Skip to content

Commit 8527ced

Browse files
committed
rb - un-guard marionette tests
1 parent 02cfa9a commit 8527ced

File tree

1 file changed

+23
-25
lines changed

1 file changed

+23
-25
lines changed

rb/spec/integration/selenium/webdriver/firefox/marionette_spec.rb

Lines changed: 23 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -141,35 +141,33 @@ def restart_remote_server(path = nil)
141141

142142
compliant_on :browser => :marionette do
143143
# These pass in isolation, but can not run in suite due to https://github.com/SeleniumHQ/selenium/issues/1150
144-
not_compliant_on :driver => :remote do
145-
context "when designated firefox binary does not include Marionette" do
146-
let(:message) { /Marionette is not supported in Firefox Version \d\d/ }
147-
148-
before do
149-
unless ENV['PRE_MARIONETTE_BINARY']
150-
pending "Set ENV['PRE_MARIONETTE_BINARY'] to test features on firefox versions without marionette"
151-
end
144+
context "when designated firefox binary does not include Marionette" do
145+
let(:message) { /Marionette is not supported in Firefox Version \d\d/ }
146+
147+
before do
148+
unless ENV['PRE_MARIONETTE_BINARY']
149+
pending "Set ENV['PRE_MARIONETTE_BINARY'] to test features on firefox versions without marionette"
152150
end
151+
end
153152

154-
it "Raises Wires Exception when setting marionette option in capabilities" do
155-
begin
156-
caps = Selenium::WebDriver::Remote::Capabilities.firefox(:marionette => true,
157-
:firefox_binary => ENV['PRE_MARIONETTE_BINARY'])
158-
@opt.merge!(:desired_capabilities => caps)
159-
expect { Selenium::WebDriver.for :firefox, @opt }.to raise_exception Error::WebDriverError, message
160-
ensure
161-
Firefox::Binary.reset_path!
162-
end
153+
it "Raises Wires Exception when setting marionette option in capabilities" do
154+
begin
155+
caps = Selenium::WebDriver::Remote::Capabilities.firefox(:marionette => true,
156+
:firefox_binary => ENV['PRE_MARIONETTE_BINARY'])
157+
@opt.merge!(:desired_capabilities => caps)
158+
expect { Selenium::WebDriver.for :firefox, @opt }.to raise_exception Error::WebDriverError, message
159+
ensure
160+
Firefox::Binary.reset_path!
163161
end
162+
end
164163

165-
it "Raises Wires Exception when setting marionette option in driver initialization" do
166-
begin
167-
caps = Selenium::WebDriver::Remote::Capabilities.firefox(:firefox_binary => ENV['PRE_MARIONETTE_BINARY'])
168-
@opt.merge!(:marionette => true, :desired_capabilities => caps)
169-
expect { Selenium::WebDriver.for :firefox, @opt }.to raise_exception Error::WebDriverError, message
170-
ensure
171-
Firefox::Binary.reset_path!
172-
end
164+
it "Raises Wires Exception when setting marionette option in driver initialization" do
165+
begin
166+
caps = Selenium::WebDriver::Remote::Capabilities.firefox(:firefox_binary => ENV['PRE_MARIONETTE_BINARY'])
167+
@opt.merge!(:marionette => true, :desired_capabilities => caps)
168+
expect { Selenium::WebDriver.for :firefox, @opt }.to raise_exception Error::WebDriverError, message
169+
ensure
170+
Firefox::Binary.reset_path!
173171
end
174172
end
175173
end

0 commit comments

Comments
 (0)