Remove dependency on return order of many-relationships in api-tests (#…

…3281)
keystonejs · Jul 23, 2020 · 50d33dd · 50d33dd
1 parent 5a38498
commit 50d33dd
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 36 deletions.
diff --git a/.changeset/popular-maps-juggle.md b/.changeset/popular-maps-juggle.md
@@ -0,0 +1,5 @@
+---
+'@keystonejs/api-tests': patch
+---
+
+Updated tests to not depend on resolver order.
diff --git a/api-tests/relationships/crud/many-to-many.test.js b/api-tests/relationships/crud/many-to-many.test.js
@@ -322,10 +322,9 @@ multiAdapterRunners().map(({ runner, adapterName }) =>
             linkedCompanies.forEach(({ locations }) => {
               expect(locations.map(({ id }) => id)).toEqual([Location.id.toString()]);
             });
-            expect(linkedCompanies[0].locations[0].companies).toEqual([
-              { id: linkedCompanies[0].id },
-              { id: linkedCompanies[1].id },
-            ]);
+            expect(linkedCompanies[0].locations[0].companies.map(({ id }) => id).sort()).toEqual(
+              [linkedCompanies[0].id, linkedCompanies[1].id].sort()
+            );
           })
         );
 
@@ -371,10 +370,9 @@ multiAdapterRunners().map(({ runner, adapterName }) =>
             allCompanies.forEach(({ locations }) => {
               expect(locations.map(({ id }) => id)).toEqual([Location.id.toString()]);
             });
-            expect(allCompanies[0].locations[0].companies).toEqual([
-              { id: allCompanies[0].id },
-              { id: allCompanies[1].id },
-            ]);
+            expect(allCompanies[0].locations[0].companies.map(({ id }) => id).sort()).toEqual(
+              [allCompanies[0].id, allCompanies[1].id].sort()
+            );
           })
         );
       });

diff --git a/api-tests/relationships/filtering/filtering.test.js b/api-tests/relationships/filtering/filtering.test.js
@@ -155,15 +155,10 @@ multiAdapterRunners().map(({ runner, adapterName }) =>
 
           expect(errors).toBe(undefined);
           expect(data).toHaveProperty('allUsers.0.posts');
+          expect(data.allUsers).toHaveLength(1);
+          expect(data.allUsers[0].id).toEqual(user.id);
           expect(data.allUsers[0].posts).toHaveLength(3);
-          expect(data).toMatchObject({
-            allUsers: [
-              {
-                id: user.id,
-                posts: [{ id: ids[0] }, { id: ids[1] }, { id: ids[2] }],
-              },
-            ],
-          });
+          expect(data.allUsers[0].posts.map(({ id }) => id).sort()).toEqual(ids.sort());
         })
       );
 
@@ -203,15 +198,10 @@ multiAdapterRunners().map(({ runner, adapterName }) =>
 
           expect(errors).toBe(undefined);
           expect(data).toHaveProperty('allUsers.0.posts');
+          expect(data.allUsers).toHaveLength(1);
+          expect(data.allUsers[0].id).toEqual(user.id);
           expect(data.allUsers[0].posts).toHaveLength(3);
-          expect(data).toMatchObject({
-            allUsers: [
-              {
-                id: user.id,
-                posts: [{ id: ids[0] }, { id: ids[1] }, { id: ids[2] }],
-              },
-            ],
-          });
+          expect(data.allUsers[0].posts.map(({ id }) => id).sort()).toEqual(ids.sort());
         })
       );
 

diff --git a/api-tests/relationships/filtering/nested.test.js b/api-tests/relationships/filtering/nested.test.js
@@ -62,19 +62,14 @@ multiAdapterRunners().map(({ runner, adapterName }) =>
 
           expect(errors).toBe(undefined);
           expect(data).toHaveProperty('allUsers.0.posts');
+          expect(data.allUsers).toHaveLength(2);
+          expect(data.allUsers[0].id).toEqual(user.id);
           expect(data.allUsers[0].posts).toHaveLength(2);
-          expect(data).toMatchObject({
-            allUsers: [
-              {
-                id: user.id,
-                posts: [{ id: ids[1] }, { id: ids[2] }],
-              },
-              {
-                id: user2.id,
-                posts: [],
-              },
-            ],
-          });
+          expect(data.allUsers[0].posts.map(({ id }) => id).sort()).toEqual(
+            [ids[1], ids[2]].sort()
+          );
+          expect(data.allUsers[1].id).toEqual(user2.id);
+          expect(data.allUsers[1].posts).toHaveLength(0);
         })
       );