[wp-trac] [WordPress Trac] #53945: Local test workflow changes for installing Composer dependencies
WordPress Trac
noreply at wordpress.org
Sat Sep 4 00:50:38 UTC 2021
#53945: Local test workflow changes for installing Composer dependencies
------------------------------------------------+--------------------------
Reporter: hellofromTonya | Owner: johnbillion
Type: task (blessed) | Status: reopened
Priority: normal | Milestone: 5.9
Component: Build/Test Tools | Version:
Severity: normal | Resolution:
Keywords: has-patch has-testing-instructions | Focuses:
------------------------------------------------+--------------------------
Comment (by azaozz):
Replying to [comment:32 jrf]:
> The delay happening **after** the "composer fund" to me indicates that
this has nothing to do with Composer as Composer is already finished by
then.
Right, but it is caused by running `composer update -W` that was added in
[51685]. If that is not run, there is no delay.
> If I remember correctly, I believe @johnbillion also mentioned something
about running the tests via Docker being significantly slower
(independently of this change and before this change was even made). He
also mentioned a time difference of ~3 minutes.
Right, it is slower, by about 1.5 min here. As far as I understand this is
caused by running from a Docker container vs. directly in the OS. All of
WP runs a bit slower. That's not the same as the delay after `composer
update` finishes and before `phpunit --group 12345` starts.
> The ~3 minute delay sounds like a 180 second process timeout, so I
wonder whether there is something going on in Docker
Yeah, could be. I'm not that good at debugging Docker, but will try
looking more there.
My other point was that it doesn't make sense to run `composer update -W`
every time when doing `npm run test:php`. When working on tests the same
test case is usually run repeatedly, so checking for Composer updates
20-30 times per hour is not good.
For that reason I think that part of [51685] should be reverted. Perhaps
another method can be found to ensure Composer updates are run from time
to time, or that can be a documented step like with npm updates.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/53945#comment:34>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list