Skip to content

ci: run vitest test suite in CI (#1) - #108

Open
ghzhost wants to merge 2 commits into
StellarSend:mainfrom
ghzhost:fix/test-suite-wire-up-1
Open

ci: run vitest test suite in CI (#1)#108
ghzhost wants to merge 2 commits into
StellarSend:mainfrom
ghzhost:fix/test-suite-wire-up-1

Conversation

@ghzhost

@ghzhost ghzhost commented Sep 3, 2026

Copy link
Copy Markdown

Summary

Fixes #1 — the repository ships vitest.config.ts, src/test/setup.ts, and 15+ test files, plus vitest / @testing-library/react / jsdom in devDependencies and a "test": "vitest run" script in package.json — but the CI workflow never invoked npm test, so the suite silently bit-rotted with no signal when it broke.

Change

Added a dedicated test job to .github/workflows/ci.yml that runs npm ci && npm test on every push and pull_request, parallel to the existing build job.

  test:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-node@v4
        with: {node-version: 20}
      - run: npm ci && npm test

No changes to package.json, vitest.config.ts, or any source files — those were already correct per the issue description.

Closes #1

ghzhost added 2 commits September 3, 2026 02:28
The repository ships vitest.config.ts, src/test/setup.ts, and 15+ test
files, plus vitest / @testing-library/react / jsdom in devDependencies and
a 'test: vitest run' script in package.json — but the CI workflow never
invoked npm test, so the suite silently bit-rotted with no signal.

Add a dedicated 'test' job to ci.yml that runs npm ci && npm test on every
push and pull_request, parallel to the existing 'build' job.

Fixes StellarSend#1
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.

Test suite exists but is never run in CI or locally (no test runner installed, no test script)

1 participant