adamjakab / SuiteCRM

tests/tests/modules/AOS_Quotes/AOS_QuotesTest.php

Spacing +7 added lines, -7 removed lines

@@ -2,13 +2,13 @@ 
 
 class AOS_QuotesTest extends PHPUnit_Framework_TestCase {
 	
-	function testAOS_Quotes(){	
+	function testAOS_Quotes() {	
 
 		//execute the contructor and check for the Object type and  attributes
 		$aosQuotes = new AOS_Quotes();
-		$this->assertInstanceOf('AOS_Quotes',$aosQuotes);
-		$this->assertInstanceOf('Basic',$aosQuotes);
-		$this->assertInstanceOf('SugarBean',$aosQuotes);
+		$this->assertInstanceOf('AOS_Quotes', $aosQuotes);
+		$this->assertInstanceOf('Basic', $aosQuotes);
+		$this->assertInstanceOf('SugarBean', $aosQuotes);
 			
 		$this->assertAttributeEquals('AOS_Quotes', 'module_dir', $aosQuotes);
 		$this->assertAttributeEquals('AOS_Quotes', 'object_name', $aosQuotes);
@@ -20,7 +20,7 @@ 
 		
 	}
 	
-	function testSaveAndMark_deleted(){
+	function testSaveAndMark_deleted() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		
@@ -28,7 +28,7 @@ 
 		
 		$aosQuotes->name = "test";
 		$aosQuotes->total_amt = 100;
-		$aosQuotes->total_amt_usdollar= 100;
+		$aosQuotes->total_amt_usdollar = 100;
 		
 		$aosQuotes->save();
 			
@@ -41,7 +41,7 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$aosQuotes->mark_deleted($aosQuotes->id);
 		$result = $aosQuotes->retrieve($aosQuotes->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	
 	}

tests/tests/modules/AOR_Fields/AOR_FieldTest.php

Spacing +5 added lines, -5 removed lines

@@ -3,13 +3,13 @@ 
 
 class AOR_FieldTest extends PHPUnit_Framework_TestCase {
 		
-	public function testAOR_Field(){
+	public function testAOR_Field() {
 
 		//execute the contructor and check for the Object type and  attributes
 		$aor_Field = new AOR_Field();
-		$this->assertInstanceOf('AOR_Field',$aor_Field);
-		$this->assertInstanceOf('Basic',$aor_Field);
-		$this->assertInstanceOf('SugarBean',$aor_Field);
+		$this->assertInstanceOf('AOR_Field', $aor_Field);
+		$this->assertInstanceOf('Basic', $aor_Field);
+		$this->assertInstanceOf('SugarBean', $aor_Field);
 			
 		$this->assertAttributeEquals('AOR_Fields', 'module_dir', $aor_Field);
 		$this->assertAttributeEquals('AOR_Field', 'object_name', $aor_Field);
@@ -21,7 +21,7 @@ 
 		
 	}
 
-    public function testsave_lines(){
+    public function testsave_lines() {
 
     	error_reporting(E_ERROR | E_PARSE);
     	 

tests/tests/modules/EmailMarketing/EmailMarketingTest.php

Spacing +11 added lines, -11 removed lines

@@ -11,8 +11,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$emailMarketing = new EmailMarketing();
 		
-		$this->assertInstanceOf('EmailMarketing',$emailMarketing);
-		$this->assertInstanceOf('SugarBean',$emailMarketing);
+		$this->assertInstanceOf('EmailMarketing', $emailMarketing);
+		$this->assertInstanceOf('SugarBean', $emailMarketing);
 			
 		$this->assertAttributeEquals('EmailMarketing', 'module_dir', $emailMarketing);
 		$this->assertAttributeEquals('EmailMarketing', 'object_name', $emailMarketing);
@@ -29,7 +29,7 @@ 
 		$emailMarketing = new EmailMarketing();
 		
 		$result = $emailMarketing->retrieve();
-		$this->assertInstanceOf('EmailMarketing',$result);
+		$this->assertInstanceOf('EmailMarketing', $result);
 		
 	}
 
@@ -38,11 +38,11 @@ 
 		$emailMarketing = new EmailMarketing();
 		
 		//test without setting name
-		$this->assertEquals(Null,$emailMarketing->get_summary_text());
+		$this->assertEquals(Null, $emailMarketing->get_summary_text());
 		
 		//test with name set
 		$emailMarketing->name = "test";
-		$this->assertEquals('test',$emailMarketing->get_summary_text());
+		$this->assertEquals('test', $emailMarketing->get_summary_text());
 		
 	}
 
@@ -52,14 +52,14 @@ 
 		
 		//test with empty string params
 		$expected = " SELECT  email_marketing.*  , jt0.name template_name , jt0.assigned_user_id template_name_owner  , 'EmailTemplates' template_name_mod FROM email_marketing   LEFT JOIN  email_templates jt0 ON email_marketing.template_id=jt0.id AND jt0.deleted=0\n\n AND jt0.deleted=0 where email_marketing.deleted=0"; 		
-		$actual = $emailMarketing->create_export_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $emailMarketing->create_export_query('', '');
+		$this->assertSame($expected, $actual);
 
 		 
 		//test with valid string params
 		$expected = " SELECT  email_marketing.*  , jt0.name template_name , jt0.assigned_user_id template_name_owner  , 'EmailTemplates' template_name_mod FROM email_marketing   LEFT JOIN  email_templates jt0 ON email_marketing.template_id=jt0.id AND jt0.deleted=0\n\n AND jt0.deleted=0 where (email_marketing.name=\"\") AND email_marketing.deleted=0";
-		$actual = $emailMarketing->create_export_query('email_marketing.id','email_marketing.name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $emailMarketing->create_export_query('email_marketing.id', 'email_marketing.name=""');
+		$this->assertSame($expected, $actual);
 	
 	}
 
@@ -69,7 +69,7 @@ 
 		$emailMarketing = new EmailMarketing();
 		
 		//execute the method and verify that it retunrs expected results
-		$expected = array (
+		$expected = array(
 				'ALL_PROSPECT_LISTS' => '0',
 		);
 		
@@ -94,7 +94,7 @@ 
 		//execute the method and verify that it retunrs expected results
 		$expected = "select prospect_lists.* from prospect_lists  left join prospect_list_campaigns on prospect_list_campaigns.prospect_list_id=prospect_lists.id where prospect_list_campaigns.deleted=0 and prospect_list_campaigns.campaign_id='' and prospect_lists.deleted=0 and prospect_lists.list_type not like 'exempt%'";
 		$actual = $emailMarketing->get_all_prospect_lists();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 

tests/tests/modules/SchedulersJobs/SchedulersJobTest.php

Spacing +24 added lines, -24 removed lines

@@ -9,9 +9,9 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$schedulersJob = new SchedulersJob();
 		
-		$this->assertInstanceOf('SchedulersJob',$schedulersJob);
-		$this->assertInstanceOf('Basic',$schedulersJob);
-		$this->assertInstanceOf('SugarBean',$schedulersJob);
+		$this->assertInstanceOf('SchedulersJob', $schedulersJob);
+		$this->assertInstanceOf('Basic', $schedulersJob);
+		$this->assertInstanceOf('SugarBean', $schedulersJob);
 			
 		$this->assertAttributeEquals('job_queue', 'table_name', $schedulersJob);
 		$this->assertAttributeEquals('SchedulersJobs', 'module_dir', $schedulersJob);
@@ -34,7 +34,7 @@ 
 		
 		$schedulersJob->check_date_relationships_load();
 		
-		$this->assertEquals("2015-01-01 00:00:00",$schedulersJob->execute_time_db);
+		$this->assertEquals("2015-01-01 00:00:00", $schedulersJob->execute_time_db);
 			
 	}
 
@@ -44,11 +44,11 @@ 
 		
 		//test with default params
 		$result = $schedulersJob->handleDateFormat();
-		$this->assertGreaterThan(0,strlen($result));
+		$this->assertGreaterThan(0, strlen($result));
 		
 		//test with a valid date param
 		$result = $schedulersJob->handleDateFormat('2015-01-01');
-		$this->assertEquals("2015-01-01 00:00:00",$result);
+		$this->assertEquals("2015-01-01 00:00:00", $result);
 		
 	}
 
@@ -59,11 +59,11 @@ 
 		
 		//test with invalid param
 		$result = $schedulersJob->fireUrl("");
-		$this->assertEquals(false,$result);		
+		$this->assertEquals(false, $result);		
 		
 		//test with valid param
 		$result = $schedulersJob->fireUrl("https://suitecrm.com/");
-		$this->assertEquals(true,$result);
+		$this->assertEquals(true, $result);
 		
 	}
 
@@ -84,7 +84,7 @@ 
 		);
 		$actual = $schedulersJob->get_list_view_data();
 
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 			
 	}
 
@@ -116,8 +116,8 @@ 
 		$this->assertTrue(isset($schedulersJob->id));
 		$this->assertEquals(36, strlen($schedulersJob->id));
 		
-		$this->assertEquals(SchedulersJob::JOB_STATUS_DONE,$schedulersJob->status);
-		$this->assertEquals(SchedulersJob::JOB_FAILURE,$schedulersJob->resolution);
+		$this->assertEquals(SchedulersJob::JOB_STATUS_DONE, $schedulersJob->status);
+		$this->assertEquals(SchedulersJob::JOB_FAILURE, $schedulersJob->resolution);
 				
 		$schedulersJob->mark_deleted($schedulersJob->id);
 	
@@ -135,8 +135,8 @@ 
 		$this->assertTrue(isset($schedulersJob->id));
 		$this->assertEquals(36, strlen($schedulersJob->id));
 		
-		$this->assertEquals(SchedulersJob::JOB_STATUS_DONE,$schedulersJob->status);
-		$this->assertEquals(SchedulersJob::JOB_SUCCESS,$schedulersJob->resolution);
+		$this->assertEquals(SchedulersJob::JOB_STATUS_DONE, $schedulersJob->status);
+		$this->assertEquals(SchedulersJob::JOB_SUCCESS, $schedulersJob->resolution);
 		
 		$schedulersJob->mark_deleted($schedulersJob->id);
 		
@@ -219,15 +219,15 @@ 
 		$this->assertEquals(36, strlen($schedulersJob->id));
 		
 		//verify the related attributes
-		$this->assertEquals(SchedulersJob::JOB_STATUS_QUEUED,$schedulersJob->status);
-		$this->assertEquals(SchedulersJob::JOB_PARTIAL,$schedulersJob->resolution);
+		$this->assertEquals(SchedulersJob::JOB_STATUS_QUEUED, $schedulersJob->status);
+		$this->assertEquals(SchedulersJob::JOB_PARTIAL, $schedulersJob->resolution);
 		
 		
 		
 		//test mark deleted method and verify record doesn't exist after deletion
 		$schedulersJob->mark_deleted($schedulersJob->id);
 		$result = $schedulersJob->retrieve($schedulersJob->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 
@@ -239,7 +239,7 @@ 
 		$schedulersJob->client = "test";
 		$schedulersJob->status = SchedulersJob::JOB_STATUS_RUNNING;
 		$schedulersJob->save();
-		$result = SchedulersJob::runJobId($schedulersJob->id,"test");
+		$result = SchedulersJob::runJobId($schedulersJob->id, "test");
 	
 	
 		//execute the method
@@ -267,7 +267,7 @@ 
 		$schedulersJob->save();
 		
 		$result = SchedulersJob::runJobId($schedulersJob->id);
-		$this->assertEquals("Job " . $schedulersJob->id . " is not marked as running.", $result);
+		$this->assertEquals("Job ".$schedulersJob->id." is not marked as running.", $result);
 		
 		
 		//test with valid job id and status but mismatch client
@@ -275,9 +275,9 @@ 
 		$schedulersJob->status = SchedulersJob::JOB_STATUS_RUNNING;
 		$schedulersJob->save();
 		
-		$result = SchedulersJob::runJobId($schedulersJob->id,"test_client");
+		$result = SchedulersJob::runJobId($schedulersJob->id, "test_client");
 		
-		$this->assertEquals("Job " . $schedulersJob->id . " belongs to another client, can not run as test_client.", $result);
+		$this->assertEquals("Job ".$schedulersJob->id." belongs to another client, can not run as test_client.", $result);
 
 		$schedulersJob->mark_deleted($schedulersJob->id);
 	
@@ -292,21 +292,21 @@ 
 		 
 		$schedulersJob->errors = "";
 		$schedulersJob->errorHandler(E_USER_WARNING, "test err", 'testfile', '1');
-		$this->assertEquals("Warning [512]: test err in testfile on line 1\n", $schedulersJob->errors );
+		$this->assertEquals("Warning [512]: test err in testfile on line 1\n", $schedulersJob->errors);
 		
 		$schedulersJob->errors = "";
 		$schedulersJob->errorHandler(E_ERROR, "test err", 'testfile', '1');
-		$this->assertEquals("Fatal Error [1]: test err in testfile on line 1\n", $schedulersJob->errors );
+		$this->assertEquals("Fatal Error [1]: test err in testfile on line 1\n", $schedulersJob->errors);
 		
 		
 		$schedulersJob->errors = "";
 		$schedulersJob->errorHandler(E_PARSE, "test err", 'testfile', '1');
-		$this->assertEquals("Parse Error [4]: test err in testfile on line 1\n", $schedulersJob->errors );
+		$this->assertEquals("Parse Error [4]: test err in testfile on line 1\n", $schedulersJob->errors);
 		
 		
 		$schedulersJob->errors = "";
 		$schedulersJob->errorHandler(E_RECOVERABLE_ERROR, "test err", 'testfile', '1');
-		$this->assertEquals("Recoverable Error [4096]: test err in testfile on line 1\n", $schedulersJob->errors );
+		$this->assertEquals("Recoverable Error [4096]: test err in testfile on line 1\n", $schedulersJob->errors);
 		
 	}
 

tests/tests/modules/AOP_Case_Updates/AOP_Case_UpdatesTest.php

Spacing +15 added lines, -15 removed lines

@@ -3,13 +3,13 @@ 
 class AOP_Case_UpdatesTest extends PHPUnit_Framework_TestCase {
 
 	
-    public function testAOP_Case_Updates(){
+    public function testAOP_Case_Updates() {
  
     	//execute the contructor and check for the Object type and  attributes
     	$aopCaseUpdates = new AOP_Case_Updates();
-    	$this->assertInstanceOf('AOP_Case_Updates',$aopCaseUpdates);
-    	$this->assertInstanceOf('Basic',$aopCaseUpdates);
-    	$this->assertInstanceOf('SugarBean',$aopCaseUpdates);
+    	$this->assertInstanceOf('AOP_Case_Updates', $aopCaseUpdates);
+    	$this->assertInstanceOf('Basic', $aopCaseUpdates);
+    	$this->assertInstanceOf('SugarBean', $aopCaseUpdates);
     	
     	$this->assertAttributeEquals('AOP_Case_Updates', 'module_dir', $aopCaseUpdates);
     	$this->assertAttributeEquals('AOP_Case_Updates', 'object_name', $aopCaseUpdates);
@@ -21,7 +21,7 @@ 
     	
     }
 
-    public function testsave(){
+    public function testsave() {
  
     	error_reporting(E_ERROR | E_PARSE);
     	
@@ -42,18 +42,18 @@ 
     }
 
     
-    public function testgetCase(){
+    public function testgetCase() {
 
     	$aopCaseUpdates = new AOP_Case_Updates();
     	
     	//execute the method and verify that it returns a Case object
     	$result = $aopCaseUpdates->getCase();
     	
-    	$this->assertInstanceOf('aCase',$result);
+    	$this->assertInstanceOf('aCase', $result);
     	    	
     }
 
-    public function testgetContacts(){
+    public function testgetContacts() {
    
     	$aopCaseUpdates = new AOP_Case_Updates();
     	
@@ -63,39 +63,39 @@ 
     	
     }
 
-    public function testgetUpdateContact(){
+    public function testgetUpdateContact() {
  
     	$aopCaseUpdates = new AOP_Case_Updates();
     	
     	//execute the method without contact_id and verify that it returns null
     	$result = $aopCaseUpdates->getUpdateContact();    	
-    	$this->assertEquals(Null,$result);
+    	$this->assertEquals(Null, $result);
     
     	
     	//execute the method without contact_id and verify that it returns false
     	$aopCaseUpdates->contact_id = 1;
     	$result = $aopCaseUpdates->getUpdateContact();
-    	$this->assertEquals(False,$result);
+    	$this->assertEquals(False, $result);
     	
     }
     
-    public function testgetUser(){
+    public function testgetUser() {
     	
     	$aopCaseUpdates = new AOP_Case_Updates();
     	
     	//execute the method and verify that it returns an instance of User
     	$result = $aopCaseUpdates->getUser();
-    	$this->assertInstanceOf('User',$result);
+    	$this->assertInstanceOf('User', $result);
     	
     }
     
-    public function testgetUpdateUser(){
+    public function testgetUpdateUser() {
 
     	$aopCaseUpdates = new AOP_Case_Updates();
     	
     	//execute the method and verify that it returns an instance of User
     	$result = $aopCaseUpdates->getUpdateUser();
-    	$this->assertInstanceOf('User',$result);    	
+    	$this->assertInstanceOf('User', $result);    	
     
     }
 

tests/tests/modules/jjwg_Markers/jjwg_MarkersTest.php

Spacing +15 added lines, -15 removed lines

@@ -9,9 +9,9 @@ 
     	//execute the contructor and check for the Object type and  attributes
     	$jjwgMarkers = new jjwg_Markers();
     	
-    	$this->assertInstanceOf('jjwg_Markers',$jjwgMarkers);
-    	$this->assertInstanceOf('Basic',$jjwgMarkers);
-    	$this->assertInstanceOf('SugarBean',$jjwgMarkers);
+    	$this->assertInstanceOf('jjwg_Markers', $jjwgMarkers);
+    	$this->assertInstanceOf('Basic', $jjwgMarkers);
+    	$this->assertInstanceOf('SugarBean', $jjwgMarkers);
     	 
     	$this->assertAttributeEquals('jjwg_Markers', 'module_dir', $jjwgMarkers);
     	$this->assertAttributeEquals('jjwg_Markers', 'object_name', $jjwgMarkers);
@@ -32,9 +32,9 @@ 
     	
     	$jjwgMarkers->configuration();
     	 
-    	$this->assertInstanceOf('jjwg_Maps',$jjwgMarkers->jjwg_Maps);
+    	$this->assertInstanceOf('jjwg_Maps', $jjwgMarkers->jjwg_Maps);
     	$this->assertTrue(is_array($jjwgMarkers->settings));
-    	$this->assertGreaterThan(0,count($jjwgMarkers->settings));
+    	$this->assertGreaterThan(0, count($jjwgMarkers->settings));
     	
     }
 
@@ -64,13 +64,13 @@ 
     	$jjwgMarkers = new jjwg_Markers();
 
     	//test with invalid values
-    	$this->assertEquals(false, $jjwgMarkers->is_valid_lng('') );
-    	$this->assertEquals(false, $jjwgMarkers->is_valid_lng(181) );
-    	$this->assertEquals(false, $jjwgMarkers->is_valid_lng(-181) );
+    	$this->assertEquals(false, $jjwgMarkers->is_valid_lng(''));
+    	$this->assertEquals(false, $jjwgMarkers->is_valid_lng(181));
+    	$this->assertEquals(false, $jjwgMarkers->is_valid_lng(-181));
     	 
     	//test with valid values
-    	$this->assertEquals(true, $jjwgMarkers->is_valid_lng(180) );
-    	$this->assertEquals(true, $jjwgMarkers->is_valid_lng(-180) );
+    	$this->assertEquals(true, $jjwgMarkers->is_valid_lng(180));
+    	$this->assertEquals(true, $jjwgMarkers->is_valid_lng(-180));
     	
     }
 
@@ -80,14 +80,14 @@ 
     	$jjwgMarkers = new jjwg_Markers();
     	
     	//test with invalid values
-    	$this->assertEquals(false, $jjwgMarkers->is_valid_lat('') );
-    	$this->assertEquals(false, $jjwgMarkers->is_valid_lat(91) );
-    	$this->assertEquals(false, $jjwgMarkers->is_valid_lat(-91) );
+    	$this->assertEquals(false, $jjwgMarkers->is_valid_lat(''));
+    	$this->assertEquals(false, $jjwgMarkers->is_valid_lat(91));
+    	$this->assertEquals(false, $jjwgMarkers->is_valid_lat(-91));
     	 
     	
     	//test with valid values
-    	$this->assertEquals(true, $jjwgMarkers->is_valid_lat(90) );
-    	$this->assertEquals(true, $jjwgMarkers->is_valid_lat(-90) );
+    	$this->assertEquals(true, $jjwgMarkers->is_valid_lat(90));
+    	$this->assertEquals(true, $jjwgMarkers->is_valid_lat(-90));
     
     }
 

tests/tests/modules/AOW_Processed/AOW_ProcessedTest.php

Spacing +6 added lines, -6 removed lines

@@ -1,15 +1,15 @@ 
 <?php
 
-class AOW_ProcessedTest extends PHPUnit_Framework_TestCase  {
+class AOW_ProcessedTest extends PHPUnit_Framework_TestCase {
 
 	
-	public function testAOW_Processed(){	
+	public function testAOW_Processed() {	
 
 		//execute the contructor and check for the Object type and  attributes
 		$aowProcessed = new AOW_Processed();
-		$this->assertInstanceOf('AOW_Processed',$aowProcessed);
-		$this->assertInstanceOf('Basic',$aowProcessed);
-		$this->assertInstanceOf('SugarBean',$aowProcessed);
+		$this->assertInstanceOf('AOW_Processed', $aowProcessed);
+		$this->assertInstanceOf('Basic', $aowProcessed);
+		$this->assertInstanceOf('SugarBean', $aowProcessed);
 			
 		$this->assertAttributeEquals('AOW_Processed', 'module_dir', $aowProcessed);
 		$this->assertAttributeEquals('AOW_Processed', 'object_name', $aowProcessed);
@@ -20,7 +20,7 @@ 
 		
 	}
 	
-	public function testbean_implements(){
+	public function testbean_implements() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		

tests/tests/modules/jjwg_Maps/jjwg_MapsTest.php

Spacing +47 added lines, -47 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-class jjwg_MapsTest extends PHPUnit_Framework_TestCase{
+class jjwg_MapsTest extends PHPUnit_Framework_TestCase {
 
 
 
@@ -9,9 +9,9 @@ 
     	//execute the contructor and check for the Object type and  attributes
     	$jjwgMaps = new jjwg_Maps();
     	 
-    	$this->assertInstanceOf('jjwg_Maps',$jjwgMaps);
-    	$this->assertInstanceOf('Basic',$jjwgMaps);
-    	$this->assertInstanceOf('SugarBean',$jjwgMaps);
+    	$this->assertInstanceOf('jjwg_Maps', $jjwgMaps);
+    	$this->assertInstanceOf('Basic', $jjwgMaps);
+    	$this->assertInstanceOf('SugarBean', $jjwgMaps);
     	
     	$this->assertAttributeEquals('jjwg_Maps', 'module_dir', $jjwgMaps);
     	$this->assertAttributeEquals('jjwg_Maps', 'object_name', $jjwgMaps);
@@ -102,7 +102,7 @@ 
     	
     	$result = $jjwgMaps->updateRelatedMeetingsGeocodeInfo($bean);
     	$this->assertSame(null, $result);
-    	$this->assertInstanceOf('jjwg_Address_Cache',$jjwgMaps->jjwg_Address_Cache);
+    	$this->assertInstanceOf('jjwg_Address_Cache', $jjwgMaps->jjwg_Address_Cache);
     	
     }
 
@@ -193,7 +193,7 @@ 
     	
     	//test with invalid geocode bean
     	$result = $jjwgMaps->getGeocodeAddressesResult("accounts");
-    	$this->assertInstanceOf('mysqli_result',$result);
+    	$this->assertInstanceOf('mysqli_result', $result);
     	
     }
 
@@ -203,7 +203,7 @@ 
     	$jjwgMaps = new jjwg_Maps();
     	
     	//test with invalid value
-    	$expected = array (
+    	$expected = array(
     			'address' => '',
     			'status' => 'ZERO_RESULTS',
     			'lat' => NULL,
@@ -211,52 +211,52 @@ 
     	);
     	
     	$actual = $jjwgMaps->getGoogleMapsGeocode("");
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
 
     	
     	//test with valid value
-    	$expected = array (
+    	$expected = array(
     			'address' => 'washington D.C',
     			'status' => 'OK',
     			'lat' => 38.90719229999999839719748706556856632232666015625,
     			'lng' => -77.0368706999999943718648864887654781341552734375,
     	);
     	$actual = $jjwgMaps->getGoogleMapsGeocode("washington D.C");
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
   
     	
     	//test with valid value and full array true
     	$expected =	
-    		array (
+    		array(
     			'results' =>
-    			array (
-    					array (
+    			array(
+    					array(
     							'address_components' =>
-    							array (
-    									array ( 'long_name' => 'Washington', 'short_name' => 'D.C.', 'types' =>  array ( 'locality', 'political', ),  ),
-    									array ( 'long_name' => 'District of Columbia', 'short_name' => 'District of Columbia', 'types' =>  array ( 'administrative_area_level_2','political', ), ),
-    									array ( 'long_name' => 'District of Columbia', 'short_name' => 'DC', 'types' =>  array ( 'administrative_area_level_1', 'political', ),
+    							array(
+    									array('long_name' => 'Washington', 'short_name' => 'D.C.', 'types' =>  array('locality', 'political',),),
+    									array('long_name' => 'District of Columbia', 'short_name' => 'District of Columbia', 'types' =>  array('administrative_area_level_2', 'political',),),
+    									array('long_name' => 'District of Columbia', 'short_name' => 'DC', 'types' =>  array('administrative_area_level_1', 'political',),
     									),
-    									array ( 'long_name' => 'United States', 'short_name' => 'US', 'types' => array ( 'country', 'political', ),  ),
+    									array('long_name' => 'United States', 'short_name' => 'US', 'types' => array('country', 'political',),),
     							),
     							'formatted_address' => 'Washington, DC, USA',
     							'geometry' =>
-    							array (
-    									'bounds' => array ( 'northeast' =>  array ( 'lat' => 38.9955479999999994333848007954657077789306640625, 'lng' => -76.909392999999994344761944375932216644287109375, ), 'southwest' =>  array ( 'lat' => 38.8031495000000035133780329488217830657958984375, 'lng' => -77.1197399999999930741978459991514682769775390625,  ), ),
-    									'location' => array ( 'lat' => 38.90719229999999839719748706556856632232666015625, 'lng' => -77.0368706999999943718648864887654781341552734375,),
+    							array(
+    									'bounds' => array('northeast' =>  array('lat' => 38.9955479999999994333848007954657077789306640625, 'lng' => -76.909392999999994344761944375932216644287109375,), 'southwest' =>  array('lat' => 38.8031495000000035133780329488217830657958984375, 'lng' => -77.1197399999999930741978459991514682769775390625,),),
+    									'location' => array('lat' => 38.90719229999999839719748706556856632232666015625, 'lng' => -77.0368706999999943718648864887654781341552734375,),
     									'location_type' => 'APPROXIMATE',
-    									'viewport' => array ( 'northeast' => array ( 'lat' => 38.9955479999999994333848007954657077789306640625, 'lng' => -76.909392999999994344761944375932216644287109375,), 'southwest' => array ( 'lat' => 38.8031495000000035133780329488217830657958984375, 'lng' => -77.1197399999999930741978459991514682769775390625,), ),
+    									'viewport' => array('northeast' => array('lat' => 38.9955479999999994333848007954657077789306640625, 'lng' => -76.909392999999994344761944375932216644287109375,), 'southwest' => array('lat' => 38.8031495000000035133780329488217830657958984375, 'lng' => -77.1197399999999930741978459991514682769775390625,),),
     							),
     							'place_id' => 'ChIJW-T2Wt7Gt4kRKl2I1CJFUsI',
-    							'types' => array ('locality', 'political'),
+    							'types' => array('locality', 'political'),
     					),
     			),
     			'status' => 'OK',
     	);
     	
-    	$actual = $jjwgMaps->getGoogleMapsGeocode("washington D.C",true);
+    	$actual = $jjwgMaps->getGoogleMapsGeocode("washington D.C", true);
     		
-    	$this->assertSame($expected['results']['geometry'],$actual['results']['geometry']);
+    	$this->assertSame($expected['results']['geometry'], $actual['results']['geometry']);
     	//$this->assertSame($expected,$actual);
     	
     }
@@ -269,7 +269,7 @@ 
     	
     	//test for Account Object type 
     	$address = array('id'=>1, 'billing_address_street'=>"addr 1", 'billing_address_city'=>"addr 2", 'billing_address_state'=>"addr 3", 'billing_address_postalcode'=>"addr 4", 'billing_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsAddress('Account',$address);
+    	$result = $jjwgMaps->defineMapsAddress('Account', $address);
     	//var_dump($result);
     	$this->assertEquals(array('address' => "addr 1, addr 2, addr 3, addr 4, addr 5"), $result);
     	
@@ -277,41 +277,41 @@ 
     	
     	//test for Contact Object type
     	$address = array('id'=>1, 'primary_address_street'=>"addr 1", 'primary_address_city'=>"addr 2", 'primary_address_state'=>"addr 3", 'primary_address_postalcode'=>"addr 4", 'primary_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsAddress('Contact',$address);
+    	$result = $jjwgMaps->defineMapsAddress('Contact', $address);
     	$this->assertEquals(array('address' => "addr 1, addr 2, addr 3, addr 4, addr 5"), $result);
     	
     	
     	
     	//test for Leads Object type
     	$address = array('id'=>1, 'primary_address_street'=>"addr 1", 'primary_address_city'=>"addr 2", 'primary_address_state'=>"addr 3", 'primary_address_postalcode'=>"addr 4", 'primary_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsAddress('Lead',$address);
+    	$result = $jjwgMaps->defineMapsAddress('Lead', $address);
     	$this->assertEquals(array('address' => "addr 1, addr 2, addr 3, addr 4, addr 5"), $result);
     	 
     	
 
     	//test for Opportunities Object type
     	$address = array('id'=>1, 'billing_address_street'=>"addr 1", 'billing_address_city'=>"addr 2", 'billing_address_state'=>"addr 3", 'billing_address_postalcode'=>"addr 4", 'billing_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsAddress('Opportunity',$address);
+    	$result = $jjwgMaps->defineMapsAddress('Opportunity', $address);
     	$this->assertEquals(false, $result);
     	
  
     	
     	//test for Case Object type
     	$address = array('id'=>1, 'billing_address_street'=>"addr 1", 'billing_address_city'=>"addr 2", 'billing_address_state'=>"addr 3", 'billing_address_postalcode'=>"addr 4", 'billing_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsAddress('Case',$address);
+    	$result = $jjwgMaps->defineMapsAddress('Case', $address);
     	$this->assertEquals(false, $result);
     	
     	
     	//test for Project Object type
     	$address = array('id'=>1, 'billing_address_street'=>"addr 1", 'billing_address_city'=>"addr 2", 'billing_address_state'=>"addr 3", 'billing_address_postalcode'=>"addr 4", 'billing_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsAddress('Project',$address);
+    	$result = $jjwgMaps->defineMapsAddress('Project', $address);
     	$this->assertEquals(false, $result);
     	 
     	
 
     	//test for Project Meetings type
     	$address = array('id'=>1, 'billing_address_street'=>"addr 1", 'billing_address_city'=>"addr 2", 'billing_address_state'=>"addr 3", 'billing_address_postalcode'=>"addr 4", 'billing_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsAddress('Meeting',$address);
+    	$result = $jjwgMaps->defineMapsAddress('Meeting', $address);
     	$this->assertEquals(false, $result);
     	 
     }
@@ -328,31 +328,31 @@ 
     	
     	//test for type billing  
     	$address = array('billing_address_street'=>"addr 1", 'billing_address_city'=>"addr 2", 'billing_address_state'=>"addr 3", 'billing_address_postalcode'=>"addr 4", 'billing_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsFormattedAddress($address,'billing');
+    	$result = $jjwgMaps->defineMapsFormattedAddress($address, 'billing');
     	$this->assertEquals("addr 1, addr 2, addr 3, addr 4, addr 5", $result);
 
     	
     	//test for type shipping    	
     	$address = array('shipping_address_street'=>"addr 1", 'shipping_address_city'=>"addr 2", 'shipping_address_state'=>"addr 3", 'shipping_address_postalcode'=>"addr 4", 'shipping_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsFormattedAddress($address,'shipping');
+    	$result = $jjwgMaps->defineMapsFormattedAddress($address, 'shipping');
     	$this->assertEquals("addr 1, addr 2, addr 3, addr 4, addr 5", $result);
     	 
     	
     	//test for type primary
     	$address = array('primary_address_street'=>"addr 1", 'primary_address_city'=>"addr 2", 'primary_address_state'=>"addr 3", 'primary_address_postalcode'=>"addr 4", 'primary_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsFormattedAddress($address,'primary');
+    	$result = $jjwgMaps->defineMapsFormattedAddress($address, 'primary');
     	$this->assertEquals("addr 1, addr 2, addr 3, addr 4, addr 5", $result);
     	
     	
     	//test for type alt
     	$address = array('alt_address_street'=>"addr 1", 'alt_address_city'=>"addr 2", 'alt_address_state'=>"addr 3", 'alt_address_postalcode'=>"addr 4", 'alt_address_country'=>"addr 5");
-    	$result = $jjwgMaps->defineMapsFormattedAddress($address,'alt');
+    	$result = $jjwgMaps->defineMapsFormattedAddress($address, 'alt');
     	$this->assertEquals("addr 1, addr 2, addr 3, addr 4, addr 5", $result);
     	
     	
     	//test for type address
     	$address = array('address_street'=>"addr 1", 'address_city'=>"addr 2", 'address_state'=>"addr 3", 'address_postalcode'=>"addr 4");
-    	$result = $jjwgMaps->defineMapsFormattedAddress($address,'address');
+    	$result = $jjwgMaps->defineMapsFormattedAddress($address, 'address');
     	$this->assertEquals("addr 1, addr 2, addr 3, addr 4", $result);
 
     	
@@ -364,13 +364,13 @@ 
     	$jjwgMaps = new jjwg_Maps();
     	
     	//test with invalid values
-    	$this->assertEquals(false, $jjwgMaps->is_valid_lng('') );
-    	$this->assertEquals(false, $jjwgMaps->is_valid_lng(181) );
-    	$this->assertEquals(false, $jjwgMaps->is_valid_lng(-181) );
+    	$this->assertEquals(false, $jjwgMaps->is_valid_lng(''));
+    	$this->assertEquals(false, $jjwgMaps->is_valid_lng(181));
+    	$this->assertEquals(false, $jjwgMaps->is_valid_lng(-181));
     	 
     	//test with valid values
-    	$this->assertEquals(true, $jjwgMaps->is_valid_lng(180) );
-    	$this->assertEquals(true, $jjwgMaps->is_valid_lng(-180) );
+    	$this->assertEquals(true, $jjwgMaps->is_valid_lng(180));
+    	$this->assertEquals(true, $jjwgMaps->is_valid_lng(-180));
 
     }
 
@@ -380,13 +380,13 @@ 
     	$jjwgMaps = new jjwg_Maps();
     	
     	//test with invalid values
-    	$this->assertEquals(false, $jjwgMaps->is_valid_lat('') );
-    	$this->assertEquals(false, $jjwgMaps->is_valid_lat(91) );
-    	$this->assertEquals(false, $jjwgMaps->is_valid_lat(-91) );
+    	$this->assertEquals(false, $jjwgMaps->is_valid_lat(''));
+    	$this->assertEquals(false, $jjwgMaps->is_valid_lat(91));
+    	$this->assertEquals(false, $jjwgMaps->is_valid_lat(-91));
     	 
     	//test with valid values
-    	$this->assertEquals(true, $jjwgMaps->is_valid_lat(90) );
-    	$this->assertEquals(true, $jjwgMaps->is_valid_lat(-90) );
+    	$this->assertEquals(true, $jjwgMaps->is_valid_lat(90));
+    	$this->assertEquals(true, $jjwgMaps->is_valid_lat(-90));
     	
     }
 

tests/tests/modules/MergeRecords/MergeRecordTest.php

Spacing +23 added lines, -23 removed lines

@@ -8,8 +8,8 @@ 
     	//execute the contructor and check for the Object type and  attributes
     	$mergeRecord = new MergeRecord();
     		
-    	$this->assertInstanceOf('MergeRecord',$mergeRecord);
-    	$this->assertInstanceOf('SugarBean',$mergeRecord);
+    	$this->assertInstanceOf('MergeRecord', $mergeRecord);
+    	$this->assertInstanceOf('SugarBean', $mergeRecord);
     	
     	$this->assertAttributeEquals('MergeRecords', 'module_dir', $mergeRecord);
     	$this->assertAttributeEquals('MergeRecord', 'object_name', $mergeRecord);
@@ -73,7 +73,7 @@ 
     	$this->assertAttributeEquals('User', 'merge_bean_class', $mergeRecord);
     	$this->assertAttributeEquals('modules/Users/User.php', 'merge_bean_file_path', $mergeRecord);
 
-    	$this->assertInstanceOf('User',$mergeRecord->merge_bean);
+    	$this->assertInstanceOf('User', $mergeRecord->merge_bean);
     	
     
     }
@@ -103,7 +103,7 @@ 
     	$this->assertAttributeEquals('User', 'merge_bean_class2', $mergeRecord);
     	$this->assertAttributeEquals('modules/Users/User.php', 'merge_bean_file_path2', $mergeRecord);
     	
-    	$this->assertInstanceOf('User',$mergeRecord->merge_bean2);
+    	$this->assertInstanceOf('User', $mergeRecord->merge_bean2);
     	 
     }
     
@@ -167,12 +167,12 @@ 
     	
     	
     	//test without setting name
-    	$this->assertEquals(Null,$mergeRecord->get_summary_text());
+    	$this->assertEquals(Null, $mergeRecord->get_summary_text());
 
     	
     	//test with name set
     	$mergeRecord->merge_bean->name = "test";
-    	$this->assertEquals('test',$mergeRecord->get_summary_text());
+    	$this->assertEquals('test', $mergeRecord->get_summary_text());
     
     }
 
@@ -203,13 +203,13 @@ 
     	//test with string
     	$expected = "contacts.last_name like 'test%' or contacts.first_name like 'test%' or accounts.name like 'test%' or contacts.assistant like 'test%' or ea.email_address like 'test%'";
     	$actual = $mergeRecord->build_generic_where_clause('test');
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
     	
     	
     	//test with number
     	$expected = "contacts.last_name like '1%' or contacts.first_name like '1%' or accounts.name like '1%' or contacts.assistant like '1%' or ea.email_address like '1%' or contacts.phone_home like '%1%' or contacts.phone_mobile like '%1%' or contacts.phone_work like '%1%' or contacts.phone_other like '%1%' or contacts.phone_fax like '%1%' or contacts.assistant_phone like '%1%'";
     	$actual = $mergeRecord->build_generic_where_clause(1);
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
     	
     }
 
@@ -224,7 +224,7 @@ 
     	
     }
     
-    public function testACLAccess(){
+    public function testACLAccess() {
 
     	$mergeRecord = new MergeRecord();
     	
@@ -235,10 +235,10 @@ 
     	//test with merge bean loaded
     	$mergeRecord->load_merge_bean("Meetings");
     	
-    	$this->assertEquals(true,$mergeRecord->ACLAccess('edit'));
-    	$this->assertEquals(true,$mergeRecord->ACLAccess('save'));
-    	$this->assertEquals(true,$mergeRecord->ACLAccess('editview'));
-    	$this->assertEquals(true,$mergeRecord->ACLAccess('delete'));
+    	$this->assertEquals(true, $mergeRecord->ACLAccess('edit'));
+    	$this->assertEquals(true, $mergeRecord->ACLAccess('save'));
+    	$this->assertEquals(true, $mergeRecord->ACLAccess('editview'));
+    	$this->assertEquals(true, $mergeRecord->ACLAccess('delete'));
     	
     }
     
@@ -254,12 +254,12 @@ 
 
     	$mergeRecord->load_merge_bean("Meetings");
     	
-    	$expected = array (
-						  'id' => array ( 'name' => 'id', 'vname' => 'LBL_ID', 'type' => 'id', 'required' => true, 'reportable' => true, 'comment' => 'Unique identifier', 'inline_edit' => false, 'value' => '1', 'search_type' => 'Exact'),
-						  'name' => array ( 'name' => 'name','vname' => 'LBL_SUBJECT','required' => true, 'type' => 'name', 'dbType' => 'varchar', 'unified_search' => true, 'full_text_search' => array ('boost' => 3), 'len' => '50','comment' => 'Meeting name', 'importable' => 'required', 'value' => 'test', 'search_type' => 'Exact' )
+    	$expected = array(
+						  'id' => array('name' => 'id', 'vname' => 'LBL_ID', 'type' => 'id', 'required' => true, 'reportable' => true, 'comment' => 'Unique identifier', 'inline_edit' => false, 'value' => '1', 'search_type' => 'Exact'),
+						  'name' => array('name' => 'name', 'vname' => 'LBL_SUBJECT', 'required' => true, 'type' => 'name', 'dbType' => 'varchar', 'unified_search' => true, 'full_text_search' => array('boost' => 3), 'len' => '50', 'comment' => 'Meeting name', 'importable' => 'required', 'value' => 'test', 'search_type' => 'Exact')
 						);
     	
-    	$mergeRecord->populate_search_params(array('nameSearchField'=>'test','idSearchField'=>'1'));
+    	$mergeRecord->populate_search_params(array('nameSearchField'=>'test', 'idSearchField'=>'1'));
     	
     	$this->assertSame($expected, $mergeRecord->field_search_params);
     	
@@ -282,7 +282,7 @@ 
     	 
     	$expected = "<input type='hidden' name='idSearchField' value='1' />\n<input type='hidden' name='idSearchType' value='' />\n<input type='hidden' name='nameSearchField' value='test' />\n<input type='hidden' name='nameSearchType' value='' />\n";
     	 
-    	$result = $mergeRecord->get_inputs_for_search_params(array('nameSearchField'=>'test','idSearchField'=>'1'));
+    	$result = $mergeRecord->get_inputs_for_search_params(array('nameSearchField'=>'test', 'idSearchField'=>'1'));
     	     	
     	$this->assertSame($expected, $result);
 
@@ -330,13 +330,13 @@ 
     	$mergeRecord = new MergeRecord();
     	
     	$mergeRecord->load_merge_bean("Contacts");
-    	$mergeRecord->populate_search_params(array('nameSearchField'=>'test','idSearchField'=>'1'));
+    	$mergeRecord->populate_search_params(array('nameSearchField'=>'test', 'idSearchField'=>'1'));
 
-    	$expected = array( "contacts.id='1'",  "contacts.name='test'", "contacts.id !=''" );
+    	$expected = array("contacts.id='1'", "contacts.name='test'", "contacts.id !=''");
 
     	$actual = $mergeRecord->create_where_statement();
     	
-    	$this->assertSame( $expected, $actual);
+    	$this->assertSame($expected, $actual);
     	
     }
 
@@ -345,12 +345,12 @@ 
 
     	$mergeRecord = new MergeRecord();
     	
-    	$clauses = array( "contacts.id='1'",  "contacts.name='test'", "contacts.id !=''" );
+    	$clauses = array("contacts.id='1'", "contacts.name='test'", "contacts.id !=''");
     	$expected = "contacts.id='1' AND contacts.name='test' AND contacts.id !=''";
     	
     	$actual = $mergeRecord->generate_where_statement($clauses);
     	
-    	$this->assertSame( $expected, $actual);
+    	$this->assertSame($expected, $actual);
     	
     	//error_reporting(E_ALL);
 	}