Files without match are left untouched

Signed-off-by: RJ Garcia <ragboyjr@icloud.com>
krakphp · Jun 1, 2020 · 9d27c96 · 9d27c96
1 parent fc15263
commit 9d27c96
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 1 deletion.
diff --git a/src/GenerateStruct.php b/src/GenerateStruct.php
@@ -37,6 +37,10 @@ public function __invoke(string $code): string {
         $nodeFinder = new NodeFinder;
         $factory = new BuilderFactory();
         $classesToGenerateStructs = $this->findClassesToGenerateStructs($nodeFinder, $ast);
+        if (!$classesToGenerateStructs) {
+            return $code;
+        }
+
         $alreadyExistingGeneratedTraits = $this->findAlreadyExistingGeneratedTraits($nodeFinder, $ast, $classesToGenerateStructs);
         $code = $this->removeExistingTraitsFromOriginalCode($code, $alreadyExistingGeneratedTraits);
 

diff --git a/src/struct-gen.php b/src/struct-gen.php
@@ -38,7 +38,7 @@ function generateStructsForFiles(iterable $files, LoggerInterface $logger = null
             $logger->debug('No changes detected.');
         } else {
             $logger->info("New Struct Info: \n". $updated);
+            file_put_contents($file->getPathname(), $updated);
         }
-        file_put_contents($file->getPathname(), $updated);
     }
 }
diff --git a/test/Feature/StructGenTest.php b/test/Feature/StructGenTest.php
@@ -49,4 +49,19 @@ public function can_generate_structs_for_files() {
             trim(file_get_contents(self::GENERATED_FILE_PATH))
         );
     }
+
+    /** @test */
+    public function files_without_any_matching_classes_are_left_untouched() {
+        $genStruct = new GenerateStruct();
+        $input = <<<'PHP'
+<?php
+
+interface Acme {}
+class Foo {}
+function bar() {
+
+}
+PHP;
+        $this->assertEquals($input, $genStruct($input));
+    }
 }