Merge branch 'master' into 131-cleanup

ryanluker · Jan 28, 2018 · 22d75cf · 22d75cf
2 parents fa32d43 + cbad7d1
commit 22d75cf
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/indicators.ts b/src/indicators.ts
@@ -61,8 +61,16 @@ export class Indicators {
                 if (err) { return reject(err); }
                 const section = data.find((lcovSection) => {
                     // consider windows and linux file paths
-                    const cleanFile = file.replace(/[\\\/]/g, "");
-                    const cleanLcovFileSection = lcovSection.file.replace(/[\\\/]/g, "");
+                    let cleanFile = file.replace(/[\\\/]/g, "");
+                    let cleanLcovFileSection = lcovSection.file.replace(/[\\\/]/g, "");
+
+                    // on Windows remove drive letter from path because of cobertura format
+                    // also convert both path to lowercase because Windows's filesystem is case insensitive
+                    if ( process.platform === "win32" ) {
+                        cleanFile = cleanFile.substr(2).toLowerCase();
+                        cleanLcovFileSection = cleanLcovFileSection.toLowerCase();
+                    }
+
                     return cleanFile.includes(cleanLcovFileSection);
                 });