Skip to content

(Chore): Attach test-collection-short-id to staging uploads#1114

Merged
trunk-io[bot] merged 1 commit into
mainfrom
dylan/staging-test-collection-short-id
Jun 3, 2026
Merged

(Chore): Attach test-collection-short-id to staging uploads#1114
trunk-io[bot] merged 1 commit into
mainfrom
dylan/staging-test-collection-short-id

Conversation

@dfrankland
Copy link
Copy Markdown
Member

@dfrankland dfrankland commented Jun 3, 2026

Summary

  • Add --test-collection-short-id 2tYJWMu7 to the PR-CI staging upload in .github/workflows/pull_request.yml so analytics-cli's own JUnit results land in the right staging collection.
  • Add --test-collection-short-id z2dzjLHm to the three smoke-test staging uploads in .github/actions/perform_smoke_test/action.yaml (Swift xcresult, baseline junit, and variant) so smoke-test runs land in their own collection.
  • Skip the flag on uploads that pass --repo-url (the flake-farm-staging-fork-prs public-repo-id upload and the GitLab smoke test), since those represent data from other repos.

Test plan

  • Verify CI PR run uploads to staging show up under the 2tYJWMu7 test collection.
  • Verify main-branch smoke-test runs show up under the z2dzjLHm test collection.
  • Confirm public-repo-id and GitLab smoke-test uploads remain unchanged in staging.

🤖 Generated with Claude Code

Tag analytics-cli staging uploads from this repo with a test collection
short ID so the staging org can route them into the right collection.
PR-CI uploads use 2tYJWMu7; the smoke-test action uses z2dzjLHm.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Jun 3, 2026

😎 Merged directly without going through the merge queue, as the queue was empty and the PR was up to date with the target branch - details.

@trunk-staging-io
Copy link
Copy Markdown

trunk-staging-io Bot commented Jun 3, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

Failed Test Failure Summary Logs
variant_quarantine_test should be quarantined when run with variant A test expected the sum of 2 + 2 to be 5, but it was actually 4, indicating a failing assertion. Logs ↗︎

View Full Report ↗︎Docs

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.39%. Comparing base (f584ee0) to head (da4c14c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1114      +/-   ##
==========================================
+ Coverage   82.12%   82.39%   +0.26%     
==========================================
  Files          69       69              
  Lines       15243    15243              
==========================================
+ Hits        12519    12559      +40     
+ Misses       2724     2684      -40     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Jun 3, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@trunk-io trunk-io Bot merged commit 4f36b83 into main Jun 3, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants