#189 - Polishing.
Made assertions in tests more strict. Original pull request: #308.
This commit is contained in:
@@ -343,6 +343,7 @@ class DefaultDatabaseClient implements DatabaseClient, ConnectionAccessor {
|
||||
|
||||
ExecuteSpecSupport(Map<Integer, SettableValue> byIndex, Map<String, SettableValue> byName,
|
||||
Supplier<String> sqlSupplier, StatementFilterFunction filterFunction) {
|
||||
|
||||
this.byIndex = byIndex;
|
||||
this.byName = byName;
|
||||
this.sqlSupplier = sqlSupplier;
|
||||
|
||||
@@ -492,7 +492,7 @@ public class DefaultDatabaseClientUnitTests {
|
||||
.as(StepVerifier::create) //
|
||||
.expectNextCount(1).verifyComplete();
|
||||
|
||||
verify(statement, never()).execute();
|
||||
verifyNoInteractions(statement);
|
||||
}
|
||||
|
||||
@Test // gh-189
|
||||
@@ -524,6 +524,7 @@ public class DefaultDatabaseClientUnitTests {
|
||||
inOrder.verify(statement).returnGeneratedValues("foo");
|
||||
inOrder.verify(statement).returnGeneratedValues("bar");
|
||||
inOrder.verify(statement).execute();
|
||||
inOrder.verifyNoMoreInteractions();
|
||||
}
|
||||
|
||||
@Test // gh-189
|
||||
@@ -551,7 +552,10 @@ public class DefaultDatabaseClientUnitTests {
|
||||
.as(StepVerifier::create) //
|
||||
.verifyComplete();
|
||||
|
||||
verify(statement).returnGeneratedValues("foo");
|
||||
InOrder inOrder = inOrder(statement);
|
||||
inOrder.verify(statement).returnGeneratedValues("foo");
|
||||
inOrder.verify(statement).execute();
|
||||
inOrder.verifyNoMoreInteractions();
|
||||
}
|
||||
|
||||
@Test // gh-189
|
||||
@@ -583,6 +587,7 @@ public class DefaultDatabaseClientUnitTests {
|
||||
inOrder.verify(statement).returnGeneratedValues("foo");
|
||||
inOrder.verify(statement).returnGeneratedValues("bar");
|
||||
inOrder.verify(statement).execute();
|
||||
inOrder.verifyNoMoreInteractions();
|
||||
}
|
||||
|
||||
static class Person {
|
||||
|
||||
Reference in New Issue
Block a user