From: Jordan Rupprecht Date: Thu, 22 Dec 2022 13:19:29 +0000 (-0800) Subject: [test][lldb-vscode] Relax assertion to allow multiple compile units returned. X-Git-Tag: upstream/17.0.6~22858 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3afe296c81b93473c1e5008514d6225a2200ad19;p=platform%2Fupstream%2Fllvm.git [test][lldb-vscode] Relax assertion to allow multiple compile units returned. I don't think the intent of this test is to make sure we only have one compile unit; it's to make sure request_compileUnits returns something sensible. Relax the test case to just make sure that the main source file is one of the compile units returned, even if there are others. Fixes llvm.org/pr49418. Reviewed By: labath Differential Revision: https://reviews.llvm.org/D138344 --- diff --git a/lldb/test/API/tools/lldb-vscode/module/TestVSCode_module.py b/lldb/test/API/tools/lldb-vscode/module/TestVSCode_module.py index b56bf98..fe168adb 100644 --- a/lldb/test/API/tools/lldb-vscode/module/TestVSCode_module.py +++ b/lldb/test/API/tools/lldb-vscode/module/TestVSCode_module.py @@ -73,7 +73,6 @@ class TestVSCode_module(lldbvscode_testcase.VSCodeTestCaseBase): @skipIfWindows @skipIfRemote - @expectedFailureAll(oslist=["freebsd"], bugnumber="llvm.org/pr49418") def test_compile_units(self): program = self.getBuildArtifact("a.out") self.build_and_launch(program) @@ -86,7 +85,5 @@ class TestVSCode_module(lldbvscode_testcase.VSCodeTestCaseBase): moduleId = self.vscode.get_modules()['a.out']['id'] response = self.vscode.request_compileUnits(moduleId) self.assertTrue(response['body']) - self.assertEqual(len(response['body']['compileUnits']), 1, - 'Only one source file should exist') - self.assertEqual(response['body']['compileUnits'][0]['compileUnitPath'], main_source_path, - 'Real path to main.cpp matches') + cu_paths = [cu['compileUnitPath'] for cu in response['body']['compileUnits']] + self.assertIn(main_source_path, cu_paths, 'Real path to main.cpp matches')