[test] TestTestgresLocal.test_pg_ctl_wait_option is corrected

dmitry-lipetsk · dmitry-lipetsk · commit c3a43afe4691 · 2025-05-04T10:58:07.000+03:00
Let's "release" all our test nodes correctly.
diff --git a/tests/test_testgres_common.py b/tests/test_testgres_common.py
@@ -883,10 +883,20 @@ def test_backup_wrong_xlog_method(self, node_svc: PostgresNodeService):
 
     def test_pg_ctl_wait_option(self, node_svc: PostgresNodeService):
         assert isinstance(node_svc, PostgresNodeService)
-        C_MAX_ATTEMPTS = 50
+        with __class__.helper__get_node(node_svc) as node:
+            self.impl__test_pg_ctl_wait_option(node_svc, node)
 
-        node = __class__.helper__get_node(node_svc)
+    def impl__test_pg_ctl_wait_option(
+        self,
+        node_svc: PostgresNodeService,
+        node: PostgresNode
+    ) -> None:
+        assert isinstance(node_svc, PostgresNodeService)
+        assert isinstance(node, PostgresNode)
         assert node.status() == NodeStatus.Uninitialized
+
+        C_MAX_ATTEMPTS = 50
+
         node.init()
         assert node.status() == NodeStatus.Stopped
         node.start(wait=False)
@@ -950,7 +960,6 @@ def test_pg_ctl_wait_option(self, node_svc: PostgresNodeService):
             raise Exception("Unexpected node status: {0}.".format(s1))
 
         logging.info("OK. Node is stopped.")
-        node.cleanup()
 
     def test_replicate(self, node_svc: PostgresNodeService):
         assert isinstance(node_svc, PostgresNodeService)
