actually do not camel case fx prefs (SeleniumHQ#9944)

follow-up to eb2d017
elgatov · Jun 27, 2022 · 2803a9d · 2803a9d
1 parent add4742
commit 2803a9d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/rb/lib/selenium/webdriver/firefox/options.rb b/rb/lib/selenium/webdriver/firefox/options.rb
@@ -175,7 +175,7 @@ def process_profile(profile)
         end
 
         def camelize?(key)
-          key != :prefs
+          key != "prefs"
         end
       end # Options
     end # Firefox

diff --git a/rb/spec/unit/selenium/webdriver/firefox/options_spec.rb b/rb/spec/unit/selenium/webdriver/firefox/options_spec.rb
@@ -149,6 +149,14 @@ module Firefox
             options.add_preference(:foo, 'bar')
             expect(options.prefs[:foo]).to eq('bar')
           end
+
+          it 'does not camelize preferences' do
+            options.add_preference('intl.accepted_languages', 'en-US')
+
+            prefs    = options.as_json['moz:firefoxOptions']['prefs']
+            expected = { 'intl.accepted_languages' => 'en-US' }
+            expect(prefs).to eq(expected)
+          end
         end
 
         describe '#enable_android' do