improve code

local/full-coverage/bin/generateFullCoverage.js

@@ -17,35 +17,43 @@ const copyReportFile = (path) => {
       path,
       join(tmpCoverageDirectory, `${basename(dirname(path))}_${Date.now()}.json`)
     );
+    return true;
   }
+  return false;
 };
 
 const generateFullCoverage = async () => {
-  copyReportFile(join(playwrightCoverage, reportFileName));
+  const copiedPlaywright = copyReportFile(join(playwrightCoverage, reportFileName));
-  copyReportFile(join(jestCoverage, reportFileName));
+  const copiedJest = copyReportFile(join(jestCoverage, reportFileName));
 
-  const mergeDestination = join(tmpCoverageDirectory, `full_${Date.now()}.json`);
+  if (copiedPlaywright || copiedJest) {
-  execSync(`nyc merge ${tmpCoverageDirectory} ${mergeDestination}`);
+    const mergeDestination = join(tmpCoverageDirectory, `full_${Date.now()}.json`);
+    execSync(`nyc merge ${tmpCoverageDirectory} ${mergeDestination}`);
 
-  const files = fs.readdirSync(tmpCoverageDirectory);
+    const files = fs.readdirSync(tmpCoverageDirectory);
-  files.forEach((file) => {
+    files.forEach((file) => {
-    const filePath = join(tmpCoverageDirectory, file);
+      const filePath = join(tmpCoverageDirectory, file);
-    if (filePath !== mergeDestination) {
+      if (filePath !== mergeDestination) {
-      fs.rmSync(filePath);
+        fs.rmSync(filePath);
-    }
+      }
-  });
+    });
 
-  execSync(
+    execSync(
-    `nyc report --reporter=lcov --reporter=clover --reporter=json --report-dir=${coverageDirectory} --temp-dir=${tmpCoverageDirectory}`
+      `nyc report --reporter=lcov --reporter=clover --reporter=json --report-dir=${coverageDirectory} --temp-dir=${tmpCoverageDirectory}`
-    /* { stdio: 'inherit' } */
+      /* { stdio: 'inherit' } */
-  );
+    );
-  fs.rmSync(tmpCoverageDirectory, { recursive: true });
+
+    fs.rmSync(tmpCoverageDirectory, { recursive: true });
+  }
 };
 
 (async () => {
   try {
     await generateFullCoverage();
   } catch (e) {
-    console.error(`Full coverage couldn't be generated.`, e);
+    // console.error(`Full coverage couldn't be generated.`, e);
+    if (fs.existsSync(tmpCoverageDirectory)) {
+      fs.rmSync(tmpCoverageDirectory, { recursive: true });
+    }
   }
 })();

local/playwright-tooling/bin/mergeCoverage.js

@@ -28,6 +28,9 @@ const mergeCoverage = async () => {
   try {
     await mergeCoverage();
   } catch (e) {
-    console.error(`Playwright coverage couldn't be merged.`, e);
+    // console.error(`Playwright coverage couldn't be merged.`, e);
+    if (fs.existsSync(tmpCoverageDirectory)) {
+      fs.rmSync(tmpCoverageDirectory, { recursive: true });
+    }
   }
 })();