ci: small fixups

classes/local/store/object_client.php

@@ -144,7 +144,7 @@ public function test_range_request($filesystem);
      */
     public function get_token_expiry_time(): int;
 
-    /*
+    /**
      * Tests setting an objects tag.
      * @return stdClass containing 'success' and 'details' properties
      */

classes/local/store/object_client_base.php

@@ -199,7 +199,7 @@ public function get_token_expiry_time(): int {
         return -1;
     }
 
-    /*
+    /**
      * Tests setting an objects tag.
      * @return stdClass containing 'success' and 'details' properties
      */

classes/tests/test_client.php

@@ -176,7 +176,7 @@ public function get_token_expiry_time(): int {
         return $CFG->objectfs_phpunit_token_expiry_time;
     }
 
-    /*
+    /**
      * Sets object tags - uses in-memory store for unit tests
      * @param string $contenthash
      * @param array $tags

tests/local/tagging_test.php

@@ -417,10 +417,12 @@ public function test_get_sync_status_string_does_not_exist() {
 
     /**
      * Tests the length of the defined tag source is checked correctly
+     * @covers \tool_objectfs\local\environment_source
      */
     public function test_environment_source_too_long() {
         global $CFG;
-        $CFG->objectfs_environment_name = 'This is a really long string. It needs to be long because it needs to be more than 128 chars for the test to trigger an exception.';
+        $CFG->objectfs_environment_name = 'This is a really long string.
+            It needs to be long because it needs to be more than 128 chars for the test to trigger an exception.';
         $source = new environment_source();
 
         $this->expectException(moodle_exception::class);