Skip to content

Take over e2e tests with minimal harness - #7

Merged
andrewchen5678 merged 2 commits into
mainfrom
e2e-harness
Sep 5, 2026
Merged

andrewchen5678 merged 2 commits into
mainfrom
e2e-harness

Conversation

@andrewchen5678

Copy link
Copy Markdown
Contributor

Transfer the end-to-end tests from tunnel-rs to this repository, implementing a minimal harness for testing the iroh layer. Enhance the negative checks and ensure cleanup processes are bounded and single-shot. Document the new testing framework and roadmap for future integration of flexaccess-keys.

Andrew Chen and others added 2 commits September 4, 2026 21:32
The relay failover suite and the auth/connectivity checks that tunnel-rs ran
end to end exercise only this crate, so they now live here, against
examples/e2e: a server that binds through the shared builder, runs the
home-relay failover beside its accept loop, and answers the endpoint-bound
auth transcript plus an echo; a client that dials through the configured
relays and proves its key. The Python fixtures (fake relay, delay proxy) and
key generation are harness subcommands, so the suites need only cargo and
iroh-relay. CI runs the offline suites.

ROADMAP.md records the idea of folding flexaccess-keys into this repository
as a subcrate, without doing it.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0163AtLdT9AWeDE3EYmhFYXK
A server that died without logging the expected failure counted as the
expected failure; now only the logged failure passes. The cleanup trap
clears itself on entry so an INT/TERM exit cannot run it twice, and it
SIGKILLs children that ignore SIGTERM after a short grace period instead
of waiting on them forever.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0163AtLdT9AWeDE3EYmhFYXK
@andrewchen5678
andrewchen5678 merged commit bb9c74c into main Sep 5, 2026
4 checks passed
@andrewchen5678
andrewchen5678 deleted the e2e-harness branch September 5, 2026 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant