only in patch2: unchanged: --- a/core/modules/migrate/tests/src/Unit/MigrateSqlSourceTestCase.php +++ b/core/modules/migrate/tests/src/Unit/MigrateSqlSourceTestCase.php @@ -122,6 +122,13 @@ public function testSourceCount() { } /** + * Test the source defines a valid ID. + */ + public function testSourceId() { + $this->assertNotEmpty($this->source->getIds()); + } + + /** * @param \Drupal\migrate\Row $row * @param string $key * @return mixed only in patch2: unchanged: --- a/core/modules/migrate/tests/src/Unit/destination/ConfigTest.php +++ b/core/modules/migrate/tests/src/Unit/destination/ConfigTest.php @@ -37,6 +37,9 @@ public function testImport() { } $config->expects($this->once()) ->method('save'); + $config->expects($this->once()) + ->method('getName') + ->willReturn('d8_config'); $config_factory = $this->getMock('Drupal\Core\Config\ConfigFactoryInterface'); $config_factory->expects($this->once()) ->method('getEditable') @@ -49,7 +52,8 @@ public function testImport() { ->method('getRawDestination') ->will($this->returnValue($source)); $destination = new Config(array('config_name' => 'd8_config'), 'd8_config', array('pluginId' => 'd8_config'), $migration, $config_factory); - $destination->import($row); + $destination_id = $destination->import($row); + $this->assertEquals($destination_id, ['d8_config']); } } only in patch2: unchanged: --- a/core/modules/migrate_drupal/tests/src/Unit/source/VariableTest.php +++ b/core/modules/migrate_drupal/tests/src/Unit/source/VariableTest.php @@ -32,6 +32,7 @@ class VariableTest extends MigrateSqlSourceTestCase { protected $expectedResults = array( array( + 'id' => 'foo', 'foo' => 1, 'bar' => FALSE, ),