Skip to content

fix broken tests - #1108

Merged
cspotcode merged 2 commits into
masterfrom
ab/fix-broken-tests
Aug 11, 2020
Merged

fix broken tests#1108
cspotcode merged 2 commits into
masterfrom
ab/fix-broken-tests

Conversation

@cspotcode

@cspotcodecspotcode commented Aug 11, 2020

Copy link
Copy Markdown
Collaborator

Some logic wasn't wrapped in a before() hook so if you used --fgrep then tests would fail. Probably a race condition where another test was installing the require() hooks.

@codecov

codecovBot commented Aug 11, 2020

Copy link
Copy Markdown

Codecov Report

Merging #1108 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #1108 +/- ##
=======================================
Coverage 75.49% 75.49% =======================================
Files 7 7 Lines 657 657 Branches 148 148 =======================================
Hits 496 496 Misses 107 107 Partials 54 54 
FlagCoverage Δ
#node_1071.96% <ø> (-0.19%)⬇️
#node_12_1572.34% <ø> (-0.19%)⬇️
#node_12_1672.53% <ø> (ø)
#node_1375.19% <ø> (ø)
#node_1475.19% <ø> (ø)
#typescript_2_775.03% <ø> (ø)
#typescript_latest74.27% <ø> (ø)
#typescript_next74.12% <ø> (ø)
#ubuntu75.03% <ø> (-0.16%)⬇️
#windows75.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b91aa85...c637f5e. Read the comment docs.

@cspotcode
cspotcode merged commit 77f568d into masterAug 11, 2020
@coveralls

coveralls commented Aug 11, 2020

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.0%) to 83.175% when pulling c637f5e on ab/fix-broken-tests into b91aa85 on master.

@cspotcode
cspotcode deleted the ab/fix-broken-tests branch August 12, 2020 18:22
@cspotcodecspotcode mentioned this pull request Aug 21, 2020
4 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@cspotcode@coveralls