Test that Rust and C++ ipc_bridge example work with each other#39
Draft
lurtz wants to merge 5 commits intoeclipse-score:mainfrom
Draft
Test that Rust and C++ ipc_bridge example work with each other#39lurtz wants to merge 5 commits intoeclipse-score:mainfrom
lurtz wants to merge 5 commits intoeclipse-score:mainfrom
Conversation
lurtz
changed the title
Contributor
Author
|
test is locally still unstable. Sometimes service discovery does not work UPDATE fixed with latest rebase |
Contributor
Author
|
Even when tests should run serialized via using tags, I see crashes. Following command may trigger failures UPDATE fixed with latest rebase |
lurtz
force-pushed
the
run-examples-in-ci
branch
from
602cece to
78ed8a2
Compare
lurtz
force-pushed
the
run-examples-in-ci
branch
6 times, most recently
from
6b566d8 to
3cc07f7
Compare
This was referenced Aug 21, 2025
lurtz
force-pushed
the
run-examples-in-ci
branch
from
d2e7de9 to
3710b44
Compare
lurtz
force-pushed
the
run-examples-in-ci
branch
from
3710b44 to
00249e7
Compare
lurtz
force-pushed
the
run-examples-in-ci
branch
2 times, most recently
from
51b515c to
4be07b3
Compare
Contributor
|
Instead of using a shell script we should use a test framework. @draganbjedov, I'm not sure whether SCTF/ITF is far enough in S-CORE to support this? Could you please take a look? |
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.
lurtz
force-pushed
the
run-examples-in-ci
branch
from
0fe616e to
90b635b
Compare
LittleHuba
reviewed
Mar 17, 2026
Contributor
LittleHuba
left a comment
LittleHuba
left a comment
There was a problem hiding this comment.
This test should be shifted to an integration test to simplify the logic and use our infrastructure.
@lurtz would you mind?
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
4 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