Skip to content

test: add unit, integration, and e2e tests #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
nightowlnerd merged 4 commits into from
Jan 31, 2026
Merged

test: add unit, integration, and e2e tests #19

nightowlnerd merged 4 commits into from
Jan 31, 2026

Conversation

@nightowlnerd
Copy link
Owner

@nightowlnerd nightowlnerd commented Jan 31, 2026

Summary

  • Add unit tests for scanner.go (isPrivateIP, BurstResult, Progress)
  • Add CLI integration tests (flags, output, error handling)
  • Add e2e tests with mock DNS server (hijacking detection, burst testing, worker pool)
  • Refactor: make DNS port configurable to enable mock server testing

Test Coverage

  • 38 tests total
  • All run locally without network dependencies
  • E2E uses mock DNS on localhost

@nightowlnerd nightowlnerd merged commit c9818bb into main Jan 31, 2026
2 checks passed
@nightowlnerd nightowlnerd deleted the test/add-scanner-tests branch January 31, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@nightowlnerd