adamjakab / SuiteCRM

tests/tests/modules/vCals/vCalTest.php

Indentation +190 added lines, -190 removed lines

@@ -3,269 +3,269 @@
 class vCalTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testvCal()
-	{
-		//execute the contructor and check for the Object type and  attributes
-		$vcal = new vCal();
+    public function testvCal()
+    {
+        //execute the contructor and check for the Object type and  attributes
+        $vcal = new vCal();
 		
-		$this->assertInstanceOf('vCal',$vcal);
-		$this->assertInstanceOf('SugarBean',$vcal);
+        $this->assertInstanceOf('vCal',$vcal);
+        $this->assertInstanceOf('SugarBean',$vcal);
 			
-		$this->assertAttributeEquals('vcals', 'table_name', $vcal);
-		$this->assertAttributeEquals('vCals', 'module_dir', $vcal);
-		$this->assertAttributeEquals('vCal', 'object_name', $vcal);
+        $this->assertAttributeEquals('vcals', 'table_name', $vcal);
+        $this->assertAttributeEquals('vCals', 'module_dir', $vcal);
+        $this->assertAttributeEquals('vCal', 'object_name', $vcal);
 		
-		$this->assertAttributeEquals(true, 'new_schema', $vcal);
-		$this->assertAttributeEquals(false, 'tracker_visibility', $vcal);
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $vcal);
+        $this->assertAttributeEquals(true, 'new_schema', $vcal);
+        $this->assertAttributeEquals(false, 'tracker_visibility', $vcal);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $vcal);
 		
-	}
+    }
 
-	public function testget_summary_text()
-	{
-		error_reporting(E_ERROR | E_PARSE);
+    public function testget_summary_text()
+    {
+        error_reporting(E_ERROR | E_PARSE);
 		
-		$vcal = new vCal();
+        $vcal = new vCal();
 		
-		//test without setting name
-		$this->assertEquals(Null,$vcal->get_summary_text());
+        //test without setting name
+        $this->assertEquals(Null,$vcal->get_summary_text());
 		
-		//test with name set
-		$vcal->name = "test";
-		$this->assertEquals('',$vcal->get_summary_text());
+        //test with name set
+        $vcal->name = "test";
+        $this->assertEquals('',$vcal->get_summary_text());
 
-	}
+    }
 
-	public function testfill_in_additional_list_fields()
-	{
+    public function testfill_in_additional_list_fields()
+    {
 
-		$vcal = new vCal();
+        $vcal = new vCal();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$vcal->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 {
+            $vcal->fill_in_additional_list_fields();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
-		$this->markTestIncomplete('method has no implementation');
-	}
+        $this->markTestIncomplete('method has no implementation');
+    }
 
-	public function testfill_in_additional_detail_fields()
-	{
+    public function testfill_in_additional_detail_fields()
+    {
 
-		$vcal = new vCal();
+        $vcal = new vCal();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$vcal->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 {
+            $vcal->fill_in_additional_detail_fields();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-		$this->markTestIncomplete('method has no implementation');
-	}
+        $this->markTestIncomplete('method has no implementation');
+    }
 
-	public function testget_list_view_data()
-	{
-		$vcal = new vCal();
+    public function testget_list_view_data()
+    {
+        $vcal = new vCal();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$vcal->get_list_view_data();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $vcal->get_list_view_data();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-		$this->markTestIncomplete('method has no implementation');
-	}
+        $this->markTestIncomplete('method has no implementation');
+    }
 
 
-	public function testget_freebusy_lines_cache()
-	{
+    public function testget_freebusy_lines_cache()
+    {
 		
-		$vcal = new vCal();
-		$user_bean = new User("1");
+        $vcal = new vCal();
+        $user_bean = new User("1");
 		
-		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\n";
-		$expectedEnd ="END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
+        $expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\n";
+        $expectedEnd ="END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		
-		$result = $vcal->get_freebusy_lines_cache($user_bean);
+        $result = $vcal->get_freebusy_lines_cache($user_bean);
 		
-		$this->assertStringStartsWith($expectedStart, $result);
-		$this->assertStringEndsWith($expectedEnd, $result);
+        $this->assertStringStartsWith($expectedStart, $result);
+        $this->assertStringEndsWith($expectedEnd, $result);
 
-	}
+    }
 
 
-	public function testcreate_sugar_freebusy()
-	{
-		global $locale, $timedate;
+    public function testcreate_sugar_freebusy()
+    {
+        global $locale, $timedate;
 		
-		$vcal = new vCal();
-		$user_bean = new User("1");
+        $vcal = new vCal();
+        $user_bean = new User("1");
 		
-		$now_date_time = $timedate->getNow(true);
-		$start_date_time = $now_date_time->get("yesterday");
-		$end_date_time = $now_date_time->get("tomorrow");
+        $now_date_time = $timedate->getNow(true);
+        $start_date_time = $now_date_time->get("yesterday");
+        $end_date_time = $now_date_time->get("tomorrow");
 		
-		$result = $vcal->create_sugar_freebusy($user_bean, $start_date_time, $end_date_time);
-		$this->assertGreaterThanOrEqual(0,strlen($result));
+        $result = $vcal->create_sugar_freebusy($user_bean, $start_date_time, $end_date_time);
+        $this->assertGreaterThanOrEqual(0,strlen($result));
 		
-	}
+    }
 
 
-	public function testget_vcal_freebusy()
-	{
-		$vcal = new vCal();
-		$user_focus = new User("1");
+    public function testget_vcal_freebusy()
+    {
+        $vcal = new vCal();
+        $user_focus = new User("1");
 		
-		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\n";
-		$expectedEnd ="END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
+        $expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\n";
+        $expectedEnd ="END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		
-		$result = $vcal->get_vcal_freebusy($user_focus);
+        $result = $vcal->get_vcal_freebusy($user_focus);
 		
-		$this->assertStringStartsWith($expectedStart, $result);
-		$this->assertStringEndsWith($expectedEnd, $result);
+        $this->assertStringStartsWith($expectedStart, $result);
+        $this->assertStringEndsWith($expectedEnd, $result);
 		
-	}
+    }
 
 
-	public function testcache_sugar_vcal()
-	{
-		$vcal = new vCal();				
-		$user_focus = new User("1");
+    public function testcache_sugar_vcal()
+    {
+        $vcal = new vCal();				
+        $user_focus = new User("1");
 
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$vcal->cache_sugar_vcal($user_focus);
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $vcal->cache_sugar_vcal($user_focus);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 
 
-	public function testcache_sugar_vcal_freebusy()
-	{
-		$vcal = new vCal();		
-		$user_focus = new User("1");
+    public function testcache_sugar_vcal_freebusy()
+    {
+        $vcal = new vCal();		
+        $user_focus = new User("1");
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$vcal->cache_sugar_vcal_freebusy($user_focus);
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $vcal->cache_sugar_vcal_freebusy($user_focus);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 				
-	}
+    }
 
 
-	public function testfold_ical_lines()
-	{
+    public function testfold_ical_lines()
+    {
 		
-		//test with short strings
-		$result = vCal::fold_ical_lines("testkey","testvalue");
-		$this->assertEquals("testkey:testvalue",$result);
+        //test with short strings
+        $result = vCal::fold_ical_lines("testkey","testvalue");
+        $this->assertEquals("testkey:testvalue",$result);
 	
 		
-		//test with longer strings
-		$expected =  "testkey11111111111111111111111111111111111111111111111111111111111111111111\r\n	11111111111111111111111111111111:testvalue11111111111111111111111111111111\r\n	11111111111111111111111111111111111111111111111111111111111111111111";
-		$result = vCal::fold_ical_lines("testkey" . str_repeat("1",100),"testvalue" . str_repeat("1",100));
-		$this->assertEquals();
+        //test with longer strings
+        $expected =  "testkey11111111111111111111111111111111111111111111111111111111111111111111\r\n	11111111111111111111111111111111:testvalue11111111111111111111111111111111\r\n	11111111111111111111111111111111111111111111111111111111111111111111";
+        $result = vCal::fold_ical_lines("testkey" . str_repeat("1",100),"testvalue" . str_repeat("1",100));
+        $this->assertEquals();
 	
-	}
+    }
 
 
-	public function testcreate_ical_array_from_string()
-	{
+    public function testcreate_ical_array_from_string()
+    {
 
-		$iCalString = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\nDTSTART:2016-01-09 00:00:00\r\nDTEND:2016-03-09 00:00:00\r\nDTSTAMP:2016-01-10 11:07:15\r\nEND:VFREEBUSY\r\nEND:VCALENDAR\r\n";
-		$expected = array(
-						array ( 'BEGIN','VCALENDAR'),
-						array ( 'VERSION','2.0'),
-						array ( 'PRODID','-//SugarCRM//SugarCRM Calendar//EN'),
-						array ( 'BEGIN', 'VFREEBUSY' ),
-						array ( 'ORGANIZER;CN= ', 'VFREEBUSY'  ),
-						array ( 'DTSTART', '2016-01-09 00:00:00' ),
-						array ( 'DTEND', '2016-03-09 00:00:00' ),
-						array ( 'DTSTAMP', '2016-01-10 11:07:15' ),
-						array ( 'END', 'VFREEBUSY' ),
-						array ( 'END', 'VCALENDAR' ),
-					);
-		$actual = vCal::create_ical_array_from_string($iCalString);
-		$this->assertSame($expected, $actual );
+        $iCalString = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\nDTSTART:2016-01-09 00:00:00\r\nDTEND:2016-03-09 00:00:00\r\nDTSTAMP:2016-01-10 11:07:15\r\nEND:VFREEBUSY\r\nEND:VCALENDAR\r\n";
+        $expected = array(
+                        array ( 'BEGIN','VCALENDAR'),
+                        array ( 'VERSION','2.0'),
+                        array ( 'PRODID','-//SugarCRM//SugarCRM Calendar//EN'),
+                        array ( 'BEGIN', 'VFREEBUSY' ),
+                        array ( 'ORGANIZER;CN= ', 'VFREEBUSY'  ),
+                        array ( 'DTSTART', '2016-01-09 00:00:00' ),
+                        array ( 'DTEND', '2016-03-09 00:00:00' ),
+                        array ( 'DTSTAMP', '2016-01-10 11:07:15' ),
+                        array ( 'END', 'VFREEBUSY' ),
+                        array ( 'END', 'VCALENDAR' ),
+                    );
+        $actual = vCal::create_ical_array_from_string($iCalString);
+        $this->assertSame($expected, $actual );
 		
-	}
+    }
 
 
-	public function testcreate_ical_string_from_array()
-	{
+    public function testcreate_ical_string_from_array()
+    {
 
-		$expected = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\nDTSTART:2016-01-09 00:00:00\r\nDTEND:2016-03-09 00:00:00\r\nDTSTAMP:2016-01-10 11:07:15\r\nEND:VFREEBUSY\r\nEND:VCALENDAR\r\n";
-		$iCalArray = array(
-				array ( 'BEGIN','VCALENDAR'),
-				array ( 'VERSION','2.0'),
-				array ( 'PRODID','-//SugarCRM//SugarCRM Calendar//EN'),
-				array ( 'BEGIN', 'VFREEBUSY' ),
-				array ( 'ORGANIZER;CN= ', 'VFREEBUSY'  ),
-				array ( 'DTSTART', '2016-01-09 00:00:00' ),
-				array ( 'DTEND', '2016-03-09 00:00:00' ),
-				array ( 'DTSTAMP', '2016-01-10 11:07:15' ),
-				array ( 'END', 'VFREEBUSY' ),
-				array ( 'END', 'VCALENDAR' ),
-		);
-		$actual = vCal::create_ical_string_from_array($iCalArray);
-		$this->assertSame($expected, $actual );
+        $expected = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\nDTSTART:2016-01-09 00:00:00\r\nDTEND:2016-03-09 00:00:00\r\nDTSTAMP:2016-01-10 11:07:15\r\nEND:VFREEBUSY\r\nEND:VCALENDAR\r\n";
+        $iCalArray = array(
+                array ( 'BEGIN','VCALENDAR'),
+                array ( 'VERSION','2.0'),
+                array ( 'PRODID','-//SugarCRM//SugarCRM Calendar//EN'),
+                array ( 'BEGIN', 'VFREEBUSY' ),
+                array ( 'ORGANIZER;CN= ', 'VFREEBUSY'  ),
+                array ( 'DTSTART', '2016-01-09 00:00:00' ),
+                array ( 'DTEND', '2016-03-09 00:00:00' ),
+                array ( 'DTSTAMP', '2016-01-10 11:07:15' ),
+                array ( 'END', 'VFREEBUSY' ),
+                array ( 'END', 'VCALENDAR' ),
+        );
+        $actual = vCal::create_ical_string_from_array($iCalArray);
+        $this->assertSame($expected, $actual );
 		
-	}
+    }
 
 
-	public function testescape_ical_chars()
-	{
-		$this->assertSame('',vCal::escape_ical_chars(''));
-		$this->assertSame('\;\,',vCal::escape_ical_chars(';,'));
-	}
+    public function testescape_ical_chars()
+    {
+        $this->assertSame('',vCal::escape_ical_chars(''));
+        $this->assertSame('\;\,',vCal::escape_ical_chars(';,'));
+    }
 
 
-	public function testunescape_ical_chars()
-	{
-		$this->assertSame('',vCal::unescape_ical_chars(''));
-		$this->assertSame("; , \\", vCal::unescape_ical_chars('\\; \\, \\\\'));	
-	}
+    public function testunescape_ical_chars()
+    {
+        $this->assertSame('',vCal::unescape_ical_chars(''));
+        $this->assertSame("; , \\", vCal::unescape_ical_chars('\\; \\, \\\\'));	
+    }
 
 
-	public function testget_ical_event(){
+    public function testget_ical_event(){
 
-		$user = new User(1);
-		$meeting = new Meeting();
+        $user = new User(1);
+        $meeting = new Meeting();
 		
-		$meeting->id = 1;
-		$meeting->date_start = "2016-02-11 17:30:00"; 
-		$meeting->date_end = "2016-02-11 17:30:00";
-		$meeting->name = "test";
-		$meeting->location = "test location";
-		$meeting->description = "test description";
+        $meeting->id = 1;
+        $meeting->date_start = "2016-02-11 17:30:00"; 
+        $meeting->date_end = "2016-02-11 17:30:00";
+        $meeting->name = "test";
+        $meeting->location = "test location";
+        $meeting->description = "test description";
 		
-		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VEVENT\r\nUID:1\r\nORGANIZED;CN=:\r\nDTSTART:20160211T173000Z\r\nDTEND:20160211T173000Z\r\n"; 
-		$expectedEnd = "\r\nSUMMARY:test\r\nLOCATION:test location\r\nDESCRIPTION:test description\r\nEND:VEVENT\r\nEND:VCALENDAR\r\n";
+        $expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VEVENT\r\nUID:1\r\nORGANIZED;CN=:\r\nDTSTART:20160211T173000Z\r\nDTEND:20160211T173000Z\r\n"; 
+        $expectedEnd = "\r\nSUMMARY:test\r\nLOCATION:test location\r\nDESCRIPTION:test description\r\nEND:VEVENT\r\nEND:VCALENDAR\r\n";
 
-		$result = vCal::get_ical_event($meeting ,$user);
+        $result = vCal::get_ical_event($meeting ,$user);
 		
-		$this->assertStringStartsWith($expectedStart, $result);
-		$this->assertStringEndsWith($expectedEnd, $result);
+        $this->assertStringStartsWith($expectedStart, $result);
+        $this->assertStringEndsWith($expectedEnd, $result);
 		
-	}
+    }
 
 }
 

Spacing +38 added lines, -38 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$vcal = new vCal();
 		
-		$this->assertInstanceOf('vCal',$vcal);
-		$this->assertInstanceOf('SugarBean',$vcal);
+		$this->assertInstanceOf('vCal', $vcal);
+		$this->assertInstanceOf('SugarBean', $vcal);
 			
 		$this->assertAttributeEquals('vcals', 'table_name', $vcal);
 		$this->assertAttributeEquals('vCals', 'module_dir', $vcal);
@@ -28,11 +28,11 @@ 
 		$vcal = new vCal();
 		
 		//test without setting name
-		$this->assertEquals(Null,$vcal->get_summary_text());
+		$this->assertEquals(Null, $vcal->get_summary_text());
 		
 		//test with name set
 		$vcal->name = "test";
-		$this->assertEquals('',$vcal->get_summary_text());
+		$this->assertEquals('', $vcal->get_summary_text());
 
 	}
 
@@ -94,7 +94,7 @@ 
 		$user_bean = new User("1");
 		
 		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\n";
-		$expectedEnd ="END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
+		$expectedEnd = "END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		
 		$result = $vcal->get_freebusy_lines_cache($user_bean);
 		
@@ -116,7 +116,7 @@ 
 		$end_date_time = $now_date_time->get("tomorrow");
 		
 		$result = $vcal->create_sugar_freebusy($user_bean, $start_date_time, $end_date_time);
-		$this->assertGreaterThanOrEqual(0,strlen($result));
+		$this->assertGreaterThanOrEqual(0, strlen($result));
 		
 	}
 
@@ -127,7 +127,7 @@ 
 		$user_focus = new User("1");
 		
 		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\n";
-		$expectedEnd ="END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
+		$expectedEnd = "END:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		
 		$result = $vcal->get_vcal_freebusy($user_focus);
 		
@@ -175,13 +175,13 @@ 
 	{
 		
 		//test with short strings
-		$result = vCal::fold_ical_lines("testkey","testvalue");
-		$this->assertEquals("testkey:testvalue",$result);
+		$result = vCal::fold_ical_lines("testkey", "testvalue");
+		$this->assertEquals("testkey:testvalue", $result);
 	
 		
 		//test with longer strings
-		$expected =  "testkey11111111111111111111111111111111111111111111111111111111111111111111\r\n	11111111111111111111111111111111:testvalue11111111111111111111111111111111\r\n	11111111111111111111111111111111111111111111111111111111111111111111";
-		$result = vCal::fold_ical_lines("testkey" . str_repeat("1",100),"testvalue" . str_repeat("1",100));
+		$expected = "testkey11111111111111111111111111111111111111111111111111111111111111111111\r\n	11111111111111111111111111111111:testvalue11111111111111111111111111111111\r\n	11111111111111111111111111111111111111111111111111111111111111111111";
+		$result = vCal::fold_ical_lines("testkey".str_repeat("1", 100), "testvalue".str_repeat("1", 100));
 		$this->assertEquals();
 	
 	}
@@ -192,19 +192,19 @@ 
 
 		$iCalString = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\nDTSTART:2016-01-09 00:00:00\r\nDTEND:2016-03-09 00:00:00\r\nDTSTAMP:2016-01-10 11:07:15\r\nEND:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		$expected = array(
-						array ( 'BEGIN','VCALENDAR'),
-						array ( 'VERSION','2.0'),
-						array ( 'PRODID','-//SugarCRM//SugarCRM Calendar//EN'),
-						array ( 'BEGIN', 'VFREEBUSY' ),
-						array ( 'ORGANIZER;CN= ', 'VFREEBUSY'  ),
-						array ( 'DTSTART', '2016-01-09 00:00:00' ),
-						array ( 'DTEND', '2016-03-09 00:00:00' ),
-						array ( 'DTSTAMP', '2016-01-10 11:07:15' ),
-						array ( 'END', 'VFREEBUSY' ),
-						array ( 'END', 'VCALENDAR' ),
+						array('BEGIN', 'VCALENDAR'),
+						array('VERSION', '2.0'),
+						array('PRODID', '-//SugarCRM//SugarCRM Calendar//EN'),
+						array('BEGIN', 'VFREEBUSY'),
+						array('ORGANIZER;CN= ', 'VFREEBUSY'),
+						array('DTSTART', '2016-01-09 00:00:00'),
+						array('DTEND', '2016-03-09 00:00:00'),
+						array('DTSTAMP', '2016-01-10 11:07:15'),
+						array('END', 'VFREEBUSY'),
+						array('END', 'VCALENDAR'),
 					);
 		$actual = vCal::create_ical_array_from_string($iCalString);
-		$this->assertSame($expected, $actual );
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -214,38 +214,38 @@ 
 
 		$expected = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VFREEBUSY\r\nORGANIZER;CN= :VFREEBUSY\r\nDTSTART:2016-01-09 00:00:00\r\nDTEND:2016-03-09 00:00:00\r\nDTSTAMP:2016-01-10 11:07:15\r\nEND:VFREEBUSY\r\nEND:VCALENDAR\r\n";
 		$iCalArray = array(
-				array ( 'BEGIN','VCALENDAR'),
-				array ( 'VERSION','2.0'),
-				array ( 'PRODID','-//SugarCRM//SugarCRM Calendar//EN'),
-				array ( 'BEGIN', 'VFREEBUSY' ),
-				array ( 'ORGANIZER;CN= ', 'VFREEBUSY'  ),
-				array ( 'DTSTART', '2016-01-09 00:00:00' ),
-				array ( 'DTEND', '2016-03-09 00:00:00' ),
-				array ( 'DTSTAMP', '2016-01-10 11:07:15' ),
-				array ( 'END', 'VFREEBUSY' ),
-				array ( 'END', 'VCALENDAR' ),
+				array('BEGIN', 'VCALENDAR'),
+				array('VERSION', '2.0'),
+				array('PRODID', '-//SugarCRM//SugarCRM Calendar//EN'),
+				array('BEGIN', 'VFREEBUSY'),
+				array('ORGANIZER;CN= ', 'VFREEBUSY'),
+				array('DTSTART', '2016-01-09 00:00:00'),
+				array('DTEND', '2016-03-09 00:00:00'),
+				array('DTSTAMP', '2016-01-10 11:07:15'),
+				array('END', 'VFREEBUSY'),
+				array('END', 'VCALENDAR'),
 		);
 		$actual = vCal::create_ical_string_from_array($iCalArray);
-		$this->assertSame($expected, $actual );
+		$this->assertSame($expected, $actual);
 		
 	}
 
 
 	public function testescape_ical_chars()
 	{
-		$this->assertSame('',vCal::escape_ical_chars(''));
-		$this->assertSame('\;\,',vCal::escape_ical_chars(';,'));
+		$this->assertSame('', vCal::escape_ical_chars(''));
+		$this->assertSame('\;\,', vCal::escape_ical_chars(';,'));
 	}
 
 
 	public function testunescape_ical_chars()
 	{
-		$this->assertSame('',vCal::unescape_ical_chars(''));
+		$this->assertSame('', vCal::unescape_ical_chars(''));
 		$this->assertSame("; , \\", vCal::unescape_ical_chars('\\; \\, \\\\'));	
 	}
 
 
