refactor: edit for integration.

Author: datamweb

src/Commands/Setup.php

@@ -87,9 +87,9 @@ private function publishConfig(): void
         $this->setupHelper();
         $this->setupRoutes();
 
-        $this->runMigrations();
-
         $this->setSecurityCSRF();
+
+        $this->runMigrations();
     }
 
     /**
@@ -260,18 +260,6 @@ private function setupRoutes(): void
         $this->add($file, $check, $pattern, $replace);
     }
 
-    private function runMigrations(): void
-    {
-        if (
-            $this->cliPrompt('  Run `spark migrate --all` now?', ['y', 'n']) === 'n'
-        ) {
-            return;
-        }
-
-        $command = new Migrate(Services::logger(), Services::commands());
-        $command->run(['all' => null]);
-    }
-
     /**
      * @see https://github.com/codeigniter4/shield/security/advisories/GHSA-5hm8-vh6r-2cjq
      */
@@ -302,12 +290,24 @@ private function setSecurityCSRF(): void
         }
 
         if (write_file($path, $output)) {
-            CLI::write(CLI::color('  Update: ', 'green') . "We have updated file '{$cleanPath}' for security reasons.");
+            CLI::write(CLI::color('  Updated: ', 'green') . "We have updated file '{$cleanPath}' for security reasons.");
         } else {
             CLI::error("  Error updating file '{$cleanPath}'.");
         }
     }
 
+    private function runMigrations(): void
+    {
+        if (
+            $this->cliPrompt('  Run `spark migrate --all` now?', ['y', 'n']) === 'n'
+        ) {
+            return;
+        }
+
+        $command = new Migrate(Services::logger(), Services::commands());
+        $command->run(['all' => null]);
+    }
+
     /**
      * This method is for testing.
      */

tests/Commands/SetupTest.php

@@ -69,17 +69,13 @@ public function testRun(): void
             '  Created: vfs://root/Config/Auth.php
   Created: vfs://root/Config/AuthGroups.php
   Updated: vfs://root/Controllers/BaseController.php
-  Updated: vfs://root/Config/Routes.php',
+  Updated: vfs://root/Config/Routes.php
+  Updated: We have updated file \'vfs://root/Config/Security.php\' for security reasons.',
             $result
         );
         $this->assertStringContainsString(
             'Running all new migrations...',
             $result
         );
-
-        $this->assertStringContainsString(
-            'Update: We have updated file \'vfs://root/Config/Security.php\' for security reasons.',
-            $result
-        );
     }
 }