diff options
author | Jon Turney <jon.turney@dronecode.org.uk> | 2023-08-15 17:13:17 +0300 |
---|---|---|
committer | Jon Turney <jon.turney@dronecode.org.uk> | 2023-08-15 17:56:59 +0300 |
commit | f2764f165b23b2e9ae03849c84fb7ed310ed12b7 (patch) | |
tree | 7563061bde0c8db6217ae298e81aa34f82e25b23 /test/test_calm.py | |
parent | 68e2201f28238209dc58f81a82ba5e3ccdf4d0f5 (diff) |
Explicitly use count keyword argument to re.sub()
This avoids flake B034: sub should pass `count` and `flags` as keyword
arguments to avoid confusion due to unintuitive argument positions.
Diffstat (limited to 'test/test_calm.py')
-rwxr-xr-x | test/test_calm.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/test_calm.py b/test/test_calm.py index 11697dc..819ce64 100755 --- a/test/test_calm.py +++ b/test/test_calm.py @@ -366,8 +366,8 @@ class CalmTest(unittest.TestCase): with open(args.inifile) as inifile: results = inifile.read() # fix the timestamp to match expected - results = re.sub('setup-timestamp: .*', 'setup-timestamp: 1458221800', results, 1) - results = re.sub('generated at .*', 'generated at 2016-03-17 13:36:40 GMT', results, 1) + results = re.sub('setup-timestamp: .*', 'setup-timestamp: 1458221800', results, count=1) + results = re.sub('generated at .*', 'generated at 2016-03-17 13:36:40 GMT', results, count=1) compare_with_expected_file(self, 'testdata/inifile', (results,), 'setup.ini') # XXX: delete a needed package, and check validate fails @@ -453,8 +453,8 @@ class CalmTest(unittest.TestCase): with open(os.path.join(args.rel_area, 'setup.ini')) as inifile: results = inifile.read() # fix the timestamp to match expected - results = re.sub('setup-timestamp: .*', 'setup-timestamp: 1473797080', results, 1) - results = re.sub('generated at .*', 'generated at 2016-09-13 21:04:40 BST', results, 1) + results = re.sub('setup-timestamp: .*', 'setup-timestamp: 1473797080', results, count=1) + results = re.sub('generated at .*', 'generated at 2016-09-13 21:04:40 BST', results, count=1) compare_with_expected_file(self, 'testdata/process_arch', (results,), 'setup.ini') for d in ARGDIRS: |