abterphp / framework

tests/Assets/CacheManager/FlysystemTest.php

Spacing +9 added lines, -9 removed lines

@@ -49,7 +49,7 @@ 
 
         $this->sut->registerFilesystem(
             $fs,
-            fn () => false
+            fn() => false
         );
 
         $path = 'foo.ext';
@@ -66,11 +66,11 @@ 
 
         $this->sut->registerFilesystem(
             $fs1,
-            fn () => false
+            fn() => false
         );
         $this->sut->registerFilesystem(
             $fs2,
-            fn () => true
+            fn() => true
         );
 
         $path = 'foo.ext';
@@ -100,7 +100,7 @@ 
 
         $this->sut->registerFilesystem(
             $fs,
-            fn () => false
+            fn() => false
         );
 
         $path = 'foo.ext';
@@ -160,11 +160,11 @@ 
 
         $this->sut->registerFilesystem(
             $fs1,
-            fn () => false
+            fn() => false
         );
         $this->sut->registerFilesystem(
             $fs2,
-            fn () => true
+            fn() => true
         );
 
         $path = 'foo.ext';
@@ -196,7 +196,7 @@ 
 
         $this->sut->registerFilesystem(
             $fs,
-            fn () => false
+            fn() => false
         );
 
         $path    = 'foo.ext';
@@ -242,11 +242,11 @@ 
 
         $this->sut->registerFilesystem(
             $fs1,
-            fn () => false
+            fn() => false
         );
         $this->sut->registerFilesystem(
             $fs2,
-            fn () => true
+            fn() => true
         );
 
         $path    = 'foo.ext';

tests/Assets/CacheManager/DummyTest.php

Spacing +7 added lines, -7 removed lines

@@ -51,7 +51,7 @@ 
 
         $this->sut->registerFilesystem(
             $fs,
-            fn () => false
+            fn() => false
         );
 
         $path = 'foo.ext';
@@ -78,7 +78,7 @@ 
 
         $this->sut->registerFilesystem(
             $fs,
-            fn () => false
+            fn() => false
         );
 
         $path = 'foo.ext';
@@ -138,11 +138,11 @@ 
 
         $this->sut->registerFilesystem(
             $fs1,
-            fn () => false
+            fn() => false
         );
         $this->sut->registerFilesystem(
             $fs2,
-            fn () => true
+            fn() => true
         );
 
         $path = 'foo.ext';
@@ -174,7 +174,7 @@ 
 
         $this->sut->registerFilesystem(
             $fs,
-            fn () => false
+            fn() => false
         );
 
         $path    = 'foo.ext';
@@ -220,11 +220,11 @@ 
 
         $this->sut->registerFilesystem(
             $fs1,
-            fn () => false
+            fn() => false
         );
         $this->sut->registerFilesystem(
             $fs2,
-            fn () => true
+            fn() => true
         );
 
         $path    = 'foo.ext';

tests/Http/Middleware/SessionTest.php

Spacing +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $actualResult = $this->sut->handle($requestMock, $next);
 
@@ -80,7 +80,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $this->sessionHandlerMock->expects($this->once())->method('gc');
 

tests/Http/Middleware/SecurityTest.php

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $this->cacheBridgeMock->expects($this->never())->method('get');
 
@@ -54,7 +54,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $this->cacheBridgeMock->expects($this->once())->method('get')->willReturn(true);
 
