The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Problem/Motivation
https://git.drupalcode.org/project/drupal/-/jobs/162191
Drupal\Tests\Core\Command\GenerateThemeTest::testGeneratingFromAnotherTheme
Symfony\Component\Finder\Exception\AccessDeniedException: RecursiveDirectoryIterator::__construct(/builds/project/drupal/sites/simpletest/21880381/files/php/twig/6526592c2580b_node.editing.html.twig__RaNgLtlrqSCHFKI6weeKQxEy): Failed to open directory: No such file or directory
/builds/project/drupal/vendor/symfony/finder/Iterator/RecursiveDirectoryIterator.php:111
/builds/project/drupal/vendor/symfony/filesystem/Filesystem.php:569
/builds/project/drupal/core/tests/Drupal/BuildTests/Framework/BuildTestBase.php:559
/builds/project/drupal/core/tests/Drupal/Tests/Core/Command/GenerateThemeTest.php:38
/builds/project/drupal/vendor/phpunit/phpunit/src/Framework/TestResult.php:728
/builds/project/drupal/vendor/phpunit/phpunit/src/Framework/TestSuite.php:684
/builds/project/drupal/vendor/phpunit/phpunit/src/TextUI/TestRunner.php:651
/builds/project/drupal/vendor/phpunit/phpunit/src/TextUI/Command.php:144
/builds/project/drupal/vendor/phpunit/phpunit/src/TextUI/Command.php:97
Steps to reproduce
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 200x_GenerateThemeTest-10conc-no-node-modules.patch | 3.16 KB | spokje |
Comments
Comment #2
catchComment #3
catchComment #4
catchComment #5
catchComment #6
spokjeInteresting stuff going on in the multiple-time patches.
For one: Concurrency kills the cat, or at least the available disk-space, it can't handle 36 concurrent instances (the default Drupal CI-setting) copying the code-base for each single test in this test class, and runs out of it.
Secondly,
looks like we're copying the
core/node_modules, which isn't needed nor helpful IMHO. Opened #3396574: Exclude copying 'core/node_modules' in getCodebaseFinderThirdly: Looking at the trace from the IS:
Why is the test trying to copy something from a directory that we explicitly (tried to?) excluded in the Iterator:
->notPath('#^sites/simpletest#')?With #3396574: Exclude copying 'core/node_modules' in getCodebaseFinder and a concurrency of 10 we can just about squeeze a 200x run out of this test.
Comment #7
spokjeWell, besides 200x being just around the limit (hence the massive amount of reruns due to DrupalCI timing out), there's just one failure in a teardown, which seems like a fluke.
Which is also what I'm going to label the original failure. Either or fluke or the FileIterator works differently on GitLab, which seems very unlikely.
With #3396574: Exclude copying 'core/node_modules' in getCodebaseFinder, I've squeezed all out of this one that I can think off.
Comment #8
catchI've committed #3396574: Exclude copying 'core/node_modules' in getCodebaseFinder. Moving this down to normal and needs more info until or if it shows up again.
Comment #9
spokjeThanks @catch, let's see if this one shows up ever again.
Comment #10
quietone commentedThere hasn't been any reports of this test failing here or in the parent issue in the last 1 yr 7 months. I think this can be closed.
Closing as a duplicate of #3396574: Exclude copying 'core/node_modules' in getCodebaseFinder.