test(llk) Add test cases for the offset feature.

Author: Hywan

Llk/Rule/Token.php

@@ -245,12 +245,15 @@ public function getValue()
     /**
      * Set token offset.
      *
-     * @param int $offset
-     * @return void
+     * @param   int  $offset    Offset.
+     * @return  int
      */
     public function setOffset($offset)
     {
+        $old           = $this->_offset;
         $this->_offset = $offset;
+
+        return $old;
     }
 
     /**

Llk/TreeNode.php

@@ -176,29 +176,29 @@ public function getValueToken()
     }
 
     /**
-     * Get token offset.
+     * Get value value.
      *
-     * @return int
+     * @return  string
      */
-    public function getOffset()
+    public function getValueValue()
     {
         return
-            isset($this->_value['offset'])
-                ? $this->_value['offset']
-                : 0;
+            isset($this->_value['value'])
+                ? $this->_value['value']
+                : null;
     }
 
     /**
-     * Get value value.
+     * Get token offset.
      *
-     * @return  string
+     * @return  int
      */
-    public function getValueValue()
+    public function getOffset()
     {
         return
-            isset($this->_value['value'])
-                ? $this->_value['value']
-                : null;
+            isset($this->_value['offset'])
+                ? $this->_value['offset']
+                : 0;
     }
 
     /**

Test/Unit/Llk/Rule/Token.php

@@ -240,12 +240,12 @@ public function case_get_value()
     {
         $this
             ->given(
-                $name           = 'foo',
-                $tokenName      = 'bar',
-                $nodeId         = 'baz',
-                $unification    = 0,
-                $value = 'qux',
-                $token          = new SUT($name, $tokenName, $nodeId, $unification),
+                $name        = 'foo',
+                $tokenName   = 'bar',
+                $nodeId      = 'baz',
+                $unification = 0,
+                $value       = 'qux',
+                $token       = new SUT($name, $tokenName, $nodeId, $unification),
                 $token->setValue($value)
             )
             ->when($result = $token->getValue())
@@ -254,6 +254,41 @@ public function case_get_value()
                     ->isEqualTo($value);
     }
 
+    public function case_set_offset()
+    {
+        $this
+            ->given(
+                $name        = 'foo',
+                $tokenName   = 'bar',
+                $nodeId      = 'baz',
+                $unification = 0,
+                $offset      = 42,
+                $token       = new SUT($name, $tokenName, $nodeId, $unification)
+            )
+            ->when($result = $token->setOffset($offset))
+            ->then
+                ->integer($result)
+                    ->isZero();
+    }
+
+    public function case_get_offset()
+    {
+        $this
+            ->given(
+                $name        = 'foo',
+                $tokenName   = 'bar',
+                $nodeId      = 'baz',
+                $unification = 0,
+                $offset      = 42,
+                $token       = new SUT($name, $tokenName, $nodeId, $unification),
+                $token->setOffset($offset)
+            )
+            ->when($result = $token->getOffset())
+            ->then
+                ->integer($result)
+                    ->isEqualTo($offset);
+    }
+
     public function case_set_kept()
     {
         $this

Test/Unit/Llk/TreeNode.php

@@ -229,6 +229,26 @@ public function case_get_value_value_undefined()
                     ->isNull();
     }
 
+    public function case_get_offset()
+    {
+        $this
+            ->given($node = new SUT('foo', ['offset' => 42]))
+            ->when($result = $node->getOffset())
+            ->then
+                ->integer($result)
+                    ->isEqualTo(42);
+    }
+
+    public function case_get_offset_undefined()
+    {
+        $this
+            ->given($node = new SUT('foo', ['bar']))
+            ->when($result = $node->getOffset())
+            ->then
+                ->integer($result)
+                    ->isZero();
+    }
+
     public function case_is_token()
     {
         $this