refactor(HasOptions): change access modifiers of methods

- Change the access modifiers of getHydratedEscapedNormalizedOptions, getEscapedNormalizedOptions, and getNormalizedOptions methods from public to protected.

src/Concerns/HasOptions.php

@@ -1502,23 +1502,23 @@ public function getOption(string $key, $default = null)
     /**
      * @throws InvalidOptionException
      */
-    public function getHydratedEscapedNormalizedOptions(): string
+    protected function getHydratedEscapedNormalizedOptions(): string
     {
         return implode(' ', $this->getEscapedNormalizedOptions());
     }
 
     /**
      * @throws InvalidOptionException
      */
-    public function getEscapedNormalizedOptions(): array
+    protected function getEscapedNormalizedOptions(): array
     {
         return array_map('escape_argument', $this->getNormalizedOptions());
     }
 
     /**
      * @throws InvalidOptionException
      */
-    public function getNormalizedOptions(): array
+    protected function getNormalizedOptions(): array
     {
         return $this->normalizeOptions($this->options);
     }

src/Concerns/WithRunable.php

@@ -13,6 +13,7 @@
 namespace Guanguans\SoarPHP\Concerns;
 
 use Guanguans\SoarPHP\Exceptions\InvalidArgumentException;
+use Guanguans\SoarPHP\Exceptions\InvalidOptionException;
 use Guanguans\SoarPHP\Exceptions\ProcessFailedException;
 use Symfony\Component\Process\Process;
 
@@ -52,6 +53,8 @@ protected function exec($withOptions = [], ?callable $processTapper = null, ?cal
 
     /**
      * @param array|string $withOptions
+     *
+     * @throws InvalidOptionException
      */
     private function createProcess($withOptions = [], ?callable $processTapper = null): Process
     {

tests/Concerns/ConcreteMagicTest.php

@@ -31,7 +31,6 @@ public function testSleep(): void
         $this->assertInstanceOf(Soar::class, $unserializedSoar);
         $this->assertNotEmpty($unserializedSoar->getSoarPath());
         $this->assertNotEmpty($unserializedSoar->getOptions());
-        $this->assertNotEmpty($unserializedSoar->getNormalizedOptions());
     }
 
     /**
@@ -47,6 +46,5 @@ public function testSetState(): void
         $this->assertInstanceOf(Soar::class, $exportedSoar);
         $this->assertNotEmpty($exportedSoar->getSoarPath());
         $this->assertNotEmpty($exportedSoar->getOptions());
-        $this->assertNotEmpty($exportedSoar->getNormalizedOptions());
     }
 }

tests/Concerns/HasOptionsTest.php

@@ -95,20 +95,6 @@ public function testMergeOption(): void
         $this->assertSame($val, $option);
     }
 
-    public function testGetNormalizedOptions(): void
-    {
-        $soar = Soar::create();
-
-        $this->assertIsArray($soar->getNormalizedOptions());
-    }
-
-    public function testGetHydratedEscapedNormalizedOptions(): void
-    {
-        $soar = Soar::create();
-
-        $this->assertIsString($soar->getHydratedEscapedNormalizedOptions());
-    }
-
     public function testGetOptions(): void
     {
         $soar = Soar::create();
@@ -151,7 +137,9 @@ public function testInvalidOptionExceptionForNormalizeOptions(): void
     {
         $this->expectException(InvalidOptionException::class);
         $this->expectExceptionMessage('object');
-        Soar::create(['foo' => $this->createMock(\stdClass::class)])->getNormalizedOptions();
+        (function () {
+            return $this->getNormalizedOptions();
+        })->call(Soar::create(['foo' => $this->createMock(\stdClass::class)]));
     }
 
     public function testNormalizeOptions(): void
@@ -183,6 +171,10 @@ public function testNormalizeOptions(): void
             'invoke' => new InvokeOption(),
         ]);
 
-        $this->assertIsArray($soar->getNormalizedOptions());
+        $this->assertIsArray(
+            (function () {
+                return $this->getNormalizedOptions();
+            })->call($soar)
+        );
     }
 }