silverstripe / silverstripe-behat-extension

src/SilverStripe/BehatExtension/Compiler/MinkExtensionBaseUrlPass.php

Doc Comments +2 added lines, -1 removed lines

@@ -58,7 +58,7 @@ 
      * Copied logic from Core.php, because it needs to be executed prior to {@link SilverStripeAwareInitializer}.
      *
      * @param String Absolute start path to search upwards from
-     * @return Boolean Absolute path to environment file
+     * @return string|null Absolute path to environment file
      */
     protected function findEnvironmentConfigFile($path) {
         $envPath = null;
@@ -85,6 +85,7 @@ 
      *
      * @param String Absolute start path to search upwards from
      * @param Array Map of paths to host names
+     * @param string $path
      * @return String URL
      */
     protected function findBaseUrlFromMapping($path, $mapping) {

Unused Use Statements +2 added lines, -2 removed lines

@@ -2,8 +2,8 @@
 
 namespace SilverStripe\BehatExtension\Compiler;
 
-use Symfony\Component\DependencyInjection\ContainerBuilder,
-    Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 
 /**
  * Behat\SilverStripe container compilation pass.

src/SilverStripe/BehatExtension/Console/Processor/InitProcessor.php

Unused Use Statements +5 added lines, -7 removed lines

@@ -2,13 +2,11 @@
 
 namespace SilverStripe\BehatExtension\Console\Processor;
 
-use Symfony\Component\DependencyInjection\ContainerInterface,
-    Symfony\Component\Console\Command\Command,
-    Symfony\Component\Console\Input\InputArgument,
-    Symfony\Component\Console\Input\InputInterface,
-    Symfony\Component\Console\Output\OutputInterface,
-    Symfony\Component\Console\Input\InputOption;
-
+use Symfony\Component\DependencyInjection\ContainerInterface;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Input\InputOption;
 use Behat\Behat\Console\Processor\InitProcessor as BaseProcessor;
 
 /**

src/SilverStripe/BehatExtension/Console/Processor/LocatorProcessor.php

Unused Use Statements +5 added lines, -6 removed lines

@@ -2,12 +2,11 @@
 
 namespace SilverStripe\BehatExtension\Console\Processor;
 
-use Symfony\Component\DependencyInjection\ContainerInterface,
-    Symfony\Component\Console\Command\Command,
-    Symfony\Component\Console\Input\InputArgument,
-    Symfony\Component\Console\Input\InputInterface,
-    Symfony\Component\Console\Output\OutputInterface;
-
+use Symfony\Component\DependencyInjection\ContainerInterface;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
 use Behat\Behat\Console\Processor\LocatorProcessor as BaseProcessor;
 
 /**

src/SilverStripe/BehatExtension/Context/BasicContext.php

Doc Comments +2 added lines, -1 removed lines

@@ -750,7 +750,7 @@ 
      * - fieldset[data-name] table
      * - table caption
      *
-     * @return Behat\Mink\Element\NodeElement
+     * @return \Behat\Mink\Element\NodeElement|null
      */
     protected function getTable($selector) {
         $selector = $this->getSession()->getSelectorsHandler()->xpathLiteral($selector);
@@ -971,6 +971,7 @@ 
 	
 	/**
 	 * We have to catch exceptions and log somehow else otherwise behat falls over
+	 * @param \WebDriver\Exception $e
 	 */
 	protected function logException($e){
 		file_put_contents('php://stderr', 'Exception caught: '.$e);

Unused Use Statements +4 added lines, -8 removed lines

@@ -2,16 +2,12 @@
 
 namespace SilverStripe\BehatExtension\Context;
 
-use Behat\Behat\Context\BehatContext,
-    Behat\Behat\Context\Step,
-    Behat\Behat\Event\StepEvent,
-    Behat\Behat\Event\ScenarioEvent;
-
+use Behat\Behat\Context\BehatContext;
+use Behat\Behat\Context\Step;
+use Behat\Behat\Event\StepEvent;
+use Behat\Behat\Event\ScenarioEvent;
 use Behat\Mink\Driver\Selenium2Driver;
 
-use Behat\Gherkin\Node\PyStringNode,
-    Behat\Gherkin\Node\TableNode;
-
 // PHPUnit
 require_once 'PHPUnit/Autoload.php';
 require_once 'PHPUnit/Framework/Assert/Functions.php';

src/SilverStripe/BehatExtension/Context/ClassGuesser/ModuleContextClassGuesser.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     /**
      * Sets bundle namespace to use for guessing.
      *
-     * @param string $namespace
+     * @param string $namespaceBase
      */
     public function setNamespaceBase($namespaceBase)
     {

src/SilverStripe/BehatExtension/Context/EmailContext.php

Unused Use Statements +4 added lines, -9 removed lines

@@ -2,15 +2,10 @@
 
 namespace SilverStripe\BehatExtension\Context;
 
-use Behat\Behat\Context\ClosuredContextInterface,
-Behat\Behat\Context\TranslatedContextInterface,
-Behat\Behat\Context\BehatContext,
-Behat\Behat\Context\Step,
-Behat\Behat\Event\FeatureEvent,
-Behat\Behat\Event\ScenarioEvent,
-Behat\Behat\Exception\PendingException;
-use Behat\Gherkin\Node\PyStringNode,
-Behat\Gherkin\Node\TableNode;
+use Behat\Behat\Context\BehatContext;
+use Behat\Behat\Context\Step;
+use Behat\Behat\Event\ScenarioEvent;
+use Behat\Gherkin\Node\TableNode;
 use Symfony\Component\DomCrawler\Crawler;
 
 // PHPUnit

src/SilverStripe/BehatExtension/Context/FixtureContext.php

Doc Comments +5 added lines

@@ -545,6 +545,10 @@ 
 		return $data;
 	}
 
+	/**
+	 * @param string $class
+	 * @param string $identifier
+	 */
 	protected function prepareAsset($class, $identifier, $data = null) {
 		if(!$data) $data = array();
 		$relativeTargetPath = (isset($data['Filename'])) ? $data['Filename'] : $identifier;
@@ -609,6 +613,7 @@ 
 	 * 
 	 * @param String Class name
 	 * @param Array Map of field names or aliases to their values.
+	 * @param string $class
 	 * @return Array Map of actual object properties to their values.
 	 */
 	protected function convertFields($class, $fields) {

Unused Use Statements +5 added lines, -5 removed lines

@@ -2,11 +2,11 @@
 
 namespace SilverStripe\BehatExtension\Context;
 
-use Behat\Behat\Context\BehatContext,
-	Behat\Behat\Event\ScenarioEvent,
-	Behat\Gherkin\Node\PyStringNode,
-	Behat\Gherkin\Node\TableNode,
-	SilverStripe\Filesystem\Storage\AssetStore;
+use Behat\Behat\Context\BehatContext;
+use Behat\Behat\Event\ScenarioEvent;
+use Behat\Gherkin\Node\PyStringNode;
+use Behat\Gherkin\Node\TableNode;
+use SilverStripe\Filesystem\Storage\AssetStore;
 
 // PHPUnit
 require_once 'PHPUnit/Autoload.php';

src/SilverStripe/BehatExtension/Context/Initializer/SilverStripeAwareInitializer.php

Unused Use Statements +2 added lines, -3 removed lines

@@ -2,9 +2,8 @@
 
 namespace SilverStripe\BehatExtension\Context\Initializer;
 
-use Behat\Behat\Context\Initializer\InitializerInterface,
-Behat\Behat\Context\ContextInterface;
-
+use Behat\Behat\Context\Initializer\InitializerInterface;
+use Behat\Behat\Context\ContextInterface;
 use SilverStripe\BehatExtension\Context\SilverStripeAwareContextInterface;
 
 /*

src/SilverStripe/BehatExtension/Context/LoginContext.php

Doc Comments +2 added lines

@@ -121,6 +121,8 @@
 
      /**
      * @When /^I log in with "(?<username>[^"]*)" and "(?<password>[^"]*)"$/
+     * @param string $email
+     * @param string $password
      */
     public function stepILogInWith($email, $password)
     {        

Unused Use Statements +2 added lines, -7 removed lines

@@ -2,13 +2,8 @@
 
 namespace SilverStripe\BehatExtension\Context;
 
-use Behat\Behat\Context\ClosuredContextInterface,
-Behat\Behat\Context\TranslatedContextInterface,
-Behat\Behat\Context\BehatContext,
-Behat\Behat\Context\Step,
-Behat\Behat\Exception\PendingException;
-use Behat\Gherkin\Node\PyStringNode,
-Behat\Gherkin\Node\TableNode;
+use Behat\Behat\Context\BehatContext;
+use Behat\Behat\Context\Step;
 
 // PHPUnit
 require_once 'PHPUnit/Autoload.php';