jquery / jquery

jQuery JavaScript Library
https://jquery.com
MIT License
58.93k stars 20.62k forks source link

Tests: reuse browser workers in BrowserStack tests #5428

Closed timmywil closed 2 months ago

timmywil commented 2 months ago

Summary

I suggest reviewing #5427 first. Any more changes made there that might apply to main I will add here as well.

Rather than spinning up a separate browser instance for each module, we send back the next test's URL when a module completes, if there is one. Then listeners.js sets window.location. At first, I tried BrowserStack's change url API, but it was slower and incredibly error-prone.

Sample BrowserStack run: https://github.com/timmywil/jquery/actions/runs/8123504592

Testing time has gone from 15-30min to 5min!

Checklist