-
Notifications
You must be signed in to change notification settings - Fork 63
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
Electron testing through karma. #107
base: master
Are you sure you want to change the base?
Conversation
I'm not sure why CircleCI is failing on |
Can you trigger a rebuild? |
Done. |
Thanks. Looks good to me. I'll merge once we get the tests passing (which is probably npm shenanigans) |
Yeah I'm thinking the same thing (npm issues). I'll try trigger a build in a few hours before bed. If that doesn't work I'll try to find some time to look into it further. |
I ssh'd into the CircleCI container to troubleshoot the npm install of slimerjs and it seems the issue is with verifying the SSL cert of Does anyone have a simple solution? For reference here is the error message from wget:
For reference here is the
|
Switching between Ubuntu versions on CircleCI requires a commit to trigger the build so this useless change is what I'm using. The reason I am trying Ubuntu 14.04 is to see if it solves the slimerjs download problems.
Hey @orther I tried running the tests again and we've made some progress. The Slimerjs issues were probably due to something broken on their end, and I would remove the new slimerjs installation commands. I do see that |
Yeah I noticed the advanced failing too and have been meaning to look into
|
Sorry for delay I am still trying to get this fixed. |
#90