-
Notifications
You must be signed in to change notification settings - Fork 103
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 for Safari & Chrome 38 on karma 6.x #239
Comments
Hi @longlho Can you provide some more information, where did it fail? |
I believe it failed at the end after all tests are run. selenium-server.log shows
Specifically
although I'm not sure what this was referred to |
This is the build link for Chrome 38 which is also failing: https://app.saucelabs.com/tests/e9aafdaf2e794cf9887562e4a235826a Logs don't seem to show any error yet Custom Data indicates there's an error:
|
Thanks @longlho What about your own server logs? And secondly, is there a reason for testing on Chrome 38? That one is really old |
Yes we maintain polyfills so we do need to test on really old browsers. Can you clarify my own server logs? From my build log all I saw (regarding err) was:
But I'm pretty sure my tests do not reload the page... |
Hi @longlho Hmm, I don't know where the page reload is coming from, but that is the one that is causing the test to be set too failed. Can you share the link of Sauce Labs of that session? |
Yup i posted it above |
Ok, problem is that I can see it there so can't debug it |
hmm any rec on what I can do to help debug? |
I've released a new version that will add the old logs back to Sauce Labs, that one might give more insights where the reload is coming from, can you update to this version and see what happens, see v4.3.4...v4.3.5 thanks |
ok I tried 4.3.5. The build URLs are: |
Hi @longlho I've been looking into this and it looks like it's something from Karma and or your testcode instead of an issue with the service. There is no webdriver command going to the browsers that would initiate this reload, so they only thing I can conclude is that there is a page reload in the test / karma is triggering this. |
@wswebcreation looks like karma-runner/karma#3643 fixed this! Thanks for looking |
Thanks for getting back on this @longlho |
All tests passed but the build error out and IDK why.
Build: https://app.saucelabs.com/tests/f000b44a703c465e9be79a2ec7ab326c
The text was updated successfully, but these errors were encountered: