Skip to content

Convert systemProperties to args in Citrus tests#2848

Closed
cunningt wants to merge 1 commit into
apache:mainfrom
cunningt:systempropertiestoargs
Closed

Convert systemProperties to args in Citrus tests#2848
cunningt wants to merge 1 commit into
apache:mainfrom
cunningt:systempropertiestoargs

Conversation

@cunningt
Copy link
Copy Markdown
Contributor

@cunningt cunningt commented May 20, 2026

I'm sticking this in draft - locally I'm only getting three integration test failures with these changes - they are all on the KafkaIT tests, which need these citrus changes citrusframework/citrus#1520 in order to enable the authentication features in the RedPanda testcontainer.

Summary

Converts Citrus integration tests from using systemProperties to args with --prop= syntax when running Camel JBang integrations.

Changes

  • Updated 42 test files to use --prop=key=value arguments instead of systemProperties block
  • Simplified test configurations by removing nested property structures
  • Maintains the same test functionality with cleaner syntax

Example Change

Before:

systemProperties:
  properties:
    - name: "http.sink.url"
      value: "${http.server.url}"

After:

args:
  - "--prop=http.sink.url=${http.server.url}"

Test plan

  • ✅ All affected Citrus integration tests should pass
  • ✅ No functional changes to test behavior
  • ✅ Verify tests still properly set properties for Camel integrations

Affected Areas

  • AWS DDB tests
  • AWS EventBridge tests
  • AWS Kinesis tests
  • AWS S3 tests
  • AWS SQS tests
  • Crypto action tests
  • Filter action tests
  • Header action tests
  • HTTP source/sink tests
  • JIRA tests
  • Log sink tests
  • OpenAPI tests
  • Timer tests
  • Transformation action tests

🤖 Generated with Claude Code on behalf of Tom Cunningham

@cunningt cunningt requested review from christophd and oscerd May 20, 2026 19:29
@cunningt cunningt marked this pull request as draft May 20, 2026 19:30
@cunningt cunningt closed this May 20, 2026
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.

2 participants