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

[JBTM-3530] no Weld classloading error logs at lra/basic FailedLRAIT #1899

Conversation

ochaloup
Copy link
Contributor

@ochaloup ochaloup commented Sep 2, 2021

https://issues.redhat.com/browse/JBTM-3530

LRA
!MAIN !CORE !QA_JTA !QA_JTS_JDKORB !QA_JTS_OPENJDKORB !QA_JTS_JACORB !BLACKTIE !XTS !PERF NO_WIN !RTS !AS_TESTS !TOMCAT !JACOCO

@jbosstm-bot
Copy link

@jbosstm-bot
Copy link

@jmfinelli
Copy link
Contributor

Unfortunately, there is not a way to tell Arquillian to avoid appending the test class to the deployed archive (even though the class is annotated with @RunAsClient). Yeah, agree with you, appending the TestBase is the only solution to get rid of the WARN. Thanks Ondra for fixing this

Copy link
Collaborator

@xstefank xstefank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can try also testable = false in @Deployment annotation but I don't see big issues with this solution.

@jmfinelli
Copy link
Contributor

@xstefank Tried it already but no luck. It seems that Arquillian appends the test class a priori

@ochaloup
Copy link
Contributor Author

ochaloup commented Sep 2, 2021

Thanks @jmfinelli for the elaboration and @xstefank for the tip.
I tried with the testable and looking about the RunAsClient and I don't understand why the testcase is loaded. Maybe because of @Before or @BeforeClass methods?
Anyway, this fix seems to me the easiest one as Manuel said.

I'm going to merge this.

@ochaloup ochaloup merged commit c24d604 into jbosstm:master Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants