Skip to content

Commit 2353f57

Browse files
JakeQZJakeQZ
authored
[TASK] Add tests for getLineNumber (#1286)
These correspond to the existing tests for `getLineNo` for classes that implement `Positionable`. Also correct an existing test method name to refer to `getLineNo`.
1 parent 4086767 commit 2353f57

11 files changed

+232
-1
lines changed

‎tests/Unit/CSSList/AtRuleBlockListTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,27 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
126126
self::assertSame($lineNumber, $subject->getLineNo());
127127
}
128128

129+
/**
130+
* @test
131+
*/
132+
public function getLineNumberByDefaultReturnsNull(): void
133+
{
134+
$subject = new AtRuleBlockList('');
135+
136+
self::assertNull($subject->getLineNumber());
137+
}
138+
139+
/**
140+
* @test
141+
*/
142+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
143+
{
144+
$lineNumber = 42;
145+
$subject = new AtRuleBlockList('', '', $lineNumber);
146+
147+
self::assertSame($lineNumber, $subject->getLineNumber());
148+
}
149+
129150
/**
130151
* @test
131152
*/

‎tests/Unit/CSSList/CSSListTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,27 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
7979
self::assertSame($lineNumber, $subject->getLineNo());
8080
}
8181

82+
/**
83+
* @test
84+
*/
85+
public function getLineNumberByDefaultReturnsNull(): void
86+
{
87+
$subject = new ConcreteCSSList();
88+
89+
self::assertNull($subject->getLineNumber());
90+
}
91+
92+
/**
93+
* @test
94+
*/
95+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
96+
{
97+
$lineNumber = 42;
98+
$subject = new ConcreteCSSList($lineNumber);
99+
100+
self::assertSame($lineNumber, $subject->getLineNumber());
101+
}
102+
82103
/**
83104
* @test
84105
*/

‎tests/Unit/CSSList/KeyFrameTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,27 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
8080
self::assertSame($lineNumber, $subject->getLineNo());
8181
}
8282

83+
/**
84+
* @test
85+
*/
86+
public function getLineNumberByDefaultReturnsNull(): void
87+
{
88+
$subject = new KeyFrame();
89+
90+
self::assertNull($subject->getLineNumber());
91+
}
92+
93+
/**
94+
* @test
95+
*/
96+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
97+
{
98+
$lineNumber = 42;
99+
$subject = new KeyFrame($lineNumber);
100+
101+
self::assertSame($lineNumber, $subject->getLineNumber());
102+
}
103+
83104
/**
84105
* @test
85106
*/

‎tests/Unit/Comment/CommentTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,4 +77,25 @@ public function getLineNoInitiallyReturnsLineNumberPassedToConstructor(): void
7777

7878
self::assertSame($lineNumber, $subject->getLineNo());
7979
}
80+
81+
/**
82+
* @test
83+
*/
84+
public function getLineNumberByDefaultReturnsNull(): void
85+
{
86+
$subject = new Comment();
87+
88+
self::assertNull($subject->getLineNumber());
89+
}
90+
91+
/**
92+
* @test
93+
*/
94+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
95+
{
96+
$lineNumber = 42;
97+
$subject = new Comment('', $lineNumber);
98+
99+
self::assertSame($lineNumber, $subject->getLineNumber());
100+
}
80101
}

‎tests/Unit/Parsing/OutputExceptionTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,27 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
5353
self::assertSame($lineNumber, $exception->getLineNo());
5454
}
5555

56+
/**
57+
* @test
58+
*/
59+
public function getLineNumberByDefaultReturnsNull(): void
60+
{
61+
$subject = new OutputException('foo');
62+
63+
self::assertNull($subject->getLineNumber());
64+
}
65+
66+
/**
67+
* @test
68+
*/
69+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
70+
{
71+
$lineNumber = 42;
72+
$subject = new OutputException('foo', $lineNumber);
73+
74+
self::assertSame($lineNumber, $subject->getLineNumber());
75+
}
76+
5677
/**
5778
* @test
5879
*/

‎tests/Unit/Parsing/SourceExceptionTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,27 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
4444
self::assertSame($lineNumber, $exception->getLineNo());
4545
}
4646

