ciandt-dev / behat-placeholders-extension

src/Ciandt/Behat/PlaceholdersExtension/Tester/ScenarioBranchingFeatureTester.php

Spacing +7 added lines, -7 removed lines

@@ -46,7 +46,7 @@ 
    *
    * @param SpecificationTester $baseTester
    */
-  public function __construct(SpecificationTester $baseTester, $variantTags, ConfigsRepository $configsRepo ) {
+  public function __construct(SpecificationTester $baseTester, $variantTags, ConfigsRepository $configsRepo) {
     $this->baseTester = $baseTester;
     $this->variantTags = $variantTags;
     $this->configsRepo = $configsRepo;
@@ -79,11 +79,11 @@ 
     foreach ($feature->getScenarios() as $scenario) {
       $configTag = $this->getScenarioConfigTag($scenario);
       $scenarioVariants = array_intersect($scenario->getTags(), $this->variantTags);
-      if (count($scenarioVariants) == 1){
+      if (count($scenarioVariants) == 1) {
         $injectedScenario = new ScenarioNode(
           $scenario->getTitle(),
           $scenario->getTags(),
-          $this->injectParametersOnSteps($scenario->getSteps(), end($scenarioVariants),$configTag),
+          $this->injectParametersOnSteps($scenario->getSteps(), end($scenarioVariants), $configTag),
           $scenario->getKeyword(),
           $scenario->getLine());
         $scenarios[] = $injectedScenario;
@@ -101,12 +101,12 @@ 
     );
   }
   
-  private function getScenarioConfigTag(ScenarioNode $scenario){
+  private function getScenarioConfigTag(ScenarioNode $scenario) {
     $availableTags = $this->configsRepo->getTags();
-    $configTags = array_intersect($scenario->getTags(),$availableTags );
+    $configTags = array_intersect($scenario->getTags(), $availableTags);
     
     if (count($configTags) > 1) {
-      throw new \Exception ("Scenario {$scenario->getTitle()}"
+      throw new \Exception("Scenario {$scenario->getTitle()}"
       . " should have only ONE of the following Placeholder"
         . " File Tags: " . implode(',', $availableTags));
     }
@@ -122,7 +122,7 @@ 
 
   private function forkScenario(ScenarioNode $scenario, $variants, $configTag) {
     $scenarios = array();
-    $nonVariantTags = array_filter($scenario->getTags(), function ($val) {
+    $nonVariantTags = array_filter($scenario->getTags(), function($val) {
       if (!in_array($val, $this->variantTags))
         return TRUE;
     }

Braces +9 added lines, -9 removed lines

@@ -68,8 +68,7 @@ 
     $variantTags = $this->variantTags;
     if ($variantTags) {
       return $tester->test($env, $this->reconstructFeature($feature), $skip);
-    }
-    else {
+    } else {
       return $tester->test($env, $feature, $skip);
     }
   }
@@ -87,11 +86,9 @@ 
           $scenario->getKeyword(),
           $scenario->getLine());
         $scenarios[] = $injectedScenario;
-      }
-      elseif (count($scenarioVariants) > 1) {
+      } elseif (count($scenarioVariants) > 1) {
         $scenarios = array_merge($scenarios, $this->forkScenario($scenario, $scenarioVariants, $configTag));
-      }
-      else {
+      } else {
         $scenarios[] = $scenario;
       }
     }
@@ -123,8 +120,9 @@ 
   private function forkScenario(ScenarioNode $scenario, $variants, $configTag) {
     $scenarios = array();
     $nonVariantTags = array_filter($scenario->getTags(), function ($val) {
-      if (!in_array($val, $this->variantTags))
-        return TRUE;
+      if (!in_array($val, $this->variantTags)) {
+              return TRUE;
+      }
     }
     );
     foreach ($variants as $variant) {
@@ -146,7 +144,9 @@ 
     foreach ($steps as $step) {
       $newStep = clone $step;
       $newStep->variant = $variant;
-      if ($configTag) $newStep->configTag = $configTag;
+      if ($configTag) {
+        $newStep->configTag = $configTag;
+      }
       $injectedSteps[] = $newStep;
     }
     return $injectedSteps;

src/Ciandt/Behat/PlaceholdersExtension/Config/ConfigsRepository.php

Spacing +6 added lines, -6 removed lines

@@ -25,7 +25,7 @@ 
    * @return string[]
    * @todo read configs and also bring alternative @config:section tags
    */
-  public function getTags(){
+  public function getTags() {
     return array_keys($this->configs);
   }
   
@@ -35,26 +35,26 @@ 
      * @param type $config_files
      * @todo user %paths.base% value
      */
-    private function loadConfigFiles($configs_mapping){
+    private function loadConfigFiles($configs_mapping) {
       $placeholder_maps = array();
-      foreach ($configs_mapping as $tag => $file_path){
+      foreach ($configs_mapping as $tag => $file_path) {
         $placeholder_maps[$tag]['config'] = Yaml::parse(file_get_contents($file_path));
         $placeholder_maps[$tag]['path'] = $file_path;
       }
       $this->configs = $placeholder_maps;
     }
     
-    public function getConfigSection($tag, $section){
+    public function getConfigSection($tag, $section) {
       if ($this->hasTag($tag)) return $this->configs[$tag]['config'][$section];
       return NULL;
     }
     
-    public function getFilePath($tag){
+    public function getFilePath($tag) {
       if ($this->hasTag($tag)) return $this->configs[$tag]['path'];
       return NULL;
     }
     
-    public function hasTag($tag){
+    public function hasTag($tag) {
         if (key_exists($tag, $this->configs)) return true;
         else return false;
     }

Braces +11 added lines, -4 removed lines

@@ -45,17 +45,24 @@
     }
     
     public function getConfigSection($tag, $section){
-      if ($this->hasTag($tag)) return $this->configs[$tag]['config'][$section];
+      if ($this->hasTag($tag)) {
+        return $this->configs[$tag]['config'][$section];
+      }
       return NULL;
     }
     
     public function getFilePath($tag){
-      if ($this->hasTag($tag)) return $this->configs[$tag]['path'];
+      if ($this->hasTag($tag)) {
+        return $this->configs[$tag]['path'];
+      }
       return NULL;
     }
     
     public function hasTag($tag){
-        if (key_exists($tag, $this->configs)) return true;
-        else return false;
+        if (key_exists($tag, $this->configs)) {
+          return true;
+        } else {
+          return false;
+        }
     }
 }