zhens
/
JimLiu-baoyu-skills
mirror of https://github.com/JimLiu/baoyu-skills.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
JimLiu-baoyu-skills/docs/testing.md

2.4 KiB
Raw Blame History

Testing Strategy

This repository has many scripts, but they do not share a single runtime or dependency graph. The lowest-risk testing strategy is to start from stable Node-based library code, then expand outward to CLI and skill-specific smoke tests.

Current Baseline

  • Root test runner: node:test
  • Entry point: npm test
  • Coverage command: npm run test:coverage
  • CI trigger: GitHub Actions on push, pull_request, and manual dispatch

This avoids introducing Jest/Vitest across a repo that already mixes plain Node scripts, Bun-based skill packages, vendored code, and browser automation.

Rollout Plan

Phase 1: Stable library coverage

Focus on pure functions under scripts/lib/ first.

  • scripts/lib/release-files.mjs
  • scripts/lib/shared-skill-packages.mjs

Goals:

  • Validate file filtering and release packaging rules
  • Catch regressions in package vendoring and dependency rewriting
  • Keep tests deterministic and free of network, Bun, or browser requirements

Phase 2: Root CLI integration tests

Add temp-directory integration tests for root CLIs that already support dry-run or local-only flows.

  • scripts/sync-shared-skill-packages.mjs
  • scripts/publish-skill.mjs --dry-run
  • scripts/sync-clawhub.mjs argument handling and local skill discovery

Goals:

  • Assert exit codes and stdout for common flows
  • Cover CLI argument parsing without hitting external services

Phase 3: Skill script smoke tests

Add opt-in smoke tests for selected skills/*/scripts/ packages, starting with those that:

  • accept local input files
  • have deterministic output
  • do not require authenticated browser sessions

Examples:

  • markdown transforms
  • file conversion helpers
  • local content analyzers

Keep browser automation, login flows, and live API publishing scripts outside the default CI path unless they are explicitly mocked.

Phase 4: Coverage gates

After the stable Node path has enough breadth, add coverage thresholds in CI for the tested root modules.

Recommended order:

  1. Start with reporting only
  2. Add line/function thresholds for scripts/lib/**
  3. Expand include patterns once skill-level smoke tests are reliable

Conventions For New Tests

  • Prefer temp directories over committed fixtures unless the fixture is reused heavily
  • Test exported functions before testing CLI wrappers
  • Avoid network, browser, and credential dependencies in default CI
  • Keep tests isolated so they can run with plain node --test