INT-17886: Fix code checker 3.1.0

open-lms-open-source · Apr 28, 2022 · d7a22cd · d7a22cd
1 parent 727d805
commit d7a22cd
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 21 deletions.
diff --git a/classes/aws_apcu_cache.php b/classes/aws_apcu_cache.php
@@ -26,8 +26,6 @@
 
 use Aws\CacheInterface;
 
-defined('MOODLE_INTERNAL') || die();
-
 /**
  * An APCu cache for AWS.
  *
@@ -92,4 +90,4 @@ public function set($key, $value, $ttl = 0) {
     public function remove($key) {
         apcu_delete($key);
     }
-}
+}
diff --git a/classes/aws_sdk.php b/classes/aws_sdk.php
@@ -24,8 +24,6 @@
 
 namespace local_aws_sdk;
 
-defined('MOODLE_INTERNAL') || die();
-
 /**
  * Utility to help use the AWS SDK.
  *
@@ -116,4 +114,4 @@ private static function resolve_credentials($name, array $cfg) {
 
         return [];
     }
-}
+}
diff --git a/classes/privacy/provider.php b/classes/privacy/provider.php
@@ -24,8 +24,6 @@
 
 namespace local_aws_sdk\privacy;
 
-defined('MOODLE_INTERNAL') || die();
-
 /**
  * Privacy Subsystem for local_aws_sdk implementing null_provider.
  *
@@ -43,4 +41,4 @@ class provider implements \core_privacy\local\metadata\null_provider {
     public static function get_reason() : string {
         return 'privacy:metadata';
     }
-}
+}
diff --git a/tests/aws_apcu_cache_test.php b/tests/aws_apcu_cache_test.php
@@ -21,21 +21,19 @@
  * @copyright Copyright (c) 2017 Open LMS (https://www.openlms.net)
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-
+namespace local_aws_sdk;
 use Aws\Credentials\Credentials;
 use local_aws_sdk\aws_apcu_cache;
 use local_aws_sdk\aws_sdk;
 
-defined('MOODLE_INTERNAL') || die();
-
 /**
  * Tests for APCu cache.
  *
  * @package   local_aws_sdk
  * @copyright Copyright (c) 2017 Open LMS (https://www.openlms.net)
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class local_aws_sdk_aws_apcu_cache_testcase extends advanced_testcase {
+class aws_apcu_cache_test extends \advanced_testcase {
 
     const TEST_KEY = 'phpunit_local_aws_sdk_test';
 

diff --git a/tests/aws_sdk_test.php b/tests/aws_sdk_test.php
@@ -21,20 +21,18 @@
  * @copyright Copyright (c) 2017 Open LMS (https://www.openlms.net)
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-
+namespace local_aws_sdk;
 use local_aws_sdk\aws_apcu_cache;
 use local_aws_sdk\aws_sdk;
 
-defined('MOODLE_INTERNAL') || die();
-
 /**
  * Tests SDK class.
  *
  * @package   local_aws_sdk
  * @copyright Copyright (c) 2017 Open LMS (https://www.openlms.net)
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class local_aws_sdk_aws_sdk_testcase extends advanced_testcase {
+class aws_sdk_test extends \advanced_testcase {
     /**
      * Test various use cases for creating config.
      *
@@ -56,7 +54,7 @@ public function test_config_from_cfg($cfg, $expected) {
      * Test error handling for missing CFG value.
      */
     public function test_config_from_cfg_missing_cfg() {
-        $this->expectException(coding_exception::class);
+        $this->expectException(\coding_exception::class);
         aws_sdk::config_from_cfg('asdf_asdf_hodor');
     }
 
@@ -70,7 +68,7 @@ public function test_config_from_cfg_not_array() {
 
         $CFG->phpunit_local_aws_sdk_test = 'hodor';
 
-        $this->expectException(coding_exception::class);
+        $this->expectException(\coding_exception::class);
         aws_sdk::config_from_cfg('phpunit_local_aws_sdk_test');
     }
 
@@ -87,7 +85,7 @@ public function test_config_from_cfg_unknown_cache() {
             'credentials_cache' => 'hodor',
         ];
 
-        $this->expectException(coding_exception::class);
+        $this->expectException(\coding_exception::class);
         aws_sdk::config_from_cfg('phpunit_local_aws_sdk_test');
     }
 

diff --git a/version.php b/version.php
@@ -29,4 +29,4 @@
 $plugin->requires  = 2015051100;
 $plugin->component = 'local_aws_sdk';
 $plugin->maturity  = MATURITY_STABLE;
-$plugin->release   = '3.1.3 (Build: 20161114)';
+$plugin->release   = '3.1.3 (Build: 20161114)';