ref(spread): rewrote tests to not modify existing ones

effector · Sep 27, 2024 · 90ec10c · 90ec10c
1 parent 7fc0e14
commit 90ec10c
Show file tree

Hide file tree

Showing 2 changed files with 395 additions and 77 deletions.
diff --git a/src/spread/spread.fork.test.ts b/src/spread/spread.fork.test.ts
@@ -4,84 +4,64 @@ import { spread } from './index';
 
 test('works in forked scope', async () => {
   const app = createDomain();
-  const source = app.createEvent<{ first: string; second: number; third: string }>();
+  const source = app.createEvent<{ first: string; second: number }>();
   const first = app.createEvent<string>();
   const second = app.createEvent<number>();
 
-  const $thirdA = app.createStore('');
-  const $thirdB = app.createStore('');
-
-  const $first = app.createStore('').on(first, (_, p) => p);
-  const $second = restore(second, 0);
+  const _$first = app.createStore('').on(first, (_, p) => p);
+  const _$second = restore(second, 0);
 
   spread({
     source,
-    targets: { first, second, third: [$thirdA, $thirdB] },
+    targets: { first, second },
   });
 
   const scope = fork();
 
-  await allSettled(source, {
-    scope,
-    params: { first: 'sergey', second: 26, third: '30' },
-  });
-
-  expect(scope.getState($first)).toBe('sergey');
-  expect(scope.getState($second)).toBe(26);
-  expect(scope.getState($thirdA)).toBe('30');
-  expect(scope.getState($thirdB)).toBe('30');
+  await allSettled(source, { scope, params: { first: 'sergey', second: 26 } });
+  expect(serialize(scope)).toMatchInlineSnapshot(`
+    {
+      "-xihhjw": 26,
+      "nln5hw": "sergey",
+    }
+  `);
 });
 
 test('do not affects original store state', async () => {
   const app = createDomain();
-  const source = app.createEvent<{ first: string; second: number; third: string }>();
+  const source = app.createEvent<{ first: string; second: number }>();
   const first = app.createEvent<string>();
   const second = app.createEvent<number>();
 
-  const $thirdA = app.createStore('');
-  const $thirdB = app.createStore('');
-
   const $first = app.createStore('').on(first, (_, p) => p);
   const $second = restore(second, 0);
 
   spread({
     source,
-    targets: { first, second, third: [$thirdA, $thirdB] },
+    targets: { first, second },
   });
 
   const scope = fork();
 
-  await allSettled(source, {
-    scope,
-    params: { first: 'sergey', second: 26, third: '30' },
-  });
-
+  await allSettled(source, { scope, params: { first: 'sergey', second: 26 } });
   expect(scope.getState($first)).toBe('sergey');
   expect(scope.getState($second)).toBe(26);
-  expect(scope.getState($thirdA)).toBe('30');
-  expect(scope.getState($thirdB)).toBe('30');
-
   expect($first.getState()).toBe('');
   expect($second.getState()).toBe(0);
-  expect($thirdA.getState()).toBe('');
-  expect($thirdB.getState()).toBe('');
 });
 
 test('do not affects another scope', async () => {
   const app = createDomain();
-  const source = app.createEvent<{ first: string; second: number; third: string }>();
+  const source = app.createEvent<{ first: string; second: number }>();
   const first = app.createEvent<string>();
   const second = app.createEvent<number>();
 
-  const $thirdA = app.createStore('');
-  const $thirdB = app.createStore('');
-
-  const $first = app.createStore('').on(first, (_, p) => p);
-  const $second = restore(second, 0);
+  const _$first = app.createStore('').on(first, (_, p) => p);
+  const _$second = restore(second, 0);
 
   spread({
     source,
-    targets: { first, second, third: [$thirdA, $thirdB] },
+    targets: { first, second },
   });
 
   const scope1 = fork();
@@ -90,21 +70,137 @@ test('do not affects another scope', async () => {
   await Promise.all([
     allSettled(source, {
       scope: scope1,
-      params: { first: 'sergey', second: 26, third: '30' },
+      params: { first: 'sergey', second: 26 },
     }),
     allSettled(source, {
       scope: scope2,
-      params: { first: 'Anon', second: 90, third: '154' },
+      params: { first: 'Anon', second: 90 },
     }),
   ]);
+  expect(serialize(scope1)).toMatchInlineSnapshot(`
+    {
+      "-w3pd79": 26,
+      "f2h7kg": "sergey",
+    }
+  `);
+  expect(serialize(scope2)).toMatchInlineSnapshot(`
+    {
+      "-w3pd79": 90,
+      "f2h7kg": "Anon",
+    }
+  `);
+});
+
+describe('targets: array of units', () => {
+  test('works in forked scope', async () => {
+    const app = createDomain();
+    const source = app.createEvent<{
+      first: string;
+      second: number;
+      third: string;
+    }>();
+    const first = app.createEvent<string>();
+    const second = app.createEvent<number>();
+
+    const $thirdA = app.createStore('');
+    const $thirdB = app.createStore('');
 
-  expect(scope1.getState($first)).toBe('sergey');
-  expect(scope1.getState($second)).toBe(26);
-  expect(scope1.getState($thirdA)).toBe('30');
-  expect(scope1.getState($thirdB)).toBe('30');
+    const $first = app.createStore('').on(first, (_, p) => p);
+    const $second = restore(second, 0);
 
-  expect(scope2.getState($first)).toBe('Anon');
-  expect(scope2.getState($second)).toBe(90);
-  expect(scope2.getState($thirdA)).toBe('154');
-  expect(scope2.getState($thirdB)).toBe('154');
+    spread({
+      source,
+      targets: { first, second, third: [$thirdA, $thirdB] },
+    });
+
+    const scope = fork();
+
+    await allSettled(source, {
+      scope,
+      params: { first: 'sergey', second: 26, third: '30' },
+    });
+
+    expect(scope.getState($first)).toBe('sergey');
+    expect(scope.getState($second)).toBe(26);
+    expect(scope.getState($thirdA)).toBe('30');
+    expect(scope.getState($thirdB)).toBe('30');
+  });
+
+  test('does not affect original store state', async () => {
+    const app = createDomain();
+    const source = app.createEvent<{
+      first: string;
+      second: number;
+      third: string;
+    }>();
+    const first = app.createEvent<string>();
+    const second = app.createEvent<number>();
+
+    const $thirdA = app.createStore('');
+    const $thirdB = app.createStore('');
+
+    const $first = app.createStore('').on(first, (_, p) => p);
+    const $second = restore(second, 0);
+
+    spread({
+      source,
+      targets: { first, second, third: [$thirdA, $thirdB] },
+    });
+
+    const scope = fork();
+
+    await allSettled(source, {
+      scope,
+      params: { first: 'sergey', second: 26, third: '30' },
+    });
+
+    expect(scope.getState($first)).toBe('sergey');
+    expect(scope.getState($second)).toBe(26);
+    expect(scope.getState($thirdA)).toBe('30');
+    expect(scope.getState($thirdB)).toBe('30');
+
+    expect($first.getState()).toBe('');
+    expect($second.getState()).toBe(0);
+    expect($thirdA.getState()).toBe('');
+    expect($thirdB.getState()).toBe('');
+  });
+
+  test('do not affects another scope', async () => {
+    const app = createDomain();
+    const source = app.createEvent<{
+      first: string;
+      second: number;
+      third: string;
+    }>();
+    const first = app.createEvent<string>();
+    const second = app.createEvent<number>();
+
+    const $thirdA = app.createStore('');
+    const $thirdB = app.createStore('');
+
+    const $first = app.createStore('').on(first, (_, p) => p);
+    const $second = restore(second, 0);
+
+    spread({
+      source,
+      targets: { first, second, third: [$thirdA, $thirdB] },
+    });
+
+    const scope1 = fork();
+    const scope2 = fork();
+
+    await Promise.all([
+      allSettled(source, {
+        scope: scope1,
+        params: { first: 'sergey', second: 26, third: '30' },
+      }),
+      allSettled(source, {
+        scope: scope2,
+        params: { first: 'Anon', second: 90, third: '154' },
+      }),
+    ]);
+
+    expect(scope1.getState($first)).toBe('sergey');
+    expect(scope1.getState($second)).toBe(26);
+  });
 });