Skip to content

Commit 27f3571

Browse files
committed
Merge remote-tracking branch 'origin/pr/795'
* origin/pr/795: tests: fix failure handling in test_031_firewall_dynamic_block
2 parents b344bd6 + f42c51a commit 27f3571

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

qubes/tests/integ/network.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -807,6 +807,8 @@ def test_031_firewall_dynamic_block(self):
807807
)
808808
)
809809

810+
client1 = None
811+
client2 = None
810812
try:
811813
self.assertEqual(
812814
self.run_cmd(self.testvm1, self.ping_ip), 0, "Ping by IP failed"
@@ -871,10 +873,10 @@ def test_031_firewall_dynamic_block(self):
871873
if server2.returncode is None:
872874
server2.terminate()
873875
self.loop.run_until_complete(server2.wait())
874-
if client1.returncode is None:
876+
if client1 and client1.returncode is None:
875877
client1.terminate()
876878
self.loop.run_until_complete(client1.wait())
877-
if client2.returncode is None:
879+
if client2 and client2.returncode is None:
878880
client2.terminate()
879881
self.loop.run_until_complete(client2.wait())
880882

0 commit comments

Comments
 (0)