adamjakab / SuiteCRM

tests/tests/modules/ProjectTask/ProjectTaskTest.php

Spacing +21 added lines, -21 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$projectTask = new ProjectTask();
 			
-		$this->assertInstanceOf('ProjectTask',$projectTask);
-		$this->assertInstanceOf('SugarBean',$projectTask);
+		$this->assertInstanceOf('ProjectTask', $projectTask);
+		$this->assertInstanceOf('SugarBean', $projectTask);
 		
 		
 		$this->assertAttributeEquals('project_task', 'table_name', $projectTask);
@@ -47,7 +47,7 @@ 
 		//$projectTask->project_id = "1";
 		$projectTask->assigned_user_id = "1";
 		$projectTask->description = "test description";
-		$projectTask->parent_task_id =1;
+		$projectTask->parent_task_id = 1;
 		
 		$projectTask->save();
 		
@@ -64,7 +64,7 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$projectTask->mark_deleted($projectTask->id);
 		$result = $projectTask->retrieve($projectTask->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 	
@@ -75,7 +75,7 @@ 
 		
 		$result = $projectTask->_get_depends_on_name($id);
 		
-		$this->assertEquals("1", $projectTask->depends_on_name_owner) ;
+		$this->assertEquals("1", $projectTask->depends_on_name_owner);
 		$this->assertEquals("ProjectTask", $projectTask->depends_on_name_mod);
 		$this->assertEquals("test", $result);
 			
@@ -104,7 +104,7 @@ 
 		
 		$projectTask = new ProjectTask();
 		
-		$projectTask->parent_task_id =1;		
+		$projectTask->parent_task_id = 1;		
 		$result = $projectTask->getProjectTaskParent();
 		$this->assertEquals(false, $result);
 		
@@ -180,11 +180,11 @@ 
 		$projectTask = new ProjectTask();
 		
 		//test without setting name
-		$this->assertEquals(Null,$projectTask->get_summary_text());
+		$this->assertEquals(Null, $projectTask->get_summary_text());
 		
 		//test with name set
 		$projectTask->name = "test";
-		$this->assertEquals('test',$projectTask->get_summary_text());
+		$this->assertEquals('test', $projectTask->get_summary_text());
 		
 	}
 
@@ -228,17 +228,17 @@ 
 		//test with empty string params
 		$expected = "project_task.name like '%'";
 		$actual = $projectTask->build_generic_where_clause('');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 			
 		//test with valid string params
 		$expected = "project_task.name like '%'";
 		$actual = $projectTask->build_generic_where_clause('test');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 	
 		$projectTask = new ProjectTask();
 		
@@ -246,7 +246,7 @@ 
 		$projectTask->description = "test assigned user";
 		$projectTask->parent_type = "Project";
 		
