hrynco/hrynco-ef
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(transaction): retain both transaction failures (IT-1588) #16

Merged
agrynco merged 1 commits from fix/IT-1588-preserve-transaction-errors into development 2026-10-15 14:27:58 +03:00
Conversation 0 Commits 1 Files Changed 8
+334 -7
agrynco commented 2026-10-15 14:14:11 +03:00
Owner
Copy Link
Copy Source

Summary

  • Preserve both the original action or commit failure and any rollback failure in an AggregateException, with the original first.
  • Clear the owned transaction reference after disposal; keep caller-owned transaction behavior unchanged.
  • Add SQLite transaction lifecycle regression tests and document the failure contract.

Validation

  • dotnet test hrynco-ef.slnx --verbosity minimal: 14 passed.
  • dotnet build hrynco-ef.slnx --no-restore --verbosity minimal: 0 warnings or errors.
  • Independent code review: no actionable findings; cognitive complexity reviewed manually.

Ref: IT-1588; related hrynco/hrynco-ef#14.

## Summary - Preserve both the original action or commit failure and any rollback failure in an AggregateException, with the original first. - Clear the owned transaction reference after disposal; keep caller-owned transaction behavior unchanged. - Add SQLite transaction lifecycle regression tests and document the failure contract. ## Validation - `dotnet test hrynco-ef.slnx --verbosity minimal`: 14 passed. - `dotnet build hrynco-ef.slnx --no-restore --verbosity minimal`: 0 warnings or errors. - Independent code review: no actionable findings; cognitive complexity reviewed manually. Ref: IT-1588; related hrynco/hrynco-ef#14.
agrynco added 1 commit 2026-10-15 14:14:11 +03:00
fix(transaction): retain both transaction failures 5dd8342bf7 
Keep the original action or commit failure visible alongside a failed rollback and cover owned transaction reuse.

Ref: IT-1588
agrynco merged commit dbd569b3d6 into development 2026-10-15 14:27:58 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agrynco (Anatolii Hrynchuk)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: hrynco/hrynco-ef#16
Delete Branch "fix/IT-1588-preserve-transaction-errors"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?