Centralize Chrome headless driver configuration

- Remove duplicate Chrome driver registration from rails_helper.rb
- Centralize headless configuration in spec/support/driver_registration.rb
- Update rails_helper to call DriverRegistration.register_selenium_chrome_headless
- Clean up leftover empty object in CommentForm.jsx
- Fix RuboCop string literal style violations

This eliminates code duplication and ensures consistent headless configuration across the test suite.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: justin808

client/app/bundles/comments/components/CommentBox/CommentForm/CommentForm.jsx

@@ -356,7 +356,6 @@ class CommentForm extends BaseComponent {
           >
             {formatMessage(defaultMessages.formInline)}
           </button>
-          {}
         </div>
         {inputForm}
       </div>

spec/rails_helper.rb

@@ -42,21 +42,8 @@
   # Ensure that if we are running js tests, we are using latest webpack assets
   # This will use the defaults of :js and :server_rendering meta tags
   ReactOnRails::TestHelper.configure_rspec_to_compile_assets(config)
-  # Configure headless Chrome driver directly here
-  Capybara.register_driver :selenium_chrome_headless do |app|
-    options = Selenium::WebDriver::Chrome::Options.new
-    options.add_argument("--headless=new")
-    options.add_argument("--disable-gpu")
-    options.add_argument("--no-sandbox")
-    options.add_argument("--disable-dev-shm-usage")
-    options.add_argument("--window-size=1920,1080")
-
-    Capybara::Selenium::Driver.new(app, browser: :chrome, options: options)
-  end
-
-  Capybara::Screenshot.register_driver(:selenium_chrome_headless) do |js_driver, path|
-    js_driver.browser.save_screenshot(path)
-  end
+  # Register headless Chrome driver using centralized configuration
+  DriverRegistration.register_selenium_chrome_headless
 
   # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
   config.fixture_path = Rails.root.join("spec/fixtures")

spec/support/driver_registration.rb

@@ -26,21 +26,22 @@ def self.register_selenium_firefox
   end
 
   def self.register_selenium_chrome_headless
-    return if @selenium_headless_registered
+    # Force re-register to ensure our configuration is used
+    Capybara.drivers.delete(:selenium_chrome_headless)
 
     Capybara.register_driver :selenium_chrome_headless do |app|
-      options = Selenium::WebDriver::Chrome::Options.new
-      options.add_argument("--headless=new")
-      options.add_argument("--disable-gpu")
-      options.add_argument("--no-sandbox")
-      options.add_argument("--disable-dev-shm-usage")
-      options.add_argument("--window-size=1920,1080")
-
-      Capybara::Selenium::Driver.new(app, browser: :chrome, options: options)
+      browser_options = ::Selenium::WebDriver::Chrome::Options.new
+      browser_options.args << "--headless"
+      browser_options.args << "--disable-gpu"
+      browser_options.args << "--no-sandbox"
+      browser_options.args << "--disable-dev-shm-usage"
+      browser_options.args << "--window-size=1920,1080"
+
+      Capybara::Selenium::Driver.new(app, browser: :chrome, options: browser_options)
     end
-    Capybara::Screenshot.register_driver(:selenium_chrome_headless) do |js_driver, path|
-      js_driver.browser.save_screenshot(path)
+
+    Capybara::Screenshot.register_driver(:selenium_chrome_headless) do |driver, path|
+      driver.browser.save_screenshot(path)
     end
-    @selenium_headless_registered = true
   end
 end