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

Akka.TestKit: configurable expect-no-message-default value #6675 #7006

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

YariSPB
Copy link
Contributor

@YariSPB YariSPB commented Nov 17, 2023

A proposed solution for Akka.TestKit: configurable expect-no-message-default value #6675 task
implements 'expect-no-message-default' setting in the Akka.NET testkit for configuring ExpectNoMessage / ExpectNoMessageAsync timeouts independently from the ExpectMsg / ExpectMsgAsync.

Fixes #

  • appended Reference.conf with a configuration 'expect-no-message-default' = 3s
  • exposed new public property NoMessageRemainingOrDefault in TestKitBase.cs based on a new config value
  • in TestKit_Expect.cs, bounded ExpectNoMsg and ExpectNoMsgAsync, which has only cancellation token as an argument, to new property.
  • updated unit test

A documentation at https://getakka.net/api/Akka.TestKit.TestKitBase.html will need to be updated.

Checklist

For significant changes, please ensure that the following have been completed (delete if not relevant):

* added this config into TestKit setting as ExpectNoMessageDefault property
* in TestKit_Expect, bounded relevant overloads of ExpectNoMsg and ExpectNoMsgAsync to the new config
* updated unit test
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.

1 participant