adamjakab / SuiteCRM

tests/tests/include/utils/encryptionUtilsTest.php

Spacing +36 added lines, -36 removed lines

@@ -4,48 +4,48 @@ 
 class encryption_utilsTest extends PHPUnit_Framework_TestCase
 {
 	
-	public function testsugarEncode(){
+	public function testsugarEncode() {
 
 		//execute the method and test if it returns expected values
 		//key param does nothing currently.
 		
 		//blank key and data
-		$expected ="";
-		$actual = sugarEncode("","");
-		$this->assertSame($expected,$actual);
+		$expected = "";
+		$actual = sugarEncode("", "");
+		$this->assertSame($expected, $actual);
 				
 		//blank key and valid data
-		$expected ="RGF0YQ==";
-		$actual = sugarEncode("","Data");
-		$this->assertSame($expected,$actual);
+		$expected = "RGF0YQ==";
+		$actual = sugarEncode("", "Data");
+		$this->assertSame($expected, $actual);
 					
 		//valid key and data
-		$expected ="RGF0YQ==";
-		$actual = sugarEncode("key","Data");
-		$this->assertSame($expected,$actual);
+		$expected = "RGF0YQ==";
+		$actual = sugarEncode("key", "Data");
+		$this->assertSame($expected, $actual);
 		
 	}
 	
 	
-	public function testsugarDecode(){
+	public function testsugarDecode() {
 
 		//execute the method and test if it returns expected values
 		//key param does nothing currently.
 		
 		//blank key and data
-		$expected ="";
-		$actual = sugarDecode("","");
-		$this->assertSame($expected,$actual);
+		$expected = "";
+		$actual = sugarDecode("", "");
+		$this->assertSame($expected, $actual);
 		
 		//blank key and valid data
-		$expected ="Data";
-		$actual = sugarDecode("","RGF0YQ==");
-		$this->assertSame($expected,$actual);
+		$expected = "Data";
+		$actual = sugarDecode("", "RGF0YQ==");
+		$this->assertSame($expected, $actual);
 			
 		//valid key and data
-		$expected ="Data";
-		$actual = sugarDecode("key","RGF0YQ==");
-		$this->assertSame($expected,$actual);
+		$expected = "Data";
+		$actual = sugarDecode("key", "RGF0YQ==");
+		$this->assertSame($expected, $actual);
 		
 		
 	}
@@ -57,50 +57,50 @@ 
 		
 		//test key
 		$actual = blowfishGetKey('test');
-		$this->assertGreaterThanOrEqual(36,strlen($actual));
+		$this->assertGreaterThanOrEqual(36, strlen($actual));
 		//var_dump($actual);
 		
 		//default key
 		$actual = blowfishGetKey('rapelcg_svryq');
-		$this->assertGreaterThanOrEqual(36,strlen($actual)); 
+		$this->assertGreaterThanOrEqual(36, strlen($actual)); 
 		
 	}
 	
 	
-	public function testblowfishEncode(){
+	public function testblowfishEncode() {
 	
 		//execute the method and test if it returns expected values
 		//it won't work with blank key, will throw an error
 
 		//valid key and blank data
-		$expected ="";
-		$actual = blowfishEncode("test","");
-		$this->assertSame($expected,$actual);
+		$expected = "";
+		$actual = blowfishEncode("test", "");
+		$this->assertSame($expected, $actual);
 		
 		
 		//valid key and valid data
-		$expected ="HI1/88NJJss=";
-		$actual = blowfishEncode("test","Data");
-		$this->assertSame($expected,$actual);
+		$expected = "HI1/88NJJss=";
+		$actual = blowfishEncode("test", "Data");
+		$this->assertSame($expected, $actual);
 		
 	}
 	
 	
-	public function testblowfishDecode(){
+	public function testblowfishDecode() {
 	
 		//execute the method and test if it returns expected values
 		//it won't work with blank key, will throw an error.
 		
 		//valid key and blank data
-		$expected ="";
-		$actual = blowfishDecode("test","");
-		$this->assertSame($expected,$actual);
+		$expected = "";
+		$actual = blowfishDecode("test", "");
+		$this->assertSame($expected, $actual);
 		
 		
 		//valid key and valid data
-		$expected ="Data";
-		$actual = blowfishDecode("test","HI1/88NJJss=");
-		$this->assertSame($expected,$actual);
+		$expected = "Data";
+		$actual = blowfishDecode("test", "HI1/88NJJss=");
+		$this->assertSame($expected, $actual);
 				
 	}
 

tests/tests/include/utils/arrayUtilsTest.php

Spacing +89 added lines, -89 removed lines

@@ -8,80 +8,80 @@ 
 	
 		//execute the method and test if it returns expected values
 		
-		$tempArray = Array( "Key1" => "value1" , "Key2" => "value2" );
+		$tempArray = Array("Key1" => "value1", "Key2" => "value2");
 		
 		$expected = "array (\n  'Key1' => 'value1',\n  'Key2' => 'value2',\n)";
 		$actual = var_export_helper($tempArray);
-		$this->assertSame($actual,$expected);
+		$this->assertSame($actual, $expected);
 	
 	}
 	
-	public function testoverride_value_to_string(){
+	public function testoverride_value_to_string() {
 	
 		//execute the method and test if it returns expected values
 		
 		$expected = "\$array_name['value_name'] = 'value';";
-		$actual =  override_value_to_string('array_name', 'value_name', 'value');
-		$this->assertSame($actual,$expected);
+		$actual = override_value_to_string('array_name', 'value_name', 'value');
+		$this->assertSame($actual, $expected);
 		
 	}
 	
-	public function testadd_blank_option(){
+	public function testadd_blank_option() {
 	
 		//execute the method with array not having any blank key value pair. function will return an array with blank key value pair added.
-		$tempArray = Array( "Key1" => "value1" , "Key2" => "value2" );
-		$expected = Array( "" => "" , "Key1" => "value1" , "Key2" => "value2" );
+		$tempArray = Array("Key1" => "value1", "Key2" => "value2");
+		$expected = Array("" => "", "Key1" => "value1", "Key2" => "value2");
 	
-		$actual =  add_blank_option($tempArray);
-		$this->assertSame($actual,$expected);
+		$actual = add_blank_option($tempArray);
+		$this->assertSame($actual, $expected);
 		
 		
 		//execute the method with array having a blank key value pair. function will return the same array back without any change.		
-		$tempArray = Array("" => "" , "Key1" => "value1" , "Key2" => "value2" );
-		$expected = Array( "" => "" , "Key1" => "value1" , "Key2" => "value2" );
+		$tempArray = Array("" => "", "Key1" => "value1", "Key2" => "value2");
+		$expected = Array("" => "", "Key1" => "value1", "Key2" => "value2");
 	
-		$actual =  add_blank_option($tempArray);
-		$this->assertSame($actual,$expected);
+		$actual = add_blank_option($tempArray);
+		$this->assertSame($actual, $expected);
 			
 	}
 	
 	
-	public function testoverride_value_to_string_recursive(){
+	public function testoverride_value_to_string_recursive() {
 	
 		//execute the method and test if it returns expected values
 		
 		//without keys
-		$tempArray = Array( "Key1" => "value1" , "Key2" => "value2" );
+		$tempArray = Array("Key1" => "value1", "Key2" => "value2");
 		$expected = "\$tempArray=array (\n  'Key1' => 'value1',\n  'Key2' => 'value2',\n);";
-		$actual =  override_value_to_string_recursive('','tempArray',$tempArray); 
-		$this->assertSame($actual,$expected);
+		$actual = override_value_to_string_recursive('', 'tempArray', $tempArray); 
+		$this->assertSame($actual, $expected);
 		
 		//with keys
 		$tempArray = Array();
-		$tempArray["Key1"]["Key2"] = Array( "Key3" => "value" , "Key4" => "value" );
+		$tempArray["Key1"]["Key2"] = Array("Key3" => "value", "Key4" => "value");
 		$expected = "\$tempArray['key1']['key2']=array (\n  'Key1' => \n  array (\n    'Key2' => \n    array (\n      'Key3' => 'value',\n      'Key4' => 'value',\n    ),\n  ),\n);";
-		$actual =  override_value_to_string_recursive(array('key1','key2'),'tempArray',$tempArray); 
+		$actual = override_value_to_string_recursive(array('key1', 'key2'), 'tempArray', $tempArray); 
 		//var_dump( nl2br($actual));
-		$this->assertSame($actual,$expected);
+		$this->assertSame($actual, $expected);
 		
 	}
 	
-	public function testoverride_recursive_helper(){
+	public function testoverride_recursive_helper() {
 	
 		//execute the method and test if it returns expected values
 		
 		//without keys
-		$tempArray = Array( "Key1" => "value1" , "Key2" => "value2" );
+		$tempArray = Array("Key1" => "value1", "Key2" => "value2");
 		$expected = "=array (\n  'Key1' => 'value1',\n  'Key2' => 'value2',\n);";
-		$actual =  override_recursive_helper('','tempArray',$tempArray);
-		$this->assertSame($actual,$expected);
+		$actual = override_recursive_helper('', 'tempArray', $tempArray);
+		$this->assertSame($actual, $expected);
 		
 		//with keys
 		$tempArray = Array();
-		$tempArray["Key1"]["Key2"] = Array( "Key3" => "value" , "Key4" => "value" );
+		$tempArray["Key1"]["Key2"] = Array("Key3" => "value", "Key4" => "value");
 		$expected = "['key1']['key2']=array (\n  'Key1' => \n  array (\n    'Key2' => \n    array (\n      'Key3' => 'value',\n      'Key4' => 'value',\n    ),\n  ),\n);";
-		$actual =  override_recursive_helper(array('key1','key2'),'tempArray',$tempArray); 
-		$this->assertSame($actual,$expected);
+		$actual = override_recursive_helper(array('key1', 'key2'), 'tempArray', $tempArray); 
+		$this->assertSame($actual, $expected);
 				
 		
 	}
@@ -92,23 +92,23 @@ 
 		
 		//null array
 		$expected = Null;
-		$actual =  override_value_to_string_recursive2( 'tempArray', 'key1','',false);
-		$this->assertSame($actual,$expected);
+		$actual = override_value_to_string_recursive2('tempArray', 'key1', '', false);
+		$this->assertSame($actual, $expected);
 		
 		 
 		//simple array
-		$tempArray = Array( "Key1" => "value1" , "Key2" => "value2" );
+		$tempArray = Array("Key1" => "value1", "Key2" => "value2");
 		$expected = "\$['tempArray']['Key1'] = 'value1';\n$['tempArray']['Key2'] = 'value2';\n";
-		$actual =  override_value_to_string_recursive2( '', 'tempArray', $tempArray);
-		$this->assertSame($actual,$expected);
+		$actual = override_value_to_string_recursive2('', 'tempArray', $tempArray);
+		$this->assertSame($actual, $expected);
 		
 		
 		//complex array
 		$tempArray = Array();
-		$tempArray["Key1"]["Key2"] = Array( "Key3" => "value" , "Key4" => "value" );
+		$tempArray["Key1"]["Key2"] = Array("Key3" => "value", "Key4" => "value");
 		$expected = "\$tempArray['key1']['Key2']['Key3'] = 'value';\n\$tempArray['key1']['Key2']['Key4'] = 'value';\n";
-		$actual =  override_value_to_string_recursive2('tempArray','key1',$tempArray["Key1"]);  
-		$this->assertSame($actual,$expected);
+		$actual = override_value_to_string_recursive2('tempArray', 'key1', $tempArray["Key1"]);  
+		$this->assertSame($actual, $expected);
 	
 	}
 	
@@ -121,14 +121,14 @@ 
 		$obj = '';
 		$expected = '';
 		$actual = object_to_array_recursive($obj);
-		$this->assertSame($actual,$expected);
+		$this->assertSame($actual, $expected);
 		
 		//test with a valid object
 		$obj = new TimeDate();
 		$expected = Array('dbDayFormat' => 'Y-m-d', 'dbTimeFormat' => 'H:i:s', 'allow_cache' => true); 
 		$actual = object_to_array_recursive($obj);
 		
-		$this->assertSame($actual,$expected);
+		$this->assertSame($actual, $expected);
 		
 	}
 
@@ -137,43 +137,43 @@ 
 		//execute the method and test if it returns expected values
 		
 		//same simple arrays
-		$tempArray1 = Array( "Key1" => "value1" , "Key2" => "value2" );
-		$tempArray2 = Array( "Key1" => "value1" , "Key2" => "value2" );
+		$tempArray1 = Array("Key1" => "value1", "Key2" => "value2");
+		$tempArray2 = Array("Key1" => "value1", "Key2" => "value2");
 		$expected = Array();
-		$actual =  deepArrayDiff( $tempArray1, $tempArray2);
-		$this->assertSame($actual,$expected);
+		$actual = deepArrayDiff($tempArray1, $tempArray2);
+		$this->assertSame($actual, $expected);
 		
 		
 		//different simple arrays
-		$tempArray1 = Array( "Key1" => "value1" , "Key2" => "value2" );
-		$tempArray2 = Array( "Key1" => "value1" , "Key2" => "value3" );
-		$expected = Array( "Key2" => "value2");
-		$actual =  deepArrayDiff( $tempArray1, $tempArray2);
-		$this->assertSame($actual,$expected);
+		$tempArray1 = Array("Key1" => "value1", "Key2" => "value2");
+		$tempArray2 = Array("Key1" => "value1", "Key2" => "value3");
+		$expected = Array("Key2" => "value2");
+		$actual = deepArrayDiff($tempArray1, $tempArray2);
+		$this->assertSame($actual, $expected);
 		
 
 		//same complex arrays
-		$tempArray1 = Array("Key1" => Array( "Key2" => "value2" , "Key3" => "value3" ));
-		$tempArray2 = Array("Key1" => Array( "Key2" => "value2" , "Key3" => "value3" ));
+		$tempArray1 = Array("Key1" => Array("Key2" => "value2", "Key3" => "value3"));
+		$tempArray2 = Array("Key1" => Array("Key2" => "value2", "Key3" => "value3"));
 		$expected = Array();
-		$actual =  deepArrayDiff( $tempArray1, $tempArray2);
-		$this->assertSame($actual,$expected);
+		$actual = deepArrayDiff($tempArray1, $tempArray2);
+		$this->assertSame($actual, $expected);
 		
 		
 		//complex arrays with different root node
-		$tempArray1 = Array("Key1" => Array( "Key2" => "value2" , "Key3" => "value3" ));
-		$tempArray2 = Array("Key2" => Array( "Key2" => "value2" , "Key3" => "value3" ));
-		$expected = Array("Key1" => Array( "Key2" => "value2" , "Key3" => "value3" ));
-		$actual =  deepArrayDiff( $tempArray1, $tempArray2);
-		$this->assertSame($actual,$expected);
+		$tempArray1 = Array("Key1" => Array("Key2" => "value2", "Key3" => "value3"));
+		$tempArray2 = Array("Key2" => Array("Key2" => "value2", "Key3" => "value3"));
+		$expected = Array("Key1" => Array("Key2" => "value2", "Key3" => "value3"));
+		$actual = deepArrayDiff($tempArray1, $tempArray2);
+		$this->assertSame($actual, $expected);
 		
 		
 		//complex arrays with different child node
-		$tempArray1 = Array("Key1" => Array( "Key2" => "value2" , "Key3" => "value3" ));
-		$tempArray2 = Array("Key1" => Array( "Key2" => "value2" , "Key4" => "value4" ));
-		$expected = Array("Key1" => Array( "Key3" => "value3" ));
-		$actual =  deepArrayDiff( $tempArray1, $tempArray2);
-		$this->assertSame($actual,$expected);
+		$tempArray1 = Array("Key1" => Array("Key2" => "value2", "Key3" => "value3"));
+		$tempArray2 = Array("Key1" => Array("Key2" => "value2", "Key4" => "value4"));
+		$expected = Array("Key1" => Array("Key3" => "value3"));
+		$actual = deepArrayDiff($tempArray1, $tempArray2);
+		$this->assertSame($actual, $expected);
 
 	}
 	
@@ -183,17 +183,17 @@ 
 		//execute the method and test if it returns expected values
 		
 		//add to existing array
-		$tempArray = Array("Key1" => Array( "Key2" => "value2" , "Key3" => "value3" ));
-		$expected = Array("Key1" => Array( "Key2" => "value2" , "Key3" => "value3" ),"key4" => "value4" );
-		setDeepArrayValue( $tempArray, "key4","value4");
-		$this->assertSame($tempArray,$expected);
+		$tempArray = Array("Key1" => Array("Key2" => "value2", "Key3" => "value3"));
+		$expected = Array("Key1" => Array("Key2" => "value2", "Key3" => "value3"), "key4" => "value4");
+		setDeepArrayValue($tempArray, "key4", "value4");
+		$this->assertSame($tempArray, $expected);
 		
 		//add to empty array
 		$tempArray = Array();
-		$expected = Array("key1" => Array( "key2" => Array("key3" => "value3")) );
-		setDeepArrayValue( $tempArray, "key1_key2_key3","value3");
+		$expected = Array("key1" => Array("key2" => Array("key3" => "value3")));
+		setDeepArrayValue($tempArray, "key1_key2_key3", "value3");
 		//var_dump($tempArray);
-		$this->assertSame($tempArray,$expected);
+		$this->assertSame($tempArray, $expected);
 		
 	}
 
@@ -202,17 +202,17 @@ 
 		//execute the method and test if it returns expected values
 		
 		//try with two different length arrays
-		$tempArray1 = Array("v1", "v2" , "v3" );
-		$tempArray2 = Array("v4", "v5" );
-		$actual = array_merge_values( $tempArray1,  $tempArray2);		
+		$tempArray1 = Array("v1", "v2", "v3");
+		$tempArray2 = Array("v4", "v5");
+		$actual = array_merge_values($tempArray1, $tempArray2);		
 		$this->assertFalse($actual);
 		
 		//try with same length arrays.
-		$tempArray1 = Array("v1", "v2" , "v3" );
-		$tempArray2 = Array("v4", "v5" , "v6" );
+		$tempArray1 = Array("v1", "v2", "v3");
+		$tempArray2 = Array("v4", "v5", "v6");
 		$expected = array("v1v4", "v2v5", "v3v6");
-		$actual = array_merge_values( $tempArray1,  $tempArray2);
-		$this->assertSame($expected,$actual);
+		$actual = array_merge_values($tempArray1, $tempArray2);
+		$this->assertSame($expected, $actual);
 	}
 	
 	
@@ -221,19 +221,19 @@ 
 		//execute the method and test if it returns expected value
 		
 		//test with invalid input
-		$tempArray ="";
-		$actual = array_search_insensitive('',$tempArray);
+		$tempArray = "";
+		$actual = array_search_insensitive('', $tempArray);
 		$this->assertFalse($actual);
 		
 		
 		//test with invalid needle..
-		$tempArray = Array("Key1" => "value1", "Key2" => "value2" , "Key3" => "value3" ,"key4" => "value4" );
+		$tempArray = Array("Key1" => "value1", "Key2" => "value2", "Key3" => "value3", "key4" => "value4");
 		$actual = array_search_insensitive('', $tempArray);
 		$this->assertFalse($actual);
 		
 		
 		//test with valid needle and haystack.
-		$tempArray = Array("Key1" => "value1", "Key2" => "value2" , "Key3" => "value3" ,"key4" => "value4" );
+		$tempArray = Array("Key1" => "value1", "Key2" => "value2", "Key3" => "value3", "key4" => "value4");
 		$actual = array_search_insensitive('value4', $tempArray);
 		$this->assertTrue($actual);
 		
@@ -245,17 +245,17 @@ 
     	//execute the method and test if it returns expected values
     	
     	//test for a top level key
-    	$tempArray = new SugarArray(Array("Key1" => Array("Key2" => "value2" , "Key3" => "value3") ,"key4" => "value4" ));
+    	$tempArray = new SugarArray(Array("Key1" => Array("Key2" => "value2", "Key3" => "value3"), "key4" => "value4"));
     	$expected = "value4";
     	$actual = $tempArray->get('key4');
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
     	
     	
     	//test for a child level key with dot notation
-    	$tempArray = new SugarArray(Array("key1" => Array("key2" => "value2" , "key3" => "value3") ,"key4" => "value4" ));
+    	$tempArray = new SugarArray(Array("key1" => Array("key2" => "value2", "key3" => "value3"), "key4" => "value4"));
     	$expected = "value3";
     	$actual = $tempArray->get('key1.key3');   	
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
     	
     }
 
@@ -264,17 +264,17 @@ 
     	//execute the method and test if it returns expected values
     	
     	//test for a top level key
-    	$haystack = Array("key1" => Array("key2" => "value2" , "key3" => "value3") ,"key4" => "value4" );
+    	$haystack = Array("key1" => Array("key2" => "value2", "key3" => "value3"), "key4" => "value4");
     	$expected = "value4";
-    	$actual = SugarArray::staticGet($haystack,"key4");
-    	$this->assertSame($expected,$actual);
+    	$actual = SugarArray::staticGet($haystack, "key4");
+    	$this->assertSame($expected, $actual);
     	 
     	 
     	//test for a child level key with dot notation
-    	$haystack = Array("key1" => Array("key2" => "value2" , "key3" => "value3") ,"key4" => "value4" );
+    	$haystack = Array("key1" => Array("key2" => "value2", "key3" => "value3"), "key4" => "value4");
     	$expected = "value3";
-    	$actual = SugarArray::staticGet($haystack,'key1.key3');
-    	$this->assertSame($expected,$actual);
+    	$actual = SugarArray::staticGet($haystack, 'key1.key3');
+    	$this->assertSame($expected, $actual);
     	
     	
     }

