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
OAK-9896 - Running unit-tests in IntelliJ dos not work #664
Conversation
@@ -1316,5 +1297,53 @@ | |||
</pluginManagement> | |||
</build> | |||
</profile> | |||
<profile> | |||
<id>intellij-build-workaround</id> |
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.
I find it a bit nasty that this workaround profile is actually containing the default configuration when it is not running in IntelliJ. Can we rather leave the default in place without any profile, and override only for IntelliJ (with this profile)?
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.
I agree with you in principle. The problem is that I don't know how to remove the _plugin.scr
instruction once it's configured. Do you know of a way to achieve that?
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.
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.
Thank you for the pointer! Please have a look at the revised change.
9aea662
to
7e08cfa
Compare
oak-parent/pom.xml
Outdated
<id>intellij-build-workaround</id> | ||
<!-- workaround for IntelliJ issue https://youtrack.jetbrains.com/issue/IDEA-141732/OSMORC-unable-to-load-bnd-plugin --> | ||
<!-- IntelliJ does not add the plugin's extra dependency to the plugin's class-path, which causes the build to fail. --> | ||
<!-- This means unit-tests cannot be run from within the IDE. Adding the relevant configuration only in the absence --> |
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.
this comment is outdated now.
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.
Thank's again :) Should be ok now.
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 now.
No description provided.