Skip to content

Commit 1646a09

Browse files
committed
Update Safari specs guards
1 parent 6ed8d51 commit 1646a09

File tree

6 files changed

+9
-9
lines changed

6 files changed

+9
-9
lines changed

rb/spec/integration/selenium/webdriver/action_builder_spec.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ module WebDriver
7878
expect(input.attribute(:value)).to eq('abcddcba')
7979
end
8080

81-
it 'can send non-ASCII keys' do
81+
it 'can send non-ASCII keys', except: {browser: :safari} do
8282
driver.navigate.to url_for('formPage.html')
8383

8484
input = driver.find_element(css: '#working')
@@ -150,15 +150,15 @@ module WebDriver
150150
expect(element.attribute(:value)).to eq('DoubleClicked')
151151
end
152152

153-
it 'context clicks an element', exclude: {browser: %i[safari safari_preview]} do
153+
it 'context clicks an element', except: {browser: %i[safari]} do
154154
driver.navigate.to url_for('javascriptPage.html')
155155
element = driver.find_element(id: 'doubleClickField')
156156

157157
driver.action.context_click(element).perform
158158
expect(element.attribute(:value)).to eq('ContextClicked')
159159
end
160160

161-
it 'can release pressed buttons via release action', exclude: {browser: :safari}, only: {browser: :firefox} do
161+
it 'can release pressed buttons via release action', except: {browser: :safari}, only: {browser: :firefox} do
162162
driver.navigate.to url_for('javascriptPage.html')
163163

164164
event_input = driver.find_element(id: 'clickField')

rb/spec/integration/selenium/webdriver/driver_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ module Selenium
2121
module WebDriver
2222
describe Driver do
2323
it_behaves_like 'driver that can be started concurrently', except: [{browser: :edge},
24-
{driver: :safari}],
25-
exclude: {browser: :safari_preview}
24+
{driver: :safari},
25+
{browser: :safari_preview}]
2626

2727
it 'should get the page title' do
2828
driver.navigate.to url_for('xhtmlTest.html')

rb/spec/integration/selenium/webdriver/element_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ module WebDriver
3333
.to raise_error(Selenium::WebDriver::Error::UnknownError, error)
3434
end
3535

36-
it 'should not raise if element is only partially covered', only: {browser: %i[ff_esr ie safari safari_preview]} do
36+
it 'should not raise if element is only partially covered', only: {browser: %i[ff_esr ie safari]} do
3737
driver.navigate.to url_for('click_tests/overlapping_elements.html')
3838
expect { driver.find_element(id: 'other_contents').click }.not_to raise_error
3939
end

rb/spec/integration/selenium/webdriver/keyboard_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ module WebDriver
5454
expect { driver.keyboard.press :return }.to raise_error(ArgumentError)
5555
end
5656

57-
it 'can press and release modifier keys', except: {browser: :safari} do
57+
it 'can press and release modifier keys', except: {browser: %i[safari safari_preview]} do
5858
driver.navigate.to url_for('javascriptPage.html')
5959

6060
event_input = driver.find_element(id: 'theworks')

rb/spec/integration/selenium/webdriver/mouse_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ module WebDriver
5757
end
5858
end
5959

60-
it 'context clicks an element', exclude: {browser: :safari} do
60+
it 'context clicks an element', except: {browser: :safari} do
6161
driver.navigate.to url_for('javascriptPage.html')
6262
element = driver.find_element(id: 'doubleClickField')
6363

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,7 @@ module WebDriver
294294
end
295295

296296
# Safari - Raises wrong error
297-
context 'unhandled alert error', exclude: {browser: %i[safari safari_preview]} do
297+
context 'unhandled alert error', except: {browser: %i[safari safari_preview]} do
298298
after { reset_driver! }
299299

300300
it 'raises an UnhandledAlertError if an alert has not been dealt with', except: {browser: %i[ie firefox]} do

0 commit comments

Comments
 (0)