adamjakab / SuiteCRM

tests/tests/modules/InboundEmail/InboundEmailTest.php

Spacing +107 added lines, -107 removed lines

@@ -15,8 +15,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$inboundEmail = new InboundEmail();
 		
-		$this->assertInstanceOf('InboundEmail',$inboundEmail);
-		$this->assertInstanceOf('SugarBean',$inboundEmail);
+		$this->assertInstanceOf('InboundEmail', $inboundEmail);
+		$this->assertInstanceOf('SugarBean', $inboundEmail);
 			
 		$this->assertAttributeEquals('InboundEmail', 'module_dir', $inboundEmail);
 		$this->assertAttributeEquals('InboundEmail', 'object_name', $inboundEmail);
@@ -132,11 +132,11 @@ 
 		$inboundEmail = new InboundEmail();
 	
 		
-		$result = $inboundEmail->getSingularRelatedId("test","inbound_email");
+		$result = $inboundEmail->getSingularRelatedId("test", "inbound_email");
 		$this->assertEquals(false, $result);
 		
 		
-		$result = $inboundEmail->getSingularRelatedId("invalid test","inbound_email");
+		$result = $inboundEmail->getSingularRelatedId("invalid test", "inbound_email");
 		$this->assertEquals(null, $result);
 		
 		
@@ -181,8 +181,8 @@ 
 		 
 		$this->assertTrue(is_array($result));
 		
-		foreach($result as $ie){		
-			$this->assertInstanceOf('InboundEmail',$ie);
+		foreach ($result as $ie) {		
+			$this->assertInstanceOf('InboundEmail', $ie);
 		}
 		
 	}
@@ -196,8 +196,8 @@ 
 			
 		$this->assertTrue(is_array($result));
 	
-		foreach($result as $ie){
-			$this->assertInstanceOf('InboundEmail',$ie);
+		foreach ($result as $ie) {
+			$this->assertInstanceOf('InboundEmail', $ie);
 		}
 	
 	}
@@ -211,8 +211,8 @@ 
 			
 		$this->assertTrue(is_array($result));
 		
-		foreach($result as $ie){
-			$this->assertInstanceOf('InboundEmail',$ie);
+		foreach ($result as $ie) {
+			$this->assertInstanceOf('InboundEmail', $ie);
 		}	
 		
 	}
@@ -244,8 +244,8 @@ 
 		$result = $inboundEmail->search($id);
 		
 		$this->assertTrue(is_array($result));
-		$this->assertEquals("Search Results" ,$result["mbox"]);
-		$this->assertEquals($id ,$result["ieId"]);
+		$this->assertEquals("Search Results", $result["mbox"]);
+		$this->assertEquals($id, $result["ieId"]);
 		 	
 	}	
 
@@ -259,11 +259,11 @@ 
 	
 		//execute he method and verify attributes
 		$inboundEmail->saveMailBoxFolders("INBOX,TRASH");
-		$this->assertEquals(array("INBOX","TRASH") , $inboundEmail->mailboxarray);
+		$this->assertEquals(array("INBOX", "TRASH"), $inboundEmail->mailboxarray);
 		
 		//retrieve it back and verify the updates
 		$inboundEmail->retrieve($id);
-		$this->assertEquals("INBOX,TRASH" , $inboundEmail->mailbox);
+		$this->assertEquals("INBOX,TRASH", $inboundEmail->mailbox);
 	
 	}
 	
@@ -279,7 +279,7 @@ 
 	
 		//retrieve it back and verify the updates
 		$inboundEmail->retrieve($id);
-		$this->assertEquals("TEST" , $inboundEmail->mailbox);
+		$this->assertEquals("TEST", $inboundEmail->mailbox);
 		
 	}
 	
@@ -292,7 +292,7 @@ 
 		$inboundEmail->mark_deleted($id);
 		
 		$result = $inboundEmail->retrieve($id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 
@@ -304,7 +304,7 @@ 
 		$inboundEmail->hardDelete($id);
 		
 		$result = $inboundEmail->retrieve($id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 	
@@ -346,12 +346,12 @@ 
 	
 		//test with array having common element
 		$result = $inboundEmail->filterMailBoxFromRaw(array("mailbox1", "mailbox2", "mailbox3"), array("mailbox1"));
-		$this->assertSame(array("mailbox1"), $result );
+		$this->assertSame(array("mailbox1"), $result);
 	
 	
 		//test with array having nothing common
-		$result = $inboundEmail->filterMailBoxFromRaw( array("mailbox1", "mailbox2"), array("mailbox4"));
-		$this->assertSame(array(), $result );
+		$result = $inboundEmail->filterMailBoxFromRaw(array("mailbox1", "mailbox2"), array("mailbox4"));
+		$this->assertSame(array(), $result);
 	
 	}
 	
@@ -413,7 +413,7 @@ 
 		$overview->subject = "subject";
 		$overview->from = "from";
 		$overview->fromaddr = "from@email.com";
-		$overview->to  = "to";
+		$overview->to = "to";
 		$overview->toaddr = "to@email.com";
 		$overview->size = 0;
 		$overview->message_id = 1;
@@ -439,7 +439,7 @@ 
 	
 	
 		//test wih default protocol
-		$result = $inboundEmail->getCacheValueForUIDs("INBOX", array(1,2,3,4,5));
+		$result = $inboundEmail->getCacheValueForUIDs("INBOX", array(1, 2, 3, 4, 5));
 	
 		$this->assertTrue(is_array($result));
 		$this->assertTrue(is_array($result['uids']));
@@ -449,7 +449,7 @@ 
 	
 		//test wih pop3 protocol
 		$inboundEmail->protocol = 'pop3';
-		$result = $inboundEmail->getCacheValueForUIDs("INBOX", array(1,2,3,4,5));
+		$result = $inboundEmail->getCacheValueForUIDs("INBOX", array(1, 2, 3, 4, 5));
 	
 		$this->assertTrue(is_array($result));
 		$this->assertTrue(is_array($result['uids']));
@@ -508,7 +508,7 @@ 
 		$ret = $inboundEmail->getCacheValue("INBOX");
 
 		//use the cache values array as parameter and verify that it returns an array
-		$result = $inboundEmail->displayFetchedSortedListXML($ret,"INBOX");
+		$result = $inboundEmail->displayFetchedSortedListXML($ret, "INBOX");
 		$this->assertTrue(is_array($result));
 	
 		
@@ -580,7 +580,7 @@ 
     	$inboundEmail->id = 1;
     	
     	//execute the method to populate answered field
-    	$inboundEmail->mark_answered(1,'pop3');
+    	$inboundEmail->mark_answered(1, 'pop3');
     	
     	//retrieve back to verify the records updated
     	$result = $inboundEmail->getCacheValue("INBOX");
@@ -597,7 +597,7 @@ 
     	 
     	$inboundEmail->id = 1;
     	
-    	$result = $inboundEmail->pop3_shiftCache(array('1'=>'1'),array('1'));
+    	$result = $inboundEmail->pop3_shiftCache(array('1'=>'1'), array('1'));
     	
     	//retrieve back to verify the records updated
     	$result = $inboundEmail->getCacheValue("INBOX");
@@ -915,7 +915,7 @@ 
 		$inboundEmail->mailbox = "INBOX,OUTBOX";
 		
 		$result = $inboundEmail->checkEmailOneMailbox("INBOX");
-		$this->assertEquals(1,$result);
+		$this->assertEquals(1, $result);
 				
 	}
 
@@ -1144,11 +1144,11 @@ 
 		//execute the method and test if it works and does not throws an exception.
 		try {
 			
-			$inboundEmail->markEmails('1','unread');
-			$inboundEmail->markEmails('1','read');
-			$inboundEmail->markEmails('1','flagged');
-			$inboundEmail->markEmails('1','unflagged');
-			$inboundEmail->markEmails('1','answered');
+			$inboundEmail->markEmails('1', 'unread');
+			$inboundEmail->markEmails('1', 'read');
+			$inboundEmail->markEmails('1', 'flagged');
+			$inboundEmail->markEmails('1', 'unflagged');
+			$inboundEmail->markEmails('1', 'answered');
 			
 			$this->assertTrue(true);
 		}
@@ -1175,7 +1175,7 @@ 
 
 		$inboundEmail = new InboundEmail();
 		
-		$result = $inboundEmail->saveNewFolder("TEST","INBOX");
+		$result = $inboundEmail->saveNewFolder("TEST", "INBOX");
 		
 		$this->assertEquals(false, $result);
 	}
@@ -1239,7 +1239,7 @@ 
 		
 		$_REQUEST['ie_name'] = "test";
 		$_REQUEST['ie_status'] = "Active";
-		$_REQUEST['server_url'] ="";
+		$_REQUEST['server_url'] = "";
 		$_REQUEST['email_user'] = "test";
 		$_REQUEST['email_password'] = "test_pass";
 		$_REQUEST['mailbox'] = "INBOX";
@@ -1307,7 +1307,7 @@ 
 	
 	
 		//test with valid user id
-		$user->id= 1;
+		$user->id = 1;
 		$result = $inboundEmail->getUserPersonalAccountCount($user);
 		$this->assertGreaterThan(0, $result);
 	
@@ -1329,8 +1329,8 @@ 
 		$result = $inboundEmail->retrieveByGroupFolderId("");
 		
 		$this->assertTrue(is_array($result));	
-		foreach($result as $ie){
-			$this->assertInstanceOf('InboundEmail',$ie);
+		foreach ($result as $ie) {
+			$this->assertInstanceOf('InboundEmail', $ie);
 		}
 
 		
@@ -1389,7 +1389,7 @@ 
 		
 		$this->assertEquals(false, $inboundEmail->findOptimumSettings(true));
 		
-		$this->assertEquals(false, $inboundEmail->findOptimumSettings(false,'test','test','','','INBOX'));
+		$this->assertEquals(false, $inboundEmail->findOptimumSettings(false, 'test', 'test', '', '', 'INBOX'));
 		
 		
 		
@@ -1401,12 +1401,12 @@ 
 		
 		//test without setting session key
 		$result = $inboundEmail->getSessionConnectionString("mail.google.com", "test", 22, "IMAP");
-		$this->assertEquals("",$result);
+		$this->assertEquals("", $result);
 				
 		//test with session key set
 		$_SESSION["mail.google.comtest22IMAP"] = "test connection string";
 		$result = $inboundEmail->getSessionConnectionString("mail.google.com", "test", 22, "IMAP");
-		$this->assertEquals("test connection string",$result);
+		$this->assertEquals("test connection string", $result);
 		
 	}
 
