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
Add Coverage reports #30
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
otherwise it runs Pester's test suite
ooops, merge conflict! |
aaronpowell
added a commit
that referenced
this pull request
Nov 8, 2017
* Decided to try my hand at tests with pester for #27 * First crack at travis integration * trying to not use ruby * Adding appveyor support * nice work on the yml file Aaron... * installing pester * using the vs2017 image as that seems to be the only one with Server 2016 * Trying to force the install of Pester * Trying to work out what PS version we have * More coverage in the install cmdlet * Adding some tests for Set-NodeVersion * Moving the install tests behind an environment variable This means that the local test run is a lot faster on the unit tests Updating appveyor to handle this * Trying to use the built in PS variables for OS detection These are only in PS Core, so we have to test for them first My if guards might not be right though... * Fixing my mock that would break the OS tests * Missing a mock * more appveyor debugging * removing a context * Some work to address #23 and enabling fuzzy install * Removing the direct usage of OSArchitecture and making it so it should only run under PSCore * removing debugging messages from appveyor config * Updating the editor config * Add Travis (#29) * Add Travis * Install fork * Use HTTPS * Add Homebrew cache * Clone Pester into home folder otherwise it runs Pester's test suite * Replace C:\tmp with GetTempPath() * Fix paths on *nix * Improving how the build status is show * Add Coverage reports (#30) * Add Travis * Install fork * Use HTTPS * Add Homebrew cache * Clone Pester into home folder otherwise it runs Pester's test suite * Replace C:\tmp with GetTempPath() * Fix paths on *nix * Add Codecov * Correct arguments * Revert version number change
Hmm, looks like there is a problem https://codecov.io/gh/aaronpowell/ps-nvmw/commit/e31f4ad63b9446b0aad0a01cd26418fa41e5057c |
Ahh, Pester only outputs a summary that is not understood at all by codecov... <?xml version="1.0" encoding="UTF-8" standalone="no"?><!DOCTYPE report PUBLIC "-//JACOCO//DTD Report 1.0//EN" "report.dtd">
<report name="Pester (11/08/2017 02:20:00)"><sessioninfo id="this" start="1510107595764" dump="1510107600257" /><counter type="INSTRUCTION" missed="105" covered="53" /><counter type="LINE" missed="80" covered="36" /><counter type="METHOD" missed="6" covered="4" /><counter type="CLASS" missed="0" covered="1" /></report> |
Oh rats 😢 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds uploads to https://codecov.io/gh/aaronpowell/ps-nvmw (you'd need to enable it)
This PR is based off #29, coverage is added in the last commit