Skip to content

test: add vitest unit tests + CI for the GitHub Action#36

Merged
finalerock44 merged 1 commit into
mainfrom
test/add-vitest-unit-tests
Jun 12, 2026
Merged

test: add vitest unit tests + CI for the GitHub Action#36
finalerock44 merged 1 commit into
mainfrom
test/add-vitest-unit-tests

Conversation

@finalerock44

Copy link
Copy Markdown
Contributor

The action had no unit tests (the old test script only built and copied dist into a sibling dir; preserved as test:gha). Add vitest and test getParameters() via mocked @actions/core / @actions/github, covering the input contract: app-file XOR app-binary-id, tag parsing, orientation / download-artifacts / report validation, numeric coercion, boolean inputs, device normalisation, default api-url + inferred name, and the github-context metadata (including include-github-context=false).

A GitHub Actions workflow runs pnpm test on push/PR.

The action had no unit tests (the old `test` script only built and copied
dist into a sibling dir; preserved as `test:gha`). Add vitest and test
getParameters() via mocked @actions/core / @actions/github, covering the
input contract: app-file XOR app-binary-id, tag parsing, orientation /
download-artifacts / report validation, numeric coercion, boolean inputs,
device normalisation, default api-url + inferred name, and the
github-context metadata (including include-github-context=false).

A GitHub Actions workflow runs `pnpm test` on push/PR.
@finalerock44 finalerock44 self-assigned this Jun 12, 2026
@finalerock44 finalerock44 merged commit 6397676 into main Jun 12, 2026
2 checks passed
@finalerock44 finalerock44 deleted the test/add-vitest-unit-tests branch June 12, 2026 12:06
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