Commit 3ec9883
committed
[CLEANUP] Remove redundant test code
`getAllValues()` only includes values that are `Value` objects.
It does not include `string` values. So the list returned was empty,
the loop had no iterations, and the `assertSame()` was never called.
It doesn't make sense to replace the code with an assertion that
`getAllValues()` is returning an empty list, because the `TestCase` is for
`Parser`, not `Document`. Besides, there is already an assertion that `red`
is set for the string `color` value.
Uncovered by PHPStan config change in #1583.
This change would also have silenced the same PHPStan warning.1 parent bdb95ea commit 3ec9883
1 file changed
Lines changed: 0 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
151 | 151 | | |
152 | 152 | | |
153 | 153 | | |
154 | | - | |
155 | | - | |
156 | | - | |
157 | 154 | | |
158 | 155 | | |
159 | 156 | | |
| |||
0 commit comments