Failure Details
Failed Jobs and Errors
The per-class coverage gate detected these regressions:
Per-class coverage regressions detected:
graphql.normalized.ExecutableNormalizedOperationFactory Line: 84.0% (was 92.0%, delta -8.0%)
graphql.normalized.ExecutableNormalizedOperationFactory Method: 75.0% (was 87.5%, delta -12.5%)
graphql.execution.directives.QueryDirectivesImpl Line: 97.3% (was 97.6%, delta -0.4%)
graphql.execution.directives.QueryDirectivesImpl Branch: 75.0% (was 80.0%, delta -5.0%)
graphql.execution.directives.QueryDirectivesImpl Method: 93.8% (was 94.4%, delta -0.7%)
Update test-baseline.json if these changes are intentional.
Root Cause
PR #4297 adds a new OperationDirectivesResolver directivesResolver field to ExecutableNormalizedOperationFactoryImpl (in src/main/java/graphql/normalized/ExecutableNormalizedOperationFactory.java) and calls directivesResolver.resolveDirectivesByName(...) inside createNormalizedQueryImpl(). This adds new code to ExecutableNormalizedOperationFactory that is not covered by existing tests for that class, causing a significant drop in line and method coverage.
QueryDirectivesImpl suffers a smaller coverage regression likely due to changes in how directive resolution is tested through the updated execution path.
Recommended Fix
Generated by CI Failure Doctor
To install this workflow, run gh aw add githubnext/agentics/workflows/ci-doctor.md@ee50a3b7d1d3eb4a8c409ac9409fd61c9a66b0f5. View source at https://github.com/githubnext/agentics/tree/ee50a3b7d1d3eb4a8c409ac9409fd61c9a66b0f5/workflows/ci-doctor.md.
Failure Details
7b92be4510f311b044ff3b71925c78aa9f350ed0(Merge branch 'master' into extract-out-operation-directives)Failed Jobs and Errors
The per-class coverage gate detected these regressions:
Root Cause
PR #4297 adds a new
OperationDirectivesResolver directivesResolverfield toExecutableNormalizedOperationFactoryImpl(insrc/main/java/graphql/normalized/ExecutableNormalizedOperationFactory.java) and callsdirectivesResolver.resolveDirectivesByName(...)insidecreateNormalizedQueryImpl(). This adds new code toExecutableNormalizedOperationFactorythat is not covered by existing tests for that class, causing a significant drop in line and method coverage.QueryDirectivesImplsuffers a smaller coverage regression likely due to changes in how directive resolution is tested through the updated execution path.Recommended Fix
test-baseline.jsonto reflect the new coverage numbers (if the regressions are acceptable), ORExecutableNormalizedOperationFactory-related test classes that exercise operations with directives (e.g.,query@foo{ field }) to recover coverage for the newdirectivesResolver.resolveDirectivesByName(...)call path inExecutableNormalizedOperationFactory.java.