Fixed naming for some unit tests.

ijuanfe · Feb 2, 2023 · ba88fa5 · ba88fa5
1 parent 1449ae2
commit ba88fa5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 25 deletions.
diff --git a/tests/local/fetchquestion_test.php b/tests/local/fetchquestion_test.php
@@ -248,31 +248,22 @@ public function test_get_set_level() {
         $fetchquestion->set_level(-22);
     }
 
-    /**
-     * @test
-     */
-    public function it_fails_when_instantiated_with_a_zero_difficulty_level(): void {
-        $this->resetAfterTest(true);
+    public function test_it_fails_when_instantiated_with_a_zero_difficulty_level(): void {
+        $this->resetAfterTest();
 
         $this->expectException('coding_exception');
         (new fetchquestion(new stdClass(), 0, 1, 100, ['phpunittag_']));
     }
 
-    /**
-     * @test
-     */
-    public function it_fails_when_instantiated_with_a_negative_difficulty_level(): void {
-        $this->resetAfterTest(true);
+    public function test_it_fails_when_instantiated_with_a_negative_difficulty_level(): void {
+        $this->resetAfterTest();
 
         $this->expectException('coding_exception');
         (new fetchquestion(new stdClass(), -11, 1, 100, ['phpunittag_']));
     }
 
-    /**
-     * @test
-     */
-    public function it_fails_when_instantiated_with_a_difficulty_level_as_a_string(): void {
-        $this->resetAfterTest(true);
+    public function test_it_fails_when_instantiated_with_a_difficulty_level_as_a_string(): void {
+        $this->resetAfterTest();
 
         $this->expectException('coding_exception');
         (new fetchquestion(new stdClass(), 'asdf', 1, 100, ['phpunittag_']));
@@ -511,10 +502,7 @@ public function test_fetch_question_search_outside_min_max_bounds() {
         $this->assertEquals([], $result);
     }
 
-    /**
-     * @test
-     */
-    public function it_retrieves_all_tag_ids(): void {
+    public function test_it_retrieves_all_tag_ids(): void {
         $this->resetAfterTest();
         $this->setup_test_data_xml();
 
@@ -527,10 +515,7 @@ public function it_retrieves_all_tag_ids(): void {
         );
     }
 
-    /**
-     * @test
-     */
-    public function it_throws_an_exception_when_retrieves_all_tag_ids_for_an_empty_tag_prefix(): void {
+    public function test_it_throws_an_exception_when_retrieves_all_tag_ids_for_an_empty_tag_prefix(): void {
         $fetchquestion = new fetchquestion(new stdClass(), 5, 1, 100);
 
         $this->expectException('invalid_parameter_exception');

diff --git a/tests/locallib_test.php b/tests/locallib_test.php
@@ -279,10 +279,9 @@ public function test_adaptivequiz_complete_attempt() {
     }
 
     /**
-     * @test
      * @covers ::adaptivequiz_complete_attempt
      */
-    public function event_is_triggered_on_attempt_completion(): void {
+    public function test_event_is_triggered_on_attempt_completion(): void {
         global $DB;
 
         $this->resetAfterTest();