Make loop in assertNextResponse looping, by making sure nextResponse returns null instead of throwing

PHPStorm tells me the loop in assertNextResponse is not looping, PHPStan tells me the throw at the end of assertNextResponse ist unreachable

Author: CReimer

src/Connection/ImapConnection.php

@@ -24,6 +24,7 @@
 use Exception;
 use Generator;
 use LogicException;
+use Throwable;
 
 class ImapConnection implements ConnectionInterface
 {
@@ -726,8 +727,12 @@ protected function assertTaggedResponse(string $tag, ?callable $exception = null
      *
      * @template T of Response
      *
-     * @param  callable(T): bool  $filter
+     * @param  callable(Response): bool  $filter
+     * @param  callable(T): bool  $assertion
+     * @param  callable(T): Throwable  $exception
      * @return T
+     *
+     * @throws ImapResponseException
      */
     protected function assertNextResponse(callable $filter, callable $assertion, callable $exception): Response
     {
@@ -768,7 +773,7 @@ protected function nextResponse(callable $filter): ?Response
             }
         }
 
-        throw new ImapResponseException('No matching response found');
+        return null;
     }
 
     /**