diff options
author | Jiri Podivin <jpodivin@redhat.com> | 2022-08-04 16:32:44 +0200 |
---|---|---|
committer | Jon Schlueter <jschluet@redhat.com> | 2022-10-24 10:22:45 -0400 |
commit | 0dfc3377f19f8aa3cb8550685d5c891950e797ef (patch) | |
tree | bbc1fe294b84d00710502c43adcb0c7ac84c562c | |
parent | e36a71b365b6058022272b27483fcba275c18aa5 (diff) | |
download | cliff-stable/wallaby.tar.gz |
Removing brackets around tested conditionalstable/wallaby
While permissible syntactically, using brackets to wrap tested conditional is unnecessary
and potentially confusing. As without inserted whitspace the code may look as a function
call, rather than a statement and expression.
The construct is also considered erroneous by linters.
Closes-Bug: 1983593
Signed-off-by: Jiri Podivin <jpodivin@redhat.com>
Change-Id: I0a086a8349e2a72cae024857e148fddc3556c319
(cherry picked from commit 56b700afcab95080182687f945c62cc96f650e45)
-rw-r--r-- | cliff/tests/test_app.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cliff/tests/test_app.py b/cliff/tests/test_app.py index 12a42f4..1172d55 100644 --- a/cliff/tests/test_app.py +++ b/cliff/tests/test_app.py @@ -294,7 +294,7 @@ class TestOptionParser(base.TestBase): return parser def take_action(self, parsed_args): - assert(parsed_args.end == '123') + assert parsed_args.end == '123' class MyCommandManager(commandmanager.CommandManager): def load_commands(self, namespace): |