delete executions when deleting a script

vitodeploy · Jun 8, 2024 · c9c2185 · c9c2185
1 parent 849d814
commit c9c2185
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/app/Models/Script.php b/app/Models/Script.php
@@ -29,6 +29,15 @@ class Script extends AbstractModel
         'content',
     ];
 
+    public static function boot(): void
+    {
+        parent::boot();
+
+        static::deleting(function (Script $script) {
+            $script->executions()->delete();
+        });
+    }
+
     public function user(): BelongsTo
     {
         return $this->belongsTo(User::class);

diff --git a/tests/Feature/ScriptTest.php b/tests/Feature/ScriptTest.php
@@ -78,6 +78,11 @@ public function test_delete_script(): void
             'user_id' => $this->user->id,
         ]);
 
+        $scriptExecution = ScriptExecution::factory()->create([
+            'script_id' => $script->id,
+            'status' => ScriptExecutionStatus::EXECUTING,
+        ]);
+
         $this->delete(
             route('scripts.delete', [
                 'script' => $script,
@@ -87,6 +92,10 @@ public function test_delete_script(): void
         $this->assertDatabaseMissing('scripts', [
             'id' => $script->id,
         ]);
+
+        $this->assertDatabaseMissing('script_executions', [
+            'id' => $scriptExecution->id,
+        ]);
     }
 
     public function test_execute_script_and_view_log(): void