Skip to content

test: add replay identity conformance case - #676

Open
zhongkechen wants to merge 4 commits into
mainfrom
test/conformance-general-11-1
Open

test: add replay identity conformance case#676
zhongkechen wants to merge 4 commits into
mainfrom
test/conformance-general-11-1

Conversation

@zhongkechen

Copy link
Copy Markdown
Contributor

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

Implements aws/aws-durable-execution-conformance-tests#111.

Description

Adds the general conformance suite and Java handler for requirement 11-1.
The handler emits a named WAIT initially, then deliberately emits a named
STEP at the same durable position on replay. Replay logging is enabled so the
suite can assert that replay occurred and that the mismatched step body never
ran.

Demo/Screenshots

Not applicable.

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

Unit Tests

  • mvn -pl conformance-tests -am test (1,157 tests passed)
  • mvn spotless:apply

Integration Tests

The new general suite is discovered by the existing conformance workflow and
will run requirement 11-1 against the deployed handler.

Examples

Added general.ReplayOperationIdentity as the conformance handler for 11-1.

@zhongkechen
zhongkechen requested a review from a team September 1, 2026 21:26
@zhongkechen
zhongkechen temporarily deployed to ai-pr-review-runtime September 1, 2026 21:26 — with GitHub Actions Inactive
Comment thread conformance-tests/template_general.yaml
@github-actions

This comment has been minimized.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Codex AI review

No actionable findings. Residual risk: the cloud-only conformance scenario and log assertions were reviewed statically but not executed under the review constraints.

Reviewed commit 9a133981d11493ae944c68f855982adb6d743428. Workflow run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant