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;
 
 /**
@@ -30,30 +29,20 @@ class Session
      * @var DocumentElement
      */
     private $page;
-    /**
-     * @var SelectorsHandler
-     */
-    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);
     }
 
@@ -144,16 +133,6 @@ public function getPage()
         return $this->page;
     }
 
-    /**
-     * Returns selectors handler.
-     *
-     * @return SelectorsHandler
-     */
-    public function getSelectorsHandler()
-    {
-        return $this->selectorsHandler;
-    }
-
     /**
      * Visit specified URL and automatically start session if not already running.
      *

tests/SessionTest.php

@@ -14,10 +14,6 @@ class SessionTest extends TestCase
      * @var DriverInterface&MockObject
      */
     private $driver;
-    /**
-     * @var SelectorsHandler
-     */
-    private $selectorsHandler;
 
     /**
      * Session.
@@ -32,8 +28,7 @@ class SessionTest extends TestCase
     protected function prepareSession(): void
     {
         $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()
@@ -46,16 +41,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()