Property localStorage is not defined in Vitest using [email protected]

[ottomated]

Describe the bug
Prior to 14.1.1, global uses of localStorage would work as expected. They no longer do.

To Reproduce
See codesandbox repro

Expected behavior
Global uses of window globals should work as expected.

Device:
All

[capricorn86]

Thank you for reporting @ottomated! 🙂

I'm trying to reproduce the problem, but it seems like it is working in my environment.

I can't access the codesandbox. Is it possible for you to change permission?

[capricorn86]

It seems like it is working here:
https://stackblitz.com/edit/stackblitz-starters-ghue6z?file=index.js

[ottomated]

Updated permissions

[capricorn86]

@ottomated thank you for updating permissions.

There is a fix in now that solves the problem:
https://github.com/capricorn86/happy-dom/releases/tag/v14.1.2

I described the problem and solution in the release notes:

• Fixes problem with properties defined as getters and setters not being registered globally by Vitest
  • The problem occurred as Vitest is using Object.getOwnPropertyDescriptors(window) to read which properties to register globally, but getters and setters are defined on the prototype
  • To solve the problem, GlobalWindow now defines the properties on the instance when it is constructed

[ottomated]

Thanks for the quick fix!