Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added testing infrastructure: karma, jasmine and chrome #982

Merged
merged 5 commits into from
Apr 17, 2017
Merged

Added testing infrastructure: karma, jasmine and chrome #982

merged 5 commits into from
Apr 17, 2017

Conversation

Deilan
Copy link

@Deilan Deilan commented Apr 16, 2017

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Adding testing infrastructure: karma, jasmine and chrome

  • What is the current behavior? (You can also link to an open issue here)
    No testing infrastructure Add test infrastructure: karma, jasmine and headless chrome #973

  • What is the new behavior (if this is a feature change)?
    Testing infrastructure in place: karma, jasmine and chrome.
    One sample test for App component.

  • Other information:
    Took the most part from the upstream AngularClass/angular2-webpack-starter

package.json Outdated
@@ -87,7 +88,17 @@
"gh-pages": "0.12.0",
"html-webpack-plugin": "2.28.0",
"imports-loader": "0.7.1",
"istanbul-instrumenter-loader": "^2.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@Deilan please update yarn lock file.

Copy link
Author

@Deilan Deilan Apr 17, 2017

Choose a reason for hiding this comment

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

@lexzhukov Well, I didn't have prior experience with yarn. Launched yarn at the root, yarn.lock got updated resulting in a lots of changes in the file. If I did something wrong, please let me know.

Copy link
Author

Choose a reason for hiding this comment

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

I also locked istanbul-instrumenter-loader's version to "2.0.0" to make it more consistent with the rest of the packages.

@lexzhukov lexzhukov merged commit 4420048 into akveo:master Apr 17, 2017
@lexzhukov
Copy link
Contributor

@Deilan thanks for PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants