Skip to content

Conversation

@ChrisJBurns
Copy link
Collaborator

@ChrisJBurns ChrisJBurns commented Nov 22, 2025

There were some overlapping scenarios across files and a bunch of unused code that clogged up the package. This POR cleans the tests up and removes unused functions

This PR is larger than 1000 diffs mainly because of the amount of unused code we are taking away.

@github-actions github-actions bot added the size/L Large PR: 600-999 lines changed label Nov 22, 2025
@codecov
Copy link

codecov bot commented Nov 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.49%. Comparing base (d8dd49a) to head (6e0eeb1).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2693   +/-   ##
=======================================
  Coverage   55.48%   55.49%           
=======================================
  Files         312      312           
  Lines       30035    30035           
=======================================
+ Hits        16666    16668    +2     
+ Misses      11901    11899    -2     
  Partials     1468     1468           

☔ 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 size/L Large PR: 600-999 lines changed size/XL Extra large PR: 1000+ lines changed and removed size/L Large PR: 600-999 lines changed labels Nov 22, 2025
@ChrisJBurns ChrisJBurns force-pushed the updates-registry-int-tests branch from d99e654 to 3a70d76 Compare November 22, 2025 16:39
@ChrisJBurns ChrisJBurns changed the title consolidates integration tests for registry controller in operator cleans up registry controller integration tests Nov 22, 2025
@github-actions github-actions bot added size/XL Extra large PR: 1000+ lines changed and removed size/XL Extra large PR: 1000+ lines changed labels Nov 22, 2025
Signed-off-by: Chris Burns <29541485+ChrisJBurns@users.noreply.github.com>
@ChrisJBurns ChrisJBurns force-pushed the updates-registry-int-tests branch from e8ffa51 to 6e0eeb1 Compare November 22, 2025 16:44
@github-actions github-actions bot added size/XL Extra large PR: 1000+ lines changed and removed size/XL Extra large PR: 1000+ lines changed labels Nov 22, 2025
@ChrisJBurns ChrisJBurns merged commit 944768a into main Nov 22, 2025
33 checks passed
@ChrisJBurns ChrisJBurns deleted the updates-registry-int-tests branch November 22, 2025 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XL Extra large PR: 1000+ lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants