adamjakab / SuiteCRM

tests/tests/modules/FP_events/FP_eventsTest.php

Spacing +5 added lines, -5 removed lines

@@ -2,13 +2,13 @@ 
 
 class FP_eventsTest extends PHPUnit_Framework_TestCase {
 	
-	public function testFP_events(){	
+	public function testFP_events() {	
 
 		//execute the contructor and check for the Object type and  attributes
 		$fpEvents = new FP_events();
-		$this->assertInstanceOf('FP_events',$fpEvents);
-		$this->assertInstanceOf('Basic',$fpEvents);
-		$this->assertInstanceOf('SugarBean',$fpEvents);
+		$this->assertInstanceOf('FP_events', $fpEvents);
+		$this->assertInstanceOf('Basic', $fpEvents);
+		$this->assertInstanceOf('SugarBean', $fpEvents);
 		
 		$this->assertAttributeEquals('FP_events', 'module_dir', $fpEvents);
 		$this->assertAttributeEquals('FP_events', 'object_name', $fpEvents);
@@ -19,7 +19,7 @@ 
 		
 	}	
 
-	public function testemail_templates(){
+	public function testemail_templates() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		

tests/tests/modules/AM_ProjectTemplates/AM_ProjectTemplatesTest.php

Spacing +4 added lines, -4 removed lines

@@ -3,13 +3,13 @@
 
 class AM_ProjectTemplatesTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAM_ProjectTemplates(){	
+	public function testAM_ProjectTemplates() {	
 		
 		//execute the contructor and check for the Object type and type attribute
 		$am_projectTemplate = new AM_ProjectTemplates();
-		$this->assertInstanceOf('AM_ProjectTemplates',$am_projectTemplate);
-		$this->assertInstanceOf('Basic',$am_projectTemplate);
-		$this->assertInstanceOf('SugarBean',$am_projectTemplate);
+		$this->assertInstanceOf('AM_ProjectTemplates', $am_projectTemplate);
+		$this->assertInstanceOf('Basic', $am_projectTemplate);
+		$this->assertInstanceOf('SugarBean', $am_projectTemplate);
 
 		
 		$this->assertAttributeEquals('AM_ProjectTemplates', 'module_dir', $am_projectTemplate);

tests/tests/modules/AM_TaskTemplates/AM_TaskTemplatesTest.php

Spacing +4 added lines, -4 removed lines

@@ -2,13 +2,13 @@
 
 class AM_TaskTemplatesTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAM_TaskTemplates(){	
+	public function testAM_TaskTemplates() {	
 
 		//execute the contructor and check for the Object type and type attribute
 		$am_taskTemplates = new AM_TaskTemplates();
-		$this->assertInstanceOf('AM_TaskTemplates',$am_taskTemplates);
-		$this->assertInstanceOf('Basic',$am_taskTemplates);
-		$this->assertInstanceOf('SugarBean',$am_taskTemplates);
+		$this->assertInstanceOf('AM_TaskTemplates', $am_taskTemplates);
+		$this->assertInstanceOf('Basic', $am_taskTemplates);
+		$this->assertInstanceOf('SugarBean', $am_taskTemplates);
 
 		$this->assertAttributeEquals('AM_TaskTemplates', 'module_dir', $am_taskTemplates);
 		$this->assertAttributeEquals('AM_TaskTemplates', 'object_name', $am_taskTemplates);

tests/tests/modules/ACLActions/ACLActionTest.php

Spacing +63 added lines, -63 removed lines

@@ -1,15 +1,15 @@ 
 <?php
 
 
-class ACLActionTest extends PHPUnit_Framework_TestCase{
+class ACLActionTest extends PHPUnit_Framework_TestCase {
 
 	
-	public function testACLAction(){
+	public function testACLAction() {
 		
 		//execute the contructor and check for the Object type and type attribute
 		$aclAction = new ACLAction();
-		$this->assertInstanceOf('ACLAction',$aclAction);
-		$this->assertInstanceOf('SugarBean',$aclAction);
+		$this->assertInstanceOf('ACLAction', $aclAction);
+		$this->assertInstanceOf('SugarBean', $aclAction);
 		
 		$this->assertAttributeEquals('ACLActions', 'module_dir', $aclAction);
 		$this->assertAttributeEquals('ACLAction', 'object_name', $aclAction);				
@@ -20,7 +20,7 @@ 
     }
 
 
-    public function testaddActions(){
+    public function testaddActions() {
 		
     	error_reporting(E_ERROR | E_PARSE);
     	
@@ -33,7 +33,7 @@ 
     }
 
 
-    public function testremoveActions(){
+    public function testremoveActions() {
 
     	//take count of actions initially and then after method execution and test if action count decreases
     	$action_count = count(ACLAction::getDefaultActions());
@@ -44,7 +44,7 @@ 
     }
 
 
-    public function testAccessName(){
+    public function testAccessName() {
 
     	error_reporting(E_ERROR | E_PARSE);
     	
@@ -55,95 +55,95 @@ 
 
 
 
-    public function testgetDefaultActions(){
+    public function testgetDefaultActions() {
         global $beanList;
         $actual = ACLAction::getDefaultActions();
         $this->assertTrue(is_array($actual)); //verify that it returns an array
-        foreach($actual as $acl){
-            $this->assertInstanceOf("ACLAction",$acl);
+        foreach ($actual as $acl) {
+            $this->assertInstanceOf("ACLAction", $acl);
         }
-        $actual = ACLAction::getDefaultActions("module","list");
+        $actual = ACLAction::getDefaultActions("module", "list");
         $this->assertTrue(is_array($actual)); //verify that it returns an array
-        foreach($actual as $acl){
-            $this->assertInstanceOf("ACLAction",$acl);
-            $this->assertEquals("list",$acl->name);
+        foreach ($actual as $acl) {
+            $this->assertInstanceOf("ACLAction", $acl);
+            $this->assertEquals("list", $acl->name);
         }
     }
 
 
-    public function testgetUserActions(){
+    public function testgetUserActions() {
         
     	$result1 = ACLAction::getUserActions('1');
-    	$result2 = ACLAction::getUserActions('1',false,'Accounts');
-    	$result3 = ACLAction::getUserActions('1',false,'Accounts','list');
+    	$result2 = ACLAction::getUserActions('1', false, 'Accounts');
+    	$result3 = ACLAction::getUserActions('1', false, 'Accounts', 'list');
     	
     	//verify that all three results retunred are different
-    	$this->assertNotSame($result1,$result2);
-    	$this->assertNotSame($result1,$result3);
-    	$this->assertNotSame($result2,$result3);
+    	$this->assertNotSame($result1, $result2);
+    	$this->assertNotSame($result1, $result3);
+    	$this->assertNotSame($result2, $result3);
     	
     }
     
 
-	public function testhasAccess(){
+	public function testhasAccess() {
 		
 		$this->assertFalse(ACLAction::hasAccess()); //check with defaults 
-		$this->assertTrue(ACLAction::hasAccess(false,false,90));  //access All with is owner false
-		$this->assertTrue(ACLAction::hasAccess(true,true,90)); //access All with is owner true 
-		$this->assertFalse(ACLAction::hasAccess(false,false,-98));// check access disabled
-		$this->assertFalse(ACLAction::hasAccess(true,true,89)); //check access enabled 
-		$this->assertTrue(ACLAction::hasAccess(true,true,75)); //check owner access with is owner true
-		$this->assertFalse(ACLAction::hasAccess(false,true,75)); //check owner access with is owner false
+		$this->assertTrue(ACLAction::hasAccess(false, false, 90)); //access All with is owner false
+		$this->assertTrue(ACLAction::hasAccess(true, true, 90)); //access All with is owner true 
+		$this->assertFalse(ACLAction::hasAccess(false, false, -98)); // check access disabled
+		$this->assertFalse(ACLAction::hasAccess(true, true, 89)); //check access enabled 
+		$this->assertTrue(ACLAction::hasAccess(true, true, 75)); //check owner access with is owner true
+		$this->assertFalse(ACLAction::hasAccess(false, true, 75)); //check owner access with is owner false
 		
     }
 
-	public function testuserNeedsSecurityGroup(){
+	public function testuserNeedsSecurityGroup() {
 
-		$this->assertFalse( ACLAction::userNeedsSecurityGroup('1','',''));//test with empty module and action 
-		$this->assertFalse( ACLAction::userNeedsSecurityGroup('1','Accounts','list')); //test with valid module and action
+		$this->assertFalse(ACLAction::userNeedsSecurityGroup('1', '', '')); //test with empty module and action 
+		$this->assertFalse(ACLAction::userNeedsSecurityGroup('1', 'Accounts', 'list')); //test with valid module and action
 		
     }
 
 
-	public function testuserHasAccess(){
+	public function testuserHasAccess() {
 
-		$this->assertFalse(ACLAction::userHasAccess('', '','')); //test with empty module and action
-		$this->assertTrue(ACLAction::userHasAccess('', 'Accounts','list')); //test with e,pty user and valid module and action
-		$this->assertTrue(ACLAction::userHasAccess('1', 'Accounts','list')); //test with valid User, module and action
-		$this->assertTrue(ACLAction::userHasAccess('1', 'SecurityGroups','list')); //test with valid User, module and action
-		$this->assertTrue(ACLAction::userHasAccess('1', 'Users','list')); //test with valid User, module and action
+		$this->assertFalse(ACLAction::userHasAccess('', '', '')); //test with empty module and action
+		$this->assertTrue(ACLAction::userHasAccess('', 'Accounts', 'list')); //test with e,pty user and valid module and action
+		$this->assertTrue(ACLAction::userHasAccess('1', 'Accounts', 'list')); //test with valid User, module and action
+		$this->assertTrue(ACLAction::userHasAccess('1', 'SecurityGroups', 'list')); //test with valid User, module and action
+		$this->assertTrue(ACLAction::userHasAccess('1', 'Users', 'list')); //test with valid User, module and action
 		
     }
 
     
-    public function testgetUserAccessLevel(){
+    public function testgetUserAccessLevel() {
     
     	//tes for accoounts module with two different actions
-    	$this->assertEquals(90,ACLAction::getUserAccessLevel('1', 'Accounts','list')); 
-    	$this->assertEquals(89,ACLAction::getUserAccessLevel('1', 'Accounts','access'));
+    	$this->assertEquals(90, ACLAction::getUserAccessLevel('1', 'Accounts', 'list')); 
+    	$this->assertEquals(89, ACLAction::getUserAccessLevel('1', 'Accounts', 'access'));
     	
     	//tes for users module with two different actions
-    	$this->assertEquals(90,ACLAction::getUserAccessLevel('1', 'Users','list'));
-    	$this->assertEquals(89,ACLAction::getUserAccessLevel('1', 'Users','access'));
+    	$this->assertEquals(90, ACLAction::getUserAccessLevel('1', 'Users', 'list'));
+    	$this->assertEquals(89, ACLAction::getUserAccessLevel('1', 'Users', 'access'));
     	
     }
 
     
-    public function testuserNeedsOwnership(){
+    public function testuserNeedsOwnership() {
       	
     	//test with invalid values
-    	$this->assertFalse(ACLAction::userNeedsOwnership('', '',''));
+    	$this->assertFalse(ACLAction::userNeedsOwnership('', '', ''));
     	
     	//test with valid values for different module and action combination
-    	$this->assertFalse(ACLAction::userNeedsOwnership('1', 'Accounts','list'));
-    	$this->assertFalse(ACLAction::userNeedsOwnership('1', 'Accounts','delete'));
-    	$this->assertFalse(ACLAction::userNeedsOwnership('1', 'Users','delete'));
-    	$this->assertFalse(ACLAction::userNeedsOwnership('1', 'Users','list'));
+    	$this->assertFalse(ACLAction::userNeedsOwnership('1', 'Accounts', 'list'));
+    	$this->assertFalse(ACLAction::userNeedsOwnership('1', 'Accounts', 'delete'));
+    	$this->assertFalse(ACLAction::userNeedsOwnership('1', 'Users', 'delete'));
+    	$this->assertFalse(ACLAction::userNeedsOwnership('1', 'Users', 'list'));
     	    	
     }
 
     
-    public function testsetupCategoriesMatrix(){
+    public function testsetupCategoriesMatrix() {
 		
     	//preset required data
     	$categories = Array();
@@ -153,13 +153,13 @@ 
 
     	$names_expected = Array("list"=> "List", "edit"=>"Edit");
     	
-    	$categories_expected = array (
+    	$categories_expected = array(
     			'Accounts' =>
-    			array (
+    			array(
     					'module' =>
-    					array (
-    							'list' => array ( 'list', 'accessColor' => false, 'accessName' => false, 'accessLabel' => false, 'accessOptions' =>  array ( 90 => 'All', 80 => 'Group', 75 => 'Owner', 0 => 'Not Set', -99 => 'None', ), ),
-    							'edit' => array ( 'edit', 'accessColor' => false, 'accessName' => false, 'accessLabel' => false, 'accessOptions' =>  array ( 90 => 'All', 80 => 'Group', 75 => 'Owner', 0 => 'Not Set', -99 => 'None' ), ),
+    					array(
+    							'list' => array('list', 'accessColor' => false, 'accessName' => false, 'accessLabel' => false, 'accessOptions' =>  array(90 => 'All', 80 => 'Group', 75 => 'Owner', 0 => 'Not Set', -99 => 'None',),),
+    							'edit' => array('edit', 'accessColor' => false, 'accessName' => false, 'accessLabel' => false, 'accessOptions' =>  array(90 => 'All', 80 => 'Group', 75 => 'Owner', 0 => 'Not Set', -99 => 'None'),),
     					),
     			),
     	);
@@ -172,37 +172,37 @@ 
     }
 
 
-	public function testtoArray(){
+	public function testtoArray() {
 	
 		$aclAction = new ACLAction();
 		
 		//wihout any fields set 
 		$expected = Array("id"=> NULL, "aclaccess"=> NULL);
 		$actual = $aclAction->toArray();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 		
 		//with fileds pre populated
-		$aclAction->populateFromRow(Array("id"=>"1234","aclaccess"=>"9999"));
-		$expected = Array("id"=>"1234","aclaccess"=>"9999");
+		$aclAction->populateFromRow(Array("id"=>"1234", "aclaccess"=>"9999"));
+		$expected = Array("id"=>"1234", "aclaccess"=>"9999");
 		$actual = $aclAction->toArray();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 			
 	}
 
-	public function testfromArray(){
+	public function testfromArray() {
 		
 		$aclAction = new ACLAction();
-		$arr = Array("id"=>"1234","name"=>"test");
+		$arr = Array("id"=>"1234", "name"=>"test");
 
 		//execute the method and verify that it retunrs expected results
 		$aclAction->fromArray($arr);
-		$this->assertSame($aclAction->id,"1234");
-		$this->assertSame($aclAction->name,"test");
+		$this->assertSame($aclAction->id, "1234");
+		$this->assertSame($aclAction->name, "test");
 	}
 
     
-	public function testclearSessionCache(){
+	public function testclearSessionCache() {
 	 
 		$aclAction = new ACLAction();
 		

tests/tests/modules/AOS_Products_Quotes/AOS_Products_QuotesTest.php

Spacing +16 added lines, -16 removed lines

@@ -2,14 +2,14 @@ 
 
 class AOS_Products_QuotesTest extends PHPUnit_Framework_TestCase {
 
-    public function testAOS_Products_Quotes(){
+    public function testAOS_Products_Quotes() {
 
     	
     	//execute the contructor and check for the Object type and  attributes
     	$aosProductsQuotes = new AOS_Products_Quotes();
-    	$this->assertInstanceOf('AOS_Products_Quotes',$aosProductsQuotes);
-    	$this->assertInstanceOf('Basic',$aosProductsQuotes);
-    	$this->assertInstanceOf('SugarBean',$aosProductsQuotes);
+    	$this->assertInstanceOf('AOS_Products_Quotes', $aosProductsQuotes);
+    	$this->assertInstanceOf('Basic', $aosProductsQuotes);
+    	$this->assertInstanceOf('SugarBean', $aosProductsQuotes);
     		
     	$this->assertAttributeEquals('AOS_Products_Quotes', 'module_dir', $aosProductsQuotes);
     	$this->assertAttributeEquals('AOS_Products_Quotes', 'object_name', $aosProductsQuotes);
@@ -20,18 +20,18 @@ 
     	
     }
 
-    public function testsave_lines(){
+    public function testsave_lines() {
 
     	error_reporting(E_ERROR | E_PARSE);
     	
     	$aosProductsQuotes = new AOS_Products_Quotes();
     	
     	//populate required values
-    	$post_data= Array();
-    	$post_data["name"] = Array('test1','test2');
-    	$post_data["group_number"] = Array('1','2');
-    	$post_data["product_id"] = Array('1','1');
-    	$post_data["product_unit_price"] = Array(100,200);
+    	$post_data = Array();
+    	$post_data["name"] = Array('test1', 'test2');
+    	$post_data["group_number"] = Array('1', '2');
+    	$post_data["product_id"] = Array('1', '1');
+    	$post_data["product_unit_price"] = Array(100, 200);
     	
     	//create parent bean
     	$aosQuote = new AOS_Quotes();
@@ -41,11 +41,11 @@ 
     	
     	//get the linked beans and verify if records created
     	$product_quote_lines = $aosQuote->get_linked_beans('aos_products_quotes', $aosQuote->object_name);
-    	$this->assertEquals(count($post_data["name"]), count($product_quote_lines) );
+    	$this->assertEquals(count($post_data["name"]), count($product_quote_lines));
     	
     }
 
-    public function testmark_lines_deleted(){
+    public function testmark_lines_deleted() {
 
     	$aosProductsQuotes = new AOS_Products_Quotes();
     	
@@ -70,16 +70,16 @@ 
     	$product_quote_lines = $aosQuote->get_linked_beans('aos_products_quotes', $aosQuote->object_name);
     	$actual = count($product_quote_lines);
 		
-    	$this->assertLessThan($expected,$actual);    	
+    	$this->assertLessThan($expected, $actual);    	
     	
     }
 
-    public function testsave(){
+    public function testsave() {
 
     	$aosProductsQuotes = new AOS_Products_Quotes();
 
     	$aosProductsQuotes->name = 'test';
-    	$aosProductsQuotes->product_id= 1;
+    	$aosProductsQuotes->product_id = 1;
     	$aosProductsQuotes->product_unit_price = 100;
     	     	
     	$aosProductsQuotes->save();
@@ -93,7 +93,7 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$aosProductsQuotes->mark_deleted($aosProductsQuotes->id);
     	$result = $aosProductsQuotes->retrieve($aosProductsQuotes->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
     		
     }
     

tests/tests/modules/AOK_KnowledgeBase/AOK_KnowledgeBaseTest.php

Spacing +4 added lines, -4 removed lines

@@ -2,13 +2,13 @@
 
 class AOK_KnowledgeBaseTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAOK_KnowledgeBase(){	
+	public function testAOK_KnowledgeBase() {	
 		
 		//execute the contructor and check for the Object type and type attribute
 		$aok_KnowledgeBase = new AOK_KnowledgeBase();
-		$this->assertInstanceOf('AOK_KnowledgeBase',$aok_KnowledgeBase);
-		$this->assertInstanceOf('Basic',$aok_KnowledgeBase);
-		$this->assertInstanceOf('SugarBean',$aok_KnowledgeBase);
+		$this->assertInstanceOf('AOK_KnowledgeBase', $aok_KnowledgeBase);
+		$this->assertInstanceOf('Basic', $aok_KnowledgeBase);
+		$this->assertInstanceOf('SugarBean', $aok_KnowledgeBase);
 		
 		$this->assertAttributeEquals('AOK_KnowledgeBase', 'module_dir', $aok_KnowledgeBase);
 		$this->assertAttributeEquals('AOK_KnowledgeBase', 'object_name', $aok_KnowledgeBase);

tests/tests/modules/AOS_Line_Item_Groups/AOS_Line_Item_GroupsTest.php

Spacing +15 added lines, -15 removed lines

@@ -2,13 +2,13 @@ 
 
 class AOS_Line_Item_GroupsTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAOS_Line_Item_Groups(){
+	public function testAOS_Line_Item_Groups() {
 
 		//execute the contructor and check for the Object type and  attributes
 		$aosLineItemGroup = new AOS_Line_Item_Groups();
-		$this->assertInstanceOf('AOS_Line_Item_Groups',$aosLineItemGroup);
-		$this->assertInstanceOf('Basic',$aosLineItemGroup);
-		$this->assertInstanceOf('SugarBean',$aosLineItemGroup);
+		$this->assertInstanceOf('AOS_Line_Item_Groups', $aosLineItemGroup);
+		$this->assertInstanceOf('Basic', $aosLineItemGroup);
+		$this->assertInstanceOf('SugarBean', $aosLineItemGroup);
 			
 		$this->assertAttributeEquals('AOS_Line_Item_Groups', 'module_dir', $aosLineItemGroup);
 		$this->assertAttributeEquals('AOS_Line_Item_Groups', 'object_name', $aosLineItemGroup);
@@ -21,18 +21,18 @@ 
 		
 	}
 	
-	public function testsave_groups(){
+	public function testsave_groups() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		
 		$aosLineItemGroup = new AOS_Line_Item_Groups();
 		
 		//populate required values		
-		$post_data= Array();
-		$post_data["group_number"] = Array(0,0);
-		$post_data["name"] = Array('test1','test2');
-		$post_data["total_amount"] = Array('100','200');
-		$post_data["total_amount_usdollar"] = Array('100','200');
+		$post_data = Array();
+		$post_data["group_number"] = Array(0, 0);
+		$post_data["name"] = Array('test1', 'test2');
+		$post_data["total_amount"] = Array('100', '200');
+		$post_data["total_amount_usdollar"] = Array('100', '200');
 		
 		//create parent bean for line item groups
 		$aosContract = new AOS_Contracts();
@@ -43,23 +43,23 @@ 
 		//get the linked beans and verify if records created
 		$line_item_groups = $aosContract->get_linked_beans('aos_line_item_groups', $aosContract->object_name);
 		
-		$this->assertEquals(count($post_data["group_number"]), count($line_item_groups) );
+		$this->assertEquals(count($post_data["group_number"]), count($line_item_groups));
 		
 		//cleanup afterwards
-		foreach($line_item_groups as $lineItem){
+		foreach ($line_item_groups as $lineItem) {
 			$lineItem->mark_deleted($lineItem->id);			
 		}
 			
 		
 	}
 
-    public function testsave(){
+    public function testsave() {
 
 
     	
     	$aosLineItemGroup = new AOS_Line_Item_Groups();
     	$aosLineItemGroup->name = 'test';
-    	$aosLineItemGroup->total_amount= 100;
+    	$aosLineItemGroup->total_amount = 100;
 		$aosLineItemGroup->total_amount_usdollar = 100;
     	
 		$aosLineItemGroup->save();
@@ -73,7 +73,7 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$aosLineItemGroup->mark_deleted($aosLineItemGroup->id);
 		$result = $aosLineItemGroup->retrieve($aosLineItemGroup->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 		
     }

tests/tests/modules/EmailTemplates/EmailTemplateTest.php

Spacing +28 added lines, -28 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$emailTemplate = new EmailTemplate();
 		
-		$this->assertInstanceOf('EmailTemplate',$emailTemplate);
-		$this->assertInstanceOf('SugarBean',$emailTemplate);
+		$this->assertInstanceOf('EmailTemplate', $emailTemplate);
+		$this->assertInstanceOf('SugarBean', $emailTemplate);
 			
 		$this->assertAttributeEquals('EmailTemplates', 'module_dir', $emailTemplate);
 		$this->assertAttributeEquals('EmailTemplate', 'object_name', $emailTemplate);
@@ -31,7 +31,7 @@ 
 		$actual = $emailTemplate->generateFieldDefsJS();
 		//$this->assertSame($expected,$actual);
 
-		$this->assertGreaterThan(0 , strlen($actual));
+		$this->assertGreaterThan(0, strlen($actual));
 		
 	}
 
@@ -40,11 +40,11 @@ 
 		$emailTemplate = new EmailTemplate();
 		
 		//test without setting name
-		$this->assertEquals(Null,$emailTemplate->get_summary_text());
+		$this->assertEquals(Null, $emailTemplate->get_summary_text());
 		
 		//test with name set
 		$emailTemplate->name = "test";
-		$this->assertEquals('test',$emailTemplate->get_summary_text());		
+		$this->assertEquals('test', $emailTemplate->get_summary_text());		
 		
 	}
 
@@ -54,14 +54,14 @@ 
 		
 		//test with empty string params
 		$expected = " SELECT  email_templates.*  , jt0.user_name assigned_user_name , jt0.created_by assigned_user_name_owner  , 'Users' assigned_user_name_mod FROM email_templates   LEFT JOIN  users jt0 ON email_templates.assigned_user_id=jt0.id AND jt0.deleted=0\n\n AND jt0.deleted=0 where email_templates.deleted=0";
-		$actual = $emailTemplate->create_export_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $emailTemplate->create_export_query('', '');
+		$this->assertSame($expected, $actual);
 
 	
 		//test with valid string params
 		$expected = " SELECT  email_templates.*  , jt0.user_name assigned_user_name , jt0.created_by assigned_user_name_owner  , 'Users' assigned_user_name_mod FROM email_templates   LEFT JOIN  users jt0 ON email_templates.assigned_user_id=jt0.id AND jt0.deleted=0\n\n AND jt0.deleted=0 where (email_templates.name=\"\") AND email_templates.deleted=0";
-		$actual = $emailTemplate->create_export_query('email_templates.id','email_templates.name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $emailTemplate->create_export_query('email_templates.id', 'email_templates.name=""');
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -121,13 +121,13 @@ 
 		$emailTemplate = new EmailTemplate();
 		
 		//execute the method and verify that it retunrs expected results
-		$expected = array (
+		$expected = array(
 				'DELETED' => 0,
 				'DATE_MODIFIED' => false,
 		);
 		
 		$actual = $emailTemplate->get_list_view_data();
-		$this->assertSame($expected,$actual);	
+		$this->assertSame($expected, $actual);	
 		
 	}
 
@@ -144,14 +144,14 @@ 
 		$expected = array(
   					"subject"=> "test subject", "body_html"=>"test html", "body"=>"test body text"
 					);
-		$actual = $emailTemplate->parse_email_template(array('subject'=>"test subject",'body_html'=>"test html",'body'=>"test body text"),'Accounts', $account, $macro_nv );
+		$actual = $emailTemplate->parse_email_template(array('subject'=>"test subject", 'body_html'=>"test html", 'body'=>"test body text"), 'Accounts', $account, $macro_nv);
 		$this->assertSame($expected, $actual);
 
 		
 		
 		//test Parse_tracker_urls
-		$tracker_url_template= 'localhost/index.php?entryPoint=campaign_trackerv2&track=%s&identifier=tracker_key';
-		$removeme_url_template= 'localhost/index.php?entryPoint=removeme&identifier=tracker_key';
+		$tracker_url_template = 'localhost/index.php?entryPoint=campaign_trackerv2&track=%s&identifier=tracker_key';
+		$removeme_url_template = 'localhost/index.php?entryPoint=removeme&identifier=tracker_key';
 		$tracker_urls = array();
 		 
 		$result = $emailTemplate->parse_tracker_urls($actual, $tracker_url_template, $tracker_urls, $removeme_url_template);
@@ -164,9 +164,9 @@ 
 
     	$emailTemplate = new EmailTemplate();
     	
-    	$this->assertEquals( 10 ,$emailTemplate->_convertToType('float',10));
-    	$this->assertEquals( "test text" ,$emailTemplate->_convertToType('text',"test text"));
-    	$this->assertEquals( "$10.00" ,$emailTemplate->_convertToType('currency',10));
+    	$this->assertEquals(10, $emailTemplate->_convertToType('float', 10));
+    	$this->assertEquals("test text", $emailTemplate->_convertToType('text', "test text"));
+    	$this->assertEquals("$10.00", $emailTemplate->_convertToType('currency', 10));
     	
     }
 
@@ -179,7 +179,7 @@ 
 		
 		//execute the method and verify that it retunrs expected results
 		
-		$expected = array (
+		$expected = array(
             'contact_user_name' => '',
                 'contact_user_id' => '',
                 'contact_user_user_name' => '',
@@ -267,11 +267,11 @@ 
 		$this->assertEquals("", $actual);
 		
 		//test with valid string
-		$actual = $emailTemplate->parse_template_bean("test", "Users", $user );
+		$actual = $emailTemplate->parse_template_bean("test", "Users", $user);
 		$this->assertEquals("test", $actual);
 		
 		//test with empty string and different module
-		$actual = $emailTemplate->parse_template_bean("", "Accounts", $account );
+		$actual = $emailTemplate->parse_template_bean("", "Accounts", $account);
 		$this->assertEquals("", $actual);
 		
 	}
@@ -285,12 +285,12 @@ 
 		
 		//test with empty string
 		$result = $emailTemplate->parse_template("", $bean_arr);
-		$this->assertEquals("",$result);
+		$this->assertEquals("", $result);
 		
 		
 		//test with valid string
 		$result = $emailTemplate->parse_template("some value", $bean_arr);
-		$this->assertEquals("some value",$result);
+		$this->assertEquals("some value", $result);
 		
 	}
 
@@ -307,13 +307,13 @@ 
     public function testgetTypeOptionsForSearch()
 	{
 		//execute the method and verify that it retunrs expected results
-		$expected = array (
+		$expected = array(
 				'' => '',
 				'campaign' => 'Campaign',
 				'email' => 'Email',
 		);
 		$actual = EmailTemplate::getTypeOptionsForSearch();
-		$this->assertSame($expected,$actual);			
+		$this->assertSame($expected, $actual);			
 
     }
 
@@ -322,12 +322,12 @@ 
 		$emailTemplate = new EmailTemplate();
 		
 		//test without id attribute
-		$this->assertEquals(false,$emailTemplate->is_used_by_email_marketing());
+		$this->assertEquals(false, $emailTemplate->is_used_by_email_marketing());
 		
 		
 		//test with id attribute
 		$emailTemplate->id = 1;
-		$this->assertEquals(false,$emailTemplate->is_used_by_email_marketing());
+		$this->assertEquals(false, $emailTemplate->is_used_by_email_marketing());
 	}
 
     public function testcleanBean()
@@ -337,13 +337,13 @@ 
     	
     	//test without body_html attribute
     	$emailTemplate->cleanBean();
-    	$this->assertEquals("",$emailTemplate->body_html);
+    	$this->assertEquals("", $emailTemplate->body_html);
     	
     	
     	//test with body_html attribute
     	$emailTemplate->body_html = "<h1>text</h1>";
     	$emailTemplate->cleanBean();
-    	$this->assertEquals("&lt;h1&gt;text&lt;/h1&gt;",$emailTemplate->body_html);
+    	$this->assertEquals("&lt;h1&gt;text&lt;/h1&gt;", $emailTemplate->body_html);
     
     }
 

tests/tests/modules/AOW_Actions/AOW_ActionTest.php

Spacing +12 added lines, -12 removed lines

@@ -3,13 +3,13 @@ 
 
 class AOW_ActionTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAOW_Action(){	
+	public function testAOW_Action() {	
 
 		//execute the contructor and check for the Object type and  attributes
 		$aowAction = new AOW_Action();
-		$this->assertInstanceOf('AOW_Action',$aowAction);
-		$this->assertInstanceOf('Basic',$aowAction);
-		$this->assertInstanceOf('SugarBean',$aowAction);
+		$this->assertInstanceOf('AOW_Action', $aowAction);
+		$this->assertInstanceOf('Basic', $aowAction);
+		$this->assertInstanceOf('SugarBean', $aowAction);
 			
 		$this->assertAttributeEquals('AOW_Actions', 'module_dir', $aowAction);
 		$this->assertAttributeEquals('AOW_Action', 'object_name', $aowAction);
@@ -21,17 +21,17 @@ 
 		
 	}
 
-    public function testsave_lines(){
+    public function testsave_lines() {
 
     	error_reporting(E_ERROR | E_PARSE);
     	 
     	$aowAction = new AOW_Action();
     	 
     	//populate required values
-    	$post_data= Array();
-    	$post_data["name"] = Array('test1','test2');
-    	$post_data["action"] = Array('action1','action2');
-    	$post_data["param"] = Array(array('param1'=>'value'),array('value'=>array('param2'=>'value')));
+    	$post_data = Array();
+    	$post_data["name"] = Array('test1', 'test2');
+    	$post_data["action"] = Array('action1', 'action2');
+    	$post_data["param"] = Array(array('param1'=>'value'), array('value'=>array('param2'=>'value')));
 
     	 
     	//create parent bean
@@ -44,17 +44,17 @@ 
     	
     	//get the linked beans and verify if records created
     	$aow_actions = $aowWorkFlow->get_linked_beans('aow_actions', $aowWorkFlow->object_name);
-    	$this->assertEquals(count($post_data["action"]), count($aow_actions) );
+    	$this->assertEquals(count($post_data["action"]), count($aow_actions));
     	
     	
     	//cleanup afterwards
-    	foreach($aow_actions as $lineItem){
+    	foreach ($aow_actions as $lineItem) {
     		$lineItem->mark_deleted($lineItem->id);
     	}
 
     }
 	
-	public function testbean_implements(){
+	public function testbean_implements() {
 
 		error_reporting(E_ERROR | E_PARSE);