Skip to content

Update Dependecies, Fix Fatal Error with mkdirp and Mac #66

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

Merged
merged 2 commits into from
Apr 27, 2020

Conversation

puneet0191
Copy link
Member

Fixes: #45
#61

@puneet0191
Copy link
Member Author

@jonatask could you please help me test this fix, if everything works we can merge your PR's along with this one and release a new version.

@jonatask
Copy link
Contributor

@puneet0191 Yes, I will test it.

@jonatask
Copy link
Contributor

jonatask commented Apr 26, 2020

@puneet0191 When installing the dependency then I see these:

npm WARN [email protected] requires a peer of bufferutil@^4.0.1 but none is installed. You must install peer dependencies yourself.
npm WARN [email protected] requires a peer of utf-8-validate@^5.0.2 but none is installed. You must install peer dependencies yourself.

The latest version also presents this warning. However, I suggest checking if there is something we can do.

@jonatask
Copy link
Contributor

The functional aspects are ok. There was not impact in the directory, mismatch calculation, elements and whole page checks, neither reports, prepareBaseline, images or any functional aspect.

@puneet0191 puneet0191 changed the base branch from master to 1.9.1 April 27, 2020 10:21
@puneet0191 puneet0191 merged commit 714c79a into codeceptjs:1.9.1 Apr 27, 2020
puneet0191 added a commit that referenced this pull request May 16, 2020
* Issue #60 - Prepare all baselines of test by setting parameter in config (#65)

Co-authored-by: Jonatas Kirsch <[email protected]>

* Update Dependecies, Fix Fatal Error with mkdirp and Mac (#66)

* Update Dependencies

* Fix Error on Mac

* Avoid failing the test for a given threshold but yet generating the difference image (#63)

* Option bypassFailure allowing the user to avoid failing the test for a given threshold but yet generating the difference image

* Renamed option to skipFailure

Co-authored-by: Jonatas Kirsch <[email protected]>

* feat(testcafe): add the support for testcafe (#62)

* passing through output settings to resemble.js (#59)

Co-authored-by: JANK Michael <[email protected]>

* Issue 48 - Add custom assert message (#64)

Co-authored-by: Jonatas Kirsch <[email protected]>

Co-authored-by: Jonatas Kirsch <[email protected]>
Co-authored-by: Jonatas Kirsch <[email protected]>
Co-authored-by: Guillaume Camus <[email protected]>
Co-authored-by: yankydoo <[email protected]>
Co-authored-by: JANK Michael <[email protected]>
Sign up for free to 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.

Not installed on MAC ERR! [email protected]
2 participants