From: Phil Sutter <phil@nwl.cc>
Date: Wed, 2 Oct 2024 17:55:49 +0000 (+0200)
Subject: tests: py: Fix for storing payload into missing file
X-Git-Tag: v1.1.2~124
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c1c0c54e237c880adaa8172b93d7450e6c617cfc;p=thirdparty%2Fnftables.git

tests: py: Fix for storing payload into missing file

When running a test for which no corresponding *.payload file exists,
the *.payload.got file name was incorrectly constructed due to
'payload_path' variable not being set.

Fixes: 2cfab7a3e10fc ("tests/py: Write dissenting payload into the right file")
Signed-off-by: Phil Sutter <phil@nwl.cc>
---

diff --git a/tests/py/nft-test.py b/tests/py/nft-test.py
index 00799e28..7acdb77f 100755
--- a/tests/py/nft-test.py
+++ b/tests/py/nft-test.py
@@ -769,10 +769,9 @@ def rule_add(rule, filename, lineno, force_all_family_option, filename_path):
 
     if rule[1].strip() == "ok":
         payload_expected = None
-        payload_path = None
+        payload_path = "%s.payload" % filename_path
         try:
-            payload_log = open("%s.payload" % filename_path)
-            payload_path = payload_log.name
+            payload_log = open(payload_path)
             payload_expected = payload_find_expected(payload_log, rule[0])
         except:
             payload_log = None