@@ -73,7 +73,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $sut->setVar(
             [
@@ -108,7 +108,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $sut->setVar(
             [
@@ -292,7 +292,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $sut->setVar($environmentData);
 

tests/Http/Middleware/EnvironmentWarningTest.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $responseMock->expects($this->never())->method('setContent');
 
@@ -53,7 +53,7 @@ 
         /** @var Response|MockObject $requestMock */
         $responseMock = $this->createMock(Response::class);
 
-        $next = fn () => $responseMock;
+        $next = fn() => $responseMock;
 
         $responseMock->expects($this->atLeastOnce())->method('setContent');
 

tests/Authorization/CombinedAdapterTest.php

Indentation -205 removed lines

@@ -12,208 +12,3 @@
 class CombinedAdapterTest extends TestCase
 {
     protected const EXAMPLE_CONFIG = <<<'EOF'
-        [policy_definition]
-        p = sub, obj, act
-
-        [role_definition]
-        g = _, _
-        EOF;
-
-    /** @var CombinedAdapter - System Under Test */
-    protected CombinedAdapter $sut;
-
-    /** @var CasbinAdapter|MockObject */
-    protected $defaultAdapterMock;
-
-    /** @var CacheManager|MockObject */
-    protected $cacheManagerMock;
-
-    public function setUp(): void
-    {
-        $this->defaultAdapterMock = $this->createMock(CasbinAdapter::class);
-
-        $this->cacheManagerMock = $this->createMock(CacheManager::class);
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock, $this->cacheManagerMock);
-
-        parent::setUp();
-    }
-
-    public function createSutWithoutCacheManager(): void
-    {
-        $this->cacheManagerMock = null;
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock);
-    }
-
-    /**
-     * @return array
-     */
-    public function loadPolicyWithoutCacheManagerProvider(): array
-    {
-        $adapterMock0 = $this->createMock(CasbinAdapter::class);
-        $adapterMock1 = $this->createMock(CasbinAdapter::class);
-        $adapterMock2 = $this->createMock(CasbinAdapter::class);
-
-        return [
-            'no-adapters'  => [[]],
-            'one-adapter'  => [[$adapterMock0]],
-            'two-adapters' => [[$adapterMock1, $adapterMock2]],
-        ];
-    }
-
-    /**
-     * @dataProvider loadPolicyWithoutCacheManagerProvider
-     *
-     * @param array $adaptersMocks
-     */
-    public function testLoadPolicyCallsAdaptersByDefault(array $adaptersMocks): void
-    {
-        $this->createSutWithoutCacheManager();
-
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        foreach ($adaptersMocks as $adapter) {
-            $adapter->expects($this->once())->method('loadPolicy')->with($model);
-            $this->sut->registerAdapter($adapter);
-        }
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanLoadPoliciesFromCache(): void
-    {
-        $cacheData = ['g' => [], 'p' => []];
-
-        $model = new CasbinModel();
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cacheData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanPopulatePoliciesFromCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cachedData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanStorePoliciesInCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $model->addPolicies('g', 'g', $cachedData['g']);
-        $model->addPolicies('p', 'p', $cachedData['p']);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn([]);
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        $this->cacheManagerMock->expects($this->once())->method('storeAll')->willReturn(true);
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testSavePolicyCallsDefaultAdapter(): void
-    {
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('savePolicy')
-            ->with($model);
-
-        $this->sut->savePolicy($model);
-    }
-
-    public function testAddPolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('addPolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->addPolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemovePolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removePolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->removePolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemoveFilteredPolicyWithoutFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec        = 'foo';
-        $ptype      = 'bar';
-        $fieldIndex = 0;
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex);
-    }
-
-    public function testRemoveFilteredPolicyWithFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec         = 'foo';
-        $ptype       = 'bar';
-        $fieldIndex  = 0;
-        $fieldValue0 = 'quix';
-        $fieldValue1 = 'tata';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-    }
-}

Switch Indentation -205 removed lines

@@ -12,208 +12,3 @@
 class CombinedAdapterTest extends TestCase
 {
     protected const EXAMPLE_CONFIG = <<<'EOF'
-        [policy_definition]
-        p = sub, obj, act
-
-        [role_definition]
-        g = _, _
-        EOF;
-
-    /** @var CombinedAdapter - System Under Test */
-    protected CombinedAdapter $sut;
-
-    /** @var CasbinAdapter|MockObject */
-    protected $defaultAdapterMock;
-
-    /** @var CacheManager|MockObject */
-    protected $cacheManagerMock;
-
-    public function setUp(): void
-    {
-        $this->defaultAdapterMock = $this->createMock(CasbinAdapter::class);
-
-        $this->cacheManagerMock = $this->createMock(CacheManager::class);
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock, $this->cacheManagerMock);
-
-        parent::setUp();
-    }
-
-    public function createSutWithoutCacheManager(): void
-    {
-        $this->cacheManagerMock = null;
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock);
-    }
-
-    /**
-     * @return array
-     */
-    public function loadPolicyWithoutCacheManagerProvider(): array
-    {
-        $adapterMock0 = $this->createMock(CasbinAdapter::class);
-        $adapterMock1 = $this->createMock(CasbinAdapter::class);
-        $adapterMock2 = $this->createMock(CasbinAdapter::class);
-
-        return [
-            'no-adapters'  => [[]],
-            'one-adapter'  => [[$adapterMock0]],
-            'two-adapters' => [[$adapterMock1, $adapterMock2]],
-        ];
-    }
-
-    /**
-     * @dataProvider loadPolicyWithoutCacheManagerProvider
-     *
-     * @param array $adaptersMocks
-     */
-    public function testLoadPolicyCallsAdaptersByDefault(array $adaptersMocks): void
-    {
-        $this->createSutWithoutCacheManager();
-
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        foreach ($adaptersMocks as $adapter) {
-            $adapter->expects($this->once())->method('loadPolicy')->with($model);
-            $this->sut->registerAdapter($adapter);
-        }
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanLoadPoliciesFromCache(): void
-    {
-        $cacheData = ['g' => [], 'p' => []];
-
-        $model = new CasbinModel();
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cacheData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanPopulatePoliciesFromCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cachedData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanStorePoliciesInCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $model->addPolicies('g', 'g', $cachedData['g']);
-        $model->addPolicies('p', 'p', $cachedData['p']);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn([]);
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        $this->cacheManagerMock->expects($this->once())->method('storeAll')->willReturn(true);
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testSavePolicyCallsDefaultAdapter(): void
-    {
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('savePolicy')
-            ->with($model);
-
-        $this->sut->savePolicy($model);
-    }
-
-    public function testAddPolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('addPolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->addPolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemovePolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removePolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->removePolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemoveFilteredPolicyWithoutFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec        = 'foo';
-        $ptype      = 'bar';
-        $fieldIndex = 0;
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex);
-    }
-
-    public function testRemoveFilteredPolicyWithFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec         = 'foo';
-        $ptype       = 'bar';
-        $fieldIndex  = 0;
-        $fieldValue0 = 'quix';
-        $fieldValue1 = 'tata';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-    }
-}

