Skip to content

fix: DynamoDB LATEST pointer + moto tests - #3

Closed
patote85 wants to merge 4 commits into
mainfrom
fix/dynamo-latest-pointer
Closed

patote85 wants to merge 4 commits into
mainfrom
fix/dynamo-latest-pointer

Conversation

@patote85

Copy link
Copy Markdown
Owner

Summary

  • Fixes DynamoDB LATEST pointer: solicitar now TransactWrite history (request_id=UUID) + pointer (request_id=LATEST with active_request_id).
  • Dedup + /status-exclusao read LATEST and return the real UUID via active_request_id.
  • Confirmar pagamento updates history + LATEST in one TransactWrite (condition on active_request_id).
  • Removes broken @idempotent (no store configured); dedup is app-level via LATEST.
  • Real moto tests in tests/test_exclusao.py; gut stub test_app.py.
  • README: additive DynamoDB — ponteiro LATEST section.
  • requirements.txt: pytest + moto.

Out of scope

  • Cognito (PR3)
  • CDK / sam validate

Test plan

  • Local: PYTHONPATH=. pytest tests/test_exclusao.py -v
  • CI / review

Do not merge without CAB.

Copy link
Copy Markdown
Owner Author

Superseded by rebased PR (main already had ADR-001 README). Use the new PR for CAB — this one stays dirty.

@patote85 patote85 mentioned this pull request Sep 20, 2026
1 of 2 tasks

Copy link
Copy Markdown
Owner Author

Superseded by #4 (rebased on main + ADR README). Use #4 for CAB.

@patote85 patote85 closed this Sep 20, 2026
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