adamjakab / SuiteCRM

tests/tests/include/MVC/View/views/view.editTest.php

Indentation +38 added lines, -38 removed lines

@@ -1,61 +1,61 @@
 <?php
 
- class ViewEditTest extends PHPUnit_Framework_TestCase
- {
+    class ViewEditTest extends PHPUnit_Framework_TestCase
+    {
 
 
- 	public function testViewEdit(){
+        public function testViewEdit(){
  
- 		//execute the contructor and check for the Object type and attributes
- 		$view = new ViewEdit();
- 		$this->assertInstanceOf('ViewEdit',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertAttributeEquals('edit','type', $view);
+            //execute the contructor and check for the Object type and attributes
+            $view = new ViewEdit();
+            $this->assertInstanceOf('ViewEdit',$view);
+            $this->assertInstanceOf('SugarView',$view);
+            $this->assertAttributeEquals('edit','type', $view);
  		
- 		$this->assertAttributeEquals(false,'useForSubpanel', $view);
- 		$this->assertAttributeEquals(false,'useModuleQuickCreateTemplate', $view);
- 		$this->assertAttributeEquals(true,'showTitle', $view);
+            $this->assertAttributeEquals(false,'useForSubpanel', $view);
+            $this->assertAttributeEquals(false,'useModuleQuickCreateTemplate', $view);
+            $this->assertAttributeEquals(true,'showTitle', $view);
  		
- 	}
+        }
 
 
     public function testpreDisplay()
     {
-    	error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
     	
-    	//execute the method with required attributes preset, it will initialize the ev(edit view) attribute.
-    	$view = new ViewEdit();
-    	$view->module = "Users";
-    	$view->bean = new User();
-    	$view->preDisplay();
-    	$this->assertInstanceOf('EditView',$view->ev);
+        //execute the method with required attributes preset, it will initialize the ev(edit view) attribute.
+        $view = new ViewEdit();
+        $view->module = "Users";
+        $view->bean = new User();
+        $view->preDisplay();
+        $this->assertInstanceOf('EditView',$view->ev);
     	
     	
-    	//execute the method again for a different module with required attributes preset, it will initialize the ev(edit view) attribute.
-    	$view = new ViewEdit();
-    	$view->module = "Meetings";
-    	$view->bean = new Meeting();
-    	$view->preDisplay();
-    	$this->assertInstanceOf('EditView',$view->ev);
+        //execute the method again for a different module with required attributes preset, it will initialize the ev(edit view) attribute.
+        $view = new ViewEdit();
+        $view->module = "Meetings";
+        $view->bean = new Meeting();
+        $view->preDisplay();
+        $this->assertInstanceOf('EditView',$view->ev);
     	
     }
 
- 	public function testdisplay(){
+        public function testdisplay(){
 
- 		//execute the method with essential parameters set. it should return some html.
- 		$view = new ViewEdit();
- 		$view->module = "Users";
- 		$view->bean = new User();
- 		$view->preDisplay();
- 		$view->ev->ss = new Sugar_Smarty();
+            //execute the method with essential parameters set. it should return some html.
+            $view = new ViewEdit();
+            $view->module = "Users";
+            $view->bean = new User();
+            $view->preDisplay();
+            $view->ev->ss = new Sugar_Smarty();
  		
- 		ob_start();
- 		$view->display();
- 		$renderedContent = ob_get_contents();
- 		ob_end_clean();
- 		$this->assertGreaterThan(0,strlen($renderedContent));
+            ob_start();
+            $view->display();
+            $renderedContent = ob_get_contents();
+            ob_end_clean();
+            $this->assertGreaterThan(0,strlen($renderedContent));
  		
- 	}
+        }
   
 }
 

Spacing +11 added lines, -11 removed lines

@@ -4,17 +4,17 @@ 
  {
 
 
- 	public function testViewEdit(){
+ 	public function testViewEdit() {
  
  		//execute the contructor and check for the Object type and attributes
  		$view = new ViewEdit();
- 		$this->assertInstanceOf('ViewEdit',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertAttributeEquals('edit','type', $view);
+ 		$this->assertInstanceOf('ViewEdit', $view);
+ 		$this->assertInstanceOf('SugarView', $view);
+ 		$this->assertAttributeEquals('edit', 'type', $view);
  		
- 		$this->assertAttributeEquals(false,'useForSubpanel', $view);
- 		$this->assertAttributeEquals(false,'useModuleQuickCreateTemplate', $view);
- 		$this->assertAttributeEquals(true,'showTitle', $view);
+ 		$this->assertAttributeEquals(false, 'useForSubpanel', $view);
+ 		$this->assertAttributeEquals(false, 'useModuleQuickCreateTemplate', $view);
+ 		$this->assertAttributeEquals(true, 'showTitle', $view);
  		
  	}
 
@@ -28,7 +28,7 @@ 
     	$view->module = "Users";
     	$view->bean = new User();
     	$view->preDisplay();
-    	$this->assertInstanceOf('EditView',$view->ev);
+    	$this->assertInstanceOf('EditView', $view->ev);
     	
     	
     	//execute the method again for a different module with required attributes preset, it will initialize the ev(edit view) attribute.
@@ -36,11 +36,11 @@ 
     	$view->module = "Meetings";
     	$view->bean = new Meeting();
     	$view->preDisplay();
-    	$this->assertInstanceOf('EditView',$view->ev);
+    	$this->assertInstanceOf('EditView', $view->ev);
     	
     }
 
- 	public function testdisplay(){
+ 	public function testdisplay() {
 
  		//execute the method with essential parameters set. it should return some html.
  		$view = new ViewEdit();
@@ -53,7 +53,7 @@ 
  		$view->display();
  		$renderedContent = ob_get_contents();
  		ob_end_clean();
- 		$this->assertGreaterThan(0,strlen($renderedContent));
+ 		$this->assertGreaterThan(0, strlen($renderedContent));
  		
  	}
   

tests/tests/include/MVC/View/views/view.serializedTest.php

Indentation +11 added lines, -11 removed lines

@@ -2,20 +2,20 @@ 
 
 class ViewSerializedTest extends PHPUnit_Framework_TestCase{
 	
-	public function testViewSerialized(){
+    public function testViewSerialized(){
 		
-		//execute the contructor and check for the Object type
-		$view = new ViewSerialized();
-		$this->assertInstanceOf('ViewSerialized',$view);
-		$this->assertInstanceOf('SugarView',$view);
+        //execute the contructor and check for the Object type
+        $view = new ViewSerialized();
+        $this->assertInstanceOf('ViewSerialized',$view);
+        $this->assertInstanceOf('SugarView',$view);
 		
- 	}
+        }
  	
- 	//Incomplete Test. method uses exit() so it cannot be tested.
-	public function testdisplay(){
+        //Incomplete Test. method uses exit() so it cannot be tested.
+    public function testdisplay(){
 		
 		
-		/* //this method call uses exit() so it cannot be tested as it forces the PHP unit to quite as well
+        /* //this method call uses exit() so it cannot be tested as it forces the PHP unit to quite as well
 		$view = new ViewSerialized();
 		$view->bean = new User();
 		
@@ -29,7 +29,7 @@ 
 		$this->assertGreaterThan(0,strlen($renderedContent));
 		*/
 		
-		$this->markTestIncomplete('Can Not be implemented');
- 	}
+        $this->markTestIncomplete('Can Not be implemented');
+        }
 }
 ?>

Spacing +5 added lines, -5 removed lines

@@ -1,18 +1,18 @@
 <?php
 
-class ViewSerializedTest extends PHPUnit_Framework_TestCase{
+class ViewSerializedTest extends PHPUnit_Framework_TestCase {
 	
-	public function testViewSerialized(){
+	public function testViewSerialized() {
 		
 		//execute the contructor and check for the Object type
 		$view = new ViewSerialized();
-		$this->assertInstanceOf('ViewSerialized',$view);
-		$this->assertInstanceOf('SugarView',$view);
+		$this->assertInstanceOf('ViewSerialized', $view);
+		$this->assertInstanceOf('SugarView', $view);
 		
  	}
  	
  	//Incomplete Test. method uses exit() so it cannot be tested.
-	public function testdisplay(){
+	public function testdisplay() {
 		
 		
 		/* //this method call uses exit() so it cannot be tested as it forces the PHP unit to quite as well

tests/tests/include/MVC/View/views/view.quickcreateTest.php

Indentation +27 added lines, -27 removed lines

@@ -6,24 +6,24 @@ 
     public function testpreDisplay() 
     {
  
-    	//check without setting any values, it should execute without any issues.
-    	$view = new ViewQuickcreate();
-    	$view->preDisplay();
-    	$this->assertEquals(0,count($_REQUEST));
+        //check without setting any values, it should execute without any issues.
+        $view = new ViewQuickcreate();
+        $view->preDisplay();
+        $this->assertEquals(0,count($_REQUEST));
     	 
-    	//check with values preset but without a valid bean id, it sould not change Request parameters
-    	$_REQUEST['source_module'] = "Users";
-    	$_REQUEST['module']= 'Users';
-    	$_REQUEST['record'] = '';
-    	$request = $_REQUEST;
+        //check with values preset but without a valid bean id, it sould not change Request parameters
+        $_REQUEST['source_module'] = "Users";
+        $_REQUEST['module']= 'Users';
+        $_REQUEST['record'] = '';
+        $request = $_REQUEST;
     	 
-    	$view->preDisplay();
-    	$this->assertSame($request,$_REQUEST);
+        $view->preDisplay();
+        $this->assertSame($request,$_REQUEST);
     	
-    	//check with values preset, it sould set some addiiotnal Request parameters
-    	$_REQUEST['record'] = 1;
-    	$view->preDisplay();
-    	$this->assertNotSame($request,$_REQUEST);
+        //check with values preset, it sould set some addiiotnal Request parameters
+        $_REQUEST['record'] = 1;
+        $view->preDisplay();
+        $this->assertNotSame($request,$_REQUEST);
     	
     	
     }    
@@ -31,26 +31,26 @@ 
     public function testdisplay()
     {
  
-    	error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 
-    	//execute the method with required child objects and parameters preset. it will return some html.
-    	$view = new ViewQuickcreate();
+        //execute the method with required child objects and parameters preset. it will return some html.
+        $view = new ViewQuickcreate();
     	 
-    	$_REQUEST['module']= 'Accounts';
-    	$view->bean = new Account();
+        $_REQUEST['module']= 'Accounts';
+        $view->bean = new Account();
     	 
-    	ob_start();
+        ob_start();
     	 
-    	$view->display();
+        $view->display();
     	 
-    	$renderedContent = ob_get_contents();
-    	ob_end_clean();
+        $renderedContent = ob_get_contents();
+        ob_end_clean();
     	 
-    	$this->assertGreaterThan(0,strlen($renderedContent));
-    	$this->assertEquals(False,json_decode($renderedContent)); //check that it doesn't return json. 
+        $this->assertGreaterThan(0,strlen($renderedContent));
+        $this->assertEquals(False,json_decode($renderedContent)); //check that it doesn't return json. 
     	
     	
-	}
+    }
 
  
 }
\ No newline at end of file

Spacing +7 added lines, -7 removed lines

@@ -9,21 +9,21 @@ 
     	//check without setting any values, it should execute without any issues.
     	$view = new ViewQuickcreate();
     	$view->preDisplay();
-    	$this->assertEquals(0,count($_REQUEST));
+    	$this->assertEquals(0, count($_REQUEST));
     	 
     	//check with values preset but without a valid bean id, it sould not change Request parameters
     	$_REQUEST['source_module'] = "Users";
-    	$_REQUEST['module']= 'Users';
+    	$_REQUEST['module'] = 'Users';
     	$_REQUEST['record'] = '';
     	$request = $_REQUEST;
     	 
     	$view->preDisplay();
-    	$this->assertSame($request,$_REQUEST);
+    	$this->assertSame($request, $_REQUEST);
     	
     	//check with values preset, it sould set some addiiotnal Request parameters
     	$_REQUEST['record'] = 1;
     	$view->preDisplay();
-    	$this->assertNotSame($request,$_REQUEST);
+    	$this->assertNotSame($request, $_REQUEST);
     	
     	
     }    
@@ -36,7 +36,7 @@ 
     	//execute the method with required child objects and parameters preset. it will return some html.
     	$view = new ViewQuickcreate();
     	 
-    	$_REQUEST['module']= 'Accounts';
+    	$_REQUEST['module'] = 'Accounts';
     	$view->bean = new Account();
     	 
     	ob_start();
@@ -46,8 +46,8 @@ 
     	$renderedContent = ob_get_contents();
     	ob_end_clean();
     	 
-    	$this->assertGreaterThan(0,strlen($renderedContent));
-    	$this->assertEquals(False,json_decode($renderedContent)); //check that it doesn't return json. 
+    	$this->assertGreaterThan(0, strlen($renderedContent));
+    	$this->assertEquals(False, json_decode($renderedContent)); //check that it doesn't return json. 
     	
     	
 	}

tests/tests/include/MVC/View/views/view.popupTest.php

Indentation +26 added lines, -26 removed lines

@@ -3,41 +3,41 @@
 class ViewPopupTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testViewPopup(){
+    public function testViewPopup(){
 
-		//execute the contructor and check for the Object type and type attribute
-		$view = new ViewPopup();
-		$this->assertInstanceOf('ViewPopup',$view);
-		$this->assertInstanceOf('SugarView',$view);
-		$this->assertAttributeEquals('list','type', $view);
-		
-		unset($view);
-	}
+        //execute the contructor and check for the Object type and type attribute
+        $view = new ViewPopup();
+        $this->assertInstanceOf('ViewPopup',$view);
+        $this->assertInstanceOf('SugarView',$view);
+        $this->assertAttributeEquals('list','type', $view);
+		
+        unset($view);
+    }
 
-	public function testdisplay(){
+    public function testdisplay(){
 		
-		//error_reporting(E_ERROR | E_PARSE |E_ALL);
+        //error_reporting(E_ERROR | E_PARSE |E_ALL);
 		
-		//execute the method with required child objects preset. it should return some html. 
-		$view = new ViewPopup();
-		$view->module = "Accounts";
+        //execute the method with required child objects preset. it should return some html. 
+        $view = new ViewPopup();
+        $view->module = "Accounts";
 		
-		try{
-			$view->bean = new Account();
-		}
-		Catch(Exception $e){
-			$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
-		}
+        try{
+            $view->bean = new Account();
+        }
+        Catch(Exception $e){
+            $this->assertStringStartsWith('mysqli_query()',$e->getMessage());
+        }
 	
-		ob_start();
+        ob_start();
 		
-		$view->display();
+        $view->display();
 		
-		$renderedContent = ob_get_contents();
-		ob_end_clean();
+        $renderedContent = ob_get_contents();
+        ob_end_clean();
 		
-		$this->assertGreaterThan(0,strlen($renderedContent));
+        $this->assertGreaterThan(0,strlen($renderedContent));
 		
-	}
+    }
 }
 

Spacing +9 added lines, -9 removed lines

@@ -3,18 +3,18 @@ 
 class ViewPopupTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testViewPopup(){
+	public function testViewPopup() {
 
 		//execute the contructor and check for the Object type and type attribute
 		$view = new ViewPopup();
-		$this->assertInstanceOf('ViewPopup',$view);
-		$this->assertInstanceOf('SugarView',$view);
-		$this->assertAttributeEquals('list','type', $view);
+		$this->assertInstanceOf('ViewPopup', $view);
+		$this->assertInstanceOf('SugarView', $view);
+		$this->assertAttributeEquals('list', 'type', $view);
 		
 		unset($view);
 	}
 
-	public function testdisplay(){
+	public function testdisplay() {
 		
 		//error_reporting(E_ERROR | E_PARSE |E_ALL);
 		
@@ -22,11 +22,11 @@ 
 		$view = new ViewPopup();
 		$view->module = "Accounts";
 		
-		try{
+		try {
 			$view->bean = new Account();
 		}
-		Catch(Exception $e){
-			$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
+		Catch (Exception $e) {
+			$this->assertStringStartsWith('mysqli_query()', $e->getMessage());
 		}
 	
 		ob_start();
@@ -36,7 +36,7 @@ 
 		$renderedContent = ob_get_contents();
 		ob_end_clean();
 		
-		$this->assertGreaterThan(0,strlen($renderedContent));
+		$this->assertGreaterThan(0, strlen($renderedContent));
 		
 	}
 }

Braces +1 added lines, -2 removed lines

@@ -24,8 +24,7 @@
 		
 		try{
 			$view->bean = new Account();
-		}
-		Catch(Exception $e){
+		} Catch(Exception $e){
 			$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
 		}
 	

tests/tests/include/MVC/View/views/view.jsonTest.php

Indentation +13 added lines, -13 removed lines

@@ -2,24 +2,24 @@ 
 
 class ViewJsonTest extends PHPUnit_Framework_TestCase
 {
-	function testViewJson(){
+    function testViewJson(){
 
-		//execute the contructor and check for the Object type and type attribute
-		$view = new ViewJson();
-		$this->assertInstanceOf('ViewJson',$view);
-		$this->assertInstanceOf('SugarView',$view);
-		$this->assertAttributeEquals('detail','type', $view);		
- 	}
+        //execute the contructor and check for the Object type and type attribute
+        $view = new ViewJson();
+        $this->assertInstanceOf('ViewJson',$view);
+        $this->assertInstanceOf('SugarView',$view);
+        $this->assertAttributeEquals('detail','type', $view);		
+        }
 
- 	//incomplete test. this method uses exit() so it cannot be tested.
-	function testdisplay(){
+        //incomplete test. this method uses exit() so it cannot be tested.
+    function testdisplay(){
 
-		/*
+        /*
 		setup required paramerers and execute the method. 
 		it uses die/exit which stops the execution of PHP unit as well so this method cannot be tested.
 		*/
 		
-		/*
+        /*
 		$view = new ViewJson();
 		$GLOBALS['module'] = "Users" ;
 		$view->bean = new User();
@@ -31,8 +31,8 @@ 
 		$this->assertGreaterThan(0,strlen($renderedContent));
 		$this->assertNotEquals(False,json_decode($renderedContent));
 		*/
-		$this->markTestIncomplete('Can Not be implemented');
-	}
+        $this->markTestIncomplete('Can Not be implemented');
+    }
 
 }
 ?>

Spacing +5 added lines, -5 removed lines

@@ -2,17 +2,17 @@
 
 class ViewJsonTest extends PHPUnit_Framework_TestCase
 {
-	function testViewJson(){
+	function testViewJson() {
 
 		//execute the contructor and check for the Object type and type attribute
 		$view = new ViewJson();
-		$this->assertInstanceOf('ViewJson',$view);
-		$this->assertInstanceOf('SugarView',$view);
-		$this->assertAttributeEquals('detail','type', $view);		
+		$this->assertInstanceOf('ViewJson', $view);
+		$this->assertInstanceOf('SugarView', $view);
+		$this->assertAttributeEquals('detail', 'type', $view);		
  	}
 
  	//incomplete test. this method uses exit() so it cannot be tested.
-	function testdisplay(){
+	function testdisplay() {
 
 		/*
 		setup required paramerers and execute the method. 

tests/tests/include/MVC/View/views/view.htmlTest.php

Indentation +15 added lines, -15 removed lines

@@ -3,28 +3,28 @@
 class ViewHtmlTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testViewHtml(){
+    public function testViewHtml(){
 		
-		//execute the contructor and check for the Object type
-		$view = new ViewHtml();
-		$this->assertInstanceOf('ViewHtml',$view);
-		$this->assertInstanceOf('SugarView',$view);
+        //execute the contructor and check for the Object type
+        $view = new ViewHtml();
+        $this->assertInstanceOf('ViewHtml',$view);
+        $this->assertInstanceOf('SugarView',$view);
 
 		
- 	}
+        }
  	
- 	public function testdisplay(){
+        public function testdisplay(){
  		
- 		$view = new ViewHtml();
+            $view = new ViewHtml();
  		 
- 		//execute the method and test if it works and does not throws an exception.
- 		try {
- 			$view->display();
+            //execute the method and test if it works and does not throws an exception.
+            try {
+                $view->display();
  			 
- 		} catch (Exception $e) {
- 			$this->fail();
- 		}
- 	}
+            } catch (Exception $e) {
+                $this->fail();
+            }
+        }
  	
 }
 ?>

Spacing +5 added lines, -5 removed lines

@@ -3,17 +3,17 @@ 
 class ViewHtmlTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testViewHtml(){
+	public function testViewHtml() {
 		
 		//execute the contructor and check for the Object type
 		$view = new ViewHtml();
-		$this->assertInstanceOf('ViewHtml',$view);
-		$this->assertInstanceOf('SugarView',$view);
+		$this->assertInstanceOf('ViewHtml', $view);
+		$this->assertInstanceOf('SugarView', $view);
 
 		
  	}
  	
- 	public function testdisplay(){
+ 	public function testdisplay() {
  		
  		$view = new ViewHtml();
  		 
@@ -21,7 +21,7 @@ 
  		try {
  			$view->display();
  			 
- 		} catch (Exception $e) {
+ 		}catch (Exception $e) {
  			$this->fail();
  		}
  	}

tests/tests/include/MVC/View/views/view.sugarpdfTest.php

Indentation +23 added lines, -23 removed lines

@@ -4,43 +4,43 @@
 class ViewSugarpdfTest extends PHPUnit_Framework_TestCase{
     
     
-     function testViewSugarpdf(){
-     	error_reporting(E_ERROR | E_WARNING | E_PARSE);
+        function testViewSugarpdf(){
+            error_reporting(E_ERROR | E_WARNING | E_PARSE);
      	      	
-     	//execute the method without request parameters and test if it works. it should output some headers and throw headers output exception.
-     	try {
-     		$view = new ViewSugarpdf();
+            //execute the method without request parameters and test if it works. it should output some headers and throw headers output exception.
+            try {
+                $view = new ViewSugarpdf();
      		 
-     	} catch (Exception $e) {
-     		$this->assertStringStartsWith('Cannot modify header information',$e->getMessage());
+            } catch (Exception $e) {
+                $this->assertStringStartsWith('Cannot modify header information',$e->getMessage());
      		
-     	}
-     	$this->assertEquals(Null,$view);
+            }
+            $this->assertEquals(Null,$view);
      	
      	
      	
-     	//execute the method with request parameters and test if it works.
-     	$_REQUEST["sugarpdf"] = 'someValue'; 
-     	$view = new ViewSugarpdf();
-     	$view->module = "Users";
-     	$this->assertInstanceOf('ViewSugarpdf',$view);
-     	$this->assertInstanceOf('SugarView',$view);
-     	$this->assertAttributeEquals('sugarpdf','type', $view);
-     	$this->assertAttributeEquals('someValue','sugarpdf', $view);
-     	$this->assertAttributeEquals(NULL,'sugarpdfBean', $view);
+            //execute the method with request parameters and test if it works.
+            $_REQUEST["sugarpdf"] = 'someValue'; 
+            $view = new ViewSugarpdf();
+            $view->module = "Users";
+            $this->assertInstanceOf('ViewSugarpdf',$view);
+            $this->assertInstanceOf('SugarView',$view);
+            $this->assertAttributeEquals('sugarpdf','type', $view);
+            $this->assertAttributeEquals('someValue','sugarpdf', $view);
+            $this->assertAttributeEquals(NULL,'sugarpdfBean', $view);
      	
-     }
+        }
 
    
     //Incomplete test. SugarpdfFactory::loadSugarpdf throws fatal error. error needs to be resolved before testing.     
     function testpreDisplay(){
-    	$this->markTestIncomplete('Can Not be implemented');
-     }
+        $this->markTestIncomplete('Can Not be implemented');
+        }
      
     //Incomplete test.  SugarpdfFactory::loadSugarpdf throws fatal error. error needs to be resolved before testing.
     function testdisplay(){
-    	$this->markTestIncomplete('Can Not be implemented');
-     }
+        $this->markTestIncomplete('Can Not be implemented');
+        }
 
 }
 ?>

Spacing +12 added lines, -12 removed lines

@@ -1,21 +1,21 @@ 
 <?php
 
 
-class ViewSugarpdfTest extends PHPUnit_Framework_TestCase{
+class ViewSugarpdfTest extends PHPUnit_Framework_TestCase {
     
     
-     function testViewSugarpdf(){
+     function testViewSugarpdf() {
      	error_reporting(E_ERROR | E_WARNING | E_PARSE);
      	      	
      	//execute the method without request parameters and test if it works. it should output some headers and throw headers output exception.
      	try {
      		$view = new ViewSugarpdf();
      		 
-     	} catch (Exception $e) {
-     		$this->assertStringStartsWith('Cannot modify header information',$e->getMessage());
+     	}catch (Exception $e) {
+     		$this->assertStringStartsWith('Cannot modify header information', $e->getMessage());
      		
      	}
-     	$this->assertEquals(Null,$view);
+     	$this->assertEquals(Null, $view);
      	
      	
      	
@@ -23,22 +23,22 @@ 
      	$_REQUEST["sugarpdf"] = 'someValue'; 
      	$view = new ViewSugarpdf();
      	$view->module = "Users";
-     	$this->assertInstanceOf('ViewSugarpdf',$view);
-     	$this->assertInstanceOf('SugarView',$view);
-     	$this->assertAttributeEquals('sugarpdf','type', $view);
-     	$this->assertAttributeEquals('someValue','sugarpdf', $view);
-     	$this->assertAttributeEquals(NULL,'sugarpdfBean', $view);
+     	$this->assertInstanceOf('ViewSugarpdf', $view);
+     	$this->assertInstanceOf('SugarView', $view);
+     	$this->assertAttributeEquals('sugarpdf', 'type', $view);
+     	$this->assertAttributeEquals('someValue', 'sugarpdf', $view);
+     	$this->assertAttributeEquals(NULL, 'sugarpdfBean', $view);
      	
      }
 
    
     //Incomplete test. SugarpdfFactory::loadSugarpdf throws fatal error. error needs to be resolved before testing.     
-    function testpreDisplay(){
+    function testpreDisplay() {
     	$this->markTestIncomplete('Can Not be implemented');
      }
      
     //Incomplete test.  SugarpdfFactory::loadSugarpdf throws fatal error. error needs to be resolved before testing.
-    function testdisplay(){
+    function testdisplay() {
     	$this->markTestIncomplete('Can Not be implemented');
      }
 

tests/tests/include/MVC/View/views/view.importvcardTest.php

Indentation +17 added lines, -17 removed lines

@@ -5,28 +5,28 @@
 
     public function test__construct()
     {
-    	//execute the contructor and check for the Object type and type attribute
-    	$view = new ViewImportvcard();
-    	$this->assertInstanceOf('ViewImportvcard',$view);
-    	$this->assertInstanceOf('SugarView',$view);
-    	$this->assertAttributeEquals('edit','type', $view);  	 
- 	}
+        //execute the contructor and check for the Object type and type attribute
+        $view = new ViewImportvcard();
+        $this->assertInstanceOf('ViewImportvcard',$view);
+        $this->assertInstanceOf('SugarView',$view);
+        $this->assertAttributeEquals('edit','type', $view);  	 
+        }
  	
 
-	public function testdisplay()
+    public function testdisplay()
     {
     	
-    	//execute the method with essential parameters set. it should return some html.
-    	$view = new ViewImportvcard();
-    	$_REQUEST['module'] = "Users";
-    	$view->ss = new Sugar_Smarty();
+        //execute the method with essential parameters set. it should return some html.
+        $view = new ViewImportvcard();
+        $_REQUEST['module'] = "Users";
+        $view->ss = new Sugar_Smarty();
     	
-    	ob_start();
-    	$view->display();
-    	$renderedContent = ob_get_contents();
-    	ob_end_clean();
-    	$this->assertGreaterThan(0,strlen($renderedContent));
+        ob_start();
+        $view->display();
+        $renderedContent = ob_get_contents();
+        ob_end_clean();
+        $this->assertGreaterThan(0,strlen($renderedContent));
     	
- 	}
+        }
 }
 ?>

Spacing +4 added lines, -4 removed lines

@@ -7,9 +7,9 @@ 
     {
     	//execute the contructor and check for the Object type and type attribute
     	$view = new ViewImportvcard();
-    	$this->assertInstanceOf('ViewImportvcard',$view);
-    	$this->assertInstanceOf('SugarView',$view);
-    	$this->assertAttributeEquals('edit','type', $view);  	 
+    	$this->assertInstanceOf('ViewImportvcard', $view);
+    	$this->assertInstanceOf('SugarView', $view);
+    	$this->assertAttributeEquals('edit', 'type', $view);  	 
  	}
  	
 
@@ -25,7 +25,7 @@ 
     	$view->display();
     	$renderedContent = ob_get_contents();
     	ob_end_clean();
-    	$this->assertGreaterThan(0,strlen($renderedContent));
+    	$this->assertGreaterThan(0, strlen($renderedContent));
     	
  	}
 }

tests/tests/include/MVC/SugarModuleTest.php

Indentation +38 added lines, -38 removed lines

@@ -3,62 +3,62 @@
 class SugarModuleTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testconstructor()
-	{
-		//test for invalid input
-		$sugarmodule = new SugarModule('');
-		$this->assertAttributeEquals(null,'_moduleName', $sugarmodule);
+    public function testconstructor()
+    {
+        //test for invalid input
+        $sugarmodule = new SugarModule('');
+        $this->assertAttributeEquals(null,'_moduleName', $sugarmodule);
 	
-		//test for valid input
-		$sugarmodule_user = SugarModule::get('User');
-		$this->assertAttributeEquals('User','_moduleName', $sugarmodule_user);
-	}
+        //test for valid input
+        $sugarmodule_user = SugarModule::get('User');
+        $this->assertAttributeEquals('User','_moduleName', $sugarmodule_user);
+    }
 	
     public function testget()
     {
-    	//test for invalid input
-    	$sugarmodule = SugarModule::get('');
-    	$this->assertInstanceOf('SugarModule',$sugarmodule);
-    	$this->assertAttributeEquals(null,'_moduleName', $sugarmodule);
+        //test for invalid input
+        $sugarmodule = SugarModule::get('');
+        $this->assertInstanceOf('SugarModule',$sugarmodule);
+        $this->assertAttributeEquals(null,'_moduleName', $sugarmodule);
     	
-    	//test for valid input
-    	$sugarmodule_user = SugarModule::get('User');
-    	$this->assertInstanceOf('SugarModule',$sugarmodule_user);
-    	$this->assertAttributeEquals('User','_moduleName', $sugarmodule_user);
+        //test for valid input
+        $sugarmodule_user = SugarModule::get('User');
+        $this->assertInstanceOf('SugarModule',$sugarmodule_user);
+        $this->assertAttributeEquals('User','_moduleName', $sugarmodule_user);
     	    	
     }
 
-     public function testmoduleImplements()
+        public function testmoduleImplements()
     {
-    	//test for invalid input
-    	$sugarmodule = new SugarModule('');
-    	$result = $sugarmodule->moduleImplements('Basic');
-    	$this->assertEquals( false, $result);
+        //test for invalid input
+        $sugarmodule = new SugarModule('');
+        $result = $sugarmodule->moduleImplements('Basic');
+        $this->assertEquals( false, $result);
     	
-    	//test for invalid input
-    	$sugarmodule_user = new SugarModule('Users');
-    	$result = $sugarmodule_user->moduleImplements('SugarModule');
-    	$this->assertFalse( $result);    	
+        //test for invalid input
+        $sugarmodule_user = new SugarModule('Users');
+        $result = $sugarmodule_user->moduleImplements('SugarModule');
+        $this->assertFalse( $result);    	
 
-    	//test for valid input
-    	$sugarmodule_user = new SugarModule('Users');
-    	$result = $sugarmodule_user->moduleImplements('Basic');
-    	$this->assertEquals( true, $result);    	
+        //test for valid input
+        $sugarmodule_user = new SugarModule('Users');
+        $result = $sugarmodule_user->moduleImplements('Basic');
+        $this->assertEquals( true, $result);    	
     	
     }
 
     
     public function testloadBean()
     {
-    	//test for invalid input
-    	$sugarmodule = new SugarModule('');
-    	$result = $sugarmodule->loadBean();
-    	$this->assertFalse( $result);
+        //test for invalid input
+        $sugarmodule = new SugarModule('');
+        $result = $sugarmodule->loadBean();
+        $this->assertFalse( $result);
     	
-    	//test for valid input
-    	$sugarmodule_user = new SugarModule('Users');
-    	$result = $sugarmodule_user->loadBean();
-    	$this->assertInstanceOf('User',$result);
+        //test for valid input
+        $sugarmodule_user = new SugarModule('Users');
+        $result = $sugarmodule_user->loadBean();
+        $this->assertInstanceOf('User',$result);
     	    	  	
     }
 }

Spacing +11 added lines, -11 removed lines

@@ -7,24 +7,24 @@ 
 	{
 		//test for invalid input
 		$sugarmodule = new SugarModule('');
-		$this->assertAttributeEquals(null,'_moduleName', $sugarmodule);
+		$this->assertAttributeEquals(null, '_moduleName', $sugarmodule);
 	
 		//test for valid input
 		$sugarmodule_user = SugarModule::get('User');
-		$this->assertAttributeEquals('User','_moduleName', $sugarmodule_user);
+		$this->assertAttributeEquals('User', '_moduleName', $sugarmodule_user);
 	}
 	
     public function testget()
     {
     	//test for invalid input
     	$sugarmodule = SugarModule::get('');
-    	$this->assertInstanceOf('SugarModule',$sugarmodule);
-    	$this->assertAttributeEquals(null,'_moduleName', $sugarmodule);
+    	$this->assertInstanceOf('SugarModule', $sugarmodule);
+    	$this->assertAttributeEquals(null, '_moduleName', $sugarmodule);
     	
     	//test for valid input
     	$sugarmodule_user = SugarModule::get('User');
-    	$this->assertInstanceOf('SugarModule',$sugarmodule_user);
-    	$this->assertAttributeEquals('User','_moduleName', $sugarmodule_user);
+    	$this->assertInstanceOf('SugarModule', $sugarmodule_user);
+    	$this->assertAttributeEquals('User', '_moduleName', $sugarmodule_user);
     	    	
     }
 
@@ -33,17 +33,17 @@ 
     	//test for invalid input
     	$sugarmodule = new SugarModule('');
     	$result = $sugarmodule->moduleImplements('Basic');
-    	$this->assertEquals( false, $result);
+    	$this->assertEquals(false, $result);
     	
     	//test for invalid input
     	$sugarmodule_user = new SugarModule('Users');
     	$result = $sugarmodule_user->moduleImplements('SugarModule');
-    	$this->assertFalse( $result);    	
+    	$this->assertFalse($result);    	
 
     	//test for valid input
     	$sugarmodule_user = new SugarModule('Users');
     	$result = $sugarmodule_user->moduleImplements('Basic');
-    	$this->assertEquals( true, $result);    	
+    	$this->assertEquals(true, $result);    	
     	
     }
 
@@ -53,12 +53,12 @@ 
     	//test for invalid input
     	$sugarmodule = new SugarModule('');
     	$result = $sugarmodule->loadBean();
-    	$this->assertFalse( $result);
+    	$this->assertFalse($result);
     	
     	//test for valid input
     	$sugarmodule_user = new SugarModule('Users');
     	$result = $sugarmodule_user->loadBean();
-    	$this->assertInstanceOf('User',$result);
+    	$this->assertInstanceOf('User', $result);
     	    	  	
     }
 }