Skip to content

nunit: use built-in bktec nunit runner#41

Draft
pda wants to merge 1 commit intonunitfrom
nunit-builtin
Draft

nunit: use built-in bktec nunit runner#41
pda wants to merge 1 commit intonunitfrom
nunit-builtin

Conversation

@pda
Copy link
Copy Markdown
Member

@pda pda commented Mar 27, 2026

Switches the NUnit example from a custom runner (bin/run-tests) to the built-in nunit runner being added to bktec in buildkite/test-engine-client#39.

Changes

  • BUILDKITE_TEST_ENGINE_TEST_RUNNER=nunit instead of custom
  • Removes bin/run-tests — bktec handles class name extraction and --filter construction internally
  • Installs bktec from source (go install ...@nunit-runner) since the runner is not yet in a release
  • Adds go to mise.toml for building from source

Depends on

@pda pda force-pushed the nunit-builtin branch 2 times, most recently from cf6aa2f to 10322a6 Compare March 27, 2026 05:42
Replace the custom runner (bin/run-tests) with bktec's built-in nunit
runner from the unreleased nunit-runner branch of test-engine-client.

- Use BUILDKITE_TEST_ENGINE_TEST_RUNNER=nunit instead of custom
- Remove bin/run-tests (bktec handles filter construction internally)
- Install bktec from source via go install (nunit-runner branch)
- Add go to mise.toml for building from source

Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d2d0d-68c7-731c-b8dc-f482d3d473d4
@pda
Copy link
Copy Markdown
Member Author

pda commented Mar 27, 2026

Awaiting server-side change to allow nunit as a runner name…

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