[GitHub] [camel] eas5 opened a new pull request #5256: Improved conditional test execution

[GitBox]

eas5 opened a new pull request #5256:
URL: https://github.com/apache/camel/pull/5256


   **Problem:**
   Conditions within the test method may produce a false test outcome. Once a 
condition is not met, and the code block inside the condition is not executed, 
a false 'passed' outcome can be produced without the test ever being executed.
   
   **Solution:**
   Instead of using a conditional to control the test execution, we propose 
using JUnit Assumptions — a failed assumption results in a test being aborted. 
Assumptions are typically used whenever it does not make sense to continue 
executing a given test method. 
   
   **Result:**
   _Before:_
   ```
   ...
   if (!items.isEmpty()) {
       ...
       assertNotNull(result, "get result");
   }
   ```
   
   _After:_
   ```
   ...
   assumeFalse(items.isEmpty());
   ...
   assertNotNull(result, "get result");
   ```


-- 
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