Skip to content
Snippets Groups Projects
Unverified Commit 11c46fc2 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3027998 by robpowell, jofitz, codekarate, heddn: Default Value... 

Issue #3027998 by robpowell, jofitz, codekarate, heddn: Default Value Migration Process plugin produces unexpected behavior when updating list field from D7 to D8
parent 54d12c13
No related branches found
No related tags found
No related merge requests found
Hide whitespace changes
Inline Side-by-side
core/modules/migrate/src/Plugin/migrate/process/DefaultValue.php
+ 2
− 1
View file @ 11c46fc2
......@@ -41,7 +41,8 @@
* @see \Drupal\migrate\Plugin\MigrateProcessInterface
*
* @MigrateProcessPlugin(
* id = "default_value"
* id = "default_value",
* handle_multiples = TRUE
* )
*/
class DefaultValue extends ProcessPluginBase {
......
core/modules/migrate/tests/src/Unit/process/DefaultValueTest.php 0 → 100644
+ 143
− 0
View file @ 11c46fc2
<?php
namespace Drupal\Tests\migrate\Unit\process;
use Drupal\migrate\Plugin\migrate\process\DefaultValue;
/**
* Tests the default_value process plugin.
*
* @group migrate
* @coversDefaultClass \Drupal\migrate\Plugin\migrate\process\DefaultValue
*/
class DefaultValueTest extends MigrateProcessTestCase {
/**
* Tests the default_value process plugin.
*
* @covers ::transform
*
* @dataProvider defaultValueDataProvider
*/
public function testDefaultValue($configuration, $expected_value, $value) {
$process = new DefaultValue($configuration, 'default_value', []);
$value = $process->transform($value, $this->migrateExecutable, $this->row, 'destinationproperty');
$this->assertSame($expected_value, $value);
}
/**
* Provides data for the successful lookup test.
*
* @return array
*/
public function defaultValueDataProvider() {
return [
'strict_true_value_populated_array' => [
'configuration' => [
'strict' => TRUE,
'default_value' => 1,
],
'expected_value' => [0, 1, 2],
'value' => [0, 1, 2],
],
'strict_true_value_empty_string' => [
'configuration' => [
'strict' => TRUE,
'default_value' => 1,
],
'expected_value' => '',
'value' => '',
],
'strict_true_value_false' => [
'configuration' => [
'strict' => TRUE,
'default_value' => 1,
],
'expected_value' => FALSE,
'value' => FALSE,
],
'strict_true_value_null' => [
'configuration' => [
'strict' => TRUE,
'default_value' => 1,
],
'expected_value' => 1,
'value' => NULL,
],
'strict_true_value_zero_string' => [
'configuration' => [
'strict' => TRUE,
'default_value' => 1,
],
'expected_value' => '0',
'value' => '0',
],
'strict_true_value_zero' => [
'configuration' => [
'strict' => TRUE,
'default_value' => 1,
],
'expected_value' => 0,
'value' => 0,
],
'strict_true_value_empty_array' => [
'configuration' => [
'strict' => TRUE,
'default_value' => 1,
],
'expected_value' => [],
'value' => [],
],
'array_populated' => [
'configuration' => [
'default_value' => 1,
],
'expected_value' => [0, 1, 2],
'value' => [0, 1, 2],
],
'empty_string' => [
'configuration' => [
'default_value' => 1,
],
'expected_value' => 1,
'value' => '',
],
'false' => [
'configuration' => [
'default_value' => 1,
],
'expected_value' => 1,
'value' => FALSE,
],
'null' => [
'configuration' => [
'default_value' => 1,
],
'expected_value' => 1,
'value' => NULL,
],
'string_zero' => [
'configuration' => [
'default_value' => 1,
],
'expected_value' => 1,
'value' => '0',
],
'int_zero' => [
'configuration' => [
'default_value' => 1,
],
'expected_value' => 1,
'value' => 0,
],
'empty_array' => [
'configuration' => [
'default_value' => 1,
],
'expected_value' => 1,
'value' => [],
],
];
}
}
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment