[CAMEL-15453] camel-jdbc: replacing deprecated code leads to test failures

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.5.0
• Fix Version/s: 3.5.0
• Component/s: camel-jdbc
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: Otavio Rodolfo Piske
• Created: 24/Aug/20 06:11
• Updated: 26/Aug/20 14:51
• Resolved: 26/Aug/20 14:51

Description

In camel-jdbc modifying the code to replace calls to getOut with getMessage, as instructed by the javadoc, leads to a test failure in JdbcOptionsTest.testInsertCommit.

 

The code fails because this assertion fails. In other words, the call to mock.getExchanges().get(0).getIn().getBody(String.class);  does return a body when applying these changes.

 

I am attaching a patch that can be used to reproduce the issue. To apply the patch, from the camel-jdbc component directory, use patch -p3 < jdbc-deprecations.patch.

 

Note 1: Removing the assertion doesn't seem to affect the results of the test

Note 2: JdbcOptionsTest is not even modified by the patch.