Skip to content

feat: wire test collection IDs to all upload steps#1119

Open
dfrankland wants to merge 3 commits into
mainfrom
feat/test-collections
Open

feat: wire test collection IDs to all upload steps#1119
dfrankland wants to merge 3 commits into
mainfrom
feat/test-collections

Conversation

@dfrankland

@dfrankland dfrankland commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

Wires Trunk Flaky Tests collection IDs to all test upload steps across repos, enabling per-collection flake tracking rather than repo-wide tracking. Adds both staging and prod collection IDs where applicable. Depends on the --test-collection-id CLI flag rename landing first.

PRs

🤖 Generated with Claude Code

dfrankland and others added 3 commits June 12, 2026 15:16
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…TION_ID

Adds test-collection-id input to test_ruby_gem_uploads action and passes
TRUNK_TEST_COLLECTION_ID through all rspec and knapsack upload steps.
Staging (T5yKSn9h) wired in ruby.yml and release_ruby_gem.yml; both
staging and prod (BiBP2neA) wired in smoke_test.yml and smoke_test_main.yml.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@trunk-io

trunk-io Bot commented Jun 12, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.47%. Comparing base (93220ba) to head (bcebf6d).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1119      +/-   ##
==========================================
- Coverage   82.39%   79.47%   -2.93%     
==========================================
  Files          69       68       -1     
  Lines       15397    15293     -104     
==========================================
- Hits        12687    12154     -533     
- Misses       2710     3139     +429     

☔ View full report in Codecov by Harness.
📢 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-staging-io

Copy link
Copy Markdown

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

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.

2 participants