diff options
| author | James O'Doherty <james@theodohertyfamily.com> | 2026-06-04 22:57:35 -0400 |
|---|---|---|
| committer | James O'Doherty <james@theodohertyfamily.com> | 2026-06-04 22:57:35 -0400 |
| commit | 04dca5dada8c2d971ff3b54eeedc5ab6e53a29ac (patch) | |
| tree | a9890073a0eb21bc7db3aef2fcbe66cdc2fc9ceb /tests/e2e/vulnerability_test.go | |
| parent | 66b782e261f1cd928ad6a8482788a65fb484db45 (diff) | |
refactor: decouple namespace operations and improve test coverage
- Introduce `namespace.Ops` interface to decouple `Manager` from system-level namespace operations, enabling easier unit testing via mocks.
- Add unit tests for `internal/paths` to verify path resolution logic across different environment configurations.
- Implement `EnsureBinary` helper in E2E tests to gracefully skip tests when `WG_WRAP_BIN` is not set, allowing `go test ./...` to pass in non-build environments.
- Apply project-wide formatting and fix linting issues.
Diffstat (limited to 'tests/e2e/vulnerability_test.go')
| -rw-r--r-- | tests/e2e/vulnerability_test.go | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/tests/e2e/vulnerability_test.go b/tests/e2e/vulnerability_test.go index 8de38a3..26536fe 100644 --- a/tests/e2e/vulnerability_test.go +++ b/tests/e2e/vulnerability_test.go @@ -56,10 +56,7 @@ func TestDNSLeak(t *testing.T) { t.Skip("Skipping DNS leak test in short mode") } - binaryPath, err := GetBinaryPath() - if err != nil { - t.Skipf("Skipping test: %v", err) - } + binaryPath := EnsureBinary(t) // 1. Setup a profile with a fake, unreachable DNS server. tmpConfigDir := t.TempDir() @@ -83,7 +80,7 @@ PublicKey = 0000000000000000000000000000000000000000000000000000000000000000 AllowedIPs = 0.0.0.0/0 Endpoint = 1.1.1.1:51820 ` - err = os.WriteFile(profilePath, []byte(conf), 0644) + err := os.WriteFile(profilePath, []byte(conf), 0644) if err != nil { t.Fatal(err) } |
