-
Notifications
You must be signed in to change notification settings - Fork 12
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
Provide code coverage information for unit test (or e2e test) execution #27
Comments
I think it's a big work. Perhaps we could start to provide a JUnit View like? I don't know if it's possible to do that. I must study result of test (never played with ng test). |
@PMJMendes I tell me if it's a really interesting feature since https://github.com/angul ar/angular-cli/wiki/test provides the capability to generate HTML pages with coverage. When you execute:
You can generate report like |
Hello,
From: Angelo ***@***.***
@PMJMendes<https://github.com/PMJMendes> I tell me if it's a really interesting feature since
https://github.com/angular/angular-cli/wiki/test provides the capability to generate HTML pages with coverage ?
Hmmm, good call. I haven’t checked out the coverage HTML pages yet. If they work well, I guess it’s not as important a feature as it would be without them. Having the code colored directly in the editor is helpful, so I guess it’s probably still interesting, but just as a nice-to-have…
Cheers,
J.
|
@angelozerr @PMJMendes I just created an Issue in typescript.java (See angelozerr/typescript.java#175), since this should be solved for typescript in general, not only for angular (in my opinion). |
Hey, :)
From: Springrbua ***@***.***
this should be solved for typescript in general, not only for angular (in my opinion).
I don't think this has a high priority, but it would be a nice feature anyways.
Agree on all counts.
Cheers,
J.
|
Provide source highliting (like EclEmma) after test execution according to which source lines and branches were executed during the test batch.
The text was updated successfully, but these errors were encountered: