Skip to content

Conversation

texasaggie97-zz
Copy link
Contributor

[X] This contribution adheres to [CONTRIBUTING.md]

What does this Pull Request accomplish?

  • Fixes No longer unit testing ModInst #221
  • Run ModInst unit tests
    • ModInst uses own session class, so some of NI-FAKE unit testing doesn't apply
  • Add to code coverage report
    • Percentage is going down because of this

Why should this Pull Request be merged?

  • Need to unit test ModInst separately

What testing has been done?

  • Travis

@coveralls
Copy link

coveralls commented Sep 1, 2017

Coverage Status

Coverage decreased (-10.2%) to 51.408% when pulling 54e2424 on bug/bug221 into 258beb0 on master.

@marcoskirsch marcoskirsch merged commit 5ce8756 into master Sep 1, 2017
@marcoskirsch marcoskirsch deleted the bug/bug221 branch September 1, 2017 20:50
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.

4 participants