go-netops

core/go-netops

Fork 0

Commit graph

Author	SHA1	Message	Date
Snider	38819660ef	fix(unifi): DX audit — fix tests, add missing Commit(), improve coverage All checks were successful Security Scan / security (pull_request) Successful in 7s Details Test / test (pull_request) Successful in 1m38s Details - Fix SaveConfig: add cfg.Commit() so credentials actually persist to disk - Fix TestResolveConfig and TestNewFromConfig: isolate from real config file by setting HOME to temp dir, preventing env/config bleed - Add RouteTypeName, GetRoutes, and GetNetworks unit tests with httptest mocks (coverage 39% → 55%) - Update CLAUDE.md: add error handling and test isolation conventions - Update docs: fix Go version (1.25 → 1.26), remove stale replace directive references, add cmd/unifi/ to architecture diagram Co-Authored-By: Virgil <virgil@lethean.io>	2026-03-17 08:44:07 +00:00

Author

SHA1

Message

Date

Snider

38819660ef

fix(unifi): DX audit — fix tests, add missing Commit(), improve coverage

Security Scan / security (pull_request) Successful in 7s

Details

Test / test (pull_request) Successful in 1m38s

Details

- Fix SaveConfig: add cfg.Commit() so credentials actually persist to disk
- Fix TestResolveConfig and TestNewFromConfig: isolate from real config
  file by setting HOME to temp dir, preventing env/config bleed
- Add RouteTypeName, GetRoutes, and GetNetworks unit tests with httptest
  mocks (coverage 39% → 55%)
- Update CLAUDE.md: add error handling and test isolation conventions
- Update docs: fix Go version (1.25 → 1.26), remove stale replace
  directive references, add cmd/unifi/ to architecture diagram

Co-Authored-By: Virgil <virgil@lethean.io>

2026-03-17 08:44:07 +00:00

1 commit