Skip to content

chore(ci): define a test script so the workflow runs the suite - #67

Merged
ralyodio merged 1 commit into
masterfrom
chore/ci-run-tests
Jul 28, 2026
Merged

chore(ci): define a test script so the workflow runs the suite#67
ralyodio merged 1 commit into
masterfrom
chore/ci-run-tests

Conversation

@ralyodio

Copy link
Copy Markdown
Collaborator

.github/workflows/test.yml gates on a test script:

if jq -e '.scripts.test' package.json >/dev/null; then
  bun run test
else
  echo 'No "test" script defined — skipping.'
fi

No such script existed, so every green run to date was green because nothing ran — the 25 tests under tests/ have never executed in CI. Recent master runs finishing in ~20s are checkout + bun install and nothing else.

Adds "test": "bun test tests/". Scoped to tests/ rather than a bare bun test so the runner doesn't wander into node_modules.

Verification

Locally, bun run test → 25 pass / 0 fail. The CI run on this PR is the real check: its test step should now report 25 tests instead of the skip message, and take longer than the ~20s no-op runs.

bun.lock pins only the root package name, not its version, so the recent v0.2.0 bump doesn't affect bun install --frozen-lockfile.

🤖 Generated with Claude Code

.github/workflows/test.yml only runs `bun run test` when package.json
defines a "test" script, and otherwise prints "No test script defined —
skipping". No such script existed, so every green run to date was green
because nothing ran: the 25 tests under tests/ were never executed in CI.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

vu1nz Security Review

0 finding(s) in PR #?

No security issues found.

@ralyodio
ralyodio merged commit 5d976ea into master Jul 28, 2026
2 checks passed
@ralyodio
ralyodio deleted the chore/ci-run-tests branch July 28, 2026 21:43
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