Skip to content
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

Testing: Include IE11 "Canary" test in Travis build #16509

Closed
aduth opened this issue Jul 10, 2019 · 6 comments
Closed

Testing: Include IE11 "Canary" test in Travis build #16509

aduth opened this issue Jul 10, 2019 · 6 comments
Labels
Browser Issues Issues or PRs that are related to browser specific problems Needs Decision Needs a decision to be actionable or relevant [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests.

Comments

@aduth
Copy link
Member

aduth commented Jul 10, 2019

There have been a number of occurrences[1][2][3][4][5][6][7][8] of catastrophic errors in Internet Explorer which would be immediately obvious on any attempt to load the block editor. In lieu of running the full set of end-to-end tests, we should explore a minimally viable solution to verify that the editor loads without errors and displays some expected elements on the screen.

Possible approach references:

@aduth aduth added [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. Browser Issues Issues or PRs that are related to browser specific problems labels Jul 10, 2019
@aduth
Copy link
Member Author

aduth commented Jul 10, 2019

Considering the future of browser support (eventually dropping Internet Explorer support), using something like BrowserStack to attempt minimal loading in a variety of browsers might be a more advisable approach.

Worth mentioning that Puppeteer support is expanding to add official support for Firefox, where we can broaden our end-to-end coverage to include this browser as well (a separate task).

@aduth
Copy link
Member Author

aduth commented Mar 27, 2020

The implementation is different than what is proposed here, but there was some improvements to the general broad goal of protecting against Internet Explorer-related issues in @tellthemachines's pull request at #18774 (simulating IE by running ESLint on the generated bundles in ES5 mode).

@tellthemachines
Copy link
Contributor

Have there been any instances of this since I added the lint check? I haven't seen immediate errors on load for a while, but from time to time there are smaller issues like #21276. This wouldn't be caught if we were just checking for page load. I wonder if it would be more useful to have a few checks for basic interaction, e.g. can write text, can add blocks etc.

@aduth
Copy link
Member Author

aduth commented Apr 1, 2020

@tellthemachines I'm not personally aware of any "catastrophic failures" (white screens), but there's been a handful of smaller issues related to interaction (the one you mention, also #19979, #20598, and a few other minor ones). I agree that would be useful have testing for those interactions. At this point, since there's so little settled here as far as an approach, it would probably depend whether the solution we choose would have the means to script those sorts of interactions. The options in the original comment are all still viable, and still encompassing of what I understand to be the most promising options available. Writing some basic Selenium tests for Internet Explorer to run in Travis' Windows environment could be the most promising for what you're suggesting, perhaps?

@sainthkh
Copy link
Contributor

I've created an experimental PR in #21528. I hope this solves the IE testing problem.

@gziolo gziolo added the [Status] In Progress Tracking issues with work in progress label May 18, 2020
@gziolo gziolo added Needs Decision Needs a decision to be actionable or relevant and removed [Status] In Progress Tracking issues with work in progress labels Feb 2, 2021
@gziolo
Copy link
Member

gziolo commented Mar 30, 2021

WordPress is dropping support for IE 11 later this year so this issue is no longer relevant:

https://make.wordpress.org/core/2021/03/25/discussion-summary-dropping-support-for-ie11/

Thank you everyone for all the time spent on exploring this idea 🙇🏻

@gziolo gziolo closed this as completed Mar 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Browser Issues Issues or PRs that are related to browser specific problems Needs Decision Needs a decision to be actionable or relevant [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants