Commit ac7e61ef authored by alexpott's avatar alexpott

Issue #2293807 by sun: Fixed Various migrate_drupal unit base test classes are...

Issue #2293807 by sun: Fixed Various migrate_drupal unit base test classes are not abstract and implement getInfo().
parent b4f282a5
......@@ -11,7 +11,7 @@
use Drupal\migrate\Source;
/**
* Provides setup and helper methods for Migrate module source tests.
* Base class for Migrate module source unit tests.
*/
abstract class MigrateSqlSourceTestCase extends MigrateTestCase {
......@@ -112,17 +112,6 @@ protected function getValue($row, $key) {
return $row->getSourceProperty($key);
}
/**
* {@inheritdoc}
*/
public static function getInfo() {
return array(
'name' => 'SQL source test',
'description' => 'Tests for SQL source plugin.',
'group' => 'Migrate',
);
}
}
class TestSource extends Source {
......
......@@ -15,4 +15,15 @@
*/
class VariableMultiRowTest extends VariableMultiRowTestBase {
/**
* {@inheritdoc}
*/
public static function getInfo() {
return array(
'name' => 'D6 variable multirow source functionality',
'description' => 'Tests D6 variable multirow source plugin.',
'group' => 'Migrate Drupal',
);
}
}
......@@ -10,9 +10,9 @@
use Drupal\migrate\Tests\MigrateSqlSourceTestCase;
/**
* Base test class for the variable multirow source tests.
* Base class for variable multirow source unit tests.
*/
class VariableMultiRowTestBase extends MigrateSqlSourceTestCase {
abstract class VariableMultiRowTestBase extends MigrateSqlSourceTestCase {
// The plugin system is not working during unit testing so the source plugin
// class needs to be manually specified.
......@@ -43,16 +43,6 @@ class VariableMultiRowTestBase extends MigrateSqlSourceTestCase {
),
);
/**
* {@inheritdoc}
*/
public static function getInfo() {
return array(
'name' => 'D6 variable multirow source functionality',
'description' => 'Tests D6 variable multirow source plugin.',
'group' => 'Migrate Drupal',
);
}
}
namespace Drupal\migrate_drupal\Tests\source;
......
......@@ -15,4 +15,15 @@
*/
class CommentTest extends CommentTestBase {
/**
* {@inheritdoc}
*/
public static function getInfo() {
return array(
'name' => 'D6 comment source functionality',
'description' => 'Tests D6 comment source plugin.',
'group' => 'Migrate Drupal',
);
}
}
......@@ -10,9 +10,9 @@
use Drupal\migrate\Tests\MigrateSqlSourceTestCase;
/**
* Base class for the comment source tests.
* Base class for comment source unit tests.
*/
class CommentTestBase extends MigrateSqlSourceTestCase {
abstract class CommentTestBase extends MigrateSqlSourceTestCase {
// The plugin system is not working during unit testing so the source plugin
// class needs to be manually specified.
......@@ -68,17 +68,6 @@ class CommentTestBase extends MigrateSqlSourceTestCase {
),
);
/**
* {@inheritdoc}
*/
public static function getInfo() {
return array(
'name' => 'D6 comment source functionality',
'description' => 'Tests D6 comment source plugin.',
'group' => 'Migrate Drupal',
);
}
/**
* {@inheritdoc}
*/
......
......@@ -15,4 +15,15 @@
*/
class TermTest extends TermTestBase {
/**
* {@inheritdoc}
*/
public static function getInfo() {
return array(
'name' => 'D6 taxonomy term source functionality',
'description' => 'Tests D6 taxonomy term source plugin.',
'group' => 'Migrate Drupal',
);
}
}
......@@ -10,9 +10,9 @@
use Drupal\migrate\Tests\MigrateSqlSourceTestCase;
/**
* Base class for the term source tests.
* Base class for taxonomy term source unit tests.
*/
class TermTestBase extends MigrateSqlSourceTestCase {
abstract class TermTestBase extends MigrateSqlSourceTestCase {
const PLUGIN_CLASS = 'Drupal\migrate_drupal\Plugin\migrate\source\d6\Term';
......@@ -76,17 +76,6 @@ class TermTestBase extends MigrateSqlSourceTestCase {
),
);
/**
* {@inheritdoc}
*/
public static function getInfo() {
return array(
'name' => 'D6 taxonomy term source functionality',
'description' => 'Tests D6 taxonomy term source plugin.',
'group' => 'Migrate Drupal',
);
}
/**
* {@inheritdoc}
*/
......
Markdown is supported
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