You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by "kirklund (GitHub)" <gi...@apache.org> on 2019/09/19 16:56:13 UTC

[GitHub] [geode] kirklund opened pull request #4071: GEODE-7165: Remove networking dependency from GatewayReceiverImplTest

The test was failing on machines with ports 2000 or 2001 in use.

This change replaces GatewayReceiverImpl's dependency on AvailablePort
with functions that are now injected by the unit tests.

Specifically, this test was failing for @dickcav and @onichols-pivotal.

[ Full content available at: https://github.com/apache/geode/pull/4071 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] kirklund commented on pull request #4071: GEODE-7165: Remove networking dependency from GatewayReceiverImplTest

Posted by "kirklund (GitHub)" <gi...@apache.org>.
Since the test doesn't need or use the input arguments, I changed it a 2nd time to remove them altogether.

[ Full content available at: https://github.com/apache/geode/pull/4071 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] kirklund closed pull request #4071: GEODE-7165: Remove networking dependency from GatewayReceiverImplTest

Posted by "kirklund (GitHub)" <gi...@apache.org>.
[ pull request closed by kirklund ]

[ Full content available at: https://github.com/apache/geode/pull/4071 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org