Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

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

Đang mở
  • Có gì không ổn?
#2,090 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@CBenoit đang làm issue này rồi.

Từ ngày 16/10/2026.

  • #2101 của @CBenoit — đang mở

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
18/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rust
Lĩnh vực
build-system, testing

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
Rust
Star
162
Fork
30
Merge trung bình
1 ngày 9 giờ
Pull request đã merge (30 ngày)
60

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Devolutions/devolutions-gateway

Tất cả issue của Devolutions/devolutions-gateway

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.