Skip to content

message_passing: one way communication integration tests - #936

Closed
gdadunashvili wants to merge 1 commit into
eclipse-score:mainfrom
gdadunashvili:message_passing_integration_tests
Closed

gdadunashvili wants to merge 1 commit into
eclipse-score:mainfrom
gdadunashvili:message_passing_integration_tests

Conversation

@gdadunashvili

Copy link
Copy Markdown
Member

The goal of these tests is to perform simple message_passing behavior and validate liveness, or validate failure modes (depending on if the test is positive or negative)

@gdadunashvili
gdadunashvili force-pushed the message_passing_integration_tests branch 5 times, most recently from e62f71d to db3ce74 Compare August 18, 2026 13:54
The goal of these tests is to perform simple message_passing behavior
and validate liveness, or validate failure modes (depending on if the test
is positive or negative)
@gdadunashvili
gdadunashvili force-pushed the message_passing_integration_tests branch from db3ce74 to ce810e5 Compare August 18, 2026 13:55
@gdadunashvili
gdadunashvili marked this pull request as ready for review August 18, 2026 13:55

std::cout << "Hello from server!" << std::endl;

const score::message_passing::ServerFactory::ServerConfig server_config{10, 1, 5};

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Magic numbers. Hard to understand.

return nullptr;
};

bool stop_listening = false;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should be atomic, no?

return target.wrap_exec("bin/server", args, cwd="/opt/ServerApp", wait_on_exit=True, **kwargs)


# gToDo: this test needs to succeed if the normal sever fails to connect

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Open TODO?

@github-actions

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. If this PR is still relevant, please leave a comment or push new changes to keep it open.

@github-actions github-actions Bot added the stale label Sep 11, 2026
@github-actions

Copy link
Copy Markdown

This pull request has been automatically closed due to inactivity. Feel free to reopen it if you would like to continue working on it.

@github-actions github-actions Bot closed this Sep 18, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in COM - Communication FT Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants