dnswatcher

31 Commits 14 Branches 0 Tags

3 Commits

Author	SHA1	Message	Date
clawbot	d0220e5814	fix: remove ErrNotImplemented stub — resolver, port, and TLS checks are fully implemented (closes #16 ) Check / check (pull_request) Failing after 5m27s Details The ErrNotImplemented sentinel error was dead code left over from initial scaffolding. The resolver performs real iterative DNS lookups from root servers, PortCheck does TCP connection checks, and TLSCheck verifies TLS certificates and expiry. Removed the unused error constant.	2026-02-21 00:55:38 -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

d0220e5814

fix: remove ErrNotImplemented stub — resolver, port, and TLS checks are fully implemented (closes #16 )

Check / check (pull_request) Failing after 5m27s

Details

The ErrNotImplemented sentinel error was dead code left over from
initial scaffolding. The resolver performs real iterative DNS
lookups from root servers, PortCheck does TCP connection checks,
and TLSCheck verifies TLS certificates and expiry. Removed the
unused error constant.

2026-02-21 00:55:38 -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