Merge pull request #10 from ruchit288/main

feat: Add pan:delete command with ID-specific deletion and test case.

Author: MrPunyapal

src/Adapters/Laravel/Console/Commands/PanDeleteCommand.php

@@ -0,0 +1,53 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Pan\Adapters\Laravel\Console\Commands;
+
+use Illuminate\Console\Command;
+use Pan\Contracts\AnalyticsRepository;
+
+final class PanDeleteCommand extends Command
+{
+    /**
+     * The name and signature of the console command.
+     *
+     * @var string
+     */
+    protected $signature = 'pan:delete {id : The ID of the analytic to delete}';
+
+    /**
+     * The console command description.
+     *
+     * @var string
+     */
+    protected $description = 'Delete analytics by ID.';
+
+    /**
+     * Execute the console command.
+     */
+    public function handle(AnalyticsRepository $repository): void
+    {
+        $id = (int) $this->argument('id');
+
+        if ($this->isInvalidId($id)) {
+            $this->error('Analytic ID must be greater than 0.');
+
+            return;
+        }
+
+        if ($repository->delete($id) !== 0) {
+            $this->info('Analytic has been deleted.');
+        } else {
+            $this->error('Record not found or already deleted.');
+        }
+    }
+
+    /**
+     * Check if the ID is invalid.
+     */
+    private function isInvalidId(int $id): bool
+    {
+        return $id <= 0;
+    }
+}

src/Adapters/Laravel/Providers/PanServiceProvider.php

@@ -10,6 +10,7 @@
 use Illuminate\Support\ServiceProvider;
 use Pan\Adapters\Laravel\Console\Commands\InstallPanCommand;
 use Pan\Adapters\Laravel\Console\Commands\PanCommand;
+use Pan\Adapters\Laravel\Console\Commands\PanDeleteCommand;
 use Pan\Adapters\Laravel\Console\Commands\PanFlushCommand;
 use Pan\Adapters\Laravel\Http\Controllers\EventController;
 use Pan\Adapters\Laravel\Http\Middleware\InjectJavascriptLibrary;
@@ -97,6 +98,7 @@ private function registerCommands(): void
                 InstallPanCommand::class,
                 PanCommand::class,
                 PanFlushCommand::class,
+                PanDeleteCommand::class,
             ]);
         }
     }

src/Adapters/Laravel/Repositories/DatabaseAnalyticsRepository.php

@@ -85,4 +85,12 @@ private function connection(): Connection
     {
         return $this->databaseManager->connection($this->config->getDatabaseConnection());
     }
+
+    /**
+     * Delete a specific analytic by ID.
+     */
+    public function delete(int $id): int
+    {
+        return DB::table('pan_analytics')->where('id', $id)->delete();
+    }
 }

src/Contracts/AnalyticsRepository.php

@@ -28,4 +28,9 @@ public function increment(string $name, EventType $event): void;
      * Flush all analytics.
      */
     public function flush(): void;
+
+    /**
+     * Delete a specific analytic by ID.
+     */
+    public function delete(int $id): int;
 }

tests/Feature/Laravel/Console/PanDeleteTest.php

@@ -0,0 +1,38 @@
+<?php
+
+use Pan\Contracts\AnalyticsRepository;
+
+beforeEach(function (): void {
+    $this->repository = mock(AnalyticsRepository::class)->makePartial();
+    app()->instance(AnalyticsRepository::class, $this->repository);
+});
+
+it('deletes a specific analytic by ID', function (): void {
+    $this->repository
+        ->expects('delete')
+        ->with(1)
+        ->once()
+        ->andReturn(1);
+
+    $this->artisan('pan:delete', ['id' => 1])
+        ->expectsOutput('Analytic has been deleted.')
+        ->assertExitCode(0);
+});
+
+it('fails when no argument is provided', function (): void {
+    $this->artisan('pan:delete')
+        ->expectsOutput('Not enough arguments (missing: "id").')
+        ->assertExitCode(1);
+});
+
+it('handles non-existent analytic gracefully', function (): void {
+    $this->repository
+        ->expects('delete')
+        ->with(26)
+        ->once()
+        ->andReturn(0);
+
+    $this->artisan('pan:delete', ['id' => 26])
+        ->expectsOutput('Record not found or already deleted.')
+        ->assertExitCode(0);
+});