Filters for JSON files, closes #8

Author: nullthoughts

src/Updater.php

@@ -106,9 +106,12 @@ protected function getFiles(string $directory, $model = null)
             return Collection::wrap($directory . '/' . $model . '.json');
         }
 
-        return collect(File::files($directory))->map(function ($path) {
-            return $path->getPathname();
-        });
+        return collect(File::files($directory))
+            ->filter(function($file) {
+                return pathinfo($file, PATHINFO_EXTENSION) == 'json';
+            })->map(function ($path) {
+                return $path->getPathname();
+            });
     }
 
     /**

tests/Unit/UpdaterTest.php

@@ -131,4 +131,13 @@ public function exception_is_thrown_if_imports_are_in_incorrect_order()
         $updater = new UpdaterFake(__DIR__ . '/../test-data/ordered');
         $updater->run();
     }
+
+    /** @test */
+    public function it_ignores_non_json_files()
+    {
+        $updater = new UpdaterFake(__DIR__ . '/../test-data/not-json');
+        $updater->run();
+
+        $this->assertDatabaseMissing('roles', ['slug' => 'update-student-records']);
+    }
 }

tests/test-data/not-json/roles.txt

@@ -0,0 +1,9 @@
+[
+    {
+        "_slug": "update-student-records",
+        "category": "testing",
+        "supervisor": {
+            "name": "CEO"
+        }
+    }
+]