fix: TestTestgresCommon::test_node_app__make_empty_with_explicit_port is corrected

dmitry-lipetsk · dmitry-lipetsk · commit 0c3d9831e8e1 · 2026-01-18T20:24:07.000+03:00
Do not release port within cycle!

All the reserved ports are released in finally section.
diff --git a/tests/test_testgres_common.py b/tests/test_testgres_common.py
@@ -2552,7 +2552,6 @@ def test_node_app__make_empty_with_explicit_port(self, node_svc: PostgresNodeSer
                         e
                     ))
                     assert node.status() == NodeStatus.Stopped
-                    node_app.port_manager.release_port(port)
                     continue
 
                 assert node.status() == NodeStatus.Running

Original file line number	Diff line number	Diff line change
`@@ -2552,7 +2552,6 @@ def test_node_app__make_empty_with_explicit_port(self, node_svc: PostgresNodeSer`
`2552`	`2552`	`e`
`2553`	`2553`	`))`
`2554`	`2554`	`assert node.status() == NodeStatus.Stopped`
`2555`		`- node_app.port_manager.release_port(port)`
`2556`	`2555`	`continue`
`2557`	`2556`
`2558`	`2557`	`assert node.status() == NodeStatus.Running`