Add return types to test methods

Author: laravel-shift

tests/Feature/SetLocaleTest.php

@@ -35,7 +35,7 @@ protected function setUp(): void
     }
 
     #[Test]
-    public function it_looks_for_a_locale_in_a_custom_route_action()
+    public function it_looks_for_a_locale_in_a_custom_route_action(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('en');
@@ -56,7 +56,7 @@ public function it_looks_for_a_locale_in_a_custom_route_action()
     }
 
     #[Test]
-    public function it_looks_for_a_locale_in_the_url()
+    public function it_looks_for_a_locale_in_the_url(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('en');
@@ -73,7 +73,7 @@ public function it_looks_for_a_locale_in_the_url()
     }
 
     #[Test]
-    public function you_can_configure_which_segment_to_use_as_locale()
+    public function you_can_configure_which_segment_to_use_as_locale(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('en');
@@ -92,7 +92,7 @@ public function you_can_configure_which_segment_to_use_as_locale()
     }
 
     #[Test]
-    public function it_looks_for_custom_slugs()
+    public function it_looks_for_custom_slugs(): void
     {
         $this->setSupportedLocales([
             'en' => 'english',
@@ -112,7 +112,7 @@ public function it_looks_for_custom_slugs()
     }
 
     #[Test]
-    public function you_can_use_multiple_slugs_for_a_locale()
+    public function you_can_use_multiple_slugs_for_a_locale(): void
     {
         $this->setSupportedLocales([
             'en' => 'english',
@@ -142,7 +142,7 @@ public function you_can_use_multiple_slugs_for_a_locale()
     }
 
     #[Test]
-    public function it_looks_for_custom_domains()
+    public function it_looks_for_custom_domains(): void
     {
         $this->setSupportedLocales([
             'en' => 'english.test',
@@ -164,7 +164,7 @@ public function it_looks_for_custom_domains()
     }
 
     #[Test]
-    public function you_can_use_multiple_domains_for_a_locale()
+    public function you_can_use_multiple_domains_for_a_locale(): void
     {
         $this->setSupportedLocales([
             'en' => 'english.test',
@@ -198,7 +198,7 @@ public function you_can_use_multiple_domains_for_a_locale()
     }
 
     #[Test]
-    public function it_checks_for_a_configured_omitted_locale()
+    public function it_checks_for_a_configured_omitted_locale(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('en');
@@ -217,7 +217,7 @@ public function it_checks_for_a_configured_omitted_locale()
     }
 
     #[Test]
-    public function it_looks_for_a_locale_on_the_authenticated_user()
+    public function it_looks_for_a_locale_on_the_authenticated_user(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('en');
@@ -238,7 +238,7 @@ public function it_looks_for_a_locale_on_the_authenticated_user()
     }
 
     #[Test]
-    public function it_will_bypass_missing_attribute_exception_if_the_locale_attribute_is_missing_on_the_user_model()
+    public function it_will_bypass_missing_attribute_exception_if_the_locale_attribute_is_missing_on_the_user_model(): void
     {
         if (version_compare(App::version(), '9.35.0') === -1) {
             $this->markTestSkipped('This test only applies to Laravel 9.35.0 and higher.');
@@ -263,7 +263,7 @@ public function it_will_bypass_missing_attribute_exception_if_the_locale_attribu
     }
 
     #[Test]
-    public function it_looks_for_a_locale_in_the_session()
+    public function it_looks_for_a_locale_in_the_session(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('en');
@@ -282,7 +282,7 @@ public function it_looks_for_a_locale_in_the_session()
     }
 
     #[Test]
-    public function it_looks_for_a_locale_in_a_cookie()
+    public function it_looks_for_a_locale_in_a_cookie(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('en');
@@ -302,7 +302,7 @@ public function it_looks_for_a_locale_in_a_cookie()
     }
 
     #[Test]
-    public function it_looks_for_a_locale_in_the_browser()
+    public function it_looks_for_a_locale_in_the_browser(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('en');
@@ -321,7 +321,7 @@ public function it_looks_for_a_locale_in_the_browser()
     }
 
     #[Test]
-    public function it_returns_the_best_match_when_a_browser_locale_is_used()
+    public function it_returns_the_best_match_when_a_browser_locale_is_used(): void
     {
         $this->setSupportedLocales(['en', 'nl', 'fr']);
         $this->setAppLocale('en');
@@ -340,7 +340,7 @@ public function it_returns_the_best_match_when_a_browser_locale_is_used()
     }
 
     #[Test]
-    public function it_looks_for_the_current_app_locale()
+    public function it_looks_for_the_current_app_locale(): void
     {
         $this->setSupportedLocales(['en', 'nl']);
         $this->setAppLocale('nl');
@@ -357,7 +357,7 @@ public function it_looks_for_the_current_app_locale()
     }
 
     #[Test]
-    public function trusted_detectors_ignore_supported_locales_and_may_set_any_locale()
+    public function trusted_detectors_ignore_supported_locales_and_may_set_any_locale(): void
     {
         $this->setSupportedLocales(['en']);
         $this->setAppLocale('en');

tests/Unit/LocalizerTest.php

@@ -13,7 +13,7 @@
 class LocalizerTest extends TestCase
 {
     #[Test]
-    public function it_loops_through_the_detectors_and_returns_the_first_supported_locale()
+    public function it_loops_through_the_detectors_and_returns_the_first_supported_locale(): void
     {
         $supportedLocales = ['en', 'nl'];
         $detectors = [
@@ -28,7 +28,7 @@ public function it_loops_through_the_detectors_and_returns_the_first_supported_l
     }
 
     #[Test]
-    public function it_returns_the_first_match_if_an_array_of_locales_is_detected()
+    public function it_returns_the_first_match_if_an_array_of_locales_is_detected(): void
     {
         $supportedLocales = ['en', 'nl'];
         $detectors = [
@@ -41,7 +41,7 @@ public function it_returns_the_first_match_if_an_array_of_locales_is_detected()
     }
 
     #[Test]
-    public function trusted_detectors_ignore_supported_locales_and_may_set_any_locale()
+    public function trusted_detectors_ignore_supported_locales_and_may_set_any_locale(): void
     {
         $supportedLocales = ['en'];
         $detectors = [
@@ -57,7 +57,7 @@ public function trusted_detectors_ignore_supported_locales_and_may_set_any_local
     }
 
     #[Test]
-    public function it_skips_null_and_false_and_empty_values()
+    public function it_skips_null_and_false_and_empty_values(): void
     {
         App::instance(Detector::class, Mockery::mock(Detector::class)->allows()->detect()->andReturns('')->getMock());
 
@@ -77,7 +77,7 @@ public function it_skips_null_and_false_and_empty_values()
     }
 
     #[Test]
-    public function it_skips_null_and_false_and_empty_values_from_trusted_detectors()
+    public function it_skips_null_and_false_and_empty_values_from_trusted_detectors(): void
     {
         App::instance(Detector::class, Mockery::mock(Detector::class)->allows()->detect()->andReturns('')->getMock());
 
@@ -100,7 +100,7 @@ public function it_skips_null_and_false_and_empty_values_from_trusted_detectors(
     }
 
     #[Test]
-    public function it_returns_false_if_no_supported_locale_could_be_detected()
+    public function it_returns_false_if_no_supported_locale_could_be_detected(): void
     {
         $supportedLocales = ['en'];
         $detectors = [
@@ -115,7 +115,7 @@ public function it_returns_false_if_no_supported_locale_could_be_detected()
     }
 
     #[Test]
-    public function it_loops_through_the_stores_and_calls_the_store_method_with_the_given_locale()
+    public function it_loops_through_the_stores_and_calls_the_store_method_with_the_given_locale(): void
     {
         $stores = [
             Mockery::mock(Store::class)->expects()->store('nl')->once()->getMock(),
@@ -129,7 +129,7 @@ public function it_loops_through_the_stores_and_calls_the_store_method_with_the_
     }
 
     #[Test]
-    public function it_accepts_class_names_instead_of_instances_in_the_constructor()
+    public function it_accepts_class_names_instead_of_instances_in_the_constructor(): void
     {
         App::instance(Store::class, Mockery::mock(Store::class)->expects()->store('nl')->once()->getMock());
         App::instance(Detector::class, Mockery::mock(Detector::class)->expects()->detect()->once()->getMock());
@@ -144,7 +144,7 @@ public function it_accepts_class_names_instead_of_instances_in_the_constructor()
     }
 
     #[Test]
-    public function you_can_set_the_supported_locales_at_runtime()
+    public function you_can_set_the_supported_locales_at_runtime(): void
     {
         $supportedLocales = ['en'];
         $detectors = [