From ed732e223cb2fae6c85c99e415facf3e4d5671e3 Mon Sep 17 00:00:00 2001
From: angrykoala <angrykoala@outlook.es>
Date: Tue, 19 Nov 2024 16:15:34 +0000
Subject: [PATCH] Update apollo subgraph compatibility tests

---
 .../src/type-defs.ts                          | 39 ++++++++++++++++---
 .../validation/validate-document.test.ts      | 24 ------------
 2 files changed, 34 insertions(+), 29 deletions(-)

diff --git a/packages/apollo-federation-subgraph-compatibility/src/type-defs.ts b/packages/apollo-federation-subgraph-compatibility/src/type-defs.ts
index 18d42e97b6..eda62ad31a 100644
--- a/packages/apollo-federation-subgraph-compatibility/src/type-defs.ts
+++ b/packages/apollo-federation-subgraph-compatibility/src/type-defs.ts
@@ -46,9 +46,31 @@ export const typeDefs = gql`
         id: ID!
         sku: String
         package: String
-        variation: ProductVariation @relationship(type: "HAS_VARIATION", direction: OUT)
-        dimensions: ProductDimension @relationship(type: "HAS_DIMENSIONS", direction: OUT)
-        createdBy: User @provides(fields: "totalProductsCreated") @relationship(type: "CREATED_BY", direction: OUT)
+        variation: ProductVariation
+            @cypher(
+                statement: """
+                MATCH (this)-[:HAS_VARIATION]->(res:ProductVariation)
+                RETURN res
+                """
+                columnName: "res"
+            )
+        dimensions: ProductDimension
+            @cypher(
+                statement: """
+                MATCH (this)-[:HAS_DIMENSIONS]->(res:ProductDimension)
+                RETURN res
+                """
+                columnName: "res"
+            )
+        createdBy: User
+            @provides(fields: "totalProductsCreated")
+            @cypher(
+                statement: """
+                MATCH (this)-[:CREATED_BY]->(res:User)
+                RETURN res
+                """
+                columnName: "res"
+            )
         notes: String @tag(name: "internal")
         research: [ProductResearch!]! @relationship(type: "HAS_RESEARCH", direction: OUT)
     }
@@ -57,7 +79,7 @@ export const typeDefs = gql`
         sku: String!
         package: String!
         reason: String
-        createdBy: User @relationship(type: "CREATED_BY", direction: OUT)
+        createdBy: [User!]! @relationship(type: "CREATED_BY", direction: OUT)
     }
 
     type ProductVariation {
@@ -65,7 +87,14 @@ export const typeDefs = gql`
     }
 
     type ProductResearch @key(fields: "study { caseNumber }") {
-        study: CaseStudy! @relationship(type: "HAS_STUDY", direction: OUT)
+        study: CaseStudy!
+            @cypher(
+                statement: """
+                MATCH (this)-[:HAS_STUDY]->(res:CaseStudy)
+                RETURN res
+                """
+                columnName: "res"
+            )
         outcome: String
     }
 
diff --git a/packages/graphql/src/schema/validation/validate-document.test.ts b/packages/graphql/src/schema/validation/validate-document.test.ts
index df20d98d64..3cf17db10f 100644
--- a/packages/graphql/src/schema/validation/validate-document.test.ts
+++ b/packages/graphql/src/schema/validation/validate-document.test.ts
@@ -139,30 +139,6 @@ describe("single relationship deprecation warning", () => {
         warn.mockReset();
     });
 
-    test("deprecated warning triggers on single relationships", () => {
-        const doc = gql`
-            type Movie @node {
-                id: ID
-                topActor: Actor @relationship(type: "TOP_ACTOR", direction: OUT)
-            }
-
-            type Actor @node {
-                name: String
-            }
-        `;
-
-        validateDocument({
-            document: doc,
-            additionalDefinitions,
-            features: {},
-        });
-
-        expect(warn).toHaveBeenCalledWith(
-            "Using @relationship directive on a non-list element is deprecated and will be removed in next major version."
-        );
-        expect(warn).toHaveBeenCalledOnce();
-    });
-
     test("deprecated warning does not trigger on list relationships", () => {
         const doc = gql`
             type Movie @node {