ciandt-dev / behat-placeholders-extension

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

Doc Comments -1 removed lines

@@ -32,7 +32,6 @@
 
   /**
      * 
-     * @param type $config_files
      * @todo user %paths.base% value
      */
     private function loadConfigFiles($configs_mapping){

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;
+        }
     }
 }

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

Doc Comments +4 added lines

@@ -104,6 +104,10 @@
                 $step->getKeyword(), $text, $arguments, $step->getLine(), $step->getKeywordType());
     }
 
+    /**
+     * @param string $string
+     * @param string $env
+     */
     private function replacePlaceholders($string, $var, $env) {
         preg_match_all('/\${(?P<key>[^}]+)}/i', $string, $placeholders, PREG_SET_ORDER);
         foreach ($placeholders as $placeholder) {

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

Doc Comments +6 added lines

@@ -120,6 +120,9 @@ 
   }
 
 
+  /**
+   * @param string[] $variants
+   */
   private function forkScenario(ScenarioNode $scenario, $variants, $configTag) {
     $scenarios = array();
     $nonVariantTags = array_filter($scenario->getTags(), function ($val) {
@@ -141,6 +144,9 @@ 
     return $scenarios;
   }
 
+  /**
+   * @param StepNode[] $steps
+   */
   private function injectParametersOnSteps($steps, $variant, $configTag) {
     $injectedSteps = array();
     foreach ($steps as $step) {

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;