Jinal-Sarvaiya commented on code in PR #119:
URL: https://github.com/apache/camel-kit/pull/119#discussion_r4361655934
##########
camel-kit-core/src/test/java/io/github/luigidemasi/camelkit/generator/ResourceConsistencyTest.java:
##########
@@ -937,13 +939,13 @@ void
catalogAndExecutionIngressRequireValidatedBindingsInsteadOfSummaryTrust() t
"Every MCP-summary consumer or subagent must inherit
context authority");
}
assertContainsAll(catalogResearcher,
- "camel_catalog_components(limit=0)",
"camel_catalog_component_maven(component, runtime,
platformBom)",
"`camel_catalog_components`, `camel_catalog_eips`",
"`camel_catalog_dataformats`, or `camel_catalog_languages`",
"A detail-call error is `UNVERIFIED`, not authoritative
absence",
"Use `INVALID_BINDING`, never `VERIFIED`",
"`NOT_FOUND` requires a successful, complete matching list
result with no exact artifact name",
+ "source=.camel-kit/config.properties",
Review Comment:
Done. Added four stale-contract patterns to
activeShippedResourcesDoNotContainKnownStaleContractTokens: limit=0 probe,
probe resolved camelVersion instruction, Catalog binding:
camel_catalog_components template, and camel_catalog_find tool name. Restoring
any of these in active resources now fails the test. All 29
ResourceConsistencyTest and 9 ShippedAssetStructureTest tests pass.
--
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]
