Issue #3250582 by huzooka, Matroskeen, danflanagan8, ravi.shankar, quietone, erik.erskine: ResponsiveImageStyles source plugin must extend DrupalSqlBase

Author: alexpott

modules/responsive_image/src/Plugin/migrate/source/d7/ResponsiveImageStyles.php

@@ -2,8 +2,8 @@
 
 namespace Drupal\responsive_image\Plugin\migrate\source\d7;
 
-use Drupal\migrate\Plugin\migrate\source\SqlBase;
 use Drupal\migrate\Row;
+use Drupal\migrate_drupal\Plugin\migrate\source\DrupalSqlBase;
 
 /**
  * Gets Drupal responsive image styles source from database.
@@ -20,7 +20,7 @@
  *   source_module = "picture"
  * )
  */
-class ResponsiveImageStyles extends SqlBase {
+class ResponsiveImageStyles extends DrupalSqlBase {
 
   /**
    * {@inheritdoc}

modules/responsive_image/tests/src/Kernel/Migrate/d7/MigrateResponsiveImageStylesTest.php

@@ -22,6 +22,11 @@ class MigrateResponsiveImageStylesTest extends MigrateDrupal7TestBase {
    */
   protected function setUp(): void {
     parent::setUp();
+    // Ensure the 'picture' module is enabled in the source.
+    $this->sourceDatabase->update('system')
+      ->condition('name', 'picture')
+      ->fields(['status' => 1])
+      ->execute();
     $this->executeMigrations(['d7_image_styles', 'd7_responsive_image_styles']);
   }
 

modules/responsive_image/tests/src/Kernel/Plugin/migrate/source/d7/ResponsiveImageStylesTest.php

@@ -15,7 +15,10 @@ class ResponsiveImageStylesTest extends MigrateSqlSourceTestBase {
   /**
    * {@inheritdoc}
    */
-  protected static $modules = ['responsive_image'];
+  protected static $modules = [
+    'migrate_drupal',
+    'responsive_image',
+  ];
 
   /**
    * {@inheritdoc}