Skip to content

[lldb][swift] Only run Swift API tests when Swift support is enabled #1892

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

Merged

Conversation

Teemperor
Copy link

Disabling Swift support in LLDB doesn't prevent the test suite from running
the Swift tests (which then end up failing instead of being marked as
unsupported). This adds swift to the SBDebugger configuration and then checks
that Swift is enabled when running Swift API tests.

@Teemperor
Copy link
Author

@swift-ci test

Copy link

@adrian-prantl adrian-prantl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That seems like a good idea.

@Teemperor Teemperor changed the title [lldb][swift] Only run Swift API tests when Swift support is enabled [lldb][swift] Only run Swift Shell tests when Swift support is enabled Oct 2, 2020
@Teemperor Teemperor force-pushed the RunSwiftTestsOnlyWithEnabledSwift branch from 7218816 to 03232f1 Compare October 2, 2020 08:51
@Teemperor
Copy link
Author

@swift-ci test

@Teemperor Teemperor changed the title [lldb][swift] Only run Swift Shell tests when Swift support is enabled [lldb][swift] Only run Swift API tests when Swift support is enabled Oct 2, 2020
@Teemperor
Copy link
Author

swift-ci test platform macos

@Teemperor
Copy link
Author

@swift-ci test platform macos

Disabling Swift support in LLDB doesn't prevent the test suite from running
the Swift tests (which then end up failing instead of being marked as
unsupported). This adds swift to the SBDebugger configuration and then checks
that Swift is enabled when running Swift API tests.
@Teemperor Teemperor force-pushed the RunSwiftTestsOnlyWithEnabledSwift branch from 03232f1 to 2c911bc Compare October 6, 2020 07:15
@Teemperor
Copy link
Author

@swift-ci test platform macos

@Teemperor
Copy link
Author

@swift-ci please test

1 similar comment
@Teemperor
Copy link
Author

@swift-ci please test

@shahmishal shahmishal merged commit 6cdb2c3 into swiftlang:swift/main Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants