Improve transaction model

* Add missing `type` variable.
* Update docblocks.
* Update transaction tests.

lib/Bitreserve/Model/Transaction.php

@@ -11,50 +11,75 @@
 class Transaction extends BaseModel implements TransactionInterface
 {
     /**
-     * @var id
+     * Id.
+     *
+     * @var string
      */
     protected $id;
 
     /**
-     * @var createdAt
+     * Created at.
+     *
+     * @var string
      */
     protected $createdAt;
 
     /**
-     * @var denomination
+     * Denomination.
+     *
+     * @var array
      */
     protected $denomination;
 
     /**
-     * @var destination
+     * Destinantion information.
+     *
+     * @var array
      */
     protected $destination;
 
     /**
-     * @var message
+     * Message.
+     *
+     * @var string
      */
     protected $message;
 
     /**
-     * @var origin
+     * Origin information.
+     *
+     * @var array
      */
     protected $origin;
 
     /**
-     * @var params
+     * Params.
+     *
+     * @var array
      */
     protected $params;
 
     /**
-     * @var refundedById
+     * Refunded Id.
+     *
+     * @var string.
      */
     protected $refundedById;
 
     /**
-     * @var status
+     * Status.
+     *
+     * @var string
      */
     protected $status;
 
+    /**
+     * Type
+     *
+     * @var string
+     */
+    protected $type;
+
     /**
      * Constructor.
      *
@@ -140,6 +165,14 @@ public function getStatus()
         return $this->status;
     }
 
+    /**
+     * {@inheritdoc}
+     */
+    public function getType()
+    {
+        return $this->type;
+    }
+
     /**
      * {@inheritdoc}
      */

lib/Bitreserve/Model/TransactionInterface.php

@@ -8,68 +8,75 @@
 interface TransactionInterface
 {
     /**
-     * Gets transaction creation date.
+     * Get transaction creation date.
      *
      * @return $createdAt
      */
     public function getCreatedAt();
 
     /**
-     * Gets transaction denomination.
+     * Get transaction denomination.
      *
      * @return $denomination
      */
     public function getDenomination();
 
     /**
-     * Gets transaction destinantion.
+     * Get transaction destinantion.
      *
      * @return $destination
      */
     public function getDestination();
 
     /**
-     * Gets transaction id.
+     * Get transaction id.
      *
      * @return $id
      */
     public function getId();
 
     /**
-     * Gets transaction message.
+     * Get transaction message.
      *
      * @return $message
      */
     public function getMessage();
 
     /**
-     * Gets transaction origin.
+     * Get transaction origin.
      *
      * @return $origin
      */
     public function getOrigin();
 
     /**
-     * Gets transaction params.
+     * Get transaction params.
      *
      * @return $params
      */
     public function getParams();
 
     /**
-     * Gets transaction refunded by id.
+     * Get transaction refunded by id.
      *
      * @return $refundedById
      */
     public function getRefundedById();
 
     /**
-     * Gets transaction current status.
+     * Get transaction current status.
      *
      * @return $status
      */
     public function getStatus();
 
+    /**
+     * Get transaction type.
+     *
+     * @return $type
+     */
+    public function getType();
+
     /**
      * Cancel current transaction.
      *

test/Bitreserve/Tests/Model/TransactionTest.php

@@ -154,6 +154,20 @@ public function shouldReturnParams()
         $this->assertEquals($data['params'], $transaction->getParams());
     }
 
+    /**
+     * @test
+     */
+    public function shouldReturnType()
+    {
+        $data = array('type' => 'foobar');
+
+        $client = $this->getBitreserveClientMock();
+
+        $transaction = new Transaction($client, $data);
+
+        $this->assertEquals($data['type'], $transaction->getType());
+    }
+
     /**
      * @test
      */