remove a few extra quotes for consistency of the changes

graphql · Mar 20, 2024 · d2275f2 · d2275f2
1 parent 80f3a1c
commit d2275f2
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 14 deletions.
diff --git a/src/type/__tests__/introspection-test.ts b/src/type/__tests__/introspection-test.ts
@@ -1644,7 +1644,7 @@ describe('Introspection', () => {
       errors: [
         {
           message:
-            'Argument <meta>.__type(name:) of type "String!" is required, but it was not provided.',
+            'Argument <meta>.__type(name:) of type String! is required, but it was not provided.',
           locations: [{ line: 3, column: 9 }],
         },
       ],

diff --git a/src/validation/__tests__/ProvidedRequiredArgumentsRule-test.ts b/src/validation/__tests__/ProvidedRequiredArgumentsRule-test.ts
@@ -168,7 +168,7 @@ describe('Validate: Provided required arguments', () => {
       `).toDeepEqual([
         {
           message:
-            'Argument ComplicatedArgs.multipleReqs(req1:) of type "Int!" is required, but it was not provided.',
+            'Argument ComplicatedArgs.multipleReqs(req1:) of type Int! is required, but it was not provided.',
           locations: [{ line: 4, column: 13 }],
         },
       ]);
@@ -184,12 +184,12 @@ describe('Validate: Provided required arguments', () => {
       `).toDeepEqual([
         {
           message:
-            'Argument ComplicatedArgs.multipleReqs(req1:) of type "Int!" is required, but it was not provided.',
+            'Argument ComplicatedArgs.multipleReqs(req1:) of type Int! is required, but it was not provided.',
           locations: [{ line: 4, column: 13 }],
         },
         {
           message:
-            'Argument ComplicatedArgs.multipleReqs(req2:) of type "Int!" is required, but it was not provided.',
+            'Argument ComplicatedArgs.multipleReqs(req2:) of type Int! is required, but it was not provided.',
           locations: [{ line: 4, column: 13 }],
         },
       ]);
@@ -205,7 +205,7 @@ describe('Validate: Provided required arguments', () => {
       `).toDeepEqual([
         {
           message:
-            'Argument ComplicatedArgs.multipleReqs(req2:) of type "Int!" is required, but it was not provided.',
+            'Argument ComplicatedArgs.multipleReqs(req2:) of type Int! is required, but it was not provided.',
           locations: [{ line: 4, column: 13 }],
         },
       ]);
@@ -244,12 +244,12 @@ describe('Validate: Provided required arguments', () => {
       `).toDeepEqual([
         {
           message:
-            'Argument @include(if:) of type "Boolean!" is required, but it was not provided.',
+            'Argument @include(if:) of type Boolean! is required, but it was not provided.',
           locations: [{ line: 3, column: 15 }],
         },
         {
           message:
-            'Argument @skip(if:) of type "Boolean!" is required, but it was not provided.',
+            'Argument @skip(if:) of type Boolean! is required, but it was not provided.',
           locations: [{ line: 4, column: 18 }],
         },
       ]);
@@ -277,7 +277,7 @@ describe('Validate: Provided required arguments', () => {
       `).toDeepEqual([
         {
           message:
-            'Argument @test(arg:) of type "String!" is required, but it was not provided.',
+            'Argument @test(arg:) of type String! is required, but it was not provided.',
           locations: [{ line: 3, column: 23 }],
         },
       ]);
@@ -291,7 +291,7 @@ describe('Validate: Provided required arguments', () => {
       `).toDeepEqual([
         {
           message:
-            'Argument @include(if:) of type "Boolean!" is required, but it was not provided.',
+            'Argument @include(if:) of type Boolean! is required, but it was not provided.',
           locations: [{ line: 3, column: 23 }],
         },
       ]);
@@ -306,7 +306,7 @@ describe('Validate: Provided required arguments', () => {
       `).toDeepEqual([
         {
           message:
-            'Argument @deprecated(reason:) of type "String!" is required, but it was not provided.',
+            'Argument @deprecated(reason:) of type String! is required, but it was not provided.',
           locations: [{ line: 3, column: 23 }],
         },
       ]);
@@ -328,7 +328,7 @@ describe('Validate: Provided required arguments', () => {
       ).toDeepEqual([
         {
           message:
-            'Argument @test(arg:) of type "String!" is required, but it was not provided.',
+            'Argument @test(arg:) of type String! is required, but it was not provided.',
           locations: [{ line: 4, column: 30 }],
         },
       ]);
@@ -350,7 +350,7 @@ describe('Validate: Provided required arguments', () => {
       ).toDeepEqual([
         {
           message:
-            'Argument @test(arg:) of type "String!" is required, but it was not provided.',
+            'Argument @test(arg:) of type String! is required, but it was not provided.',
           locations: [{ line: 2, column: 29 }],
         },
       ]);

diff --git a/src/validation/rules/ProvidedRequiredArgumentsRule.ts b/src/validation/rules/ProvidedRequiredArgumentsRule.ts
@@ -45,7 +45,7 @@ export function ProvidedRequiredArgumentsRule(
           if (!providedArgs.has(argDef.name) && isRequiredArgument(argDef)) {
             context.reportError(
               new GraphQLError(
-                `Argument ${argDef} of type "${argDef.type}" is required, but it was not provided.`,
+                `Argument ${argDef} of type ${argDef.type} is required, but it was not provided.`,
                 { nodes: fieldNode },
               ),
             );
@@ -114,7 +114,7 @@ export function ProvidedRequiredArgumentsOnDirectivesRule(
                 : print(argDef.type);
               context.reportError(
                 new GraphQLError(
-                  `Argument @${directiveName}(${argName}:) of type "${argType}" is required, but it was not provided.`,
+                  `Argument @${directiveName}(${argName}:) of type ${argType} is required, but it was not provided.`,
                   { nodes: directiveNode },
                 ),
               );