Repository navigation
Conversation
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
|
May be more involved in you all depend on a specific selenium version but the gist of what to do is included in the PR if someone more familiar with the Angular flow wants to take the ball. |
|
CLAs look good, thanks! |
|
Relaunched the build and found some issues with Edge:
@jdalton: have you seen such errors already please? |
|
I've got the same issue locally. Edge disconnects and the campaign restarts automatically, ending up in an infinite loop. When opening the dev tools to find more details, I bump into another bug due to the way systemjs loads scripts. See https://github.com/systemjs/systemjs/blob/master/lib/global-eval.js#L71 After hacking systemjs to change the load mechanism, the campaign is finally successful ... |
|
If using SystemJS you'll need to include their shim script, |
|
Hmmm... pinging @guybedford. |
|
I'd suggest upgrading to SystemJS 0.19.3 here and seeing if the error still persists as we recently fixed a critical IE loading bug. |
|
@guybedford Do you have a reference handy for the bug. I'll report it on our end. |
|
Upgrading to SystemJS 0.19.3 doesn't help, but I don't think this is the root cause here. I did some more investigations and there seems to be 2 distinct issues. 1) In Edge only, when the F12 dev tools are closed: 2) In both Edge and IE11, when the F12 dev tools are opened: Note that the campaign is successful in IE11 when the F12 dev tools are closed. |
|
@Mlaval thanks for the detailed report here. I can replicate this as well in the tests. It does sound like this is a limitation of this execution method in IE. I've created an issue in systemjs/systemjs#814 to track. I'd suggest forking SystemJS for now and including the hack to ensure eval-based execution. The thing is, we do need the inline script execution mode because many global scripts expect Ideally some queueing mechanism or otherwise can be implemented to ensure we can get this to work in IE. |
|
I'd like to stress out that this loading bug happens only when the F12 dev tools are opened. |
There was a problem hiding this comment.
I'd hold off with merging that until Sauce updates its Edge version number or it will start breaking after the fact.
@jdalton Do you know when they plan to do it? I contacted BrowserStack & they switched to "12.0".
There was a problem hiding this comment.
I'd hold off with merging that until Sauce updates its Edge version number or it will start breaking after the fact.
Sauce has been pretty good about not breaking folks with similar changes in the past, the existing '20.10240' is specific enough to avoid conflict for future versions too so I imagine they'll just alias it to '12'.
Do you know when they plan to do it?
I don't have an eta from them.
They've created an internal ticket with the request to change the version for MS Edge.
I contacted BrowserStack & they switched to "12.0".
Yap :D
There was a problem hiding this comment.
the existing '20.10240' is specific enough to avoid conflict for future versions too so I imagine they'll just alias it to '12'.
I don't fully know the Sauce Labs API as I mostly use BrowserStack but can you not pass just a major version, i.e. here 20? If you can then it will get in conflict with a future EdgeHTML 20 version, won't it?
OTH, the build number will be greater then so if they won't use the major only but continue to provide the build number, 20.SOMETHING_LARGE should be preferred to 20.10240 when you just specify 20.
|
Same issues still happening after the big update to latest Node.js and Karma |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Add
to the sauce runner.