Skip to content

test: migrate integration tests to rust - #1793

Open
hackeramitkumar wants to merge 12 commits into
agntcy:mainfrom
hackeramitkumar:test/integration-tests-migration
Open

test: migrate integration tests to rust#1793
hackeramitkumar wants to merge 12 commits into
agntcy:mainfrom
hackeramitkumar:test/integration-tests-migration

Conversation

@hackeramitkumar

@hackeramitkumar hackeramitkumar commented Jul 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Replaces the Go/Ginkgo integration test suite under tests/integration/ with Rust integration tests in tests/ (slim-integration-tests crate), including shared helpers for config rewriting, process lifecycle, log watching, and port allocation.
  • Drops legacy backward-compatibility tests (P2P/group sessions, old relay/app mixes) since v2 no longer supports cross-version compatibility; keeps all other scenarios, with routing/control-plane tests marked #[ignore] where slimctl n route add or Edge route cleanup is still pending.
  • Updates CI to run task tests:integration and hardens the new suite (port allocation across parallel test binaries, dataplane startup wait, clippy clean).

Test plan

  • task tests:integration — all active tests pass
  • cargo clippy -p slim-integration-tests -- -D warnings
  • CI ci-integration-tests workflow green on PR

Type of Change

  • Bugfix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@hackeramitkumar
hackeramitkumar requested a review from a team as a code owner July 5, 2026 11:50
@msardara msardara changed the title test: add ntegration tests migrat test: migrate integration tests to rust Jul 6, 2026
@amitami2
amitami2 force-pushed the test/integration-tests-migration branch from e68013e to 9497d9f Compare July 8, 2026 22:47
@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@amitami2
amitami2 force-pushed the test/integration-tests-migration branch 3 times, most recently from fb52e27 to 653886a Compare July 16, 2026 21:29
amitami2 added 12 commits July 20, 2026 19:38
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Sort tests/Cargo.toml dependencies for cargo-sort, centralize slimctl cm
retries with a 30s timeout, and extend session-handler waits so
add-participant succeeds on slower CI runners.

Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
Signed-off-by: amitami2 <amitami2@cisco.com>
@amitami2
amitami2 force-pushed the test/integration-tests-migration branch from d2357e2 to 60141fb Compare July 20, 2026 14:09
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.

2 participants