-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add munit test framework #536
Conversation
Thanks, but why a new test framework ? |
I'd prefer not to add more dependencies to the main testkit. Could this go into a new testkit extension module? Something like pekko-http-testkit-munit. |
* license agreements; and to You under the Apache License, version 2.0: | ||
* | ||
* https://www.apache.org/licenses/LICENSE-2.0 | ||
* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new code should get the standard ASF header - https://www.apache.org/legal/src-headers.html#headers
To me the issue is less about whether adding the dependency but more about whether the dependency itself is stable or not (talking about bincompat here). If munit breaks fairly often, adding it to Pekko will create more problems then it solves |
That true,scala gears has to self maintain a fork of munit. |
Agree that this is an extension module |
The testkit doesn't guarantee binary compatibility: https://github.com/apache/pekko-http/blob/main/build.sbt#L225 |
I'll rework the PR and do that. |
This has been setting in our private repo for a while now, I thought it would be a nice addition. And afaics MUnit is gaining some traction. |
1514a74
to
4a89989
Compare
I separated it into a module, please let me know if it is ok like this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Thanks @Philippus - merging Would you have any interest in making a similar PR for main Pekko repo to add a pekko-testkit-munit? There are a few more testkits there too (streams, actor-typed, persistence). |
There are more testkits, but I think only actor-typed and pekko-connectors-kafka have equivalent scalatest-specific functionality. |
@Philippus It isn't strictly necessary but could you consider supplying an iCLA (if you haven't done so already)? |
Thanks @Philippus - your iCLA was received by the ASF Secretary. |
Applying this PR will add support for testing routes with MUnit.