- Issue created by @catch
- Status changed to Needs review
3 months ago 1:40am 22 August 2024 - 🇬🇧United Kingdom catch
Last 11.x daily test run had this at 57s https://git.drupalcode.org/project/drupal/-/jobs/2502553
This goes down from around 8s on the MR pipeline gitlab https://git.drupalcode.org/project/drupal/-/jobs/2512911
- Status changed to RTBC
3 months ago 4:23am 22 August 2024 Came from #3390193 📌 Add drupalGet() to KernelTestBase Needs review , however this issue is already in NR, so just reviewing it
Changes looks good: used protected from public along with relevant changes included to call those protected methods, apart from nothing seems to be left, verified test changes on local, its output on the basis of time & memory usage as follow:
Before applied MR changes test takes : Time:00:21.007,
Memory: 6.00 MBAfter applied MR changes test takes: Time:
00:04.700,
Memory: 6.00 MBobserved test & its resources usage improved & attached screenshot as well, moving to RTBC
- Status changed to Fixed
3 months ago 8:16am 22 August 2024 - 🇫🇷France nod_ Lille
Committed and pushed 6aae927629 to 11.x and 1d5a6d44d6 to 11.0.x and 6adaeadd8f to 10.4.x and 20f60160dd to 10.3.x. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.