rm-hull / dot-block / 27230176089

60%

main: 61%

Build Type

Pull #150

github

Committed by gemini-pr-agent[bot]

Commit Message

chore: iterative updates from Gemini based on PR comment

I have added integration tests for both the `GET` and `POST` `/dns-query` endpoints to `internal/integration_test.go`.

### Changes Made:
- **Imports:** Added `bytes`, `encoding/base64`, `io`, and `net/http` to support HTTP requests and DNS message encoding.
- **Configuration:** Updated the test `App` configuration to use a fixed `HttpPort` (8080) to ensure the DoH tests have a reliable endpoint.
- **Test Cases:** Added four new test cases to the `TestIntegration_DNSFunctionality` suite:
    - `DoH GET - Good Domain`
    - `DoH GET - Blocked Domain`
    - `DoH POST - Good Domain`
    - `DoH POST - Blocked Domain`
- **Test Logic:** Implemented the logic to:
    - Pack DNS messages.
    - Perform `GET` requests with base64-encoded DNS queries.
    - Perform `POST` requests with `application/dns-message` bodies.
    - Unpack the response and validate that domains are correctly allowed or blocked (verified by the presence of the `ns.blocked.local.` SOA record).

### Verification:
I ran the integration tests using `go test -v ./internal`, and all tests passed, including the new DoH scenarios.

Pull Request Pull Request #150: feat: implement DNS-over-HTTPS (DoH) support

Coverage Stats

45 of 93 new or added lines in 2 files covered. (48.39%)

9 existing lines in 1 file now uncovered.

1242 of 2068 relevant lines covered (60.06%)

841.36 hits per line