@@ -1425,13 +1425,13 @@ 
 		
 		//test without setting session key
 		$result = $inboundEmail->getSessionInboundDelimiterString("mail.google.com", "test", 22, "IMAP");
-		$this->assertEquals("",$result);
+		$this->assertEquals("", $result);
 		
 		
 		//test with session key set
 		$_SESSION["mail.google.comtest22IMAPdelimiter"] = "delimit string";
 		$result = $inboundEmail->getSessionInboundDelimiterString("mail.google.com", "test", 22, "IMAP");
-		$this->assertEquals("delimit string",$result);
+		$this->assertEquals("delimit string", $result);
 				
 	}
 
@@ -1451,13 +1451,13 @@ 
 		
 		//test without setting session key
 		$result = $inboundEmail->getSessionInboundFoldersString("mail.google.com", "test", 22, "IMAP");
-		$this->assertEquals("",$result);
+		$this->assertEquals("", $result);
 		
 		
 		//test with session key set
 		$_SESSION["mail.google.comtest22IMAPfoldersList"] = "foldersList string";
 		$result = $inboundEmail->getSessionInboundFoldersString("mail.google.com", "test", 22, "IMAP");
-		$this->assertEquals("foldersList string",$result);
+		$this->assertEquals("foldersList string", $result);
 		
 	}
 
@@ -1502,13 +1502,13 @@ 
 		
 		$inboundEmail = new InboundEmail();
 		
-		$inboundEmail->group_id =1;
+		$inboundEmail->group_id = 1;
 		
 		$result = $inboundEmail->getGroupsWithSelectOptions();
 		$this->assertEquals("", $result);
 		
 		$expected = "\n<OPTION value='0'>1</OPTION>\n<OPTION selected value='1'>2</OPTION>\n<OPTION value='2'>3</OPTION>";
-		$result = $inboundEmail->getGroupsWithSelectOptions(array(1,2,3));
+		$result = $inboundEmail->getGroupsWithSelectOptions(array(1, 2, 3));
 		$this->assertEquals($expected, $result);
 		//var_dump($result);
 		
@@ -1533,7 +1533,7 @@ 
 		
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			$result = $inboundEmail->handleAutoresponse($email,$contactAddr);
+			$result = $inboundEmail->handleAutoresponse($email, $contactAddr);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -1603,8 +1603,8 @@ 
 		$this->assertEquals(false, $result);
 		
 		//test with attributes set
-		$inboundEmail->mailbox_type = 'createcase' ;
-		$inboundEmail->groupfolder_id =1;
+		$inboundEmail->mailbox_type = 'createcase';
+		$inboundEmail->groupfolder_id = 1;
 		
 		$result = $inboundEmail->isMailBoxTypeCreateCase();
 		$this->assertEquals(true, $result);
@@ -1666,7 +1666,7 @@ 
 		$inboundEmail = new InboundEmail();
 		
 		$parts = array(
-					(Object) array('encoding'=>'utf-8','parts'=> array( (Object) array('encoding'=>'utf-8','parts'=>array((Object) array('encoding'=>'utf-8','parts'=>'dummy parts 2')))))			
+					(Object)array('encoding'=>'utf-8', 'parts'=> array((Object)array('encoding'=>'utf-8', 'parts'=>array((Object)array('encoding'=>'utf-8', 'parts'=>'dummy parts 2')))))			
 				);
 		
 		//$result = $inboundEmail->getEncodingFromBreadCrumb("1.2.3", $parts);
@@ -1683,7 +1683,7 @@ 
 		$inboundEmail = new InboundEmail();
 		
 		$parts = array(
-				(Object) array('ifparameters'=>1, 'attribute'=>'charset', 'value'=>'test','parts'=> array( (Object) array('ifparameters'=>1, 'attribute'=>'charset','value'=>'test' ,'parts'=>array((Object) array('ifparameters'=>1, 'attribute'=>'charset', 'value'=>'test','parts'=>'dummy parts 2')))))
+				(Object)array('ifparameters'=>1, 'attribute'=>'charset', 'value'=>'test', 'parts'=> array((Object)array('ifparameters'=>1, 'attribute'=>'charset', 'value'=>'test', 'parts'=>array((Object)array('ifparameters'=>1, 'attribute'=>'charset', 'value'=>'test', 'parts'=>'dummy parts 2')))))
 		);
 		
 		$result = $inboundEmail->getCharsetFromBreadCrumb("1.2.3", $parts);
@@ -1705,7 +1705,7 @@ 
 		
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			$result = $inboundEmail->getMessageTextFromSingleMimePart(1,1,$structure);
+			$result = $inboundEmail->getMessageTextFromSingleMimePart(1, 1, $structure);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -1774,7 +1774,7 @@ 
 		$inboundEmail = new InboundEmail();
 		
 		
-		$expected = array (
+		$expected = array(
 						  'From' => 'Media Temple user (mt.kb.user@gmail.com)',
 						  'Subject' => 'article: How to Trace a Email',
 						  'Date' => 'January 25, 2011 3:30:58 PM PDT',
@@ -1801,12 +1801,12 @@ 
 		$inboundEmail = new InboundEmail();
 		
 		//test with default
-		$result = $inboundEmail->handleCharsetTranslation("sample text",'default');
+		$result = $inboundEmail->handleCharsetTranslation("sample text", 'default');
 		$this->assertEquals("sample text", $result);
 		
 		
 		//test with ISO-8859-8
-		$result = $inboundEmail->handleCharsetTranslation("sample text",'ISO-8859-8');
+		$result = $inboundEmail->handleCharsetTranslation("sample text", 'ISO-8859-8');
 		$this->assertEquals("sample text", $result);
 			
 	}
@@ -1933,7 +1933,7 @@ 
 		
 		$result = $inboundEmail->getNoteBeanForAttachment("1");
 		
-		$this->assertInstanceOf('Note',$result);
+		$this->assertInstanceOf('Note', $result);
 		$this->assertAttributeEquals('1', 'parent_id', $result);
 		$this->assertAttributeEquals('Emails', 'parent_type', $result);
 		
@@ -1946,8 +1946,8 @@ 
 		$inboundEmail = new InboundEmail();
 
 		//test with filename attribute
-		$part = (Object) array('dparameters'=> array((Object) array('attribute'=>'filename','value'=>'test1.txt'),(Object) array('attribute'=>'filename','value'=>'test2.txt')),
-								'parameters'=> array((Object) array('attribute'=>'name','value'=>'test1'),(Object) array('attribute'=>'name','value'=>'test2'))
+		$part = (Object)array('dparameters'=> array((Object)array('attribute'=>'filename', 'value'=>'test1.txt'), (Object)array('attribute'=>'filename', 'value'=>'test2.txt')),
+								'parameters'=> array((Object)array('attribute'=>'name', 'value'=>'test1'), (Object)array('attribute'=>'name', 'value'=>'test2'))
 								);
 
 		$result = $inboundEmail->retrieveAttachmentNameFromStructure($part);
@@ -1956,8 +1956,8 @@ 
 		
 		
 		//test with no filename attribute
-		$part = (Object) array('dparameters'=> array((Object) array('attribute'=>'name','value'=>'test1.txt')),
-								'parameters'=> array((Object) array('attribute'=>'name','value'=>'test1'),(Object) array('attribute'=>'name','value'=>'test2'))
+		$part = (Object)array('dparameters'=> array((Object)array('attribute'=>'name', 'value'=>'test1.txt')),
+								'parameters'=> array((Object)array('attribute'=>'name', 'value'=>'test1'), (Object)array('attribute'=>'name', 'value'=>'test2'))
 				
 								);
 
@@ -1977,7 +1977,7 @@ 
 				
 		$inboundEmail = new InboundEmail();
 		
-		$part = (Object) array("disposition" => "multipart", "subtype" => 10);
+		$part = (Object)array("disposition" => "multipart", "subtype" => 10);
 		
 		//execute the method and test if it works and does not throws an exception.
 		try {
@@ -1996,8 +1996,8 @@ 
 		$inboundEmail = new InboundEmail();
 		
 		$this->assertEquals("test", $inboundEmail->handleTranserEncoding("test"));
-		$this->assertEquals("test", $inboundEmail->handleTranserEncoding("dGVzdA==",3));
-		$this->assertEquals("test", $inboundEmail->handleTranserEncoding("test",4));
+		$this->assertEquals("test", $inboundEmail->handleTranserEncoding("dGVzdA==", 3));
+		$this->assertEquals("test", $inboundEmail->handleTranserEncoding("test", 4));
 
 	}
 
@@ -2024,7 +2024,7 @@ 
 		$textHeader = "From: Media Temple user (mt.kb.user@gmail.com)\r\nSubject: article: How to Trace a Email\r\nDate: January 25, 2011 3:30:58 PM PDT\r\nTo: user@example.com\r\nReturn-Path: <mt.kb.user@gmail.com>\r\nEnvelope-To: user@example.com\r\nDelivery-Date: Tue, 25 Jan 2011 15:31:01 -0700";
 	
 		$result = $inboundEmail->importDupeCheck("1", $textHeader, $textHeader);
-		$this->assertEquals(true ,$result);
+		$this->assertEquals(true, $result);
 		
 	}
 
@@ -2035,7 +2035,7 @@ 
 		
 		$result = $inboundEmail->handleMimeHeaderDecode("Subject: article: How to Trace a Email");
 		
-		$this->assertEquals("Subject: article: How to Trace a Email" ,$result);
+		$this->assertEquals("Subject: article: How to Trace a Email", $result);
 		
 	}
 
@@ -2046,7 +2046,7 @@ 
 		
 		$result = $inboundEmail->handleMimeHeaderDecode("Date: January 25, 2011 3:30:58 PM PDT");
 				
-		$this->assertEquals("Date: January 25, 2011 3:30:58 PM PDT" ,$result);
+		$this->assertEquals("Date: January 25, 2011 3:30:58 PM PDT", $result);
 		
 	}
 
