abterphp / framework

src/Debug/Exceptions/Handlers/Whoops/ExceptionRenderer.php

Indentation +1 added lines, -5 removed lines

@@ -165,8 +165,4 @@
             $values = (array)$values;
 
             foreach ($values as $value) {
-                header("$name:$value", false);
-            }
-        }
-    }
-}
+                header("$name:$value
\ No newline at end of file

tests/Console/Commands/Security/SecretGeneratorTest.php

Indentation +1 added lines, -3 removed lines

@@ -105,6 +105,4 @@
         $this->sut->execute($responseMock);
         $content2 = file_get_contents($this->configUrl);
 
-        $this->assertNotSame($content1, $content2);
-    }
-}
+        $this->assertNotSame($content1, $content2
\ No newline at end of file

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);
-    }
-}

src/Grid/Component/Filters.php

Indentation -159 removed lines

@@ -39,162 +39,3 @@
      * %2$s - nodes (filters)
      */
     protected const DEFAULT_TEMPLATE = <<<'EOT'
-        <div class="hidable">
-            <p class="hider">%1$s</p>
-            <div class="hidee">%2$s</div>
-        </div>
-        EOT;
-
-    protected string $template = self::DEFAULT_TEMPLATE;
-
-    /** @var IFilter[] */
-    protected array $nodes = [];
-
-    protected string $nodeClass = IFilter::class;
-
-    protected Action $hiderBtn;
-
-    protected Action $filterBtn;
-
-    protected Action $resetBtn;
-
-    /**
-     * Filters constructor.
-     *
-     * @param string[]        $intents
-     * @param Attributes|null $attributes
-     * @param string|null     $tag
-     */
-    public function __construct(array $intents = [], ?Attributes $attributes = null, ?string $tag = null)
-    {
-        parent::__construct(null, $intents, $attributes, $tag);
-
-        $this->hiderBtn  = new Action(static::BTN_CONTENT_FILTERS, [Action::INTENT_INFO]);
-        $this->filterBtn = new Action(static::BTN_CONTENT_FILTER, [Action::INTENT_PRIMARY], static::ATTRIBUTES_SEARCH);
-        $this->resetBtn  = new Action(static::BTN_CONTENT_RESET, [Action::INTENT_SECONDARY], static::ATTRIBUTES_RESET);
-    }
-
-    /**
-     * @param array $params
-     *
-     * @return $this
-     */
-    public function setParams(array $params): Filters
-    {
-        foreach ($this->nodes as $filter) {
-            $filter->setParams($params);
-        }
-
-        return $this;
-    }
-
-    /**
-     * @param string $baseUrl
-     *
-     * @return string
-     */
-    public function getUrl(string $baseUrl): string
-    {
-        $queryParts = [];
-        foreach ($this->nodes as $filter) {
-            $queryPart = $filter->getQueryPart();
-            if (!$queryPart) {
-                continue;
-            }
-
-            $queryParts[] = $queryPart;
-        }
-
-        if (empty($queryParts)) {
-            return $baseUrl;
-        }
-
-        return sprintf('%s%s&', $baseUrl, implode('&', $queryParts));
-    }
-
-    /**
-     * @return array
-     */
-    public function getWhereConditions(): array
-    {
-        $conditions = [];
-        foreach ($this->nodes as $filter) {
-            $conditions = array_merge($conditions, $filter->getWhereConditions());
-        }
-
-        return $conditions;
-    }
-
-    /**
-     * @return array<string,string>
-     */
-    public function getSqlParams(): array
-    {
-        $params = [];
-        foreach ($this->nodes as $filter) {
-            $params = array_merge($params, $filter->getQueryParams());
-        }
-
-        return $params;
-    }
-
-    /**
-     * @return INode[]
-     */
-    public function getExtendedNodes(): array
-    {
-        return array_merge([$this->hiderBtn, $this->filterBtn, $this->resetBtn], $this->getNodes());
-    }
-
-    /**
-     * @param string $template
-     *
-     * @return INode
-     */
-    public function setTemplate(string $template): INode
-    {
-        $this->template = $template;
-
-        return $this;
-    }
-
-    /**
-     * @return string
-     */
-    public function __toString(): string
-    {
-        $nodes = Collection::__toString();
-
-        $form = sprintf(self::WITH_CONTENT_TAG, $this->tag, $nodes, new Attributes( [Html5::ATTR_CLASS => 'filter-form']));
-
-        return sprintf(
-            $this->template,
-            (string)$this->hiderBtn,
-            $form
-        );
-    }
-
-    /**
-     * @return Action
-     */
-    public function getHiderBtn(): Action
-    {
-        return $this->hiderBtn;
-    }
-
-    /**
-     * @return Action
-     */
-    public function getFilterBtn(): Action
-    {
-        return $this->filterBtn;
-    }
-
-    /**
-     * @return Action
-     */
-    public function getResetBtn(): Action
-    {
-        return $this->resetBtn;
-    }
-}