Skip to content

Add vitest-pool-bun CI: run demo suite and selected Node unit tests in Bun pool (PoC)#801

Merged
peter-leonov-ch merged 2 commits into
copilot/vitest-pool-bunfrom
copilot/add-vitest-pool-bun-workflow
Jun 3, 2026
Merged

Add vitest-pool-bun CI: run demo suite and selected Node unit tests in Bun pool (PoC)#801
peter-leonov-ch merged 2 commits into
copilot/vitest-pool-bunfrom
copilot/add-vitest-pool-bun-workflow

Conversation

Copilot AI commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Summary

The vitest-pool-bun proof-of-concept package had no CI coverage. This PR adds a dedicated workflow that now validates both the PoC example suite and a selected subset of repository Node unit tests using the Bun pool runner.

  • New workflow .github/workflows/vitest-pool-bun.yml, path-scoped to vitest-pool-bun/** (and the workflow file), triggered on push to main, pull_request, and workflow_dispatch.
  • Keeps a pinned, least-privilege setup (permissions: {}, SHA-pinned actions, concurrency cancel-in-progress).
  • Runs npm run demo in vitest-pool-bun (green PoC suite), then installs root dependencies and runs selected Node unit tests through Bun pool using:
    • vitest-pool-bun/examples/clickhouse-node-unit.config.mjs
    • packages/client-node/__tests__/unit/node_user_agent.test.ts
    • packages/client-node/__tests__/unit/node_default_logger.test.ts

Checklist

  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG
  • For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

- name: Install dependencies (clickhouse-js root)
working-directory: .
run: |
npm install
Copilot AI changed the title [WIP] Add CI workflow for vitest-pool-bun proof of concept Add vitest-pool-bun CI: run demo suite and selected Node unit tests in Bun pool (PoC) Jun 3, 2026
Copilot AI requested a review from peter-leonov-ch June 3, 2026 14:27
@codecov

codecov Bot commented Jun 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@peter-leonov-ch peter-leonov-ch marked this pull request as ready for review June 3, 2026 14:33
@peter-leonov-ch peter-leonov-ch requested a review from mshustov as a code owner June 3, 2026 14:33
@peter-leonov-ch peter-leonov-ch merged commit 0359256 into copilot/vitest-pool-bun Jun 3, 2026
60 checks passed
@peter-leonov-ch peter-leonov-ch deleted the copilot/add-vitest-pool-bun-workflow branch June 3, 2026 14:33
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.

4 participants