fix: issue with localStorage in jsdom

[Demivan]

No description provided.

[sheremet-va]

I think it's better to create Window ourselves. Otherwise we depend on test framework implementation

We do this with happy-dom

https://github.com/Aslemammad/tinyspy/blob/c8446464e18fb6a7298a575fe8e7252f757e8ecd/test/index.test.ts#L546

[Demivan]

I'm still trying to figure out what is causing this. It is only happening with jsdom and not happy-dom.
I'll probably not use localStorage at all when I figure out the cause.

[sheremet-va]

I'm still trying to figure out what is causing this. It is only happening with jsdom and not happy-dom. I'll probably not use localStorage at all when I figure out the cause.

This aproach should help narrow it down, since it is still possible the error might be in Vitest pipeline

[Demivan]

jsdom/jsdom#2318 - you cannot actually call defineProperty to change implementation on a localStorage.

You need to replace entire localStorage implementation if you need to mock it: jestjs/jest#6798 (comment)

[sheremet-va]

jest.spyOn(window.localStorage.__proto__, 'setItem'); should work without any additional imports.

Hehe

[Demivan]

Yeah, maybe just need to document it somewhere