From a8cff29c3b1538f20b69fc503fcd12bb9f12d93e Mon Sep 17 00:00:00 2001
From: Matthew Hilton <matthewhilton@catalyst-au.net>
Date: Mon, 30 Sep 2024 09:02:12 +1000
Subject: [PATCH] ci: small fixups

---
 classes/local/store/object_client.php      | 2 +-
 classes/local/store/object_client_base.php | 2 +-
 classes/tests/test_client.php              | 2 +-
 tests/local/tagging_test.php               | 4 +++-
 4 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/classes/local/store/object_client.php b/classes/local/store/object_client.php
index ffa59bf2..55c386b3 100644
--- a/classes/local/store/object_client.php
+++ b/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
      */
diff --git a/classes/local/store/object_client_base.php b/classes/local/store/object_client_base.php
index 7361d72f..e448814e 100644
--- a/classes/local/store/object_client_base.php
+++ b/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
      */
diff --git a/classes/tests/test_client.php b/classes/tests/test_client.php
index 7ca00d56..12088abc 100644
--- a/classes/tests/test_client.php
+++ b/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
diff --git a/tests/local/tagging_test.php b/tests/local/tagging_test.php
index c8ff3a54..a5a5956e 100644
--- a/tests/local/tagging_test.php
+++ b/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);