Skip to content

test: assert the real app renders instead of a deleted starter skeleton - #14

Open
PouyanJay wants to merge 1 commit into
thebuggeddev:mainfrom
PouyanJay:fix/11-test-suite
Open

test: assert the real app renders instead of a deleted starter skeleton#14
PouyanJay wants to merge 1 commit into
thebuggeddev:mainfrom
PouyanJay:fix/11-test-suite

Conversation

@PouyanJay

Copy link
Copy Markdown

Fixes #11.

npm test failed on main, so the repo had no working automated check at all. The suite was still asserting the vinext starter's loading skeleton — a app/_sites-preview/ directory, a react-loading-skeleton dependency and a "Starter Project" title — none of which have existed since Anatomy Atelier was built on the template.

What it asserts now

Six tests against what the built worker actually serves:

  • serves the document — 200, text/html
  • page identity — title, description, og:image, apple touch icon
  • organ library — present, and all nine organs in it
  • the selected organ is server-rendered, not an empty shell — heart's poetic line, system, scientific name, tissue and a clinical condition all come through SSR
  • viewer mount and hotspot text equivalentthree-mount, viewer-tools, and the hotspot-index list. That last one is worth guarding: the hotspot dots live inside the canvas, so this list is the only thing assistive tech can reach.
  • no starter residue — the old check is kept but inverted, failing if any trace of the template reappears

Verification

✔ serves the document
✔ renders the page identity and its social metadata
✔ renders the organ library with every organ
✔ server-renders the selected organ rather than an empty shell
✔ ships the viewer mount and a text equivalent for its hotspots
✔ carries no trace of the vinext starter it was scaffolded from
ℹ pass 6   ℹ fail 0

Full npm test (build + test) passes.

Also verified against #9. Assertions were deliberately chosen to hold on both main and the viewer branch — I merged #9 into a scratch branch and re-ran the suite, 6/6. So these two can merge in either order without conflict.

Should go in before #12, since a dependency bump needs a working test before it can be trusted.

The suite asserted the vinext starter's loading skeleton — a preview
directory, a react-loading-skeleton dependency and a "Starter Project"
title, none of which have existed since Anatomy Atelier was built on top
of the template. So `npm test` failed on main, and the repo had no
working automated check of any kind.

Replaced with assertions against what the worker actually serves: the
page identity and social metadata, the organ library and all nine
organs, the server-rendered content of the default specimen, the viewer
mount, and the screen-reader equivalent of the hotspot dots — that last
one being worth guarding, since the dots themselves live inside a canvas
and the list is the only thing assistive tech can reach.

The starter check is kept, but inverted: the test now fails if any trace
of the template reappears in the output.

Fixes thebuggeddev#11
@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

@PouyanJay is attempting to deploy a commit to the thebuggeddev-9222's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

Test suite is broken on main — asserts a starter skeleton that no longer exists

1 participant