DOMTokenList test improvements

2 years ago · e1ffbfdb9e
5 changed files with 156 additions and 26 deletions
--- a/tests/cases/TestDOMTokenList.php
+++ b/tests/cases/TestDOMTokenList.php
@ -20,7 +20,7 @@ use MensBeam\HTML\DOM\{

 /** @covers \MensBeam\HTML\DOM\DOMTokenList */
 class TestDOMTokenList extends \PHPUnit\Framework\TestCase {
-    public function provideMethod_add_remove_replace_toggle__errors(): iterable {
+    public function provideMethod_add_remove_replace_toggle___errors(): iterable {
        return [
            [ function() {
                $d = new Document();
@ -65,8 +65,31 @@ class TestDOMTokenList extends \PHPUnit\Framework\TestCase {
        ];
    }

-    /** @dataProvider provideMethod_add_remove_replace_toggle__errors */
-    public function testMethod_add_remove_replace_toggle__errors(\Closure $closure, int $errorCode): void {
+    /**
+     * @dataProvider provideMethod_add_remove_replace_toggle___errors
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::add
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::remove
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::replace
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::toggle
+     *
+     * @covers \MensBeam\HTML\DOM\Document::__construct
+     * @covers \MensBeam\HTML\DOM\Document::createElement
+     * @covers \MensBeam\HTML\DOM\DOMException::__construct
+     * @covers \MensBeam\HTML\DOM\DOMImplementation::__construct
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__construct
+     * @covers \MensBeam\HTML\DOM\Element::__construct
+     * @covers \MensBeam\HTML\DOM\Element::__get_classList
+     * @covers \MensBeam\HTML\DOM\Node::__construct
+     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
+     * @covers \MensBeam\HTML\DOM\Inner\Document::getWrapperNode
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::get
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::has
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::key
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::set
+     * @covers \MensBeam\HTML\DOM\Inner\Reflection::createFromProtectedConstructor
+     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
+     */
+    public function testMethod_add_remove_replace_toggle___errors(\Closure $closure, int $errorCode): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode($errorCode);
        $closure();
@ -112,6 +135,36 @@ class TestDOMTokenList extends \PHPUnit\Framework\TestCase {
    }


+    /**
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::count
+     *
+     * @covers \MensBeam\HTML\DOM\Document::__construct
+     * @covers \MensBeam\HTML\DOM\Document::createElement
+     * @covers \MensBeam\HTML\DOM\DOMImplementation::__construct
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__construct
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__toString
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::add
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::parseOrderedSet
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::update
+     * @covers \MensBeam\HTML\DOM\Element::__construct
+     * @covers \MensBeam\HTML\DOM\Element::__get_classList
+     * @covers \MensBeam\HTML\DOM\Node::__construct
+     * @covers \MensBeam\HTML\DOM\Node::appendChild
+     * @covers \MensBeam\HTML\DOM\Node::getInnerDocument
+     * @covers \MensBeam\HTML\DOM\Node::getInnerNode
+     * @covers \MensBeam\HTML\DOM\Node::getRootNode
+     * @covers \MensBeam\HTML\DOM\Node::postInsertionBugFixes
+     * @covers \MensBeam\HTML\DOM\Node::preInsertionBugFixes
+     * @covers \MensBeam\HTML\DOM\Node::preInsertionValidity
+     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
+     * @covers \MensBeam\HTML\DOM\Inner\Document::getWrapperNode
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::get
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::has
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::key
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::set
+     * @covers \MensBeam\HTML\DOM\Inner\Reflection::createFromProtectedConstructor
+     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
+     */
    public function testMethod_count(): void {
        $d = new Document();
        $e = $d->appendChild($d->createElement('html'));
@ -202,6 +255,43 @@ class TestDOMTokenList extends \PHPUnit\Framework\TestCase {
    }


+    /**
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::replace
+     *
+     * @covers \MensBeam\HTML\DOM\Attr::__get_value
+     * @covers \MensBeam\HTML\DOM\Document::__construct
+     * @covers \MensBeam\HTML\DOM\Document::createElement
+     * @covers \MensBeam\HTML\DOM\DOMImplementation::__construct
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__construct
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__get_value
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__toString
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::parseOrderedSet
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::update
+     * @covers \MensBeam\HTML\DOM\Element::__construct
+     * @covers \MensBeam\HTML\DOM\Element::__get_classList
+     * @covers \MensBeam\HTML\DOM\Element::__get_namespaceURI
+     * @covers \MensBeam\HTML\DOM\Element::getAttribute
+     * @covers \MensBeam\HTML\DOM\Element::getAttributeNode
+     * @covers \MensBeam\HTML\DOM\Element::setAttribute
+     * @covers \MensBeam\HTML\DOM\Node::__construct
+     * @covers \MensBeam\HTML\DOM\Node::__get_ownerDocument
+     * @covers \MensBeam\HTML\DOM\Node::appendChild
+     * @covers \MensBeam\HTML\DOM\Node::getInnerDocument
+     * @covers \MensBeam\HTML\DOM\Node::getInnerNode
+     * @covers \MensBeam\HTML\DOM\Node::getRootNode
+     * @covers \MensBeam\HTML\DOM\Node::postInsertionBugFixes
+     * @covers \MensBeam\HTML\DOM\Node::preInsertionBugFixes
+     * @covers \MensBeam\HTML\DOM\Node::preInsertionValidity
+     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
+     * @covers \MensBeam\HTML\DOM\Inner\Document::__get_wrapperNode
+     * @covers \MensBeam\HTML\DOM\Inner\Document::getWrapperNode
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::get
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::has
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::key
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::set
+     * @covers \MensBeam\HTML\DOM\Inner\Reflection::createFromProtectedConstructor
+     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
+     */
    public function testMethod_replace(): void {
        $d = new Document();
        $e = $d->appendChild($d->createElement('html'));
@ -213,6 +303,45 @@ class TestDOMTokenList extends \PHPUnit\Framework\TestCase {
    }


+    /**
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::remove
+     * @covers \MensBeam\HTML\DOM\Attr::__get_value
+     * @covers \MensBeam\HTML\DOM\Document::__construct
+     * @covers \MensBeam\HTML\DOM\Document::createElement
+     * @covers \MensBeam\HTML\DOM\DOMImplementation::__construct
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__construct
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__get_length
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__get_value
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::__toString
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::item
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::offsetGet
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::parseOrderedSet
+     * @covers \MensBeam\HTML\DOM\DOMTokenList::update
+     * @covers \MensBeam\HTML\DOM\Element::__construct
+     * @covers \MensBeam\HTML\DOM\Element::__get_classList
+     * @covers \MensBeam\HTML\DOM\Element::__get_namespaceURI
+     * @covers \MensBeam\HTML\DOM\Element::getAttribute
+     * @covers \MensBeam\HTML\DOM\Element::getAttributeNode
+     * @covers \MensBeam\HTML\DOM\Element::setAttribute
+     * @covers \MensBeam\HTML\DOM\Node::__construct
+     * @covers \MensBeam\HTML\DOM\Node::__get_ownerDocument
+     * @covers \MensBeam\HTML\DOM\Node::appendChild
+     * @covers \MensBeam\HTML\DOM\Node::getInnerDocument
+     * @covers \MensBeam\HTML\DOM\Node::getInnerNode
+     * @covers \MensBeam\HTML\DOM\Node::getRootNode
+     * @covers \MensBeam\HTML\DOM\Node::postInsertionBugFixes
+     * @covers \MensBeam\HTML\DOM\Node::preInsertionBugFixes
+     * @covers \MensBeam\HTML\DOM\Node::preInsertionValidity
+     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
+     * @covers \MensBeam\HTML\DOM\Inner\Document::__get_wrapperNode
+     * @covers \MensBeam\HTML\DOM\Inner\Document::getWrapperNode
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::get
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::has
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::key
+     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::set
+     * @covers \MensBeam\HTML\DOM\Inner\Reflection::createFromProtectedConstructor
+     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
+     */
    public function testMethod_remove(): void {
        $d = new Document();
        $e = $d->appendChild($d->createElement('html'));
@ -222,6 +351,7 @@ class TestDOMTokenList extends \PHPUnit\Framework\TestCase {
        $this->assertSame('ook eek ookeek', $e->classList->value);
        $this->assertSame('ook eek ookeek', $e->getAttribute('class'));

+        // It is wasteful to do it like this of course, but this is only for testing.
        $classList = $e->classList;
        while ($classList->length > 0) {
            $classList->remove($classList[0]);
--- a/tests/cases/TestDocument.php
+++ b/tests/cases/TestDocument.php
@ -84,7 +84,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Node::__construct
     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
     */
-    public function testMethod_adoptNode_errors(): void {
+    public function testMethod_adoptNode__errors(): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::NOT_SUPPORTED);
        $d = new Document();
@ -102,7 +102,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Node::__construct
     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
     */
-    public function testMethod_createAttribute_errors(): void {
+    public function testMethod_createAttribute__errors(): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::INVALID_CHARACTER);
        $d = new Document();
@ -110,7 +110,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
    }


-    public function provideMethod_createCDATASection_errors(): iterable {
+    public function provideMethod_createCDATASection__errors(): iterable {
        return [
            [ function () {
                $d = new Document();
@ -124,7 +124,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
    }

    /**
-     * @dataProvider provideMethod_createCDATASection_errors
+     * @dataProvider provideMethod_createCDATASection__errors
     * @covers \MensBeam\HTML\DOM\Document::createCDATASection
     *
     * @covers \MensBeam\HTML\DOM\Document::__construct
@ -133,7 +133,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Node::__construct
     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
     */
-    public function testMethod_createCDATASection_errors(\Closure $closure, int $errorCode): void {
+    public function testMethod_createCDATASection__errors(\Closure $closure, int $errorCode): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode($errorCode);
        $closure();
@ -149,7 +149,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Node::__construct
     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
     */
-    public function testMethod_createElement_errors(): void {
+    public function testMethod_createElement__errors(): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::INVALID_CHARACTER);
        $d = new Document();
@ -211,7 +211,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
    }


-    public function provideMethod_importNode_errors(): iterable {
+    public function provideMethod_importNode__errors(): iterable {
        return [
            [ function () {
                $d = new Document();
@ -237,7 +237,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
    }

    /**
-     * @dataProvider provideMethod_importNode_errors
+     * @dataProvider provideMethod_importNode__errors
     * @covers \MensBeam\HTML\DOM\Document::importNode
     *
     * @covers \MensBeam\HTML\DOM\Document::__construct
@ -246,7 +246,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Node::__construct
     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
     */
-    public function testMethod_importNode_errors(\Closure $closure): void {
+    public function testMethod_importNode__errors(\Closure $closure): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::NOT_SUPPORTED);
        $closure();
@ -279,7 +279,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::createFromProtectedConstructor
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
     */
-    public function testMethod_load_errors(): void {
+    public function testMethod_load__errors(): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::NO_MODIFICATION_ALLOWED);
        $d = new Document();
@ -336,7 +336,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Node::__construct
     * @covers \MensBeam\HTML\DOM\Inner\Document::__construct
     */
-    public function testMethod_loadFile_errors(): void {
+    public function testMethod_loadFile__errors(): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::FILE_NOT_FOUND);
        $d = new Document();
@ -363,7 +363,7 @@ class TestDocument extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Inner\NodeCache::set
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::createFromProtectedConstructor
     */
-    public function testMethod_serialize_errors(): void {
+    public function testMethod_serialize__errors(): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::WRONG_DOCUMENT);
        $d = new XMLDocument();
--- a/tests/cases/TestDocumentOrElement.php
+++ b/tests/cases/TestDocumentOrElement.php
@ -110,7 +110,7 @@ class TestDocumentOrElement extends \PHPUnit\Framework\TestCase {
    }


-    public function provideMethod_validateAndExtract_errors(): iterable {
+    public function provideMethod_validateAndExtract__errors(): iterable {
        return [
            [ function() {
                $d = new Document();
@ -124,9 +124,9 @@ class TestDocumentOrElement extends \PHPUnit\Framework\TestCase {
    }

    /**
-     * @dataProvider provideMethod_validateAndExtract_errors
+     * @dataProvider provideMethod_validateAndExtract__errors
     */
-    public function testMethod_validateAndExtract_errors(\Closure $closure, int $errorCode): void {
+    public function testMethod_validateAndExtract__errors(\Closure $closure, int $errorCode): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode($errorCode);
        $closure();
--- a/tests/cases/TestElement.php
+++ b/tests/cases/TestElement.php
@ -327,7 +327,7 @@ class TestElement extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::createFromProtectedConstructor
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
     */
-    public function testMethod_setAttribute_errors() {
+    public function testMethod_setAttribute__errors() {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::INVALID_CHARACTER);
        $d = new Document();
@ -423,7 +423,7 @@ class TestElement extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::createFromProtectedConstructor
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
     */
-    public function testMethod_setAttributeNode_errors() {
+    public function testMethod_setAttributeNode__errors() {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode(DOMException::IN_USE_ATTRIBUTE);
        $d = new Document();
--- a/tests/cases/TestNode.php
+++ b/tests/cases/TestNode.php
@ -677,7 +677,7 @@ class TestNode extends \PHPUnit\Framework\TestCase {
    }


-    public function provideMethod_preInsertionValidity_errors(): iterable {
+    public function provideMethod_preInsertionValidity__errors(): iterable {
        return [
            [ function() {
                $d = new Document();
@ -783,7 +783,7 @@ class TestNode extends \PHPUnit\Framework\TestCase {
    }

    /**
-     * @dataProvider provideMethod_preInsertionValidity_errors
+     * @dataProvider provideMethod_preInsertionValidity__errors
     * @covers \MensBeam\HTML\DOM\Node::preInsertionValidity
     *
     * @covers \MensBeam\HTML\DOM\Comment::__construct
@ -823,7 +823,7 @@ class TestNode extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::setProtectedProperties
     */
-    public function testMethod_preInsertionValidity_errors(\Closure $closure, int $errorCode = DOMException::HIERARCHY_REQUEST_ERROR): void {
+    public function testMethod_preInsertionValidity__errors(\Closure $closure, int $errorCode = DOMException::HIERARCHY_REQUEST_ERROR): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode($errorCode);
        $closure();
@ -895,7 +895,7 @@ class TestNode extends \PHPUnit\Framework\TestCase {
    }


-    public function provideMethod_replaceChild_errors(): iterable {
+    public function provideMethod_replaceChild__errors(): iterable {
        return [
            [ function() {
                $d = new Document();
@ -997,7 +997,7 @@ class TestNode extends \PHPUnit\Framework\TestCase {
    }

    /**
-     * @dataProvider provideMethod_replaceChild_errors
+     * @dataProvider provideMethod_replaceChild__errors
     * @covers \MensBeam\HTML\DOM\Node::replaceChild
     *
     * @covers \MensBeam\HTML\DOM\Comment::__construct
@ -1034,7 +1034,7 @@ class TestNode extends \PHPUnit\Framework\TestCase {
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::getProtectedProperty
     * @covers \MensBeam\HTML\DOM\Inner\Reflection::setProtectedProperties
     */
-    public function testMethod_replaceChild_errors(\Closure $closure, int $errorCode = DOMException::HIERARCHY_REQUEST_ERROR): void {
+    public function testMethod_replaceChild__errors(\Closure $closure, int $errorCode = DOMException::HIERARCHY_REQUEST_ERROR): void {
        $this->expectException(DOMException::class);
        $this->expectExceptionCode($errorCode);
        $closure();