CAMEL-16853: fix resultHandlerFactory, add unit test#5945
CAMEL-16853: fix resultHandlerFactory, add unit test#5945davsclaus merged 5 commits intoapache:mainfrom
Conversation
CAMEL-16853: fix resultHandlerFactory, add unit test
|
There is a cycle dependency with maven now Can you avoid using camel-test-spring5 and just build a unit test that are Java DSL based |
|
Apologies, I borrowed some code from saxon-xslt since it's very similar and forgot to check for this. I have rewritten the unit test using DSL and camel-test-junit5, but the cyclic dependency still exists: cyclic reference: Edge between Any suggestions? |
CAMEL-16853: Change unit test to DSL version, without depending on spring (note: still cyclic!)
|
Yes the unit test should not be in camel-xslt, but in core/camel-core, there are some xslt tests there |
CAMEL-16853: Fix cyclic dep: Moved unit test to camel-core, removed test dependencies from camel-xslt
|
Moved test to core, reverted changes to camel-xslt pom. Change is now just 1 new unit test and a 3-line move |
* CAMEL-16853: fix resultHandlerFactory, add unit test * CAMEL-16853: Change unit test to DSL version, without depending on spring (note: still cyclic!) * CAMEL-16853: Fix cyclic dep: Moved unit test to camel-core, removed test dependencies from camel-xslt Co-authored-by: Rik van Ballegooijen <rik@codecast.nl>
* CAMEL-16853: fix resultHandlerFactory, add unit test * CAMEL-16853: Change unit test to DSL version, without depending on spring (note: still cyclic!) * CAMEL-16853: Fix cyclic dep: Moved unit test to camel-core, removed test dependencies from camel-xslt Co-authored-by: Rik van Ballegooijen <rik@codecast.nl>
3 participants
Unit test provided that demonstrates the flaw in current code.