Skip to content

fix(testing): use document browser storage - #165

Merged
KyleJune merged 1 commit into
mainfrom
codex/jsdom-storage-isolation
Sep 27, 2026
Merged

KyleJune merged 1 commit into
mainfrom
codex/jsdom-storage-isolation

Conversation

@KyleJune

Copy link
Copy Markdown
Member

Summary

Browser tests now use the current JSDOM document's local and session storage. Deno's own storage globals previously survived the fixture setup and could throw or persist state outside the document.

Changes

  • Bind both global storage getters to the active document.
  • Verify global identity and writes through both storage APIs.

Testing

deno task check and deno task test --parallel --reporter=dot passed: 60 tests, 852 steps.

Closes

Nothing.

@KyleJune
KyleJune merged commit d752f6b into main Sep 27, 2026
10 checks passed
@KyleJune
KyleJune deleted the codex/jsdom-storage-isolation branch September 27, 2026 02:28
KyleJune pushed a commit that referenced this pull request Sep 27, 2026
## [0.17.2](0.17.1...0.17.2) (2026-09-27)

### Bug Fixes

* **testing:** use document browser storage ([#165](#165)) ([d752f6b](d752f6b))
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 0.17.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant