adamjakab / SuiteCRM

tests/tests/modules/OAuthTokens/OAuthTokenTest.php

Indentation +162 added lines, -162 removed lines

@@ -5,58 +5,58 @@ 
 
 
     public function test__construct()
-	{
+    {
 		
-		//execute the contructor and check for the Object type and  attributes
-		$oauthToken = new OAuthToken();
+        //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);
-		$this->assertAttributeEquals('oauth_tokens', 'table_name', $oauthToken);
+        $this->assertAttributeEquals('OAuthTokens', 'module_dir', $oauthToken);
+        $this->assertAttributeEquals('OAuthToken', 'object_name', $oauthToken);
+        $this->assertAttributeEquals('oauth_tokens', 'table_name', $oauthToken);
 		
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $oauthToken);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $oauthToken);
 		
-	}
+    }
 
 
-	public function testsetState()
-	{
-		error_reporting(E_ERROR | E_PARSE);
+    public function testsetState()
+    {
+        error_reporting(E_ERROR | E_PARSE);
 		
-		$oauthToken = new OAuthToken();
-		$oauthToken->setState();
+        $oauthToken = new OAuthToken();
+        $oauthToken->setState();
 		
-		$this->assertEquals( $oauthToken->REQUEST ,$oauthToken->tstate);
+        $this->assertEquals( $oauthToken->REQUEST ,$oauthToken->tstate);
 		
-	}
+    }
 
 
-	public function testsetConsumer()
-	{
-		$oauthToken = new OAuthToken();
+    public function testsetConsumer()
+    {
+        $oauthToken = new OAuthToken();
 		
-		$oauthKey = new OAuthKey();
-		$oauthKey->id = "1";
+        $oauthKey = new OAuthKey();
+        $oauthKey->id = "1";
 		
-		$oauthToken->setConsumer($oauthKey);
+        $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); 
 		
-	}
+    }
 
 
     public function testsetCallbackURL()
     {
-    	$oauthToken = new OAuthToken();
+        $oauthToken = new OAuthToken();
     	
-    	$url = "test url";
-    	$oauthToken->setCallbackURL($url);
+        $url = "test url";
+        $oauthToken->setCallbackURL($url);
     	
-    	$this->assertEquals( $url ,$oauthToken->callback_url );
+        $this->assertEquals( $url ,$oauthToken->callback_url );
     	
     }
 
@@ -64,210 +64,210 @@ 
     public function testgenerate()
     {
     	
-    	$result = OAuthToken::generate();
+        $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));
     	
     }
     
     
     public function testSaveAndOthers()
     {
-    	$oauthToken = OAuthToken::generate();
+        $oauthToken = OAuthToken::generate();
     	
-    	$oauthToken->save();
+        $oauthToken->save();
     	
-    	//test for record ID to verify that record is saved
-    	$this->assertTrue(isset($oauthToken->id));
-    	$this->assertEquals(12, strlen($oauthToken->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($oauthToken->id));
+        $this->assertEquals(12, strlen($oauthToken->id));
     	
     	
-    	//test load method
-    	$this->load($oauthToken->id);
+        //test load method
+        $this->load($oauthToken->id);
     	
     	
-    	//test invalidate method
-    	$token = OAuthToken::load($oauthToken->id);
-    	$this->invalidate($token);
+        //test invalidate method
+        $token = OAuthToken::load($oauthToken->id);
+        $this->invalidate($token);
     	
     	
-    	//test authorize method
-    	$this->authorize($token);
+        //test authorize method
+        $this->authorize($token);
     	
     	
-    	//test mark_deleted method
-    	$this->mark_deleted($oauthToken->id);
+        //test mark_deleted method
+        $this->mark_deleted($oauthToken->id);
     	
     }
 
     public function load($id)
     {
-    	$token = OAuthToken::load($id);
+        $token = OAuthToken::load($id);
     	
-    	$this->assertInstanceOf('OAuthToken',$token);
-    	$this->assertTrue(isset($token->id));
+        $this->assertInstanceOf('OAuthToken',$token);
+        $this->assertTrue(isset($token->id));
     	
     }
     
-	public function invalidate($token)
-	{
+    public function invalidate($token)
+    {
 		
-		$token->invalidate();
+        $token->invalidate();
 		
-		$this->assertEquals( $token::INVALID ,$token->tstate);
-		$this->assertEquals( false ,$token->verify);
+        $this->assertEquals( $token::INVALID ,$token->tstate);
+        $this->assertEquals( false ,$token->verify);
 		
-	}
+    }
 
 
-	public function authorize($token)
-	{
+    public function authorize($token)
+    {
 		
-		$result = $token->authorize("test");
-		$this->assertEquals(false, $result);
+        $result = $token->authorize("test");
+        $this->assertEquals(false, $result);
 
-		$token->tstate = $token::REQUEST ;
-		$result = $token->authorize("test");
+        $token->tstate = $token::REQUEST ;
+        $result = $token->authorize("test");
 		
-		$this->assertEquals("test", $token->authdata);
-		$this->assertGreaterThan(0, strlen($result));
-		$this->assertEquals($result, $token->verify);
+        $this->assertEquals("test", $token->authdata);
+        $this->assertGreaterThan(0, strlen($result));
+        $this->assertEquals($result, $token->verify);
 		
-	}
+    }
 
 	
-	public function mark_deleted($id)
-	{
+    public function mark_deleted($id)
+    {
 		
-		$oauthToken = new OAuthToken();
+        $oauthToken = new OAuthToken();
 
-		//execute the method
-		$oauthToken->mark_deleted($id);
+        //execute the method
+        $oauthToken->mark_deleted($id);
 		
-		//verify that record can not be loaded anymore
-		$token = OAuthToken::load($id);
-		$this->assertEquals(null, $token->id);
+        //verify that record can not be loaded anymore
+        $token = OAuthToken::load($id);
+        $this->assertEquals(null, $token->id);
 
-	}
+    }
 	
 	
-	public function testcreateAuthorized()
-	{
+    public function testcreateAuthorized()
+    {
 
-		$oauthKey = new OAuthKey();
-		$oauthKey->id = "1";
+        $oauthKey = new OAuthKey();
+        $oauthKey->id = "1";
 		
-		$user = new User();
-		$user->retrieve("1");
+        $user = new User();
+        $user->retrieve("1");
 		
-		$oauthToken = OAuthToken::createAuthorized($oauthKey, $user);
+        $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
-		$oauthToken->authdata = "test";
-		$this->copyAuthData($oauthToken);
+        //execute copyAuthData method
+        $oauthToken->authdata = "test";
+        $this->copyAuthData($oauthToken);
 		
-		//finally mark deleted for cleanup
-		$oauthToken->mark_deleted($oauthToken->id);
+        //finally mark deleted for cleanup
+        $oauthToken->mark_deleted($oauthToken->id);
 		
-	}
+    }
 
 
-	public function copyAuthData($token)
-	{
-		$oauthToken = new OAuthToken();
+    public function copyAuthData($token)
+    {
+        $oauthToken = new OAuthToken();
 		
-		$oauthToken->copyAuthData($token);
-		$this->assertEquals( $token->authdata , $oauthToken->authdata);
-		$this->assertEquals( $token->assigned_user_id , $oauthToken->assigned_user_id);
+        $oauthToken->copyAuthData($token);
+        $this->assertEquals( $token->authdata , $oauthToken->authdata);
+        $this->assertEquals( $token->assigned_user_id , $oauthToken->assigned_user_id);
 		
-	}
+    }
 
 	
-	public function testqueryString()
-	{
-		$oauthToken = new OAuthToken();
+    public function testqueryString()
+    {
+        $oauthToken = new OAuthToken();
 		
-		$result = $oauthToken->queryString();
-		$this->assertEquals("oauth_token=&oauth_token_secret=", $result );
+        $result = $oauthToken->queryString();
+        $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 );
+        //test with attributes set
+        $oauthToken->token = "toekn";
+        $oauthToken->secret = "secret";
+        $result = $oauthToken->queryString();
+        $this->assertEquals("oauth_token=toekn&oauth_token_secret=secret", $result );
 		
-	}
+    }
 
-	public function testcleanup()
-	{
+    public function testcleanup()
+    {
 	
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			OAuthToken::cleanup();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
-		
-	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            OAuthToken::cleanup();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
+		
+    }
 	
-	public function testcheckNonce()
-	{
-		$result = OAuthToken::checkNonce("test", "test", 123);
-		$this->assertEquals(0, $result );		
-	}
+    public function testcheckNonce()
+    {
+        $result = OAuthToken::checkNonce("test", "test", 123);
+        $this->assertEquals(0, $result );		
+    }
 
 
-	public function testdeleteByConsumer()
-	{	
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			OAuthToken::deleteByConsumer("1");
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
-		
-	}
+    public function testdeleteByConsumer()
+    {	
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            OAuthToken::deleteByConsumer("1");
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
+		
+    }
 
 
-	public function testdeleteByUser()
-	{
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			OAuthToken::deleteByUser("1");
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
-		
-		
-	}
+    public function testdeleteByUser()
+    {
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            OAuthToken::deleteByUser("1");
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
+		
+		
+    }
 
-	public function testdisplayDateFromTs()
-	{
-		//test with empty array
-		$result = displayDateFromTs(array(''=>''), 'timestamp');
-		$this->assertEquals('', $result);
+    public function testdisplayDateFromTs()
+    {
+        //test with empty array
+        $result = displayDateFromTs(array(''=>''), 'timestamp');
+        $this->assertEquals('', $result);
 		
 		
-		//test with a valid array
-		$result = displayDateFromTs(array('TIMESTAMP'=>'1272508903'), 'timestamp');
-		$this->assertEquals('04/29/2010 02:41', $result);
+        //test with a valid array
+        $result = displayDateFromTs(array('TIMESTAMP'=>'1272508903'), 'timestamp');
+        $this->assertEquals('04/29/2010 02:41', $result);
 		
-	}
+    }
 	
 
 }

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);		
 	}
 
 

Braces +3 added lines, -6 removed lines

@@ -214,8 +214,7 @@ 
 		try {
 			OAuthToken::cleanup();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -234,8 +233,7 @@ 
 		try {
 			OAuthToken::deleteByConsumer("1");
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -248,8 +246,7 @@ 
 		try {
 			OAuthToken::deleteByUser("1");
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		

tests/tests/modules/Calls_Reschedule/Calls_RescheduleTest.php

Indentation +14 added lines, -14 removed lines

@@ -2,23 +2,23 @@
 
 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);
+        //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->assertAttributeEquals('Calls_Reschedule', 'module_dir', $callsReschedule);
-		$this->assertAttributeEquals('Calls_Reschedule', 'object_name', $callsReschedule);
-		$this->assertAttributeEquals('calls_reschedule', 'table_name', $callsReschedule);
-		$this->assertAttributeEquals(true, 'new_schema', $callsReschedule);
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $callsReschedule);
-		$this->assertAttributeEquals(true, 'importable', $callsReschedule);
-		$this->assertAttributeEquals(false, 'tracker_visibility', $callsReschedule);
+        $this->assertAttributeEquals('Calls_Reschedule', 'module_dir', $callsReschedule);
+        $this->assertAttributeEquals('Calls_Reschedule', 'object_name', $callsReschedule);
+        $this->assertAttributeEquals('calls_reschedule', 'table_name', $callsReschedule);
+        $this->assertAttributeEquals(true, 'new_schema', $callsReschedule);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $callsReschedule);
+        $this->assertAttributeEquals(true, 'importable', $callsReschedule);
+        $this->assertAttributeEquals(false, 'tracker_visibility', $callsReschedule);
 		
-	}
+    }
 
 }
 ?>
\ No newline at end of file

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);

tests/tests/modules/DocumentRevisions/DocumentRevisionTest.php

Indentation +135 added lines, -135 removed lines

@@ -3,200 +3,200 @@
 class DocumentRevisionTest extends PHPUnit_Framework_TestCase {
 
 	
-	public function testDocumentRevision() 
-	{
-		//execute the contructor and check for the Object type and  attributes
-		$documentRevision = new DocumentRevision();
-		$this->assertInstanceOf('DocumentRevision',$documentRevision);
-		$this->assertInstanceOf('SugarBean',$documentRevision);
+    public function testDocumentRevision() 
+    {
+        //execute the contructor and check for the Object type and  attributes
+        $documentRevision = new DocumentRevision();
+        $this->assertInstanceOf('DocumentRevision',$documentRevision);
+        $this->assertInstanceOf('SugarBean',$documentRevision);
 			
-		$this->assertAttributeEquals('DocumentRevisions', 'module_dir', $documentRevision);
-		$this->assertAttributeEquals('DocumentRevision', 'object_name', $documentRevision);
-		$this->assertAttributeEquals('document_revisions', 'table_name', $documentRevision);
-		$this->assertAttributeEquals(true, 'new_schema', $documentRevision);
+        $this->assertAttributeEquals('DocumentRevisions', 'module_dir', $documentRevision);
+        $this->assertAttributeEquals('DocumentRevision', 'object_name', $documentRevision);
+        $this->assertAttributeEquals('document_revisions', 'table_name', $documentRevision);
+        $this->assertAttributeEquals(true, 'new_schema', $documentRevision);
 		
-	}
+    }
 
-	public function testSaveAndRetrieve()
-	{	
-		error_reporting(E_ERROR | E_PARSE);
+    public function testSaveAndRetrieve()
+    {	
+        error_reporting(E_ERROR | E_PARSE);
 
-		$documentRevision = new DocumentRevision();
+        $documentRevision = new DocumentRevision();
 
-		$documentRevision->document_id = "1";
-		$documentRevision->doc_id = "1";
-		$documentRevision->doc_type = "text";		
-		$documentRevision->filename = "test";
-		$documentRevision->file_ext = "ext";
+        $documentRevision->document_id = "1";
+        $documentRevision->doc_id = "1";
+        $documentRevision->doc_type = "text";		
+        $documentRevision->filename = "test";
+        $documentRevision->file_ext = "ext";
 		
-		$documentRevision->save();
+        $documentRevision->save();
 			
-		//test for record ID to verify that record is saved
-		$this->assertTrue(isset($documentRevision->id));
-		$this->assertEquals(36, strlen($documentRevision->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($documentRevision->id));
+        $this->assertEquals(36, strlen($documentRevision->id));
 
 		
 		
-		//test document retrieve method
-		$docRev = $documentRevision->retrieve($documentRevision->id);
-		$this->assertEquals("1", $docRev->document_id);
-		$this->assertEquals("1", $docRev->doc_id);
-		$this->assertEquals("text", $docRev->doc_type );
-		$this->assertEquals("test", $docRev->filename);
-		$this->assertEquals("ext", $docRev->file_ext);
+        //test document retrieve method
+        $docRev = $documentRevision->retrieve($documentRevision->id);
+        $this->assertEquals("1", $docRev->document_id);
+        $this->assertEquals("1", $docRev->doc_id);
+        $this->assertEquals("text", $docRev->doc_type );
+        $this->assertEquals("test", $docRev->filename);
+        $this->assertEquals("ext", $docRev->file_ext);
 		
 		
 		
-		//mark the record as deleted and verify that this record cannot be retrieved anymore.
-		$docRev->mark_deleted($docRev->id);
-		$result = $docRev->retrieve($docRev->id);
-		$this->assertEquals(null,$result);
+        //mark the record as deleted and verify that this record cannot be retrieved anymore.
+        $docRev->mark_deleted($docRev->id);
+        $result = $docRev->retrieve($docRev->id);
+        $this->assertEquals(null,$result);
 		
-	}
+    }
 
 	
-	public function testget_summary_text()
-	{
+    public function testget_summary_text()
+    {
 		
-		$documentRevision = new DocumentRevision();
+        $documentRevision = new DocumentRevision();
 		
-		//test without setting name
-		$this->assertEquals(Null,$documentRevision->get_summary_text());
+        //test without setting name
+        $this->assertEquals(Null,$documentRevision->get_summary_text());
 		
-		//test with name set
-		$documentRevision->filename = "test";
-		$this->assertEquals('test',$documentRevision->get_summary_text());
+        //test with name set
+        $documentRevision->filename = "test";
+        $this->assertEquals('test',$documentRevision->get_summary_text());
 	
-	}
+    }
 
 
 
-	public function testis_authenticated()
-	{
+    public function testis_authenticated()
+    {
 
-		$documentRevision = new DocumentRevision();
+        $documentRevision = new DocumentRevision();
 		
-		//test wihout setting attributes
-		$this->assertEquals(null,$documentRevision->is_authenticated());
+        //test wihout setting attributes
+        $this->assertEquals(null,$documentRevision->is_authenticated());
 		
 		
-		//test with attributes preset
-		$documentRevision->authenticated= true;
-		$this->assertEquals(true,$documentRevision->is_authenticated());
-	}
+        //test with attributes preset
+        $documentRevision->authenticated= true;
+        $this->assertEquals(true,$documentRevision->is_authenticated());
+    }
 
-	public function testfill_in_additional_list_fields() 
-	{
+    public function testfill_in_additional_list_fields() 
+    {
 
-		$documentRevision = new DocumentRevision();
+        $documentRevision = new DocumentRevision();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$documentRevision->fill_in_additional_list_fields();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $documentRevision->fill_in_additional_list_fields();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
-	}
+    }
 
-	public function testfill_in_additional_detail_fields()
-	{
-		$documentRevision = new DocumentRevision();
+    public function testfill_in_additional_detail_fields()
+    {
+        $documentRevision = new DocumentRevision();
 		
-			//execute the method and test if it works and does not throws an exception.
-		try {
-			$documentRevision->fill_in_additional_detail_fields();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+            //execute the method and test if it works and does not throws an exception.
+        try {
+            $documentRevision->fill_in_additional_detail_fields();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
-	}
+    }
 	
 
-	public function testgetDocumentRevisionNameForDisplay() 
-	{
-		$documentRevision = new DocumentRevision();
+    public function testgetDocumentRevisionNameForDisplay() 
+    {
+        $documentRevision = new DocumentRevision();
 		
-		//test wihout setting attributes
-		$result = $documentRevision->getDocumentRevisionNameForDisplay();
-		$this->assertEquals(".",$result);
+        //test wihout setting attributes
+        $result = $documentRevision->getDocumentRevisionNameForDisplay();
+        $this->assertEquals(".",$result);
 		
 		
-		//test with attributes preset
-		$documentRevision->filename = "test.ext";
-		$documentRevision->revision = 1;
-		$result = $documentRevision->getDocumentRevisionNameForDisplay();
-		$this->assertEquals("-Revision_1.ext",$result);
+        //test with attributes preset
+        $documentRevision->filename = "test.ext";
+        $documentRevision->revision = 1;
+        $result = $documentRevision->getDocumentRevisionNameForDisplay();
+        $this->assertEquals("-Revision_1.ext",$result);
 		
-	}
+    }
 
-	public function testfill_document_name_revision() 
-	{
+    public function testfill_document_name_revision() 
+    {
 		
-		$documentRevision = new DocumentRevision();
+        $documentRevision = new DocumentRevision();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$documentRevision->fill_document_name_revision();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $documentRevision->fill_document_name_revision();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 	
-	public function testlist_view_parse_additional_sections(){
+    public function testlist_view_parse_additional_sections(){
 
-		$documentRevision = new DocumentRevision();
+        $documentRevision = new DocumentRevision();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$documentRevision->list_view_parse_additional_sections(new Sugar_Smarty(), $xTemplateSection);
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $documentRevision->list_view_parse_additional_sections(new Sugar_Smarty(), $xTemplateSection);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
-	}
+    }
 	
-	public function testget_list_view_data()
-	{
-		$documentRevision = new DocumentRevision();
-		$result = $documentRevision->get_list_view_data();
-		$this->assertEquals( array("DELETED"=>0), $result);
+    public function testget_list_view_data()
+    {
+        $documentRevision = new DocumentRevision();
+        $result = $documentRevision->get_list_view_data();
+        $this->assertEquals( array("DELETED"=>0), $result);
 			
-	}
+    }
 
 
-	public function testget_document_revision_name()
-	{
-		$documentRevision = new DocumentRevision();
-		$result = $documentRevision->get_document_revision_name(1);
-		$this->assertEquals(null,$result);
+    public function testget_document_revision_name()
+    {
+        $documentRevision = new DocumentRevision();
+        $result = $documentRevision->get_document_revision_name(1);
+        $this->assertEquals(null,$result);
 		
-	}
+    }
 	
 
-	public function testget_document_revisions()
-	{
-		$documentRevision = new DocumentRevision();
-		$results = $documentRevision->get_document_revisions(1);
-		$this->assertTrue(is_array($results));
+    public function testget_document_revisions()
+    {
+        $documentRevision = new DocumentRevision();
+        $results = $documentRevision->get_document_revisions(1);
+        $this->assertTrue(is_array($results));
 		
-	}	
+    }	
 
     public function testbean_implements() {
 
-    	$documentRevision = new DocumentRevision();
-    	$this->assertEquals(false, $documentRevision->bean_implements('')); //test with blank value
-    	$this->assertEquals(false, $documentRevision->bean_implements('test')); //test with invalid value
-    	$this->assertEquals(true, $documentRevision->bean_implements('FILE')); //test with valid value
+        $documentRevision = new DocumentRevision();
+        $this->assertEquals(false, $documentRevision->bean_implements('')); //test with blank value
+        $this->assertEquals(false, $documentRevision->bean_implements('test')); //test with invalid value
+        $this->assertEquals(true, $documentRevision->bean_implements('FILE')); //test with valid value
 
     }
 

Spacing +14 added lines, -14 removed lines

@@ -7,8 +7,8 @@ 
 	{
 		//execute the contructor and check for the Object type and  attributes
 		$documentRevision = new DocumentRevision();
-		$this->assertInstanceOf('DocumentRevision',$documentRevision);
-		$this->assertInstanceOf('SugarBean',$documentRevision);
+		$this->assertInstanceOf('DocumentRevision', $documentRevision);
+		$this->assertInstanceOf('SugarBean', $documentRevision);
 			
 		$this->assertAttributeEquals('DocumentRevisions', 'module_dir', $documentRevision);
 		$this->assertAttributeEquals('DocumentRevision', 'object_name', $documentRevision);
@@ -41,7 +41,7 @@ 
 		$docRev = $documentRevision->retrieve($documentRevision->id);
 		$this->assertEquals("1", $docRev->document_id);
 		$this->assertEquals("1", $docRev->doc_id);
-		$this->assertEquals("text", $docRev->doc_type );
+		$this->assertEquals("text", $docRev->doc_type);
 		$this->assertEquals("test", $docRev->filename);
 		$this->assertEquals("ext", $docRev->file_ext);
 		
@@ -50,7 +50,7 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$docRev->mark_deleted($docRev->id);
 		$result = $docRev->retrieve($docRev->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 
@@ -61,11 +61,11 @@ 
 		$documentRevision = new DocumentRevision();
 		
 		//test without setting name
-		$this->assertEquals(Null,$documentRevision->get_summary_text());
+		$this->assertEquals(Null, $documentRevision->get_summary_text());
 		
 		//test with name set
 		$documentRevision->filename = "test";
-		$this->assertEquals('test',$documentRevision->get_summary_text());
+		$this->assertEquals('test', $documentRevision->get_summary_text());
 	
 	}
 
@@ -77,12 +77,12 @@ 
 		$documentRevision = new DocumentRevision();
 		
 		//test wihout setting attributes
-		$this->assertEquals(null,$documentRevision->is_authenticated());
+		$this->assertEquals(null, $documentRevision->is_authenticated());
 		
 		
 		//test with attributes preset
-		$documentRevision->authenticated= true;
-		$this->assertEquals(true,$documentRevision->is_authenticated());
+		$documentRevision->authenticated = true;
+		$this->assertEquals(true, $documentRevision->is_authenticated());
 	}
 
 	public function testfill_in_additional_list_fields() 
@@ -123,14 +123,14 @@ 
 		
 		//test wihout setting attributes
 		$result = $documentRevision->getDocumentRevisionNameForDisplay();
-		$this->assertEquals(".",$result);
+		$this->assertEquals(".", $result);
 		
 		
 		//test with attributes preset
 		$documentRevision->filename = "test.ext";
 		$documentRevision->revision = 1;
 		$result = $documentRevision->getDocumentRevisionNameForDisplay();
-		$this->assertEquals("-Revision_1.ext",$result);
+		$this->assertEquals("-Revision_1.ext", $result);
 		
 	}
 
@@ -150,7 +150,7 @@ 
 		
 	}
 	
-	public function testlist_view_parse_additional_sections(){
+	public function testlist_view_parse_additional_sections() {
 
 		$documentRevision = new DocumentRevision();
 		
@@ -169,7 +169,7 @@ 
 	{
 		$documentRevision = new DocumentRevision();
 		$result = $documentRevision->get_list_view_data();
-		$this->assertEquals( array("DELETED"=>0), $result);
+		$this->assertEquals(array("DELETED"=>0), $result);
 			
 	}
 
@@ -178,7 +178,7 @@ 
 	{
 		$documentRevision = new DocumentRevision();
 		$result = $documentRevision->get_document_revision_name(1);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 	

Braces +4 added lines, -8 removed lines

@@ -94,8 +94,7 @@ 
 		try {
 			$documentRevision->fill_in_additional_list_fields();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 
@@ -109,8 +108,7 @@ 
 		try {
 			$documentRevision->fill_in_additional_detail_fields();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 
@@ -143,8 +141,7 @@ 
 		try {
 			$documentRevision->fill_document_name_revision();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -158,8 +155,7 @@ 
 		try {
 			$documentRevision->list_view_parse_additional_sections(new Sugar_Smarty(), $xTemplateSection);
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 

tests/tests/modules/ProjectTask/ProjectTaskTest.php

Indentation +199 added lines, -199 removed lines

@@ -3,316 +3,316 @@
 class ProjectTaskTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testProjectTask()
-	{
-		//execute the contructor and check for the Object type and  attributes
-		$projectTask = new ProjectTask();
+    public function testProjectTask()
+    {
+        //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);
-		$this->assertAttributeEquals('ProjectTask', 'module_dir', $projectTask);
-		$this->assertAttributeEquals('ProjectTask', 'object_name', $projectTask);
+        $this->assertAttributeEquals('project_task', 'table_name', $projectTask);
+        $this->assertAttributeEquals('ProjectTask', 'module_dir', $projectTask);
+        $this->assertAttributeEquals('ProjectTask', 'object_name', $projectTask);
 		
-		$this->assertAttributeEquals(true, 'new_schema', $projectTask);
-		$this->assertAttributeEquals(false, '_skipParentUpdate', $projectTask);
+        $this->assertAttributeEquals(true, 'new_schema', $projectTask);
+        $this->assertAttributeEquals(false, '_skipParentUpdate', $projectTask);
 		
-		$this->assertAttributeEquals(100, 'utilization', $projectTask);
+        $this->assertAttributeEquals(100, 'utilization', $projectTask);
 
-	}
+    }
 
     public function testskipParentUpdate()
     {
-    	error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
     	
-    	$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
     	
-    	//test with default parameter value
-    	$projectTask->skipParentUpdate();
-    	$this->assertAttributeEquals(true, '_skipParentUpdate', $projectTask);
+        //test with default parameter value
+        $projectTask->skipParentUpdate();
+        $this->assertAttributeEquals(true, '_skipParentUpdate', $projectTask);
     	
-    	//test with parameter value  = true
-    	$projectTask->skipParentUpdate(false);
-    	$this->assertAttributeEquals(false, '_skipParentUpdate', $projectTask);
+        //test with parameter value  = true
+        $projectTask->skipParentUpdate(false);
+        $this->assertAttributeEquals(false, '_skipParentUpdate', $projectTask);
     	
     }
 	
-	public function testsave()
-	{
-		$projectTask = new ProjectTask();
+    public function testsave()
+    {
+        $projectTask = new ProjectTask();
 		
-		$projectTask->name = "test";
-		//$projectTask->project_id = "1";
-		$projectTask->assigned_user_id = "1";
-		$projectTask->description = "test description";
-		$projectTask->parent_task_id =1;
+        $projectTask->name = "test";
+        //$projectTask->project_id = "1";
+        $projectTask->assigned_user_id = "1";
+        $projectTask->description = "test description";
+        $projectTask->parent_task_id =1;
 		
-		$projectTask->save();
+        $projectTask->save();
 		
-		//test for record ID to verify that record is saved
-		$this->assertTrue(isset($projectTask->id));
-		$this->assertEquals(36, strlen($projectTask->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($projectTask->id));
+        $this->assertEquals(36, strlen($projectTask->id));
 		
 		
-		//test _get_depends_on_name method
-		$this->_get_depends_on_name($projectTask->id);
+        //test _get_depends_on_name method
+        $this->_get_depends_on_name($projectTask->id);
 		
 		
 	
-		//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);
+        //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);
 		
-	}
+    }
 	
 	
-	public function _get_depends_on_name($id)
-	{
-		$projectTask = new ProjectTask();
+    public function _get_depends_on_name($id)
+    {
+        $projectTask = new ProjectTask();
 		
-		$result = $projectTask->_get_depends_on_name($id);
+        $result = $projectTask->_get_depends_on_name($id);
 		
-		$this->assertEquals("1", $projectTask->depends_on_name_owner) ;
-		$this->assertEquals("ProjectTask", $projectTask->depends_on_name_mod);
-		$this->assertEquals("test", $result);
+        $this->assertEquals("1", $projectTask->depends_on_name_owner) ;
+        $this->assertEquals("ProjectTask", $projectTask->depends_on_name_mod);
+        $this->assertEquals("test", $result);
 			
-	}
+    }
 	
 	
-	public function testupdateParentProjectTaskPercentage()
-	{
+    public function testupdateParentProjectTaskPercentage()
+    {
 		
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$projectTask->updateParentProjectTaskPercentage();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $projectTask->updateParentProjectTaskPercentage();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 	
 	
-	public function testgetProjectTaskParent()
-	{
+    public function testgetProjectTaskParent()
+    {
 		
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		
-		$projectTask->parent_task_id =1;		
-		$result = $projectTask->getProjectTaskParent();
-		$this->assertEquals(false, $result);
+        $projectTask->parent_task_id =1;		
+        $result = $projectTask->getProjectTaskParent();
+        $this->assertEquals(false, $result);
 		
-	}
+    }
 	
 	
-	public function testgetAllSubProjectTasks()
-	{
+    public function testgetAllSubProjectTasks()
+    {
 		
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		
-		$result = $projectTask->getAllSubProjectTasks();
-		$this->assertTrue(is_array($result));
+        $result = $projectTask->getAllSubProjectTasks();
+        $this->assertTrue(is_array($result));
 	
-	}
+    }
 	
 	
-	public function testupdateStatistic()
-	{
+    public function testupdateStatistic()
+    {
 		
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$projectTask->updateStatistic();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $projectTask->updateStatistic();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 	
 	
-	public function testfill_in_additional_detail_fields()
-	{
+    public function testfill_in_additional_detail_fields()
+    {
    	
-	   	$projectTask = new ProjectTask();
+            $projectTask = new ProjectTask();
 	   	
-	   	//test without setting assigned_user_id
-	   	$projectTask->fill_in_additional_detail_fields();
-	   	$this->assertEquals("", $projectTask->assigned_user_name);
+            //test without setting assigned_user_id
+            $projectTask->fill_in_additional_detail_fields();
+            $this->assertEquals("", $projectTask->assigned_user_name);
 	   	
 	   	
-	   	//test with assigned_user_id set
-	   	$projectTask->assigned_user_id = 1;
-	   	$projectTask->fill_in_additional_detail_fields();
-	   	$this->assertEquals("Administrator", $projectTask->assigned_user_name);
+            //test with assigned_user_id set
+            $projectTask->assigned_user_id = 1;
+            $projectTask->fill_in_additional_detail_fields();
+            $this->assertEquals("Administrator", $projectTask->assigned_user_name);
    	
-	}
+    }
 
 
-	public function testfill_in_additional_list_fields()
-	{
+    public function testfill_in_additional_list_fields()
+    {
    	
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		   	 
-		//test without setting assigned_user_id
-		$projectTask->fill_in_additional_list_fields();
-		$this->assertEquals("", $projectTask->assigned_user_name);
+        //test without setting assigned_user_id
+        $projectTask->fill_in_additional_list_fields();
+        $this->assertEquals("", $projectTask->assigned_user_name);
 		   	 
 		   	 
-		//test with assigned_user_id set
-		$projectTask->assigned_user_id = 1;
-		$projectTask->fill_in_additional_list_fields();
-		$this->assertEquals("Administrator", $projectTask->assigned_user_name);
+        //test with assigned_user_id set
+        $projectTask->assigned_user_id = 1;
+        $projectTask->fill_in_additional_list_fields();
+        $this->assertEquals("Administrator", $projectTask->assigned_user_name);
 
-	}
+    }
 
 
-	public function testget_summary_text()
-	{
-		$projectTask = new ProjectTask();
+    public function testget_summary_text()
+    {
+        $projectTask = new ProjectTask();
 		
-		//test without setting name
-		$this->assertEquals(Null,$projectTask->get_summary_text());
+        //test without setting name
+        $this->assertEquals(Null,$projectTask->get_summary_text());
 		
-		//test with name set
-		$projectTask->name = "test";
-		$this->assertEquals('test',$projectTask->get_summary_text());
+        //test with name set
+        $projectTask->name = "test";
+        $this->assertEquals('test',$projectTask->get_summary_text());
 		
-	}
+    }
 
 	
-	public function test_get_project_name()
-	{
-		$projectTask = new ProjectTask();
+    public function test_get_project_name()
+    {
+        $projectTask = new ProjectTask();
 	
-		//test with a empty string
-		$result = $projectTask->_get_project_name('');
-		$this->assertEquals("", $result);
+        //test with a empty string
+        $result = $projectTask->_get_project_name('');
+        $this->assertEquals("", $result);
 	
 	
-		//test with a non empty invalid id
-		$result = $projectTask->_get_project_name('1');
-		$this->assertEquals("", $result);
+        //test with a non empty invalid id
+        $result = $projectTask->_get_project_name('1');
+        $this->assertEquals("", $result);
 	
-	}
+    }
 	
 	
-	public function test_get_parent_name()
-	{
-		$projectTask = new ProjectTask();
+    public function test_get_parent_name()
+    {
+        $projectTask = new ProjectTask();
 	
-		//test with a empty string
-		$result = $projectTask->_get_parent_name('');
-		$this->assertEquals("", $result);
+        //test with a empty string
+        $result = $projectTask->_get_parent_name('');
+        $this->assertEquals("", $result);
 	
 	
-		//test with a non empty invalid id
-		$result = $projectTask->_get_parent_name('1');
-		$this->assertEquals("", $result);
+        //test with a non empty invalid id
+        $result = $projectTask->_get_parent_name('1');
+        $this->assertEquals("", $result);
 	
-	}
+    }
 	
 	
-	public function testbuild_generic_where_clause()
-	{
-		$projectTask = new ProjectTask();
+    public function testbuild_generic_where_clause()
+    {
+        $projectTask = new ProjectTask();
 		
-		//test with empty string params
-		$expected = "project_task.name like '%'";
-		$actual = $projectTask->build_generic_where_clause('');
-		$this->assertSame($expected,$actual);
+        //test with empty string params
+        $expected = "project_task.name like '%'";
+        $actual = $projectTask->build_generic_where_clause('');
+        $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);
+        //test with valid string params
+        $expected = "project_task.name like '%'";
+        $actual = $projectTask->build_generic_where_clause('test');
+        $this->assertSame($expected,$actual);
 		
-	}
+    }
 
-	public function testget_list_view_data(){
+    public function testget_list_view_data(){
 	
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		
-		$projectTask->name = "tes user";
-		$projectTask->description = "test assigned user";
-		$projectTask->parent_type = "Project";
+        $projectTask->name = "tes user";
+        $projectTask->description = "test assigned user";
+        $projectTask->parent_type = "Project";
 		
-		$expected = array (
-				'NAME' => 'tes user',
-				'DESCRIPTION' => 'test assigned user',
-				'ORDER_NUMBER' => '1',
-				'DELETED' => 0,
-				'UTILIZATION' => 100,
-				'PARENT_MODULE' => 'Project',
-				'FIRST_NAME' => '',
-				'LAST_NAME' => '',
-				'CONTACT_NAME' => ' ',
-				'TITLE' => ':  '
-		);
+        $expected = array (
+                'NAME' => 'tes user',
+                'DESCRIPTION' => 'test assigned user',
+                'ORDER_NUMBER' => '1',
+                'DELETED' => 0,
+                'UTILIZATION' => 100,
+                'PARENT_MODULE' => 'Project',
+                'FIRST_NAME' => '',
+                'LAST_NAME' => '',
+                'CONTACT_NAME' => ' ',
+                'TITLE' => ':  '
+        );
 		
-		$actual = $projectTask->get_list_view_data();
+        $actual = $projectTask->get_list_view_data();
 		
-		$this->assertSame($expected, $actual);
+        $this->assertSame($expected, $actual);
 		
 		
-	}
+    }
 
-	public function testbean_implements(){
+    public function testbean_implements(){
 	
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		
-		$this->assertEquals(false, $projectTask->bean_implements('')); //test with blank value
-		$this->assertEquals(false, $projectTask->bean_implements('test')); //test with invalid value
-		$this->assertEquals(true, $projectTask->bean_implements('ACL')); //test with valid value
-	}
+        $this->assertEquals(false, $projectTask->bean_implements('')); //test with blank value
+        $this->assertEquals(false, $projectTask->bean_implements('test')); //test with invalid value
+        $this->assertEquals(true, $projectTask->bean_implements('ACL')); //test with valid value
+    }
 	
-	public function testlistviewACLHelper(){
+    public function testlistviewACLHelper(){
 	
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		
-		$expected = array( 'MAIN' => 'a', 'PARENT' => 'a', 'PARENT_TASK' => 'a' );
-		$actual = $projectTask->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+        $expected = array( 'MAIN' => 'a', 'PARENT' => 'a', 'PARENT_TASK' => 'a' );
+        $actual = $projectTask->listviewACLHelper();
+        $this->assertSame($expected,$actual);
 		
-	}
+    }
 
-	public function testcreate_export_query()
-	{
+    public function testcreate_export_query()
+    {
 
-		$projectTask = new ProjectTask();
+        $projectTask = new ProjectTask();
 		
-		//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);
+        //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);
 
 	
-		//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);
+        //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);
 		
-	}
+    }
 
 
-	public function testgetUtilizationDropdown()
-	{
-		$projectTask = new ProjectTask();
+    public function testgetUtilizationDropdown()
+    {
+        $projectTask = new ProjectTask();
 		
-		$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);
+        $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);
 		
-	}
+    }
     
 }
 

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);
 		
 	}
     

Braces +2 added lines, -4 removed lines

@@ -91,8 +91,7 @@ 
 		try {
 			$projectTask->updateParentProjectTaskPercentage();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -131,8 +130,7 @@ 
 		try {
 			$projectTask->updateStatistic();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		

tests/tests/modules/Documents/DocumentTest.php

Indentation +144 added lines, -144 removed lines

@@ -5,192 +5,192 @@ 
 class DocumentTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testDocument() {
+    public function testDocument() {
 	
-		//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);
+        //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->assertAttributeEquals('Documents', 'module_dir', $document);
-		$this->assertAttributeEquals('Document', 'object_name', $document);
-		$this->assertAttributeEquals('documents', 'table_name', $document);
-		$this->assertAttributeEquals(true, 'new_schema', $document);
-		$this->assertAttributeEquals(false, 'disable_row_level_security', $document);
+        $this->assertAttributeEquals('Documents', 'module_dir', $document);
+        $this->assertAttributeEquals('Document', 'object_name', $document);
+        $this->assertAttributeEquals('documents', 'table_name', $document);
+        $this->assertAttributeEquals(true, 'new_schema', $document);
+        $this->assertAttributeEquals(false, 'disable_row_level_security', $document);
 		
 		
-	}
+    }
 
-	public function testSaveAndGet_document_name() {
+    public function testSaveAndGet_document_name() {
 
-		error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 		
-		$document = new Document();
+        $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->category_id = "1";
-		$document->subcategory_id = "1" ;
-		$document->document_name = "test";
+        $document->filename = "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->document_name = "test";
 		
-		$document->save();
+        $document->save();
 		
-		//test for record ID to verify that record is saved
-		$this->assertTrue(isset($document->id));
-		$this->assertEquals(36, strlen($document->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($document->id));
+        $this->assertEquals(36, strlen($document->id));
 		
 		
 		
-		//execute Get_document_name() method and verify it gets the name correctly
-		$this->assertEquals(null, $document->get_document_name(1));
-		$this->assertEquals('test', $document->get_document_name($document->id));
+        //execute Get_document_name() method and verify it gets the name correctly
+        $this->assertEquals(null, $document->get_document_name(1));
+        $this->assertEquals('test', $document->get_document_name($document->id));
 		
 		
-		//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);
+        //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);
 
-	}
+    }
 
-	public function testget_summary_text() {
+    public function testget_summary_text() {
 
 		
-		$document = new Document();
+        $document = new Document();
 		
-		//test without setting name
-		$this->assertEquals(Null,$document->get_summary_text());
+        //test without setting name
+        $this->assertEquals(Null,$document->get_summary_text());
 		
-		//test with name set
-		$document->document_name = "test";
-		$this->assertEquals('test',$document->get_summary_text());
+        //test with name set
+        $document->document_name = "test";
+        $this->assertEquals('test',$document->get_summary_text());
 		
-	}
+    }
 
-	public function testis_authenticated() 
-	{
+    public function testis_authenticated() 
+    {
 
-		$document = new Document();
+        $document = new Document();
 		
-		//test without presetting attributes
-		$this->assertEquals(null,$document->is_authenticated());
+        //test without presetting attributes
+        $this->assertEquals(null,$document->is_authenticated());
 		
 		
-		//test with attributes preset
-		$document->authenticated= true;
-		$this->assertEquals(true,$document->is_authenticated());
+        //test with attributes preset
+        $document->authenticated= true;
+        $this->assertEquals(true,$document->is_authenticated());
 		
-	}
+    }
 
-	public function testfill_in_additional_list_fields() {
+    public function testfill_in_additional_list_fields() {
 
-		$document = new Document();
+        $document = new Document();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$document->fill_in_additional_list_fields();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $document->fill_in_additional_list_fields();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
-	}
+    }
 
-	public function testfill_in_additional_detail_fields() 
-	{
+    public function testfill_in_additional_detail_fields() 
+    {
 
-		$document = new Document();
+        $document = new Document();
 
-		$document->id = "abcde-12345";
+        $document->id = "abcde-12345";
 
-		//execute the method with attributes preset and verify attributes are set accordingly
-		$document->fill_in_additional_detail_fields();
+        //execute the method with attributes preset and verify attributes are set accordingly
+        $document->fill_in_additional_detail_fields();
 
-		$this->assertRegExp(
+        $this->assertRegExp(
             '~'
             . preg_quote("<a href='index.php?entryPoint=download&id=&type=Documents' target='_blank'><img src=\"themes/SuiteR/images/def_image_inline.gif?v=")
             . '[\w-]+'
             .preg_quote("\"    border=\"0\" alt=\"View\" /></a>")
             . '~',
             $document->file_url);
-		$this->assertEquals("index.php?entryPoint=download&type=Documents&id=", $document->file_url_noimage);
+        $this->assertEquals("index.php?entryPoint=download&type=Documents&id=", $document->file_url_noimage);
 		
-	}
+    }
 
-	public function testlist_view_parse_additional_sections() {
+    public function testlist_view_parse_additional_sections() {
 
-		$document = new Document();
+        $document = new Document();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$document->list_view_parse_additional_sections(new Sugar_Smarty(), $xTemplateSection);
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $document->list_view_parse_additional_sections(new Sugar_Smarty(), $xTemplateSection);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 
     public function testcreate_export_query()
     {
-    	$document = new Document();
+        $document = new Document();
 
-    	//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);
+        //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);
     	
     	
-    	//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);
+        //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);
     	
-	}
-
-	public function testget_list_view_data() 
-	{
-		$document = new Document();
-
-		//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->category_id = "1";
-		$document->subcategory_id = "1" ;
-		$document->document_name = "test";
-		
-		
-		$expected = array (
-				'DELETED' => 0,
-				'DOCUMENT_NAME' => 'test',
-				'DOC_TYPE' => 'Sugar',
-				'FILENAME' => 'test',
-				'ACTIVE_DATE' => $document->active_date,
-				'CATEGORY_ID' => NULL,
-				'SUBCATEGORY_ID' => NULL,
-				'REVISION' => '1',
-				'LAST_REV_CREATED_NAME' => 'test',
-				'IS_TEMPLATE' => '0',
-				'FILE_URL' => '~'
+    }
+
+    public function testget_list_view_data() 
+    {
+        $document = new Document();
+
+        //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->category_id = "1";
+        $document->subcategory_id = "1" ;
+        $document->document_name = "test";
+		
+		
+        $expected = array (
+                'DELETED' => 0,
+                'DOCUMENT_NAME' => 'test',
+                'DOC_TYPE' => 'Sugar',
+                'FILENAME' => 'test',
+                'ACTIVE_DATE' => $document->active_date,
+                'CATEGORY_ID' => NULL,
+                'SUBCATEGORY_ID' => NULL,
+                'REVISION' => '1',
+                'LAST_REV_CREATED_NAME' => 'test',
+                'IS_TEMPLATE' => '0',
+                'FILE_URL' => '~'
                                 . preg_quote('<a href=\'index.php?entryPoint=download&id=&type=Documents\' target=\'_blank\'><img src="themes/SuiteR/images/def_image_inline.gif?v=')
                                 . '[\w-]+'
                                 . preg_quote('"    border="0" alt="View" /></a>')
                                 . '~',
-				'FILE_URL_NOIMAGE' => 'index.php?entryPoint=download&type=Documents&id=',
-				'LAST_REV_CREATED_BY' => 'test',
-				'NAME' => 'test',
-				'DOCUMENT_NAME_JAVASCRIPT' => NULL,
-		);
+                'FILE_URL_NOIMAGE' => 'index.php?entryPoint=download&type=Documents&id=',
+                'LAST_REV_CREATED_BY' => 'test',
+                'NAME' => 'test',
+                'DOCUMENT_NAME_JAVASCRIPT' => NULL,
+        );
 
-		$actual = $document->get_list_view_data();
+        $actual = $document->get_list_view_data();
         foreach($expected as $expectedKey => $expectedVal){
             if($expectedKey == 'FILE_URL'){
                 $this->assertRegExp($expected[$expectedKey],$actual[$expectedKey]);
@@ -198,32 +198,32 @@ 
                 $this->assertSame($expected[$expectedKey], $actual[$expectedKey]);
             }
         }
-	}
+    }
 
-	public function testmark_relationships_deleted()
+    public function testmark_relationships_deleted()
     {
-    	$document = new Document();
+        $document = new Document();
     	
-    	//execute the method and test if it works and does not throws an exception.
-    	try {
-    		$document->mark_relationships_deleted(1);
-    		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
-    		$this->fail();
-    	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $document->mark_relationships_deleted(1);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
-	}
+    }
 
 
-	public function testbean_implements() {
+    public function testbean_implements() {
 
-		$document = new Document();
-		$this->assertEquals(false, $document->bean_implements('')); //test with blank value
-		$this->assertEquals(false, $document->bean_implements('test')); //test with invalid value
-		$this->assertEquals(true, $document->bean_implements('ACL')); //test with valid value
+        $document = new Document();
+        $this->assertEquals(false, $document->bean_implements('')); //test with blank value
+        $this->assertEquals(false, $document->bean_implements('test')); //test with invalid value
+        $this->assertEquals(true, $document->bean_implements('ACL')); //test with valid value
 
-	}
+    }
 
 
 }

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]);
             }
         }

Braces +4 added lines, -7 removed lines

@@ -93,8 +93,7 @@ 
 		try {
 			$document->fill_in_additional_list_fields();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 
@@ -129,8 +128,7 @@ 
 		try {
 			$document->list_view_parse_additional_sections(new Sugar_Smarty(), $xTemplateSection);
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -194,7 +192,7 @@ 
         foreach($expected as $expectedKey => $expectedVal){
             if($expectedKey == 'FILE_URL'){
                 $this->assertRegExp($expected[$expectedKey],$actual[$expectedKey]);
-            }else {
+            } else {
                 $this->assertSame($expected[$expectedKey], $actual[$expectedKey]);
             }
         }
@@ -208,8 +206,7 @@ 
     	try {
     		$document->mark_relationships_deleted(1);
     		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
+    	} catch (Exception $e) {
     		$this->fail();
     	}
 

tests/tests/modules/AOR_Conditions/AOR_ConditionTest.php

Indentation +32 added lines, -32 removed lines

@@ -3,48 +3,48 @@
 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);
+        //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->assertAttributeEquals('AOR_Conditions', 'module_dir', $aor_Condition);
-		$this->assertAttributeEquals('AOR_Condition', 'object_name', $aor_Condition);
-		$this->assertAttributeEquals('aor_conditions', 'table_name', $aor_Condition);
-		$this->assertAttributeEquals(true, 'new_schema', $aor_Condition);
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $aor_Condition);
-		$this->assertAttributeEquals(true, 'importable', $aor_Condition);
-		$this->assertAttributeEquals(false, 'tracker_visibility', $aor_Condition);
+        $this->assertAttributeEquals('AOR_Conditions', 'module_dir', $aor_Condition);
+        $this->assertAttributeEquals('AOR_Condition', 'object_name', $aor_Condition);
+        $this->assertAttributeEquals('aor_conditions', 'table_name', $aor_Condition);
+        $this->assertAttributeEquals(true, 'new_schema', $aor_Condition);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $aor_Condition);
+        $this->assertAttributeEquals(true, 'importable', $aor_Condition);
+        $this->assertAttributeEquals(false, 'tracker_visibility', $aor_Condition);
 		
 		
-	}
+    }
 
     public function testsave_lines(){
 
-    	error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
     	
-    	$aor_Condition = new AOR_Condition();
+        $aor_Condition = new AOR_Condition();
     	
-    	//preset the required data
-    	$post_data = Array();
-    	$post_data['field'][] = 'test field';
-    	$post_data['name'][] = 'test'; 
-    	$post_data['parameter'][] = '1'; 
-    	$post_data['module_path'][] = 'test path'; 
-    	$post_data['operator'][] = 'test'; 
-    	$post_data['value_type'][] = 'test type'; 
+        //preset the required data
+        $post_data = Array();
+        $post_data['field'][] = 'test field';
+        $post_data['name'][] = 'test'; 
+        $post_data['parameter'][] = '1'; 
+        $post_data['module_path'][] = 'test path'; 
+        $post_data['operator'][] = 'test'; 
+        $post_data['value_type'][] = 'test type'; 
     	 
-    	//execute the method and test if it works and does not throws an exception.
-    	try {
-    		$aor_Condition->save_lines($post_data, new AOR_Report());
-    		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
-    		$this->fail();
-    	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $aor_Condition->save_lines($post_data, new AOR_Report());
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
     	
     	
     }

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);
     	

Braces +1 added lines, -2 removed lines

@@ -41,8 +41,7 @@
     	try {
     		$aor_Condition->save_lines($post_data, new AOR_Report());
     		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
+    	} catch (Exception $e) {
     		$this->fail();
     	}
     	

tests/tests/modules/AOD_IndexEvent/AOD_IndexEventTest.php

Indentation +14 added lines, -14 removed lines

@@ -2,23 +2,23 @@
 
 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);
+        //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->assertAttributeEquals('AOD_IndexEvent', 'module_dir', $aod_indexEvent);
-		$this->assertAttributeEquals('AOD_IndexEvent', 'object_name', $aod_indexEvent);
-		$this->assertAttributeEquals('aod_indexevent', 'table_name', $aod_indexEvent);
-		$this->assertAttributeEquals(true, 'new_schema', $aod_indexEvent);
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $aod_indexEvent);
-		$this->assertAttributeEquals(false, 'importable', $aod_indexEvent);
-		$this->assertAttributeEquals(false, 'tracker_visibility', $aod_indexEvent);
+        $this->assertAttributeEquals('AOD_IndexEvent', 'module_dir', $aod_indexEvent);
+        $this->assertAttributeEquals('AOD_IndexEvent', 'object_name', $aod_indexEvent);
+        $this->assertAttributeEquals('aod_indexevent', 'table_name', $aod_indexEvent);
+        $this->assertAttributeEquals(true, 'new_schema', $aod_indexEvent);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $aod_indexEvent);
+        $this->assertAttributeEquals(false, 'importable', $aod_indexEvent);
+        $this->assertAttributeEquals(false, 'tracker_visibility', $aod_indexEvent);
 		
-	}
+    }
 	
 }
 ?>
\ No newline at end of file

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

Indentation +38 added lines, -38 removed lines

@@ -3,77 +3,77 @@
 class TrackerTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testTracker()
+    public function testTracker()
     {
-    	//execute the contructor and check for the Object type and  attributes
-    	$tracker = new Tracker();
+        //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);
-    	$this->assertAttributeEquals('Tracker', 'object_name', $tracker);
+        $this->assertAttributeEquals('tracker', 'table_name', $tracker);
+        $this->assertAttributeEquals('Trackers', 'module_dir', $tracker);
+        $this->assertAttributeEquals('Tracker', 'object_name', $tracker);
     	    	
-    	$this->assertAttributeEquals(true, 'disable_var_defs', $tracker);
+        $this->assertAttributeEquals(true, 'disable_var_defs', $tracker);
     	
-    	$this->assertAttributeEquals('Tracker', 'acltype', $tracker);
-    	$this->assertAttributeEquals('Trackers', 'acl_category', $tracker);
-    	$this->assertAttributeEquals(true, 'disable_custom_fields', $tracker);
+        $this->assertAttributeEquals('Tracker', 'acltype', $tracker);
+        $this->assertAttributeEquals('Trackers', 'acl_category', $tracker);
+        $this->assertAttributeEquals(true, 'disable_custom_fields', $tracker);
     }
 
     public function testget_recently_viewed()
     {
-    	$tracker = new Tracker();
+        $tracker = new Tracker();
     	
-    	$result = $tracker->get_recently_viewed(1);
+        $result = $tracker->get_recently_viewed(1);
     	
-    	$this->assertInstanceOf('BreadCrumbStack',$_SESSION['breadCrumbs']);
-    	$this->assertTrue(is_array($result));
+        $this->assertInstanceOf('BreadCrumbStack',$_SESSION['breadCrumbs']);
+        $this->assertTrue(is_array($result));
     	
     }
 
     public function testmakeInvisibleForAll()
     {
     	
-    	$tracker = new Tracker();
+        $tracker = new Tracker();
 
-    	//execute the method and test if it works and does not throws an exception.
-    	try {
-    		$tracker->makeInvisibleForAll(1);
-    		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
-    		$this->fail();
-    	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $tracker->makeInvisibleForAll(1);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
     	
     }
 
    
     public function testbean_implements()
-	{
-		$tracker = new Tracker();
+    {
+        $tracker = new Tracker();
 		
-		$this->assertEquals(false, $tracker->bean_implements('')); //test with blank value
-		$this->assertEquals(false, $tracker->bean_implements('test')); //test with invalid value
-		$this->assertEquals(false, $tracker->bean_implements('ACL')); //test with valid value
+        $this->assertEquals(false, $tracker->bean_implements('')); //test with blank value
+        $this->assertEquals(false, $tracker->bean_implements('test')); //test with invalid value
+        $this->assertEquals(false, $tracker->bean_implements('ACL')); //test with valid value
 
     }
 
     
     public function testlogPage()
     {
-    	error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
        	
-    	//test without setting headerDisplayed
-    	Tracker::logPage();
-    	$this->assertEquals(null, $_SESSION['lpage']);
+        //test without setting headerDisplayed
+        Tracker::logPage();
+        $this->assertEquals(null, $_SESSION['lpage']);
     	
     	
-    	//test with headerDisplayed set
-    	$GLOBALS['app']->headerDisplayed  = 1;
-    	Tracker::logPage();
-    	$this->assertEquals(time(), $_SESSION['lpage']);
+        //test with headerDisplayed set
+        $GLOBALS['app']->headerDisplayed  = 1;
+        Tracker::logPage();
+        $this->assertEquals(time(), $_SESSION['lpage']);
     	
     }
 }

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']);
     	

Braces +1 added lines, -2 removed lines

@@ -42,8 +42,7 @@
     	try {
     		$tracker->makeInvisibleForAll(1);
     		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
+    	} catch (Exception $e) {
     		$this->fail();
     	}
     	

tests/tests/modules/Meetings/MeetingTest.php

Indentation +291 added lines, -291 removed lines

@@ -3,405 +3,405 @@ 
 class MeetingTest extends PHPUnit_Framework_TestCase  {
 
 
-	public function testMeeting() {
+    public function testMeeting() {
 
-		error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 		
-		//execute the contructor and check for the Object type and  attributes
-		$meeting = new Meeting();
+        //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);
-		$this->assertAttributeEquals('meetings', 'table_name', $meeting);
+        $this->assertAttributeEquals('Meetings', 'module_dir', $meeting);
+        $this->assertAttributeEquals('Meeting', 'object_name', $meeting);
+        $this->assertAttributeEquals('meetings', 'table_name', $meeting);
 		
-		$this->assertAttributeEquals(true, 'new_schema', $meeting);
-		$this->assertAttributeEquals(true, 'importable', $meeting);
-		$this->assertAttributeEquals(false, 'syncing', $meeting);
-		$this->assertAttributeEquals(true, 'update_vcal', $meeting); 
+        $this->assertAttributeEquals(true, 'new_schema', $meeting);
+        $this->assertAttributeEquals(true, 'importable', $meeting);
+        $this->assertAttributeEquals(false, 'syncing', $meeting);
+        $this->assertAttributeEquals(true, 'update_vcal', $meeting); 
 		
-		$this->assertAttributeEquals('meetings_users', 'rel_users_table', $meeting);
-		$this->assertAttributeEquals('meetings_contacts', 'rel_contacts_table', $meeting);
-		$this->assertAttributeEquals('meetings_leads', 'rel_leads_table', $meeting);
+        $this->assertAttributeEquals('meetings_users', 'rel_users_table', $meeting);
+        $this->assertAttributeEquals('meetings_contacts', 'rel_contacts_table', $meeting);
+        $this->assertAttributeEquals('meetings_leads', 'rel_leads_table', $meeting);
 		
-		$this->assertAttributeEquals(null, 'cached_get_users', $meeting);
-		$this->assertAttributeEquals(false, 'date_changed', $meeting); 
+        $this->assertAttributeEquals(null, 'cached_get_users', $meeting);
+        $this->assertAttributeEquals(false, 'date_changed', $meeting); 
 		
-	}
+    }
 
 
-	public function testACLAccess(){
+    public function testACLAccess(){
 
-		$meeting = new Meeting();
+        $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'));
+        //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'));
 		
 
-		//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'));
+        //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'));
 			
-	}
+    }
 
 
-	public function testhasIntegratedMeeting() {
+    public function testhasIntegratedMeeting() {
 
-		$meeting = new Meeting();
-		$result = $meeting->hasIntegratedMeeting();
-		$this->assertEquals(false, $result);
+        $meeting = new Meeting();
+        $result = $meeting->hasIntegratedMeeting();
+        $this->assertEquals(false, $result);
 		
-	}
+    }
 
 
-	public function testSaveAndMarkdeletedAndSetAcceptStatus() {
+    public function testSaveAndMarkdeletedAndSetAcceptStatus() {
 
 		
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		$meeting->name = "test";
-		$meeting->status = "Not Held";
-		$meeting->type = "Sugar";
-		$meeting->description = "test description";
-		$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";
+        $meeting->name = "test";
+        $meeting->status = "Not Held";
+        $meeting->type = "Sugar";
+        $meeting->description = "test description";
+        $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";
 		
-		$meeting->save();
+        $meeting->save();
 		
-		//test for record ID to verify that record is saved
-		$this->assertTrue(isset($meeting->id));
-		$this->assertEquals(36, strlen($meeting->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($meeting->id));
+        $this->assertEquals(36, strlen($meeting->id));
 		
 		
-		/* Test set_accept_status method */
+        /* Test set_accept_status method */
 		
-		//test set_accept_status with User object
-		$user = new User();
-		$meeting->set_accept_status($user,"accept");
+        //test set_accept_status with User object
+        $user = new User();
+        $meeting->set_accept_status($user,"accept");
 		
 		
-		//test set_accept_status with contact object
-		$contact = new Contact();
-		$meeting->set_accept_status($contact,"accept");
+        //test set_accept_status with contact object
+        $contact = new Contact();
+        $meeting->set_accept_status($contact,"accept");
 		
 		
-		//test set_accept_status with Lead object
-		$lead = new Lead();
-		$meeting->set_accept_status($lead,"accept");
+        //test set_accept_status with Lead object
+        $lead = new Lead();
+        $meeting->set_accept_status($lead,"accept");
 		
 		
-		//mark all created relationships as deleted
-		$meeting->mark_relationships_deleted($meeting->id);
+        //mark all created relationships as deleted
+        $meeting->mark_relationships_deleted($meeting->id);
 			
 		
-		//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);
+        //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);
 		
-	}
+    }
 
 	
-	public function testget_summary_text() {
+    public function testget_summary_text() {
 
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		//test without setting name
-		$this->assertEquals(Null,$meeting->get_summary_text());
+        //test without setting name
+        $this->assertEquals(Null,$meeting->get_summary_text());
 		
-		//test with name set
-		$meeting->name = "test";
-		$this->assertEquals('test',$meeting->get_summary_text());
+        //test with name set
+        $meeting->name = "test";
+        $this->assertEquals('test',$meeting->get_summary_text());
 		
 		
-	}
+    }
 
     public function testcreate_export_query(){
 
-    	$meeting = new Meeting();
+        $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);
+        //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);
     	
     		
-    	//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);
+        //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);
     	
-	}
-
-	public function testfill_in_additional_detail_fields() {
-
-		$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->fill_in_additional_detail_fields();
-
-		//verify effected atributes
-		$this->assertEquals("Administrator", $meeting->assigned_user_name);		
-		$this->assertEquals("Administrator", $meeting->created_by_name);		
-		$this->assertEquals("Administrator", $meeting->modified_by_name);		
-		$this->assertTrue(isset($meeting->time_start_hour));
-		$this->assertTrue(isset($meeting->date_start)); 
-		$this->assertTrue(isset($meeting->time_start)); 
-		$this->assertTrue(isset($meeting->duration_hours)); 
-		$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(-1, $meeting->email_reminder_time );
-		$this->assertEquals(false, $meeting->email_reminder_checked );
-		$this->assertEquals("Accounts", $meeting->parent_type);
-		
-	}
-
-	public function testget_list_view_data() {
-
-		$meeting = new Meeting();
-		
-		//preset required attribute values
-		$meeting->status == "Planned";
-		$meeting->parent_type = "Accounts";
-		$meeting->contact_id = 1;
-		$meeting->contact_name = "test";
-		$meeting->parent_name = "Account";
-		
-		$expected = array (
-					  'DELETED' => 0,
-					  'PARENT_TYPE' => 'Accounts',
-					  'STATUS' => 'Planned',
-					  'TYPE' => 'Sugar',
-					  'REMINDER_TIME' => '-1',
-					  'EMAIL_REMINDER_TIME' => '-1',
-					  'EMAIL_REMINDER_SENT' => '0',
-					  'SEQUENCE' => '0',
-					  'CONTACT_NAME' => 'test',
-					  'PARENT_NAME' => '',
-					  'CONTACT_ID' => 1,
-					  'REPEAT_INTERVAL' => '1',
-					  'PARENT_MODULE' => 'Accounts',
-					  'SET_COMPLETE' => '<a id=\'\' onclick=\'SUGAR.util.closeActivityPanel.show("Meetings","","Held","listview","1");\'><img src="themes/SuiteR/images/close_inline.png?v=fqXdFZ_r6FC1K7P_Fy3mVw"     border=\'0\' alt="Close" /></a>',
-					  'DATE_START' => '<font class=\'overdueTask\'></font>',
-					  'REMINDER_CHECKED' => false,
-					  'EMAIL_REMINDER_CHECKED' => false,
-					);
-		
-		$actual = $meeting->get_list_view_data();
-		
-		//$this->assertSame($expected, $actual);
-		$this->assertEquals($expected['PARENT_TYPE'], $actual['PARENT_TYPE']);
-		$this->assertEquals($expected['STATUS'], $actual['STATUS']);
-		$this->assertEquals($expected['TYPE'], $actual['TYPE']);
-		$this->assertEquals($expected['REMINDER_TIME'], $actual['REMINDER_TIME']);
-		$this->assertEquals($expected['EMAIL_REMINDER_TIME'], $actual['EMAIL_REMINDER_TIME']);
-		$this->assertEquals($expected['EMAIL_REMINDER_SENT'], $actual['EMAIL_REMINDER_SENT']);
-		$this->assertEquals($expected['CONTACT_NAME'], $actual['CONTACT_NAME']);
-		$this->assertEquals($expected['CONTACT_ID'], $actual['CONTACT_ID']);
-		$this->assertEquals($expected['REPEAT_INTERVAL'], $actual['REPEAT_INTERVAL']);
-		$this->assertEquals($expected['PARENT_MODULE'], $actual['PARENT_MODULE']);
-		
-		
-	}
-
-	public function testset_notification_body() {
-
-		global $current_user;
-		$current_user = new User(1);
-		
-		$meeting = new Meeting();
-		
-		//test with attributes preset and verify template variables are set accordingly		
-		$meeting->name = "test";
-		$meeting->status = "Not Held";
-		$meeting->type = "Sugar";
-		$meeting->description = "test description";
-		$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']);
+    }
+
+    public function testfill_in_additional_detail_fields() {
+
+        $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->fill_in_additional_detail_fields();
+
+        //verify effected atributes
+        $this->assertEquals("Administrator", $meeting->assigned_user_name);		
+        $this->assertEquals("Administrator", $meeting->created_by_name);		
+        $this->assertEquals("Administrator", $meeting->modified_by_name);		
+        $this->assertTrue(isset($meeting->time_start_hour));
+        $this->assertTrue(isset($meeting->date_start)); 
+        $this->assertTrue(isset($meeting->time_start)); 
+        $this->assertTrue(isset($meeting->duration_hours)); 
+        $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(-1, $meeting->email_reminder_time );
+        $this->assertEquals(false, $meeting->email_reminder_checked );
+        $this->assertEquals("Accounts", $meeting->parent_type);
+		
+    }
+
+    public function testget_list_view_data() {
+
+        $meeting = new Meeting();
+		
+        //preset required attribute values
+        $meeting->status == "Planned";
+        $meeting->parent_type = "Accounts";
+        $meeting->contact_id = 1;
+        $meeting->contact_name = "test";
+        $meeting->parent_name = "Account";
+		
+        $expected = array (
+                        'DELETED' => 0,
+                        'PARENT_TYPE' => 'Accounts',
+                        'STATUS' => 'Planned',
+                        'TYPE' => 'Sugar',
+                        'REMINDER_TIME' => '-1',
+                        'EMAIL_REMINDER_TIME' => '-1',
+                        'EMAIL_REMINDER_SENT' => '0',
+                        'SEQUENCE' => '0',
+                        'CONTACT_NAME' => 'test',
+                        'PARENT_NAME' => '',
+                        'CONTACT_ID' => 1,
+                        'REPEAT_INTERVAL' => '1',
+                        'PARENT_MODULE' => 'Accounts',
+                        'SET_COMPLETE' => '<a id=\'\' onclick=\'SUGAR.util.closeActivityPanel.show("Meetings","","Held","listview","1");\'><img src="themes/SuiteR/images/close_inline.png?v=fqXdFZ_r6FC1K7P_Fy3mVw"     border=\'0\' alt="Close" /></a>',
+                        'DATE_START' => '<font class=\'overdueTask\'></font>',
+                        'REMINDER_CHECKED' => false,
+                        'EMAIL_REMINDER_CHECKED' => false,
+                    );
+		
+        $actual = $meeting->get_list_view_data();
+		
+        //$this->assertSame($expected, $actual);
+        $this->assertEquals($expected['PARENT_TYPE'], $actual['PARENT_TYPE']);
+        $this->assertEquals($expected['STATUS'], $actual['STATUS']);
+        $this->assertEquals($expected['TYPE'], $actual['TYPE']);
+        $this->assertEquals($expected['REMINDER_TIME'], $actual['REMINDER_TIME']);
+        $this->assertEquals($expected['EMAIL_REMINDER_TIME'], $actual['EMAIL_REMINDER_TIME']);
+        $this->assertEquals($expected['EMAIL_REMINDER_SENT'], $actual['EMAIL_REMINDER_SENT']);
+        $this->assertEquals($expected['CONTACT_NAME'], $actual['CONTACT_NAME']);
+        $this->assertEquals($expected['CONTACT_ID'], $actual['CONTACT_ID']);
+        $this->assertEquals($expected['REPEAT_INTERVAL'], $actual['REPEAT_INTERVAL']);
+        $this->assertEquals($expected['PARENT_MODULE'], $actual['PARENT_MODULE']);
+		
+		
+    }
+
+    public function testset_notification_body() {
+
+        global $current_user;
+        $current_user = new User(1);
+		
+        $meeting = new Meeting();
+		
+        //test with attributes preset and verify template variables are set accordingly		
+        $meeting->name = "test";
+        $meeting->status = "Not Held";
+        $meeting->type = "Sugar";
+        $meeting->description = "test description";
+        $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']);
 	
-	}
+    }
 	
 
-	public function testcreate_notification_email(){
+    public function testcreate_notification_email(){
 
-		$meeting = new Meeting();	
+        $meeting = new Meeting();	
 		
-		$meeting->date_start = "2016-02-11 17:30:00";
-		$meeting->date_end = "2016-02-11 17:30:00";
+        $meeting->date_start = "2016-02-11 17:30:00";
+        $meeting->date_end = "2016-02-11 17:30:00";
 		
-		//test without setting user
-		$result = $meeting->create_notification_email(new User());
-		$this->assertInstanceOf('SugarPHPMailer', $result);
+        //test without setting user
+        $result = $meeting->create_notification_email(new User());
+        $this->assertInstanceOf('SugarPHPMailer', $result);
 		
-		//test with valid user
-		$result = $meeting->create_notification_email(new User(1));
-		$this->assertInstanceOf('SugarPHPMailer', $result);
+        //test with valid user
+        $result = $meeting->create_notification_email(new User(1));
+        $this->assertInstanceOf('SugarPHPMailer', $result);
 		
-	}
+    }
 	
 
-	public function testsend_assignment_notifications(){
+    public function testsend_assignment_notifications(){
 
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		$meeting->date_start = "2016-02-11 17:30:00";
-		$meeting->date_end = "2016-02-11 17:30:00";
+        $meeting->date_start = "2016-02-11 17:30:00";
+        $meeting->date_end = "2016-02-11 17:30:00";
 		
-		$admin = new Administration();
-		$admin->retrieveSettings();
-		$sendNotifications = false;
+        $admin = new Administration();
+        $admin->retrieveSettings();
+        $sendNotifications = false;
 		
-		$notify_user = new User(1);
+        $notify_user = new User(1);
 		
 
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$meeting->send_assignment_notifications($notify_user, $admin);
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $meeting->send_assignment_notifications($notify_user, $admin);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 
-	public function testget_meeting_users() {
+    public function testget_meeting_users() {
 
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		$result = $meeting->get_meeting_users();
-		$this->assertTrue(is_array($result));
+        $result = $meeting->get_meeting_users();
+        $this->assertTrue(is_array($result));
 		
-	}
+    }
 
-	public function testget_invite_meetings() {
+    public function testget_invite_meetings() {
 
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		$result = $meeting->get_invite_meetings(new User());
-		$this->assertTrue(is_array($result));
+        $result = $meeting->get_invite_meetings(new User());
+        $this->assertTrue(is_array($result));
 		
-	}
+    }
 
 
-	public function testget_notification_recipients() {
+    public function testget_notification_recipients() {
 
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		//test without special_notification
-		$result = $meeting->get_notification_recipients();
-		$this->assertTrue(is_array($result));
+        //test without special_notification
+        $result = $meeting->get_notification_recipients();
+        $this->assertTrue(is_array($result));
 		
 		
-		//test with special_notification
-		$meeting->special_notification = 1;
-		$result = $meeting->get_notification_recipients();
-		$this->assertTrue(is_array($result));
+        //test with special_notification
+        $meeting->special_notification = 1;
+        $result = $meeting->get_notification_recipients();
+        $this->assertTrue(is_array($result));
 	 	
-	}
+    }
 
 
-	public function testbean_implements() {
+    public function testbean_implements() {
 
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		$this->assertEquals(false, $meeting->bean_implements('')); //test with blank value
-		$this->assertEquals(false, $meeting->bean_implements('test')); //test with invalid value
-		$this->assertEquals(true, $meeting->bean_implements('ACL')); //test with valid value
+        $this->assertEquals(false, $meeting->bean_implements('')); //test with blank value
+        $this->assertEquals(false, $meeting->bean_implements('test')); //test with invalid value
+        $this->assertEquals(true, $meeting->bean_implements('ACL')); //test with valid value
 	
-	}
+    }
 
-	public function testlistviewACLHelper() {
+    public function testlistviewACLHelper() {
 
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		$expected = array("MAIN"=>"a", "PARENT"=>"a", "CONTACT"=>"a" );
-		$actual = $meeting->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+        $expected = array("MAIN"=>"a", "PARENT"=>"a", "CONTACT"=>"a" );
+        $actual = $meeting->listviewACLHelper();
+        $this->assertSame($expected,$actual);
 		
-	}
+    }
 
 
-	public function testsave_relationship_changes() {
+    public function testsave_relationship_changes() {
 
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$meeting->save_relationship_changes();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $meeting->save_relationship_changes();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
 		
-	}
+    }
 
 
-	public function testafterImportSave(){
+    public function testafterImportSave(){
 		
-		require_once("data/Link.php");
+        require_once("data/Link.php");
 		
-		$meeting = new Meeting();
+        $meeting = new Meeting();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
+        //execute the method and test if it works and does not throws an exception.
+        try {
 			
-			//test without parent_type 
-			$meeting->afterImportSave();
+            //test without parent_type 
+            $meeting->afterImportSave();
 			
-			//test with parent_type Contacts
-			$meeting->parent_type = 'Contacts';
-			$meeting->afterImportSave();		
+            //test with parent_type Contacts
+            $meeting->parent_type = 'Contacts';
+            $meeting->afterImportSave();		
 
-			//test with parent_type Leads
-			$meeting->parent_type = 'Leads';			
-			$meeting->afterImportSave();
+            //test with parent_type Leads
+            $meeting->parent_type = 'Leads';			
+            $meeting->afterImportSave();
 			
-			$this->assertTrue(true);
+            $this->assertTrue(true);
 			
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}		
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }		
 				
 				
-	}
+    }
 
     public function testgetDefaultStatus(){
 
-    	$meeting = new Meeting();
-    	$result = $meeting->getDefaultStatus();
-    	$this->assertEquals("Planned", $result);
+        $meeting = new Meeting();
+        $result = $meeting->getDefaultStatus();
+        $this->assertEquals("Planned", $result);
     	
     }
 
@@ -409,19 +409,19 @@ 
 
     public function testgetMeetingsExternalApiDropDown(){
     	
-    	$actual = getMeetingsExternalApiDropDown();
-    	$expected= array( "Sugar"=>"SuiteCRM");
-    	$this->assertSame($expected, $actual);
+        $actual = getMeetingsExternalApiDropDown();
+        $expected= array( "Sugar"=>"SuiteCRM");
+        $this->assertSame($expected, $actual);
     	
     }
     
     
     public function testgetMeetingTypeOptions(){
     	
-    	global $dictionary, $app_list_strings;
+        global $dictionary, $app_list_strings;
     	
-    	$result = getMeetingTypeOptions($dictionary, $app_list_strings);
-    	$this->assertTrue(is_array($result));
+        $result = getMeetingTypeOptions($dictionary, $app_list_strings);
+        $this->assertTrue(is_array($result));
     	
     }
     

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;
     	

Braces +3 added lines, -6 removed lines

@@ -286,8 +286,7 @@ 
 		try {
 			$meeting->send_assignment_notifications($notify_user, $admin);
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -358,8 +357,7 @@ 
 		try {
 			$meeting->save_relationship_changes();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -389,8 +387,7 @@ 
 			
 			$this->assertTrue(true);
 			
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}