Test that Rust and C++ ipc_bridge example work with each other#21
Closed
saksham-chawla wants to merge 5 commits intocodex/replay-pr-39-basefrom
Closed
Test that Rust and C++ ipc_bridge example work with each other#21saksham-chawla wants to merge 5 commits intocodex/replay-pr-39-basefrom
saksham-chawla wants to merge 5 commits intocodex/replay-pr-39-basefrom
Conversation
Using `--help` lead definitely to a crash, which should not happen, because `--help` is normal behavior. Also `main()` was missing a return statement which is ok in C/C++, but we have no control of the return value in this case. Now it is always 0, when terminating without errors.
CI should check that C++ and Rust code can communicate via mw::com to avoid having surprise regressions.
There is no Windows specific devcontainer anymore and all can use the same devcontainer.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI should check that C++ and Rust code can communicate via mw::com. To simplify the tests the CLI interface of the Rust code has been more aligned with the C++ interface.
Fixes #38
Integration Test for Rust and C++ IPC Bridge Examples
This pull request introduces a new integration test to verify cross-language communication between the Rust and C++
ipc_bridgeexamples usingmw::com. To facilitate this, the command-line interfaces of both examples have been aligned, and their behavior modified to support continuous sending and cycle-limited reception. The PR also includes minor cleanups and build system updates.Architecture Diagram:
flowchart TD subgraph Test_Layer TestScript["test_ipc_bridge.sh (+144 lines)"] end subgraph Example_Layer RustExample["ipc_bridge_rs (+13/-15 lines)"] CppExample["ipc_bridge_cpp (+2/-2 lines)"] end subgraph Middleware_Layer Middleware["mw::com"] end subgraph Build_System BuildFile["BUILD (+10 lines)"] end DataLayer[/"'Service Manifest (JSON)'"/] TestScript -->|runs| RustExample TestScript -->|runs| CppExample TestScript -->|generates| DataLayer RustExample -->|uses| Middleware CppExample -->|uses| Middleware RustExample -->|reads| DataLayer CppExample -->|reads| DataLayer BuildFile -->|defines| TestScript BuildFile -->|defines| RustExample BuildFile -->|defines| CppExampleThis summary was automatically generated by @propel-code-bot