Skip to content

Commit 01b96cb

Browse files
Verify no other calls in unit test (#67)
1 parent 088ae13 commit 01b96cb

File tree

1 file changed

+29
-0
lines changed

1 file changed

+29
-0
lines changed

test/ApiHealthCheck.ConsoleTest/HealthCheckManagerTest.cs

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,9 @@ public void LogHealthCheckResultSuccessTest()
8585
.Verify(v => v.SendMailToCustomer(It.IsAny<string>()),
8686
Times.Once);
8787

88+
_healthCheckMock.VerifyNoOtherCalls();
89+
_sendMailMock.VerifyNoOtherCalls();
90+
8891
Assert.Equal(
8992
"api1 status is: OK\napi2 status is: OK\n",
9093
apiStatusMessage);
@@ -150,6 +153,9 @@ public void LogHealthCheckResultFailTest()
150153
.Verify(v => v.SendMailToCustomer(It.IsAny<string>()),
151154
Times.Once);
152155

156+
_healthCheckMock.VerifyNoOtherCalls();
157+
_sendMailMock.VerifyNoOtherCalls();
158+
153159
Assert.Equal(
154160
"api1 status is: OK\napi2 status is: Error\n",
155161
apiStatusMessage);
@@ -215,6 +221,9 @@ public void LogHealthCheckResultExceptionTest()
215221
.Verify(v => v.SendMailToCustomer(It.IsAny<string>()),
216222
Times.Once);
217223

224+
_healthCheckMock.VerifyNoOtherCalls();
225+
_sendMailMock.VerifyNoOtherCalls();
226+
218227
Assert.Equal(
219228
"api1 status is: Error\napi2 status is: OK\n",
220229
apiStatusMessage);
@@ -276,6 +285,9 @@ public void LogHealthCheckResultDonotSendMailTest()
276285
.Verify(v => v.SendMailToCustomer(It.IsAny<string>()),
277286
Times.Never);
278287

288+
_healthCheckMock.VerifyNoOtherCalls();
289+
_sendMailMock.VerifyNoOtherCalls();
290+
279291
Assert.Equal(
280292
"api1 status is: Error\napi2 status is: OK\n",
281293
apiStatusMessage);
@@ -337,6 +349,9 @@ public void LogHealthCheckResultDisableUrlTest()
337349
.Verify(v => v.SendMailToCustomer(It.IsAny<string>()),
338350
Times.Never);
339351

352+
_healthCheckMock.VerifyNoOtherCalls();
353+
_sendMailMock.VerifyNoOtherCalls();
354+
340355
Assert.Equal(
341356
"api2 status is: OK\n",
342357
apiStatusMessage);
@@ -397,6 +412,17 @@ public void LogHealthCheckResultApiDetailsTest()
397412
string apiStatusMessage = _healthCheckManager!.LogHealthCheckResult();
398413

399414
Assert.Same(apiDetails, _healthCheckManager.ApiDetails);
415+
416+
_healthCheckMock
417+
.Verify(v => v.IsApiHealthy(It.IsAny<string>(), It.IsAny<ApiCredential>()),
418+
Times.Exactly(apiDetails.Count(apiDetail => apiDetail.IsEnable)));
419+
420+
_sendMailMock
421+
.Verify(v => v.SendMailToCustomer(It.IsAny<string>()),
422+
Times.Never);
423+
424+
_healthCheckMock.VerifyNoOtherCalls();
425+
_sendMailMock.VerifyNoOtherCalls();
400426
}
401427

402428
[Fact]
@@ -459,6 +485,9 @@ public void LogHealthCheckResultNoCredTest()
459485
.Verify(v => v.SendMailToCustomer(It.IsAny<string>()),
460486
Times.Once);
461487

488+
_healthCheckMock.VerifyNoOtherCalls();
489+
_sendMailMock.VerifyNoOtherCalls();
490+
462491
Assert.Equal(
463492
"api1 status is: OK\napi2 status is: OK\n",
464493
apiStatusMessage);

0 commit comments

Comments
 (0)