datasift / storyplayer

src/php/DataSift/Storyplayer/ConfigLib/ActiveConfig.php

Doc Comments +6 added lines

@@ -105,11 +105,17 @@
         $activeConfig->roles = $rolesTable->$testEnvName;
 	}
 
+	/**
+	 * @param Injectables $injectables
+	 */
 	public function mergeStoryplayerConfig($injectables, $spConf)
 	{
 		$this->mergeData('storyplayer', $spConf);
 	}
 
+	/**
+	 * @param Injectables $injectables
+	 */
 	public function mergeSystemUnderTestConfig($injectables, SystemUnderTestConfig $sutConfig = null)
 	{
         // do we have a system under test?

Indentation +27 added lines, -27 removed lines

@@ -75,43 +75,43 @@ 
  */
 class ActiveConfig extends WrappedConfig
 {
-	public function init(Injectables $injectables)
-	{
-		// we start off with the built-in config
-		$this->mergeData('storyplayer', $injectables->defaultConfig);
-
-		// these are the initial variables we want
-		$this->setData('storyplayer.ipAddress', $this->getHostIpAddress());
-		$this->setData('storyplayer.currentDir', getcwd());
-		$this->setData('storyplayer.user.home', getenv('HOME'));
-
-		// we also want to link in the hosts and roles tables, to make
-		// it a lot easier for Prose modules
-		$activeConfig         = $this->getConfig();
+    public function init(Injectables $injectables)
+    {
+        // we start off with the built-in config
+        $this->mergeData('storyplayer', $injectables->defaultConfig);
+
+        // these are the initial variables we want
+        $this->setData('storyplayer.ipAddress', $this->getHostIpAddress());
+        $this->setData('storyplayer.currentDir', getcwd());
+        $this->setData('storyplayer.user.home', getenv('HOME'));
+
+        // we also want to link in the hosts and roles tables, to make
+        // it a lot easier for Prose modules
+        $activeConfig         = $this->getConfig();
         $runtimeConfig        = $injectables->getRuntimeConfig();
         $runtimeConfigManager = $injectables->getRuntimeConfigManager();
         $testEnvName          = $injectables->activeTestEnvironmentName;
 
-		$hostsTable = $runtimeConfigManager->getTable($runtimeConfig, 'hosts');
+        $hostsTable = $runtimeConfigManager->getTable($runtimeConfig, 'hosts');
         if (!isset($hostsTable->$testEnvName)) {
             $hostsTable->$testEnvName = new BaseObject;
         }
         $activeConfig->hosts = $hostsTable->$testEnvName;
 
-		$rolesTable = $runtimeConfigManager->getTable($runtimeConfig, 'roles');
+        $rolesTable = $runtimeConfigManager->getTable($runtimeConfig, 'roles');
         if (!isset($rolesTable->$testEnvName)) {
             $rolesTable->$testEnvName = new BaseObject;
         }
         $activeConfig->roles = $rolesTable->$testEnvName;
-	}
+    }
 
-	public function mergeStoryplayerConfig($injectables, $spConf)
-	{
-		$this->mergeData('storyplayer', $spConf);
-	}
+    public function mergeStoryplayerConfig($injectables, $spConf)
+    {
+        $this->mergeData('storyplayer', $spConf);
+    }
 
-	public function mergeSystemUnderTestConfig($injectables, SystemUnderTestConfig $sutConfig = null)
-	{
+    public function mergeSystemUnderTestConfig($injectables, SystemUnderTestConfig $sutConfig = null)
+    {
         // do we have a system under test?
         if (!isset($injectables->activeSystemUnderTestName) || $sutConfig === null) {
             $this->setData('systemundertest', null);
@@ -123,7 +123,7 @@ 
 
         // merge in the loaded config
         $this->mergeData('systemundertest', $sutConfig->getConfig());
-	}
+    }
 
     /**
      *
@@ -131,8 +131,8 @@ 
      * @param  \DataSift\Storyplayer\TestEnvironmentsLib\TestEnvironmentsConfig|\DataSift\Storyplayer\DefinitionLib\TestEnvironment_Definition $envConfig
      * @return void
      */
-	public function mergeTestEnvironmentConfig($injectables, $envConfig = null)
-	{
+    public function mergeTestEnvironmentConfig($injectables, $envConfig = null)
+    {
         // do we have a test environment?
         if (!isset($injectables->activeTestEnvironmentName) || $envConfig === null) {
             $this->setData('target', null);
@@ -143,8 +143,8 @@ 
         $this->setData('target.name', $injectables->activeTestEnvironmentName);
 
         // merge in the loaded config
-		$this->mergeData('target', $envConfig->getConfig());
-	}
+        $this->mergeData('target', $envConfig->getConfig());
+    }
 
     protected function getHostIpAddress()
     {

Spacing +3 added lines, -3 removed lines

@@ -87,7 +87,7 @@ 
 
 		// we also want to link in the hosts and roles tables, to make
 		// it a lot easier for Prose modules
-		$activeConfig         = $this->getConfig();
+		$activeConfig = $this->getConfig();
         $runtimeConfig        = $injectables->getRuntimeConfig();
         $runtimeConfigManager = $injectables->getRuntimeConfigManager();
         $testEnvName          = $injectables->activeTestEnvironmentName;
@@ -179,7 +179,7 @@ 
                 // does the adapter have an IP address?
                 try {
                     $ipAddress = $netifaces->getIpAddress($adapterToTest);
-                } catch(NetifacesException $e){
+                } catch (NetifacesException $e) {
                     // We couldn't get an IP address
                     $ipAddress = null;
                 }
@@ -213,7 +213,7 @@ 
             // this sucks
             throw new NetifacesException("Unable to determine IP address");
 
-        } catch (NetifacesException $e){
+        } catch (NetifacesException $e) {
             throw new Exception("could not determine IP address of host machine");
         }
     }

Braces +4 added lines, -2 removed lines

@@ -179,7 +179,8 @@ 
                 // does the adapter have an IP address?
                 try {
                     $ipAddress = $netifaces->getIpAddress($adapterToTest);
-                } catch(NetifacesException $e){
+                }
+                catch(NetifacesException $e) {
                     // We couldn't get an IP address
                     $ipAddress = null;
                 }
@@ -213,7 +214,8 @@ 
             // this sucks
             throw new NetifacesException("Unable to determine IP address");
 
-        } catch (NetifacesException $e){
+        }
+        catch (NetifacesException $e) {
             throw new Exception("could not determine IP address of host machine");
         }
     }

src/php/DataSift/Storyplayer/Reports/TapReport.php

Doc Comments -1 removed lines

@@ -177,7 +177,6 @@
     }
 
     /**
-     * @param integer $level
      * @param string $msg
      * @return void
      */

