Make replaceExtensions work with inheritTypeDefs

.changeset/few-queens-bathe.md

@@ -0,0 +1,5 @@
+---
+'graphql-modules': patch
+---
+
+Make replaceExtensions work with inheritTypeDefs

packages/graphql-modules/src/testing/test-module.ts

@@ -76,15 +76,6 @@ export function testModule(testedModule: Module, config?: TestModuleConfig) {
 function transformModule(mod: Module, config?: TestModuleConfig) {
   const transforms: ((m: Module) => Module)[] = [];
 
-  if (config?.replaceExtensions) {
-    transforms.push((m) =>
-      moduleFactory({
-        ...m.config,
-        typeDefs: replaceExtensions(m.typeDefs),
-      })
-    );
-  }
-
   if (config?.typeDefs) {
     transforms.push((m) =>
       moduleFactory({
@@ -117,6 +108,15 @@ function transformModule(mod: Module, config?: TestModuleConfig) {
     });
   }
 
+  if (config?.replaceExtensions) {
+    transforms.push((m) =>
+      moduleFactory({
+        ...m.config,
+        typeDefs: replaceExtensions(m.typeDefs),
+      })
+    );
+  }
+
   if (transforms) {
     return transforms.reduce((m, transform) => transform(m), mod);
   }

packages/graphql-modules/tests/testing.spec.ts

@@ -1,6 +1,6 @@
 import 'reflect-metadata';
 import type { TypedDocumentNode } from '@graphql-typed-document-node/core';
-import { concatAST } from 'graphql';
+import { concatAST, printSchema } from 'graphql';
 import {
   createApplication,
   createModule,
@@ -13,6 +13,10 @@ import {
   Scope,
 } from '../src';
 
+function normalizeSDL(sdl: string): string {
+  return sdl.replace(/\s+/g, ' ').trim();
+}
+
 describe('testModule', () => {
   test('should replace extensions with definitions on demand', () => {
     const initialModule = createModule({
@@ -581,4 +585,61 @@ describe('mockApplication', () => {
       extraEnv: 'mocked',
     });
   });
+
+  test('replaceExtensions should work with inheritTypeDefs', () => {
+    const listModule = createModule({
+      id: 'list',
+      typeDefs: gql`
+        extend type Query {
+          list: List!
+        }
+
+        type List {
+          id: ID!
+          title: String!
+        }
+      `,
+    });
+
+    const productModule = createModule({
+      id: 'product',
+      typeDefs: gql`
+        extend type Query {
+          product: Product!
+        }
+
+        type Product {
+          id: ID!
+        }
+
+        extend type List {
+          product: Product!
+        }
+      `,
+    });
+
+    const app = testkit.testModule(productModule, {
+      replaceExtensions: true,
+      inheritTypeDefs: [listModule],
+    });
+
+    expect(() => app.schema).not.toThrow();
+    expect(normalizeSDL(printSchema(app.schema))).toMatch(
+      normalizeSDL(`
+        type Query {
+          product: Product!
+        }
+
+        type Product {
+          id: ID!
+        }
+
+        type List {
+          id: ID!
+          title: String!
+          product: Product!
+        }
+    `)
+    );
+  });
 });