Merge pull request #206 from creative-commoners/pulls/2.1/remove-broken-i18n-tests

FIX Remove providePermissions tests and update Travis build configuration

.travis.yml

@@ -11,15 +11,13 @@ php:
   - 5.3
   - 5.4
   - 5.5
-  - 5.6
-  - 7.0
 
 before_install:
   - pip install --user codecov
 
 env:
   global:
-    - DB=MYSQL CORE_RELEASE=3.1
+    - DB=MYSQL CORE_RELEASE=3.5
     - MODULE_PATH=comments
 
     # Set to 1 in the matrix to enable code coverage
@@ -29,15 +27,13 @@ matrix:
   include:
     - php: 5.6
       #CommentsListTest breaks with this env: DB=MYSQL CORE_RELEASE=3.2 COVERAGE=1
-      env: DB=SQLITE CORE_RELEASE=3.2 COVERAGE=1
+      env: DB=SQLITE CORE_RELEASE=3.5 COVERAGE=1
+    - php: 7.1
+      env: DB=MYSQL CORE_RELEASE=3.6
     - php: 5.6
+      env: DB=PGSQL CORE_RELEASE=3.5
+    - php: 7.1
       env: DB=MYSQL CORE_RELEASE=3
-    - php: 5.6
-      env: DB=MYSQL CORE_RELEASE=3.1
-    - php: 5.6
-      env: DB=PGSQL CORE_RELEASE=3.2
-  allow_failures:
-    - php: 7.0
 
 before_script:
   - phpenv rehash

tests/CommentAdminTest.php

@@ -2,34 +2,7 @@
 
 class CommentAdminTest extends SapphireTest
 {
-
     protected $usesDatabase = true;
-    
-    public function testProvidePermissions()
-    {
-        $commentAdmin = new CommentAdmin();
-        $locale = i18n::get_locale();
-
-        i18n::set_locale('fr');
-        $expected = array(
-            'CMS_ACCESS_CommentAdmin' => array(
-                # FIXME - this is a bug, missing from lang.yml files
-                'name' => 'Access to \'Comments\' section',
-                'category' => 'Accès au CMS'
-            )
-        );
-        $this->assertEquals($expected, $commentAdmin->providePermissions());
-
-        i18n::set_locale($locale);
-        $expected = array(
-            'CMS_ACCESS_CommentAdmin' => array(
-                # FIXME - this is a bug, missing from lang.yml files
-                'name' => 'Access to \'Comments\' section',
-                'category' => 'CMS Access'
-            )
-        );
-        $this->assertEquals($expected, $commentAdmin->providePermissions());
-    }
 
     public function testGetEditForm()
     {