- Issue created by @catch
- 🇬🇧United Kingdom catch
time ../vendor/bin/phpunit modules/ckeditor5/tests/src/FunctionalJavascript/ImageTest.php real 9m58.387s user 3m20.882s sys 0m38.975s
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Yep, functional JS tests are slow.
There's a lot of waiting for things to happen.
The only proposal I have is a very simple one, somewhat reminiscent of 📌 Add GenericModuleTestBase and use it to test general module things Downport : require all functional JS tests (in core at least) to have only a single test method. That way, they can be maximally parallellized.
- 🇬🇧United Kingdom catch
I think we need to figure out if we want to do 📌 [experiment] Explore paratest to run our phpunit tests in parallel Needs work since that runs by-method I think. If we don't, then we should do one-test-method-per-class for the very slowest tests per #3.
- Status changed to Postponed
about 1 year ago 4:41pm 20 September 2023 - Status changed to Active
about 1 year ago 5:51pm 25 September 2023 - 🇬🇧United Kingdom catch
Had a look at paratest, running by method will take a lotof refactoring and it also supports running by class, so even if we were to replace run-tests.sh with it we might not switch to per-method.
I think we should go ahead here, even if we switch to per-method concurrency eventually, it won't do any harm, it'll be just slightly unnecessary work.
I don't think we need to split these into method-per-class, a test class with 10-12 methods each or whatever logical split makes sense should be enough for now - i.e. splitting in two is ~half the overall test time and that might be enough at least to start with.
- last update
about 1 year ago Custom Commands Failed - @catch opened merge request.
- Status changed to Needs review
about 1 year ago 11:01pm 25 September 2023 - last update
about 1 year ago Custom Commands Failed - last update
about 1 year ago 30,362 pass, 1 fail - last update
about 1 year ago 30,357 pass - last update
about 1 year ago Custom Commands Failed - last update
about 1 year ago 30,363 pass - Status changed to RTBC
about 1 year ago 2:51pm 28 September 2023 - 🇺🇸United States smustgrave
THe breakup seems fine. Naming seems good too.
- last update
about 1 year ago 30,365 pass - First commit to issue fork.
- last update
about 1 year ago 30,361 pass - last update
about 1 year ago 30,361 pass - Status changed to Fixed
about 1 year ago 10:08am 29 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.