From 5034b34c88cc70c95d41d183549c709ab63cd7c9 Mon Sep 17 00:00:00 2001 From: Steven Silvester Date: Wed, 16 Oct 2024 12:53:37 -0500 Subject: [PATCH 1/2] PYTHON-4865 Skip test_write_concern_failure tests temporarily --- test/asynchronous/test_bulk.py | 2 ++ test/test_bulk.py | 2 ++ 2 files changed, 4 insertions(+) diff --git a/test/asynchronous/test_bulk.py b/test/asynchronous/test_bulk.py index 42a3311072..23c4170d97 100644 --- a/test/asynchronous/test_bulk.py +++ b/test/asynchronous/test_bulk.py @@ -971,6 +971,7 @@ async def cause_wtimeout(self, requests, ordered): @async_client_context.require_replica_set @async_client_context.require_secondaries_count(1) async def test_write_concern_failure_ordered(self): + self.skipTest("Skipping until PYTHON-4865 is resolved.") # Ensure we don't raise on wnote. coll_ww = self.coll.with_options(write_concern=WriteConcern(w=self.w)) result = await coll_ww.bulk_write([DeleteOne({"something": "that does no exist"})]) @@ -1051,6 +1052,7 @@ async def test_write_concern_failure_ordered(self): @async_client_context.require_replica_set @async_client_context.require_secondaries_count(1) async def test_write_concern_failure_unordered(self): + self.skipTest("Skipping until PYTHON-4865 is resolved.") # Ensure we don't raise on wnote. coll_ww = self.coll.with_options(write_concern=WriteConcern(w=self.w)) result = await coll_ww.bulk_write( diff --git a/test/test_bulk.py b/test/test_bulk.py index 64fd48e8cd..b873f10f90 100644 --- a/test/test_bulk.py +++ b/test/test_bulk.py @@ -969,6 +969,7 @@ def cause_wtimeout(self, requests, ordered): @client_context.require_replica_set @client_context.require_secondaries_count(1) def test_write_concern_failure_ordered(self): + self.skipTest("Skipping until PYTHON-4865 is resolved.") # Ensure we don't raise on wnote. coll_ww = self.coll.with_options(write_concern=WriteConcern(w=self.w)) result = coll_ww.bulk_write([DeleteOne({"something": "that does no exist"})]) @@ -1049,6 +1050,7 @@ def test_write_concern_failure_ordered(self): @client_context.require_replica_set @client_context.require_secondaries_count(1) def test_write_concern_failure_unordered(self): + self.skipTest("Skipping until PYTHON-4865 is resolved.") # Ensure we don't raise on wnote. coll_ww = self.coll.with_options(write_concern=WriteConcern(w=self.w)) result = coll_ww.bulk_write([DeleteOne({"something": "that does no exist"})], ordered=False) From b27040e6c1f200e14f08c82014eeebdcf556b827 Mon Sep 17 00:00:00 2001 From: Steven Silvester Date: Wed, 16 Oct 2024 12:59:41 -0500 Subject: [PATCH 2/2] lint --- test/asynchronous/test_bulk.py | 4 ++++ test/test_bulk.py | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/test/asynchronous/test_bulk.py b/test/asynchronous/test_bulk.py index 23c4170d97..c9ff167b43 100644 --- a/test/asynchronous/test_bulk.py +++ b/test/asynchronous/test_bulk.py @@ -972,6 +972,8 @@ async def cause_wtimeout(self, requests, ordered): @async_client_context.require_secondaries_count(1) async def test_write_concern_failure_ordered(self): self.skipTest("Skipping until PYTHON-4865 is resolved.") + details = None + # Ensure we don't raise on wnote. coll_ww = self.coll.with_options(write_concern=WriteConcern(w=self.w)) result = await coll_ww.bulk_write([DeleteOne({"something": "that does no exist"})]) @@ -1053,6 +1055,8 @@ async def test_write_concern_failure_ordered(self): @async_client_context.require_secondaries_count(1) async def test_write_concern_failure_unordered(self): self.skipTest("Skipping until PYTHON-4865 is resolved.") + details = None + # Ensure we don't raise on wnote. coll_ww = self.coll.with_options(write_concern=WriteConcern(w=self.w)) result = await coll_ww.bulk_write( diff --git a/test/test_bulk.py b/test/test_bulk.py index b873f10f90..ea2b803804 100644 --- a/test/test_bulk.py +++ b/test/test_bulk.py @@ -970,6 +970,8 @@ def cause_wtimeout(self, requests, ordered): @client_context.require_secondaries_count(1) def test_write_concern_failure_ordered(self): self.skipTest("Skipping until PYTHON-4865 is resolved.") + details = None + # Ensure we don't raise on wnote. coll_ww = self.coll.with_options(write_concern=WriteConcern(w=self.w)) result = coll_ww.bulk_write([DeleteOne({"something": "that does no exist"})]) @@ -1051,6 +1053,8 @@ def test_write_concern_failure_ordered(self): @client_context.require_secondaries_count(1) def test_write_concern_failure_unordered(self): self.skipTest("Skipping until PYTHON-4865 is resolved.") + details = None + # Ensure we don't raise on wnote. coll_ww = self.coll.with_options(write_concern=WriteConcern(w=self.w)) result = coll_ww.bulk_write([DeleteOne({"something": "that does no exist"})], ordered=False)