public/ip-info
Archived
6
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Chưa có unit test cho package data (đặc biệt cho regression race) #21

New Issue
Closed
opened 2026-07-22 11:26:09 +07:00 by claudecode · 1 comment
claudecode commented 2026-07-22 11:26:09 +07:00
Owner
Copy Link
Copy Source

Vấn đề

Repo chưa có file test nào. Lớp internal/data (đặc biệt là Query/Reload concurrency) rất đáng có test để:

  1. Bảo vệ regression cho fix data race (issue #10).
  2. Đảm bảo query trả đúng country/ASN cho IP đã biết (vd 8.8.8.8 → US).
  3. Validate edge case (IP sai dạng, IP private, IPv6).

Fix đề xuất

  • internal/data/query_test.go: test Query("8.8.8.8") thành công, Query("not-an-ip") lỗi, IP not found.
  • internal/data/ipdb_test.go: test Reload() không leak, IsLoaded().
  • Build test có thể -race để bắt regression race.
  • Lưu ý: cloneDBFile() dùng path tương đối (data/...) → test chạy với CWD = package dir sẽ fail. Cần xử lý (chdir trong test hoặc làm path configurable) — liên quan chặt đến issue #10.

(Lưu ý phụ: data CSV/mmdb đang commit trong repo (data/), nên test có data sẵn để chạy.)

## Vấn đề Repo **chưa có file test nào**. Lớp `internal/data` (đặc biệt là `Query`/`Reload` concurrency) rất đáng có test để: 1. Bảo vệ regression cho fix data race (issue #10). 2. Đảm bảo query trả đúng country/ASN cho IP đã biết (vd 8.8.8.8 → US). 3. Validate edge case (IP sai dạng, IP private, IPv6). ## Fix đề xuất - `internal/data/query_test.go`: test `Query("8.8.8.8")` thành công, `Query("not-an-ip")` lỗi, IP not found. - `internal/data/ipdb_test.go`: test `Reload()` không leak, `IsLoaded()`. - Build test có thể `-race` để bắt regression race. - Lưu ý: `cloneDBFile()` dùng **path tương đối** (`data/...`) → test chạy với CWD = package dir sẽ fail. Cần xử lý (chdir trong test hoặc làm path configurable) — liên quan chặt đến issue #10. (Lưu ý phụ: data CSV/mmdb đang commit trong repo (`data/`), nên test có data sẵn để chạy.)
claudecode added the cleanup label 2026-07-22 11:26:09 +07:00
claudecode commented 2026-07-22 18:37:27 +07:00
Author
Owner
Copy Link
Copy Source

Đã sửa trong PR #22 (merge commit 361eb40).

  • internal/data/data_test.go (mới): 4 test case
    • TestQueryValidIP — 8.8.8.8 → country US + network record chứa IP.
    • TestQueryInvalidIP — not-an-ip → error.
    • TestQueryNotFound — 240.0.0.1 → ErrNotFound (sentinel).
    • TestQueryReloadConcurrent — 8 goroutine Query × 5000 + 4 goroutine Reload × 30 chạy song song — regression test cho data race #10/#12.

Verify: go test -race -count=1 ./... PASS (internal/data 2.4s, không race).

Đã sửa trong PR #22 (merge commit `361eb40`). - `internal/data/data_test.go` (mới): 4 test case - `TestQueryValidIP` — `8.8.8.8` → country US + network record chứa IP. - `TestQueryInvalidIP` — `not-an-ip` → error. - `TestQueryNotFound` — `240.0.0.1` → `ErrNotFound` (sentinel). - `TestQueryReloadConcurrent` — 8 goroutine Query × 5000 + 4 goroutine Reload × 30 chạy song song — regression test cho data race #10/#12. Verify: `go test -race -count=1 ./...` PASS (`internal/data` 2.4s, không race).
This repo is archived. You cannot comment on issues.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Critical correctness/data-race issue
 cleanup
Cleanup, consistency, minor improvements
No labels cleanup
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
antigravity claudecode codex devops renovatebot thuanle
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: public/ip-info#21