UpdatePathTestBaseFilledTest is mostly pointless

Before: https://git.drupalcode.org/project/drupal/-/jobs/668164

After: https://git.drupalcode.org/project/drupal/-/jobs/668355

Potentially a full minute off the job length once a couple of other slow tests that were hidden by this one were added to @group #slow.

Very vice find, good sir!

Changes look OK to me, UpdatePathTestBaseTest will still run on its own, so no coverage loss.

I'm not sure it's *entirely* pointless. I think testDatabaseLoaded() is worth keeping ensure that both the bare and filled dumps aren't corrupt in some way? Was also unsure about testUpdateHookN() but it turns out the module lists are the same so that one does look like repeated work.

I've moved testDatabaseLoaded() to a protected method on the base class, and called it from both of these tests so we don't lose coverage.

MR appears to have phpcs errors

Fixed the cs issues.

Thanks, refactor seems good now.

Backported to 10.2.x to keep things in sync.

Committed and pushed c5e634873a to 11.x and 660b30a0c3 to 10.2.x. Thanks!

longwave → closed merge request !6255

Automatically closed - issue fixed for 2 weeks with no activity.