-	public function testget_ical_event(){
+	public function testget_ical_event() {
 
 		$user = new User(1);
 		$meeting = new Meeting();
@@ -260,7 +260,7 @@ 
 		$expectedStart = "BEGIN:VCALENDAR\r\nVERSION:2.0\r\nPRODID:-//SugarCRM//SugarCRM Calendar//EN\r\nBEGIN:VEVENT\r\nUID:1\r\nORGANIZED;CN=:\r\nDTSTART:20160211T173000Z\r\nDTEND:20160211T173000Z\r\n"; 
 		$expectedEnd = "\r\nSUMMARY:test\r\nLOCATION:test location\r\nDESCRIPTION:test description\r\nEND:VEVENT\r\nEND:VCALENDAR\r\n";
 
-		$result = vCal::get_ical_event($meeting ,$user);
+		$result = vCal::get_ical_event($meeting, $user);
 		
 		$this->assertStringStartsWith($expectedStart, $result);
 		$this->assertStringEndsWith($expectedEnd, $result);

Braces +5 added lines, -10 removed lines

@@ -45,8 +45,7 @@ 
 		try {
 			$vcal->fill_in_additional_list_fields();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 
@@ -62,8 +61,7 @@ 
 		try {
 			$vcal->fill_in_additional_detail_fields();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -78,8 +76,7 @@ 
 		try {
 			$vcal->get_list_view_data();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -146,8 +143,7 @@ 
 		try {
 			$vcal->cache_sugar_vcal($user_focus);
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -163,8 +159,7 @@ 
 		try {
 			$vcal->cache_sugar_vcal_freebusy($user_focus);
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 				

tests/tests/modules/SugarFeed/SugarFeedTest.php

Indentation +126 added lines, -126 removed lines

@@ -3,225 +3,225 @@
 class SugarFeedTest extends PHPUnit_Framework_TestCase  {
 
 	
-	public function testSugarFeed(){
+    public function testSugarFeed(){
 
-		error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 		
-		//execute the contructor and check for the Object type and  attributes
-		$sugarFeed = new SugarFeed();
+        //execute the contructor and check for the Object type and  attributes
+        $sugarFeed = new SugarFeed();
 		
-		$this->assertInstanceOf('SugarFeed',$sugarFeed);
-		$this->assertInstanceOf('Basic',$sugarFeed);
-		$this->assertInstanceOf('SugarBean',$sugarFeed);
+        $this->assertInstanceOf('SugarFeed',$sugarFeed);
+        $this->assertInstanceOf('Basic',$sugarFeed);
+        $this->assertInstanceOf('SugarBean',$sugarFeed);
 			
-		$this->assertAttributeEquals('sugarfeed', 'table_name', $sugarFeed);
-		$this->assertAttributeEquals('SugarFeed', 'module_dir', $sugarFeed);
-		$this->assertAttributeEquals('SugarFeed', 'object_name', $sugarFeed);
+        $this->assertAttributeEquals('sugarfeed', 'table_name', $sugarFeed);
+        $this->assertAttributeEquals('SugarFeed', 'module_dir', $sugarFeed);
+        $this->assertAttributeEquals('SugarFeed', 'object_name', $sugarFeed);
 		
-		$this->assertAttributeEquals(true, 'new_schema', $sugarFeed);
-		$this->assertAttributeEquals(false, 'importable', $sugarFeed);
+        $this->assertAttributeEquals(true, 'new_schema', $sugarFeed);
+        $this->assertAttributeEquals(false, 'importable', $sugarFeed);
 		
-	}
+    }
 
-	public function testactivateAndDisableModuleFeed() {
+    public function testactivateAndDisableModuleFeed() {
 
-		$admin = new Administration();
+        $admin = new Administration();
 		
-		//test activateModuleFeed method
-		SugarFeed::activateModuleFeed('Accounts');
-		$admin->retrieveSettings('sugarfeed');
-		$this->assertEquals(1,$admin->settings['sugarfeed_module_Accounts']);
+        //test activateModuleFeed method
+        SugarFeed::activateModuleFeed('Accounts');
+        $admin->retrieveSettings('sugarfeed');
+        $this->assertEquals(1,$admin->settings['sugarfeed_module_Accounts']);
 		
 
-		//test disableModuleFeed method
-		SugarFeed::disableModuleFeed('Accounts');
-		$admin->retrieveSettings('sugarfeed');
-		$this->assertEquals(0, $admin->settings['sugarfeed_module_Accounts']);
+        //test disableModuleFeed method
+        SugarFeed::disableModuleFeed('Accounts');
+        $admin->retrieveSettings('sugarfeed');
+        $this->assertEquals(0, $admin->settings['sugarfeed_module_Accounts']);
 		
-	}
+    }
 	
     
     public function testflushBackendCache() {
 
-    	//execute the method and test if it works and does not throws an exception.
-    	try {
-    		SugarFeed::flushBackendCache();
-    		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
-    		$this->fail();
-    	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            SugarFeed::flushBackendCache();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
     	
     }
 
 
     public function testgetModuleFeedFiles( ) {
 
-    	//test with invalid module
-    	$expected = array();
-    	$result = SugarFeed::getModuleFeedFiles('Accounts');
-    	$this->assertEquals($expected, $result);
+        //test with invalid module
+        $expected = array();
+        $result = SugarFeed::getModuleFeedFiles('Accounts');
+        $this->assertEquals($expected, $result);
     	
     	
-    	//test with valid module
-    	$expected = array( "CaseFeed.php"=> "modules/Cases/SugarFeeds/CaseFeed.php");
-    	$result = SugarFeed::getModuleFeedFiles('Cases');
-    	$this->assertEquals($expected, $result);
+        //test with valid module
+        $expected = array( "CaseFeed.php"=> "modules/Cases/SugarFeeds/CaseFeed.php");
+        $result = SugarFeed::getModuleFeedFiles('Cases');
+        $this->assertEquals($expected, $result);
     	    	
     }
 
     public function testgetActiveFeedModules( ) {
 
-    	$result = SugarFeed::getActiveFeedModules();
+        $result = SugarFeed::getActiveFeedModules();
     	
-    	$expected = array (
-    			'UserFeed' => 'UserFeed',
-    			'Cases' => 'Cases',
-    			'Contacts' => 'Contacts',
-    			'Leads' => 'Leads',
-    			'Opportunities' => 'Opportunities',
-    	);
+        $expected = array (
+                'UserFeed' => 'UserFeed',
+                'Cases' => 'Cases',
+                'Contacts' => 'Contacts',
+                'Leads' => 'Leads',
+                'Opportunities' => 'Opportunities',
+        );
     	
-    	$this->assertEquals($expected, $result);
+        $this->assertEquals($expected, $result);
     	
     }
 
     public function testgetAllFeedModules( ) {
 
-    	$result = SugarFeed::getAllFeedModules();
-    	$expected = array (
-					  'UserFeed' => 'UserFeed',
-					  'Cases' => 'Cases',
-					  'Contacts' => 'Contacts',
-					  'Leads' => 'Leads',
-					  'Opportunities' => 'Opportunities',
-					);
+        $result = SugarFeed::getAllFeedModules();
+        $expected = array (
+                        'UserFeed' => 'UserFeed',
+                        'Cases' => 'Cases',
+                        'Contacts' => 'Contacts',
+                        'Leads' => 'Leads',
+                        'Opportunities' => 'Opportunities',
+                    );
     	
-    	$this->assertEquals($expected, $result);
+        $this->assertEquals($expected, $result);
     	
     }
 
 
     public function testpushFeed2() {
   	
-    	$lead = new Lead();
-    	$lead->id = 1;
-    	$lead->assigned_user_id = 1;
+        $lead = new Lead();
+        $lead->id = 1;
+        $lead->assigned_user_id = 1;
     	
-    	SugarFeed::pushFeed2("some text 2", $lead,'Link', 'some url');
+        SugarFeed::pushFeed2("some text 2", $lead,'Link', 'some url');
     	
-    	//retrieve newly created bean
-    	$sugarFeed = new SugarFeed();
-    	$result = $sugarFeed->retrieve_by_string_fields(array("related_id" => '1', 'related_module'=> 'Leads'));
+        //retrieve newly created bean
+        $sugarFeed = new SugarFeed();
+        $result = $sugarFeed->retrieve_by_string_fields(array("related_id" => '1', 'related_module'=> 'Leads'));
     	
-    	//test for record ID to verify that record is saved
-    	$this->assertTrue(isset($sugarFeed->id));
-    	$this->assertEquals(36, strlen($sugarFeed->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($sugarFeed->id));
+        $this->assertEquals(36, strlen($sugarFeed->id));
     	
-    	//mark the record as deleted
-    	$sugarFeed->mark_deleted($sugarFeed->id);
+        //mark the record as deleted
+        $sugarFeed->mark_deleted($sugarFeed->id);
     	
     }
 
-	public function testpushFeed(){
+    public function testpushFeed(){
 
 		
-		SugarFeed::pushFeed("some text", 'SugarFeed', 1, 1,'Link', 'some url');
+        SugarFeed::pushFeed("some text", 'SugarFeed', 1, 1,'Link', 'some url');
 		
 		
-		//retrieve newly created bean
-		$sugarFeed = new SugarFeed();
-		$result = $sugarFeed->retrieve_by_string_fields(array("related_id" => '1', 'related_module'=> 'SugarFeed'));
+        //retrieve newly created bean
+        $sugarFeed = new SugarFeed();
+        $result = $sugarFeed->retrieve_by_string_fields(array("related_id" => '1', 'related_module'=> 'SugarFeed'));
 
 	
-		//test for record ID to verify that record is saved
-		$this->assertTrue(isset($sugarFeed->id));
-		$this->assertEquals(36, strlen($sugarFeed->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($sugarFeed->id));
+        $this->assertEquals(36, strlen($sugarFeed->id));
 		
 		
-		//test fetchReplies method
-		$this->fetchReplies();
+        //test fetchReplies method
+        $this->fetchReplies();
 		
 		
-		//mark the record as deleted 
-		$sugarFeed->mark_deleted($sugarFeed->id);
+        //mark the record as deleted 
+        $sugarFeed->mark_deleted($sugarFeed->id);
 		
-	}
+    }
 
-	public function fetchReplies() {
+    public function fetchReplies() {
 	 
-		$sugarFeed = new SugarFeed();
+        $sugarFeed = new SugarFeed();
 		
-		$actual = $sugarFeed->fetchReplies(array('ID'=>'1'));
-		$this->assertGreaterThan(0,strlen($actual));
+        $actual = $sugarFeed->fetchReplies(array('ID'=>'1'));
+        $this->assertGreaterThan(0,strlen($actual));
 		   
-	}
+    }
 	
     public function testgetLinkTypes() {
 
-    	$result = SugarFeed::getLinkTypes();
+        $result = SugarFeed::getLinkTypes();
     	
-    	$expected = array (
-    			'Image' => 'Image',
-    			'Link' => 'Link',
-    			'YouTube' => 'YouTube',
-    	);
-    	$this->assertEquals($expected, $result);
+        $expected = array (
+                'Image' => 'Image',
+                'Link' => 'Link',
+                'YouTube' => 'YouTube',
+        );
+        $this->assertEquals($expected, $result);
     	
     }
 
     public function testgetLinkClass( ) {
 
-    	//test with invalid LinkType
-    	$result = SugarFeed::getLinkClass('test');
-    	$this->assertEquals(false, $result);
+        //test with invalid LinkType
+        $result = SugarFeed::getLinkClass('test');
+        $this->assertEquals(false, $result);
     	
-    	//test with LinkType Image
-    	$result = SugarFeed::getLinkClass('Image');
-    	$this->assertInstanceOf('FeedLinkHandlerImage',$result);
+        //test with LinkType Image
+        $result = SugarFeed::getLinkClass('Image');
+        $this->assertInstanceOf('FeedLinkHandlerImage',$result);
     	
-    	//test with LinkType Link
-    	$result = SugarFeed::getLinkClass('Link');
-    	$this->assertInstanceOf('FeedLinkHandlerLink',$result);
+        //test with LinkType Link
+        $result = SugarFeed::getLinkClass('Link');
+        $this->assertInstanceOf('FeedLinkHandlerLink',$result);
 
-    	//test with LinkType YouTube
-    	$result = SugarFeed::getLinkClass('YouTube');
-    	$this->assertInstanceOf('FeedLinkHandlerYoutube',$result);
+        //test with LinkType YouTube
+        $result = SugarFeed::getLinkClass('YouTube');
+        $this->assertInstanceOf('FeedLinkHandlerYoutube',$result);
         	 	
     }
 
-	public function testget_list_view_data(){
+    public function testget_list_view_data(){
 		
-		$sugarFeed = new SugarFeed();
+        $sugarFeed = new SugarFeed();
 		
-		$result = $sugarFeed->get_list_view_data();
-		$this->assertTrue(is_array($result));
+        $result = $sugarFeed->get_list_view_data();
+        $this->assertTrue(is_array($result));
 				
-	}
+    }
 
 	
-	public function testgetTimeLapse()
-	{
-		$result = SugarFeed::getTimeLapse('2016-01-15 11:16:02');
-		$this->assertTrue(isset($result));
-		$this->assertGreaterThan(0, strlen($result));
+    public function testgetTimeLapse()
+    {
+        $result = SugarFeed::getTimeLapse('2016-01-15 11:16:02');
+        $this->assertTrue(isset($result));
+        $this->assertGreaterThan(0, strlen($result));
 		
     }
 
 
     public function testparseMessage(){
 
-    	//test with a string with no links
-    	$html = "some text with no urls";
-    	$result = SugarFeed::parseMessage($html);
-    	$this->assertEquals($html ,$result);
+        //test with a string with no links
+        $html = "some text with no urls";
+        $result = SugarFeed::parseMessage($html);
+        $this->assertEquals($html ,$result);
     	
     	
-    	//test with a string with links
-    	$html = "some text http://www.url.com with no urls";
-    	$expected = "some text <a href='http://www.url.com' target='_blank'>http://www.url.com</a> with no urls";
-    	$result = SugarFeed::parseMessage($html);
-    	$this->assertEquals($expected ,$result);
+        //test with a string with links
+        $html = "some text http://www.url.com with no urls";
+        $expected = "some text <a href='http://www.url.com' target='_blank'>http://www.url.com</a> with no urls";
+        $result = SugarFeed::parseMessage($html);
+        $this->assertEquals($expected ,$result);
     	
     }
 

Spacing +21 added lines, -21 removed lines

@@ -1,18 +1,18 @@ 
 <?PHP
 
-class SugarFeedTest extends PHPUnit_Framework_TestCase  {
+class SugarFeedTest extends PHPUnit_Framework_TestCase {
 
 	
-	public function testSugarFeed(){
+	public function testSugarFeed() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		
 		//execute the contructor and check for the Object type and  attributes
 		$sugarFeed = new SugarFeed();
 		
-		$this->assertInstanceOf('SugarFeed',$sugarFeed);
-		$this->assertInstanceOf('Basic',$sugarFeed);
-		$this->assertInstanceOf('SugarBean',$sugarFeed);
+		$this->assertInstanceOf('SugarFeed', $sugarFeed);
+		$this->assertInstanceOf('Basic', $sugarFeed);
+		$this->assertInstanceOf('SugarBean', $sugarFeed);
 			
 		$this->assertAttributeEquals('sugarfeed', 'table_name', $sugarFeed);
 		$this->assertAttributeEquals('SugarFeed', 'module_dir', $sugarFeed);
@@ -30,7 +30,7 @@ 
 		//test activateModuleFeed method
 		SugarFeed::activateModuleFeed('Accounts');
 		$admin->retrieveSettings('sugarfeed');
-		$this->assertEquals(1,$admin->settings['sugarfeed_module_Accounts']);
+		$this->assertEquals(1, $admin->settings['sugarfeed_module_Accounts']);
 		
 
 		//test disableModuleFeed method
@@ -64,7 +64,7 @@ 
     	
     	
     	//test with valid module
-    	$expected = array( "CaseFeed.php"=> "modules/Cases/SugarFeeds/CaseFeed.php");
+    	$expected = array("CaseFeed.php"=> "modules/Cases/SugarFeeds/CaseFeed.php");
     	$result = SugarFeed::getModuleFeedFiles('Cases');
     	$this->assertEquals($expected, $result);
     	    	
@@ -74,7 +74,7 @@ 
 
     	$result = SugarFeed::getActiveFeedModules();
     	
-    	$expected = array (
+    	$expected = array(
     			'UserFeed' => 'UserFeed',
     			'Cases' => 'Cases',
     			'Contacts' => 'Contacts',
@@ -89,7 +89,7 @@ 
     public function testgetAllFeedModules( ) {
 
     	$result = SugarFeed::getAllFeedModules();
-    	$expected = array (
+    	$expected = array(
 					  'UserFeed' => 'UserFeed',
 					  'Cases' => 'Cases',
 					  'Contacts' => 'Contacts',
@@ -108,7 +108,7 @@ 
     	$lead->id = 1;
     	$lead->assigned_user_id = 1;
     	
-    	SugarFeed::pushFeed2("some text 2", $lead,'Link', 'some url');
+    	SugarFeed::pushFeed2("some text 2", $lead, 'Link', 'some url');
     	
     	//retrieve newly created bean
     	$sugarFeed = new SugarFeed();
@@ -123,10 +123,10 @@ 
     	
     }
 
-	public function testpushFeed(){
+	public function testpushFeed() {
 
 		
-		SugarFeed::pushFeed("some text", 'SugarFeed', 1, 1,'Link', 'some url');
+		SugarFeed::pushFeed("some text", 'SugarFeed', 1, 1, 'Link', 'some url');
 		
 		
 		//retrieve newly created bean
@@ -153,7 +153,7 @@ 
 		$sugarFeed = new SugarFeed();
 		
 		$actual = $sugarFeed->fetchReplies(array('ID'=>'1'));
-		$this->assertGreaterThan(0,strlen($actual));
+		$this->assertGreaterThan(0, strlen($actual));
 		   
 	}
 	
@@ -161,7 +161,7 @@ 
 
     	$result = SugarFeed::getLinkTypes();
     	
-    	$expected = array (
+    	$expected = array(
     			'Image' => 'Image',
     			'Link' => 'Link',
     			'YouTube' => 'YouTube',
@@ -178,19 +178,19 @@ 
     	
     	//test with LinkType Image
     	$result = SugarFeed::getLinkClass('Image');
-    	$this->assertInstanceOf('FeedLinkHandlerImage',$result);
+    	$this->assertInstanceOf('FeedLinkHandlerImage', $result);
     	
     	//test with LinkType Link
     	$result = SugarFeed::getLinkClass('Link');
-    	$this->assertInstanceOf('FeedLinkHandlerLink',$result);
+    	$this->assertInstanceOf('FeedLinkHandlerLink', $result);
 
     	//test with LinkType YouTube
     	$result = SugarFeed::getLinkClass('YouTube');
-    	$this->assertInstanceOf('FeedLinkHandlerYoutube',$result);
+    	$this->assertInstanceOf('FeedLinkHandlerYoutube', $result);
         	 	
     }
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 		
 		$sugarFeed = new SugarFeed();
 		
@@ -209,19 +209,19 @@ 
     }
 
 
-    public function testparseMessage(){
+    public function testparseMessage() {
 
     	//test with a string with no links
     	$html = "some text with no urls";
     	$result = SugarFeed::parseMessage($html);
-    	$this->assertEquals($html ,$result);
+    	$this->assertEquals($html, $result);
     	
     	
     	//test with a string with links
     	$html = "some text http://www.url.com with no urls";
     	$expected = "some text <a href='http://www.url.com' target='_blank'>http://www.url.com</a> with no urls";
     	$result = SugarFeed::parseMessage($html);
-    	$this->assertEquals($expected ,$result);
+    	$this->assertEquals($expected, $result);
     	
     }
 

Braces +1 added lines, -2 removed lines

@@ -47,8 +47,7 @@
     	try {
     		SugarFeed::flushBackendCache();
     		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
+    	} catch (Exception $e) {
     		$this->fail();
     	}
     	

tests/tests/modules/UserPreferences/UserPreferenceTest.php

Indentation +103 added lines, -103 removed lines

@@ -3,169 +3,169 @@
 class UserPreferenceTest extends PHPUnit_Framework_TestCase
 {
 
-	public function test__construct()
-	{
-		//execute the contructor and check for the Object type and  attributes
-		$userPreference = new UserPreference();
+    public function test__construct()
+    {
+        //execute the contructor and check for the Object type and  attributes
+        $userPreference = new UserPreference();
 		
-		$this->assertInstanceOf('UserPreference',$userPreference);
-		$this->assertInstanceOf('SugarBean',$userPreference);
+        $this->assertInstanceOf('UserPreference',$userPreference);
+        $this->assertInstanceOf('SugarBean',$userPreference);
 			
-		$this->assertAttributeEquals('user_preferences', 'table_name', $userPreference);
-		$this->assertAttributeEquals('UserPreferences', 'module_dir', $userPreference);
-		$this->assertAttributeEquals('UserPreference', 'object_name', $userPreference);
+        $this->assertAttributeEquals('user_preferences', 'table_name', $userPreference);
+        $this->assertAttributeEquals('UserPreferences', 'module_dir', $userPreference);
+        $this->assertAttributeEquals('UserPreference', 'object_name', $userPreference);
 
 		
-		$this->assertAttributeEquals(true, 'new_schema', $userPreference);
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $userPreference);
+        $this->assertAttributeEquals(true, 'new_schema', $userPreference);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $userPreference);
 		
-	}
+    }
 
 	
-	public function testgetDefaultPreference( )
-	{
-		global $sugar_config;
+    public function testgetDefaultPreference( )
+    {
+        global $sugar_config;
 		
-		error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 		
-		$userPreference = new UserPreference();
+        $userPreference = new UserPreference();
 		
-		//test with non global category
-		$result = $userPreference->getDefaultPreference('chartEngine', 'Home'); 
-		$this->assertEquals(null,$result);
+        //test with non global category
+        $result = $userPreference->getDefaultPreference('chartEngine', 'Home'); 
+        $this->assertEquals(null,$result);
 		
 		
-		//test with default global category
+        //test with default global category
 		
-		$result = $userPreference->getDefaultPreference('chartEngine'); 
-		$this->assertEquals($sugar_config['chartEngine'] ,$result);
+        $result = $userPreference->getDefaultPreference('chartEngine'); 
+        $this->assertEquals($sugar_config['chartEngine'] ,$result);
 	
 		
-		$date_format = $sugar_config['datef'] != "" ? $sugar_config['datef'] : $sugar_config['default_date_format'];
-		$result = $userPreference->getDefaultPreference('datef');
-		$this->assertEquals($date_format, $result);
+        $date_format = $sugar_config['datef'] != "" ? $sugar_config['datef'] : $sugar_config['default_date_format'];
+        $result = $userPreference->getDefaultPreference('datef');
+        $this->assertEquals($date_format, $result);
 		
 		
-		$time_format = $sugar_config['timef'] != ""? $sugar_config['timef'] : $sugar_config['default_time_format'];
-		$result = $userPreference->getDefaultPreference('timef');
-		$this->assertEquals($time_format , $result);
+        $time_format = $sugar_config['timef'] != ""? $sugar_config['timef'] : $sugar_config['default_time_format'];
+        $result = $userPreference->getDefaultPreference('timef');
+        $this->assertEquals($time_format , $result);
 		
 		
-		$email_link_type = $sugar_config['email_link_type'] != "" ? $sugar_config['email_link_type'] : $sugar_config['email_default_client'];
-		$result = $userPreference->getDefaultPreference('email_link_type');
-		$this->assertEquals($email_link_type , $result);
+        $email_link_type = $sugar_config['email_link_type'] != "" ? $sugar_config['email_link_type'] : $sugar_config['email_default_client'];
+        $result = $userPreference->getDefaultPreference('email_link_type');
+        $this->assertEquals($email_link_type , $result);
 				
-	}
+    }
 
 
-	public function testSetAndGetPreference()
-	{
-		global $sugar_config;
+    public function testSetAndGetPreference()
+    {
+        global $sugar_config;
 		
-		$user = new User();
-		$user->retrieve("1");
+        $user = new User();
+        $user->retrieve("1");
 
-		$userPreference = new UserPreference($user);
+        $userPreference = new UserPreference($user);
 		
-		//test setPreference method
-		$userPreference->setPreference("test","test val","test_category");
-		$result = $_SESSION[$user->user_name.'_PREFERENCES']['test_category']['test'];
-		$this->assertEquals("test val", $result);
+        //test setPreference method
+        $userPreference->setPreference("test","test val","test_category");
+        $result = $_SESSION[$user->user_name.'_PREFERENCES']['test_category']['test'];
+        $this->assertEquals("test val", $result);
 		
 		
-		//test getPreference method
-		$result = $userPreference->getPreference("test", "test_category");
-		$this->assertEquals("test val", $result);
+        //test getPreference method
+        $result = $userPreference->getPreference("test", "test_category");
+        $this->assertEquals("test val", $result);
 		
-		$result = $userPreference->getPreference("chartEngine");
-		$this->assertEquals($sugar_config['chartEngine'], $result);
+        $result = $userPreference->getPreference("chartEngine");
+        $this->assertEquals($sugar_config['chartEngine'], $result);
 		
-	}
+    }
 
 	
-	public function testloadPreferences( )
-	{
-		$user = new User();
-		$user->retrieve("1");
+    public function testloadPreferences( )
+    {
+        $user = new User();
+        $user->retrieve("1");
 		
-		$userPreference = new UserPreference($user);
+        $userPreference = new UserPreference($user);
 		
-		$result = $userPreference->loadPreferences();
+        $result = $userPreference->loadPreferences();
 		
-		$this->assertEquals(false, $result);
-	}
+        $this->assertEquals(false, $result);
+    }
 
 
-	public function testreloadPreferences()
-	{
+    public function testreloadPreferences()
+    {
 
-		$user = new User();
-		$user->retrieve("1");
+        $user = new User();
+        $user->retrieve("1");
 		
-		$userPreference = new UserPreference($user);
+        $userPreference = new UserPreference($user);
 		
-		$result = $userPreference->reloadPreferences();
-		$this->assertEquals(false, $result);
+        $result = $userPreference->reloadPreferences();
+        $this->assertEquals(false, $result);
 		
-	}
+    }
 
 
-	public function testgetUserDateTimePreferences()
-	{
+    public function testgetUserDateTimePreferences()
+    {
 
-		$user = new User();
-		$user->retrieve("1");
+        $user = new User();
+        $user->retrieve("1");
 		
-		$userPreference = new UserPreference($user);
+        $userPreference = new UserPreference($user);
 		
-		$result = $userPreference->getUserDateTimePreferences();
-		$this->assertTrue(is_array($result));
+        $result = $userPreference->getUserDateTimePreferences();
+        $this->assertTrue(is_array($result));
 		
-	}
+    }
 
 
-	public function testSavePreferencesToDBAndResetPreferences( )
-	{
-		$user = new User();
-		$user->retrieve("1");
+    public function testSavePreferencesToDBAndResetPreferences( )
+    {
+        $user = new User();
+        $user->retrieve("1");
 		
-		$userPreference = new UserPreference($user);
+        $userPreference = new UserPreference($user);
 		
-		//create a Preference record, save it to DB
-		$userPreference->setPreference("test","test val","test_category");
-		$userPreference->savePreferencesToDB();
+        //create a Preference record, save it to DB
+        $userPreference->setPreference("test","test val","test_category");
+        $userPreference->savePreferencesToDB();
 		
 		
-		//retrieve it back and verify
-		$result = $userPreference->retrieve_by_string_fields(array(
-				'assigned_user_id' => $user->id,
-				'category' => 'test_category',
-		));
-		$this->assertTrue(isset($result->id));
+        //retrieve it back and verify
+        $result = $userPreference->retrieve_by_string_fields(array(
+                'assigned_user_id' => $user->id,
+                'category' => 'test_category',
+        ));
+        $this->assertTrue(isset($result->id));
 		
 		
-		//reset the preferences and verify that it is deleted
-		$userPreference->resetPreferences();
-		$result = $userPreference->retrieve_by_string_fields(array(
-				'assigned_user_id' => $user->id,
-				'category' => 'test_category',
-		));
-		$this->assertEquals(null,$result);
+        //reset the preferences and verify that it is deleted
+        $userPreference->resetPreferences();
+        $result = $userPreference->retrieve_by_string_fields(array(
+                'assigned_user_id' => $user->id,
+                'category' => 'test_category',
+        ));
+        $this->assertEquals(null,$result);
 		
-	}
+    }
 
 
-	public function testupdateAllUserPrefs( )
-	{	
-		global $current_user;
+    public function testupdateAllUserPrefs( )
+    {	
+        global $current_user;
 
-		$current_user = new User();
-		$current_user->retrieve("1");
+        $current_user = new User();
+        $current_user->retrieve("1");
 		
-		//UserPreference::updateAllUserPrefs("test","test val");
+        //UserPreference::updateAllUserPrefs("test","test val");
 		
-		$this->markTestIncomplete('Multiple errors in method: Unknown column user_preferences in field list');
+        $this->markTestIncomplete('Multiple errors in method: Unknown column user_preferences in field list');
 		
-	}
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$userPreference = new UserPreference();
 		
-		$this->assertInstanceOf('UserPreference',$userPreference);
-		$this->assertInstanceOf('SugarBean',$userPreference);
+		$this->assertInstanceOf('UserPreference', $userPreference);
+		$this->assertInstanceOf('SugarBean', $userPreference);
 			
 		$this->assertAttributeEquals('user_preferences', 'table_name', $userPreference);
 		$this->assertAttributeEquals('UserPreferences', 'module_dir', $userPreference);
@@ -32,13 +32,13 @@ 
 		
 		//test with non global category
 		$result = $userPreference->getDefaultPreference('chartEngine', 'Home'); 
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 		
 		//test with default global category
 		
 		$result = $userPreference->getDefaultPreference('chartEngine'); 
-		$this->assertEquals($sugar_config['chartEngine'] ,$result);
+		$this->assertEquals($sugar_config['chartEngine'], $result);
 	
 		
 		$date_format = $sugar_config['datef'] != "" ? $sugar_config['datef'] : $sugar_config['default_date_format'];
@@ -46,14 +46,14 @@ 
 		$this->assertEquals($date_format, $result);
 		
 		
-		$time_format = $sugar_config['timef'] != ""? $sugar_config['timef'] : $sugar_config['default_time_format'];
+		$time_format = $sugar_config['timef'] != "" ? $sugar_config['timef'] : $sugar_config['default_time_format'];
 		$result = $userPreference->getDefaultPreference('timef');
-		$this->assertEquals($time_format , $result);
+		$this->assertEquals($time_format, $result);
 		
 		
 		$email_link_type = $sugar_config['email_link_type'] != "" ? $sugar_config['email_link_type'] : $sugar_config['email_default_client'];
 		$result = $userPreference->getDefaultPreference('email_link_type');
-		$this->assertEquals($email_link_type , $result);
+		$this->assertEquals($email_link_type, $result);
 				
 	}
 
@@ -68,7 +68,7 @@ 
 		$userPreference = new UserPreference($user);
 		
 		//test setPreference method
-		$userPreference->setPreference("test","test val","test_category");
+		$userPreference->setPreference("test", "test val", "test_category");
 		$result = $_SESSION[$user->user_name.'_PREFERENCES']['test_category']['test'];
 		$this->assertEquals("test val", $result);
 		
@@ -132,7 +132,7 @@ 
 		$userPreference = new UserPreference($user);
 		
 		//create a Preference record, save it to DB
-		$userPreference->setPreference("test","test val","test_category");
+		$userPreference->setPreference("test", "test val", "test_category");
 		$userPreference->savePreferencesToDB();
 		
 		
@@ -150,7 +150,7 @@ 
 				'assigned_user_id' => $user->id,
 				'category' => 'test_category',
 		));
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 

tests/tests/modules/Opportunities/OpportunityTest.php

Indentation +194 added lines, -194 removed lines

@@ -3,311 +3,311 @@
 class OpportunityTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testOpportunity() {
+    public function testOpportunity() {
 
 
-		//execute the contructor and check for the Object type and  attributes
-		$opportunity = new Opportunity();
+        //execute the contructor and check for the Object type and  attributes
+        $opportunity = new Opportunity();
 			
-		$this->assertInstanceOf('Opportunity',$opportunity);
-		$this->assertInstanceOf('SugarBean',$opportunity);
+        $this->assertInstanceOf('Opportunity',$opportunity);
+        $this->assertInstanceOf('SugarBean',$opportunity);
 		
 
-		$this->assertAttributeEquals('opportunities', 'table_name', $opportunity);
-		$this->assertAttributeEquals('accounts_opportunities', 'rel_account_table', $opportunity);
-		$this->assertAttributeEquals('opportunities_contacts', 'rel_contact_table', $opportunity);
-		$this->assertAttributeEquals('Opportunities', 'module_dir', $opportunity);
-		$this->assertAttributeEquals('Opportunity', 'object_name', $opportunity);
+        $this->assertAttributeEquals('opportunities', 'table_name', $opportunity);
+        $this->assertAttributeEquals('accounts_opportunities', 'rel_account_table', $opportunity);
+        $this->assertAttributeEquals('opportunities_contacts', 'rel_contact_table', $opportunity);
+        $this->assertAttributeEquals('Opportunities', 'module_dir', $opportunity);
+        $this->assertAttributeEquals('Opportunity', 'object_name', $opportunity);
 		
-		$this->assertAttributeEquals(true, 'new_schema', $opportunity);
-		$this->assertAttributeEquals(true, 'importable', $opportunity);
+        $this->assertAttributeEquals(true, 'new_schema', $opportunity);
+        $this->assertAttributeEquals(true, 'importable', $opportunity);
 
-	}
+    }
 
 
-	public function testget_summary_text()	{
+    public function testget_summary_text()	{
 
-		error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 		
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		//test without setting name
-		$this->assertEquals(Null,$opportunity->get_summary_text());
+        //test without setting name
+        $this->assertEquals(Null,$opportunity->get_summary_text());
 		
-		//test with name set
-		$opportunity->name = "test";
-		$this->assertEquals('test',$opportunity->get_summary_text());
+        //test with name set
+        $opportunity->name = "test";
+        $this->assertEquals('test',$opportunity->get_summary_text());
 		
-	}
+    }
 
-	public function testcreate_list_query(){
+    public function testcreate_list_query(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		//test with empty string params
-		$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY opportunities.name";
-		$actual = $opportunity->create_list_query('','');
-		$this->assertSame($expected,$actual);
+        //test with empty string params
+        $expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY opportunities.name";
+        $actual = $opportunity->create_list_query('','');
+        $this->assertSame($expected,$actual);
 		
 			
-		//test with valid string params
-		$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where (accounts.name=\"\") AND \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY accounts.id";
-		$actual = $opportunity->create_list_query('accounts.id','accounts.name=""');
-		$this->assertSame($expected,$actual);
+        //test with valid string params
+        $expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where (accounts.name=\"\") AND \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY accounts.id";
+        $actual = $opportunity->create_list_query('accounts.id','accounts.name=""');
+        $this->assertSame($expected,$actual);
 			
-	}
+    }
 	
 
     public function testcreate_export_query(){
 
-    	$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
     	
-    	//test with empty string params
-    	$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY opportunities.name";
-    	$actual = $opportunity->create_list_query('','');
-    	$this->assertSame($expected,$actual);
+        //test with empty string params
+        $expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY opportunities.name";
+        $actual = $opportunity->create_list_query('','');
+        $this->assertSame($expected,$actual);
 
     	
-    	//test with valid string params
-    	$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where (accounts.name=\"\") AND \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY accounts.id";
-    	$actual = $opportunity->create_list_query('accounts.id','accounts.name=""');
-    	$this->assertSame($expected,$actual);
+        //test with valid string params
+        $expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where (accounts.name=\"\") AND \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY accounts.id";
+        $actual = $opportunity->create_list_query('accounts.id','accounts.name=""');
+        $this->assertSame($expected,$actual);
     	
     }
 
-	public function testfill_in_additional_list_fields(){
+    public function testfill_in_additional_list_fields(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		//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 force_load_details
-			$opportunity->fill_in_additional_list_fields();
+            //test without force_load_details
+            $opportunity->fill_in_additional_list_fields();
 			
 			
-			//test without force_load_details
-			$opportunity->force_load_details = true;
-			$opportunity->fill_in_additional_list_fields();
+            //test without force_load_details
+            $opportunity->force_load_details = true;
+            $opportunity->fill_in_additional_list_fields();
 			
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 
-	public function testfill_in_additional_detail_fields(){
+    public function testfill_in_additional_detail_fields(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		//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 {
 				
-			$opportunity->fill_in_additional_detail_fields();				
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+            $opportunity->fill_in_additional_detail_fields();				
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 
 
-	public function testget_contacts(){
+    public function testget_contacts(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		$result = $opportunity->get_contacts();
-		$this->assertTrue(is_array($result));
+        $result = $opportunity->get_contacts();
+        $this->assertTrue(is_array($result));
 		
-	}
+    }
 
-	public function testupdate_currency_id(){
+    public function testupdate_currency_id(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 
-		//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 {
 		
-			$opportunity->update_currency_id(array('GBP','EUR'), 'USD');
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+            $opportunity->update_currency_id(array('GBP','EUR'), 'USD');
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
 		
-	}
+    }
 
-	public function testget_list_view_data(){
+    public function testget_list_view_data(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		$opportunity->name  = "test";
+        $opportunity->name  = "test";
 				
-		$expected = array (
-				'NAME' => 'test',
-				'DELETED' => 0,
-				'SALES_STAGE' => '',
-				'ENCODED_NAME' => 'test'
-				);
+        $expected = array (
+                'NAME' => 'test',
+                'DELETED' => 0,
+                'SALES_STAGE' => '',
+                'ENCODED_NAME' => 'test'
+                );
 		
 
-		$actual = $opportunity->get_list_view_data();
-		//$this->assertSame($expected, $actual);
-		$this->assertEquals($expected['NAME'], $actual['NAME']);
-		$this->assertEquals($expected['DELETED'], $actual['DELETED']);
-		$this->assertEquals($expected['SALES_STAGE'], $actual['SALES_STAGE']);
-		$this->assertEquals($expected['ENCODED_NAME'], $actual['ENCODED_NAME']);
+        $actual = $opportunity->get_list_view_data();
+        //$this->assertSame($expected, $actual);
+        $this->assertEquals($expected['NAME'], $actual['NAME']);
+        $this->assertEquals($expected['DELETED'], $actual['DELETED']);
+        $this->assertEquals($expected['SALES_STAGE'], $actual['SALES_STAGE']);
+        $this->assertEquals($expected['ENCODED_NAME'], $actual['ENCODED_NAME']);
 		
-	}
+    }
 
     public function testget_currency_symbol(){
 
-    	$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
     	
-    	//te4st without currency id
-    	$this->assertEquals('', $opportunity->get_currency_symbol());
+        //te4st without currency id
+        $this->assertEquals('', $opportunity->get_currency_symbol());
     	
     	
-    	//test with invalid currency id
-    	$opportunity->currency_id = 1;
-    	$this->assertEquals('', $opportunity->get_currency_symbol());
+        //test with invalid currency id
+        $opportunity->currency_id = 1;
+        $this->assertEquals('', $opportunity->get_currency_symbol());
     	
     }
 
 
-	public function testbuild_generic_where_clause () {
+    public function testbuild_generic_where_clause () {
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 
-		//test with empty string params
-		$expected = "opportunities.name like '%' or accounts.name like '%'";
-		$actual = $opportunity->build_generic_where_clause('');
-		$this->assertSame($expected,$actual);
+        //test with empty string params
+        $expected = "opportunities.name like '%' or accounts.name like '%'";
+        $actual = $opportunity->build_generic_where_clause('');
+        $this->assertSame($expected,$actual);
 		
-	}
+    }
 
-	public function testsave(){
+    public function testsave(){
  
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		$opportunity->name = "test";
-		$opportunity->description ="test description";
-		$opportunity->sales_stage = "Value Proposition";
+        $opportunity->name = "test";
+        $opportunity->description ="test description";
+        $opportunity->sales_stage = "Value Proposition";
 		
-		$result = $opportunity->save();
+        $result = $opportunity->save();
 		 
-		//test for record ID to verify that record is saved
-		$this->assertTrue(isset($opportunity->id));
-		$this->assertEquals(36, strlen($opportunity->id));
-		$this->assertEquals(-99, $opportunity->currency_id);
-		$this->assertEquals(30, $opportunity->probability);
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($opportunity->id));
+        $this->assertEquals(36, strlen($opportunity->id));
+        $this->assertEquals(-99, $opportunity->currency_id);
+        $this->assertEquals(30, $opportunity->probability);
 
 				
-		//mark the record as deleted and verify that this record cannot be retrieved anymore.
-		$opportunity->mark_deleted($opportunity->id);
-		$result = $opportunity->retrieve($opportunity->id);
-		$this->assertEquals(null,$result);
+        //mark the record as deleted and verify that this record cannot be retrieved anymore.
+        $opportunity->mark_deleted($opportunity->id);
+        $result = $opportunity->retrieve($opportunity->id);
+        $this->assertEquals(null,$result);
 		
-	}
+    }
 
-	public function testsave_relationship_changes(){
+    public function testsave_relationship_changes(){
 
-		$opportunity = new Opportunity();
-		$opportunity->account_id = 1;
+        $opportunity = new Opportunity();
+        $opportunity->account_id = 1;
 		
-		try {
-			$opportunity->save_relationship_changes(true);
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        try {
+            $opportunity->save_relationship_changes(true);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 
-	public function testset_opportunity_contact_relationship(){
+    public function testset_opportunity_contact_relationship(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		try {
-			$opportunity->set_opportunity_contact_relationship("1");
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        try {
+            $opportunity->set_opportunity_contact_relationship("1");
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 
-	public function testset_notification_body(){
+    public function testset_notification_body(){
 
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		//test with attributes preset and verify template variables are set accordingly
+        //test with attributes preset and verify template variables are set accordingly
 		
-		$opportunity->name = "test";
-		$opportunity->amount ="100";
-		$opportunity->date_closed = "2015-02-11 17:30:00";
-		$opportunity->description = "tes description";
-		$opportunity->sales_stage = "Value Proposition";
+        $opportunity->name = "test";
+        $opportunity->amount ="100";
+        $opportunity->date_closed = "2015-02-11 17:30:00";
+        $opportunity->description = "tes description";
+        $opportunity->sales_stage = "Value Proposition";
 		
-		$result = $opportunity->set_notification_body(new Sugar_Smarty(), $opportunity);
+        $result = $opportunity->set_notification_body(new Sugar_Smarty(), $opportunity);
 		
-		$this->assertEquals($opportunity->name ,$result->_tpl_vars['OPPORTUNITY_NAME']);
-		$this->assertEquals($opportunity->amount  ,$result->_tpl_vars['OPPORTUNITY_AMOUNT']);
-		$this->assertEquals($opportunity->date_closed ,$result->_tpl_vars['OPPORTUNITY_CLOSEDATE']);
-		$this->assertEquals($opportunity->sales_stage ,$result->_tpl_vars['OPPORTUNITY_STAGE']);
-		$this->assertEquals($opportunity->description ,$result->_tpl_vars['OPPORTUNITY_DESCRIPTION']);
-	}
+        $this->assertEquals($opportunity->name ,$result->_tpl_vars['OPPORTUNITY_NAME']);
+        $this->assertEquals($opportunity->amount  ,$result->_tpl_vars['OPPORTUNITY_AMOUNT']);
+        $this->assertEquals($opportunity->date_closed ,$result->_tpl_vars['OPPORTUNITY_CLOSEDATE']);
+        $this->assertEquals($opportunity->sales_stage ,$result->_tpl_vars['OPPORTUNITY_STAGE']);
+        $this->assertEquals($opportunity->description ,$result->_tpl_vars['OPPORTUNITY_DESCRIPTION']);
+    }
 
-	public function testbean_implements(){
+    public function testbean_implements(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		$this->assertEquals(false, $opportunity->bean_implements('')); //test with blank value
-		$this->assertEquals(false, $opportunity->bean_implements('test')); //test with invalid value
-		$this->assertEquals(true, $opportunity->bean_implements('ACL')); //test with valid value
+        $this->assertEquals(false, $opportunity->bean_implements('')); //test with blank value
+        $this->assertEquals(false, $opportunity->bean_implements('test')); //test with invalid value
+        $this->assertEquals(true, $opportunity->bean_implements('ACL')); //test with valid value
 		
-	}
+    }
 
-	public function testlistviewACLHelper(){
+    public function testlistviewACLHelper(){
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a" );
-		$actual = $opportunity->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+        $expected = array("MAIN"=>"a", "ACCOUNT"=>"a" );
+        $actual = $opportunity->listviewACLHelper();
+        $this->assertSame($expected,$actual);
 		
-	}
+    }
 
 
-	public function testget_account_detail() {
+    public function testget_account_detail() {
 
-		$opportunity = new Opportunity();
+        $opportunity = new Opportunity();
 		
-		$result = $opportunity->get_account_detail("1");
-		$this->assertTrue(is_array($result));
+        $result = $opportunity->get_account_detail("1");
+        $this->assertTrue(is_array($result));
 		
-	}
+    }
 	
-	public function testgetCurrencyType(){
+    public function testgetCurrencyType(){
 	
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			getCurrencyType();
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
-		
-		$this->markTestIncomplete('This method has no implementation');
-	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            getCurrencyType();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
+		
+        $this->markTestIncomplete('This method has no implementation');
+    }
 	
 }
 

Spacing +43 added lines, -43 removed lines

@@ -9,8 +9,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$opportunity = new Opportunity();
 			
-		$this->assertInstanceOf('Opportunity',$opportunity);
-		$this->assertInstanceOf('SugarBean',$opportunity);
+		$this->assertInstanceOf('Opportunity', $opportunity);
+		$this->assertInstanceOf('SugarBean', $opportunity);
 		
 
 		$this->assertAttributeEquals('opportunities', 'table_name', $opportunity);
@@ -25,57 +25,57 @@ 
 	}
 
 
-	public function testget_summary_text()	{
+	public function testget_summary_text() {
 
 		error_reporting(E_ERROR | E_PARSE);
 		
 		$opportunity = new Opportunity();
 		
 		//test without setting name
-		$this->assertEquals(Null,$opportunity->get_summary_text());
+		$this->assertEquals(Null, $opportunity->get_summary_text());
 		
 		//test with name set
 		$opportunity->name = "test";
-		$this->assertEquals('test',$opportunity->get_summary_text());
+		$this->assertEquals('test', $opportunity->get_summary_text());
 		
 	}
 
-	public function testcreate_list_query(){
+	public function testcreate_list_query() {
 
 		$opportunity = new Opportunity();
 		
 		//test with empty string params
 		$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY opportunities.name";
-		$actual = $opportunity->create_list_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $opportunity->create_list_query('', '');
+		$this->assertSame($expected, $actual);
 		
 			
 		//test with valid string params
 		$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where (accounts.name=\"\") AND \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY accounts.id";
-		$actual = $opportunity->create_list_query('accounts.id','accounts.name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $opportunity->create_list_query('accounts.id', 'accounts.name=""');
+		$this->assertSame($expected, $actual);
 			
 	}
 	
 
-    public function testcreate_export_query(){
+    public function testcreate_export_query() {
 
     	$opportunity = new Opportunity();
     	
     	//test with empty string params
     	$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY opportunities.name";
-    	$actual = $opportunity->create_list_query('','');
-    	$this->assertSame($expected,$actual);
+    	$actual = $opportunity->create_list_query('', '');
+    	$this->assertSame($expected, $actual);
 
     	
     	//test with valid string params
     	$expected = "SELECT \n                            accounts.id as account_id,\n                            accounts.name as account_name,\n                            accounts.assigned_user_id account_id_owner,\n                            users.user_name as assigned_user_name  ,opportunities.*\n                            FROM opportunities LEFT JOIN users\n                            ON opportunities.assigned_user_id=users.id LEFT JOIN accounts_opportunities\n                            ON opportunities.id=accounts_opportunities.opportunity_id\n                            LEFT JOIN accounts\n                            ON accounts_opportunities.account_id=accounts.id where (accounts.name=\"\") AND \n			(accounts_opportunities.deleted is null OR accounts_opportunities.deleted=0)\n			AND (accounts.deleted is null OR accounts.deleted=0)\n			AND opportunities.deleted=0 ORDER BY accounts.id";
-    	$actual = $opportunity->create_list_query('accounts.id','accounts.name=""');
-    	$this->assertSame($expected,$actual);
+    	$actual = $opportunity->create_list_query('accounts.id', 'accounts.name=""');
+    	$this->assertSame($expected, $actual);
     	
     }
 
-	public function testfill_in_additional_list_fields(){
+	public function testfill_in_additional_list_fields() {
 
 		$opportunity = new Opportunity();
 		
@@ -98,7 +98,7 @@ 
 		
 	}
 
-	public function testfill_in_additional_detail_fields(){
+	public function testfill_in_additional_detail_fields() {
 
 		$opportunity = new Opportunity();
 		
@@ -115,7 +115,7 @@ 
 	}
 
 
-	public function testget_contacts(){
+	public function testget_contacts() {
 
 		$opportunity = new Opportunity();
 		
@@ -124,14 +124,14 @@ 
 		
 	}
 
-	public function testupdate_currency_id(){
+	public function testupdate_currency_id() {
 
 		$opportunity = new Opportunity();
 
 		//execute the method and test if it works and does not throws an exception.
 		try {
 		
-			$opportunity->update_currency_id(array('GBP','EUR'), 'USD');
+			$opportunity->update_currency_id(array('GBP', 'EUR'), 'USD');
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -141,13 +141,13 @@ 
 		
 	}
 
-	public function testget_list_view_data(){
+	public function testget_list_view_data() {
 
 		$opportunity = new Opportunity();
 		
-		$opportunity->name  = "test";
+		$opportunity->name = "test";
 				
-		$expected = array (
+		$expected = array(
 				'NAME' => 'test',
 				'DELETED' => 0,
 				'SALES_STAGE' => '',
@@ -164,7 +164,7 @@ 
 		
 	}
 
-    public function testget_currency_symbol(){
+    public function testget_currency_symbol() {
 
     	$opportunity = new Opportunity();
     	
@@ -179,23 +179,23 @@ 
     }
 
 
-	public function testbuild_generic_where_clause () {
+	public function testbuild_generic_where_clause() {
 
 		$opportunity = new Opportunity();
 
 		//test with empty string params
 		$expected = "opportunities.name like '%' or accounts.name like '%'";
 		$actual = $opportunity->build_generic_where_clause('');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
-	public function testsave(){
+	public function testsave() {
  
 		$opportunity = new Opportunity();
 		
 		$opportunity->name = "test";
-		$opportunity->description ="test description";
+		$opportunity->description = "test description";
 		$opportunity->sales_stage = "Value Proposition";
 		
 		$result = $opportunity->save();
@@ -210,11 +210,11 @@ 
 		//mark the record as deleted and verify that this record cannot be retrieved anymore.
 		$opportunity->mark_deleted($opportunity->id);
 		$result = $opportunity->retrieve($opportunity->id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 
-	public function testsave_relationship_changes(){
+	public function testsave_relationship_changes() {
 
 		$opportunity = new Opportunity();
 		$opportunity->account_id = 1;
@@ -229,7 +229,7 @@ 
 		
 	}
 
-	public function testset_opportunity_contact_relationship(){
+	public function testset_opportunity_contact_relationship() {
 
 		$opportunity = new Opportunity();
 		
@@ -243,7 +243,7 @@ 
 		
 	}
 
-	public function testset_notification_body(){
+	public function testset_notification_body() {
 
 
 		$opportunity = new Opportunity();
@@ -251,21 +251,21 @@ 
 		//test with attributes preset and verify template variables are set accordingly
 		
 		$opportunity->name = "test";
-		$opportunity->amount ="100";
+		$opportunity->amount = "100";
 		$opportunity->date_closed = "2015-02-11 17:30:00";
 		$opportunity->description = "tes description";
 		$opportunity->sales_stage = "Value Proposition";
 		
 		$result = $opportunity->set_notification_body(new Sugar_Smarty(), $opportunity);
 		
-		$this->assertEquals($opportunity->name ,$result->_tpl_vars['OPPORTUNITY_NAME']);
-		$this->assertEquals($opportunity->amount  ,$result->_tpl_vars['OPPORTUNITY_AMOUNT']);
-		$this->assertEquals($opportunity->date_closed ,$result->_tpl_vars['OPPORTUNITY_CLOSEDATE']);
-		$this->assertEquals($opportunity->sales_stage ,$result->_tpl_vars['OPPORTUNITY_STAGE']);
-		$this->assertEquals($opportunity->description ,$result->_tpl_vars['OPPORTUNITY_DESCRIPTION']);
+		$this->assertEquals($opportunity->name, $result->_tpl_vars['OPPORTUNITY_NAME']);
+		$this->assertEquals($opportunity->amount, $result->_tpl_vars['OPPORTUNITY_AMOUNT']);
+		$this->assertEquals($opportunity->date_closed, $result->_tpl_vars['OPPORTUNITY_CLOSEDATE']);
+		$this->assertEquals($opportunity->sales_stage, $result->_tpl_vars['OPPORTUNITY_STAGE']);
+		$this->assertEquals($opportunity->description, $result->_tpl_vars['OPPORTUNITY_DESCRIPTION']);
 	}
 
-	public function testbean_implements(){
+	public function testbean_implements() {
 
 		$opportunity = new Opportunity();
 		
@@ -275,13 +275,13 @@ 
 		
 	}
 
-	public function testlistviewACLHelper(){
+	public function testlistviewACLHelper() {
 
 		$opportunity = new Opportunity();
 		
-		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a" );
+		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a");
 		$actual = $opportunity->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 
@@ -295,7 +295,7 @@ 
 		
 	}
 	
-	public function testgetCurrencyType(){
+	public function testgetCurrencyType() {
 	
 		//execute the method and test if it works and does not throws an exception.
 		try {

Braces +6 added lines, -12 removed lines

@@ -91,8 +91,7 @@ 
 			$opportunity->fill_in_additional_list_fields();
 			
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -107,8 +106,7 @@ 
 				
 			$opportunity->fill_in_additional_detail_fields();				
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -133,8 +131,7 @@ 
 		
 			$opportunity->update_currency_id(array('GBP','EUR'), 'USD');
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -222,8 +219,7 @@ 
 		try {
 			$opportunity->save_relationship_changes(true);
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -236,8 +232,7 @@ 
 		try {
 			$opportunity->set_opportunity_contact_relationship("1");
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		
@@ -301,8 +296,7 @@ 
 		try {
 			getCurrencyType();
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		

tests/tests/modules/CampaignTrackers/CampaignTrackerTest.php

Indentation +34 added lines, -34 removed lines

@@ -6,39 +6,39 @@ 
 
     public function testCampaignTracker() 
     {
-    	error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
     	
-    	//execute the contructor and check for the Object type and  attributes
-    	$campaignTracker = new CampaignTracker();
-    	$this->assertInstanceOf('CampaignTracker',$campaignTracker);
-    	$this->assertInstanceOf('SugarBean',$campaignTracker);
+        //execute the contructor and check for the Object type and  attributes
+        $campaignTracker = new CampaignTracker();
+        $this->assertInstanceOf('CampaignTracker',$campaignTracker);
+        $this->assertInstanceOf('SugarBean',$campaignTracker);
     	
-    	$this->assertAttributeEquals('CampaignTrackers', 'module_dir', $campaignTracker);
-    	$this->assertAttributeEquals('CampaignTracker', 'object_name', $campaignTracker);
-    	$this->assertAttributeEquals('campaign_trkrs', 'table_name', $campaignTracker);
-    	$this->assertAttributeEquals(true, 'new_schema', $campaignTracker);
+        $this->assertAttributeEquals('CampaignTrackers', 'module_dir', $campaignTracker);
+        $this->assertAttributeEquals('CampaignTracker', 'object_name', $campaignTracker);
+        $this->assertAttributeEquals('campaign_trkrs', 'table_name', $campaignTracker);
+        $this->assertAttributeEquals(true, 'new_schema', $campaignTracker);
     	
     }
 
     public function testsave() 
     {
 
-    	$campaignTracker = new CampaignTracker();
+        $campaignTracker = new CampaignTracker();
 
-    	$campaignTracker->tracker_name = "test";
-    	$campaignTracker->is_optout = 1;
+        $campaignTracker->tracker_name = "test";
+        $campaignTracker->is_optout = 1;
     	
-    	$campaignTracker->save();
+        $campaignTracker->save();
     		
-    	//test for record ID to verify that record is saved
-    	$this->assertTrue(isset($campaignTracker->id));
-    	$this->assertEquals(36, strlen($campaignTracker->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($campaignTracker->id));
+        $this->assertEquals(36, strlen($campaignTracker->id));
     		
     		
-    	//mark the record as deleted and verify that this record cannot be retrieved anymore.
-    	$campaignTracker->mark_deleted($campaignTracker->id);
-    	$result = $campaignTracker->retrieve($campaignTracker->id);
-    	$this->assertEquals(null,$result);
+        //mark the record as deleted and verify that this record cannot be retrieved anymore.
+        $campaignTracker->mark_deleted($campaignTracker->id);
+        $result = $campaignTracker->retrieve($campaignTracker->id);
+        $this->assertEquals(null,$result);
     	
     }
 
@@ -46,30 +46,30 @@ 
     public function testget_summary_text()
     {
     	 
-    	$campaignTracker = new CampaignTracker();
+        $campaignTracker = new CampaignTracker();
     	
-    	//test without setting name
-    	$this->assertEquals(Null,$campaignTracker->get_summary_text());
+        //test without setting name
+        $this->assertEquals(Null,$campaignTracker->get_summary_text());
 
-    	//test with name set
-    	$campaignTracker->tracker_name = "test";
-    	$this->assertEquals('test',$campaignTracker->get_summary_text());
+        //test with name set
+        $campaignTracker->tracker_name = "test";
+        $this->assertEquals('test',$campaignTracker->get_summary_text());
     	 
     }
 
 
     public function testfill_in_additional_detail_fields() 
     {
-    	$campaignTracker = new CampaignTracker();
+        $campaignTracker = new CampaignTracker();
     	
-    	//test without is_optout set
-    	$campaignTracker->fill_in_additional_detail_fields();
-    	$this->assertStringEndsWith("/index.php?entryPoint=campaign_trackerv2&track=", $campaignTracker->message_url);
+        //test without is_optout set
+        $campaignTracker->fill_in_additional_detail_fields();
+        $this->assertStringEndsWith("/index.php?entryPoint=campaign_trackerv2&track=", $campaignTracker->message_url);
     	
-    	//test with is_optout set
-    	$campaignTracker->is_optout = 1;
-    	$campaignTracker->fill_in_additional_detail_fields();
-    	$this->assertStringEndsWith("/index.php?entryPoint=removeme&identifier={MESSAGE_ID}", $campaignTracker->message_url);
+        //test with is_optout set
+        $campaignTracker->is_optout = 1;
+        $campaignTracker->fill_in_additional_detail_fields();
+        $this->assertStringEndsWith("/index.php?entryPoint=removeme&identifier={MESSAGE_ID}", $campaignTracker->message_url);
     
     }
     

Spacing +6 added lines, -6 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 
-class CampaignTrackerTest extends PHPUnit_Framework_TestCase  {
+class CampaignTrackerTest extends PHPUnit_Framework_TestCase {
 
 
     public function testCampaignTracker() 
@@ -10,8 +10,8 @@ 
     	
     	//execute the contructor and check for the Object type and  attributes
     	$campaignTracker = new CampaignTracker();
-    	$this->assertInstanceOf('CampaignTracker',$campaignTracker);
-    	$this->assertInstanceOf('SugarBean',$campaignTracker);
+    	$this->assertInstanceOf('CampaignTracker', $campaignTracker);
+    	$this->assertInstanceOf('SugarBean', $campaignTracker);
     	
     	$this->assertAttributeEquals('CampaignTrackers', 'module_dir', $campaignTracker);
     	$this->assertAttributeEquals('CampaignTracker', 'object_name', $campaignTracker);
@@ -38,7 +38,7 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$campaignTracker->mark_deleted($campaignTracker->id);
     	$result = $campaignTracker->retrieve($campaignTracker->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
     	
     }
 
@@ -49,11 +49,11 @@ 
     	$campaignTracker = new CampaignTracker();
     	
     	//test without setting name
-    	$this->assertEquals(Null,$campaignTracker->get_summary_text());
+    	$this->assertEquals(Null, $campaignTracker->get_summary_text());
 
     	//test with name set
     	$campaignTracker->tracker_name = "test";
-    	$this->assertEquals('test',$campaignTracker->get_summary_text());
+    	$this->assertEquals('test', $campaignTracker->get_summary_text());
     	 
     }
 

tests/tests/modules/SavedSearch/SavedSearchTest.php

Indentation +110 added lines, -110 removed lines

@@ -3,184 +3,184 @@
 class SavedSearchTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testSavedSearch() {
+    public function testSavedSearch() {
 
-		//execute the contructor and check for the Object type and  attributes
-		$savedSearch = new SavedSearch();
+        //execute the contructor and check for the Object type and  attributes
+        $savedSearch = new SavedSearch();
 		
-		$this->assertInstanceOf('SavedSearch',$savedSearch);
-		$this->assertInstanceOf('SugarBean',$savedSearch);
+        $this->assertInstanceOf('SavedSearch',$savedSearch);
+        $this->assertInstanceOf('SugarBean',$savedSearch);
 			
-		$this->assertAttributeEquals('saved_search', 'table_name', $savedSearch);
-		$this->assertAttributeEquals('SavedSearch', 'module_dir', $savedSearch);
-		$this->assertAttributeEquals('SavedSearch', 'object_name', $savedSearch);
+        $this->assertAttributeEquals('saved_search', 'table_name', $savedSearch);
+        $this->assertAttributeEquals('SavedSearch', 'module_dir', $savedSearch);
+        $this->assertAttributeEquals('SavedSearch', 'object_name', $savedSearch);
 		
 		
-		//test with parameters
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+        //test with parameters
+        $savedSearch = new SavedSearch(array('id','name'),'id','ASC');
 		
-		$this->assertAttributeEquals(array('id','name'), 'columns', $savedSearch);
-		$this->assertAttributeEquals('id', 'orderBy', $savedSearch);
-		$this->assertAttributeEquals('ASC', 'sortOrder', $savedSearch);
+        $this->assertAttributeEquals(array('id','name'), 'columns', $savedSearch);
+        $this->assertAttributeEquals('id', 'orderBy', $savedSearch);
+        $this->assertAttributeEquals('ASC', 'sortOrder', $savedSearch);
 		
-	}
+    }
 
 
-	public function testgetForm() {
+    public function testgetForm() {
 
-		error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 		
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
-		$result = $savedSearch->getForm("Leads");
+        $savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+        $result = $savedSearch->getForm("Leads");
 		
-		$this->assertGreaterThan(0,strlen($result));
+        $this->assertGreaterThan(0,strlen($result));
 		
-	}
+    }
 
-	public function testgetSelect() {
+    public function testgetSelect() {
 
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
-		$result = $savedSearch->getSelect("Leads");
+        $savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+        $result = $savedSearch->getSelect("Leads");
 				
-		$this->assertGreaterThan(0,strlen($result));
+        $this->assertGreaterThan(0,strlen($result));
 		
-	}
+    }
 
-	public function testMain() {
+    public function testMain() {
 	
-		$savedSearch = new SavedSearch();
+        $savedSearch = new SavedSearch();
 		
-		$savedSearch->name = "test";
-		$savedSearch->search_module = "Leads";
-		$savedSearch->save();
+        $savedSearch->name = "test";
+        $savedSearch->search_module = "Leads";
+        $savedSearch->save();
 		
-		//test for record ID to verify that record is saved
-		$this->assertTrue(isset($savedSearch->id));
-		$this->assertEquals(36, strlen($savedSearch->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($savedSearch->id));
+        $this->assertEquals(36, strlen($savedSearch->id));
 		
 		
-		//test handleSave method
-		$this->handleSaveAndRetrieveSavedSearch($savedSearch->id);
+        //test handleSave method
+        $this->handleSaveAndRetrieveSavedSearch($savedSearch->id);
 		
 		
-		//test returnSavedSearch method
-		$this->returnSavedSearch($savedSearch->id);
+        //test returnSavedSearch method
+        $this->returnSavedSearch($savedSearch->id);
 		
 		
-		//test returnSavedSearchContents method
-		$this->returnSavedSearchContents($savedSearch->id);
+        //test returnSavedSearchContents method
+        $this->returnSavedSearchContents($savedSearch->id);
 		
 		
-		//test handleDelete method
-		$this->handleDelete($savedSearch->id);
+        //test handleDelete method
+        $this->handleDelete($savedSearch->id);
 		
-	}
+    }
 
-	public function handleSaveAndRetrieveSavedSearch($id) {
+    public function handleSaveAndRetrieveSavedSearch($id) {
 		
-		$savedSearch = new SavedSearch();
-		$searchModuleBean = new Lead();
+        $savedSearch = new SavedSearch();
+        $searchModuleBean = new Lead();
 		
-		$_REQUEST['search_module'] = 'Leads';
-		$_REQUEST['description'] = 'test description';
-		$_REQUEST['test_content'] = 'test text';
+        $_REQUEST['search_module'] = 'Leads';
+        $_REQUEST['description'] = 'test description';
+        $_REQUEST['test_content'] = 'test text';
 		
-		$expected = array( "search_module"=> "Leads", "description"=>"test description", "test_content"=>"test text", "advanced"=>true);
+        $expected = array( "search_module"=> "Leads", "description"=>"test description", "test_content"=>"test text", "advanced"=>true);
 		
-		//execute the method and then retrieve back to verify contents attribute
-		$savedSearch->handleSave('', false, false, $id, $searchModuleBean);
-		$savedSearch->retrieveSavedSearch($id);
-		$this->assertSame($expected, $savedSearch->contents);
+        //execute the method and then retrieve back to verify contents attribute
+        $savedSearch->handleSave('', false, false, $id, $searchModuleBean);
+        $savedSearch->retrieveSavedSearch($id);
+        $this->assertSame($expected, $savedSearch->contents);
 		
-	}
+    }
 
-	public function handleDelete($id) {
+    public function handleDelete($id) {
 	
-		$savedSearch = new SavedSearch();
+        $savedSearch = new SavedSearch();
 
-		$savedSearch->handleDelete($id);
+        $savedSearch->handleDelete($id);
 
-		$result = $savedSearch->retrieve($id);
-		$this->assertEquals(null,$result);
+        $result = $savedSearch->retrieve($id);
+        $this->assertEquals(null,$result);
 		
-	}
+    }
 	
-	public function returnSavedSearch($id) {
-		
-		$savedSearch = new SavedSearch();
-		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$savedSearch->returnSavedSearch($id);
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+    public function returnSavedSearch($id) {
+		
+        $savedSearch = new SavedSearch();
+		
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $savedSearch->returnSavedSearch($id);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 	
 		
-	}
+    }
 
-	public function returnSavedSearchContents($id) {
+    public function returnSavedSearchContents($id) {
 
-		$savedSearch = new SavedSearch();
+        $savedSearch = new SavedSearch();
 		
-		//execute the method and test if it works and does not throws an exception.
-		try {
-			$result = $savedSearch->returnSavedSearchContents($id);
-			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
-			$this->fail();
-		}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $result = $savedSearch->returnSavedSearchContents($id);
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 		
-	}
+    }
 
 
-	public function testhandleRedirect() {
+    public function testhandleRedirect() {
 
-		$savedSearch = new SavedSearch();
+        $savedSearch = new SavedSearch();
 		
-		$search_query = "&orderBy=&sortOrder=&query=&searchFormTab=&showSSDIV=";
+        $search_query = "&orderBy=&sortOrder=&query=&searchFormTab=&showSSDIV=";
 		
-		//$savedSearch->handleRedirect("Leads", $search_query, 1, 'true');
-		$this->markTestIncomplete('method uses die');
+        //$savedSearch->handleRedirect("Leads", $search_query, 1, 'true');
+        $this->markTestIncomplete('method uses die');
 		
-	}
+    }
 
-	public function testfill_in_additional_list_fields() {
+    public function testfill_in_additional_list_fields() {
 
-		$savedSearch = new SavedSearch();
+        $savedSearch = new SavedSearch();
 		
-		$savedSearch->assigned_user_id = 1;
-		$savedSearch->contents = array('search_module'=>'Leads');
+        $savedSearch->assigned_user_id = 1;
+        $savedSearch->contents = array('search_module'=>'Leads');
 		
-		$savedSearch->fill_in_additional_list_fields();
+        $savedSearch->fill_in_additional_list_fields();
 		
-		$this->assertEquals('Leads', $savedSearch->search_module);
-		$this->assertEquals('Administrator', $savedSearch->assigned_user_name);
+        $this->assertEquals('Leads', $savedSearch->search_module);
+        $this->assertEquals('Administrator', $savedSearch->assigned_user_name);
 		
-	}
+    }
 
 
-	public function testpopulateRequest(){
+    public function testpopulateRequest(){
 
-		$savedSearch = new SavedSearch();
+        $savedSearch = new SavedSearch();
 		
-		$savedSearch->contents = array( "search_module"=> "Accounts", 
-										"description"=>"test text", 
-										"test_content"=>"some content", 
-										"advanced"=>true);
+        $savedSearch->contents = array( "search_module"=> "Accounts", 
+                                        "description"=>"test text", 
+                                        "test_content"=>"some content", 
+                                        "advanced"=>true);
 		
-		$savedSearch->populateRequest();
+        $savedSearch->populateRequest();
 
-		//verify thhat Request parameters are set
-		$this->assertEquals('Accounts', $_REQUEST['search_module']); 
-		$this->assertEquals('test text',  $_REQUEST['description']); 
-		$this->assertEquals('some content', $_REQUEST['test_content']); 
+        //verify thhat Request parameters are set
+        $this->assertEquals('Accounts', $_REQUEST['search_module']); 
+        $this->assertEquals('test text',  $_REQUEST['description']); 
+        $this->assertEquals('some content', $_REQUEST['test_content']); 
 		
 		
-	}
+    }
 
 }
 

Spacing +13 added lines, -13 removed lines

@@ -8,8 +8,8 @@ 
 		//execute the contructor and check for the Object type and  attributes
 		$savedSearch = new SavedSearch();
 		
-		$this->assertInstanceOf('SavedSearch',$savedSearch);
-		$this->assertInstanceOf('SugarBean',$savedSearch);
+		$this->assertInstanceOf('SavedSearch', $savedSearch);
+		$this->assertInstanceOf('SugarBean', $savedSearch);
 			
 		$this->assertAttributeEquals('saved_search', 'table_name', $savedSearch);
 		$this->assertAttributeEquals('SavedSearch', 'module_dir', $savedSearch);
@@ -17,9 +17,9 @@ 
 		
 		
 		//test with parameters
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+		$savedSearch = new SavedSearch(array('id', 'name'), 'id', 'ASC');
 		
-		$this->assertAttributeEquals(array('id','name'), 'columns', $savedSearch);
+		$this->assertAttributeEquals(array('id', 'name'), 'columns', $savedSearch);
 		$this->assertAttributeEquals('id', 'orderBy', $savedSearch);
 		$this->assertAttributeEquals('ASC', 'sortOrder', $savedSearch);
 		
@@ -30,19 +30,19 @@ 
 
 		error_reporting(E_ERROR | E_PARSE);
 		
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+		$savedSearch = new SavedSearch(array('id', 'name'), 'id', 'ASC');
 		$result = $savedSearch->getForm("Leads");
 		
-		$this->assertGreaterThan(0,strlen($result));
+		$this->assertGreaterThan(0, strlen($result));
 		
 	}
 
 	public function testgetSelect() {
 
-		$savedSearch = new SavedSearch(array('id','name'),'id','ASC');
+		$savedSearch = new SavedSearch(array('id', 'name'), 'id', 'ASC');
 		$result = $savedSearch->getSelect("Leads");
 				
-		$this->assertGreaterThan(0,strlen($result));
+		$this->assertGreaterThan(0, strlen($result));
 		
 	}
 
@@ -85,7 +85,7 @@ 
 		$_REQUEST['description'] = 'test description';
 		$_REQUEST['test_content'] = 'test text';
 		
-		$expected = array( "search_module"=> "Leads", "description"=>"test description", "test_content"=>"test text", "advanced"=>true);
+		$expected = array("search_module"=> "Leads", "description"=>"test description", "test_content"=>"test text", "advanced"=>true);
 		
 		//execute the method and then retrieve back to verify contents attribute
 		$savedSearch->handleSave('', false, false, $id, $searchModuleBean);
@@ -101,7 +101,7 @@ 
 		$savedSearch->handleDelete($id);
 
 		$result = $savedSearch->retrieve($id);
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 		
 	}
 	
@@ -163,11 +163,11 @@ 
 	}
 
 
-	public function testpopulateRequest(){
+	public function testpopulateRequest() {
 
 		$savedSearch = new SavedSearch();
 		
-		$savedSearch->contents = array( "search_module"=> "Accounts", 
+		$savedSearch->contents = array("search_module"=> "Accounts", 
 										"description"=>"test text", 
 										"test_content"=>"some content", 
 										"advanced"=>true);
@@ -176,7 +176,7 @@ 
 
 		//verify thhat Request parameters are set
 		$this->assertEquals('Accounts', $_REQUEST['search_module']); 
-		$this->assertEquals('test text',  $_REQUEST['description']); 
+		$this->assertEquals('test text', $_REQUEST['description']); 
 		$this->assertEquals('some content', $_REQUEST['test_content']); 
 		
 		

Braces +2 added lines, -4 removed lines

@@ -113,8 +113,7 @@ 
 		try {
 			$savedSearch->returnSavedSearch($id);
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 	
@@ -129,8 +128,7 @@ 
 		try {
 			$result = $savedSearch->returnSavedSearchContents($id);
 			$this->assertTrue(true);
-		}
-		catch (Exception $e) {
+		} catch (Exception $e) {
 			$this->fail();
 		}
 		

tests/tests/modules/EmailText/EmailTextTest.php

Indentation +12 added lines, -12 removed lines

@@ -3,22 +3,22 @@
 class EmailTextTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testEmailText() {
+    public function testEmailText() {
 	
-		//execute the contructor and check for the Object type and  attributes
-		$emailText = new EmailText();
+        //execute the contructor and check for the Object type and  attributes
+        $emailText = new EmailText();
 	
-		$this->assertInstanceOf('EmailText',$emailText);
-		$this->assertInstanceOf('SugarBean',$emailText);
+        $this->assertInstanceOf('EmailText',$emailText);
+        $this->assertInstanceOf('SugarBean',$emailText);
 			
-		$this->assertAttributeEquals('EmailText', 'module_dir', $emailText);
-		$this->assertAttributeEquals('EmailText', 'module_name', $emailText);
-		$this->assertAttributeEquals('EmailText', 'object_name', $emailText);
-		$this->assertAttributeEquals('emails_text', 'table_name', $emailText);
+        $this->assertAttributeEquals('EmailText', 'module_dir', $emailText);
+        $this->assertAttributeEquals('EmailText', 'module_name', $emailText);
+        $this->assertAttributeEquals('EmailText', 'object_name', $emailText);
+        $this->assertAttributeEquals('emails_text', 'table_name', $emailText);
 	
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $emailText);
-		$this->assertAttributeEquals(true, 'disable_custom_fields', $emailText);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $emailText);
+        $this->assertAttributeEquals(true, 'disable_custom_fields', $emailText);
 	
-	}
+    }
 	
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 		//execute the contructor and check for the Object type and  attributes
 		$emailText = new EmailText();
 	
-		$this->assertInstanceOf('EmailText',$emailText);
-		$this->assertInstanceOf('SugarBean',$emailText);
+		$this->assertInstanceOf('EmailText', $emailText);
+		$this->assertInstanceOf('SugarBean', $emailText);
 			
 		$this->assertAttributeEquals('EmailText', 'module_dir', $emailText);
 		$this->assertAttributeEquals('EmailText', 'module_name', $emailText);

tests/tests/modules/AOW_WorkFlow/AOW_WorkFlowTest.php

Indentation +191 added lines, -191 removed lines

@@ -3,330 +3,330 @@
 class AOW_WorkFlowTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testAOW_WorkFlow(){
+    public function testAOW_WorkFlow(){
 
-		//execute the contructor and check for the Object type and  attributes
-		$aowWorkFlow = new AOW_WorkFlow();
-		$this->assertInstanceOf('AOW_WorkFlow',$aowWorkFlow);
-		$this->assertInstanceOf('Basic',$aowWorkFlow);
-		$this->assertInstanceOf('SugarBean',$aowWorkFlow);
+        //execute the contructor and check for the Object type and  attributes
+        $aowWorkFlow = new AOW_WorkFlow();
+        $this->assertInstanceOf('AOW_WorkFlow',$aowWorkFlow);
+        $this->assertInstanceOf('Basic',$aowWorkFlow);
+        $this->assertInstanceOf('SugarBean',$aowWorkFlow);
 
-		$this->assertAttributeEquals('AOW_WorkFlow', 'module_dir', $aowWorkFlow);
-		$this->assertAttributeEquals('AOW_WorkFlow', 'object_name', $aowWorkFlow);
-		$this->assertAttributeEquals('aow_workflow', 'table_name', $aowWorkFlow);
-		$this->assertAttributeEquals(true, 'new_schema', $aowWorkFlow);
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $aowWorkFlow);
-		$this->assertAttributeEquals(false, 'importable', $aowWorkFlow);
+        $this->assertAttributeEquals('AOW_WorkFlow', 'module_dir', $aowWorkFlow);
+        $this->assertAttributeEquals('AOW_WorkFlow', 'object_name', $aowWorkFlow);
+        $this->assertAttributeEquals('aow_workflow', 'table_name', $aowWorkFlow);
+        $this->assertAttributeEquals(true, 'new_schema', $aowWorkFlow);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $aowWorkFlow);
+        $this->assertAttributeEquals(false, 'importable', $aowWorkFlow);
 
-	}
+    }
 
-	public function testbean_implements(){
+    public function testbean_implements(){
 
-		error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 
-		$aowWorkFlow = new AOW_WorkFlow();
-		$this->assertEquals(false, $aowWorkFlow->bean_implements('')); //test with blank value
-		$this->assertEquals(false, $aowWorkFlow->bean_implements('test')); //test with invalid value
-		$this->assertEquals(true, $aowWorkFlow->bean_implements('ACL')); //test with valid value
+        $aowWorkFlow = new AOW_WorkFlow();
+        $this->assertEquals(false, $aowWorkFlow->bean_implements('')); //test with blank value
+        $this->assertEquals(false, $aowWorkFlow->bean_implements('test')); //test with invalid value
+        $this->assertEquals(true, $aowWorkFlow->bean_implements('ACL')); //test with valid value
 
-	}
+    }
 
     public function testsave(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	$aowWorkFlow->name = "test";
-    	$aowWorkFlow->flow_module = 'test';
+        $aowWorkFlow->name = "test";
+        $aowWorkFlow->flow_module = 'test';
 
-    	$aowWorkFlow->save();
+        $aowWorkFlow->save();
 
-    	//test for record ID to verify that record is saved
-    	$this->assertTrue(isset($aowWorkFlow->id));
-    	$this->assertEquals(36, strlen($aowWorkFlow->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($aowWorkFlow->id));
+        $this->assertEquals(36, strlen($aowWorkFlow->id));
 
 
-    	//mark the record as deleted and verify that this record cannot be retrieved anymore.
-    	$aowWorkFlow->mark_deleted($aowWorkFlow->id);
-    	$result = $aowWorkFlow->retrieve($aowWorkFlow->id);
-    	$this->assertEquals(null,$result);
+        //mark the record as deleted and verify that this record cannot be retrieved anymore.
+        $aowWorkFlow->mark_deleted($aowWorkFlow->id);
+        $result = $aowWorkFlow->retrieve($aowWorkFlow->id);
+        $this->assertEquals(null,$result);
 
     }
 
     public function testload_flow_beans(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//execute the method and test if it works and does not throws an exception.
-    	try {
-    		$aowWorkFlow->load_flow_beans();
-    		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
-    		$this->fail();
-    	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $aowWorkFlow->load_flow_beans();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
     }
 
 
-	public function testrun_flows(){
+    public function testrun_flows(){
 
-		$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-		$result = $aowWorkFlow->run_flows();
-		$this->assertTrue($result);
+        $result = $aowWorkFlow->run_flows();
+        $this->assertTrue($result);
 
-	}
+    }
 
 
     public function testrun_flow(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//execute the method and test if it works and does not throws an exception.
-    	try {
-    		$aowWorkFlow->run_flow();
-    		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
-    		$this->fail();
-    	}
+        //execute the method and test if it works and does not throws an exception.
+        try {
+            $aowWorkFlow->run_flow();
+            $this->assertTrue(true);
+        }
+        catch (Exception $e) {
+            $this->fail();
+        }
 
     }
 
 
     public function testrun_bean_flows(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//test with different modules. it always returns true
+        //test with different modules. it always returns true
 
-    	$result = $aowWorkFlow->run_bean_flows(new AOS_Quotes());
-    	$this->assertTrue($result);
+        $result = $aowWorkFlow->run_bean_flows(new AOS_Quotes());
+        $this->assertTrue($result);
 
-    	$result = $aowWorkFlow->run_bean_flows(new Call());
-    	$this->assertTrue($result);
+        $result = $aowWorkFlow->run_bean_flows(new Call());
+        $this->assertTrue($result);
 
     }
 
 
     public function testget_flow_beans(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//test for AOS_Quotes. it will return null as no test data is available
-    	$aowWorkFlow->flow_module = "AOS_Quotes";
-    	$result = $aowWorkFlow->get_flow_beans();
-    	$this->assertEquals(null,$result);
+        //test for AOS_Quotes. it will return null as no test data is available
+        $aowWorkFlow->flow_module = "AOS_Quotes";
+        $result = $aowWorkFlow->get_flow_beans();
+        $this->assertEquals(null,$result);
     }
 
     public function testbuild_flow_query_join(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
-    	$query = Array();
+        $aowWorkFlow = new AOW_WorkFlow();
+        $query = Array();
 
 
-    	//test without type param
-    	$result = $aowWorkFlow->build_flow_query_join('aos_products_quotes', new AOS_Quotes(), '', Array());
-    	$this->assertSame(array(), $result);
+        //test without type param
+        $result = $aowWorkFlow->build_flow_query_join('aos_products_quotes', new AOS_Quotes(), '', Array());
+        $this->assertSame(array(), $result);
 
 
-		//test with type custom
-    	$expected = Array('join' => array('c' => 'LEFT JOIN calls_cstm c ON calls.id = c.id_c ' ));
-    	$result = $aowWorkFlow->build_flow_query_join('c', new Call(), 'custom', Array());
-    	$this->assertSame($expected, $result);
+        //test with type custom
+        $expected = Array('join' => array('c' => 'LEFT JOIN calls_cstm c ON calls.id = c.id_c ' ));
+        $result = $aowWorkFlow->build_flow_query_join('c', new Call(), 'custom', Array());
+        $this->assertSame($expected, $result);
 
 
-    	//test with type relationship
-    	$expected = Array (
-    			'join' =>array ("aos_products_quotes" => "LEFT JOIN aos_products_quotes aos_products_quotes ON aos_quotes.id=aos_products_quotes.parent_id AND aos_products_quotes.deleted=0\n\n"),
-    			'select' =>array ("aos_products_quotes.id AS 'aos_products_quotes_id'"),
-    	);
-    	$result = $aowWorkFlow->build_flow_query_join('aos_products_quotes', new AOS_Quotes(), 'relationship', Array());
-    	$this->assertSame($expected, $result);
+        //test with type relationship
+        $expected = Array (
+                'join' =>array ("aos_products_quotes" => "LEFT JOIN aos_products_quotes aos_products_quotes ON aos_quotes.id=aos_products_quotes.parent_id AND aos_products_quotes.deleted=0\n\n"),
+                'select' =>array ("aos_products_quotes.id AS 'aos_products_quotes_id'"),
+        );
+        $result = $aowWorkFlow->build_flow_query_join('aos_products_quotes', new AOS_Quotes(), 'relationship', Array());
+        $this->assertSame($expected, $result);
 
     }
 
     public function testbuild_flow_query_where(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//test without presetting required object attributes
-    	$expected = Array();
-    	$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+        //test without presetting required object attributes
+        $expected = Array();
+        $query = $aowWorkFlow->build_flow_query_where();
+        $this->assertSame($expected,$query);
 
 
 
-		//test with module required attributes set
-		$aowWorkFlow->id = '1';
-		$aowWorkFlow->flow_module = "Calls";
-		$expected = array (
-				'where' => array ('NOT EXISTS (SELECT * FROM aow_processed WHERE aow_processed.aow_workflow_id=\'1\' AND aow_processed.parent_id=calls.id AND aow_processed.status = \'Complete\' AND aow_processed.deleted = 0)',
-				'calls.deleted = 0 ' )
-				);
-		$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+        //test with module required attributes set
+        $aowWorkFlow->id = '1';
+        $aowWorkFlow->flow_module = "Calls";
+        $expected = array (
+                'where' => array ('NOT EXISTS (SELECT * FROM aow_processed WHERE aow_processed.aow_workflow_id=\'1\' AND aow_processed.parent_id=calls.id AND aow_processed.status = \'Complete\' AND aow_processed.deleted = 0)',
+                'calls.deleted = 0 ' )
+                );
+        $query = $aowWorkFlow->build_flow_query_where();
+        $this->assertSame($expected,$query);
 
 
 
-		//test with flow_run_on and multiple_runs attributes set
-		$expected = array (
-						'where' =>	array ('calls.date_entered > \'\'','calls.deleted = 0 '),
-					);
-		$aowWorkFlow->flow_run_on = 'New_Records';
-		$aowWorkFlow->multiple_runs = 1;
-		$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+        //test with flow_run_on and multiple_runs attributes set
+        $expected = array (
+                        'where' =>	array ('calls.date_entered > \'\'','calls.deleted = 0 '),
+                    );
+        $aowWorkFlow->flow_run_on = 'New_Records';
+        $aowWorkFlow->multiple_runs = 1;
+        $query = $aowWorkFlow->build_flow_query_where();
+        $this->assertSame($expected,$query);
 
     }
 
     public function testbuild_query_where(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//populate required values
-    	$call = new Call();
-    	$aowCondition = new AOW_Condition();
-    	$aowCondition->name = "test";
-    	$aowCondition->module_path = base64_encode(serialize(array('')));
-    	$aowCondition->field = "name";
-    	$aowCondition->value = "testval";
+        //populate required values
+        $call = new Call();
+        $aowCondition = new AOW_Condition();
+        $aowCondition->name = "test";
+        $aowCondition->module_path = base64_encode(serialize(array('')));
+        $aowCondition->field = "name";
+        $aowCondition->value = "testval";
 
 
 
-    	//test with contains operator
-    	$aowCondition->operator = "Contains";
-    	$aowCondition->value_type = "Value";
-    	$expected = array(
-    			"where"=> array(".name LIKE CONCAT('%', 'testval' ,'%')")
-    	);
-    	$query = $aowWorkFlow->build_query_where($aowCondition, $call);
-    	$this->assertEquals($expected, $query);
+        //test with contains operator
+        $aowCondition->operator = "Contains";
+        $aowCondition->value_type = "Value";
+        $expected = array(
+                "where"=> array(".name LIKE CONCAT('%', 'testval' ,'%')")
+        );
+        $query = $aowWorkFlow->build_query_where($aowCondition, $call);
+        $this->assertEquals($expected, $query);
 
 
 
-    	//test for starts with operator
-    	$aowCondition->operator = "Starts_With";
-    	$aowCondition->value_type = "Value";
+        //test for starts with operator
+        $aowCondition->operator = "Starts_With";
+        $aowCondition->value_type = "Value";
 
-    	$expected = array(
-    		"where"=> array(".name LIKE CONCAT('testval' ,'%')")
-    	);
-    	$query = $aowWorkFlow->build_query_where($aowCondition, $call);
-    	$this->assertEquals($expected, $query);
+        $expected = array(
+            "where"=> array(".name LIKE CONCAT('testval' ,'%')")
+        );
+        $query = $aowWorkFlow->build_query_where($aowCondition, $call);
+        $this->assertEquals($expected, $query);
 
 
 
-    	//test for Equal_To operator
-    	$aowCondition->operator = "Equal_To";
-    	$aowCondition->value_type = "Value";
+        //test for Equal_To operator
+        $aowCondition->operator = "Equal_To";
+        $aowCondition->value_type = "Value";
 
-    	$expected = array(
-    			"where"=> array(".name = 'testval'")
-    	);
-    	$query = $aowWorkFlow->build_query_where($aowCondition, $call);
-    	$this->assertEquals($expected, $query);
+        $expected = array(
+                "where"=> array(".name = 'testval'")
+        );
+        $query = $aowWorkFlow->build_query_where($aowCondition, $call);
+        $this->assertEquals($expected, $query);
 
 
 
-    	//test with value type Date
-    	$aowCondition->operator = "Equal_To";
-    	$aowCondition->value_type = "Date";
+        //test with value type Date
+        $aowCondition->operator = "Equal_To";
+        $aowCondition->value_type = "Date";
 
-    	$expected = array(
-    			"where"=> array('.name = DATE_ADD(calls., INTERVAL   )')
-    	);
+        $expected = array(
+                "where"=> array('.name = DATE_ADD(calls., INTERVAL   )')
+        );
 
-    	$query = $aowWorkFlow->build_query_where($aowCondition, $call);
-    	$this->assertEquals($expected, $query);
+        $query = $aowWorkFlow->build_query_where($aowCondition, $call);
+        $this->assertEquals($expected, $query);
 
 
 
-    	//test with value type Field
-    	$aowCondition->operator = "Equal_To";
-    	$aowCondition->value_type = "Field";
+        //test with value type Field
+        $aowCondition->operator = "Equal_To";
+        $aowCondition->value_type = "Field";
 
-    	$expected = array(
-    			"where"=> array('.name = calls.testval')
-    	);
+        $expected = array(
+                "where"=> array('.name = calls.testval')
+        );
 
-    	$query = $aowWorkFlow->build_query_where($aowCondition, $call);
-    	$this->assertEquals($expected, $query);
+        $query = $aowWorkFlow->build_query_where($aowCondition, $call);
+        $this->assertEquals($expected, $query);
 
 
     }
 
     public function testcheck_valid_bean(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
-    	$aowWorkFlow->flow_run_on = "New_Records";
+        $aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow->flow_run_on = "New_Records";
 
-    	$aosQuotes = new AOS_Quotes();
+        $aosQuotes = new AOS_Quotes();
 
-    	$result = $aowWorkFlow->check_valid_bean($aosQuotes);
-    	$this->assertTrue($result);
+        $result = $aowWorkFlow->check_valid_bean($aosQuotes);
+        $this->assertTrue($result);
 
     }
 
     public function testcompare_condition(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//execute the method and verify that it returns valid values for all operators
+        //execute the method and verify that it returns valid values for all operators
 
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,1));
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,2,'Not_Equal_To' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(2,1,'Greater_Than' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,2,'Less_Than' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(5,4,'Greater_Than_or_Equal_To' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(2,3,'Less_Than_or_Equal_To' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','test','Contains' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','test','Starts_With' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','1','Ends_With' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('','','is_null' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('test2',array('test1','test2'),'One_of' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('test', array('test1','test2'),'Not_One_of' ));
+        $this->assertTrue($aowWorkFlow->compare_condition(1,1));
+        $this->assertTrue($aowWorkFlow->compare_condition(1,2,'Not_Equal_To' ));
+        $this->assertTrue($aowWorkFlow->compare_condition(2,1,'Greater_Than' ));
+        $this->assertTrue($aowWorkFlow->compare_condition(1,2,'Less_Than' ));
+        $this->assertTrue($aowWorkFlow->compare_condition(5,4,'Greater_Than_or_Equal_To' ));
+        $this->assertTrue($aowWorkFlow->compare_condition(2,3,'Less_Than_or_Equal_To' ));
+        $this->assertNotFalse($aowWorkFlow->compare_condition('test1','test','Contains' ));
+        $this->assertNotFalse($aowWorkFlow->compare_condition('test1','test','Starts_With' ));
+        $this->assertNotFalse($aowWorkFlow->compare_condition('test1','1','Ends_With' ));
+        $this->assertTrue($aowWorkFlow->compare_condition('','','is_null' ));
+        $this->assertTrue($aowWorkFlow->compare_condition('test2',array('test1','test2'),'One_of' ));
+        $this->assertTrue($aowWorkFlow->compare_condition('test', array('test1','test2'),'Not_One_of' ));
 
     }
 
     public function testcheck_in_group(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//test with two different modules
-    	$this->assertFalse($aowWorkFlow->check_in_group(1, "Users", 1));
-    	$this->assertFalse($aowWorkFlow->check_in_group(1, "Calls", 1));
+        //test with two different modules
+        $this->assertFalse($aowWorkFlow->check_in_group(1, "Users", 1));
+        $this->assertFalse($aowWorkFlow->check_in_group(1, "Calls", 1));
 
     }
 
 
     public function testrun_actions(){
 
-    	$aowWorkFlow = new AOW_WorkFlow();
+        $aowWorkFlow = new AOW_WorkFlow();
 
-    	//prepare the required objects and variables
-    	$aowWorkFlow->id = 1;
+        //prepare the required objects and variables
+        $aowWorkFlow->id = 1;
 
-    	$call = new Call();
-    	$call->id = 1;
+        $call = new Call();
+        $call->id = 1;
 
-    	//execute the method and verify if it creates records in processed table
-    	$result = $aowWorkFlow->run_actions($call);
+        //execute the method and verify if it creates records in processed table
+        $result = $aowWorkFlow->run_actions($call);
 
 
-    	//test for a entry in AOW_Processed table.
-    	$processed = new AOW_Processed();
-    	$processed->retrieve_by_string_fields(array('aow_workflow_id' => 1,'parent_id' => 1));
+        //test for a entry in AOW_Processed table.
+        $processed = new AOW_Processed();
+        $processed->retrieve_by_string_fields(array('aow_workflow_id' => 1,'parent_id' => 1));
 
-    	//test for record ID to verify that record is saved
-    	$this->assertTrue(isset($processed->id));
-    	$this->assertEquals(36, strlen($processed->id));
+        //test for record ID to verify that record is saved
+        $this->assertTrue(isset($processed->id));
+        $this->assertEquals(36, strlen($processed->id));
 
 
-    	//mark the record as deleted and verify that this record cannot be retrieved anymore.
-    	$processed->mark_deleted($processed->id);
-    	$result = $processed->retrieve($processed->id);
-    	$this->assertEquals(null,$result);
+        //mark the record as deleted and verify that this record cannot be retrieved anymore.
+        $processed->mark_deleted($processed->id);
+        $result = $processed->retrieve($processed->id);
+        $this->assertEquals(null,$result);
 
     }
 

Spacing +46 added lines, -46 removed lines

@@ -3,13 +3,13 @@ 
 class AOW_WorkFlowTest extends PHPUnit_Framework_TestCase {
 
 
-	public function testAOW_WorkFlow(){
+	public function testAOW_WorkFlow() {
 
 		//execute the contructor and check for the Object type and  attributes
 		$aowWorkFlow = new AOW_WorkFlow();
-		$this->assertInstanceOf('AOW_WorkFlow',$aowWorkFlow);
-		$this->assertInstanceOf('Basic',$aowWorkFlow);
-		$this->assertInstanceOf('SugarBean',$aowWorkFlow);
+		$this->assertInstanceOf('AOW_WorkFlow', $aowWorkFlow);
+		$this->assertInstanceOf('Basic', $aowWorkFlow);
+		$this->assertInstanceOf('SugarBean', $aowWorkFlow);
 
 		$this->assertAttributeEquals('AOW_WorkFlow', 'module_dir', $aowWorkFlow);
 		$this->assertAttributeEquals('AOW_WorkFlow', 'object_name', $aowWorkFlow);
@@ -20,7 +20,7 @@ 
 
 	}
 
-	public function testbean_implements(){
+	public function testbean_implements() {
 
 		error_reporting(E_ERROR | E_PARSE);
 
@@ -31,7 +31,7 @@ 
 
 	}
 
-    public function testsave(){
+    public function testsave() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -48,11 +48,11 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$aowWorkFlow->mark_deleted($aowWorkFlow->id);
     	$result = $aowWorkFlow->retrieve($aowWorkFlow->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
 
     }
 
-    public function testload_flow_beans(){
+    public function testload_flow_beans() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -68,7 +68,7 @@ 
     }
 
 
-	public function testrun_flows(){
+	public function testrun_flows() {
 
 		$aowWorkFlow = new AOW_WorkFlow();
 
@@ -78,7 +78,7 @@ 
 	}
 
 
-    public function testrun_flow(){
+    public function testrun_flow() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -94,7 +94,7 @@ 
     }
 
 
-    public function testrun_bean_flows(){
+    public function testrun_bean_flows() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -109,17 +109,17 @@ 
     }
 
 
-    public function testget_flow_beans(){
+    public function testget_flow_beans() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
     	//test for AOS_Quotes. it will return null as no test data is available
     	$aowWorkFlow->flow_module = "AOS_Quotes";
     	$result = $aowWorkFlow->get_flow_beans();
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
     }
 
-    public function testbuild_flow_query_join(){
+    public function testbuild_flow_query_join() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
     	$query = Array();
@@ -131,56 +131,56 @@ 
 
 
 		//test with type custom
-    	$expected = Array('join' => array('c' => 'LEFT JOIN calls_cstm c ON calls.id = c.id_c ' ));
+    	$expected = Array('join' => array('c' => 'LEFT JOIN calls_cstm c ON calls.id = c.id_c '));
     	$result = $aowWorkFlow->build_flow_query_join('c', new Call(), 'custom', Array());
     	$this->assertSame($expected, $result);
 
 
     	//test with type relationship
-    	$expected = Array (
-    			'join' =>array ("aos_products_quotes" => "LEFT JOIN aos_products_quotes aos_products_quotes ON aos_quotes.id=aos_products_quotes.parent_id AND aos_products_quotes.deleted=0\n\n"),
-    			'select' =>array ("aos_products_quotes.id AS 'aos_products_quotes_id'"),
+    	$expected = Array(
+    			'join' =>array("aos_products_quotes" => "LEFT JOIN aos_products_quotes aos_products_quotes ON aos_quotes.id=aos_products_quotes.parent_id AND aos_products_quotes.deleted=0\n\n"),
+    			'select' =>array("aos_products_quotes.id AS 'aos_products_quotes_id'"),
     	);
     	$result = $aowWorkFlow->build_flow_query_join('aos_products_quotes', new AOS_Quotes(), 'relationship', Array());
     	$this->assertSame($expected, $result);
 
     }
 
-    public function testbuild_flow_query_where(){
+    public function testbuild_flow_query_where() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
     	//test without presetting required object attributes
     	$expected = Array();
     	$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+		$this->assertSame($expected, $query);
 
 
 
 		//test with module required attributes set
 		$aowWorkFlow->id = '1';
 		$aowWorkFlow->flow_module = "Calls";
-		$expected = array (
-				'where' => array ('NOT EXISTS (SELECT * FROM aow_processed WHERE aow_processed.aow_workflow_id=\'1\' AND aow_processed.parent_id=calls.id AND aow_processed.status = \'Complete\' AND aow_processed.deleted = 0)',
-				'calls.deleted = 0 ' )
+		$expected = array(
+				'where' => array('NOT EXISTS (SELECT * FROM aow_processed WHERE aow_processed.aow_workflow_id=\'1\' AND aow_processed.parent_id=calls.id AND aow_processed.status = \'Complete\' AND aow_processed.deleted = 0)',
+				'calls.deleted = 0 ')
 				);
 		$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+		$this->assertSame($expected, $query);
 
 
 
 		//test with flow_run_on and multiple_runs attributes set
-		$expected = array (
-						'where' =>	array ('calls.date_entered > \'\'','calls.deleted = 0 '),
+		$expected = array(
+						'where' =>	array('calls.date_entered > \'\'', 'calls.deleted = 0 '),
 					);
 		$aowWorkFlow->flow_run_on = 'New_Records';
 		$aowWorkFlow->multiple_runs = 1;
 		$query = $aowWorkFlow->build_flow_query_where();
-		$this->assertSame($expected,$query);
+		$this->assertSame($expected, $query);
 
     }
 
-    public function testbuild_query_where(){
+    public function testbuild_query_where() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -256,7 +256,7 @@ 
 
     }
 
-    public function testcheck_valid_bean(){
+    public function testcheck_valid_bean() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
     	$aowWorkFlow->flow_run_on = "New_Records";
@@ -268,28 +268,28 @@ 
 
     }
 
-    public function testcompare_condition(){
+    public function testcompare_condition() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
     	//execute the method and verify that it returns valid values for all operators
 
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,1));
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,2,'Not_Equal_To' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(2,1,'Greater_Than' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(1,2,'Less_Than' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(5,4,'Greater_Than_or_Equal_To' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition(2,3,'Less_Than_or_Equal_To' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','test','Contains' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','test','Starts_With' ));
-    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1','1','Ends_With' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('','','is_null' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('test2',array('test1','test2'),'One_of' ));
-    	$this->assertTrue($aowWorkFlow->compare_condition('test', array('test1','test2'),'Not_One_of' ));
+    	$this->assertTrue($aowWorkFlow->compare_condition(1, 1));
+    	$this->assertTrue($aowWorkFlow->compare_condition(1, 2, 'Not_Equal_To'));
+    	$this->assertTrue($aowWorkFlow->compare_condition(2, 1, 'Greater_Than'));
+    	$this->assertTrue($aowWorkFlow->compare_condition(1, 2, 'Less_Than'));
+    	$this->assertTrue($aowWorkFlow->compare_condition(5, 4, 'Greater_Than_or_Equal_To'));
+    	$this->assertTrue($aowWorkFlow->compare_condition(2, 3, 'Less_Than_or_Equal_To'));
+    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1', 'test', 'Contains'));
+    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1', 'test', 'Starts_With'));
+    	$this->assertNotFalse($aowWorkFlow->compare_condition('test1', '1', 'Ends_With'));
+    	$this->assertTrue($aowWorkFlow->compare_condition('', '', 'is_null'));
+    	$this->assertTrue($aowWorkFlow->compare_condition('test2', array('test1', 'test2'), 'One_of'));
+    	$this->assertTrue($aowWorkFlow->compare_condition('test', array('test1', 'test2'), 'Not_One_of'));
 
     }
 
-    public function testcheck_in_group(){
+    public function testcheck_in_group() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -300,7 +300,7 @@ 
     }
 
 
-    public function testrun_actions(){
+    public function testrun_actions() {
 
     	$aowWorkFlow = new AOW_WorkFlow();
 
@@ -316,7 +316,7 @@ 
 
     	//test for a entry in AOW_Processed table.
     	$processed = new AOW_Processed();
-    	$processed->retrieve_by_string_fields(array('aow_workflow_id' => 1,'parent_id' => 1));
+    	$processed->retrieve_by_string_fields(array('aow_workflow_id' => 1, 'parent_id' => 1));
 
     	//test for record ID to verify that record is saved
     	$this->assertTrue(isset($processed->id));
@@ -326,7 +326,7 @@ 
     	//mark the record as deleted and verify that this record cannot be retrieved anymore.
     	$processed->mark_deleted($processed->id);
     	$result = $processed->retrieve($processed->id);
-    	$this->assertEquals(null,$result);
+    	$this->assertEquals(null, $result);
 
     }
 

Braces +2 added lines, -4 removed lines

@@ -60,8 +60,7 @@ 
     	try {
     		$aowWorkFlow->load_flow_beans();
     		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
+    	} catch (Exception $e) {
     		$this->fail();
     	}
 
@@ -86,8 +85,7 @@ 
     	try {
     		$aowWorkFlow->run_flow();
     		$this->assertTrue(true);
-    	}
-    	catch (Exception $e) {
+    	} catch (Exception $e) {
     		$this->fail();
     	}
 

tests/tests/modules/AOS_PDF_Templates/AOS_PDF_TemplatesTest.php

Indentation +13 added lines, -13 removed lines

@@ -2,23 +2,23 @@
 
 class AOS_PDF_TemplatesTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAOS_PDF_Templates(){	
+    public function testAOS_PDF_Templates(){	
 
 
-		//execute the contructor and check for the Object type and  attributes
-		$aosPdfTemplates = new AOS_PDF_Templates();
-		$this->assertInstanceOf('AOS_PDF_Templates',$aosPdfTemplates);
-		$this->assertInstanceOf('Basic',$aosPdfTemplates);
-		$this->assertInstanceOf('SugarBean',$aosPdfTemplates);
+        //execute the contructor and check for the Object type and  attributes
+        $aosPdfTemplates = new AOS_PDF_Templates();
+        $this->assertInstanceOf('AOS_PDF_Templates',$aosPdfTemplates);
+        $this->assertInstanceOf('Basic',$aosPdfTemplates);
+        $this->assertInstanceOf('SugarBean',$aosPdfTemplates);
 			
-		$this->assertAttributeEquals('AOS_PDF_Templates', 'module_dir', $aosPdfTemplates);
-		$this->assertAttributeEquals('AOS_PDF_Templates', 'object_name', $aosPdfTemplates);
-		$this->assertAttributeEquals('aos_pdf_templates', 'table_name', $aosPdfTemplates);
-		$this->assertAttributeEquals(true, 'new_schema', $aosPdfTemplates);
-		$this->assertAttributeEquals(true, 'disable_row_level_security', $aosPdfTemplates);
-		$this->assertAttributeEquals(true, 'importable', $aosPdfTemplates);
+        $this->assertAttributeEquals('AOS_PDF_Templates', 'module_dir', $aosPdfTemplates);
+        $this->assertAttributeEquals('AOS_PDF_Templates', 'object_name', $aosPdfTemplates);
+        $this->assertAttributeEquals('aos_pdf_templates', 'table_name', $aosPdfTemplates);
+        $this->assertAttributeEquals(true, 'new_schema', $aosPdfTemplates);
+        $this->assertAttributeEquals(true, 'disable_row_level_security', $aosPdfTemplates);
+        $this->assertAttributeEquals(true, 'importable', $aosPdfTemplates);
 		
-	}
+    }
 	
 }
 ?>
\ No newline at end of file

Spacing +4 added lines, -4 removed lines

@@ -2,14 +2,14 @@
 
 class AOS_PDF_TemplatesTest extends PHPUnit_Framework_TestCase {
 	
-	public function testAOS_PDF_Templates(){	
+	public function testAOS_PDF_Templates() {	
 
 
 		//execute the contructor and check for the Object type and  attributes
 		$aosPdfTemplates = new AOS_PDF_Templates();
-		$this->assertInstanceOf('AOS_PDF_Templates',$aosPdfTemplates);
-		$this->assertInstanceOf('Basic',$aosPdfTemplates);
-		$this->assertInstanceOf('SugarBean',$aosPdfTemplates);
+		$this->assertInstanceOf('AOS_PDF_Templates', $aosPdfTemplates);
+		$this->assertInstanceOf('Basic', $aosPdfTemplates);
+		$this->assertInstanceOf('SugarBean', $aosPdfTemplates);
 			
 		$this->assertAttributeEquals('AOS_PDF_Templates', 'module_dir', $aosPdfTemplates);
 		$this->assertAttributeEquals('AOS_PDF_Templates', 'object_name', $aosPdfTemplates);