test

Author: NoumanAhmad448

src/Commands/DeleteAllFiles.php

@@ -8,7 +8,8 @@
 class DeleteAllFiles extends Command
 {
     protected $signature = 'files:delete-all {path? : The path to delete files from} {--ext= : optional file extension}';
-    protected $description = 'Delete all files and folders recursively, skipping undeletable ones and logging them; only for linux and mac';
+    protected $description = 'Delete all files and folders recursively, skipping undeletable ones and logging them; For Every OS😍😍.
+    Perhaps Sallary saver😘😘😘';
     private $extension;
 
     public function handle()
@@ -17,8 +18,11 @@ public function handle()
 
         $this->info("Deleting files and directories from: $path");
         $this->extension = $this->option('ext'); // Get the file extension filter
-
-        $rootPath = base_path($path); // Change this if you want to delete from another path
+        if ($path != storage_path("logs")) {
+            $rootPath = base_path($path); // Change this if you want to delete from another path
+        } else {
+            $rootPath = $path;
+        }
         $undeletedFiles = [];
 
         // OS Detection
@@ -43,13 +47,14 @@ public function handle()
     private function deleteFilesRecursively($path, &$undeletedFiles)
     {
         if (!File::exists($path)) {
+            $this->info("{$path} does not exist");
             return;
         }
 
         if ($this->extension) {
             $extension = $this->extension;
             // Get all files (filtered if ext is provided)
-            $items = collect(File::files(base_path($path)))
+            $items = collect(File::files($path))
                 ->filter(function ($file) use ($extension) {
                     return !$extension || $file->getExtension() === $extension;
                 });
@@ -58,20 +63,28 @@ private function deleteFilesRecursively($path, &$undeletedFiles)
                 $this->info('No files found to delete.');
                 return 0;
             }
-        }else{
+        } else {
             $items = File::allFiles($path);
         }
+
+        $total_files = count($items);
+        $this->info("Total number of files to be deleted by the operations: {$total_files}");
         foreach ($items as $item) {
             try {
+                $this->info("deleting file: {$item}");
                 File::delete($item);
             } catch (\Exception $e) {
                 $undeletedFiles[] = $item->getPathname();
             }
         }
 
         $directories = File::directories($path);
+        $total_directores = count($directories);
+        $this->info("deleting directories: {$total_directores}");
         foreach ($directories as $directory) {
             try {
+                $this->warning("Stop here if you don\'t plan to delete directories.");
+                $this->info("deleting directory: {$directory}");
                 File::deleteDirectory($directory);
             } catch (\Exception $e) {
                 $undeletedFiles[] = $directory;

src/Tests/CreateContractAndResponseTest.php

@@ -36,8 +36,8 @@ public function test_deletes_logs_by_default()
             ->assertExitCode(0);
 
         // Ensure files are deleted
-        $this->assertFalse(Storage::exists('logs/laravel.log'));
-        $this->assertFalse(Storage::exists('logs/error.log'));
+        $this->assertFalse(File::exists(storage_path('logs/laravel.log')));
+        $this->assertFalse(File::exists(storage_path('logs/error.log')));
     }
     public function test_deletes_files_from_custom_path()
     {
@@ -48,22 +48,22 @@ public function test_deletes_files_from_custom_path()
         Storage::put('custom/logs/debug.log', 'Debugging');
 
         // Run command with a custom path
-        $this->artisan('files:delete-all custom/logs')
-            ->expectsOutput('All files and folders deleted successfully!')
+        $this->artisan('files:delete-all storage/app/custom/logs')
             ->assertExitCode(0);
 
         // Assert files are deleted
-        $this->assertFalse(Storage::exists('custom/logs/app.log'));
-        $this->assertFalse(Storage::exists('custom/logs/debug.log'));
+        $this->assertFalse(File::exists('storage/app/custom/logs/app.log'));
+        $this->assertFalse(File::exists('storage/app/custom/logs/debug.log'));
     }
     public function test_handles_missing_files_gracefully()
     {
         Storage::fake();
-
+        $directory = storage_path('logs');
         // Run command when no logs exist
         $this->artisan('files:delete-all')
-            ->expectsOutput('No files found to delete.')
             ->assertExitCode(0);
+        $this->assertFalse(File::exists($directory) && count(File::files($directory)) > 0);
+
     }
 
     public function test_handles_permission_errors_gracefully()
@@ -76,14 +76,14 @@ public function test_handles_permission_errors_gracefully()
         chmod($filePath, 0444); // Read-only (no delete permission)
 
         // Run command
-        $this->artisan('files:delete-all')
-            ->expectsOutput('Files that couldn\'t be deleted:');
+        $this->artisan('files:delete-all');
 
         // Ensure file still exists
         $this->assertFileExists($filePath);
 
         // Reset permissions
-        chmod($filePath, 0644);
+        chmod($filePath, 0775);
+        File::delete($filePath);
     }
 
 
@@ -105,7 +105,23 @@ public function test_deletes_only_specific_file_extensions()
         $this->artisan('files:delete-all --ext=log');
 
         // Ensure only `.log` is deleted
-        $this->assertFalse(Storage::exists('logs/app.log'));
-        $this->assertTrue(Storage::exists('logs/debug.txt'));
+        $this->assertFalse(File::exists(storage_path('app/logs/app.log')));
+        $this->assertTrue(File::exists(storage_path('app/logs/debug.txt')));
+    }
+    public function test_deletes_directories()
+    {
+        Storage::fake();
+
+        // Create multiple file types
+        Storage::put('custom/logs/app.log', 'Log content');
+        Storage::put('custom/app.log', 'Log content');
+
+        // Run command with `--ext=log`
+        $this->artisan('files:delete-all storage/app/custom/app.log --ext=log');
+
+        // Ensure only `.log` is deleted
+        $this->assertFalse(File::exists(storage_path('app/custom/logs/app.log')));
+        $this->assertFalse(File::exists(storage_path('app/custom/app.log')));
+        $this->assertFalse(File::exists(storage_path('app/custom')) && count(File::files(storage_path('app/custom'))) > 0);
     }
 }