@@ -302,7 +302,7 @@ |
||
| 302 | 302 | // do we skip destroying the test environment? |
| 303 | 303 | if ($this->st->getPersistTestEnvironment()) { |
| 304 | 304 | echo PHP_EOL . "* Warning: NOT destroying test environment" . PHP_EOL |
| 305 | - . " --reuse-target flag is set" . PHP_EOL; |
|
| 305 | + . " --reuse-target flag is set" . PHP_EOL; |
|
| 306 | 306 | } |
| 307 | 307 | |
| 308 | 308 | // cleanup |
@@ -218,8 +218,7 @@ |
||
| 218 | 218 | // $this->playerList contains one or more things to play |
| 219 | 219 | // |
| 220 | 220 | // let's play each of them in order |
| 221 | - foreach ($this->playerList as $player) |
|
| 222 | - { |
|
| 221 | + foreach ($this->playerList as $player) { |
|
| 223 | 222 | // execute each player in turn |
| 224 | 223 | // |
| 225 | 224 | // they may also have their own list of nested players |
@@ -231,7 +231,7 @@ |
||
| 231 | 231 | |
| 232 | 232 | // setup signal handling |
| 233 | 233 | pcntl_signal(SIGTERM, array($this, 'sigtermHandler')); |
| 234 | - pcntl_signal(SIGINT , array($this, 'sigtermHandler')); |
|
| 234 | + pcntl_signal(SIGINT, array($this, 'sigtermHandler')); |
|
| 235 | 235 | } |
| 236 | 236 | |
| 237 | 237 | /** |
@@ -61,7 +61,7 @@ |
||
| 61 | 61 | public function __construct() |
| 62 | 62 | { |
| 63 | 63 | $msg = "You need to use -s to specify which system to test." . PHP_EOL . PHP_EOL |
| 64 | - . "Use 'storyplayer --list-systems' to see the list of known systems under test."; |
|
| 64 | + . "Use 'storyplayer --list-systems' to see the list of known systems under test."; |
|
| 65 | 65 | parent::__construct(400, $msg, $msg); |
| 66 | 66 | } |
| 67 | 67 | } |
| 68 | 68 | \ No newline at end of file |
@@ -78,20 +78,20 @@ |
||
| 78 | 78 | |
| 79 | 79 | public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables) |
| 80 | 80 | { |
| 81 | - // which colour mode are we in? |
|
| 82 | - switch ($engine->options->color) { |
|
| 83 | - case Feature_ColorSwitch::NO_COLOR: |
|
| 84 | - $injectables->output->disableColourSupport(); |
|
| 85 | - break; |
|
| 81 | + // which colour mode are we in? |
|
| 82 | + switch ($engine->options->color) { |
|
| 83 | + case Feature_ColorSwitch::NO_COLOR: |
|
| 84 | + $injectables->output->disableColourSupport(); |
|
| 85 | + break; |
|
| 86 | 86 | |
| 87 | - case Feature_ColorSwitch::ALWAYS_COLOR: |
|
| 88 | - $injectables->output->enforceColourSupport(); |
|
| 89 | - break; |
|
| 87 | + case Feature_ColorSwitch::ALWAYS_COLOR: |
|
| 88 | + $injectables->output->enforceColourSupport(); |
|
| 89 | + break; |
|
| 90 | 90 | |
| 91 | - case Feature_ColorSwitch::AUTO_COLOR: |
|
| 92 | - $injectables->output->enableColourSupport(); |
|
| 93 | - break; |
|
| 94 | - } |
|
| 91 | + case Feature_ColorSwitch::AUTO_COLOR: |
|
| 92 | + $injectables->output->enableColourSupport(); |
|
| 93 | + break; |
|
| 94 | + } |
|
| 95 | 95 | } |
| 96 | 96 | |
| 97 | 97 | public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables) |
@@ -94,8 +94,7 @@ |
||
| 94 | 94 | $engine->options->defines = new stdClass; |
| 95 | 95 | } |
| 96 | 96 | |
| 97 | - foreach ($params as $param) |
|
| 98 | - { |
|
| 97 | + foreach ($params as $param) { |
|
| 99 | 98 | // split up the setting |
| 100 | 99 | $parts = explode('=', $param); |
| 101 | 100 | $key = array_shift($parts); |
@@ -70,14 +70,14 @@ |
||
| 70 | 70 | { |
| 71 | 71 | public function addSwitches(CliCommand $command, $injectables) |
| 72 | 72 | { |
| 73 | - $command->addSwitches([ |
|
| 74 | - new Feature_PersistDeviceSwitch, |
|
| 75 | - ]); |
|
| 73 | + $command->addSwitches([ |
|
| 74 | + new Feature_PersistDeviceSwitch, |
|
| 75 | + ]); |
|
| 76 | 76 | } |
| 77 | 77 | |
| 78 | 78 | public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables) |
| 79 | 79 | { |
| 80 | - // no-op |
|
| 80 | + // no-op |
|
| 81 | 81 | } |
| 82 | 82 | |
| 83 | 83 | public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables) |
@@ -70,14 +70,14 @@ |
||
| 70 | 70 | { |
| 71 | 71 | public function addSwitches(CliCommand $command, $injectables) |
| 72 | 72 | { |
| 73 | - $command->addSwitches([ |
|
| 74 | - new Feature_PersistProcessesSwitch, |
|
| 75 | - ]); |
|
| 73 | + $command->addSwitches([ |
|
| 74 | + new Feature_PersistProcessesSwitch, |
|
| 75 | + ]); |
|
| 76 | 76 | } |
| 77 | 77 | |
| 78 | 78 | public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables) |
| 79 | 79 | { |
| 80 | - // no-op |
|
| 80 | + // no-op |
|
| 81 | 81 | } |
| 82 | 82 | |
| 83 | 83 | public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables) |
@@ -70,21 +70,21 @@ |
||
| 70 | 70 | { |
| 71 | 71 | public function addSwitches(CliCommand $command, $injectables) |
| 72 | 72 | { |
| 73 | - $command->addSwitches([ |
|
| 74 | - new Feature_PersistTargetSwitch, |
|
| 75 | - new Feature_ReuseTargetSwitch |
|
| 76 | - ]); |
|
| 73 | + $command->addSwitches([ |
|
| 74 | + new Feature_PersistTargetSwitch, |
|
| 75 | + new Feature_ReuseTargetSwitch |
|
| 76 | + ]); |
|
| 77 | 77 | } |
| 78 | 78 | |
| 79 | 79 | public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables) |
| 80 | 80 | { |
| 81 | - // no-op |
|
| 81 | + // no-op |
|
| 82 | 82 | } |
| 83 | 83 | |
| 84 | 84 | public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables) |
| 85 | 85 | { |
| 86 | - // shorthand |
|
| 87 | - $output = $injectables->output; |
|
| 86 | + // shorthand |
|
| 87 | + $output = $injectables->output; |
|
| 88 | 88 | |
| 89 | 89 | // are we keeping the test environment hanging around afterwards? |
| 90 | 90 | if (isset($engine->options->persistTarget) && $engine->options->persistTarget) |
@@ -87,8 +87,7 @@ discard block |
||
| 87 | 87 | $output = $injectables->output; |
| 88 | 88 | |
| 89 | 89 | // are we keeping the test environment hanging around afterwards? |
| 90 | - if (isset($engine->options->persistTarget) && $engine->options->persistTarget) |
|
| 91 | - { |
|
| 90 | + if (isset($engine->options->persistTarget) && $engine->options->persistTarget) { |
|
| 92 | 91 | $injectables->activeConfig->setData('storyplayer.phases.testEnvShutdown.TestEnvironmentDestruction', false); |
| 93 | 92 | $injectables->activeConfig->unsetData('storyplayer.phases.userAbort.TestEnvironmentDestruction'); |
| 94 | 93 | $st->setPersistTestEnvironment(); |
@@ -96,8 +95,7 @@ discard block |
||
| 96 | 95 | |
| 97 | 96 | // are we trying to use a test environment that has previously |
| 98 | 97 | // been persisted? |
| 99 | - if (isset($engine->options->reuseTarget) && $engine->options->reuseTarget) |
|
| 100 | - { |
|
| 98 | + if (isset($engine->options->reuseTarget) && $engine->options->reuseTarget) { |
|
| 101 | 99 | // does the target exist to be reused? |
| 102 | 100 | $output->setSilentMode(); |
| 103 | 101 | $hasTarget = $st->fromTargetsTable()->hasCurrentTestEnvironment(); |
@@ -128,8 +126,7 @@ discard block |
||
| 128 | 126 | // if we get here, then we do not need to create the test environment |
| 129 | 127 | $injectables->activeConfig->setData('storyplayer.phases.testEnvStartup.TestEnvironmentConstruction', false); |
| 130 | 128 | } |
| 131 | - else |
|
| 132 | - { |
|
| 129 | + else { |
|
| 133 | 130 | // do we already have this target? |
| 134 | 131 | // |
| 135 | 132 | // this can happen when the test environment was previously |
@@ -70,12 +70,12 @@ |
||
| 70 | 70 | { |
| 71 | 71 | public function addSwitches(CliCommand $command, $injectables) |
| 72 | 72 | { |
| 73 | - // no-op |
|
| 73 | + // no-op |
|
| 74 | 74 | } |
| 75 | 75 | |
| 76 | 76 | public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables) |
| 77 | 77 | { |
| 78 | - $injectables->initPhaseLoaderSupport($injectables); |
|
| 78 | + $injectables->initPhaseLoaderSupport($injectables); |
|
| 79 | 79 | } |
| 80 | 80 | |
| 81 | 81 | public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables) |
@@ -70,12 +70,12 @@ |
||
| 70 | 70 | { |
| 71 | 71 | public function addSwitches(CliCommand $command, $injectables) |
| 72 | 72 | { |
| 73 | - // no-op |
|
| 73 | + // no-op |
|
| 74 | 74 | } |
| 75 | 75 | |
| 76 | 76 | public function initBeforeModulesAvailable(CliEngine $engine, CliCommand $command, Injectables $injectables) |
| 77 | 77 | { |
| 78 | - $injectables->initProseLoaderSupport($injectables); |
|
| 78 | + $injectables->initProseLoaderSupport($injectables); |
|
| 79 | 79 | } |
| 80 | 80 | |
| 81 | 81 | public function initAfterModulesAvailable(StoryTeller $st, CliEngine $engine, Injectables $injectables) |