Repository navigation
Conversation
… drives with an IPv6-first list The test resolved the host's real localhost, which Ubuntu lists IPv4 first, so a take-the-first regression passed on Linux CI.
…ns; no unlocked writes, wall-clock windows or duplicates TestIdleOnceSkipsArchived and TestIdleOnceSkipsVolumeClaim passed with their skip removed, and the restart-budget helper passed with no attempt billed. The reload test swapped refillSem without m.mu, claims counted every goroutine in the process, and several tests slept to sync or to prove a negative. The warmup-argument test moves to config, where the check lives.
…ures, bounded receives, behavior names
…s and signals, not windows The limit test asserted upper bounds on throughput after sleeps, the splice test counted every allocation in the process, the router tests polled in real time where synctest applies, and main_test could shut down before serving.
…er node, no dial to port 1
…hroughput bench and info arms removed, fixture counts pinned The per-call timeout test passed with the per-call timeout ignored, because the client timeout raised the same error. The throughput test asserted nothing and CI skipped it.
… lookup's first-success is gated, not timed The process-wide recommit count could include a retrier an earlier test left in backoff. Dropping the lookup stopwatch also dropped what it checked: a gated peer now proves lookup returns from the owner while a slow peer still holds, with the probe timeout raised past the guard so a serial lookup fails.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A whole-file audit of every test file in the repo: about 60k lines of Go tests, 5k of Rust and 3k of Python. Each file was read in full against the test rules (layout, zero comments, modern Go) and for test value. Every finding was checked against the source before it was fixed.
Tests that could not catch their regression, now strengthened
Each was proven by temporarily reverting the production behavior: the old test passed against the revert, the new one fails.
TestIdleOnceSkipsArchived,TestIdleOnceSkipsVolumeClaimhibernateLockedrefuses before any engine call, so removing the skip changed nothing observable. Now the claim'sTransitionis held across the sweep, and the test asserts the sweep did not pick the claim.assertRestartFailsUntilBudgetrestart == exit-1also holds when no attempt is billedTestAnAdoptionConfirmsTheClaimsAddressBeforeTakingItsRowTakeTestTheTunnelTargetPrefersIPv4localhost, which Ubuntu lists IPv4 first. The selection is now a puretunnelAddrwith a byte-identical result, driven with[::1, 127.0.0.1].TestCloseBoxesWaitsForEveryReleaseOncesync.Oncetest_volume_timeout_bounds_the_callFlakes removed
synctest:synctest;refillSemswap;runtime.NumGoroutineandTotalAlloc;main_testthat could shut down before serving;-race.Cut
COCOON_TEST_ATTACHED, sdkdeletes,volume.img;infoarms;--skip throughputinsilkd.yml.Not changed, on purpose
tun_test"no tun_down" sleep andfork_testsleep: making them observable needs a production hook.storetest.WaitStacksstack-text coupling: kept.// SAFETY:comments in Rust tests: required byundocumented_unsafe_blocks.Gates (this branch, on main)
make go-lint(fresh cache): 10×0 issues.asl -forwarder=false: clean for 4 modules on both GOOS.go test -race: all 5 modules ok. Changed tests also passed at-race -count=20, and the PG-gated suites passed against a live Postgres.-D warningsclean, all test binaries ok, and the CI root-only list passed as root in a Linux container.Merge order
Written against main plus the open #530 and #533–#542, then rebased onto main. Only test files overlap with those PRs:
tunnet/router_test.go(tunnet: offer USO6 so a guest kernel enables UDP segmentation offload #539's hello assertion);silkd.ymlandtun_e2e.rs(review: 7 fixes, cuts, docs against the code #530);test_fixtures.py(docs: fixture coverage, e2e usage and the proxy port doc say what the code does #537).Merge it after them; I'll rebase it once they land.