Merge pull request #141 from ARCANEDEV/update-laravel_5.8

Adding laravel 5.8 support
ARCANEDEV · Mar 4, 2019 · 185532f · 185532f
2 parents de77949 + 606eb61
commit 185532f
Show file tree

Hide file tree

Showing 12 changed files with 49 additions and 58 deletions.
diff --git a/README.md b/README.md
@@ -21,7 +21,7 @@ Feel free to check out the [releases](https://github.com/ARCANEDEV/Localization/
 ### Features
 
   * Easy setup & configuration.
-  * Laravel `5.0` to `5.7` are supported.
+  * Laravel `5.0` to `5.8` are supported.
   * SEO-Friendly (Search engine optimization).
   * New extended Router to manage your localized routes.
   * Translated Eloquent Models.

diff --git a/_docs/0-Home.md b/_docs/0-Home.md
@@ -7,7 +7,7 @@ Feel free to check out the [Releases](https://github.com/ARCANEDEV/Localization/
 ### Features
 
   * Easy setup & configuration.
-  * Laravel `5.0` to `5.7` are supported.
+  * Laravel `5.0` to `5.8` are supported.
   * SEO-Friendly (Search engine optimization).
   * New extended Router to manage your localized routes.
   * Locales selector menu (Publishable & Customizable).

diff --git a/_docs/1-Installation-and-Setup.md b/_docs/1-Installation-and-Setup.md
@@ -28,6 +28,7 @@ PHP extensions:
 | ![Localization v2.x][localization_2_x] | ![Laravel v5.5][laravel_5_5]                                                                                        |
 | ![Localization v3.x][localization_3_x] | ![Laravel v5.6][laravel_5_6]                                                                                        |
 | ![Localization v4.x][localization_4_x] | ![Laravel v5.7][laravel_5_7]                                                                                        |
+| ![Localization v5.x][localization_5_x] | ![Laravel v5.8][laravel_5_8]                                                                                        |
 
 [laravel_5_0]:  https://img.shields.io/badge/v5.0-supported-brightgreen.svg?style=flat-square "Laravel v5.0"
 [laravel_5_1]:  https://img.shields.io/badge/v5.1-supported-brightgreen.svg?style=flat-square "Laravel v5.1"
@@ -37,12 +38,14 @@ PHP extensions:
 [laravel_5_5]:  https://img.shields.io/badge/v5.5-supported-brightgreen.svg?style=flat-square "Laravel v5.5"
 [laravel_5_6]:  https://img.shields.io/badge/v5.6-supported-brightgreen.svg?style=flat-square "Laravel v5.6"
 [laravel_5_7]:  https://img.shields.io/badge/v5.7-supported-brightgreen.svg?style=flat-square "Laravel v5.7"
+[laravel_5_8]:  https://img.shields.io/badge/v5.8-supported-brightgreen.svg?style=flat-square "Laravel v5.8"
 
 [localization_0_x]: https://img.shields.io/badge/version-0.*-blue.svg?style=flat-square "Localization v0.*"
 [localization_1_x]: https://img.shields.io/badge/version-1.*-blue.svg?style=flat-square "Localization v1.*"
 [localization_2_x]: https://img.shields.io/badge/version-2.*-blue.svg?style=flat-square "Localization v2.*"
 [localization_3_x]: https://img.shields.io/badge/version-3.*-blue.svg?style=flat-square "Localization v3.*"
 [localization_4_x]: https://img.shields.io/badge/version-4.*-blue.svg?style=flat-square "Localization v4.*"
+[localization_5_x]: https://img.shields.io/badge/version-5.*-blue.svg?style=flat-square "Localization v5.*"
 
 ## Composer
 

diff --git a/composer.json b/composer.json
@@ -16,13 +16,13 @@
     "require": {
         "php":               ">=7.1.3",
         "ext-json":          "*",
-        "arcanedev/support": "~4.4.0"
+        "arcanedev/support": "~4.5.0"
     },
     "require-dev": {
-        "orchestra/testbench": "~3.7.0",
+        "orchestra/testbench": "~3.8.0",
         "mockery/mockery":     "~1.0",
-        "phpunit/phpunit":     "~7.0",
-        "phpunit/phpcov":      "~5.0"
+        "phpunit/phpcov":      "~5.0|~6.0",
+        "phpunit/phpunit":     "~7.0|~8.0"
     },
     "autoload": {
         "psr-4": {

diff --git a/tests/Entities/LocaleCollectionTest.php b/tests/Entities/LocaleCollectionTest.php
@@ -25,14 +25,14 @@ class LocaleCollectionTest extends TestCase
      | -----------------------------------------------------------------
      */
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
         $this->locales = new LocaleCollection;
     }
 
-    public function tearDown()
+    public function tearDown(): void
     {
         unset($this->locales);
 

diff --git a/tests/Entities/LocaleTest.php b/tests/Entities/LocaleTest.php
@@ -24,7 +24,7 @@ class LocaleTest extends TestCase
      | -----------------------------------------------------------------
      */
 
-    public function tearDown()
+    public function tearDown(): void
     {
         unset($this->locale);
 
@@ -80,7 +80,7 @@ public function it_can_convert_entity_to_array()
     {
         $this->locale = $this->makeLocale('en');
 
-        static::assertInternalType('array', $this->locale->toArray());
+        static::assertIsArray($this->locale->toArray());
     }
 
     /** @test */

diff --git a/tests/LocalizationServiceProviderTest.php b/tests/LocalizationServiceProviderTest.php
@@ -23,14 +23,14 @@ class LocalizationServiceProviderTest extends TestCase
      | -----------------------------------------------------------------
      */
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
         $this->provider = $this->app->getProvider(LocalizationServiceProvider::class);
     }
 
-    public function tearDown()
+    public function tearDown(): void
     {
         unset($this->provider);
 

diff --git a/tests/LocalizationTest.php b/tests/LocalizationTest.php
@@ -22,14 +22,12 @@ public function it_can_be_instantiated()
         static::assertInstanceOf(Localization::class, localization());
     }
 
-    /**
-     * @test
-     *
-     * @expectedException         \Arcanedev\Localization\Exceptions\UnsupportedLocaleException
-     * @expectedExceptionMessage  Laravel default locale [jp] is not in the `supported-locales` array.
-     */
+    /** @test */
     public function it_must_throw_unsupported_locale_exception_on_default_locale()
     {
+        $this->expectException(\Arcanedev\Localization\Exceptions\UnsupportedLocaleException::class);
+        $this->expectExceptionMessage('Laravel default locale [jp] is not in the `supported-locales` array.');
+
         app('config')->set('app.locale', 'jp');
 
         new Localization(
@@ -66,13 +64,11 @@ public function it_can_set_and_get_supported_locales()
         }
     }
 
-    /**
-     * @test
-     *
-     * @expectedException  \Arcanedev\Localization\Exceptions\UndefinedSupportedLocalesException
-     */
+    /** @test */
     public function it_must_throw_undefined_supported_locales_exception_on_set_supported_locales_with_empty_array()
     {
+        $this->expectException(\Arcanedev\Localization\Exceptions\UndefinedSupportedLocalesException::class);
+
         localization()->setSupportedLocales([]);
     }
 
@@ -333,14 +329,12 @@ public function it_can_get_url_from_route_name_translated()
         );
     }
 
-    /**
-     * @test
-     *
-     * @expectedException         \Arcanedev\Localization\Exceptions\UnsupportedLocaleException
-     * @expectedExceptionMessage  Locale 'jp' is not in the list of supported locales.
-     */
+    /** @test */
     public function it_must_throw_an_exception_on_unsupported_locale()
     {
+        $this->expectException(\Arcanedev\Localization\Exceptions\UnsupportedLocaleException::class);
+        $this->expectExceptionMessage("Locale 'jp' is not in the list of supported locales.");
+
         localization()->getUrlFromRouteName('jp', 'localization::routes.about');
     }
 
@@ -456,11 +450,11 @@ public function it_can_render_locales_navigation_bar()
     {
         $navbar = localization()->localesNavbar();
 
-        static::assertContains('<ul class="navbar-locales">', $navbar);
-        static::assertContains('<li class="active">', $navbar);
-        static::assertContains(e('English'),  $navbar);
-        static::assertContains(e('Español'),  $navbar);
-        static::assertContains(e('Français'), $navbar);
+        static::assertStringContainsString('<ul class="navbar-locales">', $navbar);
+        static::assertStringContainsString('<li class="active">', $navbar);
+        static::assertStringContainsString(e('English'),  $navbar);
+        static::assertStringContainsString(e('Español'),  $navbar);
+        static::assertStringContainsString(e('Français'), $navbar);
     }
 
     /** @test */

diff --git a/tests/Routing/RouterTest.php b/tests/Routing/RouterTest.php
@@ -24,14 +24,14 @@ class RouterTest extends TestCase
      | -----------------------------------------------------------------
      */
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
         $this->router = app('router');
     }
 
-    public function tearDown()
+    public function tearDown(): void
     {
         unset($this->router);
 

diff --git a/tests/TranslatableModelTest.php b/tests/TranslatableModelTest.php
@@ -24,7 +24,7 @@ class TranslatableModelTest extends TestCase
      | -----------------------------------------------------------------
      */
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
@@ -256,14 +256,12 @@ public function it_can_forget_all_translations()
         static::assertSame('', $this->model->getTranslation('slug', 'en'));
     }
 
-    /**
-     * @test
-     *
-     * @expectedException        \Arcanedev\Localization\Exceptions\UntranslatableAttributeException
-     * @expectedExceptionMessage The attribute `untranslated` is untranslatable because it's not available in the translatable array: `name, slug`
-     */
+    /** @test */
     public function it_will_throw_an_exception_when_trying_to_translate_an_untranslatable_attribute()
     {
+        $this->expectException(\Arcanedev\Localization\Exceptions\UntranslatableAttributeException::class);
+        $this->expectExceptionMessage("The attribute `untranslated` is untranslatable because it's not available in the translatable array: `name, slug`");
+
         $this->model->setTranslation('untranslated', 'en', 'value');
     }
 

diff --git a/tests/Utilities/LocalesManagerTest.php b/tests/Utilities/LocalesManagerTest.php
@@ -25,7 +25,7 @@ class LocalesManagerTest extends TestCase
      | -----------------------------------------------------------------
      */
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
@@ -34,7 +34,7 @@ public function setUp()
         $this->localesManager->setCurrentLocale('en');
     }
 