tests/tests/include/utils/mvcUtilsTest.php

Spacing +2 added lines, -2 removed lines

@@ -24,14 +24,14 @@
 		//test without setting REQUEST param
 		$expected = "javascript:void window.open('index.php?','printwin','menubar=1,status=0,resizable=1,scrollbars=1,toolbar=0,location=1')";
 		$actual = getPrintLink();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 		
 		//test with required REQUEST param set
 		$_REQUEST['action'] = "ajaxui";
 		$expected = "javascript:SUGAR.ajaxUI.print();";
 		$actual = getPrintLink();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 
 	}
 	

tests/tests/modules/AOS_Product_Categories/AOS_Product_CategoriesTest.php

Spacing +6 added lines, -6 removed lines

@@ -2,13 +2,13 @@ 
 
 class AOS_Product_CategoriesTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAOS_Product_Categories(){	
+	public function testAOS_Product_Categories() {	
 		
 		//execute the contructor and check for the Object type and  attributes
 		$aosProductCategories = new AOS_Product_Categories();
-		$this->assertInstanceOf('AOS_Product_Categories',$aosProductCategories);
-		$this->assertInstanceOf('Basic',$aosProductCategories);
-		$this->assertInstanceOf('SugarBean',$aosProductCategories);
+		$this->assertInstanceOf('AOS_Product_Categories', $aosProductCategories);
+		$this->assertInstanceOf('Basic', $aosProductCategories);
+		$this->assertInstanceOf('SugarBean', $aosProductCategories);
 			
 		$this->assertAttributeEquals('AOS_Product_Categories', 'module_dir', $aosProductCategories);
 		$this->assertAttributeEquals('AOS_Product_Categories', 'object_name', $aosProductCategories);
@@ -20,7 +20,7 @@ 
 	}
 
   
-    public function testsave(){
+    public function testsave() {
  
     	error_reporting(E_ERROR | E_PARSE);
     	
@@ -39,7 +39,7 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$aosProductCategories->mark_deleted($aosProductCategories->id);
     	$result = $aosProductCategories->retrieve($aosProductCategories->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
     	
     }
     

tests/tests/modules/AOS_Contracts/AOS_ContractsTest.php

Spacing +8 added lines, -8 removed lines

@@ -2,13 +2,13 @@ 
 
 class AOS_ContractsTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAOS_Contracts(){
+	public function testAOS_Contracts() {
 
 		//execute the contructor and check for the Object type and attributes
 		$aosContracts = new AOS_Contracts();
-		$this->assertInstanceOf('AOS_Contracts',$aosContracts);
-		$this->assertInstanceOf('Basic',$aosContracts);
-		$this->assertInstanceOf('SugarBean',$aosContracts);
+		$this->assertInstanceOf('AOS_Contracts', $aosContracts);
+		$this->assertInstanceOf('Basic', $aosContracts);
+		$this->assertInstanceOf('SugarBean', $aosContracts);
 			
 		$this->assertAttributeEquals('AOS_Contracts', 'module_dir', $aosContracts);
 		$this->assertAttributeEquals('AOS_Contracts', 'object_name', $aosContracts);
@@ -21,7 +21,7 @@ 
 		
 	}
 	
-	public function testsaveAndDelete(){
+	public function testsaveAndDelete() {
 	
 		error_reporting(E_ERROR | E_PARSE);
 		
@@ -39,12 +39,12 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$aosContracts->mark_deleted($aosContracts->id);
 		$result = $aosContracts->retrieve($aosContracts->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 	
 	
-	public function testCreateReminderAndCreateLinkAndDeleteCall(){
+	public function testCreateReminderAndCreateLinkAndDeleteCall() {
 
 		$call = new call();
 		
@@ -73,7 +73,7 @@ 
 		//delete the call and verify that this record cannot be retrieved anymore.		
 		$aosContracts->deleteCall();
 		$call->retrieve($aosContracts->call_id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 	

tests/tests/modules/FP_Event_Locations/FP_Event_LocationsTest.php

Spacing +4 added lines, -4 removed lines

@@ -2,14 +2,14 @@
 
 class FP_Event_LocationsTest extends PHPUnit_Framework_TestCase {
 	
-	public function testFP_Event_Locations(){	
+	public function testFP_Event_Locations() {	
 
 		
 		//execute the contructor and check for the Object type and  attributes
 		$fpEventLoc = new FP_Event_Locations();
-		$this->assertInstanceOf('FP_Event_Locations',$fpEventLoc);
-		$this->assertInstanceOf('Basic',$fpEventLoc);
-		$this->assertInstanceOf('SugarBean',$fpEventLoc);
+		$this->assertInstanceOf('FP_Event_Locations', $fpEventLoc);
+		$this->assertInstanceOf('Basic', $fpEventLoc);
+		$this->assertInstanceOf('SugarBean', $fpEventLoc);
 		
 		$this->assertAttributeEquals('FP_Event_Locations', 'module_dir', $fpEventLoc);
 		$this->assertAttributeEquals('FP_Event_Locations', 'object_name', $fpEventLoc);

tests/tests/modules/Alerts/AlertTest.php

Spacing +5 added lines, -5 removed lines

@@ -4,13 +4,13 @@ 
 class AlertTest extends PHPUnit_Framework_TestCase {
 
 
-    public function testAlert(){
+    public function testAlert() {
 
     	//execute the contructor and check for the Object type and type attribute
     	$alert = new Alert();
-    	$this->assertInstanceOf('Alert',$alert);
-    	$this->assertInstanceOf('Basic',$alert);
-    	$this->assertInstanceOf('SugarBean',$alert);
+    	$this->assertInstanceOf('Alert', $alert);
+    	$this->assertInstanceOf('Basic', $alert);
+    	$this->assertInstanceOf('SugarBean', $alert);
     	
     	$this->assertAttributeEquals('Alerts', 'module_dir', $alert);
     	$this->assertAttributeEquals('Alert', 'object_name', $alert);
@@ -21,7 +21,7 @@ 
     	    	
     }
 
-    public function testbean_implements(){
+    public function testbean_implements() {
 
     	error_reporting(E_ERROR | E_PARSE);
     	

tests/tests/modules/SecurityGroups/SecurityGroupTest.php

Spacing +30 added lines, -30 removed lines

@@ -4,14 +4,14 @@ 
 class SecurityGroupTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testSecurityGroup(){
+	public function testSecurityGroup() {
 
 		//execute the contructor and check for the Object type and  attributes
 		$securityGroup = new SecurityGroup();
 		
-		$this->assertInstanceOf('SecurityGroup',$securityGroup);
-		$this->assertInstanceOf('Basic',$securityGroup);
-		$this->assertInstanceOf('SugarBean',$securityGroup);
+		$this->assertInstanceOf('SecurityGroup', $securityGroup);
+		$this->assertInstanceOf('Basic', $securityGroup);
+		$this->assertInstanceOf('SugarBean', $securityGroup);
 			
 		$this->assertAttributeEquals('securitygroups', 'table_name', $securityGroup);
 		$this->assertAttributeEquals('SecurityGroups', 'module_dir', $securityGroup);
@@ -28,14 +28,14 @@ 
 		
 		//test with securitygroups module
 		$expected = " securitygroups.id in (\n                select secg.id from securitygroups secg\n                inner join securitygroups_users secu on secg.id = secu.securitygroup_id and secu.deleted = 0\n                    and secu.user_id = '1'\n                where secg.deleted = 0\n            )";
-		$actual = $securityGroup->getGroupWhere("securitygroups","SecurityGroups",1);
-		$this->assertSame($expected,$actual);
+		$actual = $securityGroup->getGroupWhere("securitygroups", "SecurityGroups", 1);
+		$this->assertSame($expected, $actual);
 		
 		
 		//test with //test with securitygroups module module
 		$expected = " EXISTS (SELECT  1\n                  FROM    securitygroups secg\n                          INNER JOIN securitygroups_users secu \n                            ON secg.id = secu.securitygroup_id \n                               AND secu.deleted = 0 \n                               AND secu.user_id = '1'\n                          INNER JOIN securitygroups_records secr \n                            ON secg.id = secr.securitygroup_id \n                               AND secr.deleted = 0 \n                               AND secr.module = 'Users'\n                       WHERE   secr.record_id = users.id\n                               AND secg.deleted = 0) ";
-		$actual = $securityGroup->getGroupWhere("users","Users",1);
-		$this->assertSame($expected,$actual);
+		$actual = $securityGroup->getGroupWhere("users", "Users", 1);
+		$this->assertSame($expected, $actual);
 				
 	}
 
@@ -46,7 +46,7 @@ 
 		$expected = " users.id in (\n            select sec.user_id from securitygroups_users sec\n            inner join securitygroups_users secu on sec.securitygroup_id = secu.securitygroup_id and secu.deleted = 0\n                and secu.user_id = '1'\n            where sec.deleted = 0\n        )";
 		$actual = $securityGroup->getGroupUsersWhere(1);
 	
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -57,14 +57,14 @@ 
 		
 		//test with securitygroups module
 		$expected = " LEFT JOIN (select distinct secg.id from securitygroups secg\n    inner join securitygroups_users secu on secg.id = secu.securitygroup_id and secu.deleted = 0\n            and secu.user_id = '1'\n    where secg.deleted = 0\n) securitygroup_join on securitygroup_join.id = securitygroups.id ";
-		$actual = $securityGroup->getGroupJoin("securitygroups","SecurityGroups",1);
-		$this->assertSame($expected,$actual);
+		$actual = $securityGroup->getGroupJoin("securitygroups", "SecurityGroups", 1);
+		$this->assertSame($expected, $actual);
 		
 		
 		//test with //test with securitygroups module
 		$expected = " LEFT JOIN (select distinct secr.record_id as id from securitygroups secg\n    inner join securitygroups_users secu on secg.id = secu.securitygroup_id and secu.deleted = 0\n            and secu.user_id = '1'\n    inner join securitygroups_records secr on secg.id = secr.securitygroup_id and secr.deleted = 0\n             and secr.module = 'Users'\n    where secg.deleted = 0\n) securitygroup_join on securitygroup_join.id = users.id ";
-		$actual = $securityGroup->getGroupJoin("users","Users",1);
-		$this->assertSame($expected,$actual);
+		$actual = $securityGroup->getGroupJoin("users", "Users", 1);
+		$this->assertSame($expected, $actual);
 				
 	}
 
@@ -76,7 +76,7 @@ 
 		
 		$expected = " LEFT JOIN (\n            select distinct sec.user_id as id from securitygroups_users sec\n            inner join securitygroups_users secu on sec.securitygroup_id = secu.securitygroup_id and secu.deleted = 0\n                and secu.user_id = '1'\n            where sec.deleted = 0\n        ) securitygroup_join on securitygroup_join.id = users.id ";
 		$actual = $securityGroup->getGroupUsersJoin(1);
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 
 	}
 
@@ -86,17 +86,17 @@ 
 		
 		//test for listview
 		$result = SecurityGroup::groupHasAccess('', '[SELECT_ID_LIST]');
-		$this->assertEquals(True,$result);
+		$this->assertEquals(True, $result);
 		
 		
 		//test with invalid values
 		$result = SecurityGroup::groupHasAccess('', '');
-		$this->assertEquals(false,$result);
+		$this->assertEquals(false, $result);
 		
 		
 		//test with valid values
 		$result = SecurityGroup::groupHasAccess('Users', '1');
-		$this->assertEquals(false,$result);
+		$this->assertEquals(false, $result);
 				
 	}
 
@@ -114,7 +114,7 @@ 
 			
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			SecurityGroup::inherit($account,false);
+			SecurityGroup::inherit($account, false);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -136,7 +136,7 @@ 
 		
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			SecurityGroup::assign_default_groups($account,false);
+			SecurityGroup::assign_default_groups($account, false);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -157,7 +157,7 @@ 
 		
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			SecurityGroup::inherit_creator($account,false);
+			SecurityGroup::inherit_creator($account, false);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -180,7 +180,7 @@ 
 		
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			SecurityGroup::inherit_assigned($account,false);
+			SecurityGroup::inherit_assigned($account, false);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -202,7 +202,7 @@ 
 		
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			SecurityGroup::inherit_parent($account,false);
+			SecurityGroup::inherit_parent($account, false);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -224,7 +224,7 @@ 
 
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			SecurityGroup::inherit_parentQuery($account, "Accounts", 1, 1 , $account->module_dir);
+			SecurityGroup::inherit_parentQuery($account, "Accounts", 1, 1, $account->module_dir);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -291,7 +291,7 @@ 
 
 		
 		//execute removeDefaultGroup method for each default group			
-		foreach($result as $key=>$value){			
+		foreach ($result as $key=>$value) {			
 			$securityGroup->removeDefaultGroup($key);
 		}
 		
@@ -317,7 +317,7 @@ 
 		
 		$securityGroup = new SecurityGroup();
 			
-		$expected = array (
+		$expected = array(
             'Meetings',
             'Cases',
             'AOS_Products',
@@ -359,7 +359,7 @@ 
         $actualKeys = array_keys($actual);
         sort($expected);
         sort($actualKeys);
-		$this->assertSame($expected,$actualKeys);
+		$this->assertSame($expected, $actualKeys);
 		
 	}
 
@@ -374,12 +374,12 @@ 
 		
 		$securityGroup = new SecurityGroup();
 		
-		$result = $securityGroup->getLinkName("Accounts","Contacts");
-		$this->assertEquals("contacts",$result);
+		$result = $securityGroup->getLinkName("Accounts", "Contacts");
+		$this->assertEquals("contacts", $result);
 		
 		
-		$result = $securityGroup->getLinkName("SecurityGroups","ACLRoles");
-		$this->assertEquals("aclroles",$result);
+		$result = $securityGroup->getLinkName("SecurityGroups", "ACLRoles");
+		$this->assertEquals("aclroles", $result);
 		
 		error_reporting(E_ALL);
 		//error_reporting(E_ERROR | E_PARSE);

tests/tests/modules/Releases/ReleaseTest.php

Spacing +10 added lines, -10 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$release = new Release();
 		
-		$this->assertInstanceOf('Release',$release);
-		$this->assertInstanceOf('SugarBean',$release);
+		$this->assertInstanceOf('Release', $release);
+		$this->assertInstanceOf('SugarBean', $release);
 		 
 		$this->assertAttributeEquals('releases', 'table_name', $release);
 		$this->assertAttributeEquals('Releases', 'module_dir', $release);
@@ -26,11 +26,11 @@ 
 		$release = new Release();
 		
 		//test without setting name
-		$this->assertEquals(Null,$release->get_summary_text());
+		$this->assertEquals(Null, $release->get_summary_text());
 		
 		//test with name set
 		$release->name = "test";
-		$this->assertEquals('test',$release->get_summary_text());
+		$this->assertEquals('test', $release->get_summary_text());
 	}
 
 	public function testget_releases()
@@ -42,7 +42,7 @@ 
 		$this->assertTrue(is_array($result));
 
 		//test with custom params
-		$result = $release->get_releases(true,'Hidden','name is not null');
+		$result = $release->get_releases(true, 'Hidden', 'name is not null');
 		$this->assertTrue(is_array($result));
 		
 	}
@@ -77,14 +77,14 @@ 
 		
 	}
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 
 		$release = new Release();
 		
 		$release->name = "test";
 		$release->status = "Hidden";
 		
-		$expected = array (
+		$expected = array(
 				    'NAME' => 'test',
 				    'STATUS' => 'Hidden',
 				    'ENCODED_NAME' => 'test',
@@ -98,20 +98,20 @@ 
 	}
 
 
-	public function testbuild_generic_where_clause () {
+	public function testbuild_generic_where_clause() {
 
 		$release = new Release();
 
 		//test with empty string params
 		$expected = "name like '%'";
 		$actual = $release->build_generic_where_clause('');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 			
 		//test with valid string params
 		$expected = "name like '%'";
 		$actual = $release->build_generic_where_clause('test');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 	
 	}