@@ -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 | } |