Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Commit

Permalink
Fix skipping tests break in the test runner (#27)
Browse files Browse the repository at this point in the history
  • Loading branch information
eastenluis authored Nov 23, 2018
1 parent 688f3f0 commit ed3e136
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion service/src/jestSetup/screenshotReporter.js
Original file line number Diff line number Diff line change
Expand Up @@ -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}`)
Expand Down

0 comments on commit ed3e136

Please sign in to comment.