-    public function tearDown()
+    public function tearDown(): void
     {
         unset($this->localesManager);
 
@@ -174,24 +174,20 @@ public function it_can_set_and_get_default_locale()
         }
     }
 
-    /**
-     * @test
-     *
-     * @expectedException        \Arcanedev\Localization\Exceptions\UnsupportedLocaleException
-     * @expectedExceptionMessage Laravel default locale [jp] is not in the `supported-locales` array.
-     */
+    /** @test */
     public function it_must_throw_unsupported_locale_exception_on_set_default_locale()
     {
+        $this->expectException(\Arcanedev\Localization\Exceptions\UnsupportedLocaleException::class);
+        $this->expectExceptionMessage("Laravel default locale [jp] is not in the `supported-locales` array.");
+
         $this->localesManager->setDefaultLocale('jp');
     }
 
-    /**
-     * @test
-     *
-     * @expectedException  \Arcanedev\Localization\Exceptions\UndefinedSupportedLocalesException
-     */
+    /** @test */
     public function it_must_throw_undefined_supported_locales_exception_on_set_with_empty_array()
     {
+        $this->expectException(\Arcanedev\Localization\Exceptions\UndefinedSupportedLocalesException::class);
+
         $this->localesManager->setSupportedLocales([]);
     }
 }
diff --git a/tests/Utilities/NegotiatorTest.php b/tests/Utilities/NegotiatorTest.php
@@ -26,14 +26,14 @@ class NegotiatorTest extends TestCase
      | -----------------------------------------------------------------
      */
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
         $this->negotiator = app(\Arcanedev\Localization\Contracts\Negotiator::class);
     }
 
-    public function tearDown()
+    public function tearDown(): void
     {
         unset($this->negotiator);