[PROTON-609] Assert in messenger.py test causes core dump

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: proton-0.7
• Component/s: proton-c
• Labels: messenger
• Environment: Windows or Linux
• Assignee: Unassigned
• Reporter: Charles E. Rolke
• Created: 13/Jun/14 15:40
• Updated: 20/Oct/17 22:12
• Resolved: 20/Oct/17 22:12

Description

This assert:

Index: tests/python/proton_tests/messenger.py
===================================================================
--- tests/python/proton_tests/messenger.py	(revision 1602460)
+++ tests/python/proton_tests/messenger.py	(working copy)
@@ -843,6 +843,7 @@
     msg2 = Message()
     msg2.address = self.address + "/msg2"
     self.client.put(msg2)
+    assert False, "Whoops!"
     self.pump()
     assert self.server.incoming == 1, self.server.incoming
     assert self.server.receiving == 8, self.server.receiving

causes a core dump in NBMessengerTest.teardown when the code tries to stop the client. A user may work around this issue by

+      if msgr.outgoing > 0:
+        msgr.settle()
+      while msgr.incoming > 0:
+        msgr.get()
+      msgr.stop()

when all he wants is msgr.stop().