This is an archive of the discontinued LLVM Phabricator instance.

[test][lldb-vscode] Relax assertion to allow multiple compile units returned.
ClosedPublic

Authored by rupprecht on Nov 18 2022, 5:57 PM.

Details

Summary

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.

Diff Detail

Event Timeline

rupprecht created this revision.Nov 18 2022, 5:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 18 2022, 5:57 PM
rupprecht requested review of this revision.Nov 18 2022, 5:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 18 2022, 5:57 PM
labath accepted this revision.Dec 22 2022, 3:38 AM

LGTM

This revision is now accepted and ready to land.Dec 22 2022, 3:38 AM