Fix: Move responsibility to named constructor

ergebnis · Dec 1, 2022 · f3698c1 · f3698c1
1 parent bb4c645
commit f3698c1
Showing 1 changed file with 16 additions and 15 deletions.
diff --git a/test/Util/State.php b/test/Util/State.php
@@ -15,25 +15,26 @@
 
 final class State
 {
-    private File $composerJsonFile;
-    private File $composerLockFile;
-
-    private function __construct(private Directory $directory)
-    {
-        $this->composerJsonFile = File::fromPath(\sprintf(
-            '%s/composer.json',
-            $directory->path(),
-        ));
-
-        $this->composerLockFile = File::fromPath(\sprintf(
-            '%s/composer.lock',
-            $directory->path(),
-        ));
+    private function __construct(
+        private Directory $directory,
+        private File $composerJsonFile,
+        private File $composerLockFile,
+    ) {
     }
 
     public static function fromDirectory(Directory $directory): self
     {
-        return new self($directory);
+        return new self(
+            $directory,
+            File::fromPath(\sprintf(
+                '%s/composer.json',
+                $directory->path(),
+            )),
+            File::fromPath(\sprintf(
+                '%s/composer.lock',
+                $directory->path(),
+            )),
+        );
     }
 
     public function directory(): Directory