Skip to content

[CI] Preserve xcresult bundle structure in CI artifacts#830

Merged
Kyle-Ye merged 2 commits intomainfrom
claude/dazzling-jemison
Mar 18, 2026
Merged

[CI] Preserve xcresult bundle structure in CI artifacts#830
Kyle-Ye merged 2 commits intomainfrom
claude/dazzling-jemison

Conversation

@Kyle-Ye
Copy link
Collaborator

@Kyle-Ye Kyle-Ye commented Mar 18, 2026

Summary

  • Zip the .xcresult bundle before uploading to preserve its macOS package structure
  • Bump upload-artifact from v4 to v7 with archive: false to avoid double-zipping

Fixes the issue where downloading the macos-uitest-snapshots-xcresult.zip artifact would produce a flat macos-uitest-snapshots-xcresult/ folder instead of a proper .xcresult bundle.

Reference: actions/upload-artifact#39

Test plan

  • Trigger a UI test failure and verify the downloaded xcresult artifact extracts as a valid .xcresult bundle openable in Xcode

Zip the .xcresult bundle before uploading to preserve its macOS
package structure. Bump upload-artifact to v7 with archive: false
to avoid double-zipping.
@github-actions github-actions bot added bug Something isn't working build issue platform: macOS labels Mar 18, 2026
@augmentcode
Copy link

augmentcode bot commented Mar 18, 2026

🤖 Augment PR Summary

Summary: Ensures UI-test .xcresult artifacts keep their bundle/package structure when downloaded from CI.

Changes: Zips the generated .xcresult on failure and uploads the resulting zip via actions/upload-artifact@v7 with archive: false to avoid flattening/double-zipping.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@codecov
Copy link

codecov bot commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 27.17%. Comparing base (ff8c99a) to head (7d8f107).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #830       +/-   ##
===========================================
+ Coverage   13.83%   27.17%   +13.33%     
===========================================
  Files         630      667       +37     
  Lines       38173    42953     +4780     
===========================================
+ Hits         5283    11671     +6388     
+ Misses      32890    31282     -1608     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions bot added the test Test related issue label Mar 18, 2026
@Kyle-Ye Kyle-Ye changed the title [Bug Fix] Preserve xcresult bundle structure in CI artifacts [CI] Preserve xcresult bundle structure in CI artifacts Mar 18, 2026
@Kyle-Ye Kyle-Ye merged commit 6cecbbb into main Mar 18, 2026
8 of 9 checks passed
@Kyle-Ye Kyle-Ye deleted the claude/dazzling-jemison branch March 18, 2026 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working build issue platform: macOS test Test related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant