Skip to content

Feature/implementation tester#345

Merged
IrisvdWerf merged 63 commits into
developfrom
feature/implementation-tester
May 19, 2026
Merged

Feature/implementation tester#345
IrisvdWerf merged 63 commits into
developfrom
feature/implementation-tester

Conversation

@IrisvdWerf
Copy link
Copy Markdown
Collaborator

No description provided.

@IrisvdWerf IrisvdWerf self-assigned this Mar 31, 2026
@IrisvdWerf IrisvdWerf requested a review from maarten-ic March 31, 2026 15:29
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/implementation_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/implementation_tester.py Outdated
@IrisvdWerf IrisvdWerf requested a review from maarten-ic April 3, 2026 07:28
Copy link
Copy Markdown
Collaborator

@maarten-ic maarten-ic left a comment

Choose a reason for hiding this comment

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

Couple of comments, we should probably be close to get this working for a simple case?

Comment thread libmuscle/python/libmuscle/pytest/implementation_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/implementation_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/implementation_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/implementation_tester.py
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py
Comment thread libmuscle/python/libmuscle/manager/instance_manager.py
@IrisvdWerf IrisvdWerf requested a review from maarten-ic April 8, 2026 13:13
@IrisvdWerf IrisvdWerf requested a review from maarten-ic May 4, 2026 11:08
@IrisvdWerf IrisvdWerf requested a review from LourensVeen May 4, 2026 11:16
@IrisvdWerf IrisvdWerf requested review from LourensVeen and removed request for LourensVeen May 11, 2026 11:36
Copy link
Copy Markdown
Contributor

@LourensVeen LourensVeen left a comment

Choose a reason for hiding this comment

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

This is looking really nice! I have a few comments for you to consider, but I'm going to mark this Approve and let you decide what to do with them (well, the PROJECT_DIR vs. TESTS_DIR thing in the docs needs to be fixed probably).

Comment thread docs/source/muscle_tester.rst Outdated
Comment thread docs/source/muscle_tester.rst Outdated
Comment thread docs/source/muscle_tester.rst
Comment thread docs/source/muscle_tester.rst Outdated
Comment thread docs/source/muscle_tester.rst
Comment thread integration_test/ymmsl/codes/macro.ymmsl Outdated
Comment thread integration_test/test_muscle_tester_macro_micro.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/implementation_tester.py
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py
Comment thread pyproject.toml
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py
Comment thread pyproject.toml
Comment thread pyproject.toml
Copy link
Copy Markdown
Collaborator

@maarten-ic maarten-ic left a comment

Choose a reason for hiding this comment

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

Couple of comments, let me know if you have any questions about these!

Comment thread integration_test/conftest.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
Copy link
Copy Markdown
Collaborator

@maarten-ic maarten-ic left a comment

Choose a reason for hiding this comment

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

This should be my final nitpick 😉

Comment thread libmuscle/python/libmuscle/pytest/muscle_tester.py Outdated
@maarten-ic maarten-ic self-requested a review May 19, 2026 11:44
Copy link
Copy Markdown
Collaborator

@maarten-ic maarten-ic left a comment

Choose a reason for hiding this comment

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

🚀

@IrisvdWerf IrisvdWerf merged commit 866744f into develop May 19, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants