test: add plugin conformance suite handlers (10-1..10-7) - #580
Conversation
c0a11f7 to
19c7ec1
Compare
19c7ec1 to
e5ff099
Compare
e5ff099 to
5631d04
Compare
5631d04 to
fee0513
Compare
This comment has been minimized.
This comment has been minimized.
Claude AI reviewReview — PR #580: plugin conformance suite handlers (10-1..10-7)No actionable findings. This PR adds test-only conformance handlers plus
Residual test risk (considered, not a defect): Reviewed commit |
7 handlers exercising DurableInstrumentationPlugin lifecycle hooks via durable_execution(plugins=[...]), plus template_plugin.yaml. Validated live: 7/7 PASS.
One JSON object per hook ({plugin, hook, first|status|n|outcome|op}),
values from the real hook info parameters; replaces packed text lines so
specs assert typed fields directly.
Records carry the execution ARN received via the invocation-start hook (captured in plugin state for operation/user-function hooks) so the runner's execution-scoped CloudWatch JSON filter retrieves them.
fee0513 to
2cc14d7
Compare
Codex AI reviewNo actionable findings found. Residual risk remains around live Lambda/CloudWatch log formatting and the external structured-log matcher, which are not locally covered. Reviewed commit |
|
re AI review: We use print + json intentionally, since the test runner is filtering the top level durable execution arn, and the plugin does not have access to the |
What
7 handlers for the
pluginconformance suite (instrumentation-plugin lifecycle hooks) inpackages/aws-durable-execution-sdk-python-conformance-tests/handlers/plugin/, plustemplate_plugin.yaml.Validation