test: add expects() and remove assertTrue(true)

codeigniter4 · Jun 29, 2022 · b872d7b · b872d7b
1 parent 204287c
commit b872d7b
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/tests/system/Email/EmailTest.php b/tests/system/Email/EmailTest.php
@@ -143,16 +143,14 @@ public function testDestructDoesNotThrowException()
             ->disableOriginalConstructor()
             ->onlyMethods(['sendCommand'])
             ->getMock();
-        $email->method('sendCommand')
+        $email->expects($this->once())->method('sendCommand')
             ->willThrowException(new ErrorException('SMTP Error.'));
 
         // Force resource to be injected into the property
         $SMTPConnect = fopen(__FILE__, 'rb');
         $this->setPrivateProperty($email, 'SMTPConnect', $SMTPConnect);
 
         $email->__destruct();
-
-        $this->assertTrue(true);
     }
 
     private function createMockEmail(): MockEmail