diff options
author | Brian Behlendorf <[email protected]> | 2020-05-14 09:41:29 -0700 |
---|---|---|
committer | GitHub <[email protected]> | 2020-05-14 09:41:29 -0700 |
commit | c87f9586687d53bc7c3f2e8887841267a3960269 (patch) | |
tree | 09d77ec9503bbec4a0f2c85d1780ce38b732cbfd /tests | |
parent | e1fcd940e7a550e0631fb7e80c5df6d84c446d28 (diff) |
flake8 E741 variable name warning
Update the zts-report.py script to conform to the flake8 E741 rule.
"Variables named I, O, and l can be very hard to read. This is
because the letter I and the letter l are easily confused, and
the letter O and the number 0 can be easily confused."
- https://www.flake8rules.com/rules/E741.html
Reviewed-by: George Melikov <[email protected]>
Reviewed-by: Ryan Moeller <[email protected]>
Reviewed-by: John Kennedy <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #10323
Diffstat (limited to 'tests')
-rwxr-xr-x | tests/test-runner/bin/zts-report.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/test-runner/bin/zts-report.py b/tests/test-runner/bin/zts-report.py index d74aa9d7a..5420ff9d2 100755 --- a/tests/test-runner/bin/zts-report.py +++ b/tests/test-runner/bin/zts-report.py @@ -272,8 +272,8 @@ def process_results(pathname): pattern_log = r'^\s*Log directory:\s*(\S*)' d = {} - for l in f.readlines(): - m = re.match(pattern, l) + for line in f.readlines(): + m = re.match(pattern, line) if m and len(m.groups()) == 4: summary['total'] += 1 if m.group(4) == "PASS": @@ -281,7 +281,7 @@ def process_results(pathname): d[m.group(1)] = m.group(4) continue - m = re.match(pattern_log, l) + m = re.match(pattern_log, line) if m: summary['logfile'] = m.group(1) |