Merge pull request #471 from OndraM/feature/tests-navigation

 Add functional tests for WebDriverNavigation

Author: OndraM

lib/Firefox/FirefoxDriver.php

@@ -15,6 +15,9 @@
 
 namespace Facebook\WebDriver\Firefox;
 
+/**
+ * @codeCoverageIgnore
+ */
 class FirefoxDriver
 {
     const PROFILE = 'firefox_profile';

lib/Firefox/FirefoxPreferences.php

@@ -18,6 +18,8 @@
 /**
  * Constants of common Firefox profile preferences (about:config values).
  * @see http://kb.mozillazine.org/Firefox_:_FAQs_:_About:config_Entries
+ *
+ * @codeCoverageIgnore
  */
 class FirefoxPreferences
 {

lib/Remote/DriverCommand.php

@@ -17,6 +17,8 @@
 
 /**
  * This list of command defined in the WebDriver json wire protocol.
+ *
+ * @codeCoverageIgnore
  */
 class DriverCommand
 {

lib/Remote/WebDriverBrowserType.php

@@ -16,7 +16,9 @@
 namespace Facebook\WebDriver\Remote;
 
 /**
- * All the browsers supported by selenium
+ * All the browsers supported by selenium.
+ *
+ * @codeCoverageIgnore
  */
 class WebDriverBrowserType
 {

lib/Remote/WebDriverCapabilityType.php

@@ -17,6 +17,8 @@
 
 /**
  * WebDriverCapabilityType contains all constants defined in the WebDriver Wire Protocol.
+ *
+ * @codeCoverageIgnore
  */
 class WebDriverCapabilityType
 {

lib/WebDriverNavigation.php

@@ -75,6 +75,7 @@ public function refresh()
     /**
      * Navigate to the given URL.
      *
+     * @see WebDriver::get()
      * @param string $url
      * @return WebDriverNavigation The instance.
      */

lib/WebDriverPlatform.php

@@ -17,6 +17,8 @@
 
 /**
  * The platforms supported by WebDriver.
+ *
+ * @codeCoverageIgnore
  */
 class WebDriverPlatform
 {

tests/functional/RemoteWebDriverTest.php

@@ -45,10 +45,7 @@ public function testShouldGetCurrentUrl()
     {
         $this->driver->get($this->getTestPageUrl('index.html'));
 
-        $this->assertContains(
-            '/index.html',
-            $this->driver->getCurrentURL()
-        );
+        $this->assertStringEndsWith('/index.html', $this->driver->getCurrentURL());
     }
 
     /**
@@ -173,6 +170,7 @@ function(){document.getElementById("id_test").innerHTML = "Text changed by scrip
 
     /**
      * @covers ::executeAsyncScript
+     * @covers Facebook\WebDriver\WebDriverTimeouts::setScriptTimeout
      */
     public function testShouldExecuteAsyncScriptAndWaitUntilItIsFinished()
     {

tests/functional/WebDriverNavigationTest.php

@@ -0,0 +1,85 @@
+<?php
+// Copyright 2004-present Facebook. All Rights Reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+namespace Facebook\WebDriver;
+
+/**
+ * @coversDefaultClass Facebook\WebDriver\WebDriverNavigation
+ */
+class WebDriverNavigationTest extends WebDriverTestCase
+{
+    /**
+     * @covers ::to
+     * @covers ::__construct
+     */
+    public function testShouldNavigateToUrl()
+    {
+        $this->driver->navigate()->to($this->getTestPageUrl('index.html'));
+
+        $this->assertStringEndsWith('/index.html', $this->driver->getCurrentURL());
+    }
+
+    /**
+     * @covers ::back
+     * @covers ::forward
+     */
+    public function testShouldNavigateBackAndForward()
+    {
+        $this->driver->get($this->getTestPageUrl('index.html'));
+        $linkElement = $this->driver->findElement(WebDriverBy::id('a-form'));
+
+        $linkElement->click();
+
+        $this->driver->wait()->until(
+            WebDriverExpectedCondition::urlContains('form.html')
+        );
+
+        $this->driver->navigate()->back();
+
+        $this->driver->wait()->until(
+            WebDriverExpectedCondition::urlContains('index.html')
+        );
+
+        $this->driver->navigate()->forward();
+
+        $this->driver->wait()->until(
+            WebDriverExpectedCondition::urlContains('form.html')
+        );
+    }
+
+    /**
+     * @covers ::refresh
+     */
+    public function testShouldRefreshPage()
+    {
+        $this->driver->get($this->getTestPageUrl('index.html'));
+
+        // Change input element content, to make sure it was refreshed (=> cleared to original value)
+        $inputElement = $this->driver->findElement(WebDriverBy::name('test_name'));
+        $inputElementOriginalValue = $inputElement->getAttribute('value');
+        $inputElement->clear()->sendKeys('New value');
+        $this->assertSame('New value', $inputElement->getAttribute('value'));
+
+        $this->driver->navigate()->refresh();
+
+        $this->driver->wait()->until(
+            WebDriverExpectedCondition::stalenessOf($inputElement)
+        );
+
+        $inputElementAfterRefresh = $this->driver->findElement(WebDriverBy::name('test_name'));
+
+        $this->assertSame($inputElementOriginalValue, $inputElementAfterRefresh->getAttribute('value'));
+    }
+}

tests/functional/WebDriverTimeoutsTest.php

@@ -36,6 +36,7 @@ public function testShouldFailGettingDelayedElementWithoutWait()
 
     /**
      * @covers ::implicitlyWait
+     * @covers ::__construct
      */
     public function testShouldGetDelayedElementWithImplicitWait()
     {
@@ -49,6 +50,7 @@ public function testShouldGetDelayedElementWithImplicitWait()
 
     /**
      * @covers ::pageLoadTimeout
+     * @covers ::__construct
      */
     public function testShouldFailIfPageIsLoadingLongerThanPageLoadTimeout()
     {