Skip to content

Test infra (optional): extract test-runner auditors to a plugin (pytest/go/cargo as peers) - #475

Merged
AdamFrisby merged 2 commits into
mainfrom
codeybox/1dd2be18
Sep 14, 2026
Merged

Test infra (optional): extract test-runner auditors to a plugin (pytest/go/cargo as peers)#475
AdamFrisby merged 2 commits into
mainfrom
codeybox/1dd2be18

Conversation

@AdamFrisby

Copy link
Copy Markdown
Owner

Automated via CodeyBox — work item 1dd2be18d9c546928877fec7a4055e23


Co-Authored-By: CodeyBox noreply@codeybox.invalid
🤖 Generated with CodeyBox

AdamFrisby and others added 2 commits September 13, 2026 22:41
…ference stub

Moves DotnetTestAuditor from CodeyBox.Audit.Shell into
plugins/CodeyBox.DotnetTestRunnerPlugin (referenced by the host, so the
csharp:test-pass merge gate stays default-registered with the byte-identical
command). Adds a parameterless [CodeyBoxPlugin] entry (DotnetTestRunner) with
static scoped-config run options, a PytestTestRunner reference stub
(RunAsync throws; never registered), Pytest/GoTest/CargoTest TestFramework
peers so future runners need no Core change, and packaging tests. Core doc
comments no longer name the dotnet implementation; host API 1.2 -> 1.3
(additive).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

CodeyBox-Prompt-Revision: 1
Co-Authored-By: CodeyBox <noreply@codeybox.invalid>
…urrent transaction race

GetAsync/GetByNameAsync/ListAsync/ListAuditIterationsAsync issued commands
on the shared connection without holding the per-file write gate, so a
concurrent SaveE2eReplayResultsAsync transaction on the same connection
failed polling reads with 'transaction object is not associated with the
same connection'. Reads now acquire the gate like the existing
ListE2eReplayResultsAsync path.

CodeyBox-Prompt-Revision: 1
Co-Authored-By: CodeyBox <noreply@codeybox.invalid>
@AdamFrisby
AdamFrisby merged commit e605fd5 into main Sep 14, 2026
@AdamFrisby
AdamFrisby deleted the codeybox/1dd2be18 branch September 14, 2026 00:49
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