Skip to content

[CAMEL-13426] Camel-box, error code 400 (Error parsing input model.) …#2871

Closed
JiriOndrusek wants to merge 1 commit intoapache:masterfrom
JiriOndrusek:CAMEL-13426_error-parsing-input-model
Closed

[CAMEL-13426] Camel-box, error code 400 (Error parsing input model.) …#2871
JiriOndrusek wants to merge 1 commit intoapache:masterfrom
JiriOndrusek:CAMEL-13426_error-parsing-input-model

Conversation

@JiriOndrusek
Copy link
Contributor

…during task related events

Issue: https://issues.apache.org/jira/browse/CAMEL-13426

Problem is fixed by change in date format in box sdk - box/box-java-sdk@391171a (in version 2.30.1).
I've verified that the change of format in request body json (change from "2019-04-17T08:59:35+0000" to "2019-04-18T07:26:27Z") - which is format changed in above commit - fixes problem.

Copy link
Contributor

@oscerd oscerd left a comment

Choose a reason for hiding this comment

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

Did you check the Karaf feature ITest and the Spring Boot ITest? The integration tests should be fine I think.

@JiriOndrusek
Copy link
Contributor Author

@oscerd I was executing tests from camel/components/camel-box/camel-box-component
I can run also other tests. Do I understand correctly, that I should run fuse-karaf/itests? I'm not sure where is spring boot itest

@tadayosi
Copy link
Member

@JiriOndrusek For the Karaf integration tests, you should run tests/camel-itest-karaf:
https://github.com/apache/camel/blob/master/tests/camel-itest-karaf/README.adoc
At least you should do:

$ ./run-test.sh CamelBoxTest

@oscerd
Copy link
Contributor

oscerd commented Apr 18, 2019

Thanks @tadayosi

@JiriOndrusek for the SB Itests you need to check under tests/camel-itest-spring-boot and run CamelBoxTest

@JiriOndrusek
Copy link
Contributor Author

@tadayosi Thank you for showing me karaf-integration tests.
@oscerd Thank you for showing me spring-boot integration tests.

I've executed CamelBoxTest in both suites and it is successful.

I don't think that it is necessary to run some other integration tests, what do you think?

@JiriOndrusek
Copy link
Contributor Author

@oscerd I'll rerun camel-box integration tests to be sure that change of version doesn't cause any trouble.

@oscerd
Copy link
Contributor

oscerd commented Apr 18, 2019

Thanks

@JiriOndrusek
Copy link
Contributor Author

I've executed integration tests again. There are several errors, but all of them were present also before bump of box.sdk version. (I've created issue to address these errors: https://issues.apache.org/jira/browse/CAMEL-13434)
With these facts I would say that this change could be merged.

@oscerd
Copy link
Contributor

oscerd commented Apr 18, 2019

Thanks, merged on master

@oscerd oscerd closed this Apr 18, 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.

3 participants

Comments