FROSADO / onecloudriver / 32250422878

81%

Build Type

push

github

Committed by web-flow

Commit Message

Replace global manager variable with dependency injection (issue #5) (#104)

* refactor: replace global manager variable with dependency injection via context

Implements issue #5: replaces the package-level `manager` variable with
context-based dependency injection through Cobra commands, following the
same pattern already used for the graph.Client (issue #10).

Changes:
- Add contextWithManager() and getManager() helpers in common.go
- Remove global manager variable from main.go
- Inject manager into context in PersistentPreRun
- Update all 11 commands to retrieve manager from context via getManager(cmd)
- Update test setup functions to inject manager through context

Benefits:
- Eliminates nil-panic risk from accessing uninitialized global manager
- Improves testability: tests can now inject mock managers without keyring
- Makes dependencies explicit and testable
- Aligns with Cobra best practices

* fix: correct getManager() call in TestResolveAccountNameDefault

The test was incorrectly trying to use getManager(cmd) in a context where
cmd didn't have a manager yet. Now properly retrieves manager from rootCmd
context which was set up by setupManager().

Coverage Stats

29 of 83 new or added lines in 15 files covered. (34.94%)

6 existing lines in 3 files now uncovered.

4486 of 5773 relevant lines covered (77.71%)

3416.5 hits per line