Merge pull request #25241 from Microsoft/fixFailedTestsUnlink

rbuckton · web-flow · commit cb6325d8ec58 · 2018-06-26T13:06:59.000-07:00
Add exists check before unlink
diff --git a/scripts/failed-tests.js b/scripts/failed-tests.js
@@ -80,7 +80,7 @@ class FailedTestsReporter extends Mocha.reporters.Base {
             const failed = Array.from(failingTests).join(os.EOL);
             fs.writeFile(file, failed, "utf8", done);
         }
-        else if (!keepFailed) {
+        else if (!keepFailed && fs.existsSync(file)) {
             fs.unlink(file, done);
         }
         else {

Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ class FailedTestsReporter extends Mocha.reporters.Base {`
`80`	`80`	`const failed = Array.from(failingTests).join(os.EOL);`
`81`	`81`	`fs.writeFile(file, failed, "utf8", done);`
`82`	`82`	`}`
`83`		`- else if (!keepFailed) {`
	`83`	`+ else if (!keepFailed && fs.existsSync(file)) {`
`84`	`84`	`fs.unlink(file, done);`
`85`	`85`	`}`
`86`	`86`	`else {`