From 99931016d84c9f2d5baf67e55506acfa39ae7ee3 Mon Sep 17 00:00:00 2001 From: Shane Harvey Date: Tue, 27 Apr 2021 14:27:01 -0700 Subject: [PATCH 1/2] PYTHON-2634 Skip arbiter tests when no server is running --- test/test_client.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/test/test_client.py b/test/test_client.py index e62516bd58..9dfb0d46a1 100644 --- a/test/test_client.py +++ b/test/test_client.py @@ -1964,6 +1964,7 @@ def timeout_task(): class TestClientPool(MockClientTest): + @client_context.require_connection def test_rs_client_does_not_maintain_pool_to_arbiters(self): listener = CMAPListener() c = MockClient( @@ -1993,6 +1994,7 @@ def test_rs_client_does_not_maintain_pool_to_arbiters(self): arbiter = c._topology.get_server_by_address(('d', 4)) self.assertFalse(arbiter.pool.sockets) + @client_context.require_connection def test_direct_client_maintains_pool_to_arbiter(self): listener = CMAPListener() c = MockClient( From 268b3bd0f6fb2c765fd1eaf34b8caae2f0efc18e Mon Sep 17 00:00:00 2001 From: Shane Harvey Date: Tue, 27 Apr 2021 14:27:55 -0700 Subject: [PATCH 2/2] Remove errant print statement --- test/test_client.py | 1 - 1 file changed, 1 deletion(-) diff --git a/test/test_client.py b/test/test_client.py index 9dfb0d46a1..3754cb0ac3 100644 --- a/test/test_client.py +++ b/test/test_client.py @@ -2009,7 +2009,6 @@ def test_direct_client_maintains_pool_to_arbiter(self): ) self.addCleanup(c.close) - print(c.topology_description) wait_until(lambda: len(c.nodes) == 1, 'connect') self.assertEqual(c.address, ('c', 3)) # Assert that we create 1 pooled connection.