Removed the getter for the SelectorsHandler in the Session

This allows getting rid of the SelectorsHandler as a collaborator of the
Session class.
People needing to access the SelectorsHandler directly should instantiate
the ElementFinder explicitly with a SelectorsHandler, and keep a reference
to it.

Author: stof

src/Session.php

@@ -12,7 +12,6 @@
 
 use Behat\Mink\Driver\DriverInterface;
 use Behat\Mink\Element\ElementFinder;
-use Behat\Mink\Selector\SelectorsHandler;
 use Behat\Mink\Element\DocumentElement;
 
 /**
@@ -24,27 +23,20 @@ class Session
 {
     private $driver;
     private $page;
-    private $selectorsHandler;
 
     /**
      * Initializes session.
      *
-     * @param DriverInterface       $driver
-     * @param SelectorsHandler|null $selectorsHandler
-     * @param ElementFinder|null    $elementFinder
+     * @param DriverInterface    $driver
+     * @param ElementFinder|null $elementFinder
      */
-    public function __construct(DriverInterface $driver, SelectorsHandler $selectorsHandler = null, ElementFinder $elementFinder = null)
+    public function __construct(DriverInterface $driver, ElementFinder $elementFinder = null)
     {
-        if (null === $selectorsHandler) {
-            $selectorsHandler = new SelectorsHandler();
-        }
-
         if (null === $elementFinder) {
-            $elementFinder = new ElementFinder($driver, $selectorsHandler);
+            $elementFinder = new ElementFinder($driver);
         }
 
         $this->driver = $driver;
-        $this->selectorsHandler = $selectorsHandler;
         $this->page = new DocumentElement($driver, $elementFinder);
     }
 
@@ -127,16 +119,6 @@ public function getPage()
         return $this->page;
     }
 
-    /**
-     * Returns selectors handler.
-     *
-     * @return SelectorsHandler
-     */
-    public function getSelectorsHandler()
-    {
-        return $this->selectorsHandler;
-    }
-
     /**
      * Visit specified URL.
      *

tests/SessionTest.php

@@ -10,7 +10,6 @@ class SessionTest extends \PHPUnit_Framework_TestCase
      * @var \PHPUnit_Framework_MockObject_MockObject
      */
     private $driver;
-    private $selectorsHandler;
 
     /**
      * Session.
@@ -22,8 +21,7 @@ class SessionTest extends \PHPUnit_Framework_TestCase
     protected function setUp()
     {
         $this->driver = $this->getMockBuilder('Behat\Mink\Driver\DriverInterface')->getMock();
-        $this->selectorsHandler = $this->getMockBuilder('Behat\Mink\Selector\SelectorsHandler')->getMock();
-        $this->session = new Session($this->driver, $this->selectorsHandler);
+        $this->session = new Session($this->driver);
     }
 
     public function testGetDriver()
@@ -36,16 +34,11 @@ public function testGetPage()
         $this->assertInstanceOf('Behat\Mink\Element\DocumentElement', $this->session->getPage());
     }
 
-    public function testGetSelectorsHandler()
-    {
-        $this->assertSame($this->selectorsHandler, $this->session->getSelectorsHandler());
-    }
-
     public function testInstantiateWithoutOptionalDeps()
     {
-        $session = new Session($this->driver);
+        new Session($this->driver);
 
-        $this->assertInstanceOf('Behat\Mink\Selector\SelectorsHandler', $session->getSelectorsHandler());
+        $this->assertTrue(true, 'The instantiation should work.');
     }
 
     public function testIsStarted()