Convert all test modules to use OOP Hook implementations.

Running rector got these errors:

[ERROR] Could not process
"/var/www/html/core/modules/block/tests/src/Functional/BlockTest.php" file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\block\Functional\BlockTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/block/tests/src/Functional/BlockUiTest.php" file, due
to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\block\Functional\BlockUiTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/block/tests/src/Functional/BlockXssTest.php" file, due
to:
"System error: "Method assertNoEscaped() was not found in reflection of class
Drupal\Tests\block\Functional\BlockXssTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/block/tests/src/Functional/Views/DisplayBlockTest.php"
file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\block\Functional\Views\DisplayBlockTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/block/tests/src/FunctionalJavascript/BlockDragTest.php"
file, due to:
"System error: "Method assert() was not found in reflection of class
Drupal\Tests\block\FunctionalJavascript\BlockDragTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/ckeditor5/tests/src/FunctionalJavascript/CKEditor5TestB
ase.php" file, due to:
"System error: "Method assert() was not found in reflection of class
Drupal\Tests\ckeditor5\FunctionalJavascript\CKEditor5TestBase."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/comment/tests/src/Functional/CommentPreviewTest.php"
file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\comment\Functional\CommentPreviewTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/config_translation/tests/src/Functional/ConfigTranslati
onUiTestBase.php" file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\config_translation\Functional\ConfigTranslationUiTestBase."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/config_translation/tests/src/Functional/ConfigTranslati
onCacheTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\config_translation\Functional\ConfigTranslationCacheTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/config_translation/tests/src/Functional/ConfigTranslati
onOverviewTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\config_translation\Functional\ConfigTranslationOverviewTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/config_translation/tests/src/Functional/ConfigTranslati
onUiModulesTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\config_translation\Functional\ConfigTranslationUiModulesTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/contact/tests/src/Functional/ContactPersonalTest.php"
file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\contact\Functional\ContactPersonalTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/contact/tests/src/Functional/ContactSitewideTest.php"
file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\contact\Functional\ContactSitewideTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/contextual/tests/src/Functional/ContextualDynamicContex
tTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\contextual\Functional\ContextualDynamicContextTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/field/tests/src/Functional/FormTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\field\Functional\FormTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/field/tests/src/Functional/MultipleWidgetFormTest.php"
file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\field\Functional\MultipleWidgetFormTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/field/tests/src/Functional/EntityReference/EntityRefere
nceXSSTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\field\Functional\EntityReference\EntityReferenceXSSTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/field_ui/tests/src/Traits/FieldUiJSTestTrait.php" file,
due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\field_ui\Traits\FieldUiJSTestTrait."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/field_ui/tests/src/Traits/FieldUiTestTrait.php" file,
due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\field_ui\Traits\FieldUiTestTrait."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/filter/tests/src/Functional/FilterAdminTest.php" file,
due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\filter\Functional\FilterAdminTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/help/tests/src/Functional/HelpTest.php" file, due to:
"System error: "Method assertNoEscaped() was not found in reflection of class
Drupal\Tests\help\Functional\HelpTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/field_ui/tests/src/Functional/ManageDisplayTest.php"
file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\field_ui\Functional\ManageDisplayTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/field_ui/tests/src/Functional/ManageFieldsLifecycleTest
.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\field_ui\Functional\ManageFieldsLifecycleTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/locale/tests/src/Functional/LocaleContentTest.php"
file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\locale\Functional\LocaleContentTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/node/tests/src/Functional/NodeTranslationUITest.php"
file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\node\Functional\NodeTranslationUITest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/node/tests/src/Functional/PagePreviewTest.php" file,
due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\node\Functional\PagePreviewTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/node/tests/src/Functional/NodeRevisionsUiTest.php"
file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\node\Functional\NodeRevisionsUiTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/node/tests/src/Functional/Views/NodeIntegrationTest.php
" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\node\Functional\Views\NodeIntegrationTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/search/tests/src/Functional/SearchPageTextTest.php"
file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\search\Functional\SearchPageTextTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/search/tests/src/Functional/SearchCommentTest.php"
file, due to:
"System error: "Method assertNoEscaped() was not found in reflection of class
Drupal\Tests\search\Functional\SearchCommentTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/Batch/ProcessingTest.php"
file, due to:
"System error: "Method assertNoEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\Batch\ProcessingTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/Bootstrap/DrupalMessengerSe
rviceTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\Bootstrap\DrupalMessengerServiceTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/Form/FormTest.php" file,
due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\Form\FormTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/Menu/BreadcrumbTest.php"
file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\Menu\BreadcrumbTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/Menu/MenuAccessTest.php"
file, due to:
"System error: "Method assert() was not found in reflection of class
Drupal\Tests\system\Functional\Menu\MenuAccessTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/Menu/MenuLinkSecurityTest.p
hp" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\Menu\MenuLinkSecurityTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/Menu/MenuRouterTest.php"
file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\Menu\MenuRouterTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/System/DateTimeTest.php"
file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\System\DateTimeTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/FunctionalJavascript/Batch/ProcessingT
est.php" file, due to:
"System error: "Method assertNoEscaped() was not found in reflection of class
Drupal\Tests\system\FunctionalJavascript\Batch\ProcessingTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/FunctionalJavascript/System/DateFormat
Test.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\FunctionalJavascript\System\DateFormatTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/File/FileSaveHtaccessLoggin
gTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\File\FileSaveHtaccessLoggingTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/UpdateSystem/UpdatePostUpda
teExceptionTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\UpdateSystem\UpdatePostUpdateExceptionTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/UpdateSystem/UpdatePostUpda
teFailingTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\UpdateSystem\UpdatePostUpdateFailingTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/system/tests/src/Functional/UpdateSystem/UpdateScriptTe
st.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\system\Functional\UpdateSystem\UpdateScriptTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/taxonomy/tests/src/Functional/TermTest.php" file, due
to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\taxonomy\Functional\TermTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/taxonomy/tests/src/Functional/Views/TaxonomyDefaultArgu
mentTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\taxonomy\Functional\Views\TaxonomyDefaultArgumentTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/taxonomy/tests/src/Functional/Views/TaxonomyFieldAllTer
msTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\taxonomy\Functional\Views\TaxonomyFieldAllTermsTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/taxonomy/tests/src/Functional/Views/TaxonomyIndexTidUiT
est.php" file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\taxonomy\Functional\Views\TaxonomyIndexTidUiTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/taxonomy/tests/src/Functional/Views/TaxonomyTermArgumen
tDepthTest.php" file, due to:
"System error: "Method assertEscaped() was not found in reflection of class
Drupal\Tests\taxonomy\Functional\Views\TaxonomyTermArgumentDepthTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

[ERROR] Could not process
"/var/www/html/core/modules/taxonomy/tests/src/Functional/VocabularyUiTest.php"
file, due to:
"System error: "Method buildXPathQuery() was not found in reflection of class
Drupal\Tests\taxonomy\Functional\VocabularyUiTest."
Run Rector with "--debug" option and post the report here: https://github.com/rectorphp/rector/issues/new". On line: 553

Ok running this against /core/tests hangs, so I ran this against just core/modules manually:

./vendor/bin/rector \
--config ./vendor/palantirnet/drupal-rector/rector.php \
process \
core/modules/announcements_feed/tests/modules \
core/modules/automated_cron/tests/modules \
core/modules/ban/tests/modules \
core/modules/basic_auth/tests/modules \
core/modules/big_pipe/tests/modules \
core/modules/block/tests/modules \
core/modules/block_content/tests/modules \
core/modules/breakpoint/tests/modules \
core/modules/ckeditor5/tests/modules \
core/modules/comment/tests/modules \
core/modules/config/tests/modules \
core/modules/config_translation/tests/modules \
core/modules/contact/tests/modules \
core/modules/content_moderation/tests/modules \
core/modules/content_translation/tests/modules \
core/modules/contextual/tests/modules \
core/modules/datetime/tests/modules \
core/modules/datetime_range/tests/modules \
core/modules/dblog/tests/modules \
core/modules/dynamic_page_cache/tests/modules \
core/modules/editor/tests/modules \
core/modules/field/tests/modules \
core/modules/field_layout/tests/modules \
core/modules/field_ui/tests/modules \
core/modules/file/tests/modules \
core/modules/filter/tests/modules \
core/modules/help/tests/modules \
core/modules/history/tests/modules \
core/modules/image/tests/modules \
core/modules/inline_form_errors/tests/modules \
core/modules/jsonapi/tests/modules \
core/modules/language/tests/modules \
core/modules/layout_builder/tests/modules \
core/modules/layout_discovery/tests/modules \
core/modules/link/tests/modules \
core/modules/locale/tests/modules \
core/modules/media/tests/modules \
core/modules/media_library/tests/modules \
core/modules/menu_link_content/tests/modules \
core/modules/menu_ui/tests/modules \
core/modules/migrate/tests/modules \
core/modules/migrate_drupal/tests/modules \
core/modules/migrate_drupal_ui/tests/modules \
core/modules/mysql/tests/modules \
core/modules/navigation/tests/modules \
core/modules/node/tests/modules \
core/modules/options/tests/modules \
core/modules/page_cache/tests/modules \
core/modules/path/tests/modules \
core/modules/path_alias/tests/modules \
core/modules/pgsql/tests/modules \
core/modules/phpass/tests/modules \
core/modules/responsive_image/tests/modules \
core/modules/rest/tests/modules \
core/modules/sdc/tests/modules \
core/modules/search/tests/modules \
core/modules/serialization/tests/modules \
core/modules/settings_tray/tests/modules \
core/modules/shortcut/tests/modules \
core/modules/sqlite/tests/modules \
core/modules/syslog/tests/modules \
core/modules/system/tests/modules \
core/modules/taxonomy/tests/modules \
core/modules/telephone/tests/modules \
core/modules/text/tests/modules \
core/modules/toolbar/tests/modules \
core/modules/update/tests/modules \
core/modules/user/tests/modules \
core/modules/views/tests/modules \
core/modules/views_ui/tests/modules \
core/modules/workflows/tests/modules \
core/modules/workspaces/tests/modules

To give better history I'm going to commit after each stage.

Modules that did not convert:
ckeditor5_drupalelementstyle_test
hook_jsonapi_entity_field_field_access

Test modules that do not follow the structure I ran with:
config
dynamic_page_cache
file_module
language
navigation
options

I've deleted the empty .module folders, I'm not rerunning rector right now against the missing test modules.

Some hooks have been renamed incorrectly, that is why there are so many failures.

E.g. looking at the first failure in kernel tests: EndOfTransactionQueriesTest.php
It installs the test module: delay_cache_tags_invalidation
If we look at the hooks there: core/modules/system/tests/modules/delay_cache_tags_invalidation/src/Hook/DelayCacheTagsInvalidationHooks.php

  /**
   * Implements hook_ENTITY_TYPE_insert().
   */
  #[Hook('entity_test_insert')]

This is the original

/**
 * Implements hook_ENTITY_TYPE_insert().
 */
function delay_cache_tags_invalidation_entity_test_insert(EntityTest $entity) {

So that looks correct.
This test also has entity_test installed
A look at core/modules/system/tests/modules/entity_test/src/Hook/EntityTestHooks.php
and core/modules/system/tests/modules/entity_test/src/Hook/EntityTestViewsHooks.php
both look correct too.

That specific hook has:

> \Drupal::state()->set(__FUNCTION__ . '__pretransaction_foobar', \Drupal::cache()->get('test_cache_pretransaction_foobar'));

I assume the hook is called just fine, but __FUNCTION__ inside the new class is going to have a different value than what it was before.

That's probably it, let's see if converting to __METHOD__ and updating the sets and calls to the new names is enough.

This failed again, I hardcoded the set, why hardcode the value for the test. but not hardcode it for the set in the test?

__FUNCTION__ or __METHOD__ should be the same when called from the same place, this is passing locally so I pushed it up.

We should be able to add a Rector rule that replaces __FUNCTION__ calls with the static name.

we have a script to clean up the module files and .inc files after:

find "$1" -name \*.module| xargs grep -LE "^function|^CONST" | xargs ls; git checkout HEAD core/tests/fixtures/empty_file.php.module
find "$1" -name \*.inc| xargs grep -LE "^function|^CONST" | xargs ls
Change the ls to rm when ready.

The script is great.

The __FUNCTION__ handling that was added solved some failures but not all.

I think the ones that remain will be easier to track down.

Looking through the failures some are due to the test function hooks no longer existing since they are now methods on classes. I fixed the first two obvious ones, after that I stopped being able to reproduce the failures locally.

I'm pushing up to retest. I'm also attaching a patch with the manual fixes I applied.

core/modules/system/tests/src/Functional/UpdateSystem/RebuildScriptTest.php

This needed the function_exists check to be an implementation check.

I've finished reviewing all of the failures.

I've created patches for the easy ones.
Several require discussion and follow up issues.
There were a few I could not track down when timeboxed for the review.

There are several test modules not converted because they did not follow the MODULE/test/modules structure, those are noted above.

cspell issue

I finished converting the rest of the test modules.

The patches have not made their way up yet but I think all of the failures but core/modules/system/tests/src/Functional/System/HtaccessTest.php

Have identified fixes.

I still need to test the full conversion and see what failed pop up there. But we're actually getting close.

Ok I think we've fixed the last of the tests identified here, I'm going to attach the last patches.

Now that we have these, I'm going to reconvert 📌 [ignore] Convert everything everywhere all at once Active then apply these patches and see if anything else needs work.

CommonAlterTest patch will likely have conflicts with the new conversion, I'll have to regen it.

Had to regenerate entity schema and common alter test.

In ✨ Add an API for importmaps Active I tried to use the new OO hook approach for the alter hook I'm using in tests there and found it wasn't being picked up. Could be I was doing something wrong, but ended up going back to a .module file. Would appreciate a second set of eyes if anyone has bandwidth - to save introducing a new one

I don't see where you were adding it, was it the one in Core/Assets? it has to be in a module and in the Hooks namespace, if you have a snippet or commit I can look at I will.

Adding ignore here too for now: most work is happening in 📌 [ignore] Convert everything everywhere all at once Active