@@ -2084,7 +2084,7 @@ 
 		//execute the method and test if it works and does not throws an exception.
 		try {
 			$result = $inboundEmail->importOneEmail("1", "1");
-			$this->assertEquals(false,$result);
+			$this->assertEquals(false, $result);
 		}
 		catch (Exception $e) {
 			$this->fail();
@@ -2112,7 +2112,7 @@ 
 		
 		$result = $inboundEmail->handleUUEncodedEmailBody($raw, 1);
 		
-		$this->assertEquals("\n" . $raw ,$result);
+		$this->assertEquals("\n".$raw, $result);
 
 		
 	}
@@ -2150,7 +2150,7 @@ 
 		
 		$email = new Email();
 		$email->reply_to_email = "reply@gmail.com";
-		$email->from_addr  = "from@gmail.com";
+		$email->from_addr = "from@gmail.com";
 		
 		$result = $inboundEmail->checkFilterDomain($email);
 		$this->assertEquals(true, $result);
@@ -2167,8 +2167,8 @@ 
 		
 		$inboundEmail = new InboundEmail();
 		
-		$this->assertEquals(false, $inboundEmail->checkOutOfOffice("currently Out of Office, will reply later") );
-		$this->assertEquals(true, $inboundEmail->checkOutOfOffice("test subject") );
+		$this->assertEquals(false, $inboundEmail->checkOutOfOffice("currently Out of Office, will reply later"));
+		$this->assertEquals(true, $inboundEmail->checkOutOfOffice("test subject"));
 	
 	}
 
@@ -2184,7 +2184,7 @@ 
 		
 		$inboundEmail = new InboundEmail();
 		
-		$inboundEmail->id =1;
+		$inboundEmail->id = 1;
 
 		
 		//execute the setAutoreplyStatus method to set an auto reply status for email
@@ -2193,7 +2193,7 @@ 
 
 		//test with and invalid email. it will return true as well because it's stil under max limit.
 		$result = $inboundEmail->getAutoreplyStatus("invalid@email.com");
-		$this->assertEquals(true ,$result);
+		$this->assertEquals(true, $result);
 		
 	}
 
@@ -2227,7 +2227,7 @@ 
 		$expected = "			<form action=\"index.php\" method=\"post\" name=\"Macro\" id=\"form\">\n						<input type=\"hidden\" name=\"module\" value=\"InboundEmail\">\n						<input type=\"hidden\" name=\"action\" value=\"ListView\">\n						<input type=\"hidden\" name=\"save\" value=\"true\">\n\n			<table width=\"100%\" cellpadding=\"0\" cellspacing=\"0\" border=\"0\">\n				<tr>\n					<td>\n						<input 	title=\"Save\"\n								accessKey=\"a\"\n								class=\"button\"\n								onclick=\"this.form.return_module.value='InboundEmail'; this.form.return_action.value='ListView';\"\n								type=\"submit\" name=\"Edit\" value=\"  Save  \">\n					</td>\n				</tr>\n			</table>\n\n			<table width=\"100%\" border=\"0\" cellspacing=\"0\" cellpadding=\"0\" class=\"detail view\">\n				<tr>\n					<td valign=\"top\" width='10%' NOWRAP scope=\"row\">\n						<slot>\n							<b>:</b>\n						</slot>\n					</td>\n					<td valign=\"top\" width='20%'>\n						<slot>\n							<input name=\"inbound_email_case_macro\" type=\"text\" value=\"[CASE:%1]\">\n						</slot>\n					</td>\n					<td valign=\"top\" width='70%'>\n						<slot>\n							\n							<br />\n							<i></i>\n						</slot>\n					</td>\n				</tr>\n			</table>\n			</form>";
 		$result = $inboundEmail->getSystemSettingsForm();
 		
-		$this->assertSame($expected,$result);		
+		$this->assertSame($expected, $result);		
 				
 	}
 
@@ -2245,11 +2245,11 @@ 
 		
 		$inboundEmail = new InboundEmail();
 		
-		$result = $inboundEmail->get_stored_options("test","");
+		$result = $inboundEmail->get_stored_options("test", "");
 		$this->assertEquals("", $result);
 
 		
-		$result = $inboundEmail->get_stored_options("test","default_option");
+		$result = $inboundEmail->get_stored_options("test", "default_option");
 		$this->assertEquals("default_option", $result);
 		
 	}
@@ -2294,8 +2294,8 @@ 
 		$inboundEmail = new InboundEmail();
 		
 		$this->assertEquals("{:/service=}", $inboundEmail->getConnectString()); //test with default options
-		$this->assertEquals("{:/service=mail.google.com}INBOX", $inboundEmail->getConnectString("mail.google.com","INBOX"));//test with includeMbox true
-		$this->assertEquals("{:/service=mail.google.com}", $inboundEmail->getConnectString("mail.google.com","INBOX",false));//test with includeMbox false
+		$this->assertEquals("{:/service=mail.google.com}INBOX", $inboundEmail->getConnectString("mail.google.com", "INBOX")); //test with includeMbox true
+		$this->assertEquals("{:/service=mail.google.com}", $inboundEmail->getConnectString("mail.google.com", "INBOX", false)); //test with includeMbox false
 		
 	}
 
@@ -2351,11 +2351,11 @@ 
 		$inboundEmail = new InboundEmail();
 		
 		//test without setting name
-		$this->assertEquals(Null,$inboundEmail->get_summary_text());
+		$this->assertEquals(Null, $inboundEmail->get_summary_text());
 		 
 		//test with name set
 		$inboundEmail->name = "test";
-		$this->assertEquals('test',$inboundEmail->get_summary_text());
+		$this->assertEquals('test', $inboundEmail->get_summary_text());
 		
 	}
 
@@ -2366,18 +2366,18 @@ 
 		
 		//test with empty string params
 		$expected = " SELECT  inbound_email.*  , jt0.user_name created_by_name , jt0.created_by created_by_name_owner  , 'Users' created_by_name_mod FROM inbound_email   LEFT JOIN  users jt0 ON jt0.id=inbound_email.created_by AND jt0.deleted=0\n AND jt0.deleted=0 where inbound_email.deleted=0";
-		$actual = $inboundEmail->create_export_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $inboundEmail->create_export_query('', '');
+		$this->assertSame($expected, $actual);
 
 		
 		//test with valid string params
 		$expected = " SELECT  inbound_email.*  , jt0.user_name created_by_name , jt0.created_by created_by_name_owner  , 'Users' created_by_name_mod FROM inbound_email   LEFT JOIN  users jt0 ON jt0.id=inbound_email.created_by AND jt0.deleted=0\n AND jt0.deleted=0 where (jt0.user_name=\"\") AND inbound_email.deleted=0 ORDER BY inbound_email.id";
-		$actual = $inboundEmail->create_export_query('id','jt0.user_name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $inboundEmail->create_export_query('id', 'jt0.user_name=""');
+		$this->assertSame($expected, $actual);
 		
 	}
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 
 		$inboundEmail = new InboundEmail();
 
@@ -2574,7 +2574,7 @@ 
 		$user->retrieve(1);
 		
 		//set a Outbound Server Id
-		$inboundEmail->setUsersDefaultOutboundServerId($user,"123");
+		$inboundEmail->setUsersDefaultOutboundServerId($user, "123");
 
 		
 		//retrieve Outbound Server Id back and verify
@@ -2599,7 +2599,7 @@ 
 		
 		
 		//test with pop3 protocol and all parameters true
-		$result = $inboundEmail->setEmailForDisplay("1",true,true,true);
+		$result = $inboundEmail->setEmailForDisplay("1", true, true, true);
 		$this->assertEquals("error", $result);
 		
 		
@@ -2615,12 +2615,12 @@ 
 		
 		$inboundEmail->email->name = "test";
 		$inboundEmail->email->from_addr_name = "from";
-		$inboundEmail->email->to_addrs_names ="to";
+		$inboundEmail->email->to_addrs_names = "to";
 		$inboundEmail->email->cc_addrs_names = "cc";
 		$inboundEmail->email->reply_to_addr = "reply";
 		
-		$expected = array (
-						  'meta' => array ( 'type' => 'archived', 'uid' => 1, 'ieId' => 1, 'email' => array ( 'name' => 'test','from_name' => '', 'from_addr' => 'from','date_start' => ' ', 'time_start' => '', 'message_id' => '', 'cc_addrs' => 'cc', 'attachments' => '', 'toaddrs' => 'to', 'description' => '', 'reply_to_addr' => 'reply',  ), 'mbox' => "INBOX", 'cc' => '', 'is_sugarEmail' => false, )
+		$expected = array(
+						  'meta' => array('type' => 'archived', 'uid' => 1, 'ieId' => 1, 'email' => array('name' => 'test', 'from_name' => '', 'from_addr' => 'from', 'date_start' => ' ', 'time_start' => '', 'message_id' => '', 'cc_addrs' => 'cc', 'attachments' => '', 'toaddrs' => 'to', 'description' => '', 'reply_to_addr' => 'reply',), 'mbox' => "INBOX", 'cc' => '', 'is_sugarEmail' => false,)
 						);
 		$result = $inboundEmail->displayOneEmail(1, "INBOX");
 		
@@ -2672,12 +2672,12 @@ 
 
 		
 		//execute the method to sort the objects array descending and verify the order 
-		$result = $inboundEmail->sortFetchedOverview($arr, 3,'DESC');
+		$result = $inboundEmail->sortFetchedOverview($arr, 3, 'DESC');
 		$this->assertEquals('subject 2', $result['retArr'][0]->subject);
 		
 		
 		//execute the method to sort the objects array ascending and verify the order
-		$result = $inboundEmail->sortFetchedOverview($arr, 3,'ASC');
+		$result = $inboundEmail->sortFetchedOverview($arr, 3, 'ASC');
 		$this->assertEquals('subject 1', $result['retArr'][0]->subject);
 		
 	}
