View GitHub Repository SuiteCRM

Schedule Inspection
Subscribe
Completed
Push — CI ( ee6bd7...0f01dd )
by Adam
22:32
created
Status
Category
Toggle Patches
tests/tests/include/utils/logicUtilsTest.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -18,7 +18,7 @@
 
 block discarded – undo
18 18 
 		$BugsHooks = "";
19 19 
 		try{
20 20 
 			$BugsHooks =  get_hook_array('Bugs');
21 
-		}catch(Exception $e){}
21 
+		} catch(Exception $e){}
22 22
23 23 
 		$this->assertFalse(is_array($BugsHooks));
24 24
Please login to merge, or discard this patch.
tests/tests/include/utils/progressBarUtilsTest.php 1 patch
Braces   +6 added lines, -12 removed lines patch added patch discarded remove patch 
@@ -11,8 +11,7 @@  discard block
 
 block discarded – undo
11 11 
 		try {
12 12 
 			progress_bar_flush();
13 13 
 			$this->assertTrue(true);
14 
-		}
15 
-		catch (Exception $e) {
14 
+		} catch (Exception $e) {
16 15 
 			$this->fail();
17 16 
 		}
18 17 
 		
@@ -26,8 +25,7 @@  discard block
 
 block discarded – undo
26 25 
 		try {
27 26 
 			display_flow_bar("test",0);
28 27 
 			$this->assertTrue(true);
29 
-		}
30 
-		catch (Exception $e) {
28 
+		} catch (Exception $e) {
31 29 
 			$this->fail();
32 30 
 		}
33 31 
 		
@@ -40,8 +38,7 @@  discard block
 
 block discarded – undo
40 38 
 		try {
41 39 
 			start_flow_bar("test", 1);
42 40 
 			$this->assertTrue(true);
43 
-		}
44 
-		catch (Exception $e) {
41 
+		} catch (Exception $e) {
45 42 
 			$this->fail();
46 43 
 		}
47 44 
 		
@@ -54,8 +51,7 @@  discard block
 
 block discarded – undo
54 51 
 		try {
55 52 
 			destroy_flow_bar("test");
56 53 
 			$this->assertTrue(true);
57 
-		}
58 
-		catch (Exception $e) {
54 
+		} catch (Exception $e) {
59 55 
 			$this->fail();
60 56 
 		}
61 57 
 		
@@ -69,8 +65,7 @@  discard block
 
 block discarded – undo
69 65 
 		try {
70 66 
 			display_progress_bar("test",80, 100);
71 67 
 			$this->assertTrue(true);
72 
-		}
73 
-		catch (Exception $e) {
68 
+		} catch (Exception $e) {
74 69 
 			$this->fail();
75 70 
 		}
76 71 
 		
@@ -83,8 +78,7 @@  discard block
 
 block discarded – undo
83 78 
 		try {
84 79 
 			update_progress_bar("test",80, 100);
85 80 
 			$this->assertTrue(true);
86 
-		}
87 
-		catch (Exception $e) {
81 
+		} catch (Exception $e) {
88 82 
 			$this->fail();
89 83 
 		}
90 84
Please login to merge, or discard this patch.
tests/tests/include/utils/LogicHookTest.php 1 patch
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -10,8 +10,7 @@  discard block
 
 block discarded – undo
10 10 
 		try {
11 11 
 			$LogicHook = new LogicHook();
12 12 
 			$this->assertTrue(true);
13 
-		}
14 
-		catch (Exception $e) {
13 
+		} catch (Exception $e) {
15 14 
 			$this->fail();
16 15 
 		}
17 16 
 		
@@ -128,8 +127,7 @@  discard block
 
 block discarded – undo
128 127 
     	try {
129 128 
     		LogicHook::refreshHooks();
130 129 
     		$this->assertTrue(true);
131 
-    	}
132 
-    	catch (Exception $e) {
130 
+    	} catch (Exception $e) {
133 131 
     		$this->fail();
134 132 
     	}
135 133 
     	
@@ -254,8 +252,7 @@  discard block
 
 block discarded – undo
