Added tests and removed unused code

raulfraile · Oct 3, 2013 · 3b08528 · 3b08528
1 parent e0b3e2b
commit 3b08528
Show file tree

Hide file tree

Showing 9 changed files with 35 additions and 276 deletions.
diff --git a/src/Ladybug/Model/VariableWrapper.php b/src/Ladybug/Model/VariableWrapper.php
@@ -41,15 +41,6 @@ public function __construct($id, $data, $type = self::TYPE_CLASS)
         $this->type = $type;
     }
 
-    /**
-     * Sets the variable id.
-     * @param string $id
-     */
-    public function setId($id)
-    {
-        $this->id = $id;
-    }
-
     /**
      * Gets the variable id.
      *
@@ -60,15 +51,6 @@ public function getId()
         return $this->id;
     }
 
-    /**
-     * Sets the variable type
-     * @param int $type Variable type: TYPE_CLASS or TYPE_RESOURCE
-     */
-    public function setType($type)
-    {
-        $this->type = $type;
-    }
-
     /**
      * Gets the variable type.
      *
@@ -79,15 +61,6 @@ public function getType()
         return $this->type;
     }
 
-    /**
-     * Sets the variable itself.
-     * @param mixed $data
-     */
-    public function setData($data)
-    {
-        $this->data = $data;
-    }
-
     /**
      * Gets variable.
      *

diff --git a/src/Ladybug/Type/AbstractType.php b/src/Ladybug/Type/AbstractType.php
@@ -211,11 +211,6 @@ public function __clone()
         $this->id = uniqid();
     }
 
-    public function isComposed()
-    {
-        return false;
-    }
-
     public function getInlineValue()
     {
         return $this->value;

diff --git a/src/Ladybug/Type/ArrayType.php b/src/Ladybug/Type/ArrayType.php
@@ -91,27 +91,15 @@ public function getFormattedValue()
     }
 
     /**
-     * Sets if the array is terminal
-     * @param boolean $terminal
-     */
-    public function setTerminal($terminal)
-    {
-        $this->terminal = (boolean) $terminal;
-    }
-
-    /**
+     * Checks whether the variable is terminal.
+     *
      * @return bool
      */
-    public function getTerminal()
+    public function isTerminal()
     {
         return $this->terminal;
     }
 
-    public function isComposed()
-    {
-        return true;
-    }
-
     public function getInlineValue()
     {
         $values = array();

diff --git a/src/Ladybug/Type/ArrayType/Item.php b/src/Ladybug/Type/ArrayType/Item.php
@@ -33,15 +33,6 @@ public function __construct($key, TypeInterface $value)
         $this->value = $value;
     }
 
-    /**
-     * Sets the item key
-     * @param string $key
-     */
-    public function setKey($key)
-    {
-        $this->key = $key;
-    }
-
     /**
      * Gets the item key
      *
@@ -52,15 +43,6 @@ public function getKey()
         return $this->key;
     }
 
-    /**
-     * Sets the item value
-     * @param TypeInterface $value
-     */
-    public function setValue($value)
-    {
-        $this->value = $value;
-    }
-
     /**
      * Gets the item value.
      *
@@ -71,11 +53,11 @@ public function getValue()
         return $this->value;
     }
 
-    public function setLevel($level)
-    {
-        $this->value->setLevel($level);
-    }
-
+    /**
+     * Gets the item level.
+     *
+     * @return int
+     */
     public function getLevel()
     {
         return $this->value->getLevel();