Skip live network feed test under -short for deterministic make check (closes #2) #4

Merged
clawbot merged 1 commits from issue-2-short-skip-live-test into next 2026-09-21 09:05:34 +02:00
3 changed files with 13 additions and 3 deletions
+8 -1
View File
@@ -204,7 +204,9 @@ them. We provide:
- `script/projectname` — print the project name (used for the Docker - `script/projectname` — print the project name (used for the Docker
image tag) image tag)
- `script/test` — run the test suite - `script/test` — run the test suite
(`go test -timeout 30s -race -cover ./...`, verbose rerun on failure) (`go test -short -timeout 30s -race -cover ./...`, verbose rerun on
failure). The `-short` flag skips the live-network integration test so the
default run is deterministic and offline.
- `script/lint` — run `go vet ./...` and `golangci-lint run` - `script/lint` — run `go vet ./...` and `golangci-lint run`
- `script/fmt` — format all code (writes) - `script/fmt` — format all code (writes)
- `script/fmt-check` — check formatting (read-only) - `script/fmt-check` — check formatting (read-only)
@@ -218,6 +220,11 @@ them. We provide:
- `script/install-precommit` — install the git pre-commit hook that - `script/install-precommit` — install the git pre-commit hook that
runs `script/precommit` runs `script/precommit`
The live-network integration test `TestRouteWatchLiveFeed` streams the RIPE
RIS feed for a few seconds and is skipped in short mode. To run it on demand,
invoke `go test` directly without `-short`:
`go test -run TestRouteWatchLiveFeed ./internal/routewatch/`.
## License ## License
See LICENSE file. See LICENSE file.
@@ -426,6 +426,9 @@ func (m *mockStore) Ping(ctx context.Context) error {
} }
func TestRouteWatchLiveFeed(t *testing.T) { func TestRouteWatchLiveFeed(t *testing.T) {
if testing.Short() {
t.Skip("skipping live RIPE RIS network feed test in short mode; run without -short to include it")
}
// Create mock database // Create mock database
mockDB := newMockStore() mockDB := newMockStore()
+2 -2
View File
@@ -7,9 +7,9 @@ ROOT="$(cd "$(dirname "$0")/.." && pwd -P)"
main() { main() {
cd "$ROOT" cd "$ROOT"
go test -timeout 30s -race -cover ./... || { go test -short -timeout 30s -race -cover ./... || {
echo "--- Rerunning with -v for details ---" echo "--- Rerunning with -v for details ---"
go test -timeout 30s -race -v ./... go test -short -timeout 30s -race -v ./...
exit 1 exit 1
} }
} }