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

Disable Camel integration tests for now #1135

Merged
merged 1 commit into from
Mar 1, 2019
Merged

Disable Camel integration tests for now #1135

merged 1 commit into from
Mar 1, 2019

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Mar 1, 2019

This is a workaround for #1130 as it's really in the way.

@gsmet gsmet requested a review from Sanne March 1, 2019 14:01
@@ -51,7 +51,10 @@
<module>infinispan-cache-jpa</module>
<module>infinispan-cache-jpa-stress</module>
<module>panache</module>
<!-- Camel tests are not working on Fedora + Bash
see https://github.com/jbossas/protean-shamrock/issues/1130
<module>camel-core</module>
<module>camel-salesforce</module>
Copy link
Member

Choose a reason for hiding this comment

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

shouldn't we disable the Camel IT tests as part of this too?

Copy link
Member Author

Choose a reason for hiding this comment

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

That's the IT tests that are disabled, we are in the integration-tests module :).

Copy link
Member

Choose a reason for hiding this comment

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

doh, right. Sorry I got confused by the local logged output, looks good (and it works!)

@Sanne Sanne merged commit 27ec5f4 into quarkusio:master Mar 1, 2019
@Sanne Sanne deleted the disable-camel-its branch March 1, 2019 14:38
@gsmet gsmet added this to the 0.11.0 milestone Mar 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants