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

Added unit tests for ConditionalCommand #114

Merged
merged 1 commit into from
Jan 21, 2017
Merged

Added unit tests for ConditionalCommand #114

merged 1 commit into from
Jan 21, 2017

Conversation

msoucy
Copy link
Contributor

@msoucy msoucy commented Jan 21, 2017

Also incidentally tests InstantCommand.

These are some tests for #113

Also incidentally tests InstantCommand
@codecov-io
Copy link

codecov-io commented Jan 21, 2017

Current coverage is 16.00% (diff: 100%)

Merging #114 into master will increase coverage by 0.30%

@@             master       #114   diff @@
==========================================
  Files           125        125          
  Lines          7923       7923          
  Methods        1398       1398          
  Messages          0          0          
  Branches        984        984          
==========================================
+ Hits           1244       1268    +24   
- Misses         6518       6535    +17   
+ Partials        161        120    -41   

Powered by Codecov. Last update 5292374...f6382fd

@msoucy
Copy link
Contributor Author

msoucy commented Jan 21, 2017

It might be worth upgrading NuGet to fix the issue that shows up early in the appveyor logs

@ThadHouse ThadHouse merged commit 8b8a9dc into robotdotnet:master Jan 21, 2017
@msoucy msoucy deleted the commands2017-test branch January 21, 2017 16:27
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