[rb] fix failing specs for linter & Safari

Author: titusfortner

rb/lib/selenium/webdriver/remote/capabilities.rb

@@ -102,7 +102,7 @@ def firefox(opts = {})
 
           def safari(opts = {})
             new({
-              browser_name: Selenium::WebDriver::Safari.technology_preview? ? "Safari Technology Preview" : 'safari',
+              browser_name: Selenium::WebDriver::Safari.technology_preview? ? "Safari Technology Preview" : 'safari'
             }.merge(opts))
           end
 

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

@@ -159,9 +159,8 @@ module WebDriver
           expect(element.attribute(:value)).to eq('ContextClicked')
         end
 
-        it 'can release pressed buttons via release action', except: [{browser: :safari},
-                                                                      {driver: :remote, browser: :ie}],
-                                                             only: {browser: %i[edge chrome firefox ie]} do
+        it 'can release pressed buttons via release action', except: [{browser: %i[safari safari_preview]},
+                                                                      {driver: :remote, browser: :ie}] do
           driver.navigate.to url_for('javascriptPage.html')
 
           event_input = driver.find_element(id: 'clickField')

rb/spec/integration/selenium/webdriver/devtools_spec.rb

@@ -21,7 +21,7 @@
 
 module Selenium
   module WebDriver
-    describe DevTools, only: {driver: %i[chrome edge]} do
+    describe DevTools, exclusive: {driver: %i[chrome edge]} do
       let(:username) { SpecSupport::RackServer::TestApp::BASIC_AUTH_CREDENTIALS.first }
       let(:password) { SpecSupport::RackServer::TestApp::BASIC_AUTH_CREDENTIALS.last }
 

rb/spec/integration/selenium/webdriver/manager_spec.rb

@@ -84,7 +84,7 @@ module WebDriver
         it 'should not allow domain to be set for localhost',
            except: [{browser: %i[chrome edge],
                      reason: "https://bugs.chromium.org/p/chromedriver/issues/detail?id=3733"},
-                     {browser: %i[safari safari_preview]}] do
+                    {browser: %i[safari safari_preview]}] do
           expect {
             driver.manage.add_cookie name: 'domain',
                                      value: 'localhost',
@@ -94,8 +94,8 @@ module WebDriver
 
         it 'should not allow setting on a different domain',
            except: [{browser: %i[chrome edge],
-                      reason: "https://bugs.chromium.org/p/chromedriver/issues/detail?id=3733"},
-                     {browser: %i[safari safari_preview]}] do
+                     reason: "https://bugs.chromium.org/p/chromedriver/issues/detail?id=3733"},
+                    {browser: %i[safari safari_preview]}] do
           expect {
             driver.manage.add_cookie name: 'domain',
                                      value: 'different',
@@ -179,9 +179,8 @@ module WebDriver
 
           it 'should not allow adding with value None when secure is false',
              except: [{browser: :firefox,
-                      reason: "https://github.com/mozilla/geckodriver/issues/1842"},
+                       reason: "https://github.com/mozilla/geckodriver/issues/1842"},
                       {browser: %i[safari safari_preview]}] do
-
             expect {
               driver.manage.add_cookie name: 'samesite',
                                        value: 'none-insecure',

rb/spec/integration/selenium/webdriver/window_spec.rb

@@ -120,6 +120,7 @@ module WebDriver
       # https://github.com/mozilla/geckodriver/issues/1281
       it 'can make window full screen', only: {window_manager: true},
                                         exclude: [{driver: :remote, browser: :firefox, platform: :linux},
+                                                  {driver: :remote, browser: :safari},
                                                   {browser: %i[chrome edge]}] do
         window.size = old_size = Dimension.new(700, 700)