Catch TimeoutException in FallbackSnmpClient

Author: grongor

src/Transport/FallbackSnmpClient.php

@@ -6,6 +6,7 @@
 
 use Psr\Log\LoggerInterface;
 use SimPod\PhpSnmp\Exception\GeneralException;
+use SimPod\PhpSnmp\Exception\TimeoutReached;
 
 final class FallbackSnmpClient implements SnmpClient
 {
@@ -76,7 +77,7 @@ private function tryClients(callable $requestCallback) : array
         foreach ($this->snmpClients as $key => $snmpClient) {
             try {
                 return $requestCallback($snmpClient);
-            } catch (GeneralException $exception) {
+            } catch (GeneralException | TimeoutReached $exception) {
                 $this->logger->warning(
                     'SNMP request failed',
                     [

tests/Transport/FallbackSnmpClientTest.php

@@ -8,6 +8,7 @@
 use Psr\Log\NullLogger;
 use Psr\Log\Test\TestLogger;
 use SimPod\PhpSnmp\Exception\GeneralException;
+use SimPod\PhpSnmp\Exception\TimeoutReached;
 use SimPod\PhpSnmp\Transport\FallbackSnmpClient;
 use SimPod\PhpSnmp\Transport\Request;
 use SimPod\PhpSnmp\Transport\SnmpClient;
@@ -82,7 +83,7 @@ public function testOnlyLastClientWorks() : void
         $client1->expects(self::once())
             ->method('get')
             ->with($oids = ['.1.2.3'])
-            ->willThrowException($exception1 = GeneralException::new('an error'));
+            ->willThrowException($exception1 = TimeoutReached::fromOid('127.0.0.1', '.1.2.3'));
 
         $client2 = $this->createMock(SnmpClient::class);
         $client2->expects(self::once())