ZIP Imports: Added API examples, finished testing

[bookstack] / tests / LanguageTest.php

diff --git a/tests/LanguageTest.php b/tests/LanguageTest.php
index c9d6ed6f014ed16fed5c0c25107388c307b8457f..28491c3af9d3b112e359dddc4ce4d0f809e3bdd2 100644 (file)
--- a/tests/LanguageTest.php
+++ b/tests/LanguageTest.php
@@ -1,25 +1,46 @@
-<?php namespace Tests;
+<?php
+
+namespace Tests;
+
+use BookStack\Activity\ActivityType;
+use BookStack\Translation\LocaleManager;

 
 class LanguageTest extends TestCase
 {
 
 class LanguageTest extends TestCase
 {

-
-    protected $langs;
+    protected array $langs;

 
     /**
      * LanguageTest constructor.
      */
 
     /**
      * LanguageTest constructor.
      */

-    public function setUp()
+    protected function setUp(): void

     {
         parent::setUp();
     {
         parent::setUp();

-        $this->langs = array_diff(scandir(resource_path('lang')), ['..', '.', 'check.php']);
+        $this->langs = array_diff(scandir(lang_path('')), ['..', '.']);

     }
 
     }
 

-    public function test_locales_config_key_set_properly()
+    public function test_locales_list_set_properly()

     {
     {

-        $configLocales = config('app.locales');
-        sort($configLocales);
+        $appLocales = $this->app->make(LocaleManager::class)->getAllAppLocales();
+        sort($appLocales);

         sort($this->langs);
         sort($this->langs);

-        $this->assertTrue(implode(':', $this->langs) === implode(':', $configLocales), 'app.locales configuration variable matches found lang files');
+        $this->assertEquals(implode(':', $this->langs), implode(':', $appLocales), 'app.locales configuration variable does not match those found in lang files');
+    }
+
+    // Not part of standard phpunit test runs since we sometimes expect non-added langs.
+    public function test_locales_all_have_language_dropdown_entry()
+    {
+        $this->markTestSkipped('Only used when checking language inclusion');
+
+        $dropdownLocales = array_keys(trans('settings.language_select', [], 'en'));
+        sort($dropdownLocales);
+        sort($this->langs);
+        $diffs = array_diff($this->langs, $dropdownLocales);
+        if (count($diffs) > 0) {
+            $diffText = implode(',', $diffs);
+            $warning = "Languages: {$diffText} found in files but not in language select dropdown.";
+            $this->fail($warning);
+        }
+        $this->assertTrue(true);

     }
 
     public function test_correct_language_if_not_logged_in()
     }
 
     public function test_correct_language_if_not_logged_in()


@@ -31,27 +52,23 @@ class LanguageTest extends TestCase
         $loginPageFrenchReq->assertSee('Se Connecter');
     }
 
         $loginPageFrenchReq->assertSee('Se Connecter');
     }
 

-    public function test_js_endpoint_for_each_language()
+    public function test_public_lang_autodetect_can_be_disabled()

     {
     {

+        config()->set('app.auto_detect_locale', false);
+        $loginReq = $this->get('/login');
+        $loginReq->assertSee('Log In');

 
 

-        $visibleKeys = ['common', 'components', 'entities', 'errors'];
-
-        $this->asEditor();
-        foreach ($this->langs as $lang) {
-            setting()->putUser($this->getEditor(), 'language', $lang);
-            $transResp = $this->get('/translations');
-            foreach ($visibleKeys as $key) {
-                $transResp->assertSee($key);
-            }
-        }
+        $loginPageFrenchReq = $this->get('/login', ['Accept-Language' => 'fr']);
+        $loginPageFrenchReq->assertDontSee('Se Connecter');

     }
 
     public function test_all_lang_files_loadable()
     {
     }
 
     public function test_all_lang_files_loadable()
     {

-        $files = array_diff(scandir(resource_path('lang/en')), ['..', '.']);
+        $files = array_diff(scandir(lang_path('en')), ['..', '.']);

         foreach ($this->langs as $lang) {
             foreach ($files as $file) {
                 $loadError = false;
         foreach ($this->langs as $lang) {
             foreach ($files as $file) {
                 $loadError = false;

+

                 try {
                     $translations = trans(str_replace('.php', '', $file), [], $lang);
                 } catch (\Exception $e) {
                 try {
                     $translations = trans(str_replace('.php', '', $file), [], $lang);
                 } catch (\Exception $e) {


@@ -62,4 +79,29 @@ class LanguageTest extends TestCase
         }
     }
 
         }
     }
 

-}
\ No newline at end of file
+    public function test_views_use_rtl_if_rtl_language_is_set()
+    {
+        $this->asEditor()->withHtml($this->get('/'))->assertElementExists('html[dir="ltr"]');
+
+        setting()->putUser($this->users->editor(), 'language', 'ar');
+
+        $this->withHtml($this->get('/'))->assertElementExists('html[dir="rtl"]');
+    }
+
+    public function test_unknown_lang_does_not_break_app()
+    {
+        config()->set('app.locale', 'zz');
+
+        $loginReq = $this->get('/login', ['Accept-Language' => 'zz']);
+        $loginReq->assertOk();
+        $loginReq->assertSee('Log In');
+    }
+
+    public function test_all_activity_types_have_activity_text()
+    {
+        foreach (ActivityType::all() as $activityType) {
+            $langKey = 'activities.' . $activityType;
+            $this->assertNotEquals($langKey, trans($langKey, [], 'en'));
+        }
+    }
+}