Raise coverage

simplesamlphp · Jan 25, 2024 · 3c9d692 · 3c9d692
1 parent 06a7d5e
commit 3c9d692
Show file tree

Hide file tree

Showing 7 changed files with 16 additions and 13 deletions.
diff --git a/src/XML/fed/AbstractPseudonymBasisType.php b/src/XML/fed/AbstractPseudonymBasisType.php
@@ -47,18 +47,6 @@ final public function __construct(
     }
 
 
-    /**
-     * Test if an object, at the state it's in, would produce an empty XML-element
-     *
-     * @return bool
-     */
-    public function isEmptyElement(): bool
-    {
-        return empty($this->getElements())
-            && empty($this->getAttributesNS());
-    }
-
-
     /**
      * Create an instance of this object from its XML representation.
      *

diff --git a/tests/WSSecurity/XML/wst/CodeTest.php b/tests/WSSecurity/XML/wst/CodeTest.php
@@ -16,6 +16,7 @@
  * Class \SimpleSAML\WSSecurity\XML\wst\CodeTest
  *
  * @covers \SimpleSAML\WSSecurity\XML\wst\Code
+ * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractStatusCodeOpenEnum
  * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractWstElement
  *
  * @package tvdijen/ws-security

diff --git a/tests/WSSecurity/XML/wst/ComputedKeyTest.php b/tests/WSSecurity/XML/wst/ComputedKeyTest.php
@@ -17,6 +17,7 @@
  * Class \SimpleSAML\WSSecurity\XML\wst\ComputedKeyTest
  *
  * @covers \SimpleSAML\WSSecurity\XML\wst\ComputedKey
+ * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractComputedKeyOpenEnum
  * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractWstElement
  *
  * @package tvdijen/ws-security

diff --git a/tests/WSSecurity/XML/wst/KeyTypeTest.php b/tests/WSSecurity/XML/wst/KeyTypeTest.php
@@ -17,6 +17,7 @@
  * Class \SimpleSAML\WSSecurity\XML\wst\KeyTypeTest
  *
  * @covers \SimpleSAML\WSSecurity\XML\wst\KeyType
+ * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractKeyTypeOpenEnum
  * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractWstElement
  *
  * @package tvdijen/ws-security

diff --git a/tests/WSSecurity/XML/wst/ParticipantsTest.php b/tests/WSSecurity/XML/wst/ParticipantsTest.php
@@ -21,7 +21,7 @@
  * Class \SimpleSAML\WSSecurity\XML\wst\ParticipantsTest
  *
  * @covers \SimpleSAML\WSSecurity\XML\wst\Participants
- * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractParticipantType
+ * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractParticipantsType
  * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractWstElement
  *
  * @package tvdijen/ws-security

diff --git a/tests/WSSecurity/XML/wst/SignChallengeTest.php b/tests/WSSecurity/XML/wst/SignChallengeTest.php
@@ -66,4 +66,15 @@ public function testMarshalling(): void
             strval($signChallenge),
         );
     }
+
+
+    /**
+     * Test creating an empty SignChallenge object from scratch.
+     */
+    public function testMarshallingEmpty(): void
+    {
+        $signChallenge = new SignChallenge();
+
+        $this->assertTrue($signChallenge->isEmptyElement());
+    }
 }
diff --git a/tests/WSSecurity/XML/wst/StatusTest.php b/tests/WSSecurity/XML/wst/StatusTest.php
@@ -19,6 +19,7 @@
  * Class \SimpleSAML\WSSecurity\XML\wst\StatusTest
  *
  * @covers \SimpleSAML\WSSecurity\XML\wst\Status
+ * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractStatusType
  * @covers \SimpleSAML\WSSecurity\XML\wst\AbstractWstElement
  *
  * @package tvdijen/ws-security