Skip to content
This repository was archived by the owner on Jul 7, 2020. It is now read-only.

Conversation

@haverchuck
Copy link
Contributor

Issue #, if available:
Fixes #2196

Description of changes:
Adds proper unit test tooling and updates sample test.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link
Contributor

@powerful23 powerful23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM!

@haverchuck haverchuck merged commit 3b51e19 into amazon-archives:master Dec 27, 2018
"build": "node build/build.js"
},
"dependencies": {
"@vue/cli-plugin-unit-jest": "^3.2.0",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be devDependencies?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's what I thought, but for some reason the vue cli put this in dependencies (and I followed their lead). I could probably change it, I just wanted to get this fix out quickly since the issue was stale.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants