Skip to content

test: assert live-book theme injects #enscribe-doc-theme - #473

Open
Devil1716 wants to merge 1 commit into
enscribejs:mainfrom
Devil1716:fix/471-theme-test-coverage
Open

test: assert live-book theme injects #enscribe-doc-theme#473
Devil1716 wants to merge 1 commit into
enscribejs:mainfrom
Devil1716:fix/471-theme-test-coverage

Conversation

@Devil1716

Copy link
Copy Markdown

Summary

  • Fix the stale claim that live-book-parity.test.js covers the live-book applyDocumentTheme path
  • Add a jsdom mountLiveShell case that mounts a config theme=tufte book and asserts #enscribe-doc-theme is injected with Tufte tokens

Fixes #471

Test plan

theme-consumption.test.js claimed live-book-parity covered applyDocumentTheme, but that file has no theme assertions. Mount a themed book via jsdom and pin the head style. Fixes enscribejs#471.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

theme-consumption.test.js claims live-book theme coverage that does not exist

1 participant