-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
Error [object Object] on node_modules/cypress-visual-regression/dist/command.js:33 #50
Comments
Found the issue, it happens when the filename contains newlines and/or tabs. |
Hi, I stumbled across another error and also found no helpful error in the log. After investigating a bit I found out that Not sure why rejecting an Error leads leads to a corrupt error message in the log. maybe here a similar situation is responsible for the wrong "Error [object Object]" log. |
@fabriciols @gabbersepp it also does this if you don't have the base files in snapshots/base |
@skjnldsv yes, so I think that by sanitizing the file names it will actually display this in the log since it won't be able to find the file: My guess is that it's blowing up somewhere else, causing the error to be caught somewhere where it's not being handled correctly with |
Can you fix the broken build for your PR @skjnldsv. I'm curious to see the results of this. https://travis-ci.org/github/mjhea0/cypress-visual-regression/builds/721099784 |
I have no ideas on how to fix this. My pr properly include the required deps, If the tests uses different ones, maybe the travis workflow should be adapted to use the dependencies from |
@gabbersepp |
Ok. I'll look into it. |
Thanks :) const fileName = `AppSidebar.vue
- starred_${star === null ? 'null' : star ? 'true' : 'false'}
- compact_${comp ? 'true' : 'false'}
- header_${head ? 'image' : 'none'}
- secondary_${second ? 'button' : 'none'}
` |
is this correct?
|
This is only one string. This is using the new es6 multi line template string feature. |
Hi, I'm having an error, but the logs are not helpful.
Snapshots files are in the base folder, diffs are not generated.
Cypress version 5.0.0.
Node version 10.x
cypress.json:
The text was updated successfully, but these errors were encountered: