Nove MigrateUpgradeExecuteTestBase expectedLoggedErrors property to

Whoops, typos / incomplete title

Kindly review MR !9746

Since this is a test base think assertLogError should be deprecated instead.

I think we should look at the usages before asking for the work to add a deprecation.

This change is for a base class that is used to test a migration via the UI. I know of only one contrib module that has written tests for that scenario, commerce_migrate. And a search shows that the only usages of the base class in contrib are in former core modules and, of course in, commerce_migrate. So, I think a deprecation is not needed.

I agree with #7 that this isn't an API per-se so a BC layer is overkill.

However we should be considerate and add a change record so someone who finds their contrib tests broken can search for it and find the action to take.

Thanks folks

The Migrate Drupal Module was approved for removal in 📌 [Policy] Migrate Drupal and Migrate Drupal UI after Drupal 7 EOL Fixed .

This is Postponed. The status is set according to two policies. The Remove a core extension and move it to a contributed project → and the Extensions approved for removal → policies.

The deprecation work is in 📌 [12.x] [Meta] Tasks to remove Migrate Drupal module Postponed and the removal work in 📌 [12.x] [Meta] Tasks to remove Migrate Drupal module Postponed .

Migrate Drupal will not be moved to a contributed project. It will be removed from core after the Drupal 12.x branch is open.