CR changes

ezsystems · Sep 6, 2021 · 1e153d6 · 1e153d6
1 parent 8bde58e
commit 1e153d6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/lib/Configuration/Provider/CKEditor.php b/src/lib/Configuration/Provider/CKEditor.php
@@ -38,6 +38,7 @@ public function getName(): string
 
     /**
      * Returns CKEditor configuration.
+     *
      * @phpstan-return array<array-key, array{
      *  toolbars: array<string>,
      *  customTags: array<string>,
@@ -53,6 +54,7 @@ public function getConfiguration(): array
 
     /**
      * Returns toolbars configuration.
+     *
      * @phpstan-return array<string>
      */
     private function getToolbars(): array
@@ -72,6 +74,7 @@ static function (string $key): bool {
 
     /**
      * Returns customTags configuration.
+     *
      * @phpstan-return array<string>
      */
     private function getCustomTags(): array
@@ -91,6 +94,7 @@ static function (string $key): bool {
 
     /**
      * Returns filtered Toolbars configuration.
+     *
      * @phpstan-return array<string>
      */
     private function filterToolbars(
@@ -121,13 +125,15 @@ static function (array $value): bool {
             }
         }
 
+        // Removes last separator from the toolbars list.
         array_pop($toolbars);
 
         return array_values($toolbars);
     }
 
     /**
      * Returns List of visible, sorted buttons.
+     *
      * @phpstan-return array<string>
      */
     private function getToolbarButtons(array $buttons): array

diff --git a/tests/lib/Configuration/Provider/CKEditorTest.php b/tests/lib/Configuration/Provider/CKEditorTest.php
@@ -13,7 +13,7 @@
 use EzSystems\EzPlatformRichText\SPI\Configuration\Provider;
 use EzSystems\EzPlatformRichTextBundle\DependencyInjection\Configuration\Parser\FieldType\RichText;
 
-class CKEditorTest extends BaseProviderTestCase
+final class CKEditorTest extends BaseProviderTestCase
 {
     public function createProvider(ConfigResolverInterface $configResolver = null): Provider
     {
@@ -35,7 +35,7 @@ public function testGetConfiguration(): void
         $provider = $this->createProvider();
 
         $this->configResolver
-            ->expects($this->any())
+            ->expects($this->exactly(2))
             ->method('hasParameter')
             ->willReturn(false);