tnhfw / tnh-fw

tests/include/autoloader.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 	//Autoload function
-	function test_autoload($class){
+	function test_autoload($class) {
 		$classesMap = array(
 			//Caches
 			'ApcCache' => CORE_CLASSES_CACHE_PATH . 'ApcCache.php',
@@ -39,11 +39,11 @@ 
 			'StringHash' => CORE_LIBRARY_PATH . 'StringHash.php',
 			'Upload' => CORE_LIBRARY_PATH . 'Upload.php',
 		);
-		if(isset($classesMap[$class])){
-			if(file_exists($classesMap[$class])){
+		if (isset($classesMap[$class])) {
+			if (file_exists($classesMap[$class])) {
 				include_once $classesMap[$class];
 			}
-			else{
+			else {
 				echo 'File for class ' . $class . ' not found';
 			}
 		}

Braces +1 added lines, -2 removed lines

@@ -42,8 +42,7 @@
 		if(isset($classesMap[$class])){
 			if(file_exists($classesMap[$class])){
 				include_once $classesMap[$class];
-			}
-			else{
+			} else{
 				echo 'File for class ' . $class . ' not found';
 			}
 		}

tests/include/common.php

Indentation +11 added lines, -11 removed lines

@@ -22,7 +22,7 @@ 
 	 * You should have received a copy of the GNU General Public License
 	 * along with this program; if not, write to the Free Software
 	 * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
-	*/
+	 */
 
 
 	function & class_loader($class, $dir = 'libraries', $params = null){
@@ -92,8 +92,8 @@ 
 	}
 
 	/**
-	*  @test
-	*/
+	 *  @test
+	 */
 	function get_config($key, $default = null){
 		static $cfg;
 		if(empty($cfg)){
@@ -121,8 +121,8 @@ 
 	}
 	
 	/**
-	*  @test
-	*/
+	 *  @test
+	 */
 	function is_url($url){
 		return preg_match('/^(http|https|ftp):\/\/(.*)/', $url);
 	}
@@ -142,8 +142,8 @@ 
 
 
 	/**
-	*  @test
-	*/
+	 *  @test
+	 */
 	function attributes_to_string(array $attributes){
 		$str = ' ';
 		//we check that the array passed as an argument is not empty.
@@ -173,8 +173,8 @@ 
 	}
 
 	/**
-	*  @test
-	*/
+	 *  @test
+	 */
 	function clean_input($str){
 		if(is_array($str)){
 			$str = array_map('clean_input', $str);
@@ -193,8 +193,8 @@ 
 	}
 	
 	/**
-	*  @test
-	*/
+	 *  @test
+	 */
 	function string_hidden($str, $startCount = 0, $endCount = 0, $hiddenChar = '*'){
 		//get the string length
 		$len = strlen($str);

Spacing +35 added lines, -35 removed lines

@@ -29,14 +29,14 @@ 
 		//put the first letter of class to upper case 
 		$class = ucfirst($class);
 		static $classes = array();
-		if(isset($classes[$class]) /*hack for duplicate log Logger name*/ && $class != 'Log'){
+		if (isset($classes[$class]) /*hack for duplicate log Logger name*/ && $class != 'Log') {
 			return $classes[$class];
 		}
 		$found = false;
 		foreach (array(ROOT_PATH, CORE_PATH) as $path) {
 			$file = $path . $dir . '/' . $class . '.php';
-			if(file_exists($file)){
-				if(class_exists($class, false) == false){
+			if (file_exists($file)) {
+				if (class_exists($class, false) == false) {
 					require_once $file;
 				}
 				//already found
@@ -44,7 +44,7 @@ 
 				break;
 			}
 		}
-		if(! $found){
+		if (!$found) {
 			//can't use show_error() at this time because some dependencies not yet loaded
 			set_http_status_header(503);
 			echo 'Cannot find the class [' . $class . ']';
@@ -54,7 +54,7 @@ 
 		/*
 		   TODO use the best method to get the Log instance
 		 */
-		if($class == 'Log'){
+		if ($class == 'Log') {
 			//can't use the instruction like "return new Log()" 
 			//because we need return the reference instance of the loaded class.
 			$log = new Log();
@@ -72,7 +72,7 @@ 
 
 	function & class_loaded($class = null){
 		static $list = array();
-		if($class != null){
+		if ($class != null) {
 			$list[strtolower($class)] = $class;
 		}
 		return $list;
@@ -80,7 +80,7 @@ 
 
 	function & load_configurations(array $overwrite_values = array()){
 		static $config;
-		if(empty($config)){
+		if (empty($config)) {
 			$file = CONFIG_PATH . 'config.php';
 			require_once $file;
 		
@@ -94,49 +94,49 @@ 
 	/**
 	*  @test
 	*/
-	function get_config($key, $default = null){
+	function get_config($key, $default = null) {
 		static $cfg;
-		if(empty($cfg)){
+		if (empty($cfg)) {
 			$cfg[0] = & load_configurations();
 		}
 		return array_key_exists($key, $cfg[0]) ? $cfg[0][$key] : $default;
 	}
 
-	function save_to_log($level, $message, $logger = null){
+	function save_to_log($level, $message, $logger = null) {
 		return true;
 	}
 
 	
-	function set_http_status_header($code = 200, $text = null){
+	function set_http_status_header($code = 200, $text = null) {
 		return true;
 	}
 
 	
-	function show_error($msg, $title = 'error', $logging = true){
+	function show_error($msg, $title = 'error', $logging = true) {
 		//throw new RuntimeException('TNHFW Error: '.$msg);
 	}
 
-	function is_https(){
+	function is_https() {
 		return false;
 	}
 	
 	/**
 	*  @test
 	*/
-	function is_url($url){
+	function is_url($url) {
 		return preg_match('/^(http|https|ftp):\/\/(.*)/', $url);
 	}
 		
-	function php_exception_handler($ex){
-		throw new RuntimeException('PHP Exception : '.$ex->getMessage().' | '.$ex->getFile().' | '.$ex->getLine());
+	function php_exception_handler($ex) {
+		throw new RuntimeException('PHP Exception : ' . $ex->getMessage() . ' | ' . $ex->getFile() . ' | ' . $ex->getLine());
 	}
 	
 	
-	function php_error_handler($errno , $errstr, $errfile , $errline, array $errcontext = array()){
-		throw new RuntimeException('TNHFW Exception Error : '.$errstr.' | '.$errfile.' | '.$errline);
+	function php_error_handler($errno, $errstr, $errfile, $errline, array $errcontext = array()) {
+		throw new RuntimeException('TNHFW Exception Error : ' . $errstr . ' | ' . $errfile . ' | ' . $errline);
 	}
 
-	function php_shudown_handler(){
+	function php_shudown_handler() {
 		return true;
 	}
 
@@ -144,11 +144,11 @@ 
 	/**
 	*  @test
 	*/
-	function attributes_to_string(array $attributes){
+	function attributes_to_string(array $attributes) {
 		$str = ' ';
 		//we check that the array passed as an argument is not empty.
-		if(! empty($attributes)){
-			foreach($attributes as $key => $value){
+		if (!empty($attributes)) {
+			foreach ($attributes as $key => $value) {
 				$key = trim(htmlspecialchars($key));
 				$value = trim(htmlspecialchars($value));
 				/*
@@ -158,35 +158,35 @@ 
 				* 	$attr = array('placeholder' => 'I am a "puple"')
 				* 	$str = attributes_to_string($attr); => placeholder = "I am a \"puple\""
 				 */
-				if($value && strpos('"', $value) !== false){
+				if ($value && strpos('"', $value) !== false) {
 					$value = addslashes($value);
 				}
-				$str .= $key.' = "'.$value.'" ';
+				$str .= $key . ' = "' . $value . '" ';
 			}
 		}
 		//remove the space after using rtrim()
 		return rtrim($str);
 	}
 
-	function stringfy_vars($var){
+	function stringfy_vars($var) {
 		return print_r($var, true);
 	}
 
 	/**
 	*  @test
 	*/
-	function clean_input($str){
-		if(is_array($str)){
+	function clean_input($str) {
+		if (is_array($str)) {
 			$str = array_map('clean_input', $str);
 		}
-		else if(is_object($str)){
+		else if (is_object($str)) {
 			$obj = $str;
 			foreach ($str as $var => $value) {
 				$obj->$var = clean_input($value);
 			}
 			$str = $obj;
 		}
-		else{
+		else {
 			$str = htmlspecialchars(strip_tags($str), ENT_QUOTES, 'UTF-8');
 		}
 		return $str;
@@ -195,11 +195,11 @@ 
 	/**
 	*  @test
 	*/
-	function string_hidden($str, $startCount = 0, $endCount = 0, $hiddenChar = '*'){
+	function string_hidden($str, $startCount = 0, $endCount = 0, $hiddenChar = '*') {
 		//get the string length
 		$len = strlen($str);
 		//if str is empty
-		if($len <= 0){
+		if ($len <= 0) {
 			return str_repeat($hiddenChar, 6);
 		}
 		//if the length is less than startCount and endCount
@@ -207,14 +207,14 @@ 
 		//or startCount is negative or endCount is negative
 		//return the full string hidden
 		
-		if((($startCount + $endCount) > $len) || ($startCount == 0 && $endCount == 0) || ($startCount < 0 || $endCount < 0)){
+		if ((($startCount + $endCount) > $len) || ($startCount == 0 && $endCount == 0) || ($startCount < 0 || $endCount < 0)) {
 			return str_repeat($hiddenChar, $len);
 		}
 		//the start non hidden string
 		$startNonHiddenStr = substr($str, 0, $startCount);
 		//the end non hidden string
 		$endNonHiddenStr = null;
-		if($endCount > 0){
+		if ($endCount > 0) {
 			$endNonHiddenStr = substr($str, - $endCount);
 		}
 		//the hidden string
@@ -223,12 +223,12 @@ 
 		return $startNonHiddenStr . $hiddenStr . $endNonHiddenStr;
 	}
 	
-	function set_session_config(){
+	function set_session_config() {
 		return true;
 	}
 	
 	function & get_instance(){
-		if(! Controller::get_instance()){
+		if (!Controller::get_instance()) {
 			$c = new Controller();
 		}
 		return Controller::get_instance();

Braces +2 added lines, -4 removed lines

@@ -178,15 +178,13 @@
 	function clean_input($str){
 		if(is_array($str)){
 			$str = array_map('clean_input', $str);
-		}
-		else if(is_object($str)){
+		} else if(is_object($str)){
 			$obj = $str;
 			foreach ($str as $var => $value) {
 				$obj->$var = clean_input($value);
 			}
 			$str = $obj;
-		}
-		else{
+		} else{
 			$str = htmlspecialchars(strip_tags($str), ENT_QUOTES, 'UTF-8');
 		}
 		return $str;

tests/include/testsUtil.php

Indentation +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php
 
 	/**
-	* Function to test private & protected method
-	*/
+	 * Function to test private & protected method
+	 */
 	function runPrivateOrProtectedMethod($object, $method, array $args = array()){
 		$r = new ReflectionClass(get_class($object));
 		$m = $r->getMethod($method);

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 	/**
 	* Function to test private & protected method
 	*/
-	function runPrivateOrProtectedMethod($object, $method, array $args = array()){
+	function runPrivateOrProtectedMethod($object, $method, array $args = array()) {
 		$r = new ReflectionClass(get_class($object));
 		$m = $r->getMethod($method);
 		$m->setAccessible(true);

tests/tnhfw/libraries/FormValidationTest.php

Indentation +30 added lines, -30 removed lines

@@ -3,66 +3,66 @@ 
 class FormValidationTest extends TestCase
 {   
 	public static function setUpBeforeClass()
-    {
+	{
     
-    }
+	}
 	
 	public static function tearDownAfterClass()
-    {
+	{
         
-    }
+	}
 	
-    protected function setUp()
-    {
+	protected function setUp()
+	{
 		
-    }
+	}
 
-    protected function tearDown()
-    {
+	protected function tearDown()
+	{
 		
-    }
+	}
 	
 	
 
-    // tests
-    public function testValidationDataIsEmpty()
-    {
+	// tests
+	public function testValidationDataIsEmpty()
+	{
 		$fv = new FormValidation();
 		$this->assertEmpty($fv->getData());
-    }
+	}
 	
 	public function testValidationDataIsNotEmpty()
-    {
+	{
 		$fv = new FormValidation();
 		$fv->setData(array('name' => 'mike'));
 		$this->assertNotEmpty($fv->getData());
 		$this->assertArrayHasKey('name', $fv->getData());
-    }
+	}
 	
 	public function testCannotDoValidation()
-    {
+	{
 		$fv = new FormValidation();
 		$this->assertFalse($fv->canDoValidation());
-    }
+	}
 
 	public function testSettingErrorDelimiter()
-    {
+	{
 		$fv = new FormValidation();
 		$fv->setErrorDelimiter('<a>', '</b>');
 		$this->assertContains('<a>', $fv->getErrorDelimiter());
 		$this->assertContains('</b>', $fv->getErrorDelimiter());
-    }
+	}
 	
 	public function testSettingErrorsDelimiter()
-    {
+	{
 		$fv = new FormValidation();
 		$fv->setErrorsDelimiter('<foo>', '</bar>');
 		$this->assertContains('<foo>', $fv->getErrorsDelimiter());
 		$this->assertContains('</bar>', $fv->getErrorsDelimiter());
-    }
+	}
 	
 	public function testSettingErrorMessageOverride()
-    {
+	{
 		
 		//field specific message for the rule
 		$fv = new FormValidation();
@@ -82,10 +82,10 @@ 
 
 		$this->assertFalse($fv->run());
 		$this->assertContains('global required message error', $fv->returnErrors());
-    }
+	}
 	
 	public function testSettingCustomErrorMessage()
-    {
+	{
 		
 		$fv = new FormValidation();
 		$fv->setData(array('foo' => ''));
@@ -103,20 +103,20 @@ 
 		
 		$this->assertFalse($fv->run());
 		$this->assertContains('custom "bar" message error', $fv->returnErrors());	
-    }
+	}
 	
 	public function testReturnErrorsArray()
-    {
+	{
 		$fv = new FormValidation();
 		$fv->setRule('name', 'name', 'required');
 		$fv->setData(array('name' => ''));
 		$this->assertFalse($fv->run());
 		$this->assertNotEmpty($fv->returnErrors());
-    }
+	}
 	
 	
 	public function testRequiredRule()
-    {
+	{
 		$fv = new FormValidation();
 		$fv->setRule('name', 'name', 'required');
 		$fv->setData(array('name' => ''));
@@ -131,5 +131,5 @@ 
 		$fv->setRule('name', 'name', 'required');
 		$fv->setData(array('name' => 'tony'));
 		$this->assertTrue($fv->run());
-    }
+	}
 }
\ No newline at end of file

tests/tnhfw/libraries/BackupAllTest.php

Spacing +10 added lines, -10 removed lines

@@ -30,15 +30,15 @@ 
 		// tests
 		public function testSomeFeature()
 		{
-			$config['driver']    =  'mysql';
-			$config['username']  =  'root';
-			$config['password']  =  '';
-			$config['database']  =  'db_gesco';
-			$config['hostname']  =  'localhost:3307';
+			$config['driver']    = 'mysql';
+			$config['username']  = 'root';
+			$config['password']  = '';
+			$config['database']  = 'db_gesco';
+			$config['hostname']  = 'localhost:3307';
 			$config['charset']   = 'utf8';
 			$config['collation'] = 'utf8_general_ci';
-			$config['prefix']    =  '';
-			$config['port']      =  3307;
+			$config['prefix']    = '';
+			$config['port']      = 3307;
 			
 			$l = new Log();
 			$l->setLogger('TNHBIS');
@@ -92,21 +92,21 @@ 
 		
 		
 		
-		public function testConfigChange(){
+		public function testConfigChange() {
 			//echo Config::get('base_url');
 			$c = new Config();
 			$c->set('base_url', 'foo');
 			//echo get_config('base_url'); 
 		}
 		
-		public function testCallPrivateProtectedMethod(){
+		public function testCallPrivateProtectedMethod() {
 			$c = new FileCache();
 			$expected = CACHE_PATH . md5('foo') . '.cache';
 			$result = runPrivateOrProtectedMethod($c, 'getFilePath', array('foo'));
 			$this->assertEquals($expected, $result);
 		}
 		
-		public function testMocks(){
+		public function testMocks() {
 			$mock = $this->getMockBuilder('Model')->getMock();
 			$mock->expects($this->any())->method('get')->will($this->returnValue('foobar'));
 			$this->assertEquals('foobar', $mock->get(1));

tests/tnhfw/CommonTest.php

Spacing +5 added lines, -5 removed lines

@@ -24,20 +24,20 @@ 
 		}
 
 		
-		public function testFunctionGetConfigKeyNotExist(){
+		public function testFunctionGetConfigKeyNotExist() {
 			$key = 'foo';
 			$cfg = get_config($key);
 			$this->assertNull($cfg);
 		}
 		
-		public function testFunctionGetConfigKeyNotExistUsingDefaultValue(){
+		public function testFunctionGetConfigKeyNotExistUsingDefaultValue() {
 			$key = 'foo';
 			$expected = 'bar';
 			$cfg = get_config($key, $expected);
 			$this->assertEquals($cfg, $expected);
 		}
 		
-		public function testFunctionGetConfigAfterSet(){
+		public function testFunctionGetConfigAfterSet() {
 			$key = 'foo';
 			$expected = 'bar';
 			$c = new Config();
@@ -47,9 +47,9 @@ 
 			$this->assertEquals($cfg, $expected);
 		}
 		
-		public function testVsStream(){
+		public function testVsStream() {
 		
-			$vfs =  vfsStream::setup('tnhfw');
+			$vfs = vfsStream::setup('tnhfw');
 			$this->assertFalse($vfs->hasChild('test'));
 			mkdir(vfsStream::url('tnhfw') . DS . 'test');
 			$this->assertTrue($vfs->hasChild('test'));

tests/tnhfw/classes/EventTest.php

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
 
 		
 		
-		public function testDefaultValue(){
+		public function testDefaultValue() {
 			$e = new Event('foo');
 			$this->assertSame($e->name, 'foo');
 			$this->assertSame($e->payload, array());
@@ -33,7 +33,7 @@ 
 			$this->assertFalse($e->stop);
 		}
 		
-		public function testPayloadValueIsSet(){
+		public function testPayloadValueIsSet() {
 			$e = new Event('foo', array('bar'));
 			$this->assertSame($e->name, 'foo');
 			$this->assertSame($e->payload, array('bar'));
@@ -41,7 +41,7 @@ 
 			$this->assertFalse($e->stop);
 		}
 		
-		public function testReturnBackValueIsSetToTrue(){
+		public function testReturnBackValueIsSetToTrue() {
 			$e = new Event('foo', array('bar'), true);
 			$this->assertSame($e->name, 'foo');
 			$this->assertSame($e->payload, array('bar'));
@@ -49,7 +49,7 @@ 
 			$this->assertFalse($e->stop);
 		}
 		
-		public function testStopValueIsSetToTue(){
+		public function testStopValueIsSetToTue() {
 			$e = new Event('foo', array('bar'), true, true);
 			$this->assertSame($e->name, 'foo');
 			$this->assertSame($e->payload, array('bar'));

tests/tnhfw/classes/DBSessionHandlerTest.php

Spacing +8 added lines, -8 removed lines

@@ -45,7 +45,7 @@ 
 
 		
 		
-		public function testUsingSessionConfiguration(){
+		public function testUsingSessionConfiguration() {
 			$secret = 'bXlzZWNyZXQ';
 			//using value in the configuration
 			static::$config->set('session_save_path', 'DBSessionModel');
@@ -72,7 +72,7 @@ 
 			$this->assertEquals($dbsh->decode($encoded), 'foo');
 		}
 		
-		public function testWhenDataIsExpired(){
+		public function testWhenDataIsExpired() {
 			$model = new DBSessionModel($this->db);
 			//to prevent old data conflict
 			$model->truncate();
@@ -90,7 +90,7 @@ 
 			$this->assertFalse($dbsh->read('foo'));
 		}
 		
-		public function testWhenDataAlreadyExistDoUpdate(){
+		public function testWhenDataAlreadyExistDoUpdate() {
 			$model = new DBSessionModel($this->db);
 			//to prevent old data conflict
 			$model->truncate();
@@ -108,7 +108,7 @@ 
 			$this->assertEquals($dbsh->read('foo'), '445');	
 		}
 		
-		public function testUsingCustomModelInstance(){
+		public function testUsingCustomModelInstance() {
 			
 			$model = new DBSessionModel($this->db);
 			//to prevent old data conflict
@@ -142,7 +142,7 @@ 
 		}
 			
 			
-		public function testUsingCustomLogInstance(){
+		public function testUsingCustomLogInstance() {
 			$model = new DBSessionModel($this->db);
 			//to prevent old data conflict
 			$model->truncate();
@@ -174,7 +174,7 @@ 
 			$this->assertEquals($dbsh->decode($encoded), 'foo');
 		}
 		
-		public function testUsingCustomLoaderInstance(){
+		public function testUsingCustomLoaderInstance() {
 			$model = new DBSessionModel($this->db);
 			//to prevent old data conflict
 			$model->truncate();
@@ -206,7 +206,7 @@ 
 			$this->assertEquals($dbsh->decode($encoded), 'foo');
 		}
 		
-		public function testWhenModelInsanceIsNotSet(){
+		public function testWhenModelInsanceIsNotSet() {
 			$model = new DBSessionModel($this->db);
 			//to prevent old data conflict
 			$model->truncate();
@@ -238,7 +238,7 @@ 
 			$this->assertEquals($dbsh->decode($encoded), 'foo');
 		}
 		
-		public function testWhenModelTableColumnsIsNotSet(){
+		public function testWhenModelTableColumnsIsNotSet() {
 			$model = new DBSessionModel($this->db);
 			//to prevent old data conflict
 			$model->truncate();

tests/tnhfw/classes/SessionTest.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 			$this->assertEquals('bar', Session::get('foo'));
 		}
 		
-		public function testExc(){
+		public function testExc() {
 			 //$this->expectException(InvalidArgumentException::class);
 		}
 	}
\ No newline at end of file