Skip to content

Commit d856763

Browse files
authored
Merge pull request #235 from mglaman/gh234
BrowserTestBaseDefaultThemeRule false positives with UpdatePathTests
2 parents 7c29b30 + 3642cef commit d856763

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

src/Rules/Drupal/Tests/BrowserTestBaseDefaultThemeRule.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,10 @@ public function processNode(Node $node, Scope $scope): array
3232
if ($browserTestBaseAncestor === null) {
3333
return [];
3434
}
35+
$browserTestBaseAncestor = $classType->getAncestorWithClassName('Drupal\\FunctionalTests\\Update\\UpdatePathTestBase');
36+
if ($browserTestBaseAncestor !== null) {
37+
return [];
38+
}
3539

3640
$reflection = $classType->getClassReflection();
3741
assert($reflection !== null);

tests/src/Rules/BrowserTestBaseDefaultThemeRuleTest.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,10 @@ public function fileData(): \Generator
5353
__DIR__ . '/../../fixtures/drupal/core/modules/views/tests/src/Functional/ViewTestBase.php',
5454
[]
5555
];
56+
yield [
57+
__DIR__ . '/../../fixtures/drupal/core/tests/Drupal/FunctionalTests/Update/UpdatePathTestBaseTest.php',
58+
[]
59+
];
5660
}
5761

5862
}

0 commit comments

Comments
 (0)