@@ -68,8 +68,7 @@ discard block |
||
| 68 | 68 | $variantTags = $this->variantTags; |
| 69 | 69 | if ($variantTags) { |
| 70 | 70 | return $tester->test($env, $this->reconstructFeature($feature), $skip); |
| 71 | - } |
|
| 72 | - else { |
|
| 71 | + } else { |
|
| 73 | 72 | return $tester->test($env, $feature, $skip); |
| 74 | 73 | } |
| 75 | 74 | } |
@@ -87,11 +86,9 @@ discard block |
||
| 87 | 86 | $scenario->getKeyword(), |
| 88 | 87 | $scenario->getLine()); |
| 89 | 88 | $scenarios[] = $injectedScenario; |
| 90 | - } |
|
| 91 | - elseif (count($scenarioVariants) > 1) { |
|
| 89 | + } elseif (count($scenarioVariants) > 1) { |
|
| 92 | 90 | $scenarios = array_merge($scenarios, $this->forkScenario($scenario, $scenarioVariants, $configTag)); |
| 93 | - } |
|
| 94 | - else { |
|
| 91 | + } else { |
|
| 95 | 92 | $scenarios[] = $scenario; |
| 96 | 93 | } |
| 97 | 94 | } |
@@ -123,8 +120,9 @@ discard block |
||
| 123 | 120 | private function forkScenario(ScenarioNode $scenario, $variants, $configTag) { |
| 124 | 121 | $scenarios = array(); |
| 125 | 122 | $nonVariantTags = array_filter($scenario->getTags(), function ($val) { |
| 126 | - if (!in_array($val, $this->variantTags)) |
|
| 127 | - return TRUE; |
|
| 123 | + if (!in_array($val, $this->variantTags)) { |
|
| 124 | + return TRUE; |
|
| 125 | + } |
|
| 128 | 126 | } |
| 129 | 127 | ); |
| 130 | 128 | foreach ($variants as $variant) { |
@@ -146,7 +144,9 @@ discard block |
||
| 146 | 144 | foreach ($steps as $step) { |
| 147 | 145 | $newStep = clone $step; |
| 148 | 146 | $newStep->variant = $variant; |
| 149 | - if ($configTag) $newStep->configTag = $configTag; |
|
| 147 | + if ($configTag) { |
|
| 148 | + $newStep->configTag = $configTag; |
|
| 149 | + } |
|
| 150 | 150 | $injectedSteps[] = $newStep; |
| 151 | 151 | } |
| 152 | 152 | return $injectedSteps; |
@@ -45,17 +45,24 @@ |
||
| 45 | 45 | } |
| 46 | 46 | |
| 47 | 47 | public function getConfigSection($tag, $section){ |
| 48 | - if ($this->hasTag($tag)) return $this->configs[$tag]['config'][$section]; |
|
| 48 | + if ($this->hasTag($tag)) { |
|
| 49 | + return $this->configs[$tag]['config'][$section]; |
|
| 50 | + } |
|
| 49 | 51 | return NULL; |
| 50 | 52 | } |
| 51 | 53 | |
| 52 | 54 | public function getFilePath($tag){ |
| 53 | - if ($this->hasTag($tag)) return $this->configs[$tag]['path']; |
|
| 55 | + if ($this->hasTag($tag)) { |
|
| 56 | + return $this->configs[$tag]['path']; |
|
| 57 | + } |
|
| 54 | 58 | return NULL; |
| 55 | 59 | } |
| 56 | 60 | |
| 57 | 61 | public function hasTag($tag){ |
| 58 | - if (key_exists($tag, $this->configs)) return true; |
|
| 59 | - else return false; |
|
| 62 | + if (key_exists($tag, $this->configs)) { |
|
| 63 | + return true; |
|
| 64 | + } else { |
|
| 65 | + return false; |
|
| 66 | + } |
|
| 60 | 67 | } |
| 61 | 68 | } |