CP-925 Coverage: wait for "open" command

[evanweible-wf]

Issue

#59

Changes

Source:

• Need to add an await when running the "open" command after generating coverage report

Tests:

• n/a

Areas of Regression

• n/a

Testing

• This is sort of hard to test, but on master you should see inconsistent behavior on whether or not the coverage report is automatically opened after generation.
• With this change, the report should always open (as long as --open flag is set to true)

Code Review

@trentgrover-wf
@maxwellpeterson-wf
@dustinlessard-wf
@jayudey-wf

[trentgrover-wf]

+1

[codecov-io]

Current coverage is 48.05%

Merging #60 into master will not affect coverage as of 176d8a1

Powered by Codecov. Updated on successful CI builds.

[dustinlessard-wf]

+1

[maxwellpeterson-wf]

+1

[evanweible-wf]

@jayudey-wf ready for merge

[jayudey-wf]

QA Resource Approval: +10

• Testing instruction
• Dev +1's
• Dev/QA +10 with detail of what was tested
  • verified that when running with open flag the html page is opened by default
• Unit test created/updated
• All unit tests pass

Merging into master.

[jayudey-wf]

@Rosie run_merge_script