Merge pull request #22 from loadsys/f/add-tests

F/add tests

Author: beporter

.gitignore

@@ -5,3 +5,4 @@ app/tmp/*
 app/[Cc]onfig/core.php
 app/[Cc]onfig/database.php
 !empty
+/vendor/*

composer.json

@@ -20,18 +20,24 @@
     "source": "https://github.com/loadsys/CakePHP-ConfigReadShell"
   },
   "require": {
-    "php": ">=5.4.19",
+    "php": ">=5.4",
+    "cakephp/cakephp": "~3.0",
     "composer/installers": "~1.0"
   },
+  "require-dev": {
+    "phpunit/phpunit": "~4.1",
+    "squizlabs/php_codesniffer": "~2.0",
+    "satooshi/php-coveralls": "dev-master"
+  },
   "autoload": {
     "psr-4": {
       "ConfigRead\\": "src"
     }
   },
   "autoload-dev": {
     "psr-4": {
-      "ConfigRead\\Test\\": "tests",
-      "Cake\\Test\\": "vendor/cakephp/cakephp/test"
+      "Cake\\Test\\": "vendor/cakephp/cakephp/test",
+      "ConfigRead\\Test\\": "tests"
     }
   }
 }

phpunit.xml.dist

@@ -0,0 +1,29 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit
+    backupGlobals="false"
+    backupStaticAttributes="false"
+    colors="true"
+    processIsolation="false"
+    stopOnFailure="false"
+    syntaxCheck="false"
+    bootstrap="./tests/bootstrap.php"
+>
+
+    <testsuites>
+        <testsuite name="ConfigReadShell Test Suite">
+            <directory>./tests/TestCase</directory>
+        </testsuite>
+    </testsuites>
+
+    <!-- Prevent coverage reports from looking in tests and vendors -->
+    <filter>
+        <blacklist>
+            <directory suffix=".php">./vendor/</directory>
+            <directory suffix=".ctp">./vendor/</directory>
+
+            <directory suffix=".php">./tests/</directory>
+            <directory suffix=".ctp">./tests/</directory>
+        </blacklist>
+    </filter>
+
+</phpunit>

src/Shell/ConfigReadShell.php

@@ -2,7 +2,7 @@
 /**
  * ConfigReadShell
  *
- * @package App.Console.Command
+ * @package ConfigRead\Shell
  */
 namespace ConfigRead\Shell;
 
@@ -49,8 +49,6 @@ public function _welcome() {
 	public function startup() {
 		parent::startup();
 
-		Configure::write('debug', 0);
-
 		if (isset($this->params['h'])) {
 			return $this->help();
 		}
@@ -147,7 +145,7 @@ protected function printVal($key, $val) {
 
 		if ($this->formatBash) {
 			$key = strtoupper(str_replace('.', '_', $key));
-			$format = ($this->formatBash ? '%1$s=%2$s' : '%2$s');
+			$format = '%1$s=%2$s';
 		}
 
 		$this->out(sprintf($format, $key, $val));
@@ -160,6 +158,7 @@ protected function printVal($key, $val) {
 	 *
 	 * @access	public
 	 * @return	void
+	 * @codeCoverageIgnore
 	 */
 	public function getOptionParser() {
 		$parser = parent::getOptionParser();