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

regression in IntelliJ support #573

Closed
jtnord opened this issue Jul 12, 2022 · 4 comments
Closed

regression in IntelliJ support #573

jtnord opened this issue Jul 12, 2022 · 4 comments
Labels

Comments

@jtnord
Copy link
Member

jtnord commented Jul 12, 2022

Jenkins and plugins versions report

WHen running a plugins unit test in intellij this no longer works out of the box with the plugin-pom 4.40 or higher.

This is in #524 the argLine was updated to use a surefire differed property, but IntelliJ does not support this in the default config.

Workaround is to not get surefire to parse the argline, however this does mean the the unit test can be run differently in the IDE than from maven due to different JVM settings.

IntelliJ / plugin using plugin pom 4.40 or higher

What Operating System are you using (both controller, and any agents involved in the problem)?

N/A

Reproduction steps

in a clean intellij environment clone a plugin using plugin pom 4.40 or higher.

Select a unit test and run it

Expected Results

intellij can not launch the test

Actual Results

intellij launches the test and provides the result (this is the behaviour using a parent prior to 4.40)

Anything else?

this is an intellij bug, but more people seem to use intellij than eclipse (or other IDEs) so we may want to rework how we do the java module confuguration.

@jtnord jtnord added the bug label Jul 12, 2022
@jetersen
Copy link
Member

The release notes clearly documents the workaround and future intellij will work once the fix is released.
https://github.com/jenkinsci/plugin-pom/releases/tag/plugin-4.40

@jtnord jtnord changed the title regression in INtelliJ support regression in IntelliJ support Jul 12, 2022
@jtnord
Copy link
Member Author

jtnord commented Jul 12, 2022

Alas release notes are generally consulted at the time something is updated not later on. A plugin may have multiple developers and other developers may not notice that.

I believe this issue makes things more prominent for those users. I am not one of them as I use eclipse, but am aware of an amount of time lost due to this.

@basil
Copy link
Member

basil commented Jul 12, 2022

Also documented in CONTRIBUTING.md and announced to the developer list 🤷‍♂️

BTW the fix should appear in the next major release of IntelliJ and is already available in the Early Access Program

@basil
Copy link
Member

basil commented Jul 26, 2022

Released in IntelliJ IDEA 2022.2

@basil basil closed this as completed Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants