Skip to content

message_passing: one way communication integration tests - #936

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

message_passing: one way communication integration tests#936
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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants