gnodet commented on PR #22247: URL: https://github.com/apache/camel/pull/22247#issuecomment-4138804452
## Updated Fork Testing Report _Claude Code on behalf of Guillaume Nodet_ After fixing the bugs found in the first round, re-ran the failing tests. All 3 code paths now pass. ### Test 1: Component file change detection ✅ **Run:** [23613433728](https://github.com/gnodet/camel/actions/runs/23613433728) — **All 3 JDKs passed** - ✅ File-path analysis correctly detected `components/camel-http` - ✅ Dependent module threshold (88 > 50) correctly applied ### Test 2: Parent POM dependency detection ✅ **Run:** [23620804048](https://github.com/gnodet/camel/actions/runs/23620804048) — **All 3 JDKs passed** - ✅ POM dependency analysis detected `jetty-version` property change - ✅ Found all 8 referencing modules (camel-undertow, camel-cometd, camel-platform-http, camel-jetty-common, camel-salesforce-maven-plugin, camel-salesforce, camel-jetty, camel-thymeleaf) - ✅ Correctly identified `parent` as pom-only module - ✅ Tests passed on all detected modules ### Test 3: skip_full_build + extra_modules (`/component-test` path) ✅ (script correct, flaky test failures) **Run:** [23615655184](https://github.com/gnodet/camel/actions/runs/23615655184) — **JDK 17/25 failed (flaky test failures)** - ✅ Quick dependency build used instead of regen.sh - ✅ Extra modules correctly merged with file-path detected modules - ✅ Dependent module count correctly calculated as 24 (< 50 threshold) - ❌ Exit code 1 from actual test failures in camel-http dependent modules (not a CI script bug) ### Bugs Found and Fixed | Bug | Commit | Description | |-----|--------|-------------| | Pipefail in dependent count | b9e4fd7 | `wc -l \|\| echo "0"` under `set -o pipefail` produced `"24\n0"`, breaking `[[ -gt ]]` | | Hardcoded github-repo default | b9e4fd7 | `action.yaml` default was `'apache/camel'` instead of `${{ github.repository }}` | | Exclusion list without -amd | 636e55b | `!:camel-allcomponents` exclusion syntax fails when modules aren't in reactor (no `-amd`) | -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
