fix hashing import

urql-graphql · Nov 22, 2022 · 71f3754 · 71f3754
1 parent 63f3f39
commit 71f3754
Show file tree

Hide file tree

Showing 16 changed files with 171 additions and 134 deletions.
diff --git a/exchanges/auth/src/authExchange.test.ts b/exchanges/auth/src/authExchange.test.ts
@@ -19,7 +19,7 @@ import {
   Operation,
   OperationResult,
 } from '@urql/core';
-import { queryOperation } from '@urql/core/test-utils';
+import { queryOperation } from '../../../packages/core/src/test-utils';
 
 const makeExchangeArgs = () => {
   const operations: Operation[] = [];

diff --git a/exchanges/context/src/context.test.ts b/exchanges/context/src/context.test.ts
@@ -75,7 +75,7 @@ it(`calls getContext`, () => {
   expect(result).toHaveBeenCalledTimes(1);
 });
 
-it(`calls getContext async`, done => {
+it(`calls getContext async`, async () => {
   const response = vi.fn(
     (forwardOp: Operation): OperationResult => {
       return {
@@ -108,10 +108,12 @@ it(`calls getContext async`, done => {
 
   next(op);
 
-  setTimeout(() => {
-    expect(response).toHaveBeenCalledTimes(1);
-    expect(response.mock.calls[0][0].context.headers).toEqual(headers);
-    expect(result).toHaveBeenCalledTimes(1);
-    done();
-  }, 10);
+  await new Promise(res => {
+    setTimeout(() => {
+      expect(response).toHaveBeenCalledTimes(1);
+      expect(response.mock.calls[0][0].context.headers).toEqual(headers);
+      expect(result).toHaveBeenCalledTimes(1);
+      res();
+    }, 10);
+  });
 });
diff --git a/exchanges/execute/src/execute.test.ts b/exchanges/execute/src/execute.test.ts
@@ -26,7 +26,7 @@ import {
   context,
   queryOperation,
   subscriptionOperation,
-} from '@urql/core/src/test-utils';
+} from '../../../packages/core/src/test-utils';
 import {
   makeErrorResult,
   makeOperation,

diff --git a/exchanges/request-policy/src/requestPolicyExchange.test.ts b/exchanges/request-policy/src/requestPolicyExchange.test.ts
@@ -45,7 +45,7 @@ beforeEach(() => {
   ({ source: ops$, next } = makeSubject<Operation>());
 });
 
-it(`upgrades to cache-and-network`, done => {
+it(`upgrades to cache-and-network`, async () => {
   const response = vi.fn(
     (forwardOp: Operation): OperationResult => {
       return {
@@ -78,18 +78,20 @@ it(`upgrades to cache-and-network`, done => {
   );
   expect(result).toHaveBeenCalledTimes(1);
 
-  setTimeout(() => {
-    next(op);
-    expect(response).toHaveBeenCalledTimes(2);
-    expect(response.mock.calls[1][0].context.requestPolicy).toEqual(
-      'cache-and-network'
-    );
-    expect(result).toHaveBeenCalledTimes(2);
-    done();
-  }, 10);
+  await new Promise(res => {
+    setTimeout(() => {
+      next(op);
+      expect(response).toHaveBeenCalledTimes(2);
+      expect(response.mock.calls[1][0].context.requestPolicy).toEqual(
+        'cache-and-network'
+      );
+      expect(result).toHaveBeenCalledTimes(2);
+      res();
+    }, 10);
+  });
 });
 
-it(`doesn't upgrade when shouldUpgrade returns false`, done => {
+it(`doesn't upgrade when shouldUpgrade returns false`, async () => {
   const response = vi.fn(
     (forwardOp: Operation): OperationResult => {
       return {
@@ -123,14 +125,16 @@ it(`doesn't upgrade when shouldUpgrade returns false`, done => {
   );
   expect(result).toHaveBeenCalledTimes(1);
 
-  setTimeout(() => {
-    next(op);
-    expect(response).toHaveBeenCalledTimes(2);
-    expect(response.mock.calls[1][0].context.requestPolicy).toEqual(
-      'cache-first'
-    );
-    expect(result).toHaveBeenCalledTimes(2);
-    expect(shouldUpgrade).toBeCalledTimes(2);
-    done();
-  }, 10);
+  await new Promise(res => {
+    setTimeout(() => {
+      next(op);
+      expect(response).toHaveBeenCalledTimes(2);
+      expect(response.mock.calls[1][0].context.requestPolicy).toEqual(
+        'cache-first'
+      );
+      expect(result).toHaveBeenCalledTimes(2);
+      expect(shouldUpgrade).toBeCalledTimes(2);
+      res();
+    }, 10);
+  });
 });
diff --git a/packages/core/src/utils/request.test.ts b/packages/core/src/utils/request.test.ts
@@ -1,7 +1,10 @@
-vi.mock('./hash', async () => ({
-  hash: await vi.importActual('./hash').hash,
-  phash: (x: number) => x,
-}));
+vi.mock('./hash', async () => {
+  const hash = await vi.importActual('./hash');
+  return {
+    ...hash,
+    phash: (x: number) => x,
+  };
+});
 
 import { parse, print } from 'graphql';
 import { gql } from '../gql';

diff --git a/packages/preact-urql/src/components/Mutation.test.tsx b/packages/preact-urql/src/components/Mutation.test.tsx
@@ -21,7 +21,7 @@ describe('Mutation', () => {
     cleanup();
   });
 
-  it('Should execute the mutation', done => {
+  it('Should execute the mutation', async () => {
     // eslint-disable-next-line
     let execute = () => {},
       props = {};
@@ -60,9 +60,12 @@ describe('Mutation', () => {
       fetching: true,
       error: undefined,
     });
-    setTimeout(() => {
-      expect(props).toStrictEqual({ data: 1, fetching: false, error: 2 });
-      done();
-    }, 400);
+
+    await new Promise(res => {
+      setTimeout(() => {
+        expect(props).toStrictEqual({ data: 1, fetching: false, error: 2 });
+        res();
+      }, 400);
+    });
   });
 });
diff --git a/packages/preact-urql/src/components/Query.test.tsx b/packages/preact-urql/src/components/Query.test.tsx
@@ -27,7 +27,7 @@ describe('Query', () => {
     cleanup();
   });
 
-  it('Should execute the query', done => {
+  it('Should execute the query', async () => {
     let props = {};
     const Test = () => h('p', {}, 'hi');
     const App = () => {
@@ -50,9 +50,12 @@ describe('Query', () => {
       fetching: true,
       error: undefined,
     });
-    setTimeout(() => {
-      expect(props).toStrictEqual({ data: 0, fetching: false, error: 1 });
-      done();
-    }, 250);
+
+    await new Promise(res => {
+      setTimeout(() => {
+        expect(props).toStrictEqual({ data: 0, fetching: false, error: 1 });
+        res();
+      }, 250);
+    });
   });
 });
diff --git a/packages/preact-urql/src/components/Subscription.test.tsx b/packages/preact-urql/src/components/Subscription.test.tsx
@@ -23,7 +23,7 @@ describe('Subscription', () => {
     cleanup();
   });
 
-  it('Should execute the subscription', done => {
+  it('Should execute the subscription', async () => {
     let props = {};
     const Test = () => h('p', {}, 'hi');
     const App = () => {
@@ -46,9 +46,12 @@ describe('Subscription', () => {
       fetching: true,
       error: undefined,
     });
-    setTimeout(() => {
-      expect(props).toStrictEqual({ data: 0, fetching: true, error: 1 });
-      done();
-    }, 300);
+
+    await new Promise(res => {
+      setTimeout(() => {
+        expect(props).toStrictEqual({ data: 0, fetching: true, error: 1 });
+        res();
+      }, 300);
+    });
   });
 });
diff --git a/packages/preact-urql/src/hooks/useQuery.test.tsx b/packages/preact-urql/src/hooks/useQuery.test.tsx
@@ -95,7 +95,7 @@ describe('useQuery', () => {
     expect(state).toHaveProperty('fetching', true);
   });
 
-  it('forwards data response', done => {
+  it('forwards data response', async () => {
     const { rerender } = render(
       h(Provider, {
         value: client as any,
@@ -110,19 +110,21 @@ describe('useQuery', () => {
       })
     );
 
-    setTimeout(() => {
-      rerender(
-        h(Provider, {
-          value: client as any,
-          children: [h(QueryUser, { ...props })],
-        })
-      );
-      expect(state).toHaveProperty('data', 0);
-      done();
-    }, 400);
+    await new Promise(res => {
+      setTimeout(() => {
+        rerender(
+          h(Provider, {
+            value: client as any,
+            children: [h(QueryUser, { ...props })],
+          })
+        );
+        expect(state).toHaveProperty('data', 0);
+        res();
+      }, 400);
+    });
   });
 
-  it('forwards error response', done => {
+  it('forwards error response', async () => {
     const { rerender } = render(
       h(Provider, {
         value: client as any,
@@ -137,19 +139,21 @@ describe('useQuery', () => {
       })
     );
 
-    setTimeout(() => {
-      rerender(
-        h(Provider, {
-          value: client as any,
-          children: [h(QueryUser, { ...props })],
-        })
-      );
-      expect(state).toHaveProperty('error', 1);
-      done();
-    }, 400);
+    await new Promise(res => {
+      setTimeout(() => {
+        rerender(
+          h(Provider, {
+            value: client as any,
+            children: [h(QueryUser, { ...props })],
+          })
+        );
+        expect(state).toHaveProperty('error', 1);
+        res();
+      }, 400);
+    });
   });
 
-  it('forwards extensions response', done => {
+  it('forwards extensions response', async () => {
     const { rerender } = render(
       h(Provider, {
         value: client as any,
@@ -167,20 +171,22 @@ describe('useQuery', () => {
       })
     );
 
-    setTimeout(() => {
-      rerender(
-        h(Provider, {
-          value: client as any,
-          children: [h(QueryUser, { ...props })],
-        })
-      );
+    await new Promise(res => {
+      setTimeout(() => {
+        rerender(
+          h(Provider, {
+            value: client as any,
+            children: [h(QueryUser, { ...props })],
+          })
+        );
 
-      expect(state).toHaveProperty('extensions', { i: 1 });
-      done();
-    }, 400);
+        expect(state).toHaveProperty('extensions', { i: 1 });
+        res();
+      }, 400);
+    });
   });
 
-  it('sets fetching to false', done => {
+  it('sets fetching to false', async () => {
     const { rerender } = render(
       h(Provider, {
         value: client as any,
@@ -195,16 +201,18 @@ describe('useQuery', () => {
       })
     );
 
-    setTimeout(() => {
-      rerender(
-        h(Provider, {
-          value: client as any,
-          children: [h(QueryUser, { ...props })],
-        })
-      );
-      expect(state).toHaveProperty('fetching', false);
-      done();
-    }, 400);
+    await new Promise(res => {
+      setTimeout(() => {
+        rerender(
+          h(Provider, {
+            value: client as any,
+            children: [h(QueryUser, { ...props })],
+          })
+        );
+        expect(state).toHaveProperty('fetching', false);
+        res();
+      }, 400);
+    });
   });
 
   describe('on change', () => {

diff --git a/packages/react-urql/src/components/Mutation.test.tsx b/packages/react-urql/src/components/Mutation.test.tsx
@@ -32,7 +32,7 @@ describe('Mutation', () => {
     cleanup();
   });
 
-  it('Should execute the mutation', done => {
+  it('Should execute the mutation', async () => {
     let execute = () => {
         /* noop */
       },
@@ -65,9 +65,11 @@ describe('Mutation', () => {
       fetching: true,
       error: undefined,
     });
-    setTimeout(() => {
-      expect(props).toStrictEqual({ data: 1, fetching: false, error: 2 });
-      done();
-    }, 400);
+    await new Promise(res => {
+      setTimeout(() => {
+        expect(props).toStrictEqual({ data: 1, fetching: false, error: 2 });
+        res();
+      }, 400);
+    });
   });
 });
diff --git a/packages/react-urql/src/components/Query.test.tsx b/packages/react-urql/src/components/Query.test.tsx
@@ -34,7 +34,7 @@ describe('Query', () => {
     cleanup();
   });
 
-  it('Should execute the query', done => {
+  it('Should execute the query', async () => {
     let props = {};
     const Test = () => <p>Hi</p>;
     const App = () => {
@@ -54,9 +54,11 @@ describe('Query', () => {
       fetching: true,
       error: undefined,
     });
-    setTimeout(() => {
-      expect(props).toStrictEqual({ data: 0, fetching: false, error: 1 });
-      done();
-    }, 200);
+    await new Promise(res => {
+      setTimeout(() => {
+        expect(props).toStrictEqual({ data: 0, fetching: false, error: 1 });
+        res();
+      }, 200);
+    });
   });
 });