test: move crate-local integration tests into testsuite

Ouverte
  • Quelque chose ne va pas ?
#2,109 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
55/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
Active
Stack technique
rust
Domaine
testing

Piste de recherche

Read testsuite/AGENTS.md and inspect the listed crate-local tests/ directories alongside testsuite/Cargo.toml. Choose one directory for a separate migration, starting with its test targets and dependencies; the devolutions-gateway tests may require making __debug__ knobs reachable. Done means the tests live in the testsuite crate's single test binary and still exercise a shipped binary or a public library API.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

testsuite/AGENTS.md states the policy: new integration tests go in the testsuite crate's single test binary, so the suite links once and runs as one process, and a test either drives a shipped binary or exercises the public API of a library crate under crates/.

Nine crates still carry Cargo-discovered tests/ directories that predate the policy; only testsuite/Cargo.toml sets autotests = false:

  • devolutions-gateway/tests/ (config.rs, dvls_compatibility.rs, jmux.rs, preflight.rs, token_security.rs, traffic.rs, webapp.rs)
  • crates/transport/tests/
  • crates/jmux-proto/tests/
  • crates/jmux-proxy/tests/
  • crates/http-client-proxy/tests/
  • crates/agent-tunnel-proto/tests/
  • crates/traffic-audit-libsql/tests/
  • crates/video-streamer/tests/
  • crates/devolutions-gateway-ai/tests/

Migration is deliberately deferred. Until it happens, the policy applies to new tests: add them to testsuite, not to a crate's own tests/.

Moving each directory is a candidate for its own pull request, since the devolutions-gateway tests in particular link the binary crate and may need __debug__ knobs to become reachable from outside.

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

Langage dominant
Rust
Étoiles
164
Forks
30
Merge moyen
2 j 8 h
PR mergées (30 j)
106

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de Devolutions/devolutions-gateway

Toutes les issues de Devolutions/devolutions-gateway

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.