gnodet commented on PR #22524: URL: https://github.com/apache/camel/pull/22524#issuecomment-4809874257
## Scalpel 0.3.7 validation results Created 3 clean test PRs to validate the shadow comparison with different types of managed dependency changes: | PR | Property | Grep found | Scalpel would test | Scalpel skipped | CI | |---|---|---|---|---|---| | [#24272](https://github.com/apache/camel/pull/24272) | `kafka-version` 4.3.1→4.3.0 | 0 | **2** (`camel-kafka`, `camel-kafka-azure-rebalance-listener`) | 2 | ✅ | | [#24273](https://github.com/apache/camel/pull/24273) | `junit-jupiter-version` 5.14.4→5.14.3 | 1 | **0** | 0 | ✅ | | [#24274](https://github.com/apache/camel/pull/24274) | `jackson2-version` 2.22.0→2.21.2 | 6 | **130** (157 direct, 27 skipped) | 27 | ✅ | ### Key observations - **Kafka** — Scalpel finds 2 modules that grep completely misses. Kafka is consumed via `<dependencyManagement>`, so `${kafka-version}` never appears in child POMs. This is the core value of Scalpel over grep. - **JUnit** — Scalpel correctly reports 0 testable modules. The test-scoped dependency change is handled by source-set-aware propagation — test-jar dependencies don't cascade into production code. - **Jackson** — Scalpel finds **130 testable modules** (plus 27 skipped from the exclusion list). Jackson is widely used via BOM import — grep only catches the 6 modules that explicitly write `${jackson2-version}` in their POM. The `skipTestsForDownstreamModules` correctly filters out all exclusion-list modules. All 3 test PRs passed CI ✅ with no regression to the existing grep-based test execution. _Claude Code on behalf of @gnodet_ -- 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]
