Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

test(testsuite): drop the devolutions-gateway dev-dependency

Aperta
  • Qualcosa non va?
#2,090 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@CBenoit ci sta già lavorando.

Dal 16/10/2026.

  • #2101 di @CBenoit — aperta

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
18/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
build-system, testing

Direzione di ricerca

Start with testsuite/Cargo.toml and the three devolutions_gateway imports in testsuite/tests/cli/agent/tunnel.rs, testsuite/tests/agent_tunnel/kdc.rs and testsuite/tests/agent_tunnel/integration.rs, then check cli/dgw/kdc_proxy.rs for overlap with the KDC test. Done means no test file imports devolutions_gateway and the dev-dependency is removed while the suite still passes. Open PR #2101 already targets this issue, so coordinate with it before starting.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

The testsuite links the devolutions-gateway library target as a dev-dependency, which pulls axum, hyper, rustls, and the rest of the Gateway into the single test binary's build and link. That is the largest avoidable compile cost in the suite, and it also lets tests reconstruct Gateway internals in-process instead of driving the binary.

Current usages at origin/master:

  • testsuite/tests/agent_tunnel/integration.rs: devolutions_gateway::{recording, session, subscriber, target_addr, token, traffic_audit, upstream, jmux}; wires SessionManagerTask, recording_message_channel, and jmux::handle by hand.
  • testsuite/tests/agent_tunnel/kdc.rs: devolutions_gateway::{kdc_connector::KdcConnector, target_addr::TargetAddr}.
  • testsuite/tests/cli/agent/tunnel.rs: devolutions_gateway::{target_addr::TargetAddr, token::...} for building token claims.

Proposed direction:

  • cli/agent/tunnel.rs: build claims as plain JSON; token validation is disabled in tests, so the Gateway types are not needed.
  • agent_tunnel/kdc.rs: cli/dgw/kdc_proxy.rs already covers the explicit-agent KDC path through the CLI (see #2068); check what remains unique here and move or drop it.
  • agent_tunnel/integration.rs: identify which behaviours are reachable by starting the Gateway binary with an AgentTunnel config and a real or fake agent, and which need a __debug__ knob; the remainder is unit-test material for #[cfg(test)] modules in the Gateway crate.

Once the usages are gone, remove devolutions-gateway from testsuite/Cargo.toml [dev-dependencies]. Policy context: testsuite/AGENTS.md (#2089).

[!NOTE]
Human-tuned, LLM-assisted content.

Lingua principale
Rust
Stelle
162
Fork
30
Merge medio
1g 9h
PR unite (30g)
60

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Devolutions/devolutions-gateway

Tutte le issue di Devolutions/devolutions-gateway

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.