alxpark commented on pull request #187:
URL: https://github.com/apache/camel-spring-boot/pull/187#issuecomment-703191014


   > > @oscerd
   > > > Also, this will break tests for sure.
   > > 
   > > 
   > > Yes, so I sent another PR on camel in order to fix this unit test error.
   > > this should be merged in advance. 
[apache/camel#4356](https://github.com/apache/camel/pull/4356)
   > 
   > @oscerd you're right. My codes tested based on 3.5.0 version, not 
3.6.0-SNAPSHOT. I thought I all fixed the clashes, but I re-ran the unit tests 
on 3.6.0 and found 3 more crashes. Hmm.... I will have a look tomorrow.
   
   **All cleared 3 broken modules after I recompiled today.** I assume that 
it's  camel's 3.6.0-night build tar files 
   I saw the root causes of the test errors and source generated by invalid 
default value(type error) derived from camel modules somewhere., but it's 
cleared after I compiled based on master branch.  So, my code changes will be 
ok..
   
   I think **only "camel-wordpress-starter" test will be broken by this PR 
until the other PR for "camel-wordpress" merged into camel repo.**  as conflict 
WordpressComponentConfigration class in both.
   
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to