Skip to content

Commit abba8c0

Browse files
committed
[TASK] Use Declaration::getPropertyName() in tests
This method replaces `getRule()`, which is deprecated as of #1506.
1 parent 567e390 commit abba8c0

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

src/RuleSet/RuleSet.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ public static function parseRuleSet(ParserState $parserState, RuleSet $ruleSet):
101101
*/
102102
public function addRule(Declaration $declarationToAdd, ?Declaration $sibling = null): void
103103
{
104-
$propertyName = $declarationToAdd->getRule();
104+
$propertyName = $declarationToAdd->getPropertyName();
105105
if (!isset($this->declarations[$propertyName])) {
106106
$this->declarations[$propertyName] = [];
107107
}
@@ -240,7 +240,7 @@ public function getRulesAssoc(?string $searchPattern = null): array
240240
/** @var array<string, Declaration> $result */
241241
$result = [];
242242
foreach ($this->getRules($searchPattern) as $declaration) {
243-
$result[$declaration->getRule()] = $declaration;
243+
$result[$declaration->getPropertyName()] = $declaration;
244244
}
245245

246246
return $result;
@@ -251,7 +251,7 @@ public function getRulesAssoc(?string $searchPattern = null): array
251251
*/
252252
public function removeRule(Declaration $declarationToRemove): void
253253
{
254-
$nameOfPropertyToRemove = $declarationToRemove->getRule();
254+
$nameOfPropertyToRemove = $declarationToRemove->getPropertyName();
255255
if (!isset($this->declarations[$nameOfPropertyToRemove])) {
256256
return;
257257
}

tests/ParserTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -344,8 +344,8 @@ public function ruleGetters(): void
344344
$bodyBlock = $declarationBlocks[1];
345345
$backgroundHeaderRules = $headerBlock->getRules('background-');
346346
self::assertCount(2, $backgroundHeaderRules);
347-
self::assertSame('background-color', $backgroundHeaderRules[0]->getRule());
348-
self::assertSame('background-color', $backgroundHeaderRules[1]->getRule());
347+
self::assertSame('background-color', $backgroundHeaderRules[0]->getPropertyName());
348+
self::assertSame('background-color', $backgroundHeaderRules[1]->getPropertyName());
349349
$backgroundHeaderRules = $headerBlock->getRulesAssoc('background-');
350350
self::assertCount(1, $backgroundHeaderRules);
351351
self::assertInstanceOf(Color::class, $backgroundHeaderRules['background-color']->getValue());
@@ -886,7 +886,7 @@ public function missingPropertyValueLenient(): void
886886
$rules = $block->getRules();
887887
self::assertCount(1, $rules);
888888
$rule = $rules[0];
889-
self::assertSame('display', $rule->getRule());
889+
self::assertSame('display', $rule->getPropertyName());
890890
self::assertSame('inline-block', $rule->getValue());
891891
}
892892

tests/RuleSet/DeclarationBlockTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public function overrideRules(): void
3636

3737
$rules = $wrapper->getRules();
3838
self::assertCount(1, $rules);
39-
self::assertSame('right', $rules[0]->getRule());
39+
self::assertSame('right', $rules[0]->getPropertyName());
4040
self::assertSame('-10px', $rules[0]->getValue());
4141
}
4242

0 commit comments

Comments
 (0)