Ignore abstract classes model (#19)

sethsandaru · Oct 29, 2023 · 292d4db · 292d4db
1 parent c70834d
commit 292d4db
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 1 deletion.
diff --git a/src/Commands/BulkEloquentDocsGeneratorCommand.php b/src/Commands/BulkEloquentDocsGeneratorCommand.php
@@ -76,10 +76,15 @@ private function getModels(): Collection
             ->map(function (string $file) {
                 $className = $this->getFullyQualifiedClassName($file);
 
+                $initializedModel = rescue(
+                    fn () => app()->make($className),
+                    fn () => null
+                );
+
                 return [
                     'modelClass' => $className,
                     'modelFilePath' => $file,
-                    'modelInstance' => app($className),
+                    'modelInstance' => $initializedModel,
                 ];
             })
             ->filter(fn ($model) => $model['modelInstance'] instanceof Model)

diff --git a/tests/Features/BulkEloquentDocsGeneratorCommandTest.php b/tests/Features/BulkEloquentDocsGeneratorCommandTest.php
@@ -18,6 +18,7 @@ public function testCommandReturnsOkGeneratedDocsForAllModelFilesInPath()
         $this->artisan('eloquent:bulk-phpdoc "tests/Fixtures/BulkFixtures/*.php"')
             ->expectsOutputToContain('Generated & Saved for:')
             ->expectsOutputToContain('Generated EloquentDocs for 3 model file(s)')
+            ->doesntExpectOutputToContain('BaseModel')
             ->assertSuccessful();
 
         // the generation & storing actually tested in EloquentDocsGeneratorCommandTest, no need to re-assert the same here.

diff --git a/tests/Fixtures/BulkFixtures/BaseModel.php b/tests/Fixtures/BulkFixtures/BaseModel.php
@@ -0,0 +1,10 @@
+<?php
+
+namespace SethPhat\EloquentDocs\Tests\Fixtures\BulkFixtures;
+
+use Illuminate\Database\Eloquent\Model;
+
+abstract class BaseModel extends Model
+{
+
+}