Skip to content

Commit 8e808df

Browse files
gnodetclaude
andcommitted
ci: treat test-infra modules as testable for -amd expansion
When a test-infra module changes (e.g., container version bump), its dependents should be tested via -amd even though the test-infra module itself has no src/test. Previously these were classified as "pom-only" and excluded from -amd expansion, so component tests were never run. Fixes the issue seen in PR apache#22316 where bumping the Infinispan container version didn't trigger camel-infinispan tests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b40772d commit 8e808df

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

.github/actions/incremental-build/incremental-build.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -556,6 +556,11 @@ main() {
556556
for w in $(echo "$pl" | tr ',' '\n'); do
557557
if [ -d "$w/src/test" ]; then
558558
testable_pl="${testable_pl:+${testable_pl},}${w}"
559+
elif [[ "$w" == test-infra/* ]]; then
560+
# test-infra modules have no tests themselves but their dependents do —
561+
# treat them as testable so -amd expands to the components that use them
562+
testable_pl="${testable_pl:+${testable_pl},}${w}"
563+
echo " Test-infra module (no src/test, but dependents will be tested via -amd): $w"
559564
else
560565
pom_only_pl="${pom_only_pl:+${pom_only_pl},}${w}"
561566
echo " Pom-only module (no src/test, won't expand dependents): $w"

0 commit comments

Comments
 (0)