Spacing -205 removed lines

@@ -12,208 +12,3 @@
 class CombinedAdapterTest extends TestCase
 {
     protected const EXAMPLE_CONFIG = <<<'EOF'
-        [policy_definition]
-        p = sub, obj, act
-
-        [role_definition]
-        g = _, _
-        EOF;
-
-    /** @var CombinedAdapter - System Under Test */
-    protected CombinedAdapter $sut;
-
-    /** @var CasbinAdapter|MockObject */
-    protected $defaultAdapterMock;
-
-    /** @var CacheManager|MockObject */
-    protected $cacheManagerMock;
-
-    public function setUp(): void
-    {
-        $this->defaultAdapterMock = $this->createMock(CasbinAdapter::class);
-
-        $this->cacheManagerMock = $this->createMock(CacheManager::class);
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock, $this->cacheManagerMock);
-
-        parent::setUp();
-    }
-
-    public function createSutWithoutCacheManager(): void
-    {
-        $this->cacheManagerMock = null;
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock);
-    }
-
-    /**
-     * @return array
-     */
-    public function loadPolicyWithoutCacheManagerProvider(): array
-    {
-        $adapterMock0 = $this->createMock(CasbinAdapter::class);
-        $adapterMock1 = $this->createMock(CasbinAdapter::class);
-        $adapterMock2 = $this->createMock(CasbinAdapter::class);
-
-        return [
-            'no-adapters'  => [[]],
-            'one-adapter'  => [[$adapterMock0]],
-            'two-adapters' => [[$adapterMock1, $adapterMock2]],
-        ];
-    }
-
-    /**
-     * @dataProvider loadPolicyWithoutCacheManagerProvider
-     *
-     * @param array $adaptersMocks
-     */
-    public function testLoadPolicyCallsAdaptersByDefault(array $adaptersMocks): void
-    {
-        $this->createSutWithoutCacheManager();
-
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        foreach ($adaptersMocks as $adapter) {
-            $adapter->expects($this->once())->method('loadPolicy')->with($model);
-            $this->sut->registerAdapter($adapter);
-        }
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanLoadPoliciesFromCache(): void
-    {
-        $cacheData = ['g' => [], 'p' => []];
-
-        $model = new CasbinModel();
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cacheData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanPopulatePoliciesFromCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cachedData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanStorePoliciesInCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $model->addPolicies('g', 'g', $cachedData['g']);
-        $model->addPolicies('p', 'p', $cachedData['p']);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn([]);
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        $this->cacheManagerMock->expects($this->once())->method('storeAll')->willReturn(true);
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testSavePolicyCallsDefaultAdapter(): void
-    {
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('savePolicy')
-            ->with($model);
-
-        $this->sut->savePolicy($model);
-    }
-
-    public function testAddPolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('addPolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->addPolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemovePolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removePolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->removePolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemoveFilteredPolicyWithoutFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec        = 'foo';
-        $ptype      = 'bar';
-        $fieldIndex = 0;
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex);
-    }
-
-    public function testRemoveFilteredPolicyWithFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec         = 'foo';
-        $ptype       = 'bar';
-        $fieldIndex  = 0;
-        $fieldValue0 = 'quix';
-        $fieldValue1 = 'tata';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-    }
-}

