cross-solution / YAWIK

module/Auth/src/Auth/Factory/Service/UserUniqueTokenGeneratorFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      * @param  string             $requestedName
      * @param  null|array         $options
      *
-     * @return object
+     * @return UserUniqueTokenGenerator
      * @throws ServiceNotFoundException if unable to resolve the service.
      * @throws ServiceNotCreatedException if an exception is raised when
      *     creating a service.

module/Behat/src/CommonContextTrait.php

Doc Comments +4 added lines, -1 removed lines

@@ -54,6 +54,9 @@ 
 		return $this->coreContext->generateUrl($url);
 	}
 	
+	/**
+	 * @param string $url
+	 */
 	public function visit($url)
 	{
 		$this->coreContext->iVisit($this->generateUrl($url));
@@ -69,7 +72,7 @@ 
 	}
 	
 	/**
-	 * @param $id
+	 * @param string $id
 	 *
 	 * @return RepositoryInterface
 	 */

Indentation +60 added lines, -60 removed lines

@@ -16,73 +16,73 @@
 
 trait CommonContextTrait
 {
-	/**
-	 * @var MinkContext
-	 */
-	protected $minkContext;
+    /**
+     * @var MinkContext
+     */
+    protected $minkContext;
 	
-	/**
-	 * @var CoreContext
-	 */
-	protected $coreContext;
+    /**
+     * @var CoreContext
+     */
+    protected $coreContext;
 	
-	/**
-	 * @var UserContext
-	 */
-	protected $userContext;
+    /**
+     * @var UserContext
+     */
+    protected $userContext;
 	
-	/**
-	 * @var SummaryFormContext
-	 */
-	protected $summaryFormContext;
+    /**
+     * @var SummaryFormContext
+     */
+    protected $summaryFormContext;
 	
-	/**
-	 * @BeforeScenario
-	 *
-	 * @param BeforeScenarioScope $scope
-	 */
-	public function gatherContexts(BeforeScenarioScope $scope)
-	{
-		$this->minkContext = $scope->getEnvironment()->getContext(MinkContext::class);
-		$this->coreContext = $scope->getEnvironment()->getContext(CoreContext::class);
-		$this->userContext = $scope->getEnvironment()->getContext(UserContext::class);
-		$this->summaryFormContext = $scope->getEnvironment()->getContext(SummaryFormContext::class);
-	}
+    /**
+     * @BeforeScenario
+     *
+     * @param BeforeScenarioScope $scope
+     */
+    public function gatherContexts(BeforeScenarioScope $scope)
+    {
+        $this->minkContext = $scope->getEnvironment()->getContext(MinkContext::class);
+        $this->coreContext = $scope->getEnvironment()->getContext(CoreContext::class);
+        $this->userContext = $scope->getEnvironment()->getContext(UserContext::class);
+        $this->summaryFormContext = $scope->getEnvironment()->getContext(SummaryFormContext::class);
+    }
 	
-	public function generateUrl($url)
-	{
-		return $this->coreContext->generateUrl($url);
-	}
+    public function generateUrl($url)
+    {
+        return $this->coreContext->generateUrl($url);
+    }
 	
-	public function visit($url)
-	{
-		$this->coreContext->iVisit($this->generateUrl($url));
-	}
+    public function visit($url)
+    {
+        $this->coreContext->iVisit($this->generateUrl($url));
+    }
 	
-	/**
-	 * @param $id
-	 * @return mixed|object
-	 */
-	public function getService($id)
-	{
-		return $this->coreContext->getServiceManager()->get($id);
-	}
+    /**
+     * @param $id
+     * @return mixed|object
+     */
+    public function getService($id)
+    {
+        return $this->coreContext->getServiceManager()->get($id);
+    }
 	
-	/**
-	 * @param $id
-	 *
-	 * @return RepositoryInterface
-	 */
-	public function getRepository($id)
-	{
-		return $this->coreContext->getRepositories()->get($id);
-	}
+    /**
+     * @param $id
+     *
+     * @return RepositoryInterface
+     */
+    public function getRepository($id)
+    {
+        return $this->coreContext->getRepositories()->get($id);
+    }
 	
-	/**
-	 * @return \Auth\Entity\User
-	 */
-	public function getCurrentUser()
-	{
-		return $this->userContext->getCurrentUser();
-	}
+    /**
+     * @return \Auth\Entity\User
+     */
+    public function getCurrentUser()
+    {
+        return $this->userContext->getCurrentUser();
+    }
 }
\ No newline at end of file

module/Behat/src/CoreContext.php

Doc Comments +7 added lines, -4 removed lines

@@ -76,8 +76,7 @@ 
 	}
 	
 	/**
-	 * @param $name
-	 * @param array $params
+	 * @param string $name
 	 *
 	 * @return string
 	 */
@@ -88,6 +87,7 @@ 
 	
 	/**
 	 * @When /^I hover over the element "([^"]*)"$/
+	 * @param string $locator
 	 */
 	public function iHoverOverTheElement($locator)
 	{
@@ -163,6 +163,9 @@ 
 		$this->getSession()->switchToWindow('main_window');
 	}
 	
