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.

Sign up for GitHub

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

Jump to bottom

Consider Coverlet Coverage #8

Closed
Werebunny opened this issue Mar 31, 2018 · 3 comments · Fixed by #49
Closed

Consider Coverlet Coverage #8

Werebunny opened this issue Mar 31, 2018 · 3 comments · Fixed by #49
Assignees
@Werebunny
Labels
enhancement New feature or request
Projects
Create & Track

Comments

@Werebunny
Copy link
Member

Expected Behavior

Code coverage uses Coverlet.

Current Behavior

Code coverage uses OpenCover.

Additional Information

Coverlet is cross-platform while OpenCover is Windows only.
@Werebunny Werebunny added the enhancement New feature or request label Mar 31, 2018
@Werebunny Werebunny added this to To do in Create & Track via automation Mar 31, 2018
@Werebunny Werebunny mentioned this issue Apr 23, 2018
Closed
@Werebunny
Copy link
Member Author

Weirdly, xUnit tests have issues with OpenCover not recognizing or failing to merge with preprocessor directive cases. Hopefully switching coverage tooling will resolve this (see #6).

@Werebunny Werebunny moved this from To do to In progress in Create & Track Jun 24, 2018
@Werebunny Werebunny self-assigned this Jun 24, 2018
@Werebunny Werebunny mentioned this issue Jun 25, 2018
Merged
@Werebunny
Copy link
Member Author

The preprocessor directive cases has been solved with PR #29. Unfortunately Coverlet isn't an automatic option with the new build system implemented with PR #27, so this will likely be put on hold for now.

@Werebunny
Copy link
Member Author

Coverlet now works with Nuke, can now be used.

@Werebunny Werebunny mentioned this issue Oct 11, 2019
Merged
Create & Track automation moved this from In progress to Done Oct 11, 2019
Werebunny added a commit that referenced this issue Oct 11, 2019
@Werebunny
Fix #8: Use Coverlet (#49)
fefd2b7 
* Update dependencies.
* Switch from OpenCover to Coverlet.
* Fix found test and coverage issues.
* Fix Travis builds.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Werebunny Werebunny

Labels
enhancement New feature or request
Projects
Create & Track
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #8: Use Coverlet CreateAndFake/CreateAndFake
1 participant
@Werebunny