Skip to content
Snippets Groups Projects
Commit 605e0a8d authored by catch's avatar catch
Browse files

Issue #3488834 by quietone, smustgrave: Remove Ban module from migration tests not in the module

parent be33a859
No related branches found
No related tags found
3 merge requests!11197Issue #3506427 by eduardo morales alberti: Remove responsive_image.ajax from hook,!10223132456: Fix issue where views instances are emptied before an ajax request is complete,!617Issue #3043725: Provide a Entity Handler for user cancelation
Pipeline #403733 passed with warnings
Pipeline: drupal

#403742

    Pipeline: drupal

    #403737

      Pipeline: drupal

      #403734

        name: 'Migration plugin configuration rest'
        type: module
        description: 'Provides test migrations to test altering configuration.'
        package: Testing
        version: VERSION
        id: simple_migration
        label: Simple migration test
        source:
        plugin: simple_source
        process:
        foo: foo
        destination:
        plugin: null
        <?php
        declare(strict_types=1);
        namespace Plugin\migrate\source;
        use Drupal\migrate\Plugin\migrate\source\SqlBase;
        use Drupal\Core\Database\Query\SelectInterface;
        /**
        * Simple source for testing changing configuration.
        *
        * @MigrateSource(
        * id = "simple_source",
        * )
        */
        class SimpleSource extends SqlBase {
        /**
        * {@inheritdoc}
        */
        public function query(): SelectInterface {
        return $this->select('source_table', 's')->fields('s', ['id']);
        }
        /**
        * {@inheritdoc}
        */
        public function fields(): array {
        return [
        'foo' => $this->t('Test field.'),
        ];
        }
        /**
        * {@inheritdoc}
        */
        public function getIds(): array {
        return ['foo' => ['type' => 'string']];
        }
        }
        ...@@ -21,7 +21,7 @@ class MigrationPluginConfigurationTest extends KernelTestBase { ...@@ -21,7 +21,7 @@ class MigrationPluginConfigurationTest extends KernelTestBase {
        'migrate', 'migrate',
        'migrate_drupal', 'migrate_drupal',
        // Test with a simple migration. // Test with a simple migration.
        'ban', 'migrate_plugin_config_test',
        'locale', 'locale',
        ]; ];
        ...@@ -46,7 +46,7 @@ public static function mergeProvider() { ...@@ -46,7 +46,7 @@ public static function mergeProvider() {
        // Tests adding new configuration to a migration. // Tests adding new configuration to a migration.
        [ [
        // New configuration. // New configuration.
        'd7_blocked_ips', 'simple_migration',
        [ [
        'source' => [ 'source' => [
        'constants' => [ 'constants' => [
        ...@@ -56,7 +56,7 @@ public static function mergeProvider() { ...@@ -56,7 +56,7 @@ public static function mergeProvider() {
        ], ],
        // Expected final source configuration. // Expected final source configuration.
        [ [
        'plugin' => 'd7_blocked_ips', 'plugin' => 'simple_source',
        'constants' => [ 'constants' => [
        'added_setting' => 'Ban them all!', 'added_setting' => 'Ban them all!',
        ], ],
        ...@@ -65,7 +65,7 @@ public static function mergeProvider() { ...@@ -65,7 +65,7 @@ public static function mergeProvider() {
        // Tests overriding pre-existing configuration in a migration. // Tests overriding pre-existing configuration in a migration.
        [ [
        // New configuration. // New configuration.
        'd7_blocked_ips', 'simple_migration',
        [ [
        'source' => [ 'source' => [
        'plugin' => 'a_different_plugin', 'plugin' => 'a_different_plugin',
        ......
        ...@@ -27,6 +27,7 @@ class MigrationPluginListTest extends KernelTestBase { ...@@ -27,6 +27,7 @@ class MigrationPluginListTest extends KernelTestBase {
        protected static $modules = [ protected static $modules = [
        'migrate', 'migrate',
        // Test with all modules containing Drupal migrations. // Test with all modules containing Drupal migrations.
        // @todo Remove Ban in https://www.drupal.org/project/drupal/issues/3488827
        'ban', 'ban',
        'block', 'block',
        'block_content', 'block_content',
        ......
        ...@@ -120,6 +120,7 @@ protected function getConfigurationClasses(): array { ...@@ -120,6 +120,7 @@ protected function getConfigurationClasses(): array {
        protected function getContentClasses(): array { protected function getContentClasses(): array {
        return [ return [
        EntityContentBase::class, EntityContentBase::class,
        // @todo Remove BlockedIp in https://www.drupal.org/project/drupal/issues/3488827
        BlockedIp::class, BlockedIp::class,
        UserData::class, UserData::class,
        ]; ];
        ......
        ...@@ -38,6 +38,7 @@ class StateFileExistsTest extends MigrateDrupalTestBase { ...@@ -38,6 +38,7 @@ class StateFileExistsTest extends MigrateDrupalTestBase {
        * @var array * @var array
        */ */
        protected $stateFileRequired = [ protected $stateFileRequired = [
        // @todo Remove ban in https://www.drupal.org/project/drupal/issues/3488827
        'ban', 'ban',
        'block', 'block',
        'block_content', 'block_content',
        ......
        0% Loading or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Finish editing this message first!
        Please register or to comment