Merge pull request #86 from maciesielka/allow-lowercase-keys

[Bugfix] Allow lowercase strings in `send_keys`

lib/capybara/playwright/node.rb

@@ -548,7 +548,7 @@ def initialize(element_or_keyboard, keys)
                   _key = key.last
                   code =
                     if _key.is_a?(String) && _key.length == 1
-                      _key.upcase
+                      _key
                     elsif _key.is_a?(Symbol)
                       key_for(_key)
                     else
@@ -567,7 +567,7 @@ def initialize(element_or_keyboard, keys)
                 when String
                   key.each_char do |char|
                     executables << PressKey.new(
-                      key: char.upcase,
+                      key: char,
                       modifiers: modifiers,
                     )
                   end
@@ -600,6 +600,10 @@ def execute
 
         class PressKey
           def initialize(key:, modifiers:)
+            # Shift always requires uppercase key
+            # See https://playwright.dev/docs/input#keys-and-shortcuts
+            key = key.upcase if modifiers.include?(MODIFIERS[:shift])
+
             # puts "PressKey: key=#{key} modifiers: #{modifiers}"
             if modifiers.empty?
               @key = key

spec/feature/example_spec.rb

@@ -92,6 +92,24 @@
     end
   end
 
+  it 'can send keys without modifier' do
+    Capybara.app_host = 'https://github.com'
+    visit '/'
+
+    find('body').send_keys ['s']
+
+    expect(page).to have_field('query-builder-test')
+  end
+
+  it 'can send keys with modifier' do
+    Capybara.app_host = 'https://tailwindcss.com/'
+    visit '/'
+
+    find('body').send_keys [:control, 'k']
+
+    expect(page).to have_field('docsearch-input')
+  end
+
   it 'does not silently pass when browser has not been started' do
     expect do
       page.driver.with_playwright_page do |_page|