cunningt opened a new pull request, #2833:
URL: https://github.com/apache/camel-kamelets/pull/2833

   ## Summary
   
   Fixes integration test timeouts by changing YAML literal block scalars (`|`) 
to folded scalars (`>-`) in test configuration files.
   
   ## Problem
   
   The `rest-openapi-sink-add-pet-test` was timing out after 123 seconds 
because the Camel JBang process failed to start when multi-line JSON strings 
were passed as command-line arguments using YAML's literal block scalar (`|`).
   
   When YAML literal scalars are used for system properties, the newlines are 
preserved in the resulting command-line arguments, causing shell parsing errors 
that prevent the Camel integration from starting.
   
   ## Solution
   
   Changed all instances of `value: |` to `value: >-` (folded scalar with 
trailing newline stripped) in test files. This folds multi-line content into 
single-line strings, which:
   - Avoids command-line parsing issues
   - Works correctly with shell arguments
   - Preserves the JSON content while removing problematic newlines
   
   ## Changes
   
   Updated 13 test configuration files across:
   - OpenAPI tests (1 file)
   - Transformation tests (3 files)
   - Avro tests (2 files)
   - Protobuf tests (2 files)
   - AWS tests (5 files)
   
   All changes are purely YAML formatting - the actual JSON content remains 
identical.
   
   ## Test Results
   
   ### Before
   - OpenApiIT: 1/2 passing (`rest-openapi-sink-add-pet-test` timing out after 
123s)
   
   ### After
   - OpenApiIT: 2/2 passing (add-pet test now completes in ~11s)
   
   ### No Regressions
   - All previously passing tests continue to pass
   - Pre-existing test failures in AvroIT, ProtobufIT, and CommonIT remain 
unchanged (different root causes)
   
   ## Background
   
   YAML Scalar Comparison:
   - **Literal (`|`)**: Preserves all newlines → multi-line command arguments → 
shell parsing fails
   - **Folded (`>-`)**: Joins lines with spaces → single-line arguments → shell 
parsing succeeds
   
   This is a preventive fix that also protects other tests from similar timeout 
issues in the future.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to