From 9d11fa7b2b209689acb0425cceece7d1a5bc062a Mon Sep 17 00:00:00 2001 From: Takashi Kajinami Date: Sat, 14 Feb 2026 09:56:39 +0900 Subject: [PATCH] Avoid silent skip for zookeeper config problems Change-Id: I9a50e9db9860c3736ad68136956b31d89e626552 Signed-off-by: Takashi Kajinami --- .../tests/unit/persistence/test_zk_persistence.py | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/taskflow/tests/unit/persistence/test_zk_persistence.py b/taskflow/tests/unit/persistence/test_zk_persistence.py index 07ca692a9..2be1c3343 100644 --- a/taskflow/tests/unit/persistence/test_zk_persistence.py +++ b/taskflow/tests/unit/persistence/test_zk_persistence.py @@ -58,16 +58,11 @@ class ZkPersistenceTest(test.TestCase, base.PersistenceTestMixin): # Create a unique path just for this test (so that we don't overwrite # what other tests are doing). conf['path'] = TEST_PATH_TPL % (uuidutils.generate_uuid()) - try: - self.backend = impl_zookeeper.ZkBackend(conf) - except Exception as e: - self.skipTest("Failed creating backend created from configuration" - " %s due to %s" % (conf, e)) - else: - self.addCleanup(self.backend.close) - self.addCleanup(clean_backend, self.backend, conf) - with contextlib.closing(self.backend.get_connection()) as conn: - conn.upgrade() + self.backend = impl_zookeeper.ZkBackend(conf) + self.addCleanup(self.backend.close) + self.addCleanup(clean_backend, self.backend, conf) + with contextlib.closing(self.backend.get_connection()) as conn: + conn.upgrade() def test_zk_persistence_entry_point(self): conf = {'connection': 'zookeeper:'}