From: Phil Sutter Date: Wed, 2 Oct 2024 17:55:49 +0000 (+0200) Subject: tests: py: Fix for storing payload into missing file X-Git-Url: http://git.ipfire.org/gitweb/gitweb.cgi?a=commitdiff_plain;h=c8c0b779cc02f96a9d3e534993fae10cdda93630;p=thirdparty%2Fnftables.git tests: py: Fix for storing payload into missing file commit c1c0c54e237c880adaa8172b93d7450e6c617cfc upstream. 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 --- diff --git a/tests/py/nft-test.py b/tests/py/nft-test.py index 1bc89558..50584fcf 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