Merge pull request #625 from tj-actions/feat/add-warning-message-for-…

…invalid-patterns
tj-actions · Jan 18, 2023 · 70ce922 · 70ce922
2 parents dc8e7c4 + 7dd123a
commit 70ce922
Show file tree

Hide file tree

Showing 4 changed files with 34 additions and 12 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/__tests__/main.test.ts b/src/__tests__/main.test.ts
@@ -378,3 +378,19 @@ test('matched file patterns with braces are expanded', async () => {
 
   expect(core.setOutput).toHaveBeenNthCalledWith(2, 'paths', EXPECTED_FILENAMES)
 })
+
+test('warnings are logged when files are not found', async () => {
+  mockedEnv({
+    ...defaultEnv,
+    INPUT_FILES: 'src/__tests__/not-found.txt'
+  })
+
+  // @ts-ignore
+  core.warning = jest.fn()
+
+  await run()
+
+  expect(core.warning).toHaveBeenCalledWith(
+    'No paths found using the specified patterns'
+  )
+})
diff --git a/src/main.ts b/src/main.ts
@@ -263,23 +263,25 @@ export async function run(): Promise<void> {
 
   const pathsOutput = paths.join(separator)
 
+  const hasCustomPatterns =
+    files !== '' ||
+    filesFromSourceFile !== '' ||
+    excludedFiles !== '' ||
+    excludedFilesFromSourceFile !== ''
+
   if (pathsOutput) {
     const pathsOutputFile = await tempfile('.txt')
     await fs.writeFile(pathsOutputFile, pathsOutput, {flag: 'w'})
 
     core.setOutput('paths-output-file', pathsOutputFile)
     core.saveState('paths-output-file', pathsOutputFile)
     core.info(`Successfully created paths-output-file: ${pathsOutputFile}`)
+  } else if (hasCustomPatterns) {
+    core.warning('No paths found using the specified patterns')
   }
 
   core.setOutput('paths', pathsOutput)
-  core.setOutput(
-    'has-custom-patterns',
-    files !== '' ||
-      filesFromSourceFile !== '' ||
-      excludedFiles !== '' ||
-      excludedFilesFromSourceFile !== ''
-  )
+  core.setOutput('has-custom-patterns', hasCustomPatterns)
 }
 
 if (!process.env.TESTING) {