🌱 [Bingo Upgrade] - Upgrade EnvTest and OPM tooling

[camilamacedo86]

Description

Reviewer Checklist

• API Go Documentation
• Tests: Unit Tests (and E2E Tests, if appropriate)
• Comprehensive Commit Messages
• Links to related GitHub Issue(s)

[camilamacedo86]

@anik120

[netlify]

✅ Deploy Preview for olmv1 ready!

Name	Link
🔨 Latest commit	73575a4
🔍 Latest deploy log	https://app.netlify.com/sites/olmv1/deploys/67c0b4a6234d410008f01c6f
😎 Deploy Preview	https://deploy-preview-1820--olmv1.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.49%. Comparing base (df35dcd) to head (73575a4).
Report is 5 commits behind head on main.

Additional details and impacted files

@@ Coverage Diff @@## main #1820 +/- ##
==========================================
+ Coverage 68.43% 68.49% +0.05% 
==========================================
Files 63 63 Lines 5132 5132 ==========================================
+ Hits 3512 3515 +3 + Misses 1390 1388 -2 + Partials 230 229 -1 

Flag	Coverage Δ
e2e	51.70% <ø> (+0.15%)	⬆️
unit	56.02% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[perdasilva]

@camilamacedo86 lgtm and I'm leaving it up to you to merge. I just wonder if we want/need this before #1818 gets merged (or even if we want to make this part of that PR)

[camilamacedo86]

@camilamacedo86 lgtm and I'm leaving it up to you to merge. I just wonder if we want/need this before #1818 gets merged (or even if we want to make this part of that PR)

Just updating the binary to the latest version! One thing is the version of the binary used for execution, like mocking data for E2E tests, and another is using it as a library.

If we wanted to be super precise, we could say that dependencies should be updated together, but I think is fine to do this separately. Let me know if you see any issues! 😊

[anik120]

/lgtm

for the changes in this PR. It's a little odd that bingo env can have binaries of a different version than what go pulls in, but it is what it is now.

Ideally it'd be nice to have what shows up in bingo + go mod be the same thing (which means any PR making changes in one would have to change the other)

[camilamacedo86]

Hi @anik120@perdasilva

Ideally it'd be nice to have what shows up in bingo + go mod be the same thing (which means any PR making changes in one would have to change the other)

Yep, that would not work because the dependabot will update the dependency but not the OPM bin.
So, we never do that. but I agree that is a little odd for sure.