-
Notifications
You must be signed in to change notification settings - Fork 127
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
fix(#2406): added test for FakeMaven #2658
fix(#2406): added test for FakeMaven #2658
Conversation
@Yanich96 please add to the description of pull request:
Also it would be nice to change the header of the PR |
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.
@Yanich96 good one, just a few comments from my side
.execute(AssembleMojo.class) | ||
.result(); | ||
MatcherAssert.assertThat( | ||
res, |
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.
@Yanich96 let's add assertion message here and below
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.
@maxonfjvipon I have done
@@ -70,6 +70,13 @@ | |||
* their behaviour and results. | |||
* NOT thread-safe. | |||
* @since 0.28.12 | |||
* @todo #2406:30min Fix {@link FakeMaven#allowedParams(Class)} | |||
* This function parameters of executed Mojo in {@link FakeMaven#execute(Class)} |
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.
@Yanich96 I think English needs to be corrected here. Please use translator or chatGPT
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.
@maxonfjvipon I have changed
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.
@Yanich96 LGTM! Thanks
@yegor256 please approve |
1 similar comment
@yegor256 please approve |
@rultor merge |
Ref: #2406
What's done:
PR-Codex overview
This PR focuses on fixing the configuration of custom parameters in the
FakeMaven
class.Detailed summary
FakeMaven#allowedParams(Class)
function to configure parameters of executed Mojo and parameters of Mojos inside the executed Mojo.AssembleMojo
.