Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

examples: Actor UnitTest example #1326

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from
Draft

Conversation

m3nax
Copy link
Contributor

@m3nax m3nax commented Jul 16, 2024

Description

Added to the Actor example some UnitTests to demonstrate how to write them.

Tests:

  • How to intercept state writing and check whether the persisted state is the expected state
  • How to test the method behavior with a specific initial state

Issue reference

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

m3nax and others added 13 commits July 16, 2024 20:37
Signed-off-by: Manuel Menegazzo <[email protected]>
Signed-off-by: Manuel Menegazzo <[email protected]>
…, because it is used in the example

Signed-off-by: Manuel Menegazzo <[email protected]>
Signed-off-by: Manuel Menegazzo <[email protected]>
Signed-off-by: Manuel Menegazzo <[email protected]>
Signed-off-by: Manuel Menegazzo <[email protected]>
Signed-off-by: Manuel Menegazzo <[email protected]>
@m3nax m3nax requested review from a team as code owners July 16, 2024 18:43
@m3nax m3nax mentioned this pull request Jul 16, 2024
3 tasks
@m3nax m3nax marked this pull request as draft August 21, 2024 08:28
@WhitWaldo
Copy link
Contributor

@m3nax How's this coming along? Ready for a review yet?

@m3nax
Copy link
Contributor Author

m3nax commented Oct 21, 2024

@WhitWaldo This pull request still needs some work. As soon as it is ready I will take it out of draft status.

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.

2 participants