254 252 
 		try {
255 253 
 			$LogicHook->call_custom_logic('', 'after_ui_footer');
256 254 
 			$this->assertTrue(true);
257 
-		}
258 
-		catch (Exception $e) {
255 
+		} catch (Exception $e) {
259 256 
 			$this->fail();
260 257 
 		}
261 258 
 		
@@ -273,8 +270,7 @@  discard block
 
 block discarded – undo
273 270 
 		try {
274 271 
 			$LogicHook->process_hooks($hooks, 'after_ui_footer',array() );
275 272 
 			$this->assertTrue(true);
276 
-		}
277 
-		catch (Exception $e) {
273 
+		} catch (Exception $e) {
278 274 
 			$this->fail();
279 275 
 		}
280 276
Please login to merge, or discard this patch.
tests/tests/include/utils/mvcUtilsTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -9,8 +9,7 @@
 
 block discarded – undo
9 9 
 		try {
10 10 
 			loadParentView('classic');
11 11 
 			$this->assertTrue(true);
12 
-		}
13 
-		catch (Exception $e) {
12 
+		} catch (Exception $e) {
14 13 
 			$this->fail();
15 14 
 		}
16 15
Please login to merge, or discard this patch.
tests/tests/modules/SecurityGroups/SecurityGroupTest.php 1 patch
Braces   +8 added lines, -16 removed lines patch added patch discarded remove patch 
@@ -116,8 +116,7 @@  discard block
 
 block discarded – undo
116 116 
 		try {
117 117 
 			SecurityGroup::inherit($account,false);
118 118 
 			$this->assertTrue(true);
119 
-		}
120 
-		catch (Exception $e) {
119 
+		} catch (Exception $e) {
121 120 
 			$this->fail();
122 121 
 		}
123 122 
 		
@@ -138,8 +137,7 @@  discard block
 
 block discarded – undo
138 137 
 		try {
139 138 
 			SecurityGroup::assign_default_groups($account,false);
140 139 
 			$this->assertTrue(true);
141 
-		}
142 
-		catch (Exception $e) {
140 
+		} catch (Exception $e) {
143 141 
 			$this->fail();
144 142 
 		}
145 143 
 		
@@ -159,8 +157,7 @@  discard block
 
 block discarded – undo
159 157 
 		try {
160 158 
 			SecurityGroup::inherit_creator($account,false);
161 159 
 			$this->assertTrue(true);
162 
-		}
163 
-		catch (Exception $e) {
160 
+		} catch (Exception $e) {
164 161 
 			$this->fail();
165 162 
 		}
166 163 
 		
@@ -182,8 +179,7 @@  discard block
 
 block discarded – undo
182 179 
 		try {
183 180 
 			SecurityGroup::inherit_assigned($account,false);
184 181 
 			$this->assertTrue(true);
185 
-		}
186 
-		catch (Exception $e) {
182 
+		} catch (Exception $e) {
187 183 
 			$this->fail();
188 184 
 		}
189 185 
 		
@@ -204,8 +200,7 @@  discard block
 
 block discarded – undo
204 200 
 		try {
205 201 
 			SecurityGroup::inherit_parent($account,false);
206 202 
 			$this->assertTrue(true);
207 
-		}
208 
-		catch (Exception $e) {
203 
+		} catch (Exception $e) {
209 204 
 			$this->fail();
210 205 
 		}
211 206 
 		
@@ -226,8 +221,7 @@  discard block
 
 block discarded – undo
226 221 
 		try {
227 222 
 			SecurityGroup::inherit_parentQuery($account, "Accounts", 1, 1 , $account->module_dir);
228 223 
 			$this->assertTrue(true);
229 
-		}
230 
-		catch (Exception $e) {
224 
+		} catch (Exception $e) {
231 225 
 			$this->fail();
232 226 
 		}
233 227 
 		
@@ -399,8 +393,7 @@  discard block
 
 block discarded – undo
399 393 
 		try {
400 394 
 			$securityGroup->addGroupToRecord("Accounts", 1, 1);
401 395 
 			$this->assertTrue(true);
402 
-		}
403 
-		catch (Exception $e) {
396 
+		} catch (Exception $e) {
404 397 
 			$this->fail();
405 398 
 		}
406 399 
 		
