dnswatcher

33 Commits 14 Branches 0 Tags

3 Commits

Author	SHA1	Message	Date
clawbot	b162ca743b	fix: use full Lock in State.Save() to prevent data race (closes #17 ) Check / check (pull_request) Failing after 5m31s Details State.Save() was using RLock but mutating s.snapshot.LastUpdated, which is a write operation. This created a data race since other goroutines could also hold a read lock and observe a partially written timestamp. Changed to full Lock to ensure exclusive access during the mutation.	2026-02-21 00:51:58 -08:00
clawbot	687027be53	test: add tests for no-peer-certificates error path Check / check (pull_request) Successful in 10m50s Details	2026-02-20 07:44:01 -08:00
clawbot	0486dcfd07	fix: mock DNS in resolver tests for hermetic, fast unit tests - Extract DNSClient interface from resolver to allow dependency injection - Convert all resolver methods from package-level to receiver methods using the injectable DNS client - Rewrite resolver_test.go with a mock DNS client that simulates the full delegation chain (root → TLD → authoritative) in-process - Move 2 integration tests (real DNS) behind //go:build integration tag - Add NewFromLoggerWithClient constructor for test injection - Add LookupAllRecords implementation (was returning ErrNotImplemented) All unit tests are hermetic (no network) and complete in <1s. Total make check passes in ~5s. Closes #12	2026-02-20 05:58:51 -08:00

Author

SHA1

Message

Date

clawbot

b162ca743b

fix: use full Lock in State.Save() to prevent data race (closes #17 )

Check / check (pull_request) Failing after 5m31s

Details

State.Save() was using RLock but mutating s.snapshot.LastUpdated,
which is a write operation. This created a data race since other
goroutines could also hold a read lock and observe a partially
written timestamp. Changed to full Lock to ensure exclusive access
during the mutation.

2026-02-21 00:51:58 -08:00

clawbot

687027be53

test: add tests for no-peer-certificates error path

Check / check (pull_request) Successful in 10m50s

Details

2026-02-20 07:44:01 -08:00

clawbot

0486dcfd07

fix: mock DNS in resolver tests for hermetic, fast unit tests

- Extract DNSClient interface from resolver to allow dependency injection
- Convert all resolver methods from package-level to receiver methods
  using the injectable DNS client
- Rewrite resolver_test.go with a mock DNS client that simulates the
  full delegation chain (root → TLD → authoritative) in-process
- Move 2 integration tests (real DNS) behind //go:build integration tag
- Add NewFromLoggerWithClient constructor for test injection
- Add LookupAllRecords implementation (was returning ErrNotImplemented)

All unit tests are hermetic (no network) and complete in <1s.
Total make check passes in ~5s.

Closes #12

2026-02-20 05:58:51 -08:00