Fixes preloader command.

Author: DarkGhostHunter

src/Console/Commands/Stub.php

@@ -5,7 +5,9 @@
 use Illuminate\Console\Command;
 use Illuminate\Contracts\Config\Repository as ConfigContract;
 use Illuminate\Filesystem\Filesystem;
+use Laragear\Preload\Preloader;
 use Symfony\Component\Console\Attribute\AsCommand;
+use const DIRECTORY_SEPARATOR;
 
 /**
  * @internal
@@ -39,25 +41,30 @@ class Stub extends Command
      */
     public function handle(Filesystem $file, ConfigContract $config): void
     {
-        $path = $config->get('preload.path');
+        $dir = $config->get('preload.path');
+        $filePath = $dir . '/' . Preloader::NAME_PRELOAD;
 
-        if ($file->exists($path)) {
+        $file->ensureDirectoryExists($dir);
+
+        if ($file->exists($filePath)) {
             $this->info('A preload script file already exists.');
 
             return;
         }
 
-        $file->put($path, <<<'STUB'
+        $file->put($filePath, <<<'STUB'
 <?php
 
-\fwrite(\STDOUT, 'Info: This is a stub file to be replaced for the application at runtime.');
+$date = (new DateTime())->format('d-m-Y H:i:s');
+
+echo "[$date] Info: This is a preload stub file to be replaced for the application at runtime.";
 
 STUB
         );
 
-        $this->info("Stub copied at [$path].");
+        $this->info("Stub copied at [$filePath].");
         $this->newLine();
         $this->comment('Remember to edit your [php.ini] file:');
-        $this->comment("opcache.preload = $path");
+        $this->comment("opcache.preload = $filePath");
     }
 }

src/Preloader.php

@@ -10,30 +10,30 @@
 class Preloader
 {
     /**
-     * The location of the statistic file.
+     * The filename for the statistics file.
      *
      * @const string
      */
-    public const STUB_STATISTICS = __DIR__.'/../stubs/statistics.md';
+    public const NAME_STATISTICS = 'statistics.md';
 
     /**
-     * The location of the preload script stub.
+     * The filename for the preload script file.
+     *
+     * @const string
      */
-    public const STUB_PRELOAD = __DIR__.'/../stubs/preload.php.stub';
+    public const NAME_PRELOAD = 'preload.php';
 
     /**
-     * The filename for the statistics file.
+     * The location of the statistic file.
      *
      * @const string
      */
-    public const NAME_STATISTICS = 'statistics.md';
+    public const STUB_STATISTICS = __DIR__.'/../stubs/' . self::NAME_STATISTICS;
 
     /**
-     * The filename for the preload script file.
-     *
-     * @const string
+     * The location of the preload script stub.
      */
-    public const NAME_PRELOAD = 'preload.php';
+    public const STUB_PRELOAD = __DIR__.'/../stubs/' .  self::NAME_PRELOAD . '.stub';
 
     /**
      * The filename of the list of preload files.

tests/Console/Commands/StubTest.php

@@ -3,38 +3,44 @@
 namespace Tests\Console\Commands;
 
 use Illuminate\Filesystem\Filesystem;
+use Laragear\Preload\Preloader;
 use Mockery\MockInterface;
 use Tests\TestCase;
 
 class StubTest extends TestCase
 {
     public function test_stores_placeholder_in_output_path(): void
     {
-        $path = $this->app->basePath();
+        $dir = $this->app->basePath();
+        $filePath = $dir . '/' . Preloader::NAME_PRELOAD;
 
-        $this->mock(Filesystem::class, function (MockInterface $mock) use ($path) {
-            $mock->expects('exists')->andReturnFalse();
-            $mock->expects('put')->with($path, <<<'PHP'
+        $this->mock(Filesystem::class, function (MockInterface $mock) use ($dir, $filePath) {
+            $mock->expects('ensureDirectoryExists')->with($dir);
+            $mock->expects('exists')->with($filePath)->andReturnFalse();
+            $mock->expects('put')->with($filePath, <<<'PHP'
 <?php
 
-\fwrite(\STDOUT, 'Info: This is a stub file to be replaced for the application at runtime.');
+$date = (new DateTime())->format('d-m-Y H:i:s');
+
+echo "[$date] Info: This is a preload stub file to be replaced for the application at runtime.";
 
 PHP
             );
         });
 
         $command = $this->artisan('preload:stub');
 
-        $command->expectsOutput("Stub copied at [$path].");
+        $command->expectsOutput("Stub copied at [$filePath].");
         $command->expectsOutput('Remember to edit your [php.ini] file:');
-        $command->expectsOutput("opcache.preload = $path");
+        $command->expectsOutput("opcache.preload = $filePath");
 
         $command->assertSuccessful();
     }
 
     public function test_doesnt_overwrite_same_placeholder(): void
     {
         $this->mock(Filesystem::class, function (MockInterface $mock) {
+            $mock->expects('ensureDirectoryExists');
             $mock->expects('exists')->andReturnTrue();
             $mock->expects('put')->never();
         });