-
Notifications
You must be signed in to change notification settings - Fork 145
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
Adding code coverage #11
Comments
I am also looking at this, will let you know if I get something working |
I'm going to add a sample mock to your fork. |
@morrisonbrett - I agree that example mocks are a good thing and we should get them included in this repo, but I can't see what they have to do with test-coverage or what use it would do adding them to a fork. Why don't you create a separate PR on this repo adding the mock? |
Good point. I'll do that. |
According to ionic-team/ionic-unit-testing-example#11 basic setup.
Would be great if code coverage was added to this. I'm working on getting it working with karma-coverage, but the instructions all don't really support @angular/cli and don't support webpack.
The text was updated successfully, but these errors were encountered: