Skip to content

Add support for newer [Skip|Only]TestingIdentifiers xctestplan format to test runner template - #3016

Merged
keith merged 1 commit into
bazelbuild:mainfrom
john-flanagan:jflan/update-test-filtering
Jul 10, 2026
Merged

Add support for newer [Skip|Only]TestingIdentifiers xctestplan format to test runner template#3016
keith merged 1 commit into
bazelbuild:mainfrom
john-flanagan:jflan/update-test-filtering

Conversation

@john-flanagan

Copy link
Copy Markdown
Contributor

Xcode 26.4+ doesn't honor the SkipTestIdentifiers array for Swift Testing tests.

This PR adds the newer SkipTestingIdentifiers format which supports both XCTest and Swift Testing tests on Xcode 26.4+ and supersedes SkipTestIdentifiers if both are present.

I assume we probably need to add the same update to the macOS runner. I'd be happy to include that in this PR once folks are happy with the format of this change.

@keith keith left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

i think it's fine, we need to rewrite this soon in a higher level language. I would wait on adding this to any other runners as @aaronsky is working on just merging them all #3008

@keith
keith enabled auto-merge (squash) July 6, 2026 17:23
auto-merge was automatically disabled July 9, 2026 18:32

Head branch was pushed to by a user without write access

@john-flanagan
john-flanagan force-pushed the jflan/update-test-filtering branch from fc9078c to f8fef31 Compare July 9, 2026 18:32
@keith
keith enabled auto-merge (squash) July 10, 2026 21:23
@keith
keith merged commit bf50fc7 into bazelbuild:main Jul 10, 2026
9 checks passed
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