From e88166f7a44a9e3403c1aa4eeca2ecb896e42f2a Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Fri, 16 Aug 2024 10:06:26 +0800 Subject: [PATCH] test.py: use XPath for iterating in "TestSuite/TestSuite" before this change, we check for the existence of "TestSuite" node under the root of XML tree, and then enumerating all "TestSuite" nodes under this "TestSuite", this approach works. but it * introduces unnecessary indent * is not very readable in this change, we just use "./TestSuite/TestSuite" for enumerating all "TestSuite" nodes under "TestSuite". simpler this way. Signed-off-by: Kefu Chai --- test.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/test.py b/test.py index 021afe4af5..99284b42c4 100755 --- a/test.py +++ b/test.py @@ -1721,11 +1721,9 @@ def boost_to_junit(boost_xml, junit_xml): # report produced {write_consolidated_boost_junit_xml} have the next structure suite_boost -> [suite1, suite2, ...] # so we are excluding the upper suite with name boost - boost_suite = boost_root.find('TestSuite') - if boost_suite is not None: - test_suites = boost_suite.findall('TestSuite') - - for test_suite in test_suites: + for test_suite in boost_root.findall('./TestSuite/TestSuite'): + # TODO: fix the indent + if True: suite_time = 0.0 suite_test_total = 0 suite_test_fails_number = 0