From b922d6cf8ff8febe1f093de0070b27f0435aabb6 Mon Sep 17 00:00:00 2001 From: Kamalesh Babulal Date: Thu, 3 Jul 2025 12:21:20 +0530 Subject: [PATCH] ftests/009: Fix duplicate check Remove the duplicate `if` check. No functional change. Fixes: 1545d4df54ea7 ("ftests/009: Refactor code to match outputs with same line") Acked-by: Tom Hromatka Signed-off-by: Kamalesh Babulal --- .../009-cgget-g_flag_controller_only.py | 21 +++++++++---------- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/tests/ftests/009-cgget-g_flag_controller_only.py b/tests/ftests/009-cgget-g_flag_controller_only.py index f69cb2af..32afb45f 100755 --- a/tests/ftests/009-cgget-g_flag_controller_only.py +++ b/tests/ftests/009-cgget-g_flag_controller_only.py @@ -41,17 +41,16 @@ def test(config): for expected_out in EXPECTED_OUT: if len(out.splitlines()) == len(expected_out.splitlines()): - if len(out.splitlines()) == len(expected_out.splitlines()): - result_, tmp_cause = utils.is_output_same(config, out, expected_out) - if result_ is True: - result = consts.TEST_PASSED - cause = None - break - else: - if cause is None: - cause = 'Tried Matching:\n===============' - - cause = '\n'.join(filter(None, [cause, expected_out])) + result_, tmp_cause = utils.is_output_same(config, out, expected_out) + if result_ is True: + result = consts.TEST_PASSED + cause = None + break + else: + if cause is None: + cause = 'Tried Matching:\n===============' + + cause = '\n'.join(filter(None, [cause, expected_out])) return result, cause -- 2.47.3