@@ -2687,7 +2687,7 @@ 
 
 		$inboundEmail = new InboundEmail();
 		
-		$expected = array ( 'mbox' => 'INBOX', 'ieId' => 1, 'name' => 'test', 'fromCache' => 0, 'out' => array () );
+		$expected = array('mbox' => 'INBOX', 'ieId' => 1, 'name' => 'test', 'fromCache' => 0, 'out' => array());
 		$inboundEmail->id = 1;
 		$inboundEmail->name = "test";
 		
@@ -2738,8 +2738,8 @@ 
 
 		$inboundEmail = new InboundEmail();
 		
-		$inboundEmail->mailboxarray = array("INBOX.TRASH","OUTBOX.TRASH");
-		$expected = array ( 'INBOX' => array ( 'TRASH' => 'TRASH' ), 'OUTBOX' => array ( 'TRASH' => 'TRASH') );
+		$inboundEmail->mailboxarray = array("INBOX.TRASH", "OUTBOX.TRASH");
+		$expected = array('INBOX' => array('TRASH' => 'TRASH'), 'OUTBOX' => array('TRASH' => 'TRASH'));
 		
 		
 		//test with justRaw default/false 
@@ -2759,7 +2759,7 @@ 
 		
 		$inboundEmail->mailbox = 1;
 		$inboundEmail->mailbox = "INBOX.TRASH,OUTBOX.TRASH";
-		$expected = array (	'INBOX' => 	array ( 'TRASH' => 'TRASH' ),'OUTBOX' => array ( 'TRASH' => 'TRASH'));
+		$expected = array('INBOX' => 	array('TRASH' => 'TRASH'), 'OUTBOX' => array('TRASH' => 'TRASH'));
 		
 		$result = $inboundEmail->getMailBoxesForGroupAccount();
 		
@@ -2776,7 +2776,7 @@ 
 		
 		$inboundEmail->retrieveMailBoxFolders();
 		
-		$this->assertEquals(array("INBOX","OUTBOX","TRASH"), $inboundEmail->mailboxarray);
+		$this->assertEquals(array("INBOX", "OUTBOX", "TRASH"), $inboundEmail->mailboxarray);
 		
 	} 
 
