datasift / storyplayer

src/php/DataSift/Storyplayer/PlayerLib/E5xx/NoSauceLabsConfig.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E4xx_StoryShouldFail extends Exxx_Exception
 {
-    public function __construct() {
+    public function __construct()
+    {
         $msg = "Story should fail";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/E5xx/NoSauceLabsUsername.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E4xx_StoryShouldFail extends Exxx_Exception
 {
-    public function __construct() {
+    public function __construct()
+    {
         $msg = "Story should fail";
         parent::__construct(400, $msg, $msg);
     }

src/php/DataSift/Storyplayer/PlayerLib/Story.php

Braces +1 added lines, -2 removed lines

@@ -1137,8 +1137,7 @@
     public function getOneAction()
     {
         // do we have any callbacks to pick from?
-        if (count($this->actionsCallbacks) == 0)
-        {
+        if (count($this->actionsCallbacks) == 0) {
             throw new E5xx_NoStoryActions($this->getName());
         }
 

Doc Comments +1 added lines

@@ -700,6 +700,7 @@
     }
 
     /**
+     * @param integer $version
      * @return Story
      */
     public function requiresStoryplayerVersion($version)

Spacing +13 added lines, -13 removed lines

@@ -527,7 +527,7 @@ 
     public function getScenario()
     {
         if ($this->storyScenario === null) {
-            return [ "this story has not supplied a scenario; call \$story->setScenario() in your test."];
+            return ["this story has not supplied a scenario; call \$story->setScenario() in your test."];
         }
 
         return $this->storyScenario;
@@ -682,18 +682,18 @@ 
         // tell the template which story it is being used with
         $tmpl->setStory($this);
 
-        $tmpl->hasTestCanRunCheck()         && $this->addTestCanRunCheck($tmpl->getTestCanRunCheck());
-        $tmpl->hasTestSetup()               && $this->addTestSetup($tmpl->getTestSetup());
-        $tmpl->hasTestTeardown()            && $this->addTestTeardown($tmpl->getTestTeardown());
-        $tmpl->hasPerPhaseSetup()           && $this->addPerPhaseSetup($tmpl->getPerPhaseSetup());
-        $tmpl->hasPerPhaseTeardown()        && $this->addPerPhaseTeardown($tmpl->getPerPhaseTeardown());
-        $tmpl->hasDeviceSetup()             && $this->addDeviceSetup($tmpl->getDeviceSetup());
-        $tmpl->hasDeviceTeardown()          && $this->addDeviceTeardown($tmpl->getDeviceTeardown());
-        $tmpl->hasHints()                   && $this->addHints($tmpl->getHints());
-        $tmpl->hasPreTestPrediction()       && $this->addPreTestPrediction($tmpl->getPreTestPrediction());
-        $tmpl->hasPreTestInspection()       && $this->addPreTestInspection($tmpl->getPreTestInspection());
-        $tmpl->hasAction()                  && $this->addAction($tmpl->getAction());
-        $tmpl->hasPostTestInspection()      && $this->addPostTestInspection($tmpl->getPostTestInspection());
+        $tmpl->hasTestCanRunCheck() && $this->addTestCanRunCheck($tmpl->getTestCanRunCheck());
+        $tmpl->hasTestSetup() && $this->addTestSetup($tmpl->getTestSetup());
+        $tmpl->hasTestTeardown() && $this->addTestTeardown($tmpl->getTestTeardown());
+        $tmpl->hasPerPhaseSetup() && $this->addPerPhaseSetup($tmpl->getPerPhaseSetup());
+        $tmpl->hasPerPhaseTeardown() && $this->addPerPhaseTeardown($tmpl->getPerPhaseTeardown());
+        $tmpl->hasDeviceSetup() && $this->addDeviceSetup($tmpl->getDeviceSetup());
+        $tmpl->hasDeviceTeardown() && $this->addDeviceTeardown($tmpl->getDeviceTeardown());
+        $tmpl->hasHints() && $this->addHints($tmpl->getHints());
+        $tmpl->hasPreTestPrediction() && $this->addPreTestPrediction($tmpl->getPreTestPrediction());
+        $tmpl->hasPreTestInspection() && $this->addPreTestInspection($tmpl->getPreTestInspection());
+        $tmpl->hasAction() && $this->addAction($tmpl->getAction());
+        $tmpl->hasPostTestInspection() && $this->addPostTestInspection($tmpl->getPostTestInspection());
 
         // remember this template for future use
         $this->storyTemplates[] = $tmpl;

src/php/Prose/E5xx/NoMatchingActions.php

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
  */
 class E5xx_NotImplemented extends E5xx_ProseException
 {
-    public function __construct($methodName) {
+    public function __construct($methodName)
+    {
         $msg = "Method '{$methodName}' has not been implemented yet";
         parent::__construct(500, $msg, $msg);
     }

src/php/Prose/FromRedisConn.php

Braces +1 added lines, -2 removed lines

@@ -77,8 +77,7 @@
 
             return $return;
         }
-        catch (Exception $e)
-        {
+        catch (Exception $e) {
             throw new E5xx_ActionFailed(__METHOD__, $e->getMessage());
         }
     }

storyplayer/test-environments/vagrant-vbox-centos-6.7/main.php

Indentation +3 added lines, -3 removed lines

@@ -9,14 +9,14 @@
 
 $group1 = $testEnv->newGroup('vagrant', new Vagrant_GroupAdapter);
 $group1->newHost('default', new Vagrant_VirtualboxHostAdapter)
-       ->setOperatingSystem(new CentOS_6_HostAdapter)
-       ->setRoles([
+        ->setOperatingSystem(new CentOS_6_HostAdapter)
+        ->setRoles([
             "host_target",
             "upload_target",
             "ssl_target",
             "zmq_target",
         ])
-       ->setStorySettings((object)[
+        ->setStorySettings((object)[
             "host" => (object)[
                 "expected" => "successfully retrieved this storySetting :)",
             ],

storyplayer/test-environments/vagrant-vbox-centos-7.1/main.php

Indentation +3 added lines, -3 removed lines

@@ -9,14 +9,14 @@
 
 $group1 = $testEnv->newGroup('vagrant', new Vagrant_GroupAdapter);
 $group1->newHost('default', new Vagrant_VirtualboxHostAdapter)
-       ->setOperatingSystem(new CentOS_7_HostAdapter)
-       ->setRoles([
+        ->setOperatingSystem(new CentOS_7_HostAdapter)
+        ->setRoles([
             "host_target",
             "upload_target",
             "ssl_target",
             "zmq_target",
         ])
-       ->setStorySettings((object)[
+        ->setStorySettings((object)[
             "host" => (object)[
                 "expected" => "successfully retrieved this storySetting :)",
             ],

storyplayer/test-environments/vagrant-vbox-ubuntu-14.04-server/files/zmq/zmq-echo-server.php

Indentation +19 added lines, -19 removed lines

@@ -20,24 +20,24 @@
 
 $events = 0;
 while(true) {
-	try {
-		$readable = $writeable = [];
-		$events = $poller->poll($readable, $writeable, -1);
-	}
-	catch (Exception $e) {
-		// do nothing
-	}
+    try {
+        $readable = $writeable = [];
+        $events = $poller->poll($readable, $writeable, -1);
+    }
+    catch (Exception $e) {
+        // do nothing
+    }
 
-	if ($events > 0) {
-		foreach($readable as $r) {
-			if ($r === $single['in']) {
-				$msg = $r->recv();
-				$single['out']->send($msg);
-			}
-			else {
-				$msg = $r->recvmulti();
-				$multi['out']->sendmulti($msg);
-			}
-		}
-	}
+    if ($events > 0) {
+        foreach($readable as $r) {
+            if ($r === $single['in']) {
+                $msg = $r->recv();
+                $single['out']->send($msg);
+            }
+            else {
+                $msg = $r->recvmulti();
+                $multi['out']->sendmulti($msg);
+            }
+        }
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -5,7 +5,7 @@ 
 $single  = array();
 $single['in']  = $context->getSocket(ZMQ::SOCKET_PULL);
 $single['out'] = $context->getSocket(ZMQ::SOCKET_PUSH);
-$multi  = array();
+$multi = array();
 $multi['in']  = $context->getSocket(ZMQ::SOCKET_PULL);
 $multi['out'] = $context->getSocket(ZMQ::SOCKET_PUSH);
 
@@ -19,7 +19,7 @@ 
 $poller->add($multi['in'], ZMQ::POLL_IN);
 
 $events = 0;
-while(true) {
+while (true) {
 	try {
 		$readable = $writeable = [];
 		$events = $poller->poll($readable, $writeable, -1);
@@ -29,7 +29,7 @@ 
 	}
 
 	if ($events > 0) {
-		foreach($readable as $r) {
+		foreach ($readable as $r) {
 			if ($r === $single['in']) {
 				$msg = $r->recv();
 				$single['out']->send($msg);

src/php/DataSift/Storyplayer/Cli/RuntimeConfigManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
      * a separate config file for each test environment. The original
      * runtime-v2.json file is then deleted from disk.
      *
-     * @param  BaseObject $config
+     * @param BaseObject $oldConfig
      * @return void
      */
     protected function splitUpConfig($oldConfig)

Spacing +1 added lines, -1 removed lines

@@ -145,7 +145,7 @@
         }
 
         // yes we do
-        foreach($oldConfig->storyplayer->tables->hosts as $testEnvName => $hosts) {
+        foreach ($oldConfig->storyplayer->tables->hosts as $testEnvName => $hosts) {
             // our new per-test-env config file
             $newConfig = new TestEnvironmentRuntimeConfig();
             $newConfig->setName($testEnvName);

Braces +1 added lines, -2 removed lines

@@ -70,8 +70,7 @@
         static $configDir = null;
 
         // do we have a configDir remembered yet?
-        if (!$configDir)
-        {
+        if (!$configDir) {
             $configDir = getcwd() . '/.storyplayer';
         }