adamjakab / SuiteCRM

tests/tests/modules/Opportunities/OpportunityTest.php

Spacing +43 added lines, -43 removed lines

@@ -9,8 +9,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$opportunity = new Opportunity();
 			
-		$this->assertInstanceOf('Opportunity',$opportunity);
-		$this->assertInstanceOf('SugarBean',$opportunity);
+		$this->assertInstanceOf('Opportunity', $opportunity);
+		$this->assertInstanceOf('SugarBean', $opportunity);
 		
 
 		$this->assertAttributeEquals('opportunities', 'table_name', $opportunity);
@@ -25,57 +25,57 @@ 
 	}
 
 
-	public function testget_summary_text()	{
+	public function testget_summary_text() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		
 		$opportunity = new Opportunity();
 		
 		//test without setting name
-		$this->assertEquals(Null,$opportunity->get_summary_text());
+		$this->assertEquals(Null, $opportunity->get_summary_text());
 		
 		//test with name set
 		$opportunity->name = "test";
-		$this->assertEquals('test',$opportunity->get_summary_text());
+		$this->assertEquals('test', $opportunity->get_summary_text());
 		
 	}
 
-	public function testcreate_list_query(){
+	public function testcreate_list_query() {
 
 		$opportunity = new Opportunity();
 		
 		//test with empty string params
 		$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY opportunities.name";
-		$actual = $opportunity->create_list_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $opportunity->create_list_query('', '');
+		$this->assertSame($expected, $actual);
 		
 			
 		//test with valid string params
 		$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where (accounts.name=\"\") AND \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY accounts.id";
-		$actual = $opportunity->create_list_query('accounts.id','accounts.name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $opportunity->create_list_query('accounts.id', 'accounts.name=""');
+		$this->assertSame($expected, $actual);
 			
 	}
 	
 
-    public function testcreate_export_query(){
+    public function testcreate_export_query() {
 
     	$opportunity = new Opportunity();
     	
     	//test with empty string params
     	$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY opportunities.name";
-    	$actual = $opportunity->create_list_query('','');
-    	$this->assertSame($expected,$actual);
+    	$actual = $opportunity->create_list_query('', '');
+    	$this->assertSame($expected, $actual);
 
     	
     	//test with valid string params
     	$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where (accounts.name=\"\") AND \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY accounts.id";
-    	$actual = $opportunity->create_list_query('accounts.id','accounts.name=""');
-    	$this->assertSame($expected,$actual);
+    	$actual = $opportunity->create_list_query('accounts.id', 'accounts.name=""');
+    	$this->assertSame($expected, $actual);
     	
     }
 
-	public function testfill_in_additional_list_fields(){
+	public function testfill_in_additional_list_fields() {
 
 		$opportunity = new Opportunity();
 		
@@ -98,7 +98,7 @@ 
 		
 	}
 
-	public function testfill_in_additional_detail_fields(){
+	public function testfill_in_additional_detail_fields() {
 
 		$opportunity = new Opportunity();
 		
@@ -115,7 +115,7 @@ 
 	}
 
 
-	public function testget_contacts(){
+	public function testget_contacts() {
 
 		$opportunity = new Opportunity();
 		
@@ -124,14 +124,14 @@ 
 		
 	}
 
-	public function testupdate_currency_id(){
+	public function testupdate_currency_id() {
 
 		$opportunity = new Opportunity();
 
 		//execute the method and test if it works and does not throws an exception.
 		try {
 		
-			$opportunity->update_currency_id(array('GBP','EUR'), 'USD');
+			$opportunity->update_currency_id(array('GBP', 'EUR'), 'USD');
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -141,13 +141,13 @@ 
 		
 	}
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 
 		$opportunity = new Opportunity();
 		
-		$opportunity->name  = "test";
+		$opportunity->name = "test";
 				
-		$expected = array (
+		$expected = array(
 				'NAME' => 'test',
 				'DELETED' => 0,
 				'SALES_STAGE' => '',
@@ -164,7 +164,7 @@ 
 		
 	}
 
-    public function testget_currency_symbol(){
+    public function testget_currency_symbol() {
 
     	$opportunity = new Opportunity();
     	
@@ -179,23 +179,23 @@ 
     }
 
 
-	public function testbuild_generic_where_clause () {
+	public function testbuild_generic_where_clause() {
 
 		$opportunity = new Opportunity();
 
 		//test with empty string params
 		$expected = "opportunities.name like '%' or accounts.name like '%'";
 		$actual = $opportunity->build_generic_where_clause('');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
-	public function testsave(){
+	public function testsave() {
  
 		$opportunity = new Opportunity();
 		
 		$opportunity->name = "test";
-		$opportunity->description ="test description";
+		$opportunity->description = "test description";
 		$opportunity->sales_stage = "Value Proposition";
 		
 		$result = $opportunity->save();
@@ -210,11 +210,11 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$opportunity->mark_deleted($opportunity->id);
 		$result = $opportunity->retrieve($opportunity->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 
-	public function testsave_relationship_changes(){
+	public function testsave_relationship_changes() {
 
 		$opportunity = new Opportunity();
 		$opportunity->account_id = 1;
@@ -229,7 +229,7 @@ 
 		
 	}
 
-	public function testset_opportunity_contact_relationship(){
+	public function testset_opportunity_contact_relationship() {
 
 		$opportunity = new Opportunity();
 		
@@ -243,7 +243,7 @@ 
 		
 	}
 
-	public function testset_notification_body(){
+	public function testset_notification_body() {
 
 
 		$opportunity = new Opportunity();
@@ -251,21 +251,21 @@ 
 		//test with attributes preset and verify template variables are set accordingly
 		
 		$opportunity->name = "test";
-		$opportunity->amount ="100";
+		$opportunity->amount = "100";
 		$opportunity->date_closed = "2015-02-11 17:30:00";
 		$opportunity->description = "tes description";
 		$opportunity->sales_stage = "Value Proposition";
 		
 		$result = $opportunity->set_notification_body(new Sugar_Smarty(), $opportunity);
 		
-		$this->assertEquals($opportunity->name ,$result->_tpl_vars['OPPORTUNITY_NAME']);
-		$this->assertEquals($opportunity->amount  ,$result->_tpl_vars['OPPORTUNITY_AMOUNT']);
-		$this->assertEquals($opportunity->date_closed ,$result->_tpl_vars['OPPORTUNITY_CLOSEDATE']);
-		$this->assertEquals($opportunity->sales_stage ,$result->_tpl_vars['OPPORTUNITY_STAGE']);
-		$this->assertEquals($opportunity->description ,$result->_tpl_vars['OPPORTUNITY_DESCRIPTION']);
+		$this->assertEquals($opportunity->name, $result->_tpl_vars['OPPORTUNITY_NAME']);
+		$this->assertEquals($opportunity->amount, $result->_tpl_vars['OPPORTUNITY_AMOUNT']);
+		$this->assertEquals($opportunity->date_closed, $result->_tpl_vars['OPPORTUNITY_CLOSEDATE']);
+		$this->assertEquals($opportunity->sales_stage, $result->_tpl_vars['OPPORTUNITY_STAGE']);
+		$this->assertEquals($opportunity->description, $result->_tpl_vars['OPPORTUNITY_DESCRIPTION']);
 	}
 
-	public function testbean_implements(){
+	public function testbean_implements() {
 
 		$opportunity = new Opportunity();
 		
@@ -275,13 +275,13 @@ 
 		
 	}
 
-	public function testlistviewACLHelper(){
+	public function testlistviewACLHelper() {
 
 		$opportunity = new Opportunity();
 		
-		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a" );
+		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a");
 		$actual = $opportunity->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -295,7 +295,7 @@ 
 		
 	}
 	
-	public function testgetCurrencyType(){
+	public function testgetCurrencyType() {
 	
 		//execute the method and test if it works and does not throws an exception.
 		try {

tests/tests/modules/CampaignTrackers/CampaignTrackerTest.php

Spacing +6 added lines, -6 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 
-class CampaignTrackerTest extends PHPUnit_Framework_TestCase  {
+class CampaignTrackerTest extends PHPUnit_Framework_TestCase {
 
 
     public function testCampaignTracker() 
@@ -10,8 +10,8 @@ 
     	
     	//execute the contructor and check for the Object type and  attributes
     	$campaignTracker = new CampaignTracker();
-    	$this->assertInstanceOf('CampaignTracker',$campaignTracker);
-    	$this->assertInstanceOf('SugarBean',$campaignTracker);
+    	$this->assertInstanceOf('CampaignTracker', $campaignTracker);
+    	$this->assertInstanceOf('SugarBean', $campaignTracker);
     	
     	$this->assertAttributeEquals('CampaignTrackers', 'module_dir', $campaignTracker);
     	$this->assertAttributeEquals('CampaignTracker', 'object_name', $campaignTracker);
@@ -38,7 +38,7 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$campaignTracker->mark_deleted($campaignTracker->id);
     	$result = $campaignTracker->retrieve($campaignTracker->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
     	
     }
 
@@ -49,11 +49,11 @@ 
     	$campaignTracker = new CampaignTracker();
     	
     	//test without setting name
-    	$this->assertEquals(Null,$campaignTracker->get_summary_text());
+    	$this->assertEquals(Null, $campaignTracker->get_summary_text());
 
     	//test with name set
     	$campaignTracker->tracker_name = "test";
-    	$this->assertEquals('test',$campaignTracker->get_summary_text());
+    	$this->assertEquals('test', $campaignTracker->get_summary_text());
     	 
     }
 

tests/tests/modules/SavedSearch/SavedSearchTest.php

Spacing +13 added lines, -13 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$savedSearch = new SavedSearch();
 		
-		$this->assertInstanceOf('SavedSearch',$savedSearch);
-		$this->assertInstanceOf('SugarBean',$savedSearch);
+		$this->assertInstanceOf('SavedSearch', $savedSearch);
+		$this->assertInstanceOf('SugarBean', $savedSearch);
 			
 		$this->assertAttributeEquals('saved_search', 'table_name', $savedSearch);
 		$this->assertAttributeEquals('SavedSearch', 'module_dir', $savedSearch);
@@ -17,9 +17,9 @@ 
 		
 		
 		//test with parameters
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+		$savedSearch = new SavedSearch(array('id', 'name'), 'id', 'ASC');
 		
-		$this->assertAttributeEquals(array('id','name'), 'columns', $savedSearch);
+		$this->assertAttributeEquals(array('id', 'name'), 'columns', $savedSearch);
 		$this->assertAttributeEquals('id', 'orderBy', $savedSearch);
 		$this->assertAttributeEquals('ASC', 'sortOrder', $savedSearch);
 		
@@ -30,19 +30,19 @@ 
 
 		error_reporting(E_ERROR | E_PARSE);
 		
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+		$savedSearch = new SavedSearch(array('id', 'name'), 'id', 'ASC');
 		$result = $savedSearch->getForm("Leads");
 		
-		$this->assertGreaterThan(0,strlen($result));
+		$this->assertGreaterThan(0, strlen($result));
 		
 	}
 
 	public function testgetSelect() {
 
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+		$savedSearch = new SavedSearch(array('id', 'name'), 'id', 'ASC');
 		$result = $savedSearch->getSelect("Leads");
 				
-		$this->assertGreaterThan(0,strlen($result));
+		$this->assertGreaterThan(0, strlen($result));
 		
 	}
 
@@ -85,7 +85,7 @@ 
 		$_REQUEST['description'] = 'test description';
 		$_REQUEST['test_content'] = 'test text';
 		
-		$expected = array( "search_module"=> "Leads", "description"=>"test description", "test_content"=>"test text", "advanced"=>true);
+		$expected = array("search_module"=> "Leads", "description"=>"test description", "test_content"=>"test text", "advanced"=>true);
 		
 		//execute the method and then retrieve back to verify contents attribute
 		$savedSearch->handleSave('', false, false, $id, $searchModuleBean);
@@ -101,7 +101,7 @@ 
 		$savedSearch->handleDelete($id);
 
 		$result = $savedSearch->retrieve($id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 	
@@ -163,11 +163,11 @@ 
 	}
 
 
-	public function testpopulateRequest(){
+	public function testpopulateRequest() {
 
 		$savedSearch = new SavedSearch();
 		
-		$savedSearch->contents = array( "search_module"=> "Accounts", 
+		$savedSearch->contents = array("search_module"=> "Accounts", 
 										"description"=>"test text", 
 										"test_content"=>"some content", 
 										"advanced"=>true);
@@ -176,7 +176,7 @@ 
 
 		//verify thhat Request parameters are set
 		$this->assertEquals('Accounts', $_REQUEST['search_module']); 
-		$this->assertEquals('test text',  $_REQUEST['description']); 
+		$this->assertEquals('test text', $_REQUEST['description']); 
 		$this->assertEquals('some content', $_REQUEST['test_content']); 
 		
 		

tests/tests/modules/EmailText/EmailTextTest.php

Spacing +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 		//execute the contructor and check for the Object type and  attributes
 		$emailText = new EmailText();
 	
-		$this->assertInstanceOf('EmailText',$emailText);
-		$this->assertInstanceOf('SugarBean',$emailText);
+		$this->assertInstanceOf('EmailText', $emailText);
+		$this->assertInstanceOf('SugarBean', $emailText);
 			
 		$this->assertAttributeEquals('EmailText', 'module_dir', $emailText);
 		$this->assertAttributeEquals('EmailText', 'module_name', $emailText);

tests/tests/modules/AOW_WorkFlow/AOW_WorkFlowTest.php

Spacing +46 added lines, -46 removed lines

@@ -3,13 +3,13 @@ 
 class AOW_WorkFlowTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testAOW_WorkFlow(){
+	public function testAOW_WorkFlow() {
 
 		//execute the contructor and check for the Object type and  attributes
 		$aowWorkFlow = new AOW_WorkFlow();
-		$this->assertInstanceOf('AOW_WorkFlow',$aowWorkFlow);
-		$this->assertInstanceOf('Basic',$aowWorkFlow);
-		$this->assertInstanceOf('SugarBean',$aowWorkFlow);
+		$this->assertInstanceOf('AOW_WorkFlow', $aowWorkFlow);
+		$this->assertInstanceOf('Basic', $aowWorkFlow);
+		$this->assertInstanceOf('SugarBean', $aowWorkFlow);
 
 		$this->assertAttributeEquals('AOW_WorkFlow', 'module_dir', $aowWorkFlow);
 		$this->assertAttributeEquals('AOW_WorkFlow', 'object_name', $aowWorkFlow);
@@ -20,7 +20,7 @@ 
 
 	}
 
-	public function testbean_implements(){
+	public function testbean_implements() {
 
 		error_reporting(E_ERROR | E_PARSE);
 
@@ -31,7 +31,7 @@ 
 
 	}
 
-    public function testsave(){
+    public function testsave() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -48,11 +48,11 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$aowWorkFlow->mark_deleted($aowWorkFlow->id);
     	$result = $aowWorkFlow->retrieve($aowWorkFlow->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
 
     }
 
-    public function testload_flow_beans(){
+    public function testload_flow_beans() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -68,7 +68,7 @@ 
     }
 
 
-	public function testrun_flows(){
+	public function testrun_flows() {
 
 		$aowWorkFlow = new AOW_WorkFlow();
 
@@ -78,7 +78,7 @@ 
 	}
 
 
-    public function testrun_flow(){
+    public function testrun_flow() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -94,7 +94,7 @@ 
     }
 
 
-    public function testrun_bean_flows(){
+    public function testrun_bean_flows() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -109,17 +109,17 @@ 
     }
 
 
-    public function testget_flow_beans(){
+    public function testget_flow_beans() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
     	//test for AOS_Quotes. it will return null as no test data is available
     	$aowWorkFlow->flow_module = "AOS_Quotes";
     	$result = $aowWorkFlow->get_flow_beans();
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
     }
 
-    public function testbuild_flow_query_join(){
+    public function testbuild_flow_query_join() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
     	$query = Array();
@@ -131,56 +131,56 @@ 
 
 
 		//test with type custom
-    	$expected = Array('join' => array('c' => 'LEFT JOIN calls_cstm c ON calls.id = c.id_c ' ));
+    	$expected = Array('join' => array('c' => 'LEFT JOIN calls_cstm c ON calls.id = c.id_c '));
     	$result = $aowWorkFlow->build_flow_query_join('c', new Call(), 'custom', Array());
     	$this->assertSame($expected, $result);
 
 
     	//test with type relationship
-    	$expected = Array (
-    			'join' =>array ("aos_products_quotes" => "LEFT JOIN aos_products_quotes aos_products_quotes ON aos_quotes.id=aos_products_quotes.parent_id AND aos_products_quotes.deleted=0\n\n"),
-    			'select' =>array ("aos_products_quotes.id AS 'aos_products_quotes_id'"),
+    	$expected = Array(
+    			'join' =>array("aos_products_quotes" => "LEFT JOIN aos_products_quotes aos_products_quotes ON aos_quotes.id=aos_products_quotes.parent_id AND aos_products_quotes.deleted=0\n\n"),
+    			'select' =>array("aos_products_quotes.id AS 'aos_products_quotes_id'"),
     	);
     	$result = $aowWorkFlow->build_flow_query_join('aos_products_quotes', new AOS_Quotes(), 'relationship', Array());
     	$this->assertSame($expected, $result);
 
     }
 
-    public function testbuild_flow_query_where(){
+    public function testbuild_flow_query_where() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
     	//test without presetting required object attributes
     	$expected = Array();
     	$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+		$this->assertSame($expected, $query);
 
 
 
 		//test with module required attributes set
 		$aowWorkFlow->id = '1';
 		$aowWorkFlow->flow_module = "Calls";
-		$expected = array (
-				'where' => array ('NOT EXISTS (SELECT * FROM aow_processed WHERE aow_processed.aow_workflow_id=\'1\' AND aow_processed.parent_id=calls.id AND aow_processed.status = \'Complete\' AND aow_processed.deleted = 0)',
-				'calls.deleted = 0 ' )
+		$expected = array(
+				'where' => array('NOT EXISTS (SELECT * FROM aow_processed WHERE aow_processed.aow_workflow_id=\'1\' AND aow_processed.parent_id=calls.id AND aow_processed.status = \'Complete\' AND aow_processed.deleted = 0)',
+				'calls.deleted = 0 ')
 				);
 		$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+		$this->assertSame($expected, $query);
 
 
 
 		//test with flow_run_on and multiple_runs attributes set
-		$expected = array (
-						'where' =>	array ('calls.date_entered > \'\'','calls.deleted = 0 '),
+		$expected = array(
+						'where' =>	array('calls.date_entered > \'\'', 'calls.deleted = 0 '),
 					);
 		$aowWorkFlow->flow_run_on = 'New_Records';
 		$aowWorkFlow->multiple_runs = 1;
 		$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+		$this->assertSame($expected, $query);
 
     }
 
-    public function testbuild_query_where(){
+    public function testbuild_query_where() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -256,7 +256,7 @@ 
 
     }
 
-    public function testcheck_valid_bean(){
+    public function testcheck_valid_bean() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
     	$aowWorkFlow->flow_run_on = "New_Records";
@@ -268,28 +268,28 @@ 
 
     }
 
-    public function testcompare_condition(){
+    public function testcompare_condition() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
     	//execute the method and verify that it returns valid values for all operators
 
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,1));
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,2,'Not_Equal_To' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(2,1,'Greater_Than' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,2,'Less_Than' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(5,4,'Greater_Than_or_Equal_To' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(2,3,'Less_Than_or_Equal_To' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','test','Contains' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','test','Starts_With' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','1','Ends_With' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('','','is_null' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('test2',array('test1','test2'),'One_of' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('test', array('test1','test2'),'Not_One_of' ));
+    	$this->assertTrue($aowWorkFlow->compare_condition(1, 1));
+    	$this->assertTrue($aowWorkFlow->compare_condition(1, 2, 'Not_Equal_To'));
+    	$this->assertTrue($aowWorkFlow->compare_condition(2, 1, 'Greater_Than'));
+    	$this->assertTrue($aowWorkFlow->compare_condition(1, 2, 'Less_Than'));
+    	$this->assertTrue($aowWorkFlow->compare_condition(5, 4, 'Greater_Than_or_Equal_To'));
+    	$this->assertTrue($aowWorkFlow->compare_condition(2, 3, 'Less_Than_or_Equal_To'));
+    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1', 'test', 'Contains'));
+    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1', 'test', 'Starts_With'));
+    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1', '1', 'Ends_With'));
+    	$this->assertTrue($aowWorkFlow->compare_condition('', '', 'is_null'));
+    	$this->assertTrue($aowWorkFlow->compare_condition('test2', array('test1', 'test2'), 'One_of'));
+    	$this->assertTrue($aowWorkFlow->compare_condition('test', array('test1', 'test2'), 'Not_One_of'));
 
     }
 
-    public function testcheck_in_group(){
+    public function testcheck_in_group() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -300,7 +300,7 @@ 
     }
 
 
-    public function testrun_actions(){
+    public function testrun_actions() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -316,7 +316,7 @@ 
 
     	//test for a entry in AOW_Processed table.
     	$processed = new AOW_Processed();
-    	$processed->retrieve_by_string_fields(array('aow_workflow_id' => 1,'parent_id' => 1));
+    	$processed->retrieve_by_string_fields(array('aow_workflow_id' => 1, 'parent_id' => 1));
 
     	//test for record ID to verify that record is saved
     	$this->assertTrue(isset($processed->id));
@@ -326,7 +326,7 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$processed->mark_deleted($processed->id);
     	$result = $processed->retrieve($processed->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
 
     }
 

tests/tests/modules/AOS_PDF_Templates/AOS_PDF_TemplatesTest.php

Spacing +4 added lines, -4 removed lines

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

tests/tests/modules/Cases/CaseTest.php

Spacing +32 added lines, -32 removed lines

@@ -7,9 +7,9 @@ 
 
 		//execute the contructor and check for the Object type and  attributes
 		$aCase = new aCase();
-		$this->assertInstanceOf('aCase',$aCase);
-		$this->assertInstanceOf('Basic',$aCase);
-		$this->assertInstanceOf('SugarBean',$aCase);
+		$this->assertInstanceOf('aCase', $aCase);
+		$this->assertInstanceOf('Basic', $aCase);
+		$this->assertInstanceOf('SugarBean', $aCase);
 			
 		$this->assertAttributeEquals('Cases', 'module_dir', $aCase);
 		$this->assertAttributeEquals('Case', 'object_name', $aCase);
@@ -26,19 +26,19 @@ 
 		error_reporting(E_ERROR | E_PARSE);
 		
 		$aCase = new aCase();
-		$this->assertEquals(Null,$aCase->get_summary_text());
+		$this->assertEquals(Null, $aCase->get_summary_text());
 		
 		$aCase->name = "test";
-		$this->assertEquals('test',$aCase->get_summary_text());
+		$this->assertEquals('test', $aCase->get_summary_text());
 
 	}
 
-	public function testlistviewACLHelper(){
+	public function testlistviewACLHelper() {
 
 		$aCase = new aCase();
-		$expected = array( "MAIN"=>"a", "ACCOUNT"=>"a");
+		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a");
 		$actual = $aCase->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 	
 	}
 
@@ -118,7 +118,7 @@ 
 		$aCase = new aCase();
 		
 		//test without setting attributes
-		$expected = array (
+		$expected = array(
 				'DELETED' => 0,
 				'STATE' => 'Open',
 				'UPDATE_TEXT' => '',
@@ -136,11 +136,11 @@ 
 		
 		$actual = $aCase->get_list_view_data();
 		//$this->assertSame($expected ,$actual);
-		$this->assertEquals($expected['STATE'] ,$actual['STATE']);
-		$this->assertEquals($expected['UPDATE_TEXT'] ,$actual['UPDATE_TEXT']);
-		$this->assertEquals($expected['NAME'] ,$actual['NAME']);
-		$this->assertEquals($expected['PRIORITY'] ,$actual['PRIORITY']);
-		$this->assertRegExp($expected['SET_COMPLETE'] ,$actual['SET_COMPLETE']);
+		$this->assertEquals($expected['STATE'], $actual['STATE']);
+		$this->assertEquals($expected['UPDATE_TEXT'], $actual['UPDATE_TEXT']);
+		$this->assertEquals($expected['NAME'], $actual['NAME']);
+		$this->assertEquals($expected['PRIORITY'], $actual['PRIORITY']);
+		$this->assertRegExp($expected['SET_COMPLETE'], $actual['SET_COMPLETE']);
 		
 		
 		
@@ -150,7 +150,7 @@ 
 		$aCase->status = "Open_New";
 		$aCase->case_number = 1;
 		
-		$expected = array (
+		$expected = array(
 				'NAME' => 'test',
 				'DELETED' => 0,
 				'CASE_NUMBER' => 1,
@@ -164,30 +164,30 @@ 
 		
 		$actual = $aCase->get_list_view_data();
 		//$this->assertSame($expected ,$actual);
-		$this->assertEquals($expected['NAME'] ,$actual['NAME']);
-		$this->assertEquals($expected['CASE_NUMBER'] ,$actual['CASE_NUMBER']);
-		$this->assertEquals($expected['STATUS'] ,$actual['STATUS']);
-		$this->assertEquals($expected['PRIORITY'] ,$actual['PRIORITY']);
-		$this->assertEquals($expected['STATE'] ,$actual['STATE']);
+		$this->assertEquals($expected['NAME'], $actual['NAME']);
+		$this->assertEquals($expected['CASE_NUMBER'], $actual['CASE_NUMBER']);
+		$this->assertEquals($expected['STATUS'], $actual['STATUS']);
+		$this->assertEquals($expected['PRIORITY'], $actual['PRIORITY']);
+		$this->assertEquals($expected['STATE'], $actual['STATE']);
 		
 		
 	}
 
 
-	public function testbuild_generic_where_clause () 
+	public function testbuild_generic_where_clause() 
 	{
 		$aCase = new aCase();
 		
 		//test with string
 		$expected = "(cases.name like 'test%' or accounts.name like 'test%')";
 		$actual = $aCase->build_generic_where_clause('test');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 		
 		//test with number
 		$expected = "(cases.name like '1%' or accounts.name like '1%' or cases.case_number like '1%')";
 		$actual = $aCase->build_generic_where_clause(1);
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -202,14 +202,14 @@ 
 		
 		$result = $aCase->set_notification_body(new Sugar_Smarty(), $aCase);
 		
-		$this->assertEquals($aCase->name ,$result->_tpl_vars['CASE_SUBJECT']);
-		$this->assertEquals("High" ,$result->_tpl_vars['CASE_PRIORITY']);
-		$this->assertEquals("New" ,$result->_tpl_vars['CASE_STATUS']);
-		$this->assertEquals($aCase->description ,$result->_tpl_vars['CASE_DESCRIPTION']);
+		$this->assertEquals($aCase->name, $result->_tpl_vars['CASE_SUBJECT']);
+		$this->assertEquals("High", $result->_tpl_vars['CASE_PRIORITY']);
+		$this->assertEquals("New", $result->_tpl_vars['CASE_STATUS']);
+		$this->assertEquals($aCase->description, $result->_tpl_vars['CASE_DESCRIPTION']);
 		
 	}
 
-	public function testbean_implements(){
+	public function testbean_implements() {
 	
 		$aCase = new aCase();
 		$this->assertEquals(false, $aCase->bean_implements('')); //test with blank value
@@ -218,7 +218,7 @@ 
 		
 	}
 
-	public function testsave(){
+	public function testsave() {
 
 		$aCase = new aCase();
 		$aCase->name = "test";
@@ -234,7 +234,7 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$aCase->mark_deleted($aCase->id);
 		$result = $aCase->retrieve($aCase->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		 
 	}
 
@@ -243,11 +243,11 @@ 
 		
 		$aCase = new aCase();
 		$result = $aCase->getEmailSubjectMacro();
-		$this->assertEquals("[CASE:%1]",$result);
+		$this->assertEquals("[CASE:%1]", $result);
 		
 	}
 
-	public function testgetAccount(){
+	public function testgetAccount() {
 
 		$aCase = new aCase();
 		$result = $aCase->getAccount(1);

tests/tests/modules/Groups/GroupTest.php

Spacing +7 added lines, -7 removed lines

@@ -6,9 +6,9 @@ 
 
 		//execute the contructor and check for the Object type and  attributes
 		$group = new Group();
-		$this->assertInstanceOf('Group',$group);
-		$this->assertInstanceOf('User',$group);
-		$this->assertInstanceOf('SugarBean',$group);
+		$this->assertInstanceOf('Group', $group);
+		$this->assertInstanceOf('User', $group);
+		$this->assertInstanceOf('SugarBean', $group);
 		
 		$this->assertAttributeEquals('Group', 'status', $group);
 		$this->assertAttributeEquals('', 'password', $group);
@@ -40,14 +40,14 @@ 
 		
 		//test with empty string params
 		$expected = "SELECT users.* FROM users  WHERE  users.deleted = 0 ORDER BY users.user_name";
-		$actual = $group->create_export_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $group->create_export_query('', '');
+		$this->assertSame($expected, $actual);
 
 		
 		//test with valid string params
 		$expected = "SELECT users.* FROM users  WHERE users.user_name=\"\" AND  users.deleted = 0 ORDER BY users.id";
-		$actual = $group->create_export_query('users.id','users.user_name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $group->create_export_query('users.id', 'users.user_name=""');
+		$this->assertSame($expected, $actual);
 		
 	}
 	

tests/tests/modules/AOD_Index/AOD_IndexTest.php

Spacing +29 added lines, -29 removed lines

@@ -3,13 +3,13 @@ 
 class AOD_IndexTest extends PHPUnit_Framework_TestCase {
 
 	
-	public function testAOD_Index(){
+	public function testAOD_Index() {
 
 		//execute the contructor and check for the Object type and type attribute
 		$aod_index = new AOD_Index();
-		$this->assertInstanceOf('AOD_Index',$aod_index);
-		$this->assertInstanceOf('Basic',$aod_index);
-		$this->assertInstanceOf('SugarBean',$aod_index);
+		$this->assertInstanceOf('AOD_Index', $aod_index);
+		$this->assertInstanceOf('Basic', $aod_index);
+		$this->assertInstanceOf('SugarBean', $aod_index);
 		
 		$this->assertAttributeEquals('AOD_Index', 'module_dir', $aod_index);
 		$this->assertAttributeEquals('AOD_Index', 'object_name', $aod_index);
@@ -21,7 +21,7 @@ 
 		
     }
 
-    public function testisEnabled(){
+    public function testisEnabled() {
     	
     	error_reporting(E_ERROR | E_PARSE);
     	
@@ -33,7 +33,7 @@ 
     	
     }
 
-    public function testfind(){
+    public function testfind() {
 		
     	
     	$aod_index = new AOD_Index();
@@ -53,7 +53,7 @@ 
      	    	
     }
 
-    public function testoptimise(){
+    public function testoptimise() {
 
     	$aod_index = new AOD_Index();
     	$aod_index->id = 1;
@@ -67,20 +67,20 @@ 
     	    	
     }
 
-    public function testgetIndex(){
+    public function testgetIndex() {
 		
     	$aod_index = new AOD_Index();
     	$result = $aod_index->getIndex();
     	
     	
     	//execute the method and verify it returns a different instance of samme type 
-    	$this->assertInstanceOf('AOD_Index',$result);
+    	$this->assertInstanceOf('AOD_Index', $result);
   		$this->assertNotSame($aod_index, $result);  	
     	
     }
 
 
-    public function testgetDocumentForBean(){
+    public function testgetDocumentForBean() {
     	
     	$user = new User(1);
     	
@@ -97,7 +97,7 @@ 
     	
     }
 
-    public function testcommit(){
+    public function testcommit() {
   
     	$aod_index = new AOD_Index();
     	$aod_index->id = 1;
@@ -115,32 +115,32 @@ 
  
     }
 
-    public function testisModuleSearchable(){
+    public function testisModuleSearchable() {
   
     	
     	//test with an invalid module
-    	$this->assertFalse(AOD_Index::isModuleSearchable('',''));
+    	$this->assertFalse(AOD_Index::isModuleSearchable('', ''));
     	
     	
     	//test for modules that are searchable
-    	$this->assertTrue(AOD_Index::isModuleSearchable('DocumentRevisions','DocumentRevision'));
-    	$this->assertTrue(AOD_Index::isModuleSearchable('Cases','Case'));
-    	$this->assertTrue(AOD_Index::isModuleSearchable('Accounts','Account'));
+    	$this->assertTrue(AOD_Index::isModuleSearchable('DocumentRevisions', 'DocumentRevision'));
+    	$this->assertTrue(AOD_Index::isModuleSearchable('Cases', 'Case'));
+    	$this->assertTrue(AOD_Index::isModuleSearchable('Accounts', 'Account'));
     	
     	
     	//test for modules that are not searchable
-    	$this->assertFalse(AOD_Index::isModuleSearchable('AOD_IndexEvent','AOD_IndexEvent'));
-    	$this->assertFalse(AOD_Index::isModuleSearchable('AOD_Index','AOD_Index'));
-    	$this->assertFalse(AOD_Index::isModuleSearchable('AOW_Actions','AOW_Action'));
-    	$this->assertFalse(AOD_Index::isModuleSearchable('AOW_Conditions','AOW_Condition'));
-    	$this->assertFalse(AOD_Index::isModuleSearchable('AOW_Processed','AOW_Processed'));
-    	$this->assertFalse(AOD_Index::isModuleSearchable('SchedulersJobs','SchedulersJob'));
-    	$this->assertFalse(AOD_Index::isModuleSearchable('Users','User'));
+    	$this->assertFalse(AOD_Index::isModuleSearchable('AOD_IndexEvent', 'AOD_IndexEvent'));
+    	$this->assertFalse(AOD_Index::isModuleSearchable('AOD_Index', 'AOD_Index'));
+    	$this->assertFalse(AOD_Index::isModuleSearchable('AOW_Actions', 'AOW_Action'));
+    	$this->assertFalse(AOD_Index::isModuleSearchable('AOW_Conditions', 'AOW_Condition'));
+    	$this->assertFalse(AOD_Index::isModuleSearchable('AOW_Processed', 'AOW_Processed'));
+    	$this->assertFalse(AOD_Index::isModuleSearchable('SchedulersJobs', 'SchedulersJob'));
+    	$this->assertFalse(AOD_Index::isModuleSearchable('Users', 'User'));
     	
  
     }
 
-    public function testindex(){
+    public function testindex() {
   
     	$aod_index = new AOD_Index();
     	$aod_index->id = 1;
@@ -154,13 +154,13 @@ 
     	
     	//test with a searchable module but invalid bean id, it will still index it
     	$result = $aod_index->index("Accounts", 1);
-    	$this->assertEquals(Null,$result);
+    	$this->assertEquals(Null, $result);
     	
  
     }
   
 
-    public function testremove(){
+    public function testremove() {
   
     	$aod_index = new AOD_Index();
     	$aod_index->id = 1;
@@ -178,10 +178,10 @@ 
     	  	
     }
 
-    public function testgetIndexableModules(){
+    public function testgetIndexableModules() {
   
     	
-    	$expected = Array (
+    	$expected = Array(
     			'AM_ProjectTemplates' => 'AM_ProjectTemplates',
     			'AM_TaskTemplates' => 'AM_TaskTemplates',
     			'AOK_KnowledgeBase' => 'AOK_KnowledgeBase',
@@ -222,7 +222,7 @@ 
     	
     	//execute the method and verify that it retunrs expected results
     	$actual = $aod_index->getIndexableModules();
-    	$this->assertSame( $expected, $actual);
+    	$this->assertSame($expected, $actual);
     
     }