Skip to content

Conversation

@basvandijk
Copy link
Collaborator

@basvandijk basvandijk commented Jan 27, 2026

After #8449 it's required to specify runtime dependencies of system-tests using the runtime_deps parameter. This already prepares the //rs/tests/message_routing:queues_compatibility_test for this requirement.

To make this work the rust_test_with_binary bazel macro had to be rewritten to not symlink to the test binary but to copy it instead. Otherwise the system-test would use the no_op executable as its dependency.

@github-actions github-actions bot added the chore label Jan 27, 2026
@basvandijk basvandijk marked this pull request as ready for review January 27, 2026 20:25
@basvandijk basvandijk requested review from a team as code owners January 27, 2026 20:25
Copy link
Contributor

@alin-at-dfinity alin-at-dfinity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you.

@basvandijk basvandijk enabled auto-merge January 27, 2026 22:14
@basvandijk basvandijk added this pull request to the merge queue Jan 28, 2026
@basvandijk basvandijk removed this pull request from the merge queue due to a manual request Jan 28, 2026
@basvandijk basvandijk added this pull request to the merge queue Jan 28, 2026
Merged via the queue into master with commit 8f5d87d Jan 28, 2026
69 of 70 checks passed
@basvandijk basvandijk deleted the basvandijk/use-runtime_deps-in-queues_compatibility_test branch January 28, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants