[rb] update headless! deprecations

titusfortner · titusfortner · commit 6c847fb19347 · 2023-01-14T17:38:28.000-06:00
diff --git a/rb/lib/selenium/webdriver/chromium/options.rb b/rb/lib/selenium/webdriver/chromium/options.rb
@@ -161,6 +161,9 @@ def add_preference(name, value)
 
         #
         # Run Chrome in headless mode.
+        # Old headless uses a non-production browser and is set with `--headless`
+        # Native headless from v86 - v108 is set with `--headless=chrome`
+        # Native headless from v109+ is set with `--headless=new`
         #
         # @example Enable headless mode
         #   options = Selenium::WebDriver::Chrome::Options.new
@@ -169,7 +172,7 @@ def add_preference(name, value)
 
         def headless!
           WebDriver.logger.deprecate('`Options#headless!`',
-                                     "`options = Selenium::WebDriver::Options.chrome(args: ['--headless=chrome'])`",
+                                     "`Options#add_argument('--headless=new')`",
                                      id: :headless)
           add_argument '--headless'
         end
diff --git a/rb/lib/selenium/webdriver/firefox/options.rb b/rb/lib/selenium/webdriver/firefox/options.rb
@@ -109,7 +109,7 @@ def add_preference(name, value)
 
         def headless!
           WebDriver.logger.deprecate('`Options#headless!`',
-                                     "`options = Selenium::WebDriver::Options.firefox(args: ['-headless'])`",
+                                     "`Options#add_argument('-headless')`",
                                      id: :headless)
           add_argument '-headless'
         end
diff --git a/rb/spec/unit/selenium/webdriver/chrome/options_spec.rb b/rb/spec/unit/selenium/webdriver/chrome/options_spec.rb
@@ -181,7 +181,9 @@ module Chrome
 
         describe '#headless!' do
           it 'adds necessary command-line arguments' do
-            options.headless!
+            expect {
+              options.headless!
+            }.to have_deprecated(:headless)
             expect(options.args).to eql(['--headless'])
           end
         end
diff --git a/rb/spec/unit/selenium/webdriver/edge/options_spec.rb b/rb/spec/unit/selenium/webdriver/edge/options_spec.rb
@@ -136,7 +136,9 @@ module Edge
 
         describe '#headless!' do
           it 'adds necessary command-line arguments' do
-            options.headless!
+            expect {
+              options.headless!
+            }.to have_deprecated(:headless)
             expect(options.args.to_a).to eql(['--headless'])
           end
         end
diff --git a/rb/spec/unit/selenium/webdriver/firefox/options_spec.rb b/rb/spec/unit/selenium/webdriver/firefox/options_spec.rb
@@ -127,7 +127,9 @@ module Firefox
 
         describe '#headless!' do
           it 'adds the -headless command-line flag' do
-            options.headless!
+            expect {
+              options.headless!
+            }.to have_deprecated(:headless)
             expect(options.as_json['moz:firefoxOptions']['args']).to include('-headless')
           end
         end
