alb3rtobr commented on a change in pull request #5600:
URL: https://github.com/apache/geode/pull/5600#discussion_r534079046
##########
File path:
geode-wan/src/test/java/org/apache/geode/internal/cache/wan/GatewaySenderEventRemoteDispatcherJUnitTest.java
##########
@@ -77,4 +132,160 @@ public void
getConnectionShouldCreateNewConnectionWhenServerIsNull() {
verify(dispatcher, times(1)).initializeConnection();
verify(dispatcher, times(2)).getConnectionLifeCycleLock();
}
+
+ @Test
+ public void
enforceThreadsConnectSameReceiver_initializeParallelSenderConnection_retriesAreNotUsed()
{
Review comment:
I read the article when you posted it the first time (interesting
reading, thanks!) and tried to follow the
`UnitOfWork_StateUnderTest_ExpectedBehavior` schema they propose (which btw I
found it very similar to the `given-when-then` naming approach). It seems I did
not understand the article in the same way as you, sorry. I used
`enforceThreadsConnectSameReceiver` as `UnitOfWork` because this is what this
PR is introducing.
The names you have proposed look fine for me, they really look like
requirements when you read them.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
