qpid-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From astitc...@apache.org
Subject [2/3] qpid-proton git commit: PROTON-1280: Fix Messenger Selectable test - I hate this kind of fix! - We now have a very short time out in select instead of 0 timeout (polling behaviour) so that we don't finish if it takes a little time to receive an
Date Thu, 11 Aug 2016 17:31:44 GMT
PROTON-1280: Fix Messenger Selectable test
- I hate this kind of fix!
- We now have a very short time out in select instead of 0 timeout
  (polling behaviour) so that we don't finish if it takes a little time to
  receive an end of stream on one of our sockets.
- It seems that on Linux you reliably get the end of stream on the select()
  call immediately after closing the socket at the other end locally
- But on FreeBSD you usually don't get the EOS immediately, you need to enter
  the kernel first: this was causing the test to fail on FreeBSD.


Project: http://git-wip-us.apache.org/repos/asf/qpid-proton/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-proton/commit/ec38ff0b
Tree: http://git-wip-us.apache.org/repos/asf/qpid-proton/tree/ec38ff0b
Diff: http://git-wip-us.apache.org/repos/asf/qpid-proton/diff/ec38ff0b

Branch: refs/heads/0.14.x
Commit: ec38ff0b99518a6453617d3285523ad38f0f4a30
Parents: a77eaf2
Author: Andrew Stitcher <astitcher@apache.org>
Authored: Tue Aug 9 06:37:10 2016 -0400
Committer: Andrew Stitcher <astitcher@apache.org>
Committed: Thu Aug 11 13:26:19 2016 -0400

----------------------------------------------------------------------
 tests/python/proton_tests/messenger.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/ec38ff0b/tests/python/proton_tests/messenger.py
----------------------------------------------------------------------
diff --git a/tests/python/proton_tests/messenger.py b/tests/python/proton_tests/messenger.py
index 9656033..8e2251a 100644
--- a/tests/python/proton_tests/messenger.py
+++ b/tests/python/proton_tests/messenger.py
@@ -970,7 +970,7 @@ class Pump:
         if sel.writing:
           writing.append(sel)
 
-    readable, writable, _ = select(reading, writing, [], 0)
+    readable, writable, _ = select(reading, writing, [], 0.1)
 
     count = 0
     for s in readable:
@@ -984,7 +984,6 @@ class Pump:
   def pump(self):
     while self.pump_once(): pass
 
-
 class SelectableMessengerTest(common.Test):
 
   def testSelectable(self, count = 1):


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org


Mime
View raw message