TestsGenerator: do not throw exceptions - failure is not an option

mrtnzlml-archive · Jul 31, 2016 · c94ea10 · c94ea10
1 parent 5134803
commit c94ea10
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/Scaffold/TestsGenerator.php b/src/Scaffold/TestsGenerator.php
@@ -135,16 +135,18 @@ public function generateTests($outputFolder)
 				try {
 					$form->onSuccess($form, $form->getValues());
 					$testClass->addMethod('test' . ucfirst($testMethod))->addBody(
-						"\$this->checkForm(?, ?, [\n" .
-						$controls .
-						"], ?);", [$destination . ':', $action, FALSE]);
+						"\$this->checkForm(?, ?, [\n" . $controls . '], ?);',
+						[$destination . ':', $action, FALSE]
+					);
 				} catch (\Nette\Application\AbortException $exc) {
 					$extra = $this->getResponse($service);
 					$path = $extra ? preg_replace('~^https?://([/a-z0-9]+).*~i', '$1', $extra->getUrl()) : '/';
 					$testClass->addMethod('test' . ucfirst($testMethod))->addBody(
-						"\$this->checkForm(?, ?, [\n" .
-						$controls .
-						"], ?);", [$destination . ':', $action, $path]);
+						"\$this->checkForm(?, ?, [\n" . $controls . '], ?);',
+						[$destination . ':', $action, $path]
+					);
+				} catch (\Exception $exc) {
+					//This sucks but we have to move on - failure is not an option
 				}
 			}