[ 
https://issues.apache.org/jira/browse/GEODE-8157?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17219750#comment-17219750
 ] 

ASF GitHub Bot commented on GEODE-8157:
---------------------------------------

pdxcodemonkey commented on a change in pull request #674:
URL: https://github.com/apache/geode-native/pull/674#discussion_r510963460



##########
File path: clicache/acceptance-test/SNITests.cs
##########
@@ -127,5 +133,38 @@ public void ConnectionWithoutProxyFails()
 
             Assert.Throws<NotConnectedException>(() => region.Put("1", "one"));
         }
+

Review comment:
       This is probably a correct test until such time as we resolve 
GEODE-8628.  If it turns out NC should be honoring the read timeout value and 
throwing a timeout exception (what I would've expected), then when we fix it to 
do so properly, we can update this test to expect throw rather than just 
checking if the put is complete.  @echobravopapa I'm in favor of resolving this 
bug and merging the new test, then writing one to actually shut down and 
restart the proxy and catch the proper exception etc.  To my mind, these are 
different tests, both valid.  Let us know what you think.




----------------------------------------------------------------
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:
us...@infra.apache.org


> Write "drop proxy test" for C# (clicache)
> -----------------------------------------
>
>                 Key: GEODE-8157
>                 URL: https://issues.apache.org/jira/browse/GEODE-8157
>             Project: Geode
>          Issue Type: New Feature
>          Components: native client
>            Reporter: Blake Bender
>            Priority: Major
>              Labels: pull-request-available
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to