Merge pull request #162 from kabalin/bump_codechecker_310

Bump to codechecker v3.1.0

Author: stronk7

.github/workflows/test.yml

@@ -53,10 +53,6 @@ jobs:
             moodle-branch: 'MOODLE_39_STABLE'
           - php: '7.4'
             moodle-branch: 'MOODLE_38_STABLE'
-          - php: '7.2'
-            moodle-branch: 'MOODLE_35_STABLE'
-          - php: '7.0'
-            moodle-branch: 'MOODLE_35_STABLE'
 
     steps:
     - name: Check out repository code

.travis.yml

@@ -81,7 +81,3 @@ jobs:
       env: MOODLE_BRANCH=MOODLE_39_STABLE
     - php: 7.4
       env: MOODLE_BRANCH=MOODLE_38_STABLE
-    - php: 7.2
-      env: MOODLE_BRANCH=MOODLE_35_STABLE
-    - php: 7.0
-      env: MOODLE_BRANCH=MOODLE_35_STABLE

composer.json

@@ -40,11 +40,11 @@
       "type": "package",
       "package": {
         "name": "moodlehq/moodle-local_codechecker",
-        "version": "3.0.6",
+        "version": "3.1.0",
         "source": {
           "url": "https://github.com/moodlehq/moodle-local_codechecker.git",
           "type": "git",
-          "reference": "v3.0.6"
+          "reference": "v3.1.0"
         }
       }
     },
@@ -75,7 +75,7 @@
   ],
   "require": {
     "php": ">=7.0.8",
-    "moodlehq/moodle-local_codechecker": "^3.0.6",
+    "moodlehq/moodle-local_codechecker": "^3.1",
     "moodlehq/moodle-local_ci": "^1.0.12",
     "moodlehq/moodle-local_moodlecheck": "^1.0.9",
     "sebastian/phpcpd": "^3.0",

composer.lock

@@ -88,7 +88,7 @@ public function testExecuteFail()
         $output = $this->getActualOutput();
         $this->assertRegExp('/E\.* 8\.* \/ 8 \(100%\)/', $output);                  // Progress.
         $this->assertRegExp('/\/fixable.php/', $output);                            // File.
-        $this->assertRegExp('/ (4|5) ERRORS AND 2 WARNINGS AFFECTING 6 /', $output); // Summary (php70 shows one less)
+        $this->assertRegExp('/ (4|5) ERRORS AND (1|2) WARNINGS? AFFECTING 6 /', $output); // Summary (php70 shows one less)
         $this->assertRegexp('/moodle\.Files\.BoilerplateComment\.Wrong/', $output); // Moodle sniff.
         $this->assertRegexp('/print_object\(\) is forbidden/', $output);            // Moodle sniff.
         $this->assertRegexp('/FunctionUse\.RemovedFunctions\.ldap_sort/', $output); // PHPCompatibility sniff.

tests/Command/CodeCheckerCommandTest.php

@@ -40,6 +40,8 @@
 class lib_test extends \basic_testcase {
     /**
      * Test addition.
+     *
+     * @covers ::local_ci_add
      */
     public function test_local_ci_add() {
         $this->assertEquals(4, local_ci_add(2, 2));
@@ -49,6 +51,8 @@ public function test_local_ci_add() {
 
     /**
      * Test subtraction.
+     *
+     * @covers ::local_ci_substract
      */
     public function test_local_ci_subtract() {
         $this->assertEquals(0, local_ci_subtract(2, 2));
@@ -58,6 +62,8 @@ public function test_local_ci_subtract() {
 
     /**
      * Test math class.
+     *
+     * @covers \local_ci\math\add
      */
     public function test_local_ci_math() {
         $math = new \local_ci_math();
@@ -68,6 +74,8 @@ public function test_local_ci_math() {
 
     /**
      * Test math class.
+     *
+     * @covers \local_ci\math\add
      */
     public function test_local_ci_math_class() {
         $math = new math();