Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Storyshots puppeteer images do not match #4045

Merged
merged 2 commits into from Aug 20, 2018

Conversation

igor-dv
Copy link
Member

@igor-dv igor-dv commented Aug 20, 2018

Issue: Changes from #4037 by @markelog . Merged it here to check what happens with CircleCI.

markelog and others added 2 commits August 17, 2018 17:54
When current screenshots do not much with previous ones,
we show connection error since `expect` method also throws.

Workaround that and also lowercase the error message so it would look
like more then other errors

Fixes #4036
…-not-match

Storyshots puppeteer images do not match
@codecov
Copy link

codecov bot commented Aug 20, 2018

Codecov Report

Merging #4045 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4045      +/-   ##
==========================================
- Coverage   40.82%   40.82%   -0.01%     
==========================================
  Files         444      444              
  Lines        5555     5556       +1     
  Branches      748      748              
==========================================
  Hits         2268     2268              
- Misses       2911     2912       +1     
  Partials      376      376
Impacted Files Coverage Δ
...ddons/storyshots/storyshots-puppeteer/src/index.js 0% <0%> (ø) ⬆️

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 494bda5...9ca612f. Read the comment docs.

@igor-dv igor-dv merged commit b7054f3 into master Aug 20, 2018
@igor-dv igor-dv deleted the storyshots-puppeteer-images-do-not-match branch August 20, 2018 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants