From 0dfc3377f19f8aa3cb8550685d5c891950e797ef Mon Sep 17 00:00:00 2001 From: Jiri Podivin Date: Thu, 4 Aug 2022 16:32:44 +0200 Subject: Removing brackets around tested conditional 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 Change-Id: I0a086a8349e2a72cae024857e148fddc3556c319 (cherry picked from commit 56b700afcab95080182687f945c62cc96f650e45) --- cliff/tests/test_app.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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): -- cgit v1.2.1