+	/**
+	 * @param string $url
+	 */
 	public function iVisit($url)
 	{
 		$this->minkContext->getSession()->visit($url);
@@ -233,10 +236,10 @@ 
 	}
 	
 	/**
-	 * @param $locator
+	 * @param string $locator
 	 * @param string $selector
 	 *
-	 * @return \Behat\Mink\Element\NodeElement|mixed|null
+	 * @return \Behat\Mink\Element\NodeElement|null
 	 */
 	public function getElement($locator,$selector='css')
 	{

Indentation +196 added lines, -196 removed lines

@@ -22,244 +22,244 @@
  */
 class CoreContext extends RawMinkContext
 {
-	static protected $application;
+    static protected $application;
 	
-	/**
-	 * @var MinkContext
-	 */
-	protected $minkContext;
+    /**
+     * @var MinkContext
+     */
+    protected $minkContext;
 	
-	/**
-	 * @BeforeScenario
-	 * @param BeforeScenarioScope $scope
-	 */
-	public function gatherContexts(BeforeScenarioScope $scope)
-	{
-		$this->minkContext = $scope->getEnvironment()->getContext(MinkContext::class);
-	}
+    /**
+     * @BeforeScenario
+     * @param BeforeScenarioScope $scope
+     */
+    public function gatherContexts(BeforeScenarioScope $scope)
+    {
+        $this->minkContext = $scope->getEnvironment()->getContext(MinkContext::class);
+    }
 	
-	/**
-	 * @return Application
-	 */
-	public function getApplication()
-	{
-		if(!is_object(static::$application)){
-			$configFile = realpath(__DIR__.'/../../../config/config.php');
-			$config = include($configFile);
-			static::$application = Application::init($config);
-		}
-		return static::$application;
-	}
+    /**
+     * @return Application
+     */
+    public function getApplication()
+    {
+        if(!is_object(static::$application)){
+            $configFile = realpath(__DIR__.'/../../../config/config.php');
+            $config = include($configFile);
+            static::$application = Application::init($config);
+        }
+        return static::$application;
+    }
 	
-	/**
-	 * @return \Zend\ServiceManager\ServiceManager
-	 */
-	public function getServiceManager()
-	{
-		return $this->getApplication()->getServiceManager();
-	}
+    /**
+     * @return \Zend\ServiceManager\ServiceManager
+     */
+    public function getServiceManager()
+    {
+        return $this->getApplication()->getServiceManager();
+    }
 	
-	/**
-	 * @return \Zend\EventManager\EventManagerInterface
-	 */
-	public function getEventManager()
-	{
-		return $this->getApplication()->getEventManager();
-	}
+    /**
+     * @return \Zend\EventManager\EventManagerInterface
+     */
+    public function getEventManager()
+    {
+        return $this->getApplication()->getEventManager();
+    }
 	
-	/**
-	 * @return RepositoryService
-	 */
-	public function getRepositories()
-	{
-		return $this->getServiceManager()->get('repositories');
-	}
+    /**
+     * @return RepositoryService
+     */
+    public function getRepositories()
+    {
+        return $this->getServiceManager()->get('repositories');
+    }
 	
-	/**
-	 * @param $name
-	 * @param array $params
-	 *
-	 * @return string
-	 */
-	public function generateUrl($name)
-	{
-		return $this->minkContext->locatePath($name);
-	}
+    /**
+     * @param $name
+     * @param array $params
+     *
+     * @return string
+     */
+    public function generateUrl($name)
+    {
+        return $this->minkContext->locatePath($name);
+    }
 	
-	/**
-	 * @When /^I hover over the element "([^"]*)"$/
-	 */
-	public function iHoverOverTheElement($locator)
-	{
-		$session = $this->minkContext->getSession(); // get the mink session
-		$element = $session->getPage()->find('css', $locator); // runs the actual query and returns the element
+    /**
+     * @When /^I hover over the element "([^"]*)"$/
+     */
+    public function iHoverOverTheElement($locator)
+    {
+        $session = $this->minkContext->getSession(); // get the mink session
+        $element = $session->getPage()->find('css', $locator); // runs the actual query and returns the element
 		
-		// errors must not pass silently
-		if (null === $element) {
-			throw new \InvalidArgumentException(sprintf('Could not evaluate CSS selector: "%s"', $locator));
-		}
+        // errors must not pass silently
+        if (null === $element) {
+            throw new \InvalidArgumentException(sprintf('Could not evaluate CSS selector: "%s"', $locator));
+        }
 		
-		// ok, let's hover it
-		$element->mouseOver();
-	}
+        // ok, let's hover it
+        $element->mouseOver();
+    }
 	
-	/**
-	 * @Given /^I wait for (\d+) seconds$/
-	 */
-	public function iWaitForSecond($second)
-	{
-		sleep($second);
-	}
+    /**
+     * @Given /^I wait for (\d+) seconds$/
+     */
+    public function iWaitForSecond($second)
+    {
+        sleep($second);
+    }
 	
-	/**
-	 * @Then /^I wait for the ajax response$/
-	 */
-	public function iWaitForTheAjaxResponse()
-	{
-		$this->getSession()->wait(5000, '(0 === jQuery.active)');
-	}
+    /**
+     * @Then /^I wait for the ajax response$/
+     */
+    public function iWaitForTheAjaxResponse()
+    {
+        $this->getSession()->wait(5000, '(0 === jQuery.active)');
+    }
 	
-	/**
-	 * Some forms do not have a Submit button just pass the ID
-	 *
-	 * @Given /^I submit the form with id "([^"]*)"$/
-	 */
-	public function iSubmitTheFormWithId($arg)
-	{
-		$node = $this->minkContext->getSession()->getPage()->find('css', $arg);
-		if($node) {
-			$this->minkContext->getSession()->executeScript("jQuery('$arg').submit();");
-		} else {
-			throw new \Exception('Element not found');
-		}
-	}
+    /**
+     * Some forms do not have a Submit button just pass the ID
+     *
+     * @Given /^I submit the form with id "([^"]*)"$/
+     */
+    public function iSubmitTheFormWithId($arg)
+    {
+        $node = $this->minkContext->getSession()->getPage()->find('css', $arg);
+        if($node) {
+            $this->minkContext->getSession()->executeScript("jQuery('$arg').submit();");
+        } else {
+            throw new \Exception('Element not found');
+        }
+    }
 	
-	/**
-	 * @Then I switch to popup :name
-	 *
-	 * @param $name
-	 */
-	public function iSwitchToPopup($name)
-	{
-		$this->iSetMainWindowName();
-		$this->getSession()->switchToWindow($name);
-	}
+    /**
+     * @Then I switch to popup :name
+     *
+     * @param $name
+     */
+    public function iSwitchToPopup($name)
+    {
+        $this->iSetMainWindowName();
+        $this->getSession()->switchToWindow($name);
+    }
 	
-	/**
-	 * @Then I set main window name
-	 */
-	public function iSetMainWindowName()
-	{
-		$window_name = 'main_window';
-		$script = 'window.name = "' . $window_name . '"';
-		$this->getSession()->executeScript($script);
-	}
+    /**
+     * @Then I set main window name
+     */
+    public function iSetMainWindowName()
+    {
+        $window_name = 'main_window';
+        $script = 'window.name = "' . $window_name . '"';
+        $this->getSession()->executeScript($script);
+    }
 	
-	/**
-	 * @Then I switch back to main window
-	 */
-	public function iSwitchBackToMainWindow()
-	{
-		$this->getSession()->switchToWindow('main_window');
-	}
+    /**
+     * @Then I switch back to main window
+     */
+    public function iSwitchBackToMainWindow()
+    {
+        $this->getSession()->switchToWindow('main_window');
+    }
 	
-	public function iVisit($url)
-	{
-		$this->minkContext->getSession()->visit($url);
-	}
+    public function iVisit($url)
+    {
+        $this->minkContext->getSession()->visit($url);
+    }
 	
-	/**
-	 * @When I scroll :selector into view
-	 *
-	 * @param string $selector Allowed selectors: #id, .className, //xpath
-	 *
-	 * @throws \Exception
-	 */
-	public function scrollIntoView($selector)
-	{
-		$locator = substr($selector, 0, 1);
+    /**
+     * @When I scroll :selector into view
+     *
+     * @param string $selector Allowed selectors: #id, .className, //xpath
+     *
+     * @throws \Exception
+     */
+    public function scrollIntoView($selector)
+    {
+        $locator = substr($selector, 0, 1);
 		
-		switch ($locator) {
-			case '/' : // XPath selector
-				$function = <<<JS
+        switch ($locator) {
+            case '/' : // XPath selector
+                $function = <<<JS
 (function(){
   var elem = document.evaluate($selector, document, null, XPathResult.FIRST_ORDERED_NODE_TYPE, null).singleNodeValue;
   elem.scrollIntoView(false);
 })()
 JS;
-				break;
+                break;
 			
-			case '#' : // ID selector
-				$selector = substr($selector, 1);
-				$function = <<<JS
+            case '#' : // ID selector
+                $selector = substr($selector, 1);
+                $function = <<<JS
 (function(){
   var elem = document.getElementById("$selector");
   elem.scrollIntoView(false);
 })()
 JS;
-				break;
+                break;
 			
-			case '.' : // Class selector
-				$selector = substr($selector, 1);
-				$function = <<<JS
+            case '.' : // Class selector
+                $selector = substr($selector, 1);
+                $function = <<<JS
 (function(){
   var elem = document.getElementsByClassName("$selector");
   elem[0].scrollIntoView(false);
 })()
 JS;
-				break;
+                break;
 			
-			default:
-				throw new \Exception(__METHOD__ . ' Couldn\'t find selector: ' . $selector . ' - Allowed selectors: #id, .className, //xpath');
-				break;
-		}
+            default:
+                throw new \Exception(__METHOD__ . ' Couldn\'t find selector: ' . $selector . ' - Allowed selectors: #id, .className, //xpath');
+                break;
+        }
 		
-		try {
-			$this->getSession()->executeScript($function);
-		} catch (\Exception $e) {
-			throw new \Exception(__METHOD__ . ' failed'. ' Message: for this locator:"'.$selector.'"');
-		}
-	}
+        try {
+            $this->getSession()->executeScript($function);
+        } catch (\Exception $e) {
+            throw new \Exception(__METHOD__ . ' failed'. ' Message: for this locator:"'.$selector.'"');
+        }
+    }
 	
 	
-	/**
-	 * @When I click location selector
-	 */
-	public function iClickLocationSelector()
-	{
-		$locator = '#jobBase-geoLocation-span .select2';
-		$element = $this->getElement($locator);
-		$element->click();
-	}
+    /**
+     * @When I click location selector
+     */
+    public function iClickLocationSelector()
+    {
+        $locator = '#jobBase-geoLocation-span .select2';
+        $element = $this->getElement($locator);
+        $element->click();
+    }
 	
-	/**
-	 * @param $locator
-	 * @param string $selector
-	 *
-	 * @return \Behat\Mink\Element\NodeElement|mixed|null
-	 */
-	public function getElement($locator,$selector='css')
-	{
-		$page = $this->minkContext->getSession()->getPage();
-		$element = $page->find('css',$locator);
-		return $element;
-	}
+    /**
+     * @param $locator
+     * @param string $selector
+     *
+     * @return \Behat\Mink\Element\NodeElement|mixed|null
+     */
+    public function getElement($locator,$selector='css')
+    {
+        $page = $this->minkContext->getSession()->getPage();
+        $element = $page->find('css',$locator);
+        return $element;
+    }
 	
-	/**
-	 * @When I fill in location search with :term
-	 * @param $term
-	 */
-	public function iFillInLocationSearch($term)
-	{
-		$locator = '.select2-container--open .select2-search__field';
-		$element = $this->getElement($locator);
-		$element->focus();
-		$element->setValue($term);
-		$this->iWaitForTheAjaxResponse();
-	}
+    /**
+     * @When I fill in location search with :term
+     * @param $term
+     */
+    public function iFillInLocationSearch($term)
+    {
+        $locator = '.select2-container--open .select2-search__field';
+        $element = $this->getElement($locator);
+        $element->focus();
+        $element->setValue($term);
+        $this->iWaitForTheAjaxResponse();
+    }
 	
-	public function iClickOn()
-	{
+    public function iClickOn()
+    {
 	
-	}
+    }
 }
\ No newline at end of file

Spacing +7 added lines, -7 removed lines

@@ -43,7 +43,7 @@ 
 	 */
 	public function getApplication()
 	{
-		if(!is_object(static::$application)){
+		if (!is_object(static::$application)) {
 			$configFile = realpath(__DIR__.'/../../../config/config.php');
 			$config = include($configFile);
 			static::$application = Application::init($config);
@@ -127,7 +127,7 @@ 
 	public function iSubmitTheFormWithId($arg)
 	{
 		$node = $this->minkContext->getSession()->getPage()->find('css', $arg);
-		if($node) {
+		if ($node) {
 			$this->minkContext->getSession()->executeScript("jQuery('$arg').submit();");
 		} else {
 			throw new \Exception('Element not found');
@@ -151,7 +151,7 @@ 
 	public function iSetMainWindowName()
 	{
 		$window_name = 'main_window';
-		$script = 'window.name = "' . $window_name . '"';
+		$script = 'window.name = "'.$window_name.'"';
 		$this->getSession()->executeScript($script);
 	}
 	
@@ -210,14 +210,14 @@ 
 				break;
 			
 			default:
-				throw new \Exception(__METHOD__ . ' Couldn\'t find selector: ' . $selector . ' - Allowed selectors: #id, .className, //xpath');
+				throw new \Exception(__METHOD__.' Couldn\'t find selector: '.$selector.' - Allowed selectors: #id, .className, //xpath');
 				break;
 		}
 		
 		try {
 			$this->getSession()->executeScript($function);
 		} catch (\Exception $e) {
-			throw new \Exception(__METHOD__ . ' failed'. ' Message: for this locator:"'.$selector.'"');
+			throw new \Exception(__METHOD__.' failed'.' Message: for this locator:"'.$selector.'"');
 		}
 	}
 	
@@ -238,10 +238,10 @@ 
 	 *
 	 * @return \Behat\Mink\Element\NodeElement|mixed|null
 	 */
-	public function getElement($locator,$selector='css')
+	public function getElement($locator, $selector = 'css')
 	{
 		$page = $this->minkContext->getSession()->getPage();
-		$element = $page->find('css',$locator);
+		$element = $page->find('css', $locator);
 		return $element;
 	}
 	

module/Behat/src/Select2Context.php

Doc Comments +2 added lines, -1 removed lines

@@ -31,6 +31,7 @@ 
 	 *
 	 * @When /^(?:|I )fill in select2 "(?P<field>(?:[^"]|\\")*)" with "(?P<value>(?:[^"]|\\")*)"$/
 	 * @When /^(?:|I )fill in select2 "(?P<value>(?:[^"]|\\")*)" for "(?P<field>(?:[^"]|\\")*)"$/
+	 * @param string $field
 	 */
 	public function iFillInSelect2Field($field, $value)
 	{
@@ -42,7 +43,7 @@ 
 	
 	/**
 	 * @When I fill in select2 search :field with :search and I choose :choice
-	 * @param $field
+	 * @param string $field
 	 * @param $value
 	 */
 	public function iFillInSelect2FieldWith($field,$search,$choice=null)

Indentation +111 added lines, -111 removed lines

@@ -24,130 +24,130 @@
  */
 class Select2Context extends RawMinkContext implements Context
 {
-	protected $timeout = 5;
+    protected $timeout = 5;
 	
-	/**
-	 * Fills in Select2 field with specified
-	 *
-	 * @When /^(?:|I )fill in select2 "(?P<field>(?:[^"]|\\")*)" with "(?P<value>(?:[^"]|\\")*)"$/
-	 * @When /^(?:|I )fill in select2 "(?P<value>(?:[^"]|\\")*)" for "(?P<field>(?:[^"]|\\")*)"$/
-	 */
-	public function iFillInSelect2Field($field, $value)
-	{
-		$page = $this->getSession()->getPage();
+    /**
+     * Fills in Select2 field with specified
+     *
+     * @When /^(?:|I )fill in select2 "(?P<field>(?:[^"]|\\")*)" with "(?P<value>(?:[^"]|\\")*)"$/
+     * @When /^(?:|I )fill in select2 "(?P<value>(?:[^"]|\\")*)" for "(?P<field>(?:[^"]|\\")*)"$/
+     */
+    public function iFillInSelect2Field($field, $value)
+    {
+        $page = $this->getSession()->getPage();
 		
-		$this->openField($page, $field);
-		$this->selectValue($page, $field, $value, $this->timeout);
-	}
+        $this->openField($page, $field);
+        $this->selectValue($page, $field, $value, $this->timeout);
+    }
 	
-	/**
-	 * @When I fill in select2 search :field with :search and I choose :choice
-	 * @param $field
-	 * @param $value
-	 */
-	public function iFillInSelect2FieldWith($field,$search,$choice=null)
-	{
-		$page = $this->getSession()->getPage();
-		$this->openField($page, $field);
-		$this->fillSearchField($page,$field,$search);
-		$this->selectValue($page, $field, $choice);
-	}
+    /**
+     * @When I fill in select2 search :field with :search and I choose :choice
+     * @param $field
+     * @param $value
+     */
+    public function iFillInSelect2FieldWith($field,$search,$choice=null)
+    {
+        $page = $this->getSession()->getPage();
+        $this->openField($page, $field);
+        $this->fillSearchField($page,$field,$search);
+        $this->selectValue($page, $field, $choice);
+    }
 	
-	/**
-	 * Fill Select2 search field
-	 *
-	 * @param DocumentElement $page
-	 * @param string          $field
-	 * @param string          $value
-	 * @throws \Exception
-	 */
-	private function fillSearchField(DocumentElement $page, $field, $value)
-	{
-		$driver = $this->getSession()->getDriver();
-		if ('Behat\Mink\Driver\Selenium2Driver' === get_class($driver)) {
-			// Can't use `$this->getSession()->getPage()->find()` because of https://github.com/minkphp/MinkSelenium2Driver/issues/188
+    /**
+     * Fill Select2 search field
+     *
+     * @param DocumentElement $page
+     * @param string          $field
+     * @param string          $value
+     * @throws \Exception
+     */
+    private function fillSearchField(DocumentElement $page, $field, $value)
+    {
+        $driver = $this->getSession()->getDriver();
+        if ('Behat\Mink\Driver\Selenium2Driver' === get_class($driver)) {
+            // Can't use `$this->getSession()->getPage()->find()` because of https://github.com/minkphp/MinkSelenium2Driver/issues/188
 			
-			$element = $page->find('css','.select2-container--open .select2-search__field');
-			$xpath = $element->getXpath();
-			$select2Input = $this->getSession()
-				->getDriver()
-				->getWebDriverSession()
-				->element('xpath',$xpath)
-				//->element('xpath', "//html/descendant-or-self::*[@class and contains(concat(' ', normalize-space(@class), ' '), ' select2-search__field ')]")
-			;
-			if (!$select2Input) {
-				throw new \Exception(sprintf('No field "%s" found', $field));
-			}
+            $element = $page->find('css','.select2-container--open .select2-search__field');
+            $xpath = $element->getXpath();
+            $select2Input = $this->getSession()
+                ->getDriver()
+                ->getWebDriverSession()
+                ->element('xpath',$xpath)
+                //->element('xpath', "//html/descendant-or-self::*[@class and contains(concat(' ', normalize-space(@class), ' '), ' select2-search__field ')]")
+            ;
+            if (!$select2Input) {
+                throw new \Exception(sprintf('No field "%s" found', $field));
+            }
 			
-			$select2Input->postValue(['value' => [$value]]);
-		} else {
-			$select2Input = $page->find('css', '.select2-search__field');
-			if (!$select2Input) {
-				throw new \Exception(sprintf('No input found for "%s"', $field));
-			}
-			$select2Input->setValue($value);
-		}
+            $select2Input->postValue(['value' => [$value]]);
+        } else {
+            $select2Input = $page->find('css', '.select2-search__field');
+            if (!$select2Input) {
+                throw new \Exception(sprintf('No input found for "%s"', $field));
+            }
+            $select2Input->setValue($value);
+        }
 		
-		$this->waitForLoadingResults($this->timeout);
-	}
+        $this->waitForLoadingResults($this->timeout);
+    }
 	
-	/**
-	 * Select value in choice list
-	 *
-	 * @param DocumentElement $page
-	 * @param string          $field
-	 * @param string          $value
-	 * @param int             $time
-	 * @throws \Exception
-	 */
-	private function selectValue(DocumentElement $page, $field, $value, $time=5)
-	{
-		$this->waitForLoadingResults($time);
+    /**
+     * Select value in choice list
+     *
+     * @param DocumentElement $page
+     * @param string          $field
+     * @param string          $value
+     * @param int             $time
+     * @throws \Exception
+     */
+    private function selectValue(DocumentElement $page, $field, $value, $time=5)
+    {
+        $this->waitForLoadingResults($time);
 		
-		$chosenResults = $page->findAll('css', '.select2-results li');
-		foreach ($chosenResults as $result) {
-			$text = $result->getText();
-			if (false!==strpos($text,$value)) {
-				$result->click();
-				return;
-			}
-		}
+        $chosenResults = $page->findAll('css', '.select2-results li');
+        foreach ($chosenResults as $result) {
+            $text = $result->getText();
+            if (false!==strpos($text,$value)) {
+                $result->click();
+                return;
+            }
+        }
 		
-		throw new \Exception(sprintf('Value "%s" not found for "%s"', $value, $field));
-	}
+        throw new \Exception(sprintf('Value "%s" not found for "%s"', $value, $field));
+    }
 	
-	private function openField(DocumentElement $page, $field)
-	{
-		$inputField = $page->find('css',$field);
-		if(!$inputField){
-			$fieldName = sprintf('select[name="%s"] + .select2-container', $field);
-			$inputField = $page->find('css', $fieldName);
-		}
-		if (!$inputField) {
-			throw new \Exception(sprintf('No field "%s" found', $field));
-		}
+    private function openField(DocumentElement $page, $field)
+    {
+        $inputField = $page->find('css',$field);
+        if(!$inputField){
+            $fieldName = sprintf('select[name="%s"] + .select2-container', $field);
+            $inputField = $page->find('css', $fieldName);
+        }
+        if (!$inputField) {
+            throw new \Exception(sprintf('No field "%s" found', $field));
+        }
 		
-		$choice = $inputField->find('css', '.select2-selection');
-		if (!$choice) {
-			throw new \Exception(sprintf('No select2 choice found for "%s"', $field));
-		}
-		$choice->press();
-	}
+        $choice = $inputField->find('css', '.select2-selection');
+        if (!$choice) {
+            throw new \Exception(sprintf('No select2 choice found for "%s"', $field));
+        }
+        $choice->press();
+    }
 	
-	/**
-	 * Wait the end of fetching Select2 results
-	 *
-	 * @param int $time Time to wait in seconds
-	 */
-	private function waitForLoadingResults($time)
-	{
-		for ($i = 0; $i < $time; $i++) {
-			if (!$this->getSession()->getPage()->find('css', '.select2-results__option.loading-results')) {
-				return;
-			}
+    /**
+     * Wait the end of fetching Select2 results
+     *
+     * @param int $time Time to wait in seconds
+     */
+    private function waitForLoadingResults($time)
+    {
+        for ($i = 0; $i < $time; $i++) {
+            if (!$this->getSession()->getPage()->find('css', '.select2-results__option.loading-results')) {
+                return;
+            }
 			
-			sleep(1);
-		}
-	}
+            sleep(1);
+        }
+    }
 	
 }
\ No newline at end of file

Spacing +8 added lines, -8 removed lines

@@ -45,11 +45,11 @@ 
 	 * @param $field
 	 * @param $value
 	 */
-	public function iFillInSelect2FieldWith($field,$search,$choice=null)
+	public function iFillInSelect2FieldWith($field, $search, $choice = null)
 	{
 		$page = $this->getSession()->getPage();
 		$this->openField($page, $field);
-		$this->fillSearchField($page,$field,$search);
+		$this->fillSearchField($page, $field, $search);
 		$this->selectValue($page, $field, $choice);
 	}
 	
@@ -67,12 +67,12 @@ 
 		if ('Behat\Mink\Driver\Selenium2Driver' === get_class($driver)) {
 			// Can't use `$this->getSession()->getPage()->find()` because of https://github.com/minkphp/MinkSelenium2Driver/issues/188
 			
-			$element = $page->find('css','.select2-container--open .select2-search__field');
+			$element = $page->find('css', '.select2-container--open .select2-search__field');
 			$xpath = $element->getXpath();
 			$select2Input = $this->getSession()
 				->getDriver()
 				->getWebDriverSession()
-				->element('xpath',$xpath)
+				->element('xpath', $xpath)
 				//->element('xpath', "//html/descendant-or-self::*[@class and contains(concat(' ', normalize-space(@class), ' '), ' select2-search__field ')]")
 			;
 			if (!$select2Input) {
@@ -100,14 +100,14 @@ 
 	 * @param int             $time
 	 * @throws \Exception
 	 */
-	private function selectValue(DocumentElement $page, $field, $value, $time=5)
+	private function selectValue(DocumentElement $page, $field, $value, $time = 5)
 	{
 		$this->waitForLoadingResults($time);
 		
 		$chosenResults = $page->findAll('css', '.select2-results li');
 		foreach ($chosenResults as $result) {
 			$text = $result->getText();
-			if (false!==strpos($text,$value)) {
+			if (false !== strpos($text, $value)) {
 				$result->click();
 				return;
 			}
@@ -118,8 +118,8 @@ 
 	
 	private function openField(DocumentElement $page, $field)
 	{
-		$inputField = $page->find('css',$field);
-		if(!$inputField){
+		$inputField = $page->find('css', $field);
+		if (!$inputField) {
 			$fieldName = sprintf('select[name="%s"] + .select2-container', $field);
 			$inputField = $page->find('css', $fieldName);
 		}

module/Behat/src/UserContext.php

Doc Comments +4 added lines, -1 removed lines

@@ -198,6 +198,9 @@ 
 		$this->startLogin($user,'test');
 	}
 	
+	/**
+	 * @param string $password
+	 */
 	private function startLogin(UserInterface $user, $password)
 	{
 		$currentUser = $this->currentUser;
@@ -242,7 +245,7 @@ 
 	 * @param string $fullname
 	 * @param string $role
 	 *
-	 * @return \Auth\Entity\UserInterface
+	 * @return \Core\Entity\EntityInterface
 	 */
 	public function createUser($email,$password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter")
 	{

Indentation +372 added lines, -372 removed lines

@@ -29,384 +29,384 @@
 
 class UserContext implements Context
 {
-	/**
-	 * @var CoreContext
-	 */
-	private $coreContext;
-	
-	/**
-	 * @var MinkContext
-	 */
-	private $minkContext;
-	
-	/**
-	 * @var User
-	 */
-	private $currentUser;
-	
-	/**
-	 * @var User[]
-	 */
-	static private $users = [];
-	
-	/**
-	 * @var UserRepository
-	 */
-	static private $userRepo;
-	
-	/**
-	 * @var string
-	 */
-	static private $currentSession;
-	
-	private $socialLoginInfo = [];
-	
-	/**
-	 * @var UserInterface
-	 */
-	private $loggedInUser;
-	
-	public function __construct($parameters=[])
-	{
-		$defaultLoginInfo = [
-			'facebook' => [
-				'email' => getenv('FACEBOOK_USER_EMAIL'),
-				'pass' => getenv('FACEBOOK_USER_PASSWORD')
-			],
-			'linkedin' => [
-				'session_key-login' => getenv('LINKEDIN_USER_EMAIL'),
-				'session_password-login' => getenv('LINKEDIN_USER_PASSWORD')
-			],
-		];
-		$socialLoginConfig = isset($parameters['social_login_info']) ? $parameters['social_login_info']:[];
-		$this->socialLoginInfo = array_merge($defaultLoginInfo,$socialLoginConfig);
-	}
-	
-	/**
-	 * @AfterSuite
-	 * @param AfterSuiteScope $scope
-	 */
-	static public function afterSuite(AfterSuiteScope $scope)
-	{
-		$repo = static::$userRepo;
-		foreach(static::$users as $user){
-			if($repo->findByLogin($user->getLogin())){
-				try{
-					JobContext::removeJobByUser($user);
-					$repo->remove($user,true);
-				}catch (\Exception $e){
+    /**
+     * @var CoreContext
+     */
+    private $coreContext;
+	
+    /**
+     * @var MinkContext
+     */
+    private $minkContext;
+	
+    /**
+     * @var User
+     */
+    private $currentUser;
+	
+    /**
+     * @var User[]
+     */
+    static private $users = [];
+	
+    /**
+     * @var UserRepository
+     */
+    static private $userRepo;
+	
+    /**
+     * @var string
+     */
+    static private $currentSession;
+	
+    private $socialLoginInfo = [];
+	
+    /**
+     * @var UserInterface
+     */
+    private $loggedInUser;
+	
+    public function __construct($parameters=[])
+    {
+        $defaultLoginInfo = [
+            'facebook' => [
+                'email' => getenv('FACEBOOK_USER_EMAIL'),
+                'pass' => getenv('FACEBOOK_USER_PASSWORD')
+            ],
+            'linkedin' => [
+                'session_key-login' => getenv('LINKEDIN_USER_EMAIL'),
+                'session_password-login' => getenv('LINKEDIN_USER_PASSWORD')
+            ],
+        ];
+        $socialLoginConfig = isset($parameters['social_login_info']) ? $parameters['social_login_info']:[];
+        $this->socialLoginInfo = array_merge($defaultLoginInfo,$socialLoginConfig);
+    }
+	
+    /**
+     * @AfterSuite
+     * @param AfterSuiteScope $scope
+     */
+    static public function afterSuite(AfterSuiteScope $scope)
+    {
+        $repo = static::$userRepo;
+        foreach(static::$users as $user){
+            if($repo->findByLogin($user->getLogin())){
+                try{
+                    JobContext::removeJobByUser($user);
+                    $repo->remove($user,true);
+                }catch (\Exception $e){
 				
-				}
-			}
-		}
-	}
-	
-	/**
-	 * @BeforeScenario
-	 * @param BeforeScenarioScope $scope
-	 */
-	public function beforeScenario(BeforeScenarioScope $scope)
-	{
-		$this->minkContext = $scope->getEnvironment()->getContext(MinkContext::class);
-		$this->coreContext = $scope->getEnvironment()->getContext(CoreContext::class);
-		static::$userRepo = $this->getUserRepository();
-	}
-	
-	/**
-	 * @return User
-	 */
-	public function getCurrentUser()
-	{
-		return $this->currentUser;
-	}
-	
-	/**
-	 * @When I fill in login form with :provider user
-	 */
-	public function iSignInWithSocialUser($provider)
-	{
-		$provider = strtolower($provider);
-		$mink = $this->minkContext;
-		foreach($this->socialLoginInfo[$provider] as $field=>$value){
-			$mink->fillField($field,$value);
-		}
-	}
-	
-	/**
-	 * @Given I am logged in as a recruiter
-	 * @Given I am logged in as a recruiter with :organization as organization
-	 */
-	public function iAmLoggedInAsARecruiter($organization=null)
-	{
-		$user = $this->thereIsAUserIdentifiedBy(
-			'test@recruiter.com',
-			'test',User::ROLE_RECRUITER,
-			'Test Recruiter',
-			$organization
-		);
-		$this->startLogin($user,'test');
-	}
-	
-	/**
-	 * @Given I don't have :login user
-	 * @param string $login
-	 */
-	public function iDonTHaveUser($login)
-	{
-		$repo = $this->getUserRepository();
-		$user=$repo->findByLogin($login);
-		if($user instanceof UserInterface){
-			$repo->remove($user,true);
-		}
-	}
-	
-	/**
-	 * @Given I have a :role with the following:
-	 * @param $role
-	 * @param TableNode $fields
-	 */
-	public function iHaveUserWithTheFollowing($role,TableNode $fields)
-	{
-		$normalizedFields = [
-			'login' => 'test@login.com',
-			'fullname' => 'Test Login',
-			'role' => User::ROLE_USER,
-			'password' => 'test',
-			'organization' => 'Cross Solution'
-		];
-		foreach($fields->getRowsHash() as $field=>$value){
-			$field = Inflector::camelize($field);
-			$normalizedFields[$field] = $value;
-		}
+                }
+            }
+        }
+    }
+	
+    /**
+     * @BeforeScenario
+     * @param BeforeScenarioScope $scope
+     */
+    public function beforeScenario(BeforeScenarioScope $scope)
+    {
+        $this->minkContext = $scope->getEnvironment()->getContext(MinkContext::class);
+        $this->coreContext = $scope->getEnvironment()->getContext(CoreContext::class);
+        static::$userRepo = $this->getUserRepository();
+    }
+	
+    /**
+     * @return User
+     */
+    public function getCurrentUser()
+    {
+        return $this->currentUser;
+    }
+	
+    /**
+     * @When I fill in login form with :provider user
+     */
+    public function iSignInWithSocialUser($provider)
+    {
+        $provider = strtolower($provider);
+        $mink = $this->minkContext;
+        foreach($this->socialLoginInfo[$provider] as $field=>$value){
+            $mink->fillField($field,$value);
+        }
+    }
+	
+    /**
+     * @Given I am logged in as a recruiter
+     * @Given I am logged in as a recruiter with :organization as organization
+     */
+    public function iAmLoggedInAsARecruiter($organization=null)
+    {
+        $user = $this->thereIsAUserIdentifiedBy(
+            'test@recruiter.com',
+            'test',User::ROLE_RECRUITER,
+            'Test Recruiter',
+            $organization
+        );
+        $this->startLogin($user,'test');
+    }
+	
+    /**
+     * @Given I don't have :login user
+     * @param string $login
+     */
+    public function iDonTHaveUser($login)
+    {
+        $repo = $this->getUserRepository();
+        $user=$repo->findByLogin($login);
+        if($user instanceof UserInterface){
+            $repo->remove($user,true);
+        }
+    }
+	
+    /**
+     * @Given I have a :role with the following:
+     * @param $role
+     * @param TableNode $fields
+     */
+    public function iHaveUserWithTheFollowing($role,TableNode $fields)
+    {
+        $normalizedFields = [
+            'login' => 'test@login.com',
+            'fullname' => 'Test Login',
+            'role' => User::ROLE_USER,
+            'password' => 'test',
+            'organization' => 'Cross Solution'
+        ];
+        foreach($fields->getRowsHash() as $field=>$value){
+            $field = Inflector::camelize($field);
+            $normalizedFields[$field] = $value;
+        }
 		
-		$this->thereIsAUserIdentifiedBy(
-			$normalizedFields['login'],
-			$normalizedFields['password'],
-			$role,
-			$normalizedFields['fullname'],
-			$normalizedFields['organization']
-		);
+        $this->thereIsAUserIdentifiedBy(
+            $normalizedFields['login'],
+            $normalizedFields['password'],
+            $role,
+            $normalizedFields['fullname'],
+            $normalizedFields['organization']
+        );
 		
-	}
-	
-	/**
-	 * @Given I am logged in as an administrator
-	 */
-	public function iAmLoggedInAsAnAdmin()
-	{
-		$user = $this->thereIsAUserIdentifiedBy('test@admin.com','test',User::ROLE_ADMIN);
-		$this->startLogin($user,'test');
-	}
-	
-	private function startLogin(UserInterface $user, $password)
-	{
-		$currentUser = $this->currentUser;
-		if(!is_object($currentUser) || $user->getId()!=$currentUser->getId()){
-			$this->iWantToLogIn();
-			$this->iSpecifyTheUsernameAs($user->getLogin());
-			$this->iSpecifyThePasswordAs($password);
-			$this->iLogIn();
-			$this->currentUser = $user;
-		}
-	}
-	
-	/**
-	 * @return UserRepository
-	 */
-	public function getUserRepository()
-	{
-		return $this->coreContext->getRepositories()->get('Auth\Entity\User');
-	}
-	
-	/**
-	 * @Given there is a user :email identified by :password
-	 */
-	public function thereIsAUserIdentifiedBy($email, $password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter",$organization=null)
-	{
-		$repo = $this->getUserRepository();
-		if(!is_object($user=$repo->findByEmail($email))){
-			$user = $this->createUser($email,$password,$role,$fullname,$organization);
-		}
+    }
+	
+    /**
+     * @Given I am logged in as an administrator
+     */
+    public function iAmLoggedInAsAnAdmin()
+    {
+        $user = $this->thereIsAUserIdentifiedBy('test@admin.com','test',User::ROLE_ADMIN);
+        $this->startLogin($user,'test');
+    }
+	
+    private function startLogin(UserInterface $user, $password)
+    {
+        $currentUser = $this->currentUser;
+        if(!is_object($currentUser) || $user->getId()!=$currentUser->getId()){
+            $this->iWantToLogIn();
+            $this->iSpecifyTheUsernameAs($user->getLogin());
+            $this->iSpecifyThePasswordAs($password);
+            $this->iLogIn();
+            $this->currentUser = $user;
+        }
+    }
+	
+    /**
+     * @return UserRepository
+     */
+    public function getUserRepository()
+    {
+        return $this->coreContext->getRepositories()->get('Auth\Entity\User');
+    }
+	
+    /**
+     * @Given there is a user :email identified by :password
+     */
+    public function thereIsAUserIdentifiedBy($email, $password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter",$organization=null)
+    {
+        $repo = $this->getUserRepository();
+        if(!is_object($user=$repo->findByEmail($email))){
+            $user = $this->createUser($email,$password,$role,$fullname,$organization);
+        }
 		
-		if(!is_null($organization)){
-			$this->iHaveMainOrganization($user,$organization);
-		}
-		$this->addCreatedUser($user);
-		return $user;
-	}
-	
-	/**
-	 * @param $email
-	 * @param $password
-	 * @param $username
-	 * @param string $fullname
-	 * @param string $role
-	 *
-	 * @return \Auth\Entity\UserInterface
-	 */
-	public function createUser($email,$password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter")
-	{
-		/* @var Register $service */
-		$repo = $this->getUserRepository();
-		$user = $repo->create([]);
-		$user->setLogin($email);
-		$user->setPassword($password);
-		$user->setRole($role);
+        if(!is_null($organization)){
+            $this->iHaveMainOrganization($user,$organization);
+        }
+        $this->addCreatedUser($user);
+        return $user;
+    }
+	
+    /**
+     * @param $email
+     * @param $password
+     * @param $username
+     * @param string $fullname
+     * @param string $role
+     *
+     * @return \Auth\Entity\UserInterface
+     */
+    public function createUser($email,$password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter")
+    {
+        /* @var Register $service */
+        $repo = $this->getUserRepository();
+        $user = $repo->create([]);
+        $user->setLogin($email);
+        $user->setPassword($password);
+        $user->setRole($role);
 		
-		$expFullName = explode(' ',$fullname);
-		$info = $user->getInfo();
-		$info->setFirstName(array_shift($expFullName));
-		$info->setLastName(count($expFullName)>0 ? implode(' ',$expFullName):'');
-		$info->setEmail($email);
-		$info->setEmailVerified(true);
-		$repo->store($user);
-		/* @var \Core\EventManager\EventManager $events */
-		/* @var \Auth\Listener\Events\AuthEvent $event */
-		//@TODO: [Behat] event not working in travis
-		//$events = $this->coreContext->getEventManager();
-		//$event  = $events->getEvent(AuthEvent::EVENT_USER_REGISTERED, $this);
-		//$event->setUser($user);
-		//$events->triggerEvent($event);
-		return $user;
-	}
-	
-	/**
-	 * @When I have :organization as my main organization
-	 * @param $orgName
-	 */
-	public function iHaveMainOrganization(UserInterface $user,$orgName)
-	{
-		/* @var $repoOrganization OrganizationRepository */
-		$repoOrganization = $this->coreContext->getRepositories()->get('Organizations/Organization');
-		$organization=$repoOrganization->findByName($orgName);
-		if(!$organization instanceof Organization){
-			$organization = new Organization();
-			$organizationName = new OrganizationName($orgName);
-			$organization->setOrganizationName($organizationName);
-		}
-		$organization->setUser($user);
-		$repoOrganization->store($organization);
-	}
-	
-	/**
-	 * @When I want to log in
-	 */
-	public function iWantToLogIn()
-	{
-		$session = $this->minkContext->getSession();
-		$url = $this->minkContext->locatePath('/en/login');
-		$session->visit($url);
-	}
-	
-	/**
-	 * @When I specify the username as :username
-	 */
-	public function iSpecifyTheUsernameAs($username)
-	{
-		$this->minkContext->fillField('Login name',$username);
-	}
-	
-	/**
-	 * @When I specify the password as :password
-	 */
-	public function iSpecifyThePasswordAs($password)
-	{
-		$this->minkContext->fillField('Password',$password);
-	}
-	
-	/**
-	 * @Given I am logged in as :username identified by :password
-	 */
-	public function iAmLoggedInAsIdentifiedBy($username, $password)
-	{
-		$repo = $this->getUserRepository();
-		$user = $repo->findByLogin($username);
+        $expFullName = explode(' ',$fullname);
+        $info = $user->getInfo();
+        $info->setFirstName(array_shift($expFullName));
+        $info->setLastName(count($expFullName)>0 ? implode(' ',$expFullName):'');
+        $info->setEmail($email);
+        $info->setEmailVerified(true);
+        $repo->store($user);
+        /* @var \Core\EventManager\EventManager $events */
+        /* @var \Auth\Listener\Events\AuthEvent $event */
+        //@TODO: [Behat] event not working in travis
+        //$events = $this->coreContext->getEventManager();
+        //$event  = $events->getEvent(AuthEvent::EVENT_USER_REGISTERED, $this);
+        //$event->setUser($user);
+        //$events->triggerEvent($event);
+        return $user;
+    }
+	
+    /**
+     * @When I have :organization as my main organization
+     * @param $orgName
+     */
+    public function iHaveMainOrganization(UserInterface $user,$orgName)
+    {
+        /* @var $repoOrganization OrganizationRepository */
+        $repoOrganization = $this->coreContext->getRepositories()->get('Organizations/Organization');
+        $organization=$repoOrganization->findByName($orgName);
+        if(!$organization instanceof Organization){
+            $organization = new Organization();
+            $organizationName = new OrganizationName($orgName);
+            $organization->setOrganizationName($organizationName);
+        }
+        $organization->setUser($user);
+        $repoOrganization->store($organization);
+    }
+	
+    /**
+     * @When I want to log in
+     */
+    public function iWantToLogIn()
+    {
+        $session = $this->minkContext->getSession();
+        $url = $this->minkContext->locatePath('/en/login');
+        $session->visit($url);
+    }
+	
+    /**
+     * @When I specify the username as :username
+     */
+    public function iSpecifyTheUsernameAs($username)
+    {
+        $this->minkContext->fillField('Login name',$username);
+    }
+	
+    /**
+     * @When I specify the password as :password
+     */
+    public function iSpecifyThePasswordAs($password)
+    {
+        $this->minkContext->fillField('Password',$password);
+    }
+	
+    /**
+     * @Given I am logged in as :username identified by :password
+     */
+    public function iAmLoggedInAsIdentifiedBy($username, $password)
+    {
+        $repo = $this->getUserRepository();
+        $user = $repo->findByLogin($username);
 		
-		if(!$user instanceof User){
-			throw new \Exception(sprintf('There is no user with this login: "%s"',$username));
-		}
-		$this->currentUser = $user;
-		$this->iWantToLogIn();
-		$this->iSpecifyTheUsernameAs($username);
-		$this->iSpecifyThePasswordAs($password);
-		$this->iLogIn();
-	}
-	
-	/**
-	 * @When I log in
-	 */
-	public function iLogIn()
-	{
-		$this->minkContext->pressButton('login');
-	}
-	
-	/**
-	 * @When I press logout link
-	 */
-	public function iPressLogoutLink()
-	{
-		//@TODO: [ZF3] replace this with click method
-		$url = $this->coreContext->generateUrl('/logout');
-		$this->minkContext->visit($url);
-	}
-	
-	/**
-	 * @Given I log in with username :username and password :password
-	 */
-	public function iLogInWith($username, $password)
-	{
-		$repo = $this->getUserRepository();
-		$user = $repo->findByLogin($username);
-		$this->iWantToLogIn();
-		$this->iSpecifyTheUsernameAs($username);
-		$this->iSpecifyThePasswordAs($password);
-		$this->iLogIn();
-		$this->loggedInUser = $user;
-	}
-	
-	/**
-	 * @When I go to profile page
-	 */
-	public function iGoToProfilePage()
-	{
-		$url = $this->coreContext->generateUrl('/en/my/profile');
-		$this->minkContext->visit($url);
-	}
-	
-	/**
-	 * @Given there is a user with the following:
-	 */
-	public function thereIsAUserWithTheFollowing(TableNode $table)
-	{
-		$repo = $this->getUserRepository();
-		$data = $table->getRowsHash();
-		$email = isset($data['email']) ? $data['email']:'test@example.com';
-		$password = isset($data['password']) ? $data['password']:'test';
-		$fullname = isset($data['fullname']) ? $data['fullname']:'Test User';
-		$role = isset($data['role']) ? $data['role']:User::ROLE_RECRUITER;
+        if(!$user instanceof User){
+            throw new \Exception(sprintf('There is no user with this login: "%s"',$username));
+        }
+        $this->currentUser = $user;
+        $this->iWantToLogIn();
+        $this->iSpecifyTheUsernameAs($username);
+        $this->iSpecifyThePasswordAs($password);
+        $this->iLogIn();
+    }
+	
+    /**
+     * @When I log in
+     */
+    public function iLogIn()
+    {
+        $this->minkContext->pressButton('login');
+    }
+	
+    /**
+     * @When I press logout link
+     */
+    public function iPressLogoutLink()
+    {
+        //@TODO: [ZF3] replace this with click method
+        $url = $this->coreContext->generateUrl('/logout');
+        $this->minkContext->visit($url);
+    }
+	
+    /**
+     * @Given I log in with username :username and password :password
+     */
+    public function iLogInWith($username, $password)
+    {
+        $repo = $this->getUserRepository();
+        $user = $repo->findByLogin($username);
+        $this->iWantToLogIn();
+        $this->iSpecifyTheUsernameAs($username);
+        $this->iSpecifyThePasswordAs($password);
+        $this->iLogIn();
+        $this->loggedInUser = $user;
+    }
+	
+    /**
+     * @When I go to profile page
+     */
+    public function iGoToProfilePage()
+    {
+        $url = $this->coreContext->generateUrl('/en/my/profile');
+        $this->minkContext->visit($url);
+    }
+	
+    /**
+     * @Given there is a user with the following:
+     */
+    public function thereIsAUserWithTheFollowing(TableNode $table)
+    {
+        $repo = $this->getUserRepository();
+        $data = $table->getRowsHash();
+        $email = isset($data['email']) ? $data['email']:'test@example.com';
+        $password = isset($data['password']) ? $data['password']:'test';
+        $fullname = isset($data['fullname']) ? $data['fullname']:'Test User';
+        $role = isset($data['role']) ? $data['role']:User::ROLE_RECRUITER;
 		
-		if(!is_object($user=$repo->findByLogin($email))){
-			$user = $this->createUser($email,$password,$role,$fullname);
-		}
-		$this->currentUser = $user;
-		$this->addCreatedUser($user);
-	}
-	
-	private function addCreatedUser(UserInterface $user)
-	{
-		if(!in_array($user,static::$users)){
-			static::$users[] = $user;
-		}
-	}
-	
-	/**
-	 * @When I want to change my password
-	 */
-	public function iWantToChangeMyPassword()
-	{
-		$mink = $this->minkContext;
-		$url = $this->coreContext->generateUrl('/en/my/password');
-		$mink->getSession()->visit($url);
-	}
+        if(!is_object($user=$repo->findByLogin($email))){
+            $user = $this->createUser($email,$password,$role,$fullname);
+        }
+        $this->currentUser = $user;
+        $this->addCreatedUser($user);
+    }
+	
+    private function addCreatedUser(UserInterface $user)
+    {
+        if(!in_array($user,static::$users)){
+            static::$users[] = $user;
+        }
+    }
+	
+    /**
+     * @When I want to change my password
+     */
+    public function iWantToChangeMyPassword()
+    {
+        $mink = $this->minkContext;
+        $url = $this->coreContext->generateUrl('/en/my/password');
+        $mink->getSession()->visit($url);
+    }
 	
 }
\ No newline at end of file

Spacing +43 added lines, -43 removed lines

@@ -66,7 +66,7 @@ 
 	 */
 	private $loggedInUser;
 	
-	public function __construct($parameters=[])
+	public function __construct($parameters = [])
 	{
 		$defaultLoginInfo = [
 			'facebook' => [
@@ -78,8 +78,8 @@ 
 				'session_password-login' => getenv('LINKEDIN_USER_PASSWORD')
 			],
 		];
-		$socialLoginConfig = isset($parameters['social_login_info']) ? $parameters['social_login_info']:[];
-		$this->socialLoginInfo = array_merge($defaultLoginInfo,$socialLoginConfig);
+		$socialLoginConfig = isset($parameters['social_login_info']) ? $parameters['social_login_info'] : [];
+		$this->socialLoginInfo = array_merge($defaultLoginInfo, $socialLoginConfig);
 	}
 	
 	/**
@@ -89,12 +89,12 @@ 
 	static public function afterSuite(AfterSuiteScope $scope)
 	{
 		$repo = static::$userRepo;
-		foreach(static::$users as $user){
-			if($repo->findByLogin($user->getLogin())){
-				try{
+		foreach (static::$users as $user) {
+			if ($repo->findByLogin($user->getLogin())) {
+				try {
 					JobContext::removeJobByUser($user);
-					$repo->remove($user,true);
-				}catch (\Exception $e){
+					$repo->remove($user, true);
+				} catch (\Exception $e) {
 				
 				}
 			}
@@ -127,8 +127,8 @@ 
 	{
 		$provider = strtolower($provider);
 		$mink = $this->minkContext;
-		foreach($this->socialLoginInfo[$provider] as $field=>$value){
-			$mink->fillField($field,$value);
+		foreach ($this->socialLoginInfo[$provider] as $field=>$value) {
+			$mink->fillField($field, $value);
 		}
 	}
 	
@@ -136,15 +136,15 @@ 
 	 * @Given I am logged in as a recruiter
 	 * @Given I am logged in as a recruiter with :organization as organization
 	 */
-	public function iAmLoggedInAsARecruiter($organization=null)
+	public function iAmLoggedInAsARecruiter($organization = null)
 	{
 		$user = $this->thereIsAUserIdentifiedBy(
 			'test@recruiter.com',
-			'test',User::ROLE_RECRUITER,
+			'test', User::ROLE_RECRUITER,
 			'Test Recruiter',
 			$organization
 		);
-		$this->startLogin($user,'test');
+		$this->startLogin($user, 'test');
 	}
 	
 	/**
@@ -154,9 +154,9 @@ 
 	public function iDonTHaveUser($login)
 	{
 		$repo = $this->getUserRepository();
-		$user=$repo->findByLogin($login);
-		if($user instanceof UserInterface){
-			$repo->remove($user,true);
+		$user = $repo->findByLogin($login);
+		if ($user instanceof UserInterface) {
+			$repo->remove($user, true);
 		}
 	}
 	
@@ -165,7 +165,7 @@ 
 	 * @param $role
 	 * @param TableNode $fields
 	 */
-	public function iHaveUserWithTheFollowing($role,TableNode $fields)
+	public function iHaveUserWithTheFollowing($role, TableNode $fields)
 	{
 		$normalizedFields = [
 			'login' => 'test@login.com',
@@ -174,7 +174,7 @@ 
 			'password' => 'test',
 			'organization' => 'Cross Solution'
 		];
-		foreach($fields->getRowsHash() as $field=>$value){
+		foreach ($fields->getRowsHash() as $field=>$value) {
 			$field = Inflector::camelize($field);
 			$normalizedFields[$field] = $value;
 		}
@@ -194,14 +194,14 @@ 
 	 */
 	public function iAmLoggedInAsAnAdmin()
 	{
-		$user = $this->thereIsAUserIdentifiedBy('test@admin.com','test',User::ROLE_ADMIN);
-		$this->startLogin($user,'test');
+		$user = $this->thereIsAUserIdentifiedBy('test@admin.com', 'test', User::ROLE_ADMIN);
+		$this->startLogin($user, 'test');
 	}
 	
 	private function startLogin(UserInterface $user, $password)
 	{
 		$currentUser = $this->currentUser;
-		if(!is_object($currentUser) || $user->getId()!=$currentUser->getId()){
+		if (!is_object($currentUser) || $user->getId() != $currentUser->getId()) {
 			$this->iWantToLogIn();
 			$this->iSpecifyTheUsernameAs($user->getLogin());
 			$this->iSpecifyThePasswordAs($password);
@@ -221,15 +221,15 @@ 
 	/**
 	 * @Given there is a user :email identified by :password
 	 */
-	public function thereIsAUserIdentifiedBy($email, $password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter",$organization=null)
+	public function thereIsAUserIdentifiedBy($email, $password, $role = User::ROLE_RECRUITER, $fullname = "Test Recruiter", $organization = null)
 	{
 		$repo = $this->getUserRepository();
-		if(!is_object($user=$repo->findByEmail($email))){
-			$user = $this->createUser($email,$password,$role,$fullname,$organization);
+		if (!is_object($user = $repo->findByEmail($email))) {
+			$user = $this->createUser($email, $password, $role, $fullname, $organization);
 		}
 		
-		if(!is_null($organization)){
-			$this->iHaveMainOrganization($user,$organization);
+		if (!is_null($organization)) {
+			$this->iHaveMainOrganization($user, $organization);
 		}
 		$this->addCreatedUser($user);
 		return $user;
@@ -244,7 +244,7 @@ 
 	 *
 	 * @return \Auth\Entity\UserInterface
 	 */
-	public function createUser($email,$password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter")
+	public function createUser($email, $password, $role = User::ROLE_RECRUITER, $fullname = "Test Recruiter")
 	{
 		/* @var Register $service */
 		$repo = $this->getUserRepository();
@@ -253,10 +253,10 @@ 
 		$user->setPassword($password);
 		$user->setRole($role);
 		
-		$expFullName = explode(' ',$fullname);
+		$expFullName = explode(' ', $fullname);
 		$info = $user->getInfo();
 		$info->setFirstName(array_shift($expFullName));
-		$info->setLastName(count($expFullName)>0 ? implode(' ',$expFullName):'');
+		$info->setLastName(count($expFullName) > 0 ? implode(' ', $expFullName) : '');
 		$info->setEmail($email);
 		$info->setEmailVerified(true);
 		$repo->store($user);
@@ -274,12 +274,12 @@ 
 	 * @When I have :organization as my main organization
 	 * @param $orgName
 	 */
-	public function iHaveMainOrganization(UserInterface $user,$orgName)
+	public function iHaveMainOrganization(UserInterface $user, $orgName)
 	{
 		/* @var $repoOrganization OrganizationRepository */
 		$repoOrganization = $this->coreContext->getRepositories()->get('Organizations/Organization');
-		$organization=$repoOrganization->findByName($orgName);
-		if(!$organization instanceof Organization){
+		$organization = $repoOrganization->findByName($orgName);
+		if (!$organization instanceof Organization) {
 			$organization = new Organization();
 			$organizationName = new OrganizationName($orgName);
 			$organization->setOrganizationName($organizationName);
@@ -303,7 +303,7 @@ 
 	 */
 	public function iSpecifyTheUsernameAs($username)
 	{
-		$this->minkContext->fillField('Login name',$username);
+		$this->minkContext->fillField('Login name', $username);
 	}
 	
 	/**
@@ -311,7 +311,7 @@ 
 	 */
 	public function iSpecifyThePasswordAs($password)
 	{
-		$this->minkContext->fillField('Password',$password);
+		$this->minkContext->fillField('Password', $password);
 	}
 	
 	/**
@@ -322,8 +322,8 @@ 
 		$repo = $this->getUserRepository();
 		$user = $repo->findByLogin($username);
 		
-		if(!$user instanceof User){
-			throw new \Exception(sprintf('There is no user with this login: "%s"',$username));
+		if (!$user instanceof User) {
+			throw new \Exception(sprintf('There is no user with this login: "%s"', $username));
 		}
 		$this->currentUser = $user;
 		$this->iWantToLogIn();
@@ -380,13 +380,13 @@ 
 	{
 		$repo = $this->getUserRepository();
 		$data = $table->getRowsHash();
-		$email = isset($data['email']) ? $data['email']:'test@example.com';
-		$password = isset($data['password']) ? $data['password']:'test';
-		$fullname = isset($data['fullname']) ? $data['fullname']:'Test User';
-		$role = isset($data['role']) ? $data['role']:User::ROLE_RECRUITER;
+		$email = isset($data['email']) ? $data['email'] : 'test@example.com';
+		$password = isset($data['password']) ? $data['password'] : 'test';
+		$fullname = isset($data['fullname']) ? $data['fullname'] : 'Test User';
+		$role = isset($data['role']) ? $data['role'] : User::ROLE_RECRUITER;
 		
-		if(!is_object($user=$repo->findByLogin($email))){
-			$user = $this->createUser($email,$password,$role,$fullname);
+		if (!is_object($user = $repo->findByLogin($email))) {
+			$user = $this->createUser($email, $password, $role, $fullname);
 		}
 		$this->currentUser = $user;
 		$this->addCreatedUser($user);
@@ -394,7 +394,7 @@ 
 	
 	private function addCreatedUser(UserInterface $user)
 	{
-		if(!in_array($user,static::$users)){
+		if (!in_array($user, static::$users)) {
 			static::$users[] = $user;
 		}
 	}

Braces +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
 				try{
 					JobContext::removeJobByUser($user);
 					$repo->remove($user,true);
-				}catch (\Exception $e){
+				} catch (\Exception $e){
 				
 				}
 			}

module/Core/src/Core/Controller/Plugin/CreatePaginator.php

Doc Comments -1 removed lines

@@ -128,7 +128,6 @@
     }
     
     /**
-     * @param ControllerManager $controllerManager
      * @return CreatePaginator
      * @codeCoverageIgnore
      */

Indentation +10 added lines, -10 removed lines

@@ -44,12 +44,12 @@ 
      */
     protected $request;
 	
-	/**
-	 * CreatePaginator constructor.
-	 *
-	 * @param ContainerInterface $container
-	 * @param HttpRequest $request
-	 */
+    /**
+     * CreatePaginator constructor.
+     *
+     * @param ContainerInterface $container
+     * @param HttpRequest $request
+     */
     public function __construct(ContainerInterface $container, HttpRequest $request)
     {
         $this->serviceManager = $container->get('ServiceManager');
@@ -105,9 +105,9 @@ 
         /* @var \Core\EventManager\EventManager $events */
         /* @var \Zend\Paginator\Paginator $paginator */
         /* @var CreatePaginatorEvent $event */
-	    $events = $this->serviceManager->get('Core/CreatePaginator/Events');
+        $events = $this->serviceManager->get('Core/CreatePaginator/Events');
 
-	    $event = $events->getEvent(CreatePaginatorEvent::EVENT_CREATE_PAGINATOR,$this,[
+        $event = $events->getEvent(CreatePaginatorEvent::EVENT_CREATE_PAGINATOR,$this,[
             'paginatorParams' => $params,
             'paginators' => $paginators,
             'paginatorName' => $paginatorName
@@ -121,8 +121,8 @@ 
             $paginator = $paginators->get($paginatorName,$params);
         }
         $paginator->setCurrentPageNumber(isset($params['page']) ? $params['page'] : 1)
-                  ->setItemCountPerPage(isset($params['count']) ? $params['count'] : 10)
-                  ->setPageRange(isset($params['range']) ? $params['range'] : 5);
+                    ->setItemCountPerPage(isset($params['count']) ? $params['count'] : 10)
+                    ->setPageRange(isset($params['range']) ? $params['range'] : 5);
 
         return $paginator;
     }

Spacing +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
         /* @var CreatePaginatorEvent $event */
 	    $events = $this->serviceManager->get('Core/CreatePaginator/Events');
 
-	    $event = $events->getEvent(CreatePaginatorEvent::EVENT_CREATE_PAGINATOR,$this,[
+	    $event = $events->getEvent(CreatePaginatorEvent::EVENT_CREATE_PAGINATOR, $this, [
             'paginatorParams' => $params,
             'paginators' => $paginators,
             'paginatorName' => $paginatorName
@@ -116,9 +116,9 @@ 
 
         $paginator = $event->getPaginator();
 
-        if(!$paginator instanceof Paginator){
+        if (!$paginator instanceof Paginator) {
             // no paginator created by listener, so let's create default paginator
-            $paginator = $paginators->get($paginatorName,$params);
+            $paginator = $paginators->get($paginatorName, $params);
         }
         $paginator->setCurrentPageNumber(isset($params['page']) ? $params['page'] : 1)
                   ->setItemCountPerPage(isset($params['count']) ? $params['count'] : 10)

module/Core/src/Core/EventManager/EventManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -60,8 +60,8 @@
      *
      * If no event instance is passed, it creates one prior to triggering.
      *
-     * @param EventInterface|string $eventName
-     * @param object|string|null $target
+     * @param EventInterface $eventName
+     * @param \Core\Controller\AdminController $target
      * @param array $argv
      *
      * @return \Zend\EventManager\ResponseCollection

Indentation +3 added lines, -3 removed lines

@@ -55,8 +55,8 @@ 
         return $event;
     }
 	
-	/**
-	 * Trigger an event.
+    /**
+     * Trigger an event.
      *
      * If no event instance is passed, it creates one prior to triggering.
      *
@@ -65,7 +65,7 @@ 
      * @param array $argv
      *
      * @return \Zend\EventManager\ResponseCollection
-	 */
+     */
     public function trigger($eventName, $target = null, $argv = [])
     {
         $event = $eventName instanceOf EventInterface

module/Core/src/Core/Factory/Service/RestClientFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     }
 
     /**
-     * @return mixed
+     * @return null|string
      */
     abstract protected function getUri();
 

Indentation +7 added lines, -7 removed lines

@@ -38,15 +38,15 @@
      */
     protected $serviceLocator;
 	
-	public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
-	{
-		$this->serviceLocator = $container;
-		$service = new RestClient($this->getUri(), $this->getConfig());
-		return $service;
-	}
+    public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
+    {
+        $this->serviceLocator = $container;
+        $service = new RestClient($this->getUri(), $this->getConfig());
+        return $service;
+    }
 	
 	
-	/**
+    /**
      * Create the settings service
      *
      * @param  ServiceLocatorInterface $serviceLocator

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
      */
     protected $serviceLocator;
 	
-	public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
+	public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
 	{
 		$this->serviceLocator = $container;
 		$service = new RestClient($this->getUri(), $this->getConfig());
@@ -54,7 +54,7 @@ 
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {
-        return $this($serviceLocator,RestClient::class);
+        return $this($serviceLocator, RestClient::class);
     }
 
     /**

module/Core/src/Core/I18n/LocaleFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      * @param  string             $requestedName
      * @param  null|array         $options
      *
-     * @return object
+     * @return Locale
      * @throws ServiceNotFoundException if unable to resolve the service.
      * @throws ServiceNotCreatedException if an exception is raised when
      *     creating a service.