datasift / storyplayer

src/php/DataSift/Storyplayer/Phases/PreTestInspectionPhase.php

Braces +1 added lines, -2 removed lines

@@ -73,8 +73,7 @@
         $phaseResult = $this->getNewPhaseResult();
 
         // do we have anything to do?
-        if (!$story->hasPreTestInspection())
-        {
+        if (!$story->hasPreTestInspection()) {
             $phaseResult->setContinuePlaying(
                 $phaseResult::HASNOACTIONS,
                 "story has no pre-test inspection instructions"

src/php/DataSift/Storyplayer/Phases/PreTestPredictionPhase.php

Braces +1 added lines, -2 removed lines

@@ -75,8 +75,7 @@
 
         try {
             // do we have anything to do?
-            if (!$story->hasPreTestPrediction())
-            {
+            if (!$story->hasPreTestPrediction()) {
                 $phaseResult->setContinuePlaying(
                     $phaseResult::HASNOACTIONS,
                     "story has no pre-test prediction instructions; skipping"

src/php/DataSift/Storyplayer/Phases/StartupHandlersPhase.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
     public function doPhase($phaseResults)
     {
         // shorthand
-        $st      = $this->st;
+        $st = $this->st;
 
         // our result object
         $phaseResult = $this->getNewPhaseResult();

src/php/DataSift/Storyplayer/Phases/TableHandlersHelper.php

Indentation +8 added lines, -8 removed lines

@@ -59,13 +59,13 @@ 
 
 class TableHandlersHelper
 {
-	/**
-	 * @param string $type
-	 */
-	public function runHandlers(StoryTeller $st, $type)
-	{
-		// shorthand
-		$output = $st->getOutput();
+    /**
+     * @param string $type
+     */
+    public function runHandlers(StoryTeller $st, $type)
+    {
+        // shorthand
+        $output = $st->getOutput();
 
         // Do we have any persistent tables to cleanup?
         $runtimeConfig = $st->getRuntimeConfig();
@@ -103,5 +103,5 @@ 
             }
             exit(1);
         }
-	}
+    }
 }
\ No newline at end of file

Spacing +4 added lines, -4 removed lines

@@ -69,7 +69,7 @@ 
 
         // Do we have any persistent tables to cleanup?
         $runtimeConfig = $st->getRuntimeConfig();
-        if (!isset($runtimeConfig->storyplayer, $runtimeConfig->storyplayer->tables)){
+        if (!isset($runtimeConfig->storyplayer, $runtimeConfig->storyplayer->tables)) {
             // there are no tables at all
             return;
         }
@@ -83,12 +83,12 @@ 
 
         // Take a look at all of our process list tables
         foreach ($runtimeConfig->storyplayer->tables as $key => $value) {
-            $className = "cleanup".ucfirst($key);
+            $className = "cleanup" . ucfirst($key);
             try {
                 $st->$className($key)->$type();
-            } catch(E5xx_NoMatchingActions $e){
+            } catch (E5xx_NoMatchingActions $e) {
                 // We don't know about a cleanup module for this, SHOUT LOUDLY
-                $missingCleanupHandlers[] = "Missing cleanup module for '{$key}'".PHP_EOL;
+                $missingCleanupHandlers[] = "Missing cleanup module for '{$key}'" . PHP_EOL;
             }
         }
 

Braces +3 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
 
         // Do we have any persistent tables to cleanup?
         $runtimeConfig = $st->getRuntimeConfig();
-        if (!isset($runtimeConfig->storyplayer, $runtimeConfig->storyplayer->tables)){
+        if (!isset($runtimeConfig->storyplayer, $runtimeConfig->storyplayer->tables)) {
             // there are no tables at all
             return;
         }
@@ -86,7 +86,8 @@ 
             $className = "cleanup".ucfirst($key);
             try {
                 $st->$className($key)->$type();
-            } catch(E5xx_NoMatchingActions $e){
+            }
+            catch(E5xx_NoMatchingActions $e) {
                 // We don't know about a cleanup module for this, SHOUT LOUDLY
                 $missingCleanupHandlers[] = "Missing cleanup module for '{$key}'".PHP_EOL;
             }

src/php/DataSift/Storyplayer/Phases/TestCanRunCheckPhase.php

Braces +2 added lines, -4 removed lines

@@ -74,8 +74,7 @@ 
         $phaseResult = $this->getNewPhaseResult();
 
         // do we have anything to do?
-        if (!$story->hasTestCanRunCheck())
-        {
+        if (!$story->hasTestCanRunCheck()) {
             $phaseResult->setContinuePlaying(
                 $phaseResult::COMPLETED,
                 "story can always run"
@@ -154,8 +153,7 @@ 
             );
             $storyResult->setStoryHasBeenSkipped($phaseResult);
         }
-        catch (Exception $e)
-        {
+        catch (Exception $e) {
             // something went wrong ... the test cannot continue
             $phaseResult->setPlayingFailed(
                 $phaseResult::ERROR,

src/php/DataSift/Storyplayer/Phases/TestEnvironmentDestructionPhase.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     public function doPhase($thingBeingPlayed = null)
     {
         // shorthand
-        $st    = $this->st;
+        $st = $this->st;
 
         // our return value
         $phaseResult = $this->getNewPhaseResult();

src/php/DataSift/Storyplayer/Phases/TestSetupPhase.php

Braces +2 added lines, -4 removed lines

@@ -73,8 +73,7 @@ 
         $phaseResult = $this->getNewPhaseResult();
 
         // do we have anything to do?
-        if (!$story->hasTestSetup())
-        {
+        if (!$story->hasTestSetup()) {
             $phaseResult->setContinuePlaying(
                 $phaseResult::HASNOACTIONS,
                 "story has no test setup instructions"
@@ -122,8 +121,7 @@ 
             );
             $storyResult->setStoryIsIncomplete($phaseResult);
         }
-        catch (Exception $e)
-        {
+        catch (Exception $e) {
             // something went wrong ... the test cannot continue
             $phaseResult->setPlayingFailed(
                 $phaseResult::ERROR,

src/php/DataSift/Storyplayer/Phases/TestTeardownPhase.php

Braces +2 added lines, -4 removed lines

@@ -73,8 +73,7 @@ 
         $phaseResult = $this->getNewPhaseResult();
 
         // do we have anything to do?
-        if (!$story->hasTestTeardown())
-        {
+        if (!$story->hasTestTeardown()) {
             $phaseResult->setContinuePlaying(
                 $phaseResult::HASNOACTIONS,
                 "story has no test teardown instructions"
@@ -124,8 +123,7 @@ 
             );
             $storyResult->setStoryIsIncomplete($phaseResult);
         }
-        catch (Exception $e)
-        {
+        catch (Exception $e) {
             // we still want to continue at this stage
             $phaseResult->setContinuePlaying(
                 $phaseResult::ERROR,

src/php/DataSift/Storyplayer/PlayerLib/Action/Logger.php

Braces +2 added lines, -4 removed lines

@@ -78,8 +78,7 @@ 
     public function startAction($message, $codeLine = null)
     {
         // do we have an open action?
-        if (!$this->action || !$this->action->getIsOpen())
-        {
+        if (!$this->action || !$this->action->getIsOpen()) {
             $openItem = $this->action = new Action_LogItem($this->injectables, 1);
         }
         else {
@@ -93,8 +92,7 @@ 
     public function closeAllOpenActions()
     {
         // do we have any empty log items?
-        if (!$this->action)
-        {
+        if (!$this->action) {
             return;
         }