Braces -205 removed lines

@@ -12,208 +12,3 @@
 class CombinedAdapterTest extends TestCase
 {
     protected const EXAMPLE_CONFIG = <<<'EOF'
-        [policy_definition]
-        p = sub, obj, act
-
-        [role_definition]
-        g = _, _
-        EOF;
-
-    /** @var CombinedAdapter - System Under Test */
-    protected CombinedAdapter $sut;
-
-    /** @var CasbinAdapter|MockObject */
-    protected $defaultAdapterMock;
-
-    /** @var CacheManager|MockObject */
-    protected $cacheManagerMock;
-
-    public function setUp(): void
-    {
-        $this->defaultAdapterMock = $this->createMock(CasbinAdapter::class);
-
-        $this->cacheManagerMock = $this->createMock(CacheManager::class);
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock, $this->cacheManagerMock);
-
-        parent::setUp();
-    }
-
-    public function createSutWithoutCacheManager(): void
-    {
-        $this->cacheManagerMock = null;
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock);
-    }
-
-    /**
-     * @return array
-     */
-    public function loadPolicyWithoutCacheManagerProvider(): array
-    {
-        $adapterMock0 = $this->createMock(CasbinAdapter::class);
-        $adapterMock1 = $this->createMock(CasbinAdapter::class);
-        $adapterMock2 = $this->createMock(CasbinAdapter::class);
-
-        return [
-            'no-adapters'  => [[]],
-            'one-adapter'  => [[$adapterMock0]],
-            'two-adapters' => [[$adapterMock1, $adapterMock2]],
-        ];
-    }
-
-    /**
-     * @dataProvider loadPolicyWithoutCacheManagerProvider
-     *
-     * @param array $adaptersMocks
-     */
-    public function testLoadPolicyCallsAdaptersByDefault(array $adaptersMocks): void
-    {
-        $this->createSutWithoutCacheManager();
-
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        foreach ($adaptersMocks as $adapter) {
-            $adapter->expects($this->once())->method('loadPolicy')->with($model);
-            $this->sut->registerAdapter($adapter);
-        }
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanLoadPoliciesFromCache(): void
-    {
-        $cacheData = ['g' => [], 'p' => []];
-
-        $model = new CasbinModel();
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cacheData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanPopulatePoliciesFromCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cachedData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanStorePoliciesInCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $model->addPolicies('g', 'g', $cachedData['g']);
-        $model->addPolicies('p', 'p', $cachedData['p']);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn([]);
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        $this->cacheManagerMock->expects($this->once())->method('storeAll')->willReturn(true);
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testSavePolicyCallsDefaultAdapter(): void
-    {
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('savePolicy')
-            ->with($model);
-
-        $this->sut->savePolicy($model);
-    }
-
-    public function testAddPolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('addPolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->addPolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemovePolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removePolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->removePolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemoveFilteredPolicyWithoutFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec        = 'foo';
-        $ptype      = 'bar';
-        $fieldIndex = 0;
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex);
-    }
-
-    public function testRemoveFilteredPolicyWithFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec         = 'foo';
-        $ptype       = 'bar';
-        $fieldIndex  = 0;
-        $fieldValue0 = 'quix';
-        $fieldValue1 = 'tata';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-    }
-}

