feat!: 删除 engines.localStorage 和 engines.sessionStorage 两个属性

foca-js · Feb 26, 2024 · 298ea1d · 298ea1d
1 parent 3cce1ab
commit 298ea1d
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 88 deletions.
diff --git a/src/engines/index.ts b/src/engines/index.ts
diff --git a/src/engines/local.ts b/src/engines/local.ts
diff --git a/src/engines/session.ts b/src/engines/session.ts
diff --git a/src/index.ts b/src/index.ts
@@ -14,7 +14,6 @@ export { connect } from './redux/connect';
 export { compose } from 'redux';
 export { modelStore as store } from './store/model-store';
 export { FocaProvider } from './redux/foca-provider';
-export { engines } from './engines';
 export { memoryStorage } from './engines/memory';
 
 // 可能用到的TS类型

diff --git a/test/engine.test.ts b/test/engine.test.ts
@@ -1,44 +1,34 @@
 import 'fake-indexeddb/auto';
 import localforage from 'localforage';
 import ReactNativeStorage from '@react-native-async-storage/async-storage';
-import { engines } from '../src';
 import { toPromise } from '../src/utils/to-promise';
+import { memoryStorage } from '../src';
 
 const storages = [
-  [engines.localStorage, localStorage, 'local'],
-  [engines.sessionStorage, sessionStorage, 'session'],
-  [engines.memoryStorage, undefined, 'memory'],
+  [localStorage, 'local'],
+  [sessionStorage, 'session'],
+  [memoryStorage, 'memory'],
   [
     localforage.createInstance({ driver: localforage.LOCALSTORAGE }),
-    undefined,
     'localforage local',
   ],
   [
     localforage.createInstance({ driver: localforage.INDEXEDDB }),
-    undefined,
     'localforage indexedDb',
   ],
-  [ReactNativeStorage, undefined, 'react-native'],
+  [ReactNativeStorage, 'react-native'],
 ] as const;
 
-describe.each(storages)('storage io', (storage, syncOrigin, name) => {
+describe.each(storages)('storage io', (storage, name) => {
   beforeEach(() => storage.clear());
   afterEach(() => storage.clear());
 
   test(`[${name}] Get and set data`, async () => {
-    if (syncOrigin) {
-      expect(syncOrigin.getItem('test1')).toBeNull();
-    }
-
     await expect(toPromise(() => storage.getItem('test1'))).resolves.toBeNull();
     await storage.setItem('test1', 'yes');
     await expect(toPromise(() => storage.getItem('test1'))).resolves.toBe(
       'yes',
     );
-
-    if (syncOrigin) {
-      expect(syncOrigin.getItem('test1')).toBe('yes');
-    }
   });
 
   test(`[${name}] Update data`, async () => {
@@ -48,10 +38,6 @@ describe.each(storages)('storage io', (storage, syncOrigin, name) => {
     );
     await storage.setItem('test2', 'no');
     await expect(toPromise(() => storage.getItem('test2'))).resolves.toBe('no');
-
-    if (syncOrigin) {
-      expect(syncOrigin.getItem('test2')).toBe('no');
-    }
   });
 
   test(`[${name}] Delete data`, async () => {
@@ -61,10 +47,6 @@ describe.each(storages)('storage io', (storage, syncOrigin, name) => {
     );
     await storage.removeItem('test3');
     await expect(toPromise(() => storage.getItem('test3'))).resolves.toBeNull();
-
-    if (syncOrigin) {
-      expect(syncOrigin.getItem('test3')).toBeNull();
-    }
   });
 
   test(`[${name}] Clear all data`, async () => {
@@ -75,10 +57,5 @@ describe.each(storages)('storage io', (storage, syncOrigin, name) => {
 
     await expect(toPromise(() => storage.getItem('test4'))).resolves.toBeNull();
     await expect(toPromise(() => storage.getItem('test5'))).resolves.toBeNull();
-
-    if (syncOrigin) {
-      expect(syncOrigin.getItem('test4')).toBeNull();
-      expect(syncOrigin.getItem('test5')).toBeNull();
-    }
   });
 });