tests bug fix

Signed-off-by: MarioRadu <magda_marior@yahoo.com>

Author: MarioRadu

test/Unit/Admin/Handler/Account/PostAccountChangePasswordHandlerTest.php

@@ -112,7 +112,7 @@ public function testInvalidConfirmPasswordProvidedWillReturnRedirectResponse():
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addError')
             ->with(Message::CURRENT_PASSWORD_INCORRECT);
 
@@ -147,13 +147,13 @@ public function testThrowIdentityExceptionWillReturnRedirectResponse(): void
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addError')
             ->with((new IdentityException())->getMessage());
 
         $this
             ->adminService
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('updateAdmin');
 
         $handler = new PostAccountChangePasswordHandler(
@@ -187,7 +187,7 @@ public function testThrowExceptionWillReturnRedirectResponse(): void
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addError')
             ->with(Message::AN_ERROR_OCCURRED);
 
@@ -221,7 +221,7 @@ public function testValidFormDataWillReturnRedirectResponse(): void
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addSuccess')
             ->with(Message::ACCOUNT_UPDATE_SUCCESSFULLY);
 

test/Unit/Admin/Handler/Account/PostAccountEditHandlerTest.php

@@ -106,7 +106,7 @@ public function testThrowIdentityExceptionWillReturnRedirectResponse(): void
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addError')
             ->with(Message::AN_ERROR_OCCURRED);
 
@@ -141,7 +141,7 @@ public function testThrowExceptionWillReturnRedirectResponse(): void
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addError')
             ->with(Message::AN_ERROR_OCCURRED);
 
@@ -175,7 +175,7 @@ public function testValidFormDataWillReturnRedirectResponse(): void
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addSuccess')
             ->with(Message::ACCOUNT_UPDATE_SUCCESSFULLY);
 

test/Unit/Admin/Handler/Account/PostAccountLoginHandlerTest.php

@@ -24,6 +24,7 @@
 use PHPUnit\Framework\MockObject\Exception as MockObjectException;
 use PHPUnit\Framework\MockObject\MockObject;
 use Psr\Http\Message\ServerRequestInterface;
+use Psr\Http\Message\UriInterface;
 
 class PostAccountLoginHandlerTest extends UnitTest
 {
@@ -95,7 +96,7 @@ public function testInvalidLoginFormDataProvidedWillReturnRedirectResponse(): vo
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addError');
 
         $handler = new PostAccountLoginHandler(
@@ -130,8 +131,8 @@ public function testInvalidPasswordProvidedWillReturnRedirectResponse(): void
         $this->authenticationAdapter->method('setCredential')->willReturn($this->authenticationAdapter);
         $this->authenticationService->method('getAdapter')->willReturn($this->authenticationAdapter);
 
-        $this->messenger->expects($this->exactly(1))->method('addError');
-        $this->adminService->expects($this->exactly(1))->method('logAdminVisit');
+        $this->messenger->expects($this->once())->method('addError');
+        $this->adminService->expects($this->once())->method('logAdminVisit');
 
         $handler = new PostAccountLoginHandler(
             $this->adminService,
@@ -166,13 +167,13 @@ public function testAdminInactiveWillReturnRedirectResponse(): void
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addError')
             ->with(Message::ADMIN_INACTIVE);
 
         $this
             ->authenticationService
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('clearIdentity');
 
         $handler = new PostAccountLoginHandler(
@@ -190,13 +191,17 @@ public function testAdminInactiveWillReturnRedirectResponse(): void
         $this->assertSame(StatusCodeInterface::STATUS_SEE_OTHER, $response->getStatusCode());
     }
 
+    /**
+     * @throws MockObjectException
+     */
     public function testAdminLoginThrowsExceptionWillReturnRedirectResponse(): void
     {
+        $this->request->method('getUri')->willReturn($this->createMock(UriInterface::class));
         $this->throwException(new Exception());
 
         $this
             ->messenger
-            ->expects($this->exactly(1))
+            ->expects($this->once())
             ->method('addError')
             ->with(Message::AN_ERROR_OCCURRED);
 
@@ -232,8 +237,8 @@ public function testAdminLoginSuccessfulWillReturnRedirectResponse(): void
         $this->authenticationAdapter->method('setCredential')->willReturn($this->authenticationAdapter);
         $this->authenticationService->method('getAdapter')->willReturn($this->authenticationAdapter);
 
-        $this->adminService->expects($this->exactly(1))->method('logAdminVisit');
-        $this->storage->expects($this->exactly(1))->method('write')->with($this->identity);
+        $this->adminService->expects($this->once())->method('logAdminVisit');
+        $this->storage->expects($this->once())->method('write')->with($this->identity);
 
         $handler = new PostAccountLoginHandler(
             $this->adminService,