test: increase test coverage for pkg/version and slackdeps

[mwbrooks]

Changelog

• N/A

Summary

This pull request increases our test coverage in internal/pkg/version and internal/slackdeps.

I skipped updating a few other internal/pkg since there are some active refactoring going on in those packages and I want to avoid overcomplicating that work.

Tests are ordered alphabetically, grouping getters/setters, and constructors are at the top.

Requirements

• I've read and understood the Contributing Guidelines and have done my best effort to follow them.
• I've read and agree to the Code of Conduct.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.28%. Comparing base (34ac465) to head (db2adf6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #409      +/-   ##
==========================================
+ Coverage   68.16%   68.28%   +0.11%     
==========================================
  Files         218      218              
  Lines       18058    18058              
==========================================
+ Hits        12310    12331      +21     
+ Misses       4594     4569      -25     
- Partials     1154     1158       +4     

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

[mwbrooks]

A little guidance for the readers

[mwbrooks]

note: t.Setenv(...) is a built-in that backs up the original environment variable value, sets the new value, and then restores the original when the test completes. I'd rather use a 100% mocked call, but this is a good and safe start.

[zimeg]

@mwbrooks LGTM! 🎁 🧪 ✨

And thanks for keeping adjacent changes minded in updates here! I hope it doesn't stall testing efforts much but I do think it'll help us in those updates!

[mwbrooks]

And thanks for keeping adjacent changes minded in updates here! I hope it doesn't stall testing efforts much but I do think it'll help us in those updates!

No stales at all! I'll keep little test coverage PRs coming and once the sensitive areas are ready, we can see if there's testing opportunities hidden! 🕵🏻

Thanks for the quick review! 🙇🏻