From 94e159d6955e8c183e3d18f7537a99450000a87c Mon Sep 17 00:00:00 2001 From: Chris Lalancette Date: Fri, 26 Jan 2024 08:56:00 -0500 Subject: [PATCH] Fix flake8 warnings in launch_yaml. (#756) We don't need parentheses after assert. This fixes warnings when using the new flake8. Signed-off-by: Chris Lalancette --- .../test/launch_yaml/test_executable.py | 23 +++++++++---------- 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/launch_yaml/test/launch_yaml/test_executable.py b/launch_yaml/test/launch_yaml/test_executable.py index a4e1a9bf2..982245c74 100644 --- a/launch_yaml/test/launch_yaml/test_executable.py +++ b/launch_yaml/test/launch_yaml/test_executable.py @@ -46,23 +46,22 @@ def test_executable(): ld = parser.parse_description(root_entity) executable = ld.entities[0] cmd = [i[0].perform(None) for i in executable.cmd] - assert( - cmd == ['ls', '-l', '-a', '-s']) - assert(executable.cwd[0].perform(None) == '/') - assert(executable.name[0].perform(None) == 'my_ls') - assert(executable.shell is True) - assert(executable.emulate_tty is True) - assert(executable.output[0].perform(None) == 'log') - assert(executable.sigkill_timeout[0].perform(None) == '4.0') - assert(executable.sigterm_timeout[0].perform(None) == '7.0') + assert cmd == ['ls', '-l', '-a', '-s'] + assert executable.cwd[0].perform(None) == '/' + assert executable.name[0].perform(None) == 'my_ls' + assert executable.shell is True + assert executable.emulate_tty is True + assert executable.output[0].perform(None) == 'log' + assert executable.sigkill_timeout[0].perform(None) == '4.0' + assert executable.sigterm_timeout[0].perform(None) == '7.0' key, value = executable.additional_env[0] key = key[0].perform(None) value = value[0].perform(None) - assert(key == 'var') - assert(value == '1') + assert key == 'var' + assert value == '1' ls = LaunchService() ls.include_launch_description(ld) - assert(0 == ls.run()) + assert 0 == ls.run() def test_executable_on_exit():