ViewMultieditTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testViewMultiedit()	0	8	1
B	testdisplay()	0	40	1

<?php


 class ViewMultieditTest  extends PHPUnit_Framework_TestCase{

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

Expected 1 space after class name; 2 found

Expected 1 space before extends keyword; 2 found

 	function testViewMultiedit(){
 		
 		//execute the contructor and check for the Object type and type attribute
 		$view = new ViewMultiedit();
 		$this->assertInstanceOf('ViewMultiedit',$view);
 		$this->assertInstanceOf('SugarView',$view);
 		$this->assertAttributeEquals('edit','type', $view);
 	}
 	

There is some trailing whitespace on this line which should be avoided as per coding-style.

 	function testdisplay(){


 		//test without action value and modules list in REQUEST object
 		$view = new ViewMultiedit();
 		ob_start();
 		$view->display();
 		$renderedContent = ob_get_contents();
 		ob_end_clean();
 		$this->assertEquals(0,strlen($renderedContent));
 		

 		
 		//test with valid action value to get link in return
 		$view = new ViewMultiedit();
 		$view->action = 'AjaxFormSave';
 		$view->module = 'Users';
 		$view->bean = new User();
 		$view->bean->id =1;
 		ob_start(); 		
 		$view->display(); 		
 		$renderedContent = ob_get_contents();
 		ob_end_clean(); 		
 		$this->assertGreaterThan(0,strlen($renderedContent));
 			
 		
 		//Fails with a fatal error, method creates editview without properly setting it up causing fatal errors.
 		/*

54% of this comment could be valid code. Did you maybe forget this after debugging?

 		//test only with modules list in REQUEST object
 		$view = new ViewMultiedit();
 		$GLOBALS['current_language']= 'en_us';
 		$_REQUEST['modules']= Array('Calls','Accounts');
 		ob_start();
 		$view->display();
 		$renderedContent = ob_get_contents();
 		ob_end_clean();
 		$this->assertGreaterThan(0,strlen($renderedContent));
 		*/	
 		
 	}
 }
?>