@@ -420,8 +413,7 @@  discard block
 
 block discarded – undo
420 413 
 		try {
421 414 
 			$securityGroup->removeGroupFromRecord("Accounts", 1, 1);
422 415 
 			$this->assertTrue(true);
423 
-		}
424 
-		catch (Exception $e) {
416 
+		} catch (Exception $e) {
425 417 
 			$this->fail();
426 418 
 		}
427 419
Please login to merge, or discard this patch.
tests/tests/modules/Releases/ReleaseTest.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -55,8 +55,7 @@  discard block
 
 block discarded – undo
55 55 
 		try {
56 56 
 			$release->fill_in_additional_list_fields();
57 57 
 			$this->assertTrue(true);
58 
-		}
59 
-		catch (Exception $e) {
58 
+		} catch (Exception $e) {
60 59 
 			$this->fail();
61 60 
 		}
62 61 
 		
@@ -70,8 +69,7 @@  discard block
 
 block discarded – undo
70 69 
 		try {
71 70 
 			$release->fill_in_additional_detail_fields();
72 71 
 			$this->assertTrue(true);
73 
-		}
74 
-		catch (Exception $e) {
72 
+		} catch (Exception $e) {
75 73 
 			$this->fail();
76 74 
 		}
77 75
Please login to merge, or discard this patch.
tests/tests/modules/Calls/CallTest.php 1 patch
Braces   +2 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -180,7 +180,7 @@  discard block
 
 block discarded – undo
180 180 
         foreach($expected as $expectedKey => $expectedVal){
181 181 
             if($expectedKey == 'SET_COMPLETE'){
182 182 
                 $this->assertRegExp($expected[$expectedKey],$actual[$expectedKey]);
183 
-            }else {
183 
+            } else {
184 184 
                 $this->assertSame($expected[$expectedKey], $actual[$expectedKey]);
185 185 
             }
186 186 
         }
 
@@ -312,8 +312,7 @@  discard block
 
 block discarded – undo
312 312 
 		try {
313 313 
 			$call->save_relationship_changes(true);
314 314 
 			$this->assertTrue(true);
315 
-		}
316 
-		catch (Exception $e) {
315 
+		} catch (Exception $e) {
317 316 
 			$this->fail();
318 317 
 		}
319 318
Please login to merge, or discard this patch.
tests/tests/modules/AOR_Fields/AOR_FieldTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -46,8 +46,7 @@
 
 block discarded – undo
46 46 
     	try {
47 47 
     		$aor_Field->save_lines($post_data, new AOR_Report());
48 48 
     		$this->assertTrue(true);
49 
-    	}
50 
-    	catch (Exception $e) {
49 
+    	} catch (Exception $e) {
51 50 
     		$this->fail();
52 51 
     	}
53 52
Please login to merge, or discard this patch.
tests/tests/modules/SchedulersJobs/SchedulersJobTest.php 1 patch
Braces   +3 added lines, -6 removed lines patch added patch discarded remove patch 
@@ -97,8 +97,7 @@  discard block
 
 block discarded – undo
97 97 
 		try {
98 98 
 			$schedulersJob->fill_in_additional_list_fields();
99 99 
 			$this->assertTrue(true);
100 
-		}
101 
-		catch (Exception $e) {
100 
+		} catch (Exception $e) {
102 101 
 			$this->fail();
103 102 
 		}
104 103 
 
@@ -151,8 +150,7 @@  discard block
 
 block discarded – undo
151 150 
 		try {
152 151 
 			$schedulersJob->onFailureRetry();
153 152 
 			$this->assertTrue(true);
154 
-		}
155 
-		catch (Exception $e) {
153 
+		} catch (Exception $e) {
156 154 
 			$this->fail();
157 155 
 		}
158 156 
 		
@@ -168,8 +166,7 @@  discard block
 
 block discarded – undo
168 166 
 		try {
169 167 
 			$schedulersJob->onFinalFailure();
170 168 
 			$this->assertTrue(true);
171 
-		}
172 
-		catch (Exception $e) {
169 
+		} catch (Exception $e) {
173 170 
 			$this->fail();
174 171 
 		}
175 172
Please login to merge, or discard this patch.