Merge pull request #5738 from neo4j/deprecate-unique

deprecation warning for unique

.changeset/brave-chicken-hunt.md

@@ -0,0 +1,5 @@
+---
+"@neo4j/graphql": patch
+---
+
+Add deprecation warning when using `@unique`

packages/graphql/src/schema/validation/custom-rules/warnings/deprecated-unique.ts

@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) "Neo4j"
+ * Neo4j Sweden AB [http://neo4j.com]
+ *
+ * This file is part of Neo4j.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import type { ASTVisitor, ObjectTypeDefinitionNode } from "graphql";
+import { uniqueDirective } from "../../../../graphql/directives";
+
+export function WarnUniqueDeprecation() {
+    return function (): ASTVisitor {
+        let warningAlreadyIssued = false;
+
+        return {
+            ObjectTypeDefinition(objectTypeDefinition: ObjectTypeDefinitionNode) {
+                if (["Query", "Mutation", "Subscription"].includes(objectTypeDefinition.name.value)) {
+                    return;
+                }
+                if (warningAlreadyIssued) {
+                    return;
+                }
+                let hasUniqueDirective = false;
+                for (const field of objectTypeDefinition.fields || []) {
+                    for (const directive of field.directives ?? []) {
+                        if (directive.name.value === uniqueDirective.name) {
+                            hasUniqueDirective = true;
+                        }
+                    }
+                }
+
+                if (hasUniqueDirective) {
+                    console.warn(`Future library versions will not support @unique directive.`);
+                    warningAlreadyIssued = true;
+                }
+            },
+        };
+    };
+}

packages/graphql/src/schema/validation/validate-document.ts

@@ -62,6 +62,7 @@ import {
 import { ValidFieldTypes } from "./custom-rules/valid-types/valid-field-types";
 import { ValidObjectType } from "./custom-rules/valid-types/valid-object-type";
 import { WarnIfAuthorizationFeatureDisabled } from "./custom-rules/warnings/authorization-feature-disabled";
+import { WarnUniqueDeprecation } from "./custom-rules/warnings/deprecated-unique";
 import { WarnIfAMaxLimitCanBeBypassedThroughInterface } from "./custom-rules/warnings/limit-max-can-be-bypassed";
 import { WarnIfListOfListsFieldDefinition } from "./custom-rules/warnings/list-of-lists";
 import { WarnObjectFieldsWithoutResolver } from "./custom-rules/warnings/object-fields-without-resolver";
@@ -233,6 +234,7 @@ function runValidationRulesOnFilteredDocument({
             WarnIfSubscriptionsAuthorizationMissing(Boolean(features?.subscriptions)),
             WarnIfTypeIsNotMarkedAsNode(),
             WarnIfQueryDirectionIsUsedWithDeprecatedValues,
+            WarnUniqueDeprecation(),
         ],
         schema
     );

packages/graphql/tests/integration/migration-warnings/unique-directive.test.ts

@@ -0,0 +1,52 @@
+/*
+ * Copyright (c) "Neo4j"
+ * Neo4j Sweden AB [http://neo4j.com]
+ *
+ * This file is part of Neo4j.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { Neo4jGraphQL } from "../../../src/classes";
+
+describe("deprecated @unique warnings", () => {
+    let warn: jest.SpyInstance;
+
+    beforeEach(() => {
+        warn = jest.spyOn(console, "warn").mockImplementation(() => {});
+    });
+
+    afterEach(() => {
+        warn.mockReset();
+    });
+
+    test("warning on unique usage", async () => {
+        const typeDefs = /* GraphQL */ `
+            type User @node {
+                id: ID! @unique
+                firstName: String!
+            }
+
+            type Movie @node {
+                id: ID! @unique
+            }
+        `;
+
+        const neoSchema = new Neo4jGraphQL({
+            typeDefs: typeDefs,
+            validate: true,
+        });
+        await neoSchema.getSchema();
+        expect(warn).toHaveBeenCalledWith("Future library versions will not support @unique directive.");
+    });
+});