Skip to content

Conversation

@ChrisJBurns
Copy link
Collaborator

@ChrisJBurns ChrisJBurns commented Oct 7, 2025

This was caused in this commit where the mock_collector.go was committed to git, but the actual file thats generated is mock_status.go this results in 2 mock files being present. This causes issues in the tests because the contents of the files are the same. This simply removes the collector variant in favour for the generated status one.

Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Oct 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.62%. Comparing base (5b0826f) to head (4bfe671).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2110      +/-   ##
==========================================
- Coverage   48.63%   48.62%   -0.01%     
==========================================
  Files         240      240              
  Lines       30264    30264              
==========================================
- Hits        14719    14717       -2     
- Misses      14440    14442       +2     
  Partials     1105     1105              

☔ 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.

@ChrisJBurns ChrisJBurns merged commit 44aa334 into main Oct 7, 2025
25 of 26 checks passed
@ChrisJBurns ChrisJBurns deleted the fixes-mocks branch October 7, 2025 17:20
@dmartinol
Copy link
Collaborator

Good catch @ChrisJBurns! Git rebase restored this after I fixed it 😞 , thank you!

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.

4 participants