Add assertHttpRequest to test

Shopify · Feb 9, 2021 · db71552 · db71552
1 parent 245d208
commit db71552
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 10 deletions.
diff --git a/src/clients/graphql/test/graphql_client.test.ts b/src/clients/graphql/test/graphql_client.test.ts
@@ -83,6 +83,14 @@ describe('GraphQL client', () => {
     fetchMock.mockResponseOnce(JSON.stringify(expectedResponse));
 
     await expect(client.query({data: queryWithVariables})).resolves.toEqual(buildExpectedResponse(expectedResponse));
+
+    assertHttpRequest(
+      'POST',
+      DOMAIN,
+      '/admin/api/unstable/graphql.json',
+      {'Content-Length': 219, 'Content-Type': 'application/json', 'X-Shopify-Access-Token': 'bork'},
+      JSON.stringify(queryWithVariables),
+    );
   });
 });
 

diff --git a/src/clients/http_client/test/test_helper.ts b/src/clients/http_client/test/test_helper.ts
@@ -16,15 +16,13 @@ export function assertHttpRequest(
   const maxCall = currentCall + tries;
   for (let i = currentCall; i < maxCall; i++) {
     currentCall++;
-    expect(fetchMock.mock.calls[i]).toEqual(
-      [
-        `https://${domain}${path}`,
-        expect.objectContaining({
-          method,
-          headers: expect.objectContaining(headers),
-          body: data,
-        }),
-      ],
-    );
+    expect(fetchMock.mock.calls[i]).toEqual([
+      `https://${domain}${path}`,
+      expect.objectContaining({
+        method,
+        headers: expect.objectContaining(headers),
+        body: data,
+      }),
+    ]);
   }
 }