fixed tests that were using print (#4341)

There were a few tests that broke due to changes to print params
graphql · Feb 7, 2025 · 7121006 · 7121006
1 parent e0c2425
commit 7121006
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 9 deletions.
diff --git a/src/utilities/__tests__/buildASTSchema-test.ts b/src/utilities/__tests__/buildASTSchema-test.ts
@@ -60,7 +60,7 @@ function expectASTNode(obj: Maybe<{ readonly astNode: Maybe<ASTNode> }>) {
 function expectExtensionASTNodes(obj: {
   readonly extensionASTNodes: ReadonlyArray<ASTNode>;
 }) {
-  return expect(obj.extensionASTNodes.map(print).join('\n\n'));
+  return expect(obj.extensionASTNodes.map((node) => print(node)).join('\n\n'));
 }
 
 describe('Schema Builder', () => {

diff --git a/src/utilities/__tests__/extendSchema-test.ts b/src/utilities/__tests__/extendSchema-test.ts
@@ -39,7 +39,7 @@ import { printSchema } from '../printSchema';
 function expectExtensionASTNodes(obj: {
   readonly extensionASTNodes: ReadonlyArray<ASTNode>;
 }) {
-  return expect(obj.extensionASTNodes.map(print).join('\n\n'));
+  return expect(obj.extensionASTNodes.map((node) => print(node)).join('\n\n'));
 }
 
 function expectASTNode(obj: Maybe<{ readonly astNode: Maybe<ASTNode> }>) {
@@ -51,10 +51,12 @@ function expectSchemaChanges(
   schema: GraphQLSchema,
   extendedSchema: GraphQLSchema,
 ) {
-  const schemaDefinitions = parse(printSchema(schema)).definitions.map(print);
+  const schemaDefinitions = parse(printSchema(schema)).definitions.map((node) =>
+    print(node),
+  );
   return expect(
     parse(printSchema(extendedSchema))
-      .definitions.map(print)
+      .definitions.map((node) => print(node))
       .filter((def) => !schemaDefinitions.includes(def))
       .join('\n\n'),
   );

diff --git a/src/utilities/__tests__/separateOperations-test.ts b/src/utilities/__tests__/separateOperations-test.ts
@@ -49,7 +49,9 @@ describe('separateOperations', () => {
       }
     `);
 
-    const separatedASTs = mapValue(separateOperations(ast), print);
+    const separatedASTs = mapValue(separateOperations(ast), (node) =>
+      print(node),
+    );
     expect(separatedASTs).to.deep.equal({
       '': dedent`
         {
@@ -128,7 +130,9 @@ describe('separateOperations', () => {
       }
     `);
 
-    const separatedASTs = mapValue(separateOperations(ast), print);
+    const separatedASTs = mapValue(separateOperations(ast), (node) =>
+      print(node),
+    );
     expect(separatedASTs).to.deep.equal({
       One: dedent`
         query One {
@@ -178,7 +182,9 @@ describe('separateOperations', () => {
       }
     `);
 
-    const separatedASTs = mapValue(separateOperations(ast), print);
+    const separatedASTs = mapValue(separateOperations(ast), (node) =>
+      print(node),
+    );
     expect(separatedASTs).to.deep.equal({
       '': dedent`
         {
@@ -215,7 +221,9 @@ describe('separateOperations', () => {
       type Bar
     `);
 
-    const separatedASTs = mapValue(separateOperations(ast), print);
+    const separatedASTs = mapValue(separateOperations(ast), (node) =>
+      print(node),
+    );
     expect(separatedASTs).to.deep.equal({
       Foo: dedent`
         query Foo {
@@ -241,7 +249,9 @@ describe('separateOperations', () => {
       }
     `);
 
-    const separatedASTs = mapValue(separateOperations(ast), print);
+    const separatedASTs = mapValue(separateOperations(ast), (node) =>
+      print(node),
+    );
     expect(separatedASTs).to.deep.equal({
       '': dedent`
         {