fix: add CTA to warning message

aws-amplify · Oct 13, 2023 · b77b9ad · b77b9ad
1 parent a02a48b
commit b77b9ad
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/packages/amplify-codegen/src/commands/types.js b/packages/amplify-codegen/src/commands/types.js
@@ -77,7 +77,9 @@ async function generateTypes(context, forceDownloadSchema, withoutInit = false,
           })
           .filter(([queryFilePath, source]) => {
             if (!source) {
-              context.print.warning(`Unable to extract GraphQL queries from ${queryFilePath}. Skipping source.`);
+              context.print.warning(
+                `Unable to extract GraphQL queries from ${queryFilePath}. Skipping source. This source matched the includes target in .grapqhlconfig.yml. Modify the includes or excludes target if this file should not be included.`,
+              );
               return false;
             }
             return true;

diff --git a/packages/amplify-codegen/tests/commands/types-mock-fs.test.js b/packages/amplify-codegen/tests/commands/types-mock-fs.test.js
@@ -132,10 +132,14 @@ describe('command - types (mock fs)', () => {
     await generateStatements(MOCK_CONTEXT, false);
     await generateTypes(MOCK_CONTEXT, false);
     expect(MOCK_CONTEXT.print.warning).toHaveBeenCalledWith(
-      expect.stringMatching('Unable to extract GraphQL queries from .*/src/graphql/foo.ts. Skipping source.'),
+      expect.stringMatching(
+        'Unable to extract GraphQL queries from .*/src/graphql/foo.ts. Skipping source. This source matched the includes target in .grapqhlconfig.yml. Modify the includes or excludes target if this file should not be included.',
+      ),
     );
     expect(MOCK_CONTEXT.print.warning).not.toHaveBeenCalledWith(
-      expect.stringMatching('Unable to extract GraphQL queries from .*/src/graphql/excluded.ts. Skipping source.'),
+      expect.stringMatching(
+        'Unable to extract GraphQL queries from .*/src/graphql/excluded.ts. Skipping source. This source matched the includes target in .grapqhlconfig.yml. Modify the includes or excludes target if this file should not be included.',
+      ),
     );
     expect(fs.existsSync(generatedFileName)).toBeTruthy();
   });