Upper-Lower-Casing -205 removed lines

@@ -12,208 +12,3 @@
 class CombinedAdapterTest extends TestCase
 {
     protected const EXAMPLE_CONFIG = <<<'EOF'
-        [policy_definition]
-        p = sub, obj, act
-
-        [role_definition]
-        g = _, _
-        EOF;
-
-    /** @var CombinedAdapter - System Under Test */
-    protected CombinedAdapter $sut;
-
-    /** @var CasbinAdapter|MockObject */
-    protected $defaultAdapterMock;
-
-    /** @var CacheManager|MockObject */
-    protected $cacheManagerMock;
-
-    public function setUp(): void
-    {
-        $this->defaultAdapterMock = $this->createMock(CasbinAdapter::class);
-
-        $this->cacheManagerMock = $this->createMock(CacheManager::class);
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock, $this->cacheManagerMock);
-
-        parent::setUp();
-    }
-
-    public function createSutWithoutCacheManager(): void
-    {
-        $this->cacheManagerMock = null;
-
-        $this->sut = new CombinedAdapter($this->defaultAdapterMock);
-    }
-
-    /**
-     * @return array
-     */
-    public function loadPolicyWithoutCacheManagerProvider(): array
-    {
-        $adapterMock0 = $this->createMock(CasbinAdapter::class);
-        $adapterMock1 = $this->createMock(CasbinAdapter::class);
-        $adapterMock2 = $this->createMock(CasbinAdapter::class);
-
-        return [
-            'no-adapters'  => [[]],
-            'one-adapter'  => [[$adapterMock0]],
-            'two-adapters' => [[$adapterMock1, $adapterMock2]],
-        ];
-    }
-
-    /**
-     * @dataProvider loadPolicyWithoutCacheManagerProvider
-     *
-     * @param array $adaptersMocks
-     */
-    public function testLoadPolicyCallsAdaptersByDefault(array $adaptersMocks): void
-    {
-        $this->createSutWithoutCacheManager();
-
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        foreach ($adaptersMocks as $adapter) {
-            $adapter->expects($this->once())->method('loadPolicy')->with($model);
-            $this->sut->registerAdapter($adapter);
-        }
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanLoadPoliciesFromCache(): void
-    {
-        $cacheData = ['g' => [], 'p' => []];
-
-        $model = new CasbinModel();
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cacheData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanPopulatePoliciesFromCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn($cachedData);
-
-        $this->defaultAdapterMock->expects($this->never())->method('loadPolicy');
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testLoadPolicyCanStorePoliciesInCache(): void
-    {
-        $cachedData = [
-            'g' => [
-                ['joe', 'admin', '', '', ','],
-                ['jane', 'user', '', '', ','],
-            ],
-            'p' => [
-                ['user', 'admin_resource_users', 'read', '', ','],
-                ['admin', 'admin_resource_users', 'write', '', ','],
-            ],
-        ];
-
-        $model = new CasbinModel();
-        $model->loadModelFromText(static::EXAMPLE_CONFIG);
-
-        $model->addPolicies('g', 'g', $cachedData['g']);
-        $model->addPolicies('p', 'p', $cachedData['p']);
-
-        $this->cacheManagerMock->expects($this->once())->method('getAll')->willReturn([]);
-
-        $this->defaultAdapterMock->expects($this->once())->method('loadPolicy')->with($model);
-
-        $this->cacheManagerMock->expects($this->once())->method('storeAll')->willReturn(true);
-
-        $this->sut->loadPolicy($model);
-    }
-
-    public function testSavePolicyCallsDefaultAdapter(): void
-    {
-        $model = new CasbinModel();
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('savePolicy')
-            ->with($model);
-
-        $this->sut->savePolicy($model);
-    }
-
-    public function testAddPolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('addPolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->addPolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemovePolicyCallsDefaultAdapter(): void
-    {
-        $sec   = 'foo';
-        $ptype = 'bar';
-        $rule  = 'baz';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removePolicy')
-            ->with($sec, $ptype, [$rule]);
-
-        $this->sut->removePolicy($sec, $ptype, [$rule]);
-    }
-
-    public function testRemoveFilteredPolicyWithoutFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec        = 'foo';
-        $ptype      = 'bar';
-        $fieldIndex = 0;
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex);
-    }
-
-    public function testRemoveFilteredPolicyWithFieldValuesCallsDefaultAdapter(): void
-    {
-        $sec         = 'foo';
-        $ptype       = 'bar';
-        $fieldIndex  = 0;
-        $fieldValue0 = 'quix';
-        $fieldValue1 = 'tata';
-
-        $this->defaultAdapterMock
-            ->expects($this->once())
-            ->method('removeFilteredPolicy')
-            ->with($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-
-        $this->sut->removeFilteredPolicy($sec, $ptype, $fieldIndex, $fieldValue0, $fieldValue1);
-    }
-}

tests/Bootstrappers/Events/EventDispatcherBootstrapperTest.php

Spacing +4 added lines, -4 removed lines

@@ -22,8 +22,8 @@ 
     public function testRegisterBindings(): void
     {
         $listenerName = 'foo';
-        $events1      = [$listenerName => [fn () => null]];
-        $events2      = [$listenerName => [fn () => null]];
+        $events1      = [$listenerName => [fn() => null]];
+        $events2      = [$listenerName => [fn() => null]];
 
         $this->sut->setBaseEvents($events1);
         $this->sut->setModuleEvents($events2);
@@ -44,10 +44,10 @@ 
     public function testRegisterBindingsMixed(): void
     {
         $listenerName1 = 'foo';
-        $events1       = [$listenerName1 => [fn () => null]];
+        $events1       = [$listenerName1 => [fn() => null]];
 
         $listenerName2 = 'bar';
-        $events2       = [$listenerName2 => [fn () => null]];
+        $events2       = [$listenerName2 => [fn() => null]];
 
         $this->sut->setBaseEvents($events1);
         $this->sut->setModuleEvents($events2);

tests/Navigation/UserBlockTest.php

Spacing +4 added lines, -4 removed lines

@@ -33,10 +33,10 @@ 
 
         $sessionData = &$this->sessionData;
         $this->sessionMock->expects($this->any())->method('has')->willReturnCallback(
-            fn ($key) => isset($sessionData[$key])
+            fn($key) => isset($sessionData[$key])
         );
         $this->sessionMock->expects($this->any())->method('get')->willReturnCallback(
-            fn ($key) => $sessionData[$key] ?? ''
+            fn($key) => $sessionData[$key] ?? ''
         );
     }
 
@@ -64,10 +64,10 @@ 
         $sessionMock = $this->createMock(Session::class);
 
         $sessionMock->expects($this->any())->method('has')->willReturnCallback(
-            fn ($key) => isset($sessionData[$key])
+            fn($key) => isset($sessionData[$key])
         );
         $sessionMock->expects($this->any())->method('get')->willReturnCallback(
-            fn ($key) => $sessionData[$key] ?? ''
+            fn($key) => $sessionData[$key] ?? ''
         );
 
         $sut = new UserBlock($sessionMock);

tests/Filesystem/UploaderTest.php

Spacing +1 added lines, -1 removed lines

@@ -273,7 +273,7 @@
     {
         $this->sut = new Uploader($this->filesystemMock);
 
-        $fooPathFactory = fn ($filename) => "/foo-base-dir/$filename";
+        $fooPathFactory = fn($filename) => "/foo-base-dir/$filename";
 
         $key  = 'foo';
         $path = 'example.foo';