Skip to content

test: add update artwork seams - #1238

Open
TB516 wants to merge 1 commit into
projectbluefin:mainfrom
TB516:quality/update-artwork-test-seam
Open

test: add update artwork seams#1238
TB516 wants to merge 1 commit into
projectbluefin:mainfrom
TB516:quality/update-artwork-test-seam

Conversation

@TB516

@TB516 TB516 commented Sep 12, 2026

Copy link
Copy Markdown

Summary

  • export the artwork driver's fetch, conversion, manifest, sync, and entry-point functions
  • guard main() so importing the module is side-effect free
  • inject network, command, path, time, and write dependencies at the relevant seams
  • cover manifest round-tripping, conversion commands, all collection planners, preflight failure, and the no-change path

Verification

  • node --test scripts/update-artwork.test.js
  • node --test --experimental-test-coverage scripts/update-artwork.test.js (78.27% line coverage for scripts/update-artwork.mjs)
  • npx eslint scripts/update-artwork.mjs scripts/update-artwork.test.js
  • npm run lint (0 errors; pre-existing warnings)
  • npm test (101/102 files pass; unrelated render-release-card.test.js is blocked by sandbox spawnSync /usr/bin/node EPERM)
  • npm run typecheck (blocked by generated JSON seeds absent from a clean checkout)

Fixes #1230

— hive: backend=codex

Signed-off-by: TB516 <TB516@users.noreply.github.com>
@TB516
TB516 requested a review from castrojo as a code owner September 12, 2026 05:47
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.

[quality] scripts/update-artwork.mjs (651 lines) has zero coverage and exports nothing — needs a test seam before it can be covered

1 participant