refactor: improve getWorkDir() implementation

__tests__/pytest-junit.test.ts

@@ -30,7 +30,6 @@ describe('pytest-junit tests', () => {
 
     const opts: ParseOptions = {
       parseErrors: true,
-      workDir: 'mnt/extra-addons',
       trackedFiles: ['addons/product_changes/tests/first_test.py']
     }
 

action.yml

@@ -30,6 +30,7 @@ inputs:
         - flutter-json
         - java-junit
         - jest-junit
+        - pytest-junit
         - mocha-json
     required: true
   list-suites:

src/report/get-annotations.ts

@@ -69,9 +69,10 @@ export function getAnnotations(results: TestRunResult[], maxCount: number): Anno
   errors.splice(maxCount + 1)
 
   const annotations = errors.map(e => {
+    const paths = e.path ? [e.path] : e.testRunPaths
     const message = [
       'Failed test found in:',
-      e.testRunPaths.map(p => `  ${p}`).join('\n'),
+      paths.map(p => `  ${p}`).join('\n'),
       'Error:',
       ident(fixEol(e.message), '  ')
     ].join('\n')

src/utils/path-utils.ts

@@ -23,17 +23,21 @@ export function getBasePath(path: string, trackedFiles: string[]): string | unde
     return ''
   }
 
-  let max = ''
   for (const file of trackedFiles) {
-    if (path.endsWith(file) && file.length > max.length) {
+    const pathParts = path.split('/')
-      max = file
+    const originalLength = pathParts.length
+    const fileParts = file.split('/')
+
+    while (pathParts.length && pathParts.slice(-1)[0] === fileParts.slice(-1)[0]) {
+      pathParts.pop()
+      fileParts.pop()
+    }
+
+    // we found some matching path parts
+    if (pathParts.length !== originalLength) {
+      return pathParts.join('/')
     }
   }
 
-  if (max === '') {
+  return undefined
-    return undefined
-  }
-
-  const base = path.substr(0, path.length - max.length)
-  return base
 }