Differential D8608

[CashTab] Only show console messages if test fails
ClosedPublic
Actions

Authored by alcipir on Dec 6 2020, 08:03.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABC81333d8c462c: [CashTab] Only show console messages if test fails

Summary

Adds a script to show console messages only if the tests fails. Currently when running our tests, there is a lot of noise from console even if everything is fine and the test passes.
Also adds CI=true flag to npm run test:coverage script, thus making watchMode false. This is required for further CI usage.

Test Plan

npm install && npm run test:coverage. Should not show any console messages unless a test fails. Should also not be in watchMode.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

alcipir created this revision.Dec 6 2020, 08:03

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 6 2020, 08:03

alcipir requested review of this revision.Dec 6 2020, 08:03

Harbormaster completed remote builds in B14319: Diff 26207.Dec 6 2020, 08:03

Fix wrong argument name being passed

Harbormaster completed remote builds in B14320: Diff 26208.Dec 6 2020, 08:11

alcipir edited the summary of this revision. (Show Details)Dec 6 2020, 08:14

majcosta accepted this revision.Dec 6 2020, 20:54

This revision is now accepted and ready to land.Dec 6 2020, 20:54

Closed by commit rABC81333d8c462c: [CashTab] Only show console messages if test fails (authored by alcipir). · Explain WhyDec 6 2020, 21:01

This revision was automatically updated to reflect the committed changes.

alcipir added a commit: rABC81333d8c462c: [CashTab] Only show console messages if test fails.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

6 lines

scripts/

58 lines

Diff 26212

web/cashtab/package.json

Loading...

web/cashtab/scripts/testSetup.js

Loading...