Remove unused private methods

alamirault · alamirault · commit be39a0993ca1 · 2023-02-23T21:12:05.000+01:00
diff --git a/src/Symfony/Component/ErrorHandler/Tests/Exception/FlattenExceptionTest.php b/src/Symfony/Component/ErrorHandler/Tests/Exception/FlattenExceptionTest.php
@@ -292,9 +292,4 @@ public function testToStringParent()
         $this->assertSame($exception->getTraceAsString(), $flattened->getTraceAsString());
         $this->assertSame($exception->__toString(), $flattened->getAsString());
     }
-
-    private function createException($foo): \Exception
-    {
-        return new \Exception();
-    }
 }
diff --git a/src/Symfony/Component/Form/Tests/ChoiceList/Factory/DefaultChoiceListFactoryTest.php b/src/Symfony/Component/Form/Tests/ChoiceList/Factory/DefaultChoiceListFactoryTest.php
@@ -854,25 +854,6 @@ public function testCreateViewFlatlabelTranslationParametersClosureReceivesValue
         $this->assertFlatViewWithlabelTranslationParameters($view);
     }
 
-    private function assertScalarListWithChoiceValues(ChoiceListInterface $list)
-    {
-        $this->assertSame(['a', 'b', 'c', 'd'], $list->getValues());
-
-        $this->assertSame([
-            'a' => 'a',
-            'b' => 'b',
-            'c' => 'c',
-            'd' => 'd',
-        ], $list->getChoices());
-
-        $this->assertSame([
-            'a' => 'A',
-            'b' => 'B',
-            'c' => 'C',
-            'd' => 'D',
-        ], $list->getOriginalKeys());
-    }
-
     private function assertObjectListWithGeneratedValues(ChoiceListInterface $list)
     {
         $this->assertSame(['0', '1', '2', '3'], $list->getValues());
@@ -892,25 +873,6 @@ private function assertObjectListWithGeneratedValues(ChoiceListInterface $list)
         ], $list->getOriginalKeys());
     }
 
-    private function assertScalarListWithCustomValues(ChoiceListInterface $list)
-    {
-        $this->assertSame(['a', 'b', '1', '2'], $list->getValues());
-
-        $this->assertSame([
-            'a' => 'a',
-            'b' => 'b',
-            1 => 'c',
-            2 => 'd',
-        ], $list->getChoices());
-
-        $this->assertSame([
-            'a' => 'A',
-            'b' => 'B',
-            1 => 'C',
-            2 => 'D',
-        ], $list->getOriginalKeys());
-    }
-
     private function assertObjectListWithCustomValues(ChoiceListInterface $list)
     {
         $this->assertSame(['a', 'b', '1', '2'], $list->getValues());
diff --git a/src/Symfony/Component/Messenger/Bridge/Redis/Transport/Connection.php b/src/Symfony/Component/Messenger/Bridge/Redis/Transport/Connection.php
@@ -524,11 +524,6 @@ public function setup(): void
         $this->autoSetup = false;
     }
 
-    private function getCurrentTimeInMilliseconds(): int
-    {
-        return (int) (microtime(true) * 1000);
-    }
-
     public function cleanup(): void
     {
         static $unlink = true;
diff --git a/src/Symfony/Component/Routing/Loader/Configurator/CollectionConfigurator.php b/src/Symfony/Component/Routing/Loader/Configurator/CollectionConfigurator.php
@@ -114,9 +114,4 @@ final public function host(string|array $host): static
 
         return $this;
     }
-
-    private function createRoute(string $path): Route
-    {
-        return (clone $this->route)->setPath($path);
-    }
 }
diff --git a/src/Symfony/Component/VarDumper/Tests/Dumper/CliDumperTest.php b/src/Symfony/Component/VarDumper/Tests/Dumper/CliDumperTest.php
@@ -449,22 +449,4 @@ public function testDumpArrayWithColor($value, $flags, $expectedOut)
 
         $this->assertSame($expectedOut, $out);
     }
-
-    private function getSpecialVars()
-    {
-        foreach (array_keys($GLOBALS) as $var) {
-            if ('GLOBALS' !== $var) {
-                unset($GLOBALS[$var]);
-            }
-        }
-
-        $var = function &() {
-            $var = [];
-            $var[] = &$var;
-
-            return $var;
-        };
-
-        return eval('return [$var(), $GLOBALS, &$GLOBALS];');
-    }
 }
diff --git a/src/Symfony/Component/VarExporter/LazyGhostTrait.php b/src/Symfony/Component/VarExporter/LazyGhostTrait.php
@@ -379,13 +379,4 @@ public function __destruct()
             parent::__destruct();
         }
     }
-
-    private function setLazyObjectAsInitialized(bool $initialized): void
-    {
-        $state = $this->lazyObjectState ?? null;
-
-        if ($state && !\is_array($state->initializer)) {
-            $state->status = $initialized ? LazyObjectState::STATUS_INITIALIZED_FULL : LazyObjectState::STATUS_UNINITIALIZED_FULL;
-        }
-    }
 }

Original file line number	Diff line number	Diff line change
`@@ -292,9 +292,4 @@ public function testToStringParent()`
`292`	`292`	`$this->assertSame($exception->getTraceAsString(), $flattened->getTraceAsString());`
`293`	`293`	`$this->assertSame($exception->__toString(), $flattened->getAsString());`
`294`	`294`	`}`
`295`		`-`
`296`		`- private function createException($foo): \Exception`
`297`		`- {`
`298`		`- return new \Exception();`
`299`		`- }`
`300`	`295`	`}`
Original file line number	Diff line number	Diff line change
`@@ -524,11 +524,6 @@ public function setup(): void`
`524`	`524`	`$this->autoSetup = false;`
`525`	`525`	`}`
`526`	`526`
`527`		`- private function getCurrentTimeInMilliseconds(): int`
`528`		`- {`
`529`		`- return (int) (microtime(true) * 1000);`
`530`		`- }`
`531`		`-`
`532`	`527`	`public function cleanup(): void`
`533`	`528`	`{`
`534`	`529`	`static $unlink = true;`
Original file line number	Diff line number	Diff line change
`@@ -114,9 +114,4 @@ final public function host(string\|array $host): static`
`114`	`114`
`115`	`115`	`return $this;`
`116`	`116`	`}`
`117`		`-`
`118`		`- private function createRoute(string $path): Route`
`119`		`- {`
`120`		`- return (clone $this->route)->setPath($path);`
`121`		`- }`
`122`	`117`	`}`
Original file line number	Diff line number	Diff line change
`@@ -379,13 +379,4 @@ public function __destruct()`
`379`	`379`	`parent::__destruct();`
`380`	`380`	`}`
`381`	`381`	`}`
`382`		`-`
`383`		`- private function setLazyObjectAsInitialized(bool $initialized): void`
`384`		`- {`
`385`		`- $state = $this->lazyObjectState ?? null;`
`386`		`-`
`387`		`- if ($state && !\is_array($state->initializer)) {`
`388`		`- $state->status = $initialized ? LazyObjectState::STATUS_INITIALIZED_FULL : LazyObjectState::STATUS_UNINITIALIZED_FULL;`
`389`		`- }`
`390`		`- }`
`391`	`382`	`}`