From 3f8947b275845e767bafc4dd4d7341fa28f46570 Mon Sep 17 00:00:00 2001
From: Alexander Hofbauer <a.hofbauer@fify.at>
Date: Thu, 27 Feb 2025 21:22:29 +0100
Subject: [PATCH] Handle fault strings as defined by the WSDL

---
 src/Exception/InvalidInputException.php          | 13 +++++++++++++
 src/Exception/MaxConcurrentRequestsException.php | 13 +++++++++++++
 .../MaxConcurrentStateRequestsException.php      | 13 +++++++++++++
 src/Exception/ServiceUnavailableException.php    | 13 +++++++++++++
 .../StateServiceUnavailableException.php         | 13 +++++++++++++
 src/Exception/TimeoutException.php               | 13 +++++++++++++
 src/Vies/Client.php                              | 16 +++++++++++++++-
 tests/Vies/ClientTest.php                        |  9 +++++++++
 8 files changed, 102 insertions(+), 1 deletion(-)
 create mode 100644 src/Exception/InvalidInputException.php
 create mode 100644 src/Exception/MaxConcurrentRequestsException.php
 create mode 100644 src/Exception/MaxConcurrentStateRequestsException.php
 create mode 100644 src/Exception/ServiceUnavailableException.php
 create mode 100644 src/Exception/StateServiceUnavailableException.php
 create mode 100644 src/Exception/TimeoutException.php

diff --git a/src/Exception/InvalidInputException.php b/src/Exception/InvalidInputException.php
new file mode 100644
index 0000000..25148e7
--- /dev/null
+++ b/src/Exception/InvalidInputException.php
@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Ddeboer\Vatin\Exception;
+
+class InvalidInputException extends \RuntimeException implements ViesExceptionInterface
+{
+    public function __construct(?\Throwable $previous = null)
+    {
+        parent::__construct('The provided country code is invalid or the VAT number is empty', previous: $previous);
+    }
+}
diff --git a/src/Exception/MaxConcurrentRequestsException.php b/src/Exception/MaxConcurrentRequestsException.php
new file mode 100644
index 0000000..a8a4a3e
--- /dev/null
+++ b/src/Exception/MaxConcurrentRequestsException.php
@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Ddeboer\Vatin\Exception;
+
+class MaxConcurrentRequestsException extends \RuntimeException implements ViesExceptionInterface
+{
+    public function __construct(?\Throwable $previous = null)
+    {
+        parent::__construct('The maximum number of concurrent requests has been reached', previous: $previous);
+    }
+}
diff --git a/src/Exception/MaxConcurrentStateRequestsException.php b/src/Exception/MaxConcurrentStateRequestsException.php
new file mode 100644
index 0000000..3fc952e
--- /dev/null
+++ b/src/Exception/MaxConcurrentStateRequestsException.php
@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Ddeboer\Vatin\Exception;
+
+class MaxConcurrentStateRequestsException extends \RuntimeException implements ViesExceptionInterface
+{
+    public function __construct(?\Throwable $previous = null)
+    {
+        parent::__construct('The maximum number of concurrent requests for this member state has been reached', previous: $previous);
+    }
+}
diff --git a/src/Exception/ServiceUnavailableException.php b/src/Exception/ServiceUnavailableException.php
new file mode 100644
index 0000000..498e680
--- /dev/null
+++ b/src/Exception/ServiceUnavailableException.php
@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Ddeboer\Vatin\Exception;
+
+class ServiceUnavailableException extends \RuntimeException implements ViesExceptionInterface
+{
+    public function __construct(?\Throwable $previous = null)
+    {
+        parent::__construct('An error was encountered at the network or web application level', previous: $previous);
+    }
+}
diff --git a/src/Exception/StateServiceUnavailableException.php b/src/Exception/StateServiceUnavailableException.php
new file mode 100644
index 0000000..8c4ec50
--- /dev/null
+++ b/src/Exception/StateServiceUnavailableException.php
@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Ddeboer\Vatin\Exception;
+
+class StateServiceUnavailableException extends \RuntimeException implements ViesExceptionInterface
+{
+    public function __construct(?\Throwable $previous = null)
+    {
+        parent::__construct('The application at the member state is not replying or not available', previous: $previous);
+    }
+}
diff --git a/src/Exception/TimeoutException.php b/src/Exception/TimeoutException.php
new file mode 100644
index 0000000..c1cc36d
--- /dev/null
+++ b/src/Exception/TimeoutException.php
@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Ddeboer\Vatin\Exception;
+
+class TimeoutException extends \RuntimeException implements ViesExceptionInterface
+{
+    public function __construct(?\Throwable $previous = null)
+    {
+        parent::__construct('The application did not receive a reply within the allocated time period', previous: $previous);
+    }
+}
diff --git a/src/Vies/Client.php b/src/Vies/Client.php
index cef7797..ec46d21 100644
--- a/src/Vies/Client.php
+++ b/src/Vies/Client.php
@@ -2,7 +2,13 @@
 
 namespace Ddeboer\Vatin\Vies;
 
+use Ddeboer\Vatin\Exception\InvalidInputException;
+use Ddeboer\Vatin\Exception\MaxConcurrentRequestsException;
+use Ddeboer\Vatin\Exception\MaxConcurrentStateRequestsException;
+use Ddeboer\Vatin\Exception\ServiceUnavailableException;
 use Ddeboer\Vatin\Exception\ServiceUnreachableException;
+use Ddeboer\Vatin\Exception\StateServiceUnavailableException;
+use Ddeboer\Vatin\Exception\TimeoutException;
 use Ddeboer\Vatin\Exception\ViesException;
 use Ddeboer\Vatin\Exception\ViesExceptionInterface;
 use Ddeboer\Vatin\Vies\Response\CheckVatResponse;
@@ -82,6 +88,14 @@ private function buildExceptionFromSoapFault(\SoapFault $e): ViesExceptionInterf
             return new ServiceUnreachableException($e);
         }
 
-        return new ViesException($e);
+        return match($e->faultstring) {
+            'INVALID_INPUT' => new InvalidInputException($e),
+            'GLOBAL_MAX_CONCURRENT_REQ' => new MaxConcurrentRequestsException($e),
+            'MS_MAX_CONCURRENT_REQ' => new MaxConcurrentStateRequestsException($e),
+            'SERVICE_UNAVAILABLE' => new ServiceUnavailableException($e),
+            'MS_UNAVAILABLE' => new StateServiceUnavailableException($e),
+            'TIMEOUT' => new TimeoutException($e),
+            default => new ViesException($e),
+        };
     }
 }
diff --git a/tests/Vies/ClientTest.php b/tests/Vies/ClientTest.php
index 6397d40..c6e324c 100644
--- a/tests/Vies/ClientTest.php
+++ b/tests/Vies/ClientTest.php
@@ -2,6 +2,7 @@
 
 namespace Ddeboer\Vatin\Test\Vies;
 
+use Ddeboer\Vatin\Exception\InvalidInputException;
 use Ddeboer\Vatin\Vies\Client;
 use PHPUnit\Framework\TestCase;
 
@@ -19,4 +20,12 @@ public function testCheckVat(): void
         $this->assertEquals('---', $response->name);
         $this->assertEquals('---', $response->address);
     }
+
+    public function testInvalidInputException(): void
+    {
+        $this->expectException(InvalidInputException::class);
+
+        $client = new Client();
+        $client->checkVat('XYZ', '123');
+    }
 }