Start using PHP 8.1 features

phpunit.xml

@@ -1,9 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" backupGlobals="false" colors="true" processIsolation="false" stopOnFailure="false" bootstrap="./tests/bootstrap.php" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.0/phpunit.xsd" cacheDirectory=".phpunit.cache" backupStaticProperties="false">
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" backupGlobals="false" colors="true" processIsolation="false" stopOnFailure="false" bootstrap="./tests/bootstrap.php" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" cacheDirectory=".phpunit.cache" backupStaticProperties="false">
   <coverage>
-    <include>
-      <directory>./src</directory>
-    </include>
     <report>
       <clover outputFile="build/logs/clover.xml"/>
     </report>
@@ -14,4 +11,9 @@
     </testsuite>
   </testsuites>
   <logging/>
+  <source>
+    <include>
+      <directory>./src</directory>
+    </include>
+  </source>
 </phpunit>

src/Constants.php

@@ -28,21 +28,6 @@ class Constants extends \SimpleSAML\XML\Constants
     public const NS_SOAP_ENC = 'http://www.w3.org/2003/05/soap-encoding';
 
     /**
-     * The fault codes defined by the specification
      */
-    public const FAULT_VERSION_MISMATCH = 'VersionMismatch';
-    public const FAULT_MUST_UNDERSTAND = 'MustUnderstand';
-    public const FAULT_SENDER = 'Sender';
-    public const FAULT_RESPONDER = 'Responder';
-    public const FAULT_DATA_ENCODING_UNKNOWN = 'DataEncodingUnknown';
-
-    public const FAULT_CODES = [
-        self::FAULT_VERSION_MISMATCH,
-        self::FAULT_MUST_UNDERSTAND,
-        self::FAULT_SENDER,
-        self::FAULT_RESPONDER,
-        self::FAULT_DATA_ENCODING_UNKNOWN,
-    ];
-
     public const SOAP_ACTOR_NEXT = 'http://schemas.xmlsoap.org/soap/actor/next';
 }

src/SOAP11/XML/env/Body.php

@@ -12,6 +12,7 @@
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\ExtendableElementTrait;
 use SimpleSAML\XML\ExtendableAttributesTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 use function array_diff;
 use function array_filter;
@@ -29,10 +30,10 @@ final class Body extends AbstractSoapElement
     use ExtendableElementTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ELT_NAMESPACE = NS::ANY;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ATTR_NAMESPACE = NS::ANY;
 
     /**
      * @var \SimpleSAML\SOAP11\XML\env\Fault|null

src/SOAP11/XML/env/Detail.php

@@ -8,10 +8,10 @@
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\XML\AbstractElement;
 use SimpleSAML\XML\Chunk;
-use SimpleSAML\XML\Constants as C;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
 use SimpleSAML\XML\ExtendableElementTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 /**
  * Class representing a env:Detail element.
@@ -33,10 +33,10 @@ final class Detail extends AbstractElement
     public const NS_PREFIX = null;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ELT_NAMESPACE = NS::ANY;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ATTR_NAMESPACE = NS::ANY;
 
 
     /**

src/SOAP11/XML/env/Envelope.php

@@ -13,6 +13,7 @@
 use SimpleSAML\XML\Exception\TooManyElementsException;
 use SimpleSAML\XML\ExtendableElementTrait;
 use SimpleSAML\XML\ExtendableAttributesTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 /**
  * Class representing a env:Envelope element.
@@ -25,10 +26,10 @@ final class Envelope extends AbstractSoapElement
     use ExtendableAttributesTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ELT_NAMESPACE = NS::OTHER;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**

src/SOAP11/XML/env/Header.php

@@ -7,10 +7,10 @@
 use DOMElement;
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\XML\Chunk;
-use SimpleSAML\XML\Constants as C;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
 use SimpleSAML\XML\ExtendableElementTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 /**
  * Class representing a env:Header element.
@@ -23,10 +23,10 @@ final class Header extends AbstractSoapElement
     use ExtendableElementTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ELT_NAMESPACE = NS::OTHER;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**

src/SOAP12/Fault.php

@@ -0,0 +1,19 @@
+<?php
+
+declare(strict_types=1);
+
+namespace SimpleSAML\SOAP12;
+
+/**
+ * The fault codes defined by the specification
+ *
+ * @package simplesamlphp/xml-soap
+ */
+enum Fault: string
+{
+    case VERSION_MISMATCH = 'VersionMismatch';
+    case MUST_UNDERSTAND = 'MustUnderstand';
+    case SENDER = 'Sender';
+    case RESPONDER = 'Responder';
+    case DATA_ENCODING_UNKNOWN = 'DataEncodingUnknown';
+}

src/SOAP12/XML/env/Body.php

@@ -12,6 +12,7 @@
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
 use SimpleSAML\XML\ExtendableElementTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 use function array_diff;
 use function array_filter;
@@ -29,10 +30,10 @@ final class Body extends AbstractSoapElement
     use ExtendableElementTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ELT_NAMESPACE = NS::ANY;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
     /**
      * @var \SimpleSAML\SOAP12\XML\env\Fault|null

src/SOAP12/XML/env/Code.php

@@ -8,6 +8,7 @@
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\SOAP\Constants as C;
 use SimpleSAML\SOAP\Exception\ProtocolViolationException;
+use SimpleSAML\SOAP12\Fault;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\Exception\MissingElementException;
 use SimpleSAML\XML\Exception\TooManyElementsException;
@@ -36,8 +37,8 @@ public function __construct(
         }
 
         Assert::oneOf(
-            $localName,
-            C::FAULT_CODES,
+            Fault::from($localName),
+            Fault::cases(),
             'Invalid top-level Value',
             ProtocolViolationException::class
         );

src/SOAP12/XML/env/Detail.php

@@ -7,10 +7,10 @@
 use DOMElement;
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\XML\Chunk;
-use SimpleSAML\XML\Constants as C;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
 use SimpleSAML\XML\ExtendableElementTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 /**
  * Class representing a env:Detail element.
@@ -23,10 +23,10 @@ final class Detail extends AbstractSoapElement
     use ExtendableElementTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ELT_NAMESPACE = NS::ANY;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**

src/SOAP12/XML/env/Envelope.php

@@ -6,11 +6,11 @@
 
 use DOMElement;
 use SimpleSAML\Assert\Assert;
-use SimpleSAML\XML\Constants as C;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\Exception\MissingElementException;
 use SimpleSAML\XML\Exception\TooManyElementsException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 /**
  * Class representing a env:Envelope element.
@@ -22,7 +22,7 @@ final class Envelope extends AbstractSoapElement
     use ExtendableAttributesTrait;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**

src/SOAP12/XML/env/Header.php

@@ -7,10 +7,10 @@
 use DOMElement;
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\XML\Chunk;
-use SimpleSAML\XML\Constants as C;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
 use SimpleSAML\XML\ExtendableElementTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 /**
  * Class representing a env:Header element.
@@ -23,10 +23,10 @@ final class Header extends AbstractSoapElement
     use ExtendableElementTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ELT_NAMESPACE = NS::ANY;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**

src/SOAP12/XML/env/Text.php

@@ -8,7 +8,6 @@
 use DOMElement;
 use DOMNameSpaceNode;
 use SimpleSAML\Assert\Assert;
-use SimpleSAML\XML\Constants as C;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\LocalizedStringElementTrait;