Merge pull request repejota#22 from octante/changed-vars-visibility-connection-options

Changed vars visibility from ConnectionOptions

Author: repejota

src/ConnectionOptions.php

@@ -12,63 +12,63 @@ class ConnectionOptions
      *
      * @var string
      */
-    public $host = "localhost";
+    private $host = "localhost";
 
     /**
      * Port number to connect
      *
      * @var integer
      */
-    public $port = 4222;
+    private $port = 4222;
 
     /**
      * Username to connect
      *
      * @var string
      */
-    public $user = null;
+    private $user = null;
 
     /**
      * Password to connect
      *
      * @var string
      */
-    public $pass = null;
+    private $pass = null;
 
     /**
      * Language of this client
      *
      * @var string
      */
-    public $lang = "php";
+    private $lang = "php";
 
     /**
      * Version of this client
      *
      * @var string
      */
-    public $version = "0.0.5";
+    private $version = "0.0.5";
 
     /**
      * If verbose mode is enabled
      *
      * @var boolean
      */
-    public $verbose = false;
+    private $verbose = false;
 
     /**
      * If pedantic mode is enabled
      *
      * @var boolean
      */
-    public $pedantic = false;
+    private $pedantic = false;
 
     /**
      * If reconnect mode is enabled
      *
      * @var boolean
      */
-    public $reconnect = true;
+    private $reconnect = true;
 
     /**
      * Get the URI for a server
@@ -101,4 +101,220 @@ public function toJSON()
         }
         return json_encode($a);
     }
+
+    /**
+     * Get host
+     *
+     * @return string
+     */
+    public function getHost()
+    {
+        return $this->host;
+    }
+
+    /**
+     * Set host
+     *
+     * @param string $host Host.
+     *
+     * @return $this
+     */
+    public function setHost($host)
+    {
+        $this->host = $host;
+
+        return $this;
+    }
+
+    /**
+     * Get port
+     *
+     * @return integer
+     */
+    public function getPort()
+    {
+        return $this->port;
+    }
+
+    /**
+     * Set port
+     *
+     * @param integer $port Port.
+     *
+     * @return $this
+     */
+    public function setPort($port)
+    {
+        $this->port = $port;
+
+        return $this;
+    }
+
+    /**
+     * Get user
+     *
+     * @return string
+     */
+    public function getUser()
+    {
+        return $this->user;
+    }
+
+    /**
+     * Set user
+     *
+     * @param string $user User.
+     *
+     * @return $this
+     */
+    public function setUser($user)
+    {
+        $this->user = $user;
+
+        return $this;
+    }
+
+    /**
+     * Get password
+     *
+     * @return string
+     */
+    public function getPass()
+    {
+        return $this->pass;
+    }
+
+    /**
+     * Set password
+     *
+     * @param string $pass Password.
+     *
+     * @return $this
+     */
+    public function setPass($pass)
+    {
+        $this->pass = $pass;
+
+        return $this;
+    }
+
+    /**
+     * Get language
+     *
+     * @return string
+     */
+    public function getLang()
+    {
+        return $this->lang;
+    }
+
+    /**
+     * Set language
+     *
+     * @param string $lang Language.
+     *
+     * @return $this
+     */
+    public function setLang($lang)
+    {
+        $this->lang = $lang;
+
+        return $this;
+    }
+
+    /**
+     * Get version
+     *
+     * @return string
+     */
+    public function getVersion()
+    {
+        return $this->version;
+    }
+
+    /**
+     * Set version
+     *
+     * @param string $version Version number.
+     *
+     * @return $this
+     */
+    public function setVersion($version)
+    {
+        $this->version = $version;
+
+        return $this;
+    }
+
+    /**
+     * Get verbose
+     *
+     * @return boolean
+     */
+    public function isVerbose()
+    {
+        return $this->verbose;
+    }
+
+    /**
+     * Set verbose
+     *
+     * @param boolean $verbose Verbose flag.
+     *
+     * @return $this
+     */
+    public function setVerbose($verbose)
+    {
+        $this->verbose = $verbose;
+
+        return $this;
+    }
+
+    /**
+     * Get pedantic
+     *
+     * @return boolean
+     */
+    public function isPedantic()
+    {
+        return $this->pedantic;
+    }
+
+    /**
+     * Set pedantic
+     *
+     * @param boolean $pedantic Pedantic flag.
+     *
+     * @return $this
+     */
+    public function setPedantic($pedantic)
+    {
+        $this->pedantic = $pedantic;
+
+        return $this;
+    }
+
+    /**
+     * Get reconnect
+     *
+     * @return boolean
+     */
+    public function isReconnect()
+    {
+        return $this->reconnect;
+    }
+
+    /**
+     * Set reconnect
+     *
+     * @param boolean $reconnect Reconnect flag.
+     *
+     * @return $this
+     */
+    public function setReconnect($reconnect)
+    {
+        $this->reconnect = $reconnect;
+
+        return $this;
+    }
 }

tests/Unit/ConnectionOptionsTest.php

@@ -0,0 +1,47 @@
+<?php
+/**
+ * Created by PhpStorm.
+ * User: isselguberna
+ * Date: 29/9/15
+ * Time: 23:48
+ */
+
+namespace Nats\tests\Unit;
+
+use Nats\ConnectionOptions;
+
+/**
+ * Class ConnectionOptionsTest
+ */
+class ConnectionOptionsTest extends \PHPUnit_Framework_TestCase
+{
+    /**
+     * Tests Connection Options getters and setters. Only necessary for code coverage.
+     *
+     * @return void
+     */
+    public function testSettersAndGetters()
+    {
+        $options = new ConnectionOptions();
+        $options
+            ->setHost('host')
+            ->setPort(4222)
+            ->setUser('user')
+            ->setPass('password')
+            ->setLang('lang')
+            ->setVersion('version')
+            ->setVerbose(true)
+            ->setPedantic(true)
+            ->setReconnect(true);
+
+        $this->assertEquals('host', $options->getHost());
+        $this->assertEquals(4222, $options->getPort());
+        $this->assertEquals('user', $options->getUser());
+        $this->assertEquals('password', $options->getPass());
+        $this->assertEquals('lang', $options->getLang());
+        $this->assertEquals('version', $options->getVersion());
+        $this->assertTrue($options->isVerbose());
+        $this->assertTrue($options->isPedantic());
+        $this->assertTrue($options->isReconnect());
+    }
+}

tests/Unit/ConnectionTest.php

@@ -62,7 +62,7 @@ public function setUp()
         $options = new ConnectionOptions();
         if (!self::$isGnatsd) {
             time_nanosleep(1, 700000000);
-            $options->port = 4222;
+            $options->setPort(4222);
         }
         $this->c = new Nats\Connection($options);
         $this->c->connect();