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.

Sign up for GitHub

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

Jump to bottom

Enhancement: BaseMojoTestCase should initialise @parameter properties with default values #676

Closed
jarmoniuk opened this issue Sep 8, 2022 · 4 comments
Closed

Enhancement: BaseMojoTestCase should initialise @parameter properties with default values #676

jarmoniuk opened this issue Sep 8, 2022 · 4 comments

Comments

@jarmoniuk
Copy link
Contributor

jarmoniuk commented Sep 8, 2022 •
edited
Loading

Currently this is not being done, as a consequence, test writers must care to initialise test values manually. I think this could be automated, for example using reflection.
@slawekjaranowski
Copy link
Member

IMHO should be done in testing library for plugins

@jarmoniuk
Copy link
Contributor Author

Maven testing harness?

@jarmoniuk
Copy link
Contributor Author

jarmoniuk commented Sep 8, 2022 •
edited
Loading

-> I'll create a ticket there + implement it

@slawekjaranowski
Copy link
Member

https://issues.apache.org/jira/projects/MPLUGINTESTING/
https://github.com/apache/maven-plugin-testing

the latest release 2014 😄

jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
be181f9 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
afc06bf 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
ac41eaa 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
2d8e27d 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
21cc1f0 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
7458b40 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
05a2400 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
4c4e832 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: implementing default parameter resolution. Switch…
ee6d77a 
…ing to using the class loader to resolve test files.
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 10, 2022
@jarmoniuk
Fixed mojohaus#676: Removing BaseMojoTestCase -- not needed
2088e18
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 13, 2022
@jarmoniuk
Fixed mojohaus#676: Removing BaseMojoTestCase -- not needed
413e7f2
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 16, 2022
@jarmoniuk
mojohaus#676: Removing BaseTestCase
9e97cc0
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 16, 2022
@jarmoniuk
mojohaus#676: Removing BaseTestCase
33344f8
jarmoniuk added a commit to jarmoniuk/versions-maven-plugin that referenced this issue Sep 16, 2022
@jarmoniuk
Fixed mojohaus#676: Removing BaseMojoTestCase -- not needed
2c11d02
@jarmoniuk jarmoniuk closed this as not planned Won't fix, can't repro, duplicate, stale Sep 17, 2022
slawekjaranowski pushed a commit that referenced this issue Sep 17, 2022
@jarmoniuk @slawekjaranowski
Fixed #676: Removing BaseMojoTestCase -- not needed
0e1d456
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jarmoniuk @slawekjaranowski