Skip to content

test(persist): assert the open test transaction and the optional commit_ids result - #87

Open
wolfy-j wants to merge 1 commit into
masterfrom
test/assert-test-transaction
Open

wolfy-j wants to merge 1 commit into
masterfrom
test/assert-test-transaction

Conversation

@wolfy-j

@wolfy-j wolfy-j commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

The persist tests read a transaction that before_each opens and after_each resets to nil, and a commit_ids field the commit result sets only when commits were applied. The go-lua type checker fixes in wippyai/go-lua#51 now see both as possibly nil (29 errors in the module's lint). The helpers assert what the tests rely on.

Test-only; no runtime behavior change.

@wolfy-j
wolfy-j requested a review from msmakouz September 24, 2026 15:44
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