src/php/DataSift/Storyplayer/Cli/BuildTestEnvironment/Command.php

Indentation +1 added lines, -1 removed lines

@@ -302,7 +302,7 @@
         // do we skip destroying the test environment?
         if ($this->st->getPersistTestEnvironment()) {
             echo PHP_EOL . "* Warning: NOT destroying test environment" . PHP_EOL
-                 .         "           --reuse-target flag is set" . PHP_EOL;
+                    .         "           --reuse-target flag is set" . PHP_EOL;
         }
 
         // cleanup

Braces +1 added lines, -2 removed lines

@@ -218,8 +218,7 @@
         // $this->playerList contains one or more things to play
         //
         // let's play each of them in order
-        foreach ($this->playerList as $player)
-        {
+        foreach ($this->playerList as $player) {
             // execute each player in turn
             //
             // they may also have their own list of nested players

Spacing +1 added lines, -1 removed lines

@@ -231,7 +231,7 @@
 
         // setup signal handling
         pcntl_signal(SIGTERM, array($this, 'sigtermHandler'));
-        pcntl_signal(SIGINT , array($this, 'sigtermHandler'));
+        pcntl_signal(SIGINT, array($this, 'sigtermHandler'));
     }
 
     /**

src/php/DataSift/Storyplayer/Cli/E4xx/NoSystemUnderTestSpecified.php

Indentation +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
     public function __construct()
     {
         $msg = "You need to use -s to specify which system to test." . PHP_EOL . PHP_EOL
-             . "Use 'storyplayer --list-systems' to see the list of known systems under test.";
+                . "Use 'storyplayer --list-systems' to see the list of known systems under test.";
         parent::__construct(400, $msg, $msg);
     }
 }
\ No newline at end of file

src/php/DataSift/Storyplayer/Cli/Feature/ColorSupport.php

Indentation +12 added lines, -12 removed lines

@@ -78,20 +78,20 @@
 
     public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables)
     {
-    	// which colour mode are we in?
-    	switch ($engine->options->color) {
-    		case Feature_ColorSwitch::NO_COLOR:
-    			$injectables->output->disableColourSupport();
-    			break;
+        // which colour mode are we in?
+        switch ($engine->options->color) {
+            case Feature_ColorSwitch::NO_COLOR:
+                $injectables->output->disableColourSupport();
+                break;
 
-    		case Feature_ColorSwitch::ALWAYS_COLOR:
-    			$injectables->output->enforceColourSupport();
-    			break;
+            case Feature_ColorSwitch::ALWAYS_COLOR:
+                $injectables->output->enforceColourSupport();
+                break;
 
-    		case Feature_ColorSwitch::AUTO_COLOR:
-    			$injectables->output->enableColourSupport();
-    			break;
-    	}
+            case Feature_ColorSwitch::AUTO_COLOR:
+                $injectables->output->enableColourSupport();
+                break;
+        }
     }
 
     public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables)

src/php/DataSift/Storyplayer/Cli/Feature/DefineSwitch.php

Braces +1 added lines, -2 removed lines

@@ -94,8 +94,7 @@
             $engine->options->defines = new stdClass;
         }
 
-        foreach ($params as $param)
-        {
+        foreach ($params as $param) {
             // split up the setting
             $parts = explode('=', $param);
             $key   = array_shift($parts);

src/php/DataSift/Storyplayer/Cli/Feature/PersistDeviceSupport.php

Indentation +4 added lines, -4 removed lines

@@ -70,14 +70,14 @@
 {
     public function addSwitches(CliCommand $command, $injectables)
     {
-    	$command->addSwitches([
-    		new Feature_PersistDeviceSwitch,
-    	]);
+        $command->addSwitches([
+            new Feature_PersistDeviceSwitch,
+        ]);
     }
 
     public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables)
     {
-    	// no-op
+        // no-op
     }
 
     public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables)

src/php/DataSift/Storyplayer/Cli/Feature/PersistProcessesSupport.php

Indentation +4 added lines, -4 removed lines

@@ -70,14 +70,14 @@
 {
     public function addSwitches(CliCommand $command, $injectables)
     {
-    	$command->addSwitches([
-    		new Feature_PersistProcessesSwitch,
-    	]);
+        $command->addSwitches([
+            new Feature_PersistProcessesSwitch,
+        ]);
     }
 
     public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables)
     {
-    	// no-op
+        // no-op
     }
 
     public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables)

src/php/DataSift/Storyplayer/Cli/Feature/PersistReuseTargetSupport.php

Indentation +7 added lines, -7 removed lines

@@ -70,21 +70,21 @@
 {
     public function addSwitches(CliCommand $command, $injectables)
     {
-    	$command->addSwitches([
-    		new Feature_PersistTargetSwitch,
-    		new Feature_ReuseTargetSwitch
-    	]);
+        $command->addSwitches([
+            new Feature_PersistTargetSwitch,
+            new Feature_ReuseTargetSwitch
+        ]);
     }
 
     public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables)
     {
-    	// no-op
+        // no-op
     }
 
     public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables)
     {
-    	// shorthand
-    	$output = $injectables->output;
+        // shorthand
+        $output = $injectables->output;
 
         // are we keeping the test environment hanging around afterwards?
         if (isset($engine->options->persistTarget) && $engine->options->persistTarget)

Braces +3 added lines, -6 removed lines

@@ -87,8 +87,7 @@ 
     	$output = $injectables->output;
 
         // are we keeping the test environment hanging around afterwards?
-        if (isset($engine->options->persistTarget) && $engine->options->persistTarget)
-        {
+        if (isset($engine->options->persistTarget) && $engine->options->persistTarget) {
             $injectables->activeConfig->setData('storyplayer.phases.testEnvShutdown.TestEnvironmentDestruction', false);
             $injectables->activeConfig->unsetData('storyplayer.phases.userAbort.TestEnvironmentDestruction');
             $st->setPersistTestEnvironment();
@@ -96,8 +95,7 @@ 
 
         // are we trying to use a test environment that has previously
         // been persisted?
-        if (isset($engine->options->reuseTarget) && $engine->options->reuseTarget)
-        {
+        if (isset($engine->options->reuseTarget) && $engine->options->reuseTarget) {
             // does the target exist to be reused?
             $output->setSilentMode();
             $hasTarget = $st->fromTargetsTable()->hasCurrentTestEnvironment();
@@ -128,8 +126,7 @@ 
             // if we get here, then we do not need to create the test environment
             $injectables->activeConfig->setData('storyplayer.phases.testEnvStartup.TestEnvironmentConstruction', false);
         }
-        else
-        {
+        else {
             // do we already have this target?
             //
             // this can happen when the test environment was previously