This is an automated email from the ASF dual-hosted git repository.

jdanek pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git


The following commit(s) were added to refs/heads/main by this push:
     new f964ab7  DISPATCH-2186 Mark test helpers so that PyTest does not 
attempt test collection on them (#1276)
f964ab7 is described below

commit f964ab7d6fa4eb63b729ec326ea8d35b1803fbde
Author: Jiri Daněk <jda...@redhat.com>
AuthorDate: Mon Jun 28 15:48:19 2021 +0200

    DISPATCH-2186 Mark test helpers so that PyTest does not attempt test 
collection on them (#1276)
---
 tests/http1_tests.py              |  1 +
 tests/system_test.py              |  1 +
 tests/system_tests_core_client.py | 19 +++++++++++++++----
 3 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/tests/http1_tests.py b/tests/http1_tests.py
index e74b6f1..3d2730c 100644
--- a/tests/http1_tests.py
+++ b/tests/http1_tests.py
@@ -212,6 +212,7 @@ class TestServer(object):
     """
     A HTTPServer running in a separate thread
     """
+    __test__ = False
 
     @classmethod
     def new_server(cls, server_port, client_port, tests, handler_cls=None):
diff --git a/tests/system_test.py b/tests/system_test.py
index bc35300..7a63dc6 100755
--- a/tests/system_test.py
+++ b/tests/system_test.py
@@ -1342,6 +1342,7 @@ class TestTimeout(object):
     A callback object for MessagingHandler class
     parent: A MessagingHandler with a timeout() method
     """
+    __test__ = False
 
     def __init__(self, parent):
         self.parent = parent
diff --git a/tests/system_tests_core_client.py 
b/tests/system_tests_core_client.py
index d4cd0b8..ae1667f 100644
--- a/tests/system_tests_core_client.py
+++ b/tests/system_tests_core_client.py
@@ -90,7 +90,9 @@ class CoreClientAPITest(TestCase):
 
 
 class TestService(MessagingHandler):
-    # a service that the core client can communicate with
+    """a service that the core client can communicate with"""
+    __test__ = False
+
     class Timeout(object):
         def __init__(self, service):
             self.service = service
@@ -170,8 +172,10 @@ class TestService(MessagingHandler):
         self._container.process()
 
 
-# wait until all credit is exhausted, then re-flow more credit
 class TestCreditStarve(TestService):
+    """wait until all credit is exhausted, then re-flow more credit"""
+    __test__ = False
+
     def __init__(self, address):
         super(TestCreditStarve, self).__init__(address, credit=5)
         self.starved = False
@@ -187,8 +191,10 @@ class TestCreditStarve(TestService):
                 self._conn.close()
 
 
-# grant 10, but don't respond and close early
 class TestEarlyClose(TestService):
+    """grant 10, but don't respond and close early"""
+    __test__ = False
+
     def __init__(self, address):
         super(TestEarlyClose, self).__init__(address, credit=10)
 
@@ -199,6 +205,8 @@ class TestEarlyClose(TestService):
 
 
 class TestNoCorrelationId(TestService):
+    __test__ = False
+
     def __init__(self, address):
         super(TestNoCorrelationId, self).__init__(address, credit=1)
         self.rejected = False
@@ -211,6 +219,8 @@ class TestNoCorrelationId(TestService):
 
 
 class TestOldCorrelationId(TestService):
+    __test__ = False
+
     def __init__(self, address):
         super(TestOldCorrelationId, self).__init__(address, credit=1)
         self.accepted = False
@@ -224,7 +234,8 @@ class TestOldCorrelationId(TestService):
 
 
 class TestCallTimeout(TestService):
-    # test that the timeout is handled properly
+    """test that the timeout is handled properly"""
+    __test__ = False
 
     class PeriodicLogScrape(object):
         # periodically scan the log for the timeout error

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

Reply via email to