Fix tests

Author: PatrickePatate

tests/Console/ConfigureCmsCommandTest.php

@@ -14,14 +14,16 @@
     config(['ozu-client.website_key' => 'test']);
     Http::fake();
 
-    Schema::swap(new class {
+    Schema::swap(new class
+    {
         public function getColumnListing($table): array
         {
             return ['dummy-text', 'dummy-check', 'dummy-image', 'dummy-date', 'dummy-select'];
         }
+
         public function getColumnType($table, $column): string
         {
-            return match($column) {
+            return match ($column) {
                 'dummy-check' => 'tinyint',
                 'dummy-date' => 'date',
                 default => 'string',
@@ -48,7 +50,7 @@ public function ozuCollectionKey(): string
         },
     ]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutput('CMS configuration sent to Ozu.')
         ->assertExitCode(Command::SUCCESS);
 
@@ -93,7 +95,7 @@ public static function configureOzuCollection(OzuCollectionConfig $config): OzuC
         },
     ]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutput('CMS configuration sent to Ozu.')
         ->assertExitCode(Command::SUCCESS);
 
@@ -136,7 +138,7 @@ public static function configureOzuCollectionList(OzuCollectionListConfig $confi
         },
     ]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutput('CMS configuration sent to Ozu.')
         ->assertExitCode(Command::SUCCESS);
 
@@ -207,7 +209,7 @@ public static function configureOzuCollectionForm(OzuCollectionFormConfig $confi
         },
     ]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutput('CMS configuration sent to Ozu.')
         ->assertExitCode(Command::SUCCESS);
 
@@ -241,11 +243,13 @@ public static function configureOzuCollectionForm(OzuCollectionFormConfig $confi
 });
 
 it('sends custom fields configuration to Ozu', function () {
-    Schema::swap(new class {
+    Schema::swap(new class
+    {
         public function getColumnListing($table): array
         {
             return ['dummy-text'];
         }
+
         public function getColumnType($table, $column): string
         {
             return 'string';
@@ -262,7 +266,7 @@ public function ozuCollectionKey(): string
         },
     ]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutput('CMS configuration sent to Ozu.')
         ->assertExitCode(Command::SUCCESS);
 
@@ -290,7 +294,7 @@ public function ozuCollectionKey(): string
         },
     ]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutput('CMS configuration sent to Ozu.')
         ->assertExitCode(Command::SUCCESS);
 
@@ -329,7 +333,7 @@ public static function configureOzuCollection(OzuCollectionConfig $config): OzuC
 
     config(['ozu-client.collections' => [$parentCollectionClass]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutput('CMS configuration sent to Ozu.')
         ->assertExitCode(Command::SUCCESS);
 
@@ -370,10 +374,10 @@ public static function configureOzuCollectionList(OzuCollectionListConfig $confi
                     ->addColumn(OzuColumn::makeText('dummy-text'))
                     ->addColumn(OzuColumn::makeCheck('dummy-another-check'));
             }
-        }
+        },
     ]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutputToContain('The keys [dummy-another-check] are defined either in the list or in the form but are not custom fields of the model')
         ->assertExitCode(Command::SUCCESS);
 });
@@ -394,10 +398,10 @@ public static function configureOzuCollectionForm(OzuCollectionFormConfig $confi
                         OzuField::makeText('dummy-another-text')
                     );
             }
-        }
+        },
     ]]);
 
-    $this->artisan('ozu:configure-cms')
+    $this->artisan('ozu:configure')
         ->expectsOutputToContain('The keys [dummy-another-text] are defined either in the list or in the form but are not custom fields of the model')
         ->assertExitCode(Command::SUCCESS);
 });

tests/Console/FetchSettingsFromOzuTest.php

@@ -1,25 +1,17 @@
 <?php
 
 use Code16\OzuClient\Client as OzuClient;
-use Code16\OzuClient\Console\FetchSettingsFromOzu;
+use Code16\OzuClient\Console\FetchDataFromOzu;
 use Code16\OzuClient\Tests\Fixtures\DummySettings;
 use Illuminate\Support\Facades\Cache;
 use Mockery as m;
 
-it('fails when no settings class is configured', function () {
+it('succeeds when settings are not configured', function () {
     config(['ozu-client.settings' => null, 'cache.default' => 'array']);
 
-    $this->artisan('ozu:fetch-settings')
-        ->expectsOutput('OZU settings are not configured.')
-        ->assertExitCode(FetchSettingsFromOzu::FAILURE);
-});
-
-it('succeeds with dontFailIfNoSettingsClass option when settings are not configured', function () {
-    config(['ozu-client.settings' => null, 'cache.default' => 'array']);
-
-    $this->artisan('ozu:fetch-settings --dontFailIfNoSettingsClass')
-        ->expectsOutput('OZU settings are not configured.')
-        ->assertExitCode(FetchSettingsFromOzu::SUCCESS);
+    $this->artisan('ozu:import --settings')
+        ->expectsOutput('❌ OZU settings are not configured.')
+        ->assertExitCode(FetchDataFromOzu::SUCCESS);
 });
 
 it('fails when the Ozu client returns null (unsuccessful response)', function () {
@@ -29,9 +21,9 @@
     $mock->shouldReceive('fetchSettings')->once()->andReturn(null);
     $this->app->instance(OzuClient::class, $mock);
 
-    $this->artisan('ozu:fetch-settings')
-        ->expectsOutput('Ozu’s response was not successful.')
-        ->assertExitCode(FetchSettingsFromOzu::FAILURE);
+    $this->artisan('ozu:import --settings')
+        ->expectsOutput('❌ Ozu’s response was not successful.')
+        ->assertExitCode(FetchDataFromOzu::FAILURE);
 });
 
 it('caches fetched settings with class-based prefix and decodes JSON values', function () {
@@ -46,9 +38,9 @@
     ]);
     $this->app->instance(OzuClient::class, $mock);
 
-    $this->artisan('ozu:fetch-settings')
-        ->expectsOutput('Settings fetched successfully')
-        ->assertExitCode(FetchSettingsFromOzu::SUCCESS);
+    $this->artisan('ozu:import --settings')
+        ->expectsOutput('✅ Settings fetched successfully')
+        ->assertExitCode(FetchDataFromOzu::SUCCESS);
 
     expect(Cache::get('DummySettings_foo'))->toBe('value-foo');
     expect(Cache::get('DummySettings_bar'))->toBe(['nested' => true]);