test: add test for getStringStorage method from createJSONStorage to not be called in server

Author: mhsattarian

tests/react/vanilla-utils/atomWithStorage.test.tsx

@@ -9,7 +9,10 @@ import {
   createJSONStorage,
   unstable_withStorageValidator as withStorageValidator,
 } from 'jotai/vanilla/utils'
-import type { SyncStringStorage } from 'jotai/vanilla/utils/atomWithStorage'
+import type {
+  AsyncStringStorage,
+  SyncStringStorage,
+} from 'jotai/vanilla/utils/atomWithStorage'
 
 const resolve: (() => void)[] = []
 
@@ -337,7 +340,7 @@ describe('atomWithStorage (in non-browser environment)', () => {
   const asyncStorageData: Record<string, string> = {
     count: '10',
   }
-  const asyncDummyStorage = {
+  const asyncDummyStorage: AsyncStringStorage = {
     getItem: async (key: string) => {
       await new Promise<void>((r) => resolve.push(r))
       return asyncStorageData[key] as string
@@ -367,6 +370,13 @@ describe('atomWithStorage (in non-browser environment)', () => {
 
     expect(storage.subscribe).toBeUndefined()
   })
+
+  it('createJSONStorage with custom subscribe', async () => {
+    const subscriber = vi.fn()
+    createJSONStorage(subscriber)
+
+    expect(subscriber).not.toBeCalled()
+  })
 })
 
 describe('atomWithStorage (with browser storage)', () => {