-		$expected = array (
+		$expected = array(
 				'NAME' => 'tes user',
 				'DESCRIPTION' => 'test assigned user',
 				'ORDER_NUMBER' => '1',
@@ -266,7 +266,7 @@ 
 		
 	}
 
-	public function testbean_implements(){
+	public function testbean_implements() {
 	
 		$projectTask = new ProjectTask();
 		
@@ -275,13 +275,13 @@ 
 		$this->assertEquals(true, $projectTask->bean_implements('ACL')); //test with valid value
 	}
 	
-	public function testlistviewACLHelper(){
+	public function testlistviewACLHelper() {
 	
 		$projectTask = new ProjectTask();
 		
-		$expected = array( 'MAIN' => 'a', 'PARENT' => 'a', 'PARENT_TASK' => 'a' );
+		$expected = array('MAIN' => 'a', 'PARENT' => 'a', 'PARENT_TASK' => 'a');
 		$actual = $projectTask->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -292,14 +292,14 @@ 
 		
 		//test with empty string params
 		$expected = "SELECT\n				project_task.*,\n                users.user_name as assigned_user_name  FROM project_task LEFT JOIN project ON project_task.project_id=project.id AND project.deleted=0  LEFT JOIN users\n                   	ON project_task.assigned_user_id=users.id where  project_task.deleted=0 ";
-		$actual = $projectTask->create_export_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $projectTask->create_export_query('', '');
+		$this->assertSame($expected, $actual);
 
 	
 		//test with valid string params
 		$expected = "SELECT\n				project_task.*,\n                users.user_name as assigned_user_name  FROM project_task LEFT JOIN project ON project_task.project_id=project.id AND project.deleted=0  LEFT JOIN users\n                   	ON project_task.assigned_user_id=users.id where (users.user_name= \"\") AND  project_task.deleted=0  ORDER BY project_task.id";
-		$actual = $projectTask->create_export_query('project_task.id','users.user_name= ""');
-		$this->assertSame($expected,$actual);
+		$actual = $projectTask->create_export_query('project_task.id', 'users.user_name= ""');
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -310,7 +310,7 @@ 
 		
 		$expected = "<select name=\"utilization\">\n<OPTION value='0'>none</OPTION>\n<OPTION value='25'>25</OPTION>\n<OPTION value='50'>50</OPTION>\n<OPTION value='75'>75</OPTION>\n<OPTION value='100'>100</OPTION></select>";
 		$actual = getUtilizationDropdown($projectTask, 'utilization', '0', 'EditView');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
     

tests/tests/modules/Documents/DocumentTest.php

Spacing +24 added lines, -24 removed lines

@@ -9,9 +9,9 @@ 
 	
 		//execute the contructor and check for the Object type and  attributes
 		$document = new Document();
-		$this->assertInstanceOf('Document',$document);
-		$this->assertInstanceOf('File',$document);
-		$this->assertInstanceOf('SugarBean',$document);
+		$this->assertInstanceOf('Document', $document);
+		$this->assertInstanceOf('File', $document);
+		$this->assertInstanceOf('SugarBean', $document);
 			
 		$this->assertAttributeEquals('Documents', 'module_dir', $document);
 		$this->assertAttributeEquals('Document', 'object_name', $document);
@@ -29,11 +29,11 @@ 
 		$document = new Document();
 		
 		$document->filename = "test";
-		$document->file_url ="test_url";
-		$document->file_url_noimage= "test_image_url";
-		$document->last_rev_created_name= "test";
+		$document->file_url = "test_url";
+		$document->file_url_noimage = "test_image_url";
+		$document->last_rev_created_name = "test";
 		$document->category_id = "1";
-		$document->subcategory_id = "1" ;
+		$document->subcategory_id = "1";
 		$document->document_name = "test";
 		
 		$document->save();
@@ -52,7 +52,7 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$document->mark_deleted($document->id);
 		$result = $document->retrieve($document->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 
 	}
 
@@ -62,11 +62,11 @@ 
 		$document = new Document();
 		
 		//test without setting name
-		$this->assertEquals(Null,$document->get_summary_text());
+		$this->assertEquals(Null, $document->get_summary_text());
 		
 		//test with name set
 		$document->document_name = "test";
-		$this->assertEquals('test',$document->get_summary_text());
+		$this->assertEquals('test', $document->get_summary_text());
 		
 	}
 
@@ -76,12 +76,12 @@ 
 		$document = new Document();
 		
 		//test without presetting attributes
-		$this->assertEquals(null,$document->is_authenticated());
+		$this->assertEquals(null, $document->is_authenticated());
 		
 		
 		//test with attributes preset
-		$document->authenticated= true;
-		$this->assertEquals(true,$document->is_authenticated());
+		$document->authenticated = true;
+		$this->assertEquals(true, $document->is_authenticated());
 		
 	}
 
@@ -143,13 +143,13 @@ 
     	//test with empty string parameters
     	$expected = "SELECT\n						documents.* FROM documents  WHERE  documents.deleted = 0 ORDER BY documents.document_name";
    		$actual = $document->create_export_query('', '');
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
     	
     	
     	//test with valid string parameters
     	$expected = "SELECT\n						documents.* FROM documents  WHERE documents.document_name = \"\" AND  documents.deleted = 0 ORDER BY documents.id";
     	$actual = $document->create_export_query('documents.id', 'documents.document_name = ""');
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
     	
 	}
 
@@ -160,15 +160,15 @@ 
 		//execute the method and verify that it retunrs expected results
 		
 		$document->filename = "test";
-		$document->file_url ="test_url";
-		$document->file_url_noimage= "test_image_url";
-		$document->last_rev_created_name= "test";
+		$document->file_url = "test_url";
+		$document->file_url_noimage = "test_image_url";
+		$document->last_rev_created_name = "test";
 		$document->category_id = "1";
-		$document->subcategory_id = "1" ;
+		$document->subcategory_id = "1";
 		$document->document_name = "test";
 		
 		
-		$expected = array (
+		$expected = array(
 				'DELETED' => 0,
 				'DOCUMENT_NAME' => 'test',
 				'DOC_TYPE' => 'Sugar',
@@ -191,10 +191,10 @@ 
 		);
 
 		$actual = $document->get_list_view_data();
-        foreach($expected as $expectedKey => $expectedVal){
-            if($expectedKey == 'FILE_URL'){
-                $this->assertRegExp($expected[$expectedKey],$actual[$expectedKey]);
-            }else {
+        foreach ($expected as $expectedKey => $expectedVal) {
+            if ($expectedKey == 'FILE_URL') {
+                $this->assertRegExp($expected[$expectedKey], $actual[$expectedKey]);
+            } else {
                 $this->assertSame($expected[$expectedKey], $actual[$expectedKey]);
             }
         }

tests/tests/modules/AOR_Conditions/AOR_ConditionTest.php

Spacing +5 added lines, -5 removed lines

@@ -3,13 +3,13 @@ 
 class AOR_ConditionTest extends PHPUnit_Framework_TestCase {
 
 	
-	public function testAOR_Condition(){
+	public function testAOR_Condition() {
 
 		//execute the contructor and check for the Object type and  attributes
 		$aor_Condition = new AOR_Condition();
-		$this->assertInstanceOf('AOR_Condition',$aor_Condition);
-		$this->assertInstanceOf('Basic',$aor_Condition);
-		$this->assertInstanceOf('SugarBean',$aor_Condition);
+		$this->assertInstanceOf('AOR_Condition', $aor_Condition);
+		$this->assertInstanceOf('Basic', $aor_Condition);
+		$this->assertInstanceOf('SugarBean', $aor_Condition);
 			
 		$this->assertAttributeEquals('AOR_Conditions', 'module_dir', $aor_Condition);
 		$this->assertAttributeEquals('AOR_Condition', 'object_name', $aor_Condition);
@@ -22,7 +22,7 @@ 
 		
 	}
 
-    public function testsave_lines(){
+    public function testsave_lines() {
 
     	error_reporting(E_ERROR | E_PARSE);
     	

tests/tests/modules/AOD_IndexEvent/AOD_IndexEventTest.php

Spacing +4 added lines, -4 removed lines

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

tests/tests/modules/Trackers/TrackerTest.php

Spacing +4 added lines, -4 removed lines

@@ -8,8 +8,8 @@ 
     	//execute the contructor and check for the Object type and  attributes
     	$tracker = new Tracker();
     		
-    	$this->assertInstanceOf('Tracker',$tracker);
-    	$this->assertInstanceOf('SugarBean',$tracker);
+    	$this->assertInstanceOf('Tracker', $tracker);
+    	$this->assertInstanceOf('SugarBean', $tracker);
     	
     	$this->assertAttributeEquals('tracker', 'table_name', $tracker);
     	$this->assertAttributeEquals('Trackers', 'module_dir', $tracker);
@@ -28,7 +28,7 @@ 
     	
     	$result = $tracker->get_recently_viewed(1);
     	
-    	$this->assertInstanceOf('BreadCrumbStack',$_SESSION['breadCrumbs']);
+    	$this->assertInstanceOf('BreadCrumbStack', $_SESSION['breadCrumbs']);
     	$this->assertTrue(is_array($result));
     	
     }
@@ -71,7 +71,7 @@ 
     	
     	
     	//test with headerDisplayed set
-    	$GLOBALS['app']->headerDisplayed  = 1;
+    	$GLOBALS['app']->headerDisplayed = 1;
     	Tracker::logPage();
     	$this->assertEquals(time(), $_SESSION['lpage']);
     	

tests/tests/modules/Meetings/MeetingTest.php

Spacing +47 added lines, -47 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-class MeetingTest extends PHPUnit_Framework_TestCase  {
+class MeetingTest extends PHPUnit_Framework_TestCase {
 
 
 	public function testMeeting() {
@@ -10,8 +10,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$meeting = new Meeting();
 			
-		$this->assertInstanceOf('Meeting',$meeting);
-		$this->assertInstanceOf('SugarBean',$meeting);
+		$this->assertInstanceOf('Meeting', $meeting);
+		$this->assertInstanceOf('SugarBean', $meeting);
 		
 		$this->assertAttributeEquals('Meetings', 'module_dir', $meeting);
 		$this->assertAttributeEquals('Meeting', 'object_name', $meeting);
@@ -32,23 +32,23 @@ 
 	}
 
 
-	public function testACLAccess(){
+	public function testACLAccess() {
 
 		$meeting = new Meeting();
 		
 		//test without recurring_source
-		$this->assertEquals(true,$meeting->ACLAccess('edit'));
-		$this->assertEquals(true,$meeting->ACLAccess('save'));
-		$this->assertEquals(true,$meeting->ACLAccess('editview'));
-		$this->assertEquals(true,$meeting->ACLAccess('delete'));
+		$this->assertEquals(true, $meeting->ACLAccess('edit'));
+		$this->assertEquals(true, $meeting->ACLAccess('save'));
+		$this->assertEquals(true, $meeting->ACLAccess('editview'));
+		$this->assertEquals(true, $meeting->ACLAccess('delete'));
 		
 
 		//test with recurring_source
 		$meeting->recurring_source = "test";
-		$this->assertEquals(false,$meeting->ACLAccess('edit'));
-		$this->assertEquals(false,$meeting->ACLAccess('save'));
-		$this->assertEquals(false,$meeting->ACLAccess('editview'));
-		$this->assertEquals(false,$meeting->ACLAccess('delete'));
+		$this->assertEquals(false, $meeting->ACLAccess('edit'));
+		$this->assertEquals(false, $meeting->ACLAccess('save'));
+		$this->assertEquals(false, $meeting->ACLAccess('editview'));
+		$this->assertEquals(false, $meeting->ACLAccess('delete'));
 			
 	}
 
@@ -71,7 +71,7 @@ 
 		$meeting->status = "Not Held";
 		$meeting->type = "Sugar";
 		$meeting->description = "test description";
-		$meeting->duration_hours = 1 ;
+		$meeting->duration_hours = 1;
 		$meeting->duration_minutes = 1;
 		$meeting->date_start = "2016-02-11 17:30:00"; 
 		$meeting->date_end = "2016-02-11 17:30:00";
@@ -87,17 +87,17 @@ 
 		
 		//test set_accept_status with User object
 		$user = new User();
-		$meeting->set_accept_status($user,"accept");
+		$meeting->set_accept_status($user, "accept");
 		
 		
 		//test set_accept_status with contact object
 		$contact = new Contact();
-		$meeting->set_accept_status($contact,"accept");
+		$meeting->set_accept_status($contact, "accept");
 		
 		
 		//test set_accept_status with Lead object
 		$lead = new Lead();
-		$meeting->set_accept_status($lead,"accept");
+		$meeting->set_accept_status($lead, "accept");
 		
 		
 		//mark all created relationships as deleted
@@ -107,7 +107,7 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$meeting->mark_deleted($meeting->id);
 		$result = $meeting->retrieve($meeting->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 
@@ -117,29 +117,29 @@ 
 		$meeting = new Meeting();
 		
 		//test without setting name
-		$this->assertEquals(Null,$meeting->get_summary_text());
+		$this->assertEquals(Null, $meeting->get_summary_text());
 		
 		//test with name set
 		$meeting->name = "test";
-		$this->assertEquals('test',$meeting->get_summary_text());
+		$this->assertEquals('test', $meeting->get_summary_text());
 		
 		
 	}
 
-    public function testcreate_export_query(){
+    public function testcreate_export_query() {
 
     	$meeting = new Meeting();
     	
     	//test with empty string params
     	$expected = "SELECT meetings.*, users.user_name as assigned_user_name   FROM meetings   LEFT JOIN users ON meetings.assigned_user_id=users.id  where meetings.deleted=0";
-    	$actual = $meeting->create_export_query('','');
-    	$this->assertSame($expected,$actual);
+    	$actual = $meeting->create_export_query('', '');
+    	$this->assertSame($expected, $actual);
     	
     		
     	//test with valid string params
     	$expected = "SELECT meetings.*, users.user_name as assigned_user_name   FROM meetings   LEFT JOIN users ON meetings.assigned_user_id=users.id  where users.user_name=\"\" AND meetings.deleted=0";
-    	$actual = $meeting->create_export_query('meetings.id','users.user_name=""');
-    	$this->assertSame($expected,$actual);
+    	$actual = $meeting->create_export_query('meetings.id', 'users.user_name=""');
+    	$this->assertSame($expected, $actual);
     	
 	}
 
@@ -148,10 +148,10 @@ 
 		$meeting = new Meeting();
 		
 		//preset required attributes
-		$meeting->assigned_user_id =1;
-		$meeting->modified_user_id =1;
-		$meeting->created_by =1;
-		$meeting->contact_id =1;
+		$meeting->assigned_user_id = 1;
+		$meeting->modified_user_id = 1;
+		$meeting->created_by = 1;
+		$meeting->contact_id = 1;
 		
 		$meeting->fill_in_additional_detail_fields();
 
@@ -166,9 +166,9 @@ 
 		$this->assertTrue(isset($meeting->duration_minutes)); 
 		$this->assertEquals(-1, $meeting->reminder_time );
 		$this->assertTrue(isset($meeting->reminder_time));	
-		$this->assertEquals(false, $meeting->reminder_checked );
+		$this->assertEquals(false, $meeting->reminder_checked);
 		$this->assertEquals(-1, $meeting->email_reminder_time );
-		$this->assertEquals(false, $meeting->email_reminder_checked );
+		$this->assertEquals(false, $meeting->email_reminder_checked);
 		$this->assertEquals("Accounts", $meeting->parent_type);
 		
 	}
@@ -184,7 +184,7 @@ 
 		$meeting->contact_name = "test";
 		$meeting->parent_name = "Account";
 		
-		$expected = array (
+		$expected = array(
 					  'DELETED' => 0,
 					  'PARENT_TYPE' => 'Accounts',
 					  'STATUS' => 'Planned',
@@ -233,24 +233,24 @@ 
 		$meeting->status = "Not Held";
 		$meeting->type = "Sugar";
 		$meeting->description = "test description";
-		$meeting->duration_hours = 1 ;
+		$meeting->duration_hours = 1;
 		$meeting->duration_minutes = 1;
 		$meeting->date_start = "2016-02-11 17:30:00"; 
 		$meeting->date_end = "2016-02-11 17:30:00";
 
 		$result = $meeting->set_notification_body(new Sugar_Smarty(), $meeting);
 		
-		$this->assertEquals($meeting->name ,$result->_tpl_vars['MEETING_SUBJECT']);
-		$this->assertEquals($meeting->status ,$result->_tpl_vars['MEETING_STATUS']);
-		$this->assertEquals("SuiteCRM" ,$result->_tpl_vars['MEETING_TYPE']);
-		$this->assertEquals($meeting->duration_hours ,$result->_tpl_vars['MEETING_HOURS']);
-		$this->assertEquals($meeting->duration_minutes ,$result->_tpl_vars['MEETING_MINUTES']);								
-		$this->assertEquals($meeting->description ,$result->_tpl_vars['MEETING_DESCRIPTION']);
+		$this->assertEquals($meeting->name, $result->_tpl_vars['MEETING_SUBJECT']);
+		$this->assertEquals($meeting->status, $result->_tpl_vars['MEETING_STATUS']);
+		$this->assertEquals("SuiteCRM", $result->_tpl_vars['MEETING_TYPE']);
+		$this->assertEquals($meeting->duration_hours, $result->_tpl_vars['MEETING_HOURS']);
+		$this->assertEquals($meeting->duration_minutes, $result->_tpl_vars['MEETING_MINUTES']);								
+		$this->assertEquals($meeting->description, $result->_tpl_vars['MEETING_DESCRIPTION']);
 	
 	}
 	
 
-	public function testcreate_notification_email(){
+	public function testcreate_notification_email() {
 
 		$meeting = new Meeting();	
 		
@@ -268,7 +268,7 @@ 
 	}
 	
 
-	public function testsend_assignment_notifications(){
+	public function testsend_assignment_notifications() {
 
 		$meeting = new Meeting();
 		
@@ -343,9 +343,9 @@ 
 
 		$meeting = new Meeting();
 		
-		$expected = array("MAIN"=>"a", "PARENT"=>"a", "CONTACT"=>"a" );
+		$expected = array("MAIN"=>"a", "PARENT"=>"a", "CONTACT"=>"a");
 		$actual = $meeting->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -367,7 +367,7 @@ 
 	}
 
 
-	public function testafterImportSave(){
+	public function testafterImportSave() {
 		
 		require_once("data/Link.php");
 		
@@ -397,7 +397,7 @@ 
 				
 	}
 
-    public function testgetDefaultStatus(){
+    public function testgetDefaultStatus() {
 
     	$meeting = new Meeting();
     	$result = $meeting->getDefaultStatus();
@@ -407,16 +407,16 @@ 
 
     
 
-    public function testgetMeetingsExternalApiDropDown(){
+    public function testgetMeetingsExternalApiDropDown() {
     	
     	$actual = getMeetingsExternalApiDropDown();
-    	$expected= array( "Sugar"=>"SuiteCRM");
+    	$expected = array("Sugar"=>"SuiteCRM");
     	$this->assertSame($expected, $actual);
     	
     }
     
     
-    public function testgetMeetingTypeOptions(){
+    public function testgetMeetingTypeOptions() {
     	
     	global $dictionary, $app_list_strings;
     	

tests/tests/modules/Project/ProjectTest.php

Spacing +15 added lines, -15 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$project = new Project();
 			
-		$this->assertInstanceOf('Project',$project);
-		$this->assertInstanceOf('SugarBean',$project);
+		$this->assertInstanceOf('Project', $project);
+		$this->assertInstanceOf('SugarBean', $project);
 		
 		
 		$this->assertAttributeEquals('project', 'table_name', $project);
@@ -61,7 +61,7 @@ 
     	
     	$project = new Project();
 
-    	$project->id =1;
+    	$project->id = 1;
     	$_REQUEST['relate_id'] = 2;
     	$_REQUEST['relate_to'] = "contacts";
     	
@@ -97,16 +97,16 @@ 
 		$project = new Project();
 		
 		//test without setting name
-		$this->assertEquals(Null,$project->get_summary_text());
+		$this->assertEquals(Null, $project->get_summary_text());
 		
 		//test with name set
 		$project->name = "test";
-		$this->assertEquals('test',$project->get_summary_text());
+		$this->assertEquals('test', $project->get_summary_text());
 
 	}
 
 
-	public function testbuild_generic_where_clause ()
+	public function testbuild_generic_where_clause()
 	{
 		
 		$project = new Project();
@@ -114,13 +114,13 @@ 
 		//test with empty string params
 		$expected = "project.name LIKE '%%'";
 		$actual = $project->build_generic_where_clause('');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 		 
 		//test with valid string params
 		$expected = "project.name LIKE '%%'";
 		$actual = $project->build_generic_where_clause('test');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		 
 	}
 
@@ -131,7 +131,7 @@ 
 		$project->user_name = "tes user";
 		$project->assigned_user_name = "test assigned user";
 		
-		$expected = array (
+		$expected = array(
 			'DELETED' => '0',
 			'ASSIGNED_USER_NAME' => 'test assigned user',
 			'USER_NAME' => 'tes user'
@@ -143,7 +143,7 @@ 
 		
 	}
 
-	public function testbean_implements(){
+	public function testbean_implements() {
 
 		$project = new Project();
 		
@@ -159,18 +159,18 @@ 
     	
     	//test with empty string params
     	$expected = "SELECT\n				project.*,\n                users.user_name as assigned_user_name  FROM project  LEFT JOIN users\n                   	ON project.assigned_user_id=users.id where  project.deleted=0 ";
-    	$actual = $project->create_export_query('','');
-    	$this->assertSame($expected,$actual);
+    	$actual = $project->create_export_query('', '');
+    	$this->assertSame($expected, $actual);
      		
     	
     	//test with valid string params
     	$expected = "SELECT\n				project.*,\n                users.user_name as assigned_user_name  FROM project  LEFT JOIN users\n                   	ON project.assigned_user_id=users.id where (users.user_name) AND  project.deleted=0  ORDER BY project.id";
-    	$actual = $project->create_export_query('project.id','users.user_name');
-    	$this->assertSame($expected,$actual);
+    	$actual = $project->create_export_query('project.id', 'users.user_name');
+    	$this->assertSame($expected, $actual);
     	
     }
 
-	public function testgetAllProjectTasks(){
+	public function testgetAllProjectTasks() {
 
 		$project = new Project();
 		

tests/tests/modules/vCals/vCalTest.php

Spacing +38 added lines, -38 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$vcal = new vCal();
 		
-		$this->assertInstanceOf('vCal',$vcal);
-		$this->assertInstanceOf('SugarBean',$vcal);
+		$this->assertInstanceOf('vCal', $vcal);
+		$this->assertInstanceOf('SugarBean', $vcal);
 			
 		$this->assertAttributeEquals('vcals', 'table_name', $vcal);
 		$this->assertAttributeEquals('vCals', 'module_dir', $vcal);
@@ -28,11 +28,11 @@ 
 		$vcal = new vCal();
 		
 		//test without setting name
-		$this->assertEquals(Null,$vcal->get_summary_text());
+		$this->assertEquals(Null, $vcal->get_summary_text());
 		
 		//test with name set
 		$vcal->name = "test";
-		$this->assertEquals('',$vcal->get_summary_text());
+		$this->assertEquals('', $vcal->get_summary_text());
 
 	}
 
@@ -94,7 +94,7 @@ 
 		$user_bean = new User("1");
 		
 		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\n";
-		$expectedEnd ="END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
+		$expectedEnd = "END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		
 		$result = $vcal->get_freebusy_lines_cache($user_bean);
 		
@@ -116,7 +116,7 @@ 
 		$end_date_time = $now_date_time->get("tomorrow");
 		
 		$result = $vcal->create_sugar_freebusy($user_bean, $start_date_time, $end_date_time);
-		$this->assertGreaterThanOrEqual(0,strlen($result));
+		$this->assertGreaterThanOrEqual(0, strlen($result));
 		
 	}
 
@@ -127,7 +127,7 @@ 
 		$user_focus = new User("1");
 		
 		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\n";
-		$expectedEnd ="END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
+		$expectedEnd = "END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		
 		$result = $vcal->get_vcal_freebusy($user_focus);
 		
@@ -175,13 +175,13 @@ 
 	{
 		
 		//test with short strings
-		$result = vCal::fold_ical_lines("testkey","testvalue");
-		$this->assertEquals("testkey:testvalue",$result);
+		$result = vCal::fold_ical_lines("testkey", "testvalue");
+		$this->assertEquals("testkey:testvalue", $result);
 	
 		
 		//test with longer strings
-		$expected =  "testkey11111111111111111111111111111111111111111111111111111111111111111111\r\n	11111111111111111111111111111111:testvalue11111111111111111111111111111111\r\n	11111111111111111111111111111111111111111111111111111111111111111111";
-		$result = vCal::fold_ical_lines("testkey" . str_repeat("1",100),"testvalue" . str_repeat("1",100));
+		$expected = "testkey11111111111111111111111111111111111111111111111111111111111111111111\r\n	11111111111111111111111111111111:testvalue11111111111111111111111111111111\r\n	11111111111111111111111111111111111111111111111111111111111111111111";
+		$result = vCal::fold_ical_lines("testkey".str_repeat("1", 100), "testvalue".str_repeat("1", 100));
 		$this->assertEquals();
 	
 	}
@@ -192,19 +192,19 @@ 
 
 		$iCalString = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\nDTSTART:2016-01-09 00:00:00\r\nDTEND:2016-03-09 00:00:00\r\nDTSTAMP:2016-01-10 11:07:15\r\nEND:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		$expected = array(
-						array ( 'BEGIN','VCALENDAR'),
-						array ( 'VERSION','2.0'),
-						array ( 'PRODID','-//SugarCRM//SugarCRM Calendar//EN'),
-						array ( 'BEGIN', 'VFREEBUSY' ),
-						array ( 'ORGANIZER;CN= ', 'VFREEBUSY'  ),
-						array ( 'DTSTART', '2016-01-09 00:00:00' ),
-						array ( 'DTEND', '2016-03-09 00:00:00' ),
-						array ( 'DTSTAMP', '2016-01-10 11:07:15' ),
-						array ( 'END', 'VFREEBUSY' ),
-						array ( 'END', 'VCALENDAR' ),
+						array('BEGIN', 'VCALENDAR'),
+						array('VERSION', '2.0'),
+						array('PRODID', '-//SugarCRM//SugarCRM Calendar//EN'),
+						array('BEGIN', 'VFREEBUSY'),
+						array('ORGANIZER;CN= ', 'VFREEBUSY'),
+						array('DTSTART', '2016-01-09 00:00:00'),
+						array('DTEND', '2016-03-09 00:00:00'),
+						array('DTSTAMP', '2016-01-10 11:07:15'),
+						array('END', 'VFREEBUSY'),
+						array('END', 'VCALENDAR'),
 					);
 		$actual = vCal::create_ical_array_from_string($iCalString);
-		$this->assertSame($expected, $actual );
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -214,38 +214,38 @@ 
 
 		$expected = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\nDTSTART:2016-01-09 00:00:00\r\nDTEND:2016-03-09 00:00:00\r\nDTSTAMP:2016-01-10 11:07:15\r\nEND:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		$iCalArray = array(
-				array ( 'BEGIN','VCALENDAR'),
-				array ( 'VERSION','2.0'),
-				array ( 'PRODID','-//SugarCRM//SugarCRM Calendar//EN'),
-				array ( 'BEGIN', 'VFREEBUSY' ),
-				array ( 'ORGANIZER;CN= ', 'VFREEBUSY'  ),
-				array ( 'DTSTART', '2016-01-09 00:00:00' ),
-				array ( 'DTEND', '2016-03-09 00:00:00' ),
-				array ( 'DTSTAMP', '2016-01-10 11:07:15' ),
-				array ( 'END', 'VFREEBUSY' ),
-				array ( 'END', 'VCALENDAR' ),
+				array('BEGIN', 'VCALENDAR'),
+				array('VERSION', '2.0'),
+				array('PRODID', '-//SugarCRM//SugarCRM Calendar//EN'),
+				array('BEGIN', 'VFREEBUSY'),
+				array('ORGANIZER;CN= ', 'VFREEBUSY'),
+				array('DTSTART', '2016-01-09 00:00:00'),
+				array('DTEND', '2016-03-09 00:00:00'),
+				array('DTSTAMP', '2016-01-10 11:07:15'),
+				array('END', 'VFREEBUSY'),
+				array('END', 'VCALENDAR'),
 		);
 		$actual = vCal::create_ical_string_from_array($iCalArray);
-		$this->assertSame($expected, $actual );
+		$this->assertSame($expected, $actual);
 		
 	}
 
 
 	public function testescape_ical_chars()
 	{
-		$this->assertSame('',vCal::escape_ical_chars(''));
-		$this->assertSame('\;\,',vCal::escape_ical_chars(';,'));
+		$this->assertSame('', vCal::escape_ical_chars(''));
+		$this->assertSame('\;\,', vCal::escape_ical_chars(';,'));
 	}
 
 
 	public function testunescape_ical_chars()
 	{
-		$this->assertSame('',vCal::unescape_ical_chars(''));
+		$this->assertSame('', vCal::unescape_ical_chars(''));
 		$this->assertSame("; , \\", vCal::unescape_ical_chars('\\; \\, \\\\'));	
 	}
 
 
-	public function testget_ical_event(){
+	public function testget_ical_event() {
 
 		$user = new User(1);
 		$meeting = new Meeting();
@@ -260,7 +260,7 @@ 
 		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VEVENT\r\nUID:1\r\nORGANIZED;CN=:\r\nDTSTART:20160211T173000Z\r\nDTEND:20160211T173000Z\r\n"; 
 		$expectedEnd = "\r\nSUMMARY:test\r\nLOCATION:test location\r\nDESCRIPTION:test description\r\nEND:VEVENT\r\nEND:VCALENDAR\r\n";
 
-		$result = vCal::get_ical_event($meeting ,$user);
+		$result = vCal::get_ical_event($meeting, $user);
 		
 		$this->assertStringStartsWith($expectedStart, $result);
 		$this->assertStringEndsWith($expectedEnd, $result);

tests/tests/modules/SugarFeed/SugarFeedTest.php

Spacing +21 added lines, -21 removed lines

@@ -1,18 +1,18 @@ 
 <?PHP
 
-class SugarFeedTest extends PHPUnit_Framework_TestCase  {
+class SugarFeedTest extends PHPUnit_Framework_TestCase {
 
 	
-	public function testSugarFeed(){
+	public function testSugarFeed() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		
 		//execute the contructor and check for the Object type and  attributes
 		$sugarFeed = new SugarFeed();
 		
-		$this->assertInstanceOf('SugarFeed',$sugarFeed);
-		$this->assertInstanceOf('Basic',$sugarFeed);
-		$this->assertInstanceOf('SugarBean',$sugarFeed);
+		$this->assertInstanceOf('SugarFeed', $sugarFeed);
+		$this->assertInstanceOf('Basic', $sugarFeed);
+		$this->assertInstanceOf('SugarBean', $sugarFeed);
 			
 		$this->assertAttributeEquals('sugarfeed', 'table_name', $sugarFeed);
 		$this->assertAttributeEquals('SugarFeed', 'module_dir', $sugarFeed);
@@ -30,7 +30,7 @@ 
 		//test activateModuleFeed method
 		SugarFeed::activateModuleFeed('Accounts');
 		$admin->retrieveSettings('sugarfeed');
-		$this->assertEquals(1,$admin->settings['sugarfeed_module_Accounts']);
+		$this->assertEquals(1, $admin->settings['sugarfeed_module_Accounts']);
 		
 
 		//test disableModuleFeed method
@@ -64,7 +64,7 @@ 
     	
     	
     	//test with valid module
-    	$expected = array( "CaseFeed.php"=> "modules/Cases/SugarFeeds/CaseFeed.php");
+    	$expected = array("CaseFeed.php"=> "modules/Cases/SugarFeeds/CaseFeed.php");
     	$result = SugarFeed::getModuleFeedFiles('Cases');
     	$this->assertEquals($expected, $result);
     	    	
@@ -74,7 +74,7 @@ 
 
     	$result = SugarFeed::getActiveFeedModules();
     	
-    	$expected = array (
+    	$expected = array(
     			'UserFeed' => 'UserFeed',
     			'Cases' => 'Cases',
     			'Contacts' => 'Contacts',
@@ -89,7 +89,7 @@ 
     public function testgetAllFeedModules( ) {
 
     	$result = SugarFeed::getAllFeedModules();
-    	$expected = array (
+    	$expected = array(
 					  'UserFeed' => 'UserFeed',
 					  'Cases' => 'Cases',
 					  'Contacts' => 'Contacts',
@@ -108,7 +108,7 @@ 
     	$lead->id = 1;
     	$lead->assigned_user_id = 1;
     	
-    	SugarFeed::pushFeed2("some text 2", $lead,'Link', 'some url');
+    	SugarFeed::pushFeed2("some text 2", $lead, 'Link', 'some url');
     	
     	//retrieve newly created bean
     	$sugarFeed = new SugarFeed();
@@ -123,10 +123,10 @@ 
     	
     }
 
-	public function testpushFeed(){
+	public function testpushFeed() {
 
 		
-		SugarFeed::pushFeed("some text", 'SugarFeed', 1, 1,'Link', 'some url');
+		SugarFeed::pushFeed("some text", 'SugarFeed', 1, 1, 'Link', 'some url');
 		
 		
 		//retrieve newly created bean
@@ -153,7 +153,7 @@ 
 		$sugarFeed = new SugarFeed();
 		
 		$actual = $sugarFeed->fetchReplies(array('ID'=>'1'));
-		$this->assertGreaterThan(0,strlen($actual));
+		$this->assertGreaterThan(0, strlen($actual));
 		   
 	}
 	
@@ -161,7 +161,7 @@ 
 
     	$result = SugarFeed::getLinkTypes();
     	
-    	$expected = array (
+    	$expected = array(
     			'Image' => 'Image',
     			'Link' => 'Link',
     			'YouTube' => 'YouTube',
@@ -178,19 +178,19 @@ 
     	
     	//test with LinkType Image
     	$result = SugarFeed::getLinkClass('Image');
-    	$this->assertInstanceOf('FeedLinkHandlerImage',$result);
+    	$this->assertInstanceOf('FeedLinkHandlerImage', $result);
     	
     	//test with LinkType Link
     	$result = SugarFeed::getLinkClass('Link');
-    	$this->assertInstanceOf('FeedLinkHandlerLink',$result);
+    	$this->assertInstanceOf('FeedLinkHandlerLink', $result);
 
     	//test with LinkType YouTube
     	$result = SugarFeed::getLinkClass('YouTube');
-    	$this->assertInstanceOf('FeedLinkHandlerYoutube',$result);
+    	$this->assertInstanceOf('FeedLinkHandlerYoutube', $result);
         	 	
     }
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 		
 		$sugarFeed = new SugarFeed();
 		
@@ -209,19 +209,19 @@ 
     }
 
 
-    public function testparseMessage(){
+    public function testparseMessage() {
 
     	//test with a string with no links
     	$html = "some text with no urls";
     	$result = SugarFeed::parseMessage($html);
-    	$this->assertEquals($html ,$result);
+    	$this->assertEquals($html, $result);
     	
     	
     	//test with a string with links
     	$html = "some text http://www.url.com with no urls";
     	$expected = "some text <a href='http://www.url.com' target='_blank'>http://www.url.com</a> with no urls";
     	$result = SugarFeed::parseMessage($html);
-    	$this->assertEquals($expected ,$result);
+    	$this->assertEquals($expected, $result);
     	
     }