adamjakab / SuiteCRM

tests/tests/modules/Employees/EmployeeTest.php

Indentation +100 added lines, -100 removed lines

@@ -3,143 +3,143 @@ 
 class EmployeeTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testEmployee() {
+    public function testEmployee() {
 
-		//execute the contructor and check for the Object type and  attributes
-		$employee = new Employee();
-		$this->assertInstanceOf('Employee',$employee);
-		$this->assertInstanceOf('Person',$employee);
-		$this->assertInstanceOf('SugarBean',$employee);
+        //execute the contructor and check for the Object type and  attributes
+        $employee = new Employee();
+        $this->assertInstanceOf('Employee',$employee);
+        $this->assertInstanceOf('Person',$employee);
+        $this->assertInstanceOf('SugarBean',$employee);
 
-		$this->assertAttributeEquals('Employees', 'module_dir', $employee);
-		$this->assertAttributeEquals('Employee', 'object_name', $employee);
-		$this->assertAttributeEquals('users', 'table_name', $employee);
-		$this->assertAttributeEquals(true, 'new_schema', $employee);
+        $this->assertAttributeEquals('Employees', 'module_dir', $employee);
+        $this->assertAttributeEquals('Employee', 'object_name', $employee);
+        $this->assertAttributeEquals('users', 'table_name', $employee);
+        $this->assertAttributeEquals(true, 'new_schema', $employee);
 
-	}
+    }
 
 
-	public function testget_summary_text() {
+    public function testget_summary_text() {
 
-		error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 
-		$employee = new Employee();
+        $employee = new Employee();
 
-		//test without setting name
-		$this->assertEquals(' ',$employee->get_summary_text());
+        //test without setting name
+        $this->assertEquals(' ',$employee->get_summary_text());
 
-		//test with name set
-		$employee->retrieve(1);
-		$this->assertEquals('Administrator',$employee->get_summary_text());
+        //test with name set
+        $employee->retrieve(1);
+        $this->assertEquals('Administrator',$employee->get_summary_text());
 
     }
 
 
-	public function testfill_in_additional_list_fields() {
-
-		$employee = new Employee();
+    public function testfill_in_additional_list_fields() {
 
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$employee->fill_in_additional_list_fields();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        $employee = new Employee();
 
-	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $employee->fill_in_additional_list_fields();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
-	public function testfill_in_additional_detail_fields()
-	{
-		$employee = new Employee();
+    }
 
+    public function testfill_in_additional_detail_fields()
+    {
+        $employee = new Employee();
 
-		//test with a empty employee bean
-		$employee->fill_in_additional_detail_fields();
-		$this->assertEquals("", $employee->reports_to_name);
 
+        //test with a empty employee bean
+        $employee->fill_in_additional_detail_fields();
+        $this->assertEquals("", $employee->reports_to_name);
 
-		//test with a valid employee bean
-		$employee->retrieve(1);
-		$employee->fill_in_additional_detail_fields();
-		$this->assertEquals("", $employee->reports_to_name);
 
-	}
+        //test with a valid employee bean
+        $employee->retrieve(1);
+        $employee->fill_in_additional_detail_fields();
+        $this->assertEquals("", $employee->reports_to_name);
 
-	public function testretrieve_employee_id()
-	{
-		$employee = new Employee();
-		//$this->assertEquals('1' ,$employee->retrieve_employee_id('admin'));
+    }
 
-		$this->markTestSkipped('Bug in query: employee_name parameter is wrongly used as user_name');
+    public function testretrieve_employee_id()
+    {
+        $employee = new Employee();
+        //$this->assertEquals('1' ,$employee->retrieve_employee_id('admin'));
 
-	}
+        $this->markTestSkipped('Bug in query: employee_name parameter is wrongly used as user_name');
 
+    }
 
-	public function testverify_data()
-	{
-		$employee = new Employee();
-		$this->assertEquals(true ,$employee->verify_data() );
 
-	}
+    public function testverify_data()
+    {
+        $employee = new Employee();
+        $this->assertEquals(true ,$employee->verify_data() );
 
-	public function testget_list_view_data(){
+    }
 
-		$employee = new Employee();
+    public function testget_list_view_data(){
 
-		$expected = array (
-					  'SUGAR_LOGIN' => '1',
-					  'FULL_NAME' => ' ',
-					  'NAME' => ' ',
-					  'IS_ADMIN' => '0',
-					  'EXTERNAL_AUTH_ONLY' => '0',
-					  'RECEIVE_NOTIFICATIONS' => '1',
-					  'DELETED' => 0,
-					  'PORTAL_ONLY' => '0',
-					  'SHOW_ON_EMPLOYEES' => '1',
-					  'ENCODED_NAME' => ' ',
-					  'EMAIL1' => '',
-					  'EMAIL1_LINK' => '<a href=\'javascript:void(0);\' onclick=\'SUGAR.quickCompose.init({"fullComposeUrl":"contact_id=\\u0026parent_type=Employees\\u0026parent_id=\\u0026parent_name=+\\u0026to_addrs_ids=\\u0026to_addrs_names=\\u0026to_addrs_emails=\\u0026to_email_addrs=+%26nbsp%3B%26lt%3B%26gt%3B\\u0026return_module=Employees\\u0026return_action=ListView\\u0026return_id=","composePackage":{"contact_id":"","parent_type":"Employees","parent_id":"","parent_name":" ","to_addrs_ids":"","to_addrs_names":"","to_addrs_emails":"","to_email_addrs":"  \\u003C\\u003E","return_module":"Employees","return_action":"ListView","return_id":""}});\' class=\'\'>',
-					  'MESSENGER_TYPE' => '',
-					  'REPORTS_TO_NAME' => NULL,
-					);
+        $employee = new Employee();
 
-		$actual = $employee->get_list_view_data();
-		$this->assertSame($expected, $actual);
+        $expected = array (
+                        'SUGAR_LOGIN' => '1',
+                        'FULL_NAME' => ' ',
+                        'NAME' => ' ',
+                        'IS_ADMIN' => '0',
+                        'EXTERNAL_AUTH_ONLY' => '0',
+                        'RECEIVE_NOTIFICATIONS' => '1',
+                        'DELETED' => 0,
+                        'PORTAL_ONLY' => '0',
+                        'SHOW_ON_EMPLOYEES' => '1',
+                        'ENCODED_NAME' => ' ',
+                        'EMAIL1' => '',
+                        'EMAIL1_LINK' => '<a href=\'javascript:void(0);\' onclick=\'SUGAR.quickCompose.init({"fullComposeUrl":"contact_id=\\u0026parent_type=Employees\\u0026parent_id=\\u0026parent_name=+\\u0026to_addrs_ids=\\u0026to_addrs_names=\\u0026to_addrs_emails=\\u0026to_email_addrs=+%26nbsp%3B%26lt%3B%26gt%3B\\u0026return_module=Employees\\u0026return_action=ListView\\u0026return_id=","composePackage":{"contact_id":"","parent_type":"Employees","parent_id":"","parent_name":" ","to_addrs_ids":"","to_addrs_names":"","to_addrs_emails":"","to_email_addrs":"  \\u003C\\u003E","return_module":"Employees","return_action":"ListView","return_id":""}});\' class=\'\'>',
+                        'MESSENGER_TYPE' => '',
+                        'REPORTS_TO_NAME' => NULL,
+                    );
+
+        $actual = $employee->get_list_view_data();
+        $this->assertSame($expected, $actual);
 
-	}
+    }
 
-	public function testlist_view_parse_additional_sections(){
+    public function testlist_view_parse_additional_sections(){
 
-		$employee = new Employee();
+        $employee = new Employee();
 
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$employee->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 {
+            $employee->list_view_parse_additional_sections(new Sugar_Smarty(), $xTemplateSection);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
-	}
+    }
 
 
-	public function testcreate_export_query() {
+    public function testcreate_export_query() {
 
-		$employee = new Employee();
+        $employee = new Employee();
 
-		//test with empty string params
-		$expected = "SELECT id, user_name, first_name, last_name, description, date_entered, date_modified, modified_user_id, created_by, title, department, is_admin, phone_home, phone_mobile, phone_work, phone_other, phone_fax, address_street, address_city, address_state, address_postalcode, address_country, reports_to_id, portal_only, status, receive_notifications, employee_status, messenger_id, messenger_type, is_group FROM users  WHERE  users.deleted = 0 ORDER BY users.user_name";
-		$actual = $employee->create_export_query('','');
-		$this->assertSame($expected,$actual);
+        //test with empty string params
+        $expected = "SELECT id, user_name, first_name, last_name, description, date_entered, date_modified, modified_user_id, created_by, title, department, is_admin, phone_home, phone_mobile, phone_work, phone_other, phone_fax, address_street, address_city, address_state, address_postalcode, address_country, reports_to_id, portal_only, status, receive_notifications, employee_status, messenger_id, messenger_type, is_group FROM users  WHERE  users.deleted = 0 ORDER BY users.user_name";
+        $actual = $employee->create_export_query('','');
+        $this->assertSame($expected,$actual);
 
 
-		//test with valid string params
-		$expected = "SELECT id, user_name, first_name, last_name, description, date_entered, date_modified, modified_user_id, created_by, title, department, is_admin, phone_home, phone_mobile, phone_work, phone_other, phone_fax, address_street, address_city, address_state, address_postalcode, address_country, reports_to_id, portal_only, status, receive_notifications, employee_status, messenger_id, messenger_type, is_group FROM users  WHERE users.user_name=\"\" AND  users.deleted = 0 ORDER BY users.id";
-		$actual = $employee->create_export_query('users.id','users.user_name=""');
-		$this->assertSame($expected,$actual);
+        //test with valid string params
+        $expected = "SELECT id, user_name, first_name, last_name, description, date_entered, date_modified, modified_user_id, created_by, title, department, is_admin, phone_home, phone_mobile, phone_work, phone_other, phone_fax, address_street, address_city, address_state, address_postalcode, address_country, reports_to_id, portal_only, status, receive_notifications, employee_status, messenger_id, messenger_type, is_group FROM users  WHERE users.user_name=\"\" AND  users.deleted = 0 ORDER BY users.id";
+        $actual = $employee->create_export_query('users.id','users.user_name=""');
+        $this->assertSame($expected,$actual);
         
     }
     
@@ -158,12 +158,12 @@ 
             $this->fail();
         }
 
-	}
+    }
 
     /**
      * @todo: NEEDS FIXING!
      */
-	public function testcreate_new_list_query()
+    public function testcreate_new_list_query()
     {
         /*
     	$employee = new Employee();
@@ -185,9 +185,9 @@ 
 
     public function testhasCustomFields()
     {
-    	$employee = new Employee();
-    	$result = $employee->hasCustomFields();
-    	$this->assertEquals(false,$result);
+        $employee = new Employee();
+        $result = $employee->hasCustomFields();
+        $this->assertEquals(false,$result);
     }
 
 }

Spacing +15 added lines, -15 removed lines

@@ -7,9 +7,9 @@ 
 
 		//execute the contructor and check for the Object type and  attributes
 		$employee = new Employee();
-		$this->assertInstanceOf('Employee',$employee);
-		$this->assertInstanceOf('Person',$employee);
-		$this->assertInstanceOf('SugarBean',$employee);
+		$this->assertInstanceOf('Employee', $employee);
+		$this->assertInstanceOf('Person', $employee);
+		$this->assertInstanceOf('SugarBean', $employee);
 
 		$this->assertAttributeEquals('Employees', 'module_dir', $employee);
 		$this->assertAttributeEquals('Employee', 'object_name', $employee);
@@ -26,11 +26,11 @@ 
 		$employee = new Employee();
 
 		//test without setting name
-		$this->assertEquals(' ',$employee->get_summary_text());
+		$this->assertEquals(' ', $employee->get_summary_text());
 
 		//test with name set
 		$employee->retrieve(1);
-		$this->assertEquals('Administrator',$employee->get_summary_text());
+		$this->assertEquals('Administrator', $employee->get_summary_text());
 
     }
 
@@ -80,15 +80,15 @@ 
 	public function testverify_data()
 	{
 		$employee = new Employee();
-		$this->assertEquals(true ,$employee->verify_data() );
+		$this->assertEquals(true, $employee->verify_data());
 
 	}
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 
 		$employee = new Employee();
 
-		$expected = array (
+		$expected = array(
 					  'SUGAR_LOGIN' => '1',
 					  'FULL_NAME' => ' ',
 					  'NAME' => ' ',
@@ -110,7 +110,7 @@ 
 
 	}
 
-	public function testlist_view_parse_additional_sections(){
+	public function testlist_view_parse_additional_sections() {
 
 		$employee = new Employee();
 
@@ -132,14 +132,14 @@ 
 
 		//test with empty string params
 		$expected = "SELECT id, user_name, first_name, last_name, description, date_entered, date_modified, modified_user_id, created_by, title, department, is_admin, phone_home, phone_mobile, phone_work, phone_other, phone_fax, address_street, address_city, address_state, address_postalcode, address_country, reports_to_id, portal_only, status, receive_notifications, employee_status, messenger_id, messenger_type, is_group FROM users  WHERE  users.deleted = 0 ORDER BY users.user_name";
-		$actual = $employee->create_export_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $employee->create_export_query('', '');
+		$this->assertSame($expected, $actual);
 
 
 		//test with valid string params
 		$expected = "SELECT id, user_name, first_name, last_name, description, date_entered, date_modified, modified_user_id, created_by, title, department, is_admin, phone_home, phone_mobile, phone_work, phone_other, phone_fax, address_street, address_city, address_state, address_postalcode, address_country, reports_to_id, portal_only, status, receive_notifications, employee_status, messenger_id, messenger_type, is_group FROM users  WHERE users.user_name=\"\" AND  users.deleted = 0 ORDER BY users.id";
-		$actual = $employee->create_export_query('users.id','users.user_name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $employee->create_export_query('users.id', 'users.user_name=""');
+		$this->assertSame($expected, $actual);
         
     }
     
@@ -153,7 +153,7 @@ 
         {
             $employee->preprocess_fields_on_save();
             $this->assertTrue(TRUE);
-        } catch(Exception $e)
+        }catch (Exception $e)
         {
             $this->fail();
         }
@@ -187,7 +187,7 @@ 
     {
     	$employee = new Employee();
     	$result = $employee->hasCustomFields();
-    	$this->assertEquals(false,$result);
+    	$this->assertEquals(false, $result);
     }
 
 }