Skip to content

Commit 24bb671

Browse files
committed
Pass in the line number to the color test so that it never fails
1 parent 61f90b8 commit 24bb671

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

tests/Sabberworm/CSS/ParserTest.php

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -52,27 +52,21 @@ function testColorParsing() {
5252
$this->assertSame('red', $oColor);
5353
$aColorRule = $oRuleSet->getRules('background-');
5454
$oColor = $aColorRule[0]->getValue();
55-
$oColor->setLineNo(0);
56-
$this->assertEquals(array('r' => new Size(35.0, null, true), 'g' => new Size(35.0, null, true), 'b' => new Size(35.0, null, true)), $oColor->getColor());
55+
$this->assertEquals(array('r' => new Size(35.0, null, true, $oColor->getLineNo()), 'g' => new Size(35.0, null, true, $oColor->getLineNo()), 'b' => new Size(35.0, null, true, $oColor->getLineNo())), $oColor->getColor());
5756
$aColorRule = $oRuleSet->getRules('border-color');
5857
$oColor = $aColorRule[0]->getValue();
59-
$oColor->setLineNo(0);
60-
$this->assertEquals(array('r' => new Size(10.0, null, true), 'g' => new Size(100.0, null, true), 'b' => new Size(230.0, null, true)), $oColor->getColor());
58+
$this->assertEquals(array('r' => new Size(10.0, null, true, $oColor->getLineNo()), 'g' => new Size(100.0, null, true, $oColor->getLineNo()), 'b' => new Size(230.0, null, true, $oColor->getLineNo())), $oColor->getColor());
6159
$oColor = $aColorRule[1]->getValue();
62-
$oColor->setLineNo(0);
63-
$this->assertEquals(array('r' => new Size(10.0, null, true), 'g' => new Size(100.0, null, true), 'b' => new Size(231.0, null, true), 'a' => new Size("0000.3", null, true)), $oColor->getColor());
60+
$this->assertEquals(array('r' => new Size(10.0, null, true, $oColor->getLineNo()), 'g' => new Size(100.0, null, true, $oColor->getLineNo()), 'b' => new Size(231.0, null, true, $oColor->getLineNo()), 'a' => new Size("0000.3", null, true, $oColor->getLineNo())), $oColor->getColor());
6461
$aColorRule = $oRuleSet->getRules('outline-color');
6562
$oColor = $aColorRule[0]->getValue();
66-
$oColor->setLineNo(0);
67-
$this->assertEquals(array('r' => new Size(34.0, null, true), 'g' => new Size(34.0, null, true), 'b' => new Size(34.0, null, true)), $oColor->getColor());
63+
$this->assertEquals(array('r' => new Size(34.0, null, true, $oColor->getLineNo()), 'g' => new Size(34.0, null, true, $oColor->getLineNo()), 'b' => new Size(34.0, null, true, $oColor->getLineNo())), $oColor->getColor());
6864
} else if($sSelector === '#yours') {
6965
$aColorRule = $oRuleSet->getRules('background-color');
7066
$oColor = $aColorRule[0]->getValue();
71-
$oColor->setLineNo(0);
72-
$this->assertEquals(array('h' => new Size(220.0, null, true), 's' => new Size(10.0, '%', true), 'l' => new Size(220.0, '%', true)), $oColor->getColor());
67+
$this->assertEquals(array('h' => new Size(220.0, null, true, $oColor->getLineNo()), 's' => new Size(10.0, '%', true, $oColor->getLineNo()), 'l' => new Size(220.0, '%', true, $oColor->getLineNo())), $oColor->getColor());
7368
$oColor = $aColorRule[1]->getValue();
74-
$oColor->setLineNo(0);
75-
$this->assertEquals(array('h' => new Size(220.0, null, true), 's' => new Size(10.0, '%', true), 'l' => new Size(220.0, '%', true), 'a' => new Size(0000.3, null, true)), $oColor->getColor());
69+
$this->assertEquals(array('h' => new Size(220.0, null, true, $oColor->getLineNo()), 's' => new Size(10.0, '%', true, $oColor->getLineNo()), 'l' => new Size(220.0, '%', true, $oColor->getLineNo()), 'a' => new Size(0000.3, null, true, $oColor->getLineNo())), $oColor->getColor());
7670
}
7771
}
7872
foreach ($oDoc->getAllValues('color') as $sColor) {

0 commit comments

Comments
 (0)