Add unit tests for internal/service module

[salignatmoandal]

What

This PR adds a comprehensive test suite for the internal/services module.

Why

• Improves code reliability and maintainability
• Makes future refactoring safer
• Provides documentation for expected behavior

Details

• Adds unit tests for:
  • Run()
  • CheckVersions()
  • listRemoteImages()
  • suggestUpdateCmd()
• Uses afero.NewMemMapFs() for in-memory filesystem testing
• Uses gock for HTTP request mocking
• Uses testify for assertions

Result

• Test coverage for the module is now above 90%
• All tests pass locally

---

Let me know if you need any changes or improvements!

[sweatybridge]

To make these tests pass, you need to mock all api requests using gock.

[salignatmoandal]

Got it — I’ll use gock to mock the API calls and make the tests pass. Thanks!

[salignatmoandal]

• Use a fake but valid access token for tests, with #nosec G101 to avoid gosec false positives
• Fix race condition in listRemoteImages by protecting map writes with sync.Mutex
• All tests now pass with -race and are CI-friendly

[coveralls]

Pull Request Test Coverage Report for Build 16030849241

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 8 of 8 (100.0%) changed or added relevant lines in 1 file are covered.
• 7 unchanged lines in 2 files lost coverage.
• Overall coverage increased (+0.6%) to 56.237%

---

Files with Coverage Reduction	New Missed Lines	%
internal/storage/rm/rm.go	2	89.53%
internal/gen/keys/keys.go	5	12.9%

Totals
Change from base Build 16026012543:	0.6%
Covered Lines:	6158
Relevant Lines:	10950

---

💛 - Coveralls