47+
/**
48+
* @test
49+
*/
50+
public function getLineNumberByDefaultReturnsNull(): void
51+
{
52+
$subject = new SourceException('foo');
53+
54+
self::assertNull($subject->getLineNumber());
55+
}
56+
57+
/**
58+
* @test
59+
*/
60+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
61+
{
62+
$lineNumber = 42;
63+
$subject = new SourceException('foo', $lineNumber);
64+
65+
self::assertSame($lineNumber, $subject->getLineNumber());
66+
}
67+
4768
/**
4869
* @test
4970
*/

‎tests/Unit/Parsing/UnexpectedEOFExceptionTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,27 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
4242
self::assertSame($lineNumber, $exception->getLineNo());
4343
}
4444

45+
/**
46+
* @test
47+
*/
48+
public function getLineNumberByDefaultReturnsNull(): void
49+
{
50+
$subject = new UnexpectedEOFException('expected', 'found');
51+
52+
self::assertNull($subject->getLineNumber());
53+
}
54+
55+
/**
56+
* @test
57+
*/
58+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
59+
{
60+
$lineNumber = 42;
61+
$subject = new UnexpectedEOFException('expected', 'found', 'literal', $lineNumber);
62+
63+
self::assertSame($lineNumber, $subject->getLineNumber());
64+
}
65+
4566
/**
4667
* @test
4768
*/

‎tests/Unit/Parsing/UnexpectedTokenExceptionTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,27 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
4242
self::assertSame($lineNumber, $exception->getLineNo());
4343
}
4444

45+
/**
46+
* @test
47+
*/
48+
public function getLineNumberByDefaultReturnsNull(): void
49+
{
50+
$subject = new UnexpectedTokenException('expected', 'found');
51+
52+
self::assertNull($subject->getLineNumber());
53+
}
54+
55+
/**
56+
* @test
57+
*/
58+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
59+
{
60+
$lineNumber = 42;
61+
$subject = new UnexpectedTokenException('expected', 'found', 'literal', $lineNumber);
62+
63+
self::assertSame($lineNumber, $subject->getLineNumber());
64+
}
65+
4566
/**
4667
* @test
4768
*/

‎tests/Unit/Value/CSSStringTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,4 +81,25 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
8181

8282
self::assertSame($lineNumber, $subject->getLineNo());
8383
}
84+
85+
/**
86+
* @test
87+
*/
88+
public function getLineNumberByDefaultReturnsNull(): void
89+
{
90+
$subject = new CSSString('');
91+
92+
self::assertNull($subject->getLineNumber());
93+
}
94+
95+
/**
96+
* @test
97+
*/
98+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
99+
{
100+
$lineNumber = 42;
101+
$subject = new CSSString('', $lineNumber);
102+
103+
self::assertSame($lineNumber, $subject->getLineNumber());
104+
}
84105
}

‎tests/Unit/Value/CalcRuleValueListTest.php

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public function isRuleValueList(): void
2929
/**
3030
* @test
3131
*/
32-
public function getLineNumberByDefaultReturnsZero(): void
32+
public function getLineNoByDefaultReturnsZero(): void
3333
{
3434
$subject = new CalcRuleValueList();
3535

@@ -48,6 +48,27 @@ public function getLineNoReturnsLineNumberProvidedToConstructor(): void
4848
self::assertSame($lineNumber, $subject->getLineNo());
4949
}
5050

51+
/**
52+
* @test
53+
*/
54+
public function getLineNumberByDefaultReturnsNull(): void
55+
{
56+
$subject = new CalcRuleValueList();
57+
58+
self::assertNull($subject->getLineNumber());
59+
}
60+
61+
/**
62+
* @test
63+
*/
64+
public function getLineNumberReturnsLineNumberProvidedToConstructor(): void
65+
{
66+
$lineNumber = 42;
67+
$subject = new CalcRuleValueList($lineNumber);
68+
69+
self::assertSame($lineNumber, $subject->getLineNumber());
70+
}
71+
5172
/**
5273
* @test
5374
*/

0 commit comments

Comments
 (0)