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

chore(test): add tests for components + improve config (migrate to webpack2) #35

Merged
merged 2 commits into from
Jan 21, 2017

Conversation

tinesoft
Copy link
Contributor

@tinesoft tinesoft commented Jan 21, 2017

Update config files (migrate to webpack 2 + improve karma config):

  • add option to deactivate code coverage (useful for debugging tests)
  • add new gulp task test:watch-no-cc that runs tests in watch without coverage
  • pin codelyzer version to 1.0.0-beta.0 (for compatibility with angular 2.0.0)
  • add a test helper module containing mock classes and helper functions

Add tests for components:

-add a test helper module containing mock classes and helper functions

Some branches still need to be tested(see coverage report), but basically, most have been.

@tinesoft tinesoft force-pushed the chore_test_components branch from 44eb4bd to aa4c61a Compare January 21, 2017 15:53
…rma config)

- add option to deactivate code coverage (useful for debugging tests)
- add new gulp task `test:watch-no-cc` that runs tests in watch without coverage
- ping `codelyzer` version  to 1.0.0-beta.0 (for compatibility with angular 2.0.0)
-add a test helper module containing mock classes and helper functions
@tinesoft tinesoft force-pushed the chore_test_components branch from aa4c61a to e1294dd Compare January 21, 2017 16:11
@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 96.319% when pulling e1294dd on tinesoft:chore_test_components into ef72569 on MurhafSousli:master.

@MurhafSousli MurhafSousli merged commit f1f5b33 into MurhafSousli:master Jan 21, 2017
@tinesoft tinesoft deleted the chore_test_components branch January 21, 2017 16:41
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