Skip to content

Commit ab924e5

Browse files
committed
fix initializing key for reachable roles
1 parent eead81d commit ab924e5

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

Role/RoleHierarchy.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ public function __construct(
3030

3131
public function getReachableRoleNames(array $roles): array
3232
{
33-
$reachableRoles = $roles;
33+
$reachableRoles = array_combine($roles, $roles);
3434

3535
foreach ($roles as $role) {
3636
if (!isset($this->map[$role])) {

Tests/Dumper/MermaidDumperTest.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
namespace Symfony\Component\Security\Core\Tests\Dumper;
1313

1414
use PHPUnit\Framework\TestCase;
15-
use Symfony\Component\Security\Core\Dumper\MermaidDirectionEnum;
15+
use Symfony\Component\Security\Core\Dumper\MermaidDirection;
1616
use Symfony\Component\Security\Core\Dumper\MermaidDumper;
1717
use Symfony\Component\Security\Core\Role\RoleHierarchy;
1818

@@ -46,7 +46,7 @@ public function testDumpWithDirection()
4646

4747
$roleHierarchy = new RoleHierarchy($hierarchy);
4848
$dumper = new MermaidDumper();
49-
$output = $dumper->dump($roleHierarchy, MermaidDirectionEnum::LEFT_TO_RIGHT);
49+
$output = $dumper->dump($roleHierarchy, MermaidDirection::LEFT_TO_RIGHT);
5050

5151
$this->assertStringContainsString('graph LR', $output);
5252
}
@@ -97,7 +97,7 @@ public function testInvalidDirection()
9797
}
9898

9999
#[\PHPUnit\Framework\Attributes\DataProvider('dataProviderValidDirection')]
100-
public function testValidDirections(MermaidDirectionEnum $direction)
100+
public function testValidDirections(MermaidDirection $direction)
101101
{
102102
$this->expectNotToPerformAssertions();
103103
$dumper = new MermaidDumper();
@@ -107,11 +107,11 @@ public function testValidDirections(MermaidDirectionEnum $direction)
107107
public static function dataProviderValidDirection()
108108
{
109109
return [
110-
[MermaidDirectionEnum::TOP_TO_BOTTOM],
111-
[MermaidDirectionEnum::TOP_DOWN],
112-
[MermaidDirectionEnum::BOTTOM_TO_TOP],
113-
[MermaidDirectionEnum::RIGHT_TO_LEFT],
114-
[MermaidDirectionEnum::LEFT_TO_RIGHT],
110+
[MermaidDirection::TOP_TO_BOTTOM],
111+
[MermaidDirection::TOP_DOWN],
112+
[MermaidDirection::BOTTOM_TO_TOP],
113+
[MermaidDirection::RIGHT_TO_LEFT],
114+
[MermaidDirection::LEFT_TO_RIGHT],
115115
];
116116
}
117117

0 commit comments

Comments
 (0)