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
WINDUPRULE-404: eap 7.3 maven-artemis-jms-client rules #384
Conversation
please do not merge, to be fixed once EAP 7.3 is released |
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.
Some changes required to have the validation test to work (i.e. mvn -Dtest=WindupRulesMultipleTests -DrunTestsMatching=maven-artemis-jms-client clean surefire-report:report
) and some wording in message and tag.
rules-reviewed/eap7/eap72/tests/maven-artemis-jms-client.rhamt.test.xml
Outdated
Show resolved
Hide resolved
1c77917
to
93277f8
Compare
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.
Tested executing mvn -Dtest=WindupRulesMultipleTests -DrunTestsMatching=maven-artemis-jms-client clean surefire-report:report
and suggested how to fix one of the failures.
rules-reviewed/eap7/eap72/tests/maven-artemis-jms-client.rhamt.test.xml
Outdated
Show resolved
Hide resolved
93277f8
to
f39a8ac
Compare
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.
The rule is fine.
The current failure in the build is related to:
[main] ERROR org.jboss.windup.rules.tests.WindupRulesLinksTest - Response code 404 for https://access.redhat.com/documentation/en-us/red_hat_jboss_enterprise_application_platform/7.3/html-single/migration_guide/index#migrate_messaging_application_changes_7_2
which is supposed to happen since this doc is not publicly available yet.
It has been decided to merge this PR even if the link test fails because the documentation link has been manually tested. |
Issue: https://issues.jboss.org/browse/WINDUPRULE-404