refactor: code

refs: #17

Author: pirs1337

src/Commands/InitCommand.php

@@ -65,13 +65,7 @@ class InitCommand extends Command implements Isolatable
 
     public function handle(): void
     {
-        $this->appName = $this->argument('application-name');
-
-        $pascalCaseAppName = $this->toPascalCase($this->appName);
-
-        if (!$this->isPascalCase($this->appName) && $this->confirm("The application name is not in PascalCase, would you like to use {$pascalCaseAppName}")) {
-            $this->appName = $pascalCaseAppName;
-        }
+        $this->prepareAppName();
 
         $kebabName = Str::kebab($this->appName);
 
@@ -367,20 +361,14 @@ protected function saveReadme(): void
         file_put_contents('README.md', $this->readmeContent);
     }
 
-    protected function toPascalCase(string $string): string
+    protected function prepareAppName(): void
     {
-        // Remove non-alphanumeric characters except underscores
-        $string = preg_replace('/[^a-zA-Z0-9_]/', '', $string);
-
-        // Replace underscores with spaces, then uppercase the first letter of each word
-        $string = ucwords(str_replace('_', ' ', $string));
+        $this->appName = $this->argument('application-name');
 
-        // Remove spaces
-        return str_replace(' ', '', $string);
-    }
+        $pascalCaseAppName = ucfirst(Str::camel($this->appName));
 
-    protected function isPascalCase(string $string): bool
-    {
-        return preg_match('/^[A-Z][a-zA-Z0-9]*$/', $string);
+        if ($this->appName !== $pascalCaseAppName && $this->confirm("The application name is not in PascalCase, would you like to use {$pascalCaseAppName}")) {
+            $this->appName = $pascalCaseAppName;
+        }
     }
 }

tests/InitCommandTest.php

@@ -13,15 +13,18 @@ public function testRunWithoutAdminAndReadmeCreation()
         $this->mockFileGetContent(
             [
                 'arguments' => ['.env.example'],
-                'result' => $this->getFixture('env.example.yml'),
+                'result' => $this->getFixture('env.example_app_name_pascal_case.yml'),
             ],
             [
                 'arguments' => ['.env.development'],
-                'result' => $this->getFixture('env.development.yml'),
+                'result' => $this->getFixture('env.development_app_name_pascal_case.yml'),
             ],
         );
 
-        $this->mockFilePutContent();
+        $this->mockFilePutContent(
+            'env.example_app_name_pascal_case.yml',
+            'env.development_app_name_pascal_case.yml',
+        );
 
         $this->mockShellExec(
             ['arguments' => 'composer require ronasit/laravel-helpers --ansi'],
@@ -32,8 +35,7 @@ public function testRunWithoutAdminAndReadmeCreation()
         );
 
         $this
-            ->artisan('init "My App"')
-            ->expectsConfirmation('The application name is not in PascalCase, would you like to use MyApp')
+            ->artisan('init "MyApp"')
             ->expectsOutput('Project initialized successfully!')
             ->expectsQuestion('Please enter an application URL', 'https://mysite.com')
             ->expectsConfirmation('Do you want to generate an admin user?')

tests/Support/Traits/InitCommandMockTrait.php

@@ -12,7 +12,7 @@ trait InitCommandMockTrait
     public function mockFilePutContent(
         string $exampleEnvFixtureName = 'env.example.yml',
         string $developmentEnvFixtureName = 'env.development.yml',
-        ...$arguments
+        ...$arguments,
     ): void {
         $callChain = [
             ['.env.example', $this->getFixture($exampleEnvFixtureName), 'optionalParameter', 'optionalParameter'],