@@ -2789,7 +2789,7 @@ 
 		
 		//execute the method and test if it works and does not throws an exception.
 		try {
-			$inboundEmail->insertMailBoxFolders(array("INBOX","OUTBOX"));
+			$inboundEmail->insertMailBoxFolders(array("INBOX", "OUTBOX"));
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -2813,9 +2813,9 @@ 
 
 		$inboundEmail = new InboundEmail();
 		
-		$arraymbox = array ( 'INBOX' => 	array ( 'TRASH' => 'TRASH' ), 'OUTBOX' => array ( 'TRASH' => 'TRASH') );
+		$arraymbox = array('INBOX' => 	array('TRASH' => 'TRASH'), 'OUTBOX' => array('TRASH' => 'TRASH'));
 		
-		$expected = array('INBOX','INBOX.TRASH','OUTBOX', 'OUTBOX.TRASH');
+		$expected = array('INBOX', 'INBOX.TRASH', 'OUTBOX', 'OUTBOX.TRASH');
 		
 		$result = $inboundEmail->generateFlatArrayFromMultiDimArray($arraymbox, ".");
 		
@@ -2827,9 +2827,9 @@ 
 
 		$inboundEmail = new InboundEmail();
 		
-		$expected = array (	'INBOX' => 	array ( 'TRASH' => 'TRASH' ),'OUTBOX' => array ( 'TRASH' => 'TRASH'));
+		$expected = array('INBOX' => 	array('TRASH' => 'TRASH'), 'OUTBOX' => array('TRASH' => 'TRASH'));
 		
-		$result = $inboundEmail->generateMultiDimArrayFromFlatArray(array("INBOX.TRASH","OUTBOX.TRASH"), ".");
+		$result = $inboundEmail->generateMultiDimArrayFromFlatArray(array("INBOX.TRASH", "OUTBOX.TRASH"), ".");
 
 		$this->assertEquals($expected, $result);
 	
@@ -2840,8 +2840,8 @@ 
 		$inboundEmail = new InboundEmail();
 		
 		$result = array(); 
-		$arraymbox = array ( 'INBOX' => array ('TRASH' => 'TRASH'));
-		$expected = array( 'MAIN', 'MAIN.INBOX','MAIN.INBOX.TRASH' );
+		$arraymbox = array('INBOX' => array('TRASH' => 'TRASH'));
+		$expected = array('MAIN', 'MAIN.INBOX', 'MAIN.INBOX.TRASH');
 		
 		$inboundEmail->generateArrayData("MAIN", $arraymbox, $result, ".");
 		
@@ -2855,9 +2855,9 @@ 
 
 		$inboundEmail = new InboundEmail();
 		
-		$result= $inboundEmail->sortMailboxes("INBOX.TRASH",array());
+		$result = $inboundEmail->sortMailboxes("INBOX.TRASH", array());
 		
-		$expected = array ( 'INBOX' => array ('TRASH' => 'TRASH'));
+		$expected = array('INBOX' => array('TRASH' => 'TRASH'));
 		
 		$this->assertEquals($expected, $result);
 		
@@ -2918,7 +2918,7 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$overview = new Overview();
 		
-		$this->assertInstanceOf('Overview',$overview);
+		$this->assertInstanceOf('Overview', $overview);
 			
 		$this->assertTrue(is_array($overview->fieldDefs));
 		$this->assertTrue(is_array($overview->indices));

tests/tests/modules/AOR_Reports/AOR_ReportTest.php

Spacing +49 added lines, -49 removed lines

@@ -3,15 +3,15 @@ 
 
 class AOR_ReportTest extends PHPUnit_Framework_TestCase {
 
-	public function testAOR_Report(){
+	public function testAOR_Report() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		
 		//execute the contructor and check for the Object type and  attributes
 		$aor_Report = new AOR_Report();
-		$this->assertInstanceOf('AOR_Report',$aor_Report);
-		$this->assertInstanceOf('Basic',$aor_Report);
-		$this->assertInstanceOf('SugarBean',$aor_Report);
+		$this->assertInstanceOf('AOR_Report', $aor_Report);
+		$this->assertInstanceOf('Basic', $aor_Report);
+		$this->assertInstanceOf('SugarBean', $aor_Report);
 			
 		$this->assertAttributeEquals('AOR_Reports', 'module_dir', $aor_Report);
 		$this->assertAttributeEquals('AOR_Report', 'object_name', $aor_Report);
@@ -22,7 +22,7 @@ 
 		
 	}
 
-	public function testbean_implements(){
+	public function testbean_implements() {
 
 		$aor_Report = new AOR_Report();
 		 
@@ -32,7 +32,7 @@ 
 		
 	}
 
-    public function testsave(){
+    public function testsave() {
 
     	$aor_Report = new AOR_Report();
     	
@@ -75,7 +75,7 @@ 
     }
     
 
-    public function testload_report_beans(){
+    public function testload_report_beans() {
 
     	$aor_Report = new AOR_Report();
 
@@ -91,7 +91,7 @@ 
     }
 
 
-    public function testgetReportFields(){
+    public function testgetReportFields() {
     	
     	//execute the method and verify that it returns an array
     	$aor_Report = new AOR_Report();
@@ -101,7 +101,7 @@ 
     }
 
 
-    public function testbuild_report_chart(){
+    public function testbuild_report_chart() {
 
     	$aor_Report = new AOR_Report();
     	$aor_Report->report_module = "Accounts";
@@ -114,7 +114,7 @@ 
     }
 
 
-    public function testbuild_group_report(){
+    public function testbuild_group_report() {
 
     	
     	$aor_Report = new AOR_Report();
@@ -123,26 +123,26 @@ 
 
     	//execute the method without any parameters and verify it returns html string
     	$html1 = $aor_Report->build_group_report();
-    	$this->assertGreaterThan(0,strlen($html1));
+    	$this->assertGreaterThan(0, strlen($html1));
     	
     	//execute the method wit offset parameter and verify it returns html string
     	$html2 = $aor_Report->build_group_report(1);
-    	$this->assertGreaterThan(0,strlen($html2));
+    	$this->assertGreaterThan(0, strlen($html2));
     	
     	//execute the method with both parameters and verify it returns html string
-    	$html3 = $aor_Report->build_group_report(0,false);
-    	$this->assertGreaterThan(0,strlen($html3));
+    	$html3 = $aor_Report->build_group_report(0, false);
+    	$this->assertGreaterThan(0, strlen($html3));
     	
     	//verify that all three html strings are different.
-    	$this->assertNotEquals($html1,$html2);
-    	$this->assertNotEquals($html1,$html3);
-    	$this->assertNotEquals($html2,$html3); 
+    	$this->assertNotEquals($html1, $html2);
+    	$this->assertNotEquals($html1, $html3);
+    	$this->assertNotEquals($html2, $html3); 
     	
     	
     }
 
 
-    public function testbuild_report_html(){
+    public function testbuild_report_html() {
 
     	$aor_Report = new AOR_Report();
     	$aor_Report->report_module = "Accounts";
@@ -150,56 +150,56 @@ 
     	
     	//execute the method without any parameters and verify it returns html string
     	$html1 = $aor_Report->build_report_html();
-    	$this->assertGreaterThan(0,strlen($html1));
+    	$this->assertGreaterThan(0, strlen($html1));
 		
     	
     	//execute the method with both parameters and verify it returns html string
-    	$html2 = $aor_Report->build_report_html(0,false);
-    	$this->assertGreaterThan(0,strlen($html2));
+    	$html2 = $aor_Report->build_report_html(0, false);
+    	$this->assertGreaterThan(0, strlen($html2));
     	
     	
     	//execute the method with group and identifier parameters and verify it returns html string
-    	$html3 = $aor_Report->build_report_html(1,false,'grouptest','testidentifier');
-    	$this->assertGreaterThan(0,strlen($html3));
+    	$html3 = $aor_Report->build_report_html(1, false, 'grouptest', 'testidentifier');
+    	$this->assertGreaterThan(0, strlen($html3));
     	
     	
     	//verify that group and identifier exist in the strings
-    	$this->assertNotFalse(strpos($html3,'grouptest'));
-    	$this->assertNotFalse(strpos($html3,'testidentifier'));
+    	$this->assertNotFalse(strpos($html3, 'grouptest'));
+    	$this->assertNotFalse(strpos($html3, 'testidentifier'));
     	
     }
 
-    public function testgetTotalHTML(){
+    public function testgetTotalHTML() {
 
     	
     	//execute the method with required data preset and verify it returns expected result
-    	$fields =Array('label'=>Array('display'=>1 , 'total'=> 'SUM','label'=>'total'));
-    	$totals = Array('label'=> array(10,20,30));
+    	$fields = Array('label'=>Array('display'=>1, 'total'=> 'SUM', 'label'=>'total'));
+    	$totals = Array('label'=> array(10, 20, 30));
     	$expected = "<tbody><tr><th>total Sum</th></tr><tr><td>60</td></tr></tbody>";
     	
     	$aor_Report = new AOR_Report();
-    	$actual = $aor_Report->getTotalHTML($fields,$totals);
+    	$actual = $aor_Report->getTotalHTML($fields, $totals);
 
     	$this->assertSame($expected, $actual);
     	
     }
 
-    public function testcalculateTotal(){
+    public function testcalculateTotal() {
 
     	//execute the method with data preset and verify it returns expected result
-    	$totals = Array(10,20,30);
+    	$totals = Array(10, 20, 30);
     	
     	$aor_Report = new AOR_Report();
 
     	$this->assertEquals('', $aor_Report->calculateTotal('', $totals));
     	$this->assertEquals(60, $aor_Report->calculateTotal('SUM', $totals));
     	$this->assertEquals(3, $aor_Report->calculateTotal('COUNT', $totals));
-    	$this->assertEquals(20 , $aor_Report->calculateTotal('AVG', $totals)); 	
+    	$this->assertEquals(20, $aor_Report->calculateTotal('AVG', $totals)); 	
     	    	
     }
 
 
-    public function testbuild_report_csv(){
+    public function testbuild_report_csv() {
 
     	//this method uses exit so it cannot be tested
     	
@@ -214,7 +214,7 @@ 
 
 
 
-    public function testbuild_report_query(){
+    public function testbuild_report_query() {
 
     	$aor_Report = new AOR_Report();
     	$aor_Report->report_module = "Accounts";
@@ -229,7 +229,7 @@ 
     	
     }
 
-    public function testbuild_report_query_select(){
+    public function testbuild_report_query_select() {
 
     	$aor_Report = new AOR_Report();
     	$aor_Report->report_module = "Accounts";    	
@@ -242,50 +242,50 @@ 
     }
 
 
-    public function testbuild_report_query_join( ){
+    public function testbuild_report_query_join( ) {
 
     	$aor_Report = new AOR_Report();
     	$aor_Report->report_module = "Accounts";
 
     	
 		//test with type custom and verify that it retunrs expected results
-    	$expected = array ('join' => array ('accounts_contacts' => 'LEFT JOIN `accounts_cstm` `contacts` ON `accounts`.id = `contacts`.id_c '));     	
-    	$actual = $aor_Report->build_report_query_join('contacts', 'accounts_contacts', 'accounts', new Account() , 'custom', array() );
-		$this->assertSame($expected ,$actual);
+    	$expected = array('join' => array('accounts_contacts' => 'LEFT JOIN `accounts_cstm` `contacts` ON `accounts`.id = `contacts`.id_c '));     	
+    	$actual = $aor_Report->build_report_query_join('contacts', 'accounts_contacts', 'accounts', new Account(), 'custom', array());
+		$this->assertSame($expected, $actual);
     	
 		
 		//test with type relationship and verify that it retunrs expected results
-		$expected = array ('join' =>array ('accounts_contacts' => "LEFT JOIN accounts_contacts `accounts|accounts_contacts` ON `accounts`.id=`accounts|accounts_contacts`.account_id AND `accounts|accounts_contacts`.deleted=0\n\nLEFT JOIN contacts `accounts_contacts` ON `accounts_contacts`.id=`accounts|accounts_contacts`.contact_id AND `accounts_contacts`.deleted=0\n"),
-						   'select' =>array ('`accounts_contacts`.id AS \'accounts_contacts_id\''));
-		$actual = $aor_Report->build_report_query_join('contacts', 'accounts_contacts', 'accounts', new Account() , 'relationship', array() );
-		$this->assertSame($expected , $actual);
+		$expected = array('join' =>array('accounts_contacts' => "LEFT JOIN accounts_contacts `accounts|accounts_contacts` ON `accounts`.id=`accounts|accounts_contacts`.account_id AND `accounts|accounts_contacts`.deleted=0\n\nLEFT JOIN contacts `accounts_contacts` ON `accounts_contacts`.id=`accounts|accounts_contacts`.contact_id AND `accounts_contacts`.deleted=0\n"),
+						   'select' =>array('`accounts_contacts`.id AS \'accounts_contacts_id\''));
+		$actual = $aor_Report->build_report_query_join('contacts', 'accounts_contacts', 'accounts', new Account(), 'relationship', array());
+		$this->assertSame($expected, $actual);
     			
     }
 
-    public function testbuild_report_access_query(){
+    public function testbuild_report_access_query() {
 
     	$aor_Report = new AOR_Report();
     	
     	//test without alias and verify that it retunrs expected results
     	$result = $aor_Report->build_report_access_query(new AOR_Report(), '');
-    	$this->assertEquals("",$result);
+    	$this->assertEquals("", $result);
     	
     	//test with alias and verify that it retunrs expected results
     	$result = $aor_Report->build_report_access_query(new AOR_Report(), 'rep');
-    	$this->assertEquals("",$result);
+    	$this->assertEquals("", $result);
     	
     }
 
 
-    public function testbuild_report_query_where(){
+    public function testbuild_report_query_where() {
 
     	$aor_Report = new AOR_Report();
     	$aor_Report->report_module = "Accounts";
 
     	//execute the method and verify that it retunrs expected results
-    	$expected = Array ('where' => array ('accounts.deleted = 0 ')); 
+    	$expected = Array('where' => array('accounts.deleted = 0 ')); 
     	$actual = $aor_Report->build_report_query_where();
-    	$this->assertSame($expected , $actual);
+    	$this->assertSame($expected, $actual);
     	    	
     }
 

tests/tests/modules/Notes/NoteTest.php

Spacing +16 added lines, -16 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-class NoteTest extends PHPUnit_Framework_TestCase  {
+class NoteTest extends PHPUnit_Framework_TestCase {
 
 
 	public function testNote() {
@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$note = new Note();
 		
-		$this->assertInstanceOf('Note',$note);
-		$this->assertInstanceOf('SugarBean',$note);
+		$this->assertInstanceOf('Note', $note);
+		$this->assertInstanceOf('SugarBean', $note);
 		 
 		$this->assertAttributeEquals('Notes', 'module_dir', $note);
 		$this->assertAttributeEquals('Note', 'object_name', $note);
@@ -29,14 +29,14 @@ 
 		//test with valid file name
 		$note->filename = "test.txt";
 		$note->safeAttachmentName();
-		$this->assertEquals("test.txt" ,$note->filename);
+		$this->assertEquals("test.txt", $note->filename);
 		
 		
 		//test with invalid file name
 		$note->filename = "test.php";
 		$note->safeAttachmentName();
-		$this->assertEquals(".txt" ,$note->name);
-		$this->assertEquals("test.php.txt" ,$note->filename);
+		$this->assertEquals(".txt", $note->name);
+		$this->assertEquals("test.php.txt", $note->filename);
 		
 	}
 
@@ -56,7 +56,7 @@ 
 		
 	}
 
-	public function testdeleteAttachment(){
+	public function testdeleteAttachment() {
 
 		$note = new Note();
 		
@@ -72,11 +72,11 @@ 
 		$note = new Note();
 		
 		//test without setting name
-		$this->assertEquals('',$note->get_summary_text());
+		$this->assertEquals('', $note->get_summary_text());
 		
 		//test with name set
 		$note->name = "test";
-		$this->assertEquals('test',$note->get_summary_text());
+		$this->assertEquals('test', $note->get_summary_text());
 		
 	}
 
@@ -87,14 +87,14 @@ 
     	
     	//test with empty string params
     	$expected = "SELECT notes.*, contacts.first_name, contacts.last_name, users.user_name as assigned_user_name  FROM notes 	LEFT JOIN contacts ON notes.contact_id=contacts.id   LEFT JOIN users ON notes.assigned_user_id=users.id where  notes.deleted=0 AND (contacts.deleted IS NULL OR contacts.deleted=0) ORDER BY notes.name";
-    	$actual = $note->create_export_query('','');
-    	$this->assertSame($expected,$actual);
+    	$actual = $note->create_export_query('', '');
+    	$this->assertSame($expected, $actual);
 
     	
     	//test with valid string params
     	$expected = "SELECT notes.*, contacts.first_name, contacts.last_name, users.user_name as assigned_user_name  FROM notes 	LEFT JOIN contacts ON notes.contact_id=contacts.id   LEFT JOIN users ON notes.assigned_user_id=users.id where users.user_name=\"\" AND  notes.deleted=0 AND (contacts.deleted IS NULL OR contacts.deleted=0) ORDER BY notes.name";
-    	$actual = $note->create_export_query('notes.id','users.user_name=""');
-    	$this->assertSame($expected,$actual);
+    	$actual = $note->create_export_query('notes.id', 'users.user_name=""');
+    	$this->assertSame($expected, $actual);
     	
 	}
 
@@ -138,7 +138,7 @@ 
 		$note->filename = "test.txt";
 		$note->contact_name = "test contact";
 
-		$expected = array (
+		$expected = array(
                       'ID' => $id,
 					  'FILENAME' => 'test.txt',
 					  'PARENT_TYPE' => 'Account',
@@ -159,9 +159,9 @@ 
 
 		$note = new Note();
 		
-		$expected = array("MAIN"=>"a", "PARENT"=>"a", "CONTACT"=>"a" );
+		$expected = array("MAIN"=>"a", "PARENT"=>"a", "CONTACT"=>"a");
 		$actual = $note->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 	
 	}
 

tests/tests/modules/Leads/LeadTest.php

Spacing +50 added lines, -50 removed lines

@@ -9,9 +9,9 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$lead = new Lead();
 		 
-		$this->assertInstanceOf('Lead',$lead);
-		$this->assertInstanceOf('Person',$lead);
-		$this->assertInstanceOf('SugarBean',$lead);
+		$this->assertInstanceOf('Lead', $lead);
+		$this->assertInstanceOf('Person', $lead);
+		$this->assertInstanceOf('SugarBean', $lead);
 		
 		$this->assertAttributeEquals('Leads', 'module_dir', $lead);
 		$this->assertAttributeEquals('Lead', 'object_name', $lead);
@@ -31,13 +31,13 @@ 
 		
 		//test without pre settting attributes
 		$result = $lead->get_account();
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 		
 		//test with required attributes preset
-		$lead->account_id =1;
+		$lead->account_id = 1;
 		$result = $lead->get_account();
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 		
 	}
@@ -49,13 +49,13 @@ 
 		
 		//test without pre settting attributes
 		$result = $lead->get_opportunity();
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 		
 		//test with required attributes preset
-		$lead->opportunity_id =1;
+		$lead->opportunity_id = 1;
 		$result = $lead->get_opportunity();
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 	
@@ -66,13 +66,13 @@ 
 		
 		//test without pre settting attributes
 		$result = $lead->get_contact();
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 		
 		//test with required attributes preset
-		$lead->contact_id =1;
+		$lead->contact_id = 1;
 		$result = $lead->get_contact();
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 		
 	}
@@ -83,14 +83,14 @@ 
 		
 		//test with empty string params
 		$expected = "SELECT leads.*, users.user_name assigned_user_name FROM leads 			LEFT JOIN users\n                                ON leads.assigned_user_id=users.id LEFT JOIN email_addr_bean_rel eabl  ON eabl.bean_id = leads.id AND eabl.bean_module = 'Leads' and eabl.primary_address = 1 and eabl.deleted=0 LEFT JOIN email_addresses ea ON (ea.id = eabl.email_address_id) where  leads.deleted=0 ";
-		$actual = $lead->create_list_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $lead->create_list_query('', '');
+		$this->assertSame($expected, $actual);
 		
 		 
 		//test with valid string params
 		$expected = "SELECT leads.*, users.user_name assigned_user_name FROM leads 			LEFT JOIN users\n                                ON leads.assigned_user_id=users.id LEFT JOIN email_addr_bean_rel eabl  ON eabl.bean_id = leads.id AND eabl.bean_module = 'Leads' and eabl.primary_address = 1 and eabl.deleted=0 LEFT JOIN email_addresses ea ON (ea.id = eabl.email_address_id) where (users.user_name=\"\") AND  leads.deleted=0  ORDER BY leads.id";
-		$actual = $lead->create_list_query('leads.id','users.user_name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $lead->create_list_query('leads.id', 'users.user_name=""');
+		$this->assertSame($expected, $actual);
 				
 	}
 
@@ -101,14 +101,14 @@ 
 		
 		//test with empty string params
 		$expected = " SELECT  leads.* , '                                                                                                                                                                                                                                                              ' c_accept_status_fields , '                                    '  call_id , '                                                                                                                                                                                                                                                              ' e_invite_status_fields , '                                    '  fp_events_leads_1fp_events_ida , '                                                                                                                                                                                                                                                              ' e_accept_status_fields , LTRIM(RTRIM(CONCAT(IFNULL(leads.first_name,''),' ',IFNULL(leads.last_name,'')))) as name , jt3.user_name modified_by_name , jt3.created_by modified_by_name_owner  , 'Users' modified_by_name_mod , jt4.user_name created_by_name , jt4.created_by created_by_name_owner  , 'Users' created_by_name_mod , jt5.user_name assigned_user_name , jt5.created_by assigned_user_name_owner  , 'Users' assigned_user_name_mod, LTRIM(RTRIM(CONCAT(IFNULL(leads.first_name,''),' ',IFNULL(leads.last_name,'')))) as full_name , jt6.name campaign_name , jt6.assigned_user_id campaign_name_owner  , 'Campaigns' campaign_name_mod, '                                                                                                                                                                                                                                                              ' m_accept_status_fields , '                                    '  meeting_id  FROM leads   LEFT JOIN  users jt3 ON leads.modified_user_id=jt3.id AND jt3.deleted=0\n\n AND jt3.deleted=0  LEFT JOIN  users jt4 ON leads.created_by=jt4.id AND jt4.deleted=0\n\n AND jt4.deleted=0  LEFT JOIN  users jt5 ON leads.assigned_user_id=jt5.id AND jt5.deleted=0\n\n AND jt5.deleted=0  LEFT JOIN  campaigns jt6 ON leads.campaign_id=jt6.id AND jt6.deleted=0\n\n AND jt6.deleted=0 where leads.deleted=0";
-		$actual = $lead->create_new_list_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $lead->create_new_list_query('', '');
+		$this->assertSame($expected, $actual);
 
 			
 		//test with valid string params
 		$expected = " SELECT  leads.* , '                                                                                                                                                                                                                                                              ' c_accept_status_fields , '                                    '  call_id , '                                                                                                                                                                                                                                                              ' e_invite_status_fields , '                                    '  fp_events_leads_1fp_events_ida , '                                                                                                                                                                                                                                                              ' e_accept_status_fields , LTRIM(RTRIM(CONCAT(IFNULL(leads.first_name,''),' ',IFNULL(leads.last_name,'')))) as name , jt3.user_name modified_by_name , jt3.created_by modified_by_name_owner  , 'Users' modified_by_name_mod , jt4.user_name created_by_name , jt4.created_by created_by_name_owner  , 'Users' created_by_name_mod , jt5.user_name assigned_user_name , jt5.created_by assigned_user_name_owner  , 'Users' assigned_user_name_mod, LTRIM(RTRIM(CONCAT(IFNULL(leads.first_name,''),' ',IFNULL(leads.last_name,'')))) as full_name , jt6.name campaign_name , jt6.assigned_user_id campaign_name_owner  , 'Campaigns' campaign_name_mod, '                                                                                                                                                                                                                                                              ' m_accept_status_fields , '                                    '  meeting_id  FROM leads   LEFT JOIN  users jt3 ON leads.modified_user_id=jt3.id AND jt3.deleted=0\n\n AND jt3.deleted=0  LEFT JOIN  users jt4 ON leads.created_by=jt4.id AND jt4.deleted=0\n\n AND jt4.deleted=0  LEFT JOIN  users jt5 ON leads.assigned_user_id=jt5.id AND jt5.deleted=0\n\n AND jt5.deleted=0  LEFT JOIN  campaigns jt6 ON leads.campaign_id=jt6.id AND jt6.deleted=0\n\n AND jt6.deleted=0 where (users.user_name=\"\") AND leads.deleted=0";
-		$actual = $lead->create_new_list_query('leads.id','users.user_name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $lead->create_new_list_query('leads.id', 'users.user_name=""');
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -117,7 +117,7 @@ 
     	$lead = new Lead();
 
     	$lead->first_name = "firstn";
-    	$lead->last_name ="lastnn";
+    	$lead->last_name = "lastnn";
     	$lead->lead_source = "test";
     	    	
     	$result = $lead->save();
@@ -146,7 +146,7 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$lead->mark_deleted($lead->id);
     	$result = $lead->retrieve($lead->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
     	
     }
 
@@ -157,11 +157,11 @@ 
 		$lead = new Lead();
 		
 		$lead->first_name = "firstn";
-		$lead->last_name ="lastn";
+		$lead->last_name = "lastn";
 		
 		$lead->fill_in_additional_list_fields();
 		
-		$this->assertEquals("firstn lastn", $lead->name );
+		$this->assertEquals("firstn lastn", $lead->name);
 		
 	}
 
@@ -172,21 +172,21 @@ 
 		$lead = new Lead();
 		
 		$lead->first_name = "firstn";
-		$lead->last_name ="lastn";
+		$lead->last_name = "lastn";
 		
 		$lead->fill_in_additional_detail_fields();
 		
-		$this->assertEquals("firstn lastn", $lead->name );
+		$this->assertEquals("firstn lastn", $lead->name);
 		
 		
 		
 	}
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 
 		$lead = new Lead();
 		
-		$expected = array (
+		$expected = array(
 					  'NAME' => ' ',
 					  'DELETED' => 0,
 					  'FULL_NAME' => ' ',
@@ -214,7 +214,7 @@ 
     {
     	$lead = new Lead();
 
-    	$expected = array (
+    	$expected = array(
 					  'created_by_link',
 					  'modified_user_link',
 					  'assigned_user_link',
@@ -242,11 +242,11 @@ 
         sort($expected);
         $actualKeys = array_keys($actual);
         sort($actualKeys);
-    	$this->assertSame($expected ,$actualKeys);
+    	$this->assertSame($expected, $actualKeys);
     	
     }
 
-	public function testbuild_generic_where_clause () 
+	public function testbuild_generic_where_clause() 
 	{
 		
 		$lead = new Lead();
@@ -254,13 +254,13 @@ 
 		//test with empty string params
 		$expected = "leads.last_name like '%' or leads.account_name like '%' or leads.first_name like '%' or ea.email_address like '%'";
 		$actual = $lead->build_generic_where_clause("");
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 
 		
 		//test with valid string params
 		$expected = "leads.last_name like '%' or leads.account_name like '%' or leads.first_name like '%' or ea.email_address like '%'";
 		$actual = $lead->build_generic_where_clause("123");
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -272,7 +272,7 @@ 
 		//test with attributes preset and verify template variables are set accordingly
 		
 		$lead->first_name = "firstn";
-		$lead->last_name ="lastn";
+		$lead->last_name = "lastn";
 		$lead->salutation = "Mr";
 		$lead->lead_source = "Email";
 		$lead->status = "New";
@@ -280,10 +280,10 @@ 
 		
 		$result = $lead->set_notification_body(new Sugar_Smarty(), $lead);
 		
-		$this->assertEquals("Mr firstn lastn" ,$result->_tpl_vars['LEAD_NAME']);
-		$this->assertEquals($lead->lead_source ,$result->_tpl_vars['LEAD_SOURCE']);
-		$this->assertEquals($lead->status ,$result->_tpl_vars['LEAD_STATUS']);
-		$this->assertEquals($lead->description ,$result->_tpl_vars['LEAD_DESCRIPTION']);
+		$this->assertEquals("Mr firstn lastn", $result->_tpl_vars['LEAD_NAME']);
+		$this->assertEquals($lead->lead_source, $result->_tpl_vars['LEAD_SOURCE']);
+		$this->assertEquals($lead->status, $result->_tpl_vars['LEAD_STATUS']);
+		$this->assertEquals($lead->description, $result->_tpl_vars['LEAD_DESCRIPTION']);
 
 	}
 
@@ -301,9 +301,9 @@ 
 	{
 		$lead = new Lead();
 
-		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a", "OPPORTUNITY"=>"a", "CONTACT"=>"a" );
+		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a", "OPPORTUNITY"=>"a", "CONTACT"=>"a");
 		$actual = $lead->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 	
 	}
 
@@ -318,7 +318,7 @@ 
 				
 		$result = $lead->convertCustomFieldsForm($form, $tempBean, $prefix);
 		
-		$this->assertEquals(true,$result);
+		$this->assertEquals(true, $result);
 		$this->assertgreaterThanOrEqual("", $form); //no filed with source = custom_fields 
 		
 	}
@@ -330,7 +330,7 @@ 
 
 		$expected = "SELECT emails.id FROM emails  JOIN (select DISTINCT email_id from emails_email_addr_rel eear\n\n	join email_addr_bean_rel eabr on eabr.bean_id ='' and eabr.bean_module = 'Leads' and\n	eabr.email_address_id = eear.email_address_id and eabr.deleted=0\n	where eear.deleted=0 and eear.email_id not in\n	(select eb.email_id from emails_beans eb where eb.bean_module ='Leads' and eb.bean_id = '')\n	) derivedemails on derivedemails.email_id = emails.id";
 		$actual = $lead->get_unlinked_email_query();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 
 	}
     
@@ -339,16 +339,16 @@ 
     {
     	$lead = new Lead();
     	
-    	$expected = array (
+    	$expected = array(
 					  "select" => "SELECT calls.id ",
 					  "from" => "FROM calls ",
 					  "where" => " WHERE calls.parent_id = '' \n            AND calls.parent_type = 'Leads' AND calls.id NOT IN ( SELECT call_id FROM calls_leads ) ",
 					  "join" => "",
-					  "join_tables" =>  array (""),
+					  "join_tables" =>  array(""),
 					);
     	
     	$actual = $lead->get_old_related_calls();
-    	$this->assertSame($expected,$actual);
+    	$this->assertSame($expected, $actual);
     	
     }
 
@@ -357,9 +357,9 @@ 
 	{
 		$lead = new Lead();
 		
-		$expected = array( "copy"=>"Copy", "move"=>"Move", "donothing"=>"Do Nothing");
+		$expected = array("copy"=>"Copy", "move"=>"Move", "donothing"=>"Do Nothing");
 		$actual = $lead->getActivitiesOptions();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
     }
 
@@ -368,16 +368,16 @@ 
     {
     	$lead = new Lead();
 
-    	$expected = array (
+    	$expected = array(
 					  "select" => "SELECT meetings.id ",
 					  "from" => "FROM meetings ",
 					  "where" => " WHERE meetings.parent_id = '' \n            AND meetings.parent_type = 'Leads' AND meetings.id NOT IN ( SELECT meeting_id FROM meetings_leads ) ",
 					  "join" => "",
-					  "join_tables" => array (""),
+					  "join_tables" => array(""),
 					);
     	 
     	$actual = $lead->get_old_related_meetings();
-    	 $this->assertSame($expected,$actual);
+    	 $this->assertSame($expected, $actual);
     	
     }
 

tests/tests/modules/OAuthKeys/OAuthKeyTest.php

Spacing +4 added lines, -4 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 
-class OAuthKeyTest extends PHPUnit_Framework_TestCase{
+class OAuthKeyTest extends PHPUnit_Framework_TestCase {
 
 
 	public function testOAuthKey()
@@ -10,9 +10,9 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$oauthKey = new OAuthKey();
 		
-		$this->assertInstanceOf('OAuthKey',$oauthKey);
-		$this->assertInstanceOf('Basic',$oauthKey);
-		$this->assertInstanceOf('SugarBean',$oauthKey);
+		$this->assertInstanceOf('OAuthKey', $oauthKey);
+		$this->assertInstanceOf('Basic', $oauthKey);
+		$this->assertInstanceOf('SugarBean', $oauthKey);
 			
 		$this->assertAttributeEquals('OAuthKeys', 'module_dir', $oauthKey);
 		$this->assertAttributeEquals('OAuthKey', 'object_name', $oauthKey);

tests/tests/modules/AOS_Products/AOS_ProductsTest.php

Spacing +8 added lines, -8 removed lines

@@ -2,13 +2,13 @@ 
 
 class AOS_ProductsTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAOS_Products(){	
+	public function testAOS_Products() {	
 
 		//execute the contructor and check for the Object type and  attributes
 		$aosProducts = new AOS_Products();
-		$this->assertInstanceOf('AOS_Products',$aosProducts);
-		$this->assertInstanceOf('Basic',$aosProducts);
-		$this->assertInstanceOf('SugarBean',$aosProducts);
+		$this->assertInstanceOf('AOS_Products', $aosProducts);
+		$this->assertInstanceOf('Basic', $aosProducts);
+		$this->assertInstanceOf('SugarBean', $aosProducts);
 			
 		$this->assertAttributeEquals('AOS_Products', 'module_dir', $aosProducts);
 		$this->assertAttributeEquals('AOS_Products', 'object_name', $aosProducts);
@@ -19,7 +19,7 @@ 
 		
 	}
 	
-	public function testsave(){
+	public function testsave() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		
@@ -27,7 +27,7 @@ 
 		
 		$aosProducts->name = "test";
 		$aosProducts->category = 1;
-		$aosProducts->product_image='test img';
+		$aosProducts->product_image = 'test img';
 		$_POST['deleteAttachment'] = '1';
 		
 		$aosProducts->save();
@@ -41,7 +41,7 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$aosProducts->mark_deleted($aosProducts->id);
 		$result = $aosProducts->retrieve($aosProducts->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
     }
 
@@ -53,7 +53,7 @@ 
 		//execute the method and verify that it retunrs expected results
 		$expected = "\n 			SELECT * FROM (\n 				SELECT\n					aos_quotes.*,\n					accounts.id AS account_id,\n					accounts.name AS billing_account,\n					\n					opportunity_id AS opportunity,\n					billing_contact_id AS billing_contact,\n					'' AS created_by_name,\n					'' AS modified_by_name,\n					'' AS assigned_user_name\n				FROM\n					aos_products\n\n				JOIN aos_products_quotes ON aos_products_quotes.product_id = aos_products.id AND aos_products.id = '1' AND aos_products_quotes.deleted = 0 AND aos_products.deleted = 0\n				JOIN aos_quotes ON aos_quotes.id = aos_products_quotes.parent_id AND aos_quotes.stage = 'Closed Accepted' AND aos_quotes.deleted = 0\n				JOIN accounts ON accounts.id = aos_quotes.billing_account_id -- AND accounts.deleted = 0\n\n				GROUP BY accounts.id\n			) AS aos_quotes\n\n		";
 		$actual = $aosProducts->getCustomersPurchasedProductsQuery();		
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}	
 	

tests/tests/modules/Tasks/TaskTest.php

Spacing +24 added lines, -24 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$task = new Task();
 		
-		$this->assertInstanceOf('Task',$task);
-		$this->assertInstanceOf('SugarBean',$task);
+		$this->assertInstanceOf('Task', $task);
+		$this->assertInstanceOf('SugarBean', $task);
 			
 		$this->assertAttributeEquals('tasks', 'table_name', $task);
 		$this->assertAttributeEquals('Tasks', 'module_dir', $task);
@@ -27,7 +27,7 @@ 
     	
     	$task = new Task();
     	
-		$task->name  = "test";
+		$task->name = "test";
 		$task->priority = "Medium";
 		$task->parent_type = "Accounts";
 		$task->status = "In Progress";
@@ -42,7 +42,7 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$task->mark_deleted($task->id);
     	$result = $task->retrieve($task->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
     	
     }
 
@@ -52,11 +52,11 @@ 
 		$task = new Task();
 		
 		//test without setting name
-		$this->assertEquals(Null,$task->get_summary_text());
+		$this->assertEquals(Null, $task->get_summary_text());
 		
 		//test with name set
 		$task->name = "test";
-		$this->assertEquals('test',$task->get_summary_text());
+		$this->assertEquals('test', $task->get_summary_text());
 
 	}
 
@@ -67,14 +67,14 @@ 
     	
     	//test with empty string params
     	$expected = "SELECT tasks.*, users.user_name as assigned_user_name  FROM tasks   LEFT JOIN users ON tasks.assigned_user_id=users.id where tasks.deleted=0 ORDER BY tasks.name";
-    	$actual = $task->create_export_query('','');
-    	$this->assertSame($expected,$actual);
+    	$actual = $task->create_export_query('', '');
+    	$this->assertSame($expected, $actual);
     	
     		
     	//test with valid string params
     	$expected = "SELECT tasks.*, users.user_name as assigned_user_name  FROM tasks   LEFT JOIN users ON tasks.assigned_user_id=users.id where users.user_name = \"\" AND tasks.deleted=0 ORDER BY tasks.name";
-    	$actual = $task->create_export_query('tasks.id','users.user_name = ""');
-    	$this->assertSame($expected,$actual);
+    	$actual = $task->create_export_query('tasks.id', 'users.user_name = ""');
+    	$this->assertSame($expected, $actual);
     	
     }
 
@@ -131,7 +131,7 @@ 
 	{
 		$task = new Task();
 		
-		$task->name  = "test";
+		$task->name = "test";
 		$task->priority = "Medium";
 		$task->parent_type = "Accounts";
 		$task->status = "In Progress";
@@ -139,7 +139,7 @@ 
 		$task->contact_phone = "1234567";
 		$task->parent_name = "test";
 		
-		$expected = array (
+		$expected = array(
 				'NAME' => 'test',
 				'DELETED' => 0,
 				'STATUS' => 'In Progress',
@@ -159,10 +159,10 @@ 
 		);
 		
 		$actual = $task->get_list_view_data();
-        foreach($expected as $expectedKey => $expectedVal){
-            if($expectedKey == 'SET_COMPLETE'){
-                $this->assertRegExp($expected[$expectedKey],$actual[$expectedKey]);
-            }else {
+        foreach ($expected as $expectedKey => $expectedVal) {
+            if ($expectedKey == 'SET_COMPLETE') {
+                $this->assertRegExp($expected[$expectedKey], $actual[$expectedKey]);
+            } else {
                 $this->assertSame($expected[$expectedKey], $actual[$expectedKey]);
             }
         }
@@ -179,17 +179,17 @@ 
 		$task->priority = "Medium";
 		$task->date_due = "2016-02-11 17:30:00";
 		$task->status = "In Progress";		
-		$task->description = "test description" ;
+		$task->description = "test description";
 		
 		$task->current_notify_user = new User(1);
 		
 		$result = $task->set_notification_body(new Sugar_Smarty(), $task);
 		
-		$this->assertEquals($task->name ,$result->_tpl_vars['TASK_SUBJECT']);
-		$this->assertEquals($task->status ,$result->_tpl_vars['TASK_STATUS']);
-		$this->assertEquals($task->priority ,$result->_tpl_vars['TASK_PRIORITY']);
-		$this->assertEquals("02/11/2016 17:30 UTC(+00:00)" ,$result->_tpl_vars['TASK_DUEDATE']);
-		$this->assertEquals($task->description ,$result->_tpl_vars['TASK_DESCRIPTION']);
+		$this->assertEquals($task->name, $result->_tpl_vars['TASK_SUBJECT']);
+		$this->assertEquals($task->status, $result->_tpl_vars['TASK_STATUS']);
+		$this->assertEquals($task->priority, $result->_tpl_vars['TASK_PRIORITY']);
+		$this->assertEquals("02/11/2016 17:30 UTC(+00:00)", $result->_tpl_vars['TASK_DUEDATE']);
+		$this->assertEquals($task->description, $result->_tpl_vars['TASK_DESCRIPTION']);
 		
 	}
 
@@ -207,9 +207,9 @@ 
 	{
 		$task = new Task();
 		
-		$expected = array("MAIN"=>"a", "PARENT"=>"a", "CONTACT"=>"a" );
+		$expected = array("MAIN"=>"a", "PARENT"=>"a", "CONTACT"=>"a");
 		$actual = $task->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 			
 	}
 

tests/tests/modules/OAuthTokens/OAuthTokenTest.php

Spacing +22 added lines, -22 removed lines

@@ -10,8 +10,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$oauthToken = new OAuthToken();
 		
-		$this->assertInstanceOf('OAuthToken',$oauthToken);
-		$this->assertInstanceOf('SugarBean',$oauthToken);
+		$this->assertInstanceOf('OAuthToken', $oauthToken);
+		$this->assertInstanceOf('SugarBean', $oauthToken);
 			
 		$this->assertAttributeEquals('OAuthTokens', 'module_dir', $oauthToken);
 		$this->assertAttributeEquals('OAuthToken', 'object_name', $oauthToken);
@@ -29,7 +29,7 @@ 
 		$oauthToken = new OAuthToken();
 		$oauthToken->setState();
 		
-		$this->assertEquals( $oauthToken->REQUEST ,$oauthToken->tstate);
+		$this->assertEquals($oauthToken->REQUEST, $oauthToken->tstate);
 		
 	}
 
@@ -43,8 +43,8 @@ 
 		
 		$oauthToken->setConsumer($oauthKey);
 		
-		$this->assertEquals( $oauthKey->id ,$oauthToken->consumer); 
-		$this->assertEquals( $oauthKey ,$oauthToken->consumer_obj); 
+		$this->assertEquals($oauthKey->id, $oauthToken->consumer); 
+		$this->assertEquals($oauthKey, $oauthToken->consumer_obj); 
 		
 	}
 
@@ -56,7 +56,7 @@ 
     	$url = "test url";
     	$oauthToken->setCallbackURL($url);
     	
-    	$this->assertEquals( $url ,$oauthToken->callback_url );
+    	$this->assertEquals($url, $oauthToken->callback_url);
     	
     }
 
@@ -66,9 +66,9 @@ 
     	
     	$result = OAuthToken::generate();
     	
-    	$this->assertInstanceOf('OAuthToken',$result);
-    	$this->assertGreaterThan(0,strlen($result->token));
-    	$this->assertGreaterThan(0,strlen($result->secret));
+    	$this->assertInstanceOf('OAuthToken', $result);
+    	$this->assertGreaterThan(0, strlen($result->token));
+    	$this->assertGreaterThan(0, strlen($result->secret));
     	
     }
     
@@ -106,7 +106,7 @@ 
     {
     	$token = OAuthToken::load($id);
     	
-    	$this->assertInstanceOf('OAuthToken',$token);
+    	$this->assertInstanceOf('OAuthToken', $token);
     	$this->assertTrue(isset($token->id));
     	
     }
@@ -116,8 +116,8 @@ 
 		
 		$token->invalidate();
 		
-		$this->assertEquals( $token::INVALID ,$token->tstate);
-		$this->assertEquals( false ,$token->verify);
+		$this->assertEquals($token::INVALID, $token->tstate);
+		$this->assertEquals(false, $token->verify);
 		
 	}
 
@@ -128,7 +128,7 @@ 
 		$result = $token->authorize("test");
 		$this->assertEquals(false, $result);
 
-		$token->tstate = $token::REQUEST ;
+		$token->tstate = $token::REQUEST;
 		$result = $token->authorize("test");
 		
 		$this->assertEquals("test", $token->authdata);
@@ -164,10 +164,10 @@ 
 		
 		$oauthToken = OAuthToken::createAuthorized($oauthKey, $user);
 		
-		$this->assertEquals( $oauthKey->id ,$oauthToken->consumer);
-		$this->assertEquals( $oauthKey ,$oauthToken->consumer_obj);
-		$this->assertEquals( $oauthToken::ACCESS ,$oauthToken->tstate);
-		$this->assertEquals( $user->id ,$oauthToken->assigned_user_id);
+		$this->assertEquals($oauthKey->id, $oauthToken->consumer);
+		$this->assertEquals($oauthKey, $oauthToken->consumer_obj);
+		$this->assertEquals($oauthToken::ACCESS, $oauthToken->tstate);
+		$this->assertEquals($user->id, $oauthToken->assigned_user_id);
 		
 		
 		//execute copyAuthData method
@@ -185,8 +185,8 @@ 
 		$oauthToken = new OAuthToken();
 		
 		$oauthToken->copyAuthData($token);
-		$this->assertEquals( $token->authdata , $oauthToken->authdata);
-		$this->assertEquals( $token->assigned_user_id , $oauthToken->assigned_user_id);
+		$this->assertEquals($token->authdata, $oauthToken->authdata);
+		$this->assertEquals($token->assigned_user_id, $oauthToken->assigned_user_id);
 		
 	}
 
@@ -196,14 +196,14 @@ 
 		$oauthToken = new OAuthToken();
 		
 		$result = $oauthToken->queryString();
-		$this->assertEquals("oauth_token=&oauth_token_secret=", $result );
+		$this->assertEquals("oauth_token=&oauth_token_secret=", $result);
 		
 
 		//test with attributes set
 		$oauthToken->token = "toekn";
 		$oauthToken->secret = "secret";
 		$result = $oauthToken->queryString();
-		$this->assertEquals("oauth_token=toekn&oauth_token_secret=secret", $result );
+		$this->assertEquals("oauth_token=toekn&oauth_token_secret=secret", $result);
 		
 	}
 
@@ -224,7 +224,7 @@ 
 	public function testcheckNonce()
 	{
 		$result = OAuthToken::checkNonce("test", "test", 123);
-		$this->assertEquals(0, $result );		
+		$this->assertEquals(0, $result);		
 	}
 
 

tests/tests/modules/Calls_Reschedule/Calls_RescheduleTest.php

Spacing +5 added lines, -5 removed lines

@@ -1,14 +1,14 @@
 <?PHP
 
-class Calls_RescheduleTest extends PHPUnit_Framework_TestCase  {
+class Calls_RescheduleTest extends PHPUnit_Framework_TestCase {
 	
-	public function testCalls_Reschedule(){	
+	public function testCalls_Reschedule() {	
 
 		//execute the contructor and check for the Object type and  attributes
 		$callsReschedule = new Calls_Reschedule();
-		$this->assertInstanceOf('Calls_Reschedule',$callsReschedule);
-		$this->assertInstanceOf('Basic',$callsReschedule);
-		$this->assertInstanceOf('SugarBean',$callsReschedule);
+		$this->assertInstanceOf('Calls_Reschedule', $callsReschedule);
+		$this->assertInstanceOf('Basic', $callsReschedule);
+		$this->assertInstanceOf('SugarBean', $callsReschedule);
 			
 		$this->assertAttributeEquals('Calls_Reschedule', 'module_dir', $callsReschedule);
 		$this->assertAttributeEquals('Calls_Reschedule', 'object_name', $callsReschedule);