Skip to content

chore: add Citrus test for counter-source#2883

Open
oscerd wants to merge 1 commit into
apache:mainfrom
oscerd:quick-fix/counter-source-test
Open

chore: add Citrus test for counter-source#2883
oscerd wants to merge 1 commit into
apache:mainfrom
oscerd:quick-fix/counter-source-test

Conversation

@oscerd

@oscerd oscerd commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Summary

counter-source carries the camel.apache.org/kamelet.verified: "true" label, but had no behaviour test under tests/camel-kamelets-itest. Per the catalog convention the verified label means "passing behaviour tests", so this adds the missing Citrus test rather than dropping the label.

What it does

  • tests/.../resources/counter/counter-source-pipe.yaml — a Pipe running counter-sourcelog-sink (start: 777777, numbers: 1).
  • tests/.../resources/counter/counter-source-pipe.citrus.it.yaml — runs the Pipe via Camel JBang and verifies the generated sequential number reaches the sink (logMessage: "777777").
  • Registers the new counter package in CommonIT.

Verification

  • mvn verify -Dit.test=CommonIT#counterTEST SUCCESS (Verified Camel integration 'counter-source-pipe' ... All values OK!, ~29s).
  • Smoke-tested the Kamelet directly via camel run: it loads, the counter AtomicInteger bean resolves, and it emits the sequence with Content-Type: text/plain. (Relevant to Can't load counter kamelet with a bean #2771 — the bean-backed counter-source loads and runs on the current catalog version, 4.21.0-SNAPSHOT.)

AI-generated by Claude Code on behalf of Andrea Cosentino (@oscerd).

counter-source carried camel.apache.org/kamelet.verified="true" but had
no behaviour test. Add a Citrus test that runs the counter-source ->
log-sink Pipe via Camel JBang and verifies the generated sequential
number reaches the sink, registering the new "counter" package in
CommonIT. Verified with: mvn verify -Dit.test=CommonIT#counter.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@oscerd oscerd requested review from christophd and davsclaus June 25, 2026 16:20
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