|
19 | 19 |
|
20 | 20 | /** |
21 | 21 | * Unit test for CustomizableOptions resolver |
| 22 | + * @SuppressWarnings(PHPMD.UnusedFormalParameter) |
22 | 23 | */ |
23 | 24 | class CustomizableOptionsTest extends TestCase |
24 | 25 | { |
@@ -153,7 +154,10 @@ public function testResolveReturnsCustomizableOptionsData(): void |
153 | 154 | $this->customizableOptionMock->expects($this->exactly(3)) |
154 | 155 | ->method('getData') |
155 | 156 | ->willReturnCallback( |
156 | | - function ($unused, $optionId) use ( |
| 157 | + function ( |
| 158 | + QuoteItem $item, |
| 159 | + int $optionId |
| 160 | + ) use ( |
157 | 161 | $expectedOptionData1, |
158 | 162 | $expectedOptionData2, |
159 | 163 | $expectedOptionData3 |
@@ -208,7 +212,10 @@ public function testResolveSkipsEmptyCustomizableOptions(): void |
208 | 212 | $this->customizableOptionMock->expects($this->exactly(3)) |
209 | 213 | ->method('getData') |
210 | 214 | ->willReturnCallback( |
211 | | - function ($unused, $optionId) use ($expectedOptionData1, $expectedOptionData3) { |
| 215 | + function ( |
| 216 | + QuoteItem $item, |
| 217 | + int $optionId |
| 218 | + ) use ($expectedOptionData1, $expectedOptionData3) { |
212 | 219 | switch ($optionId) { |
213 | 220 | case 1: |
214 | 221 | return $expectedOptionData1; |
@@ -281,7 +288,10 @@ public function testResolveHandlesMixedNullAndEmptyArrayReturns(): void |
281 | 288 | $this->customizableOptionMock->expects($this->exactly(4)) |
282 | 289 | ->method('getData') |
283 | 290 | ->willReturnCallback( |
284 | | - function ($unused, $optionId) use ($expectedOptionData1) { |
| 291 | + function ( |
| 292 | + QuoteItem $item, |
| 293 | + int $optionId |
| 294 | + ) use ($expectedOptionData1) { |
285 | 295 | switch ($optionId) { |
286 | 296 | case 1: |
287 | 297 | return $expectedOptionData1; |
@@ -430,7 +440,10 @@ public function testResolveHandlesDeletedCustomOption(): void |
430 | 440 | $this->customizableOptionMock->expects($this->exactly(3)) |
431 | 441 | ->method('getData') |
432 | 442 | ->willReturnCallback( |
433 | | - function ($unused, $optionId) use ($validOption1, $validOption2) { |
| 443 | + function ( |
| 444 | + QuoteItem $item, |
| 445 | + int $optionId |
| 446 | + ) use ($validOption1, $validOption2) { |
434 | 447 | switch ($optionId) { |
435 | 448 | case 1: |
436 | 449 | return $validOption1; |
|
0 commit comments