silverstripe / silverstripe-behat-extension

src/SilverStripe/BehatExtension/Context/SilverStripeContext.php

Doc Comments +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
 	 * When using the "data-title" attribute, ensure not to include double quotes.
 	 *
 	 * @param String $region Region name or CSS selector
-	 * @return MinkElement|null
+	 * @return \Behat\Mink\Element\NodeElement
 	 */
 	public function getRegionObj($region) {
 		// Try to find regions directly by CSS selector.

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

@@ -2,21 +2,15 @@
 
 namespace SilverStripe\BehatExtension\Context;
 
-use Behat\Behat\Context\Step,
-	Behat\Behat\Event\FeatureEvent,
-	Behat\Behat\Event\ScenarioEvent,
-	Behat\Behat\Event\SuiteEvent;
-use Behat\Gherkin\Node\PyStringNode;
+use Behat\Behat\Context\Step;
+use Behat\Behat\Event\ScenarioEvent;
 use Behat\MinkExtension\Context\MinkContext;
-use Behat\Mink\Driver\GoutteDriver,
-	Behat\Mink\Driver\Selenium2Driver,
-	Behat\Mink\Exception\UnsupportedDriverActionException,
-	Behat\Mink\Exception\ElementNotFoundException;
-
+use Behat\Mink\Driver\GoutteDriver;
+use Behat\Mink\Driver\Selenium2Driver;
+use Behat\Mink\Exception\UnsupportedDriverActionException;
+use Behat\Mink\Exception\ElementNotFoundException;
 use SilverStripe\BehatExtension\Context\SilverStripeAwareContextInterface;
 
-use Symfony\Component\Yaml\Yaml;
-
 // Mink etc.
 require_once 'vendor/autoload.php';
 

src/SilverStripe/BehatExtension/Extension.php

Doc Comments +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
     }
 
     /**
-     * @return array
+     * @return Compiler\CoreInitializationPass[]
      */
     public function getCompilerPasses()
     {

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

@@ -2,11 +2,10 @@
 
 namespace SilverStripe\BehatExtension;
 
-use Symfony\Component\Config\FileLocator,
-    Symfony\Component\DependencyInjection\ContainerBuilder,
-    Symfony\Component\DependencyInjection\Loader\YamlFileLoader,
-    Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
-
+use Symfony\Component\Config\FileLocator;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
+use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Behat\Behat\Extension\ExtensionInterface;
 
 /*

src/SilverStripe/BehatExtension/Compiler/CoreInitializationPass.php

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.

appendixes/mysite/code/testing/FakeDatabase.php

Doc Comments +4 added lines

@@ -82,6 +82,10 @@
 		return $return;
 	}
 
+	/**
+	 * @param string $type
+	 * @param string $key
+	 */
 	public function get($type, $key) {
 		$data = $this->getData();
 		$return = (isset($data[$type][$key])) ? FakeObject::create_from_array($data[$type][$key]) : null;

appendixes/mysite/code/testing/SampleFakeManager.php

Doc Comments +6 added lines

@@ -97,6 +97,9 @@ 
 		// Don't initialize client
 	}
 
+	/**
+	 * @param FakeDatabase $db
+	 */
 	public function setDb($db) {
 		$this->db = $db;
 	}
@@ -126,6 +129,9 @@ 
 		// Don't initialize client
 	}
 
+	/**
+	 * @param FakeDatabase $db
+	 */
 	public function setDb($db) {
 		$this->db = $db;
 	}

appendixes/mysite/tests/behat/features/bootstrap/Context/FeatureContext.php

Doc Comments +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
     }
 
     /**
-     * @return FixtureFactory
+     * @return \FixtureFactory
      */
     public function getFixtureFactory() {
         if(!$this->fixtureFactory) {

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

@@ -2,13 +2,13 @@
 
 namespace Mysite\Test\Behaviour;
 
-use SilverStripe\BehatExtension\Context\SilverStripeContext,
-    SilverStripe\BehatExtension\Context\BasicContext,
-    SilverStripe\BehatExtension\Context\LoginContext,
-    SilverStripe\BehatExtension\Context\FixtureContext,
-    SilverStripe\Framework\Test\Behaviour\CmsFormsContext,
-    SilverStripe\Framework\Test\Behaviour\CmsUiContext,
-    SilverStripe\Cms\Test\Behaviour;
+use SilverStripe\BehatExtension\Context\SilverStripeContext;
+use SilverStripe\BehatExtension\Context\BasicContext;
+use SilverStripe\BehatExtension\Context\LoginContext;
+use SilverStripe\BehatExtension\Context\FixtureContext;
+use SilverStripe\Framework\Test\Behaviour\CmsFormsContext;
+use SilverStripe\Framework\Test\Behaviour\CmsUiContext;
+use SilverStripe\Cms\Test\Behaviour;
 
 // PHPUnit
 require_once 'PHPUnit/Autoload.php';