Skip to content

CAMEL-25925: camel-aws2-ddb-starter - align DeleteTable/DescribeTable tests with TABLE_NAME header removal - #2058

Merged
Croway merged 1 commit into
apache:mainfrom
Croway:CAMEL-25925-aws2-ddb-delete-table-header
Oct 26, 2026
Merged

Croway merged 1 commit into
apache:mainfrom
Croway:CAMEL-25925-aws2-ddb-delete-table-header

Conversation

@Croway

@Croway Croway commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Ddb2DeleteTableTest.deleteTable has failed on every Camel-Spring-Boot Daily main build since #423 (Oct 25): expected: <randomTable> but was: <null>.

apache/camel#28365 (CAMEL-25925) deliberately stopped DeleteTableCommand and DescribeTableCommand from setting the CamelAwsDdbTableName result header, so that a following aws2-ddb step doesn't pick it up as its table. Upstream updated AWS2DeleteTableRuleIT and AWS2DescribeTableRuleIT to assertNull. This PR makes the same change in our copies:

  • Ddb2DeleteTableTest
  • Ddb2DescribeTableRuleIT

Test plan

  • mvn verify -Dtest=Ddb2DeleteTableTest in camel-aws2-ddb-starter against the current 4.24.0-SNAPSHOT
  • mvn verify -Dtest=Ddb2DescribeTableRuleIT in camel-aws2-ddb-starter

Claude Code (Opus 5.5) on behalf of Federico Mariani

… tests with TABLE_NAME header removal

Camel core no longer sets the CamelAwsDdbTableName header on DeleteTable and
DescribeTable results (apache/camel#28365), so assert it is absent, as the
upstream AWS2DeleteTableRuleIT/AWS2DescribeTableRuleIT now do.
@Croway
Croway requested review from davsclaus and oscerd October 26, 2026 10:09
@Croway
Croway merged commit 366c6f2 into apache:main Oct 26, 2026
4 checks passed
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