Skip to content

Commit

Permalink
Merge branch '7.0' into 7.1
Browse files Browse the repository at this point in the history
* 7.0:
  fix merge
  fix merge
  fix merge
  • Loading branch information
xabbuh committed May 17, 2024
2 parents 4e01371 + 0e1883e commit 232d6d0
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -41,17 +41,17 @@ protected function validateType($value): void
{
}

protected function normalizeValue($value)
protected function normalizeValue($value): mixed
{
return null;
}

protected function mergeValues($leftSide, $rightSide)
protected function mergeValues($leftSide, $rightSide): mixed
{
return null;
}

protected function finalizeValue($value)
protected function finalizeValue($value): mixed
{
return null;
}
Expand All @@ -61,7 +61,7 @@ public function hasDefaultValue(): bool
return true;
}

public function getDefaultValue()
public function getDefaultValue(): mixed
{
return null;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ protected function tearDown(): void
@rmdir($this->savePath);
}

$this->savePath = null;
ini_set('session.save_handler', $this->initialSessionSaveHandler);
ini_set('session.save_path', $this->initialSessionSavePath);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,6 @@ protected function tearDown(): void
@rmdir($this->savePath);
}

$this->savePath = null;
ini_set('session.save_handler', $this->initialSessionSaveHandler);
ini_set('session.save_path', $this->initialSessionSavePath);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
*/
class AbstractProxyTest extends TestCase
{
protected MockObject&AbstractProxy $proxy;
protected AbstractProxy $proxy;

protected function setUp(): void
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ public function testRenderExceptionIgnoreErrors()

public function testRenderExceptionIgnoreErrorsWithAlt()
{
$strategy = new InlineFragmentRenderer($this->getKernel($this->returnCallback(function () {
$strategy = new InlineFragmentRenderer($this->getKernel(function () {
static $firstCall = true;

if ($firstCall) {
Expand All @@ -112,7 +112,7 @@ public function testRenderExceptionIgnoreErrorsWithAlt()
}

return new Response('bar');
})));
}));

$this->assertEquals('bar', $strategy->render('/', Request::create('/'), ['ignore_errors' => true, 'alt' => '/foo'])->getContent());
}
Expand All @@ -127,6 +127,8 @@ private function getKernel($returnValue)

if ($returnValue instanceof \Exception) {
$mocker->willThrowException($returnValue);
} elseif ($returnValue instanceof \Closure) {
$mocker->willReturnCallback($returnValue);
} else {
$mocker->willReturn(...(\is_array($returnValue) ? $returnValue : [$returnValue]));
}
Expand Down

0 comments on commit 232d6d0

Please sign in to comment.