Skip to content

Commit 8c83a37

Browse files
committed
Fix retrieving strategy for custom AccessDecisionManager
1 parent db1e522 commit 8c83a37

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

Authorization/TraceableAccessDecisionManager.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
class TraceableAccessDecisionManager implements AccessDecisionManagerInterface
2727
{
2828
private AccessDecisionManagerInterface $manager;
29-
private AccessDecisionStrategyInterface $strategy;
29+
private ?AccessDecisionStrategyInterface $strategy = null;
3030
/** @var iterable<mixed, VoterInterface> */
3131
private iterable $voters = [];
3232
private array $decisionLog = []; // All decision logs

Tests/Authorization/TraceableAccessDecisionManagerTest.php

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -274,4 +274,13 @@ public function testAccessDecisionManagerCalledByVoter()
274274
],
275275
], $sut->getDecisionLog());
276276
}
277+
278+
public function testCustomAccessDecisionManagerReturnsEmptyStrategy()
279+
{
280+
$admMock = $this->createMock(AccessDecisionManagerInterface::class);
281+
282+
$adm = new TraceableAccessDecisionManager($admMock);
283+
284+
$this->assertEquals('-', $adm->getStrategy());
285+
}
277286
}

0 commit comments

Comments
 (0)