rb - update more Edge specs

Author: titusfortner

rb/spec/integration/selenium/webdriver/options_spec.rb

@@ -60,13 +60,14 @@ module WebDriver
             expect(cookies.first[:value]).to eq("bar")
           end
 
-          # TODO - File bug with Microsoft; This command returns unknown error:
-          # DELETE session/EC3D38BB-BF62-4224-BB6D-E6820EF7C519/cookie/foo
-          it "should delete one" do
-            driver.navigate.to url_for("xhtmlTest.html")
-            driver.manage.add_cookie :name => "foo", :value => "bar"
+          # Edge BUG - https://connect.microsoft.com/IE/feedbackdetail/view/1864122
+          not_compliant_on :browser => :edge do
+            it "should delete one" do
+              driver.navigate.to url_for("xhtmlTest.html")
+              driver.manage.add_cookie :name => "foo", :value => "bar"
 
-            driver.manage.delete_cookie("foo")
+              driver.manage.delete_cookie("foo")
+            end
           end
 
           # This is not a w3c supported spec

rb/spec/integration/selenium/webdriver/target_locator_spec.rb

@@ -207,38 +207,33 @@
     end
   end
 
-  describe "alerts" do
-    not_compliant_on :browser => [:iphone, :safari, :phantomjs] do
+  # Edge BUG - https://connect.microsoft.com/IE/feedback/details/1850030
+  not_compliant_on :browser => [:iphone, :safari, :phantomjs, :edge] do
+    describe "alerts" do
       it "allows the user to accept an alert" do
-        driver.navigate.to url_for("alerts.html")
-        driver.find_element(:id => "alert").click
+          driver.navigate.to url_for("alerts.html")
+          driver.find_element(:id => "alert").click
 
-        alert = wait_for_alert
-        alert.accept
+          alert = wait_for_alert
+          alert.accept
 
-        expect(driver.title).to eq("Testing Alerts")
-      end
-    end
+          expect(driver.title).to eq("Testing Alerts")
+        end
 
-    not_compliant_on({:browser => :chrome, :platform => :macosx},
-                     {:browser => :iphone},
-                     {:browser => :safari},
-                     {:browser => :phantomjs}) do
-      it "allows the user to dismiss an alert" do
-        driver.navigate.to url_for("alerts.html")
-        driver.find_element(:id => "alert").click
+      not_compliant_on({:browser => :chrome, :platform => :macosx}) do
+        it "allows the user to dismiss an alert" do
+          driver.navigate.to url_for("alerts.html")
+          driver.find_element(:id => "alert").click
 
-        alert = wait_for_alert
-        alert.dismiss
+          alert = wait_for_alert
+          alert.dismiss
 
-        wait_for_no_alert
+          wait_for_no_alert
 
-        expect(driver.title).to eq("Testing Alerts")
+          expect(driver.title).to eq("Testing Alerts")
+        end
       end
-    end
 
-    # Edge BUG - https://connect.microsoft.com/IE/feedback/details/1850030
-    not_compliant_on :browser => [:iphone, :safari, :phantomjs, :edge] do
        it "allows the user to set the value of a prompt" do
         driver.navigate.to url_for("alerts.html")
         driver.find_element(:id => "prompt").click
@@ -272,27 +267,24 @@
         expect { alert.text }.to raise_error(Selenium::WebDriver::Error::NoAlertPresentError)
       end
 
-    end
-
-    not_compliant_on :browser => [:ie, :iphone, :safari, :phantomjs] do
-      it "raises NoAlertOpenError if no alert is present" do
-        expect { driver.switch_to.alert }.to raise_error(
-          Selenium::WebDriver::Error::NoAlertPresentError, /alert|modal dialog/i)
+      not_compliant_on :browser => :ie do
+        it "raises NoAlertOpenError if no alert is present" do
+          expect { driver.switch_to.alert }.to raise_error(
+            Selenium::WebDriver::Error::NoAlertPresentError, /alert|modal dialog/i)
+        end
       end
-    end
 
-    # TODO - File Edge bug; clicking for alert causes driver to hang
-    compliant_on :browser => [:firefox, :ie, :edge] do
-      it "raises an UnhandledAlertError if an alert has not been dealt with" do
-        driver.navigate.to url_for("alerts.html")
-        driver.find_element(:id => "alert").click
-        wait_for_alert
+      compliant_on :browser => [:firefox, :ie] do
+        it "raises an UnhandledAlertError if an alert has not been dealt with" do
+          driver.navigate.to url_for("alerts.html")
+          driver.find_element(:id => "alert").click
+          wait_for_alert
 
-        expect { driver.title }.to raise_error(Selenium::WebDriver::Error::UnhandledAlertError, /cheese/)
+          expect { driver.title }.to raise_error(Selenium::WebDriver::Error::UnhandledAlertError, /cheese/)
 
-        expect(driver.title).to eq("Testing Alerts") # :chrome does not auto-dismiss the alert
+          expect(driver.title).to eq("Testing Alerts") # :chrome does not auto-dismiss the alert
+        end
       end
     end
-
   end
 end

rb/spec/integration/selenium/webdriver/timeout_spec.rb

@@ -63,8 +63,6 @@
       expect(driver.find_elements(:class_name => "redbox")).to be_empty
     end
 
-    # TODO - File bug with Microsoft - This should not return Not Found Error - {POST} /session/{sessionId}/elements
-    # It should be a list
     it "should return after first attempt to find many after disabling implicit waits" do
       add = driver.find_element(:id => "adder")