From ed3e1365929058340befe58da5df8e7ada2617c0 Mon Sep 17 00:00:00 2001 From: Martin Lai Date: Fri, 23 Nov 2018 12:32:39 -0500 Subject: [PATCH] Fix skipping tests break in the test runner (#27) --- service/src/jestSetup/screenshotReporter.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/service/src/jestSetup/screenshotReporter.js b/service/src/jestSetup/screenshotReporter.js index d73ea108..f5548565 100644 --- a/service/src/jestSetup/screenshotReporter.js +++ b/service/src/jestSetup/screenshotReporter.js @@ -29,7 +29,7 @@ class PuppeteerScreenshotReporter { for (let result of testResult.testResults) { const relativePath = path.join(result.fullName, this._options.filename || 'screenshot.png') const screenshot = path.join(this._options.output, relativePath) - if (result.status !== 'passed') { + if (result.status === 'failed') { const downloadLink = await this.uploadScreenshotToS3(screenshot) result.failureMessages.push(`Screenshot available at ${relativePath}`)