Converted the isBlock method to use a case insensitive mask and added tests for it.

1 files changed, 43 insertions, 0 deletions

diff --git a/test/HTML5/Serializer/TraverserTest.php b/test/HTML5/Serializer/TraverserTest.php
index c7e43d8..8638e22 100644
--- a/test/HTML5/Serializer/TraverserTest.php
+++ b/test/HTML5/Serializer/TraverserTest.php
@@ -7,4 +7,47 @@ require_once __DIR__ . '/../TestCase.php';
 
 class TraverserTest extends \HTML5\Tests\TestCase {
 
+  /**
+   * Using reflection we make a protected method accessible for testing.
+   * 
+   * @param string $name
+   *   The name of the method on the Traverser class to test.
+   *
+   * @return \ReflectionMethod
+   *   \ReflectionMethod for the specified method
+   */
+  function getProtectedMethod($name) {
+    $class = new \ReflectionClass('\HTML5\Serializer\Traverser');
+    $method = $class->getMethod($name);
+    $method->setAccessible(true);
+    return $method;
+  }
+
+  public function testIsBlock() {
+    $blocks = array('html', 'body', 'head', 'p', 'div', 'h1', 'h2', 'h3', 'h4',
+      'h5', 'h6', 'title', 'script', 'link', 'meta', 'section', 'article',
+      'table', 'tbody', 'tr', 'th', 'td',
+      //'form',
+    );
+
+    // Mocking the required input because there is no checking.
+    $t = new Traverser('', '');
+    $method = $this->getProtectedMethod('isBlock');
+
+    foreach ($blocks as $block) {
+      $this->assertTrue($method->invoke($t, $block), 'Block test failed on: ' . $block);
+
+      // Also test the uppercase version.
+      $this->assertTrue($method->invoke($t, strtoupper($block)), 'Block test failed on: ' . $block);
+    }
+
+    $nonblocks = array('span', 'a', 'img');
+    foreach ($nonblocks as $tag) {
+      $this->assertFalse($method->invoke($t, $tag),  'Block test failed on: ' . $tag);
+
+      // Also test the uppercase version.
+      $this->assertFalse($method->invoke($t, strtoupper($tag)),  'Block test failed on: ' . $tag);
+    }
+  }
+
 }
\ No newline at end of file