Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix nightly debugging test failures #1388

Merged
merged 5 commits into from
Feb 20, 2025

Conversation

matthewbastien
Copy link
Member

@matthewbastien matthewbastien commented Feb 13, 2025

Recent changes to launch configuration options and prompts for CodeLLDB broke the nightly tests. Fix those issues here by:

  • Updating tests to use the swift launch config type instead of swift-lldb
  • Use the new setting swift.debugger.debugAdapter to set the debug adapter that the tests use
  • Add a new setting called swift.debugger.setupCodeLLDB that controls how CodeLLDB settings are updated. Tests now set this to "alwaysUpdateGlobal" so that they don't have to deal with the prompt at all.

I also had to disable the use of liblldb in tests with Swift versions <5.10 since those versions of liblldb don't seem to be compatible with CodeLLDB. The tests fail to start due to the debug adapter crashing.

@matthewbastien matthewbastien added the full-test-run Perform a full test suite run. Apply this label before pushing up a PR or commit label Feb 13, 2025
@matthewbastien matthewbastien force-pushed the fix-debugging-tests branch 2 times, most recently from 8efc158 to 3063a45 Compare February 13, 2025 20:40
@matthewbastien matthewbastien changed the title Change "swift-lldb" debug configs to "swift" in tests Fix nightly debugging test failures Feb 13, 2025
@award999
Copy link
Contributor

@matthewbastien pre-6.0 you may have to disable these tests or use codelldb

@matthewbastien matthewbastien force-pushed the fix-debugging-tests branch 2 times, most recently from 324f079 to 82459fd Compare February 18, 2025 22:26
@matthewbastien matthewbastien merged commit acf8084 into swiftlang:main Feb 20, 2025
16 checks passed
@matthewbastien matthewbastien deleted the fix-debugging-tests branch February 20, 2025 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-test-run Perform a full test suite run. Apply this label before pushing up a PR or commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants