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

More CXF tests and cleanup #4207

Merged
merged 4 commits into from
Oct 25, 2022
Merged

More CXF tests and cleanup #4207

merged 4 commits into from
Oct 25, 2022

Conversation

ppalaga
Copy link
Contributor

@ppalaga ppalaga commented Oct 24, 2022

fix #4203

... because camel-cxf-soap depends on cxf-rt-features-logging
package names, test names and property names among the individual cxf
test modules so that they do not clash when run together
@ppalaga
Copy link
Contributor Author

ppalaga commented Oct 24, 2022

@llowinge I have added some fixes for MTOM in native mode, most notably https://github.com/apache/camel-quarkus/pull/4207/files#diff-7c81397449fd40a8260c63c1f182320f8aab02b7e56879589ffce1d46b36df6fR35-R38 and https://github.com/apache/camel-quarkus/pull/4207/files#diff-a6565d3563e7493df31d39fe68eba12b8d5892af636f96541f78d22e385cac8fR72-R74 but it still fails.
So I filed #4208 and disabled the test in native mode https://github.com/apache/camel-quarkus/pull/4207/files#diff-e9e596e55a8e034e9d04f6b83ba0a06cc87694d14824540234ece03404e73082R37
Feel free to try to fix it.
I think it would be worth to try to see an MTOM test with java.awt.Image working on plain quarkus-cxf. If it won't work there, then we know that it is an AWT problem rather than CXF problem. Otherwise it will point in direction CQ. Note that a DataHandler MTOM test is already there in quarkus-cxf: https://github.com/quarkiverse/quarkus-cxf/tree/main/integration-tests/mtom

@llowinge
Copy link
Contributor

I've fixed it with not using the awt.* classes in the test app (it is needed for the test scenario anyway). I was unable to work on that with macOS because of oracle/graal#4124. Fix is in commit llowinge@2fae5ab.

@ppalaga ppalaga merged commit 3836ce8 into apache:main Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create test coverage for CXF SOAP MTOM
4 participants