Skip to content

CAMEL-23354: camel-jbang use quarkus-junit for Quarkus 3.31+#24200

Merged
davsclaus merged 1 commit into
mainfrom
fix/CAMEL-23354
Jun 23, 2026
Merged

CAMEL-23354: camel-jbang use quarkus-junit for Quarkus 3.31+#24200
davsclaus merged 1 commit into
mainfrom
fix/CAMEL-23354

Conversation

@davsclaus

Copy link
Copy Markdown
Contributor

Summary

  • Replace deprecated quarkus-junit5 artifact with quarkus-junit (renamed in Quarkus 3.31)
  • Add version-conditional logic in FreeMarker templates: use quarkus-junit for Quarkus >= 3.31, keep quarkus-junit5 for older versions
  • Update camel-jbang-mcp/pom.xml directly (fixed Quarkus 3.36.3, no conditional needed)

Changes

  • ExportQuarkus.java: add UseQuarkusJunit boolean flag to template model using VersionHelper.isGE(version, "3.31.0")
  • quarkus-pom.ftl / quarkus-kubernetes-pom.ftl: conditional artifactId based on Quarkus version
  • camel-jbang-mcp/pom.xml: rename quarkus-junit5quarkus-junit
  • TemplateHelperTest.java: add tests for both pre-3.31 and current Quarkus versions

Test plan

  • Existing testQuarkusPomTemplate renamed and updated to verify quarkus-junit5 for Quarkus < 3.31
  • New testQuarkusPomTemplateCurrentVersion verifies quarkus-junit for Quarkus >= 3.31
  • Both modules build cleanly

Claude Code on behalf of Claus Ibsen

Co-Authored-By: Claude noreply@anthropic.com

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Claus Ibsen <claus.ibsen@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions github-actions Bot added the dsl label Jun 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • dsl/camel-jbang/camel-jbang-core
  • dsl/camel-jbang/camel-jbang-mcp

POM dependency changes: targeted tests included

Modules affected by dependency changes (2)
  • :camel-jbang-core
  • :camel-jbang-mcp

⚠️ Some tests are disabled on GitHub Actions (@DisabledIfSystemProperty(named = "ci.env.name")) and require manual verification:

  • dsl/camel-jbang/camel-jbang-core: 1 test(s) disabled on GitHub Actions
  • dsl/camel-jbang/camel-jbang-mcp: 1 test(s) disabled on GitHub Actions

💡 Manual integration tests recommended:

You modified dsl/camel-jbang/camel-jbang-core. The related integration tests in dsl/camel-jbang/camel-jbang-it are excluded from CI. Consider running them manually:

mvn verify -f dsl/camel-jbang/camel-jbang-it -Djbang-it-test
All tested modules (6 modules)
  • Camel :: JBang :: Core
  • Camel :: JBang :: MCP
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: Launcher :: Container

⚙️ View full build and test results

@davsclaus davsclaus merged commit 29c92b5 into main Jun 23, 2026
5 checks passed
@davsclaus davsclaus deleted the fix/CAMEL-23354 branch June 23, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant