-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Labels
Description
Our provider tests are supposed to call mockProvider.VerifyAll after every test. But the code to do so is decorated with a SetUp attribute instead of a TearDown attribute, which never fires. Here is an example:
Gravity/Gravity/Gravity.Test.Unit/RsapiDaoInsertTests.cs
Lines 41 to 46 in a4874bc
| [SetUp] | |
| public void End() | |
| { | |
| //ensure any defined methods called | |
| mockProvider.VerifyAll(); | |
| } |
Flipping this value shows that 10 tests are not written to meet the verification criterion:
In the process of fixing this, we should create a base unit test class to handle the mockProvider setup and verification.
