Replace test-collector-swift with JUnit XML upload#25243
Closed
Replace test-collector-swift with JUnit XML upload#25243
Conversation
CI uses Fastlane exclusively; these local wrappers around `xcodebuild` were obsolete and hardcoded a stale simulator. --- Generated with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Lets agents run lint, build, test, and simulator queries without prompting. Other A8C repos (e.g. woocommerce.com) use the same pattern. --- Generated with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Skips CI prerequisites (env files, signing, toolkit checks) and reuses DerivedData for incremental builds. Supports only_testing, scheme, device, and clean options. --- Generate with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Ideally, this would build faster than the rest and allow focused testing. The blocker is that since the Modules don't build with `swift build` we can test with `swift test`. Plus, auto-generated schemes allow building but not testing!
--- Generate with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
--- Generate with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
--- Generate with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
--- Generate with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The file was in a `WordPressUITests` directory with no matching SPM target, so it was never compiled or run. Updated its import from `WordPress` to `WordPressUI`. --- Generate with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
--- Generate with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Applies the same test-bundle workaround already used elsewhere in WordPressUI so the storyboard loads in SPM test context. --- Generate with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Test Engine results will be uploaded via the test-collector Buildkite plugin instead of the in-process SPM library, which doesn't capture Swift Testing results. --- Generated with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Upload JUnit XML test results to Test Engine via the test-collector plugin, which runs as a post-command hook and picks up results from trainer's XML output. --- Generated with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Collaborator
Generated by 🚫 Danger |
Contributor
|
| App Name | WordPress | |
| Configuration | Release-Alpha | |
| Build Number | 30878 | |
| Version | PR #25243 | |
| Bundle ID | org.wordpress.alpha | |
| Commit | 41d4353 | |
| Installation URL | 3u824ijl7bg28 |
Contributor
|
| App Name | Jetpack | |
| Configuration | Release-Alpha | |
| Build Number | 30878 | |
| Version | PR #25243 | |
| Bundle ID | com.jetpack.alpha | |
| Commit | 41d4353 | |
| Installation URL | 217jd316lt00g |
--- Generated with the help of Claude Code, https://code.claude.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Contributor
🤖 Build Failure AnalysisThis build has failures. Claude has analyzed them - check the build annotations for details. |
Contributor
Author
CI Status NotesUI Tests (iPhone/iPad): Failing, but these are also failing on Unit Tests: Failing. I can't access Buildkite logs to check which specific tests failed. This might be flaky — unit tests pass on Danger: Failing because the PR needs a label and milestone — cosmetic. Everything else passes: builds, linters, SonarCloud, Reader tests, WordPressData tests. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Important
Closed in favor of paNNhX-14c-p2
test-collector-swiftSPM package, which hooks into XCTest at runtime and doesn't capture Swift Testing resultstest-collector-buildkite-pluginv1.11.0 to the pipeline steps for unit tests, UI tests (iPhone), and UI tests (iPad)trainerfrom.xcresultbundles) to Buildkite Test Engine in a post-command hookTest plan
🤖 Generated with Claude Code