Merge pull request #4 from zacharyrankin/default-fix

Setting a default value for the token variable.
zacharymarshal · Nov 7, 2016 · 250a77d · 250a77d
2 parents 3b68c8f + 030169b
commit 250a77d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/src/Tokenizer.php b/src/Tokenizer.php
@@ -6,6 +6,8 @@ class Tokenizer
 {
     public function tokenize($html)
     {
+        $tokens = [];
+
         $length = strlen($html);
         while ($length) {
             $token = $tokens[] = $this->getToken($html);

diff --git a/tests/lexer-test.php b/tests/lexer-test.php
@@ -5,6 +5,15 @@
 
 require_once __DIR__ . '/../vendor/autoload.php';
 
+Test::create('should return an empty array for no value', function(Test $test) {
+    $tokenizer = new Tokenizer;
+    $tokens = $tokenizer->tokenize("");
+    $test->equals(
+        $tokens,
+        []
+    );
+});
+
 Test::create('should tokenize whitespace before word', function(Test $test) {
     $tokenizer = new Tokenizer;
     $tokens = $tokenizer->tokenize("    a");