Skip to content

test: add module-aligned source coverage#4

Merged
jasonsaayman merged 1 commit into
mainfrom
feat/implement-testing
May 15, 2026
Merged

test: add module-aligned source coverage#4
jasonsaayman merged 1 commit into
mainfrom
feat/implement-testing

Conversation

@jasonsaayman
Copy link
Copy Markdown
Member

@jasonsaayman jasonsaayman commented May 15, 2026

Summary

  • Split the catch-all test file into module-aligned tests for action, comparison, config, paths, report, and tarball behavior.
  • Expanded edge-case coverage for path safety, tarball parsing/download failures, report writing, gzip comparison output, and action orchestration with a mocked axios npm tarball.
  • Synced and archived the OpenSpec add-src-test-suite change, and strengthened repo guidance so code changes include tests by default.

Testing

  • pnpm test

Summary by cubic

Reorganized the test suite into module-aligned files and added focused edge-case coverage to improve safety and maintainability. Added an OpenSpec “source-test-suite” requirement and repo guidance so future changes include tests by default.

  • Refactors
    • Split the catch-all test into tests/action.test.js, comparison.test.js, config.test.js, paths.test.js, report.test.js, and tarball.test.js; removed tests/index.test.js.
    • Added an orchestration test for run() that mocks an npm axios tarball download and verifies GITHUB_OUTPUT and report output.
    • Expanded coverage for path normalization/safety, tarball parsing and failures, gzip size/delta math, and safe report writing.
    • Kept tests deterministic and offline; no new runtime or dev dependencies.
    • Synced OpenSpec: added source-test-suite specs, archived the change, and updated openspec/config.yaml and AGENTS.md to require module-aligned tests.
    • No runtime behavior changes; pnpm test compiles to lib/ before running.

Written for commit 0c1d537. Summary will update on new commits. Review in cubic

@jasonsaayman jasonsaayman self-assigned this May 15, 2026
@jasonsaayman jasonsaayman changed the title Add module-aligned source test coverage test: add module-aligned source coverage May 15, 2026
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 15 files

You're on the cubic free plan with 16 free PR reviews remaining this month. Upgrade for unlimited reviews.
Re-trigger cubic

@jasonsaayman jasonsaayman merged commit f51c163 into main May 15, 2026
2 checks passed
@jasonsaayman jasonsaayman deleted the feat/implement-testing branch May 15, 2026 19:52
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