Skip to content

Conversation

bkhouri
Copy link
Contributor

@bkhouri bkhouri commented Sep 2, 2025

The _InternalTestSupport and _InternalBuildTestSupport modules have XCTest assert helper functions, that, when called from a Swift Testing test, is essentially a no-op. During migration to Swift Testing, this could lead to false positives if the helpers function are not converted to use Swift Testing APIs.

During the transition, update all helper functions to detect whether the caller is executing in the XCTest or Swift Testing context, and call the appropriate API given the library.

@bkhouri
Copy link
Contributor Author

bkhouri commented Sep 2, 2025

@swift-ci test

@bkhouri bkhouri force-pushed the t/main/test_helpers_support_both_st_and_xctest branch from 2cb180b to 4c35205 Compare September 3, 2025 18:57
@bkhouri
Copy link
Contributor Author

bkhouri commented Sep 3, 2025

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Sep 3, 2025

@swift-ci test windows

@bkhouri bkhouri enabled auto-merge (squash) September 3, 2025 19:41
@bripeticca
Copy link
Contributor

@swift-ci please test self hosted windows

The `_InternalTestSupport` and `_InternalBuildTestSupport` modules have
XCTest assert helper functions, that, when called from a Swift Testing
test, is essentially a no-op.  During migration to Swift Testing, this
could lead to false positives if the helpers function are not converted
to use Swift Testing APIs.

During the transition, update all helper functions to detect whether the
caller is executing in the XCTest or Swift Testing context, and call the
appropriate API given the library.
@bkhouri bkhouri force-pushed the t/main/test_helpers_support_both_st_and_xctest branch from 4c35205 to aba200a Compare September 4, 2025 19:47
@bkhouri
Copy link
Contributor Author

bkhouri commented Sep 4, 2025

@swift-ci test

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.

3 participants