Remove code for running specs without docker since that does not work

Author: mattbooks

spec/integration/ruby_server_js_client_spec.rb

@@ -55,7 +55,7 @@ def poll_for_js_result(script)
     # Start a Capybara server for our rack app
     Capybara::Server.new(rack_app).tap(&:boot)
   end
-  let(:server_host) { CapybaraServerHelper.server_host_for_browser(server) }
+  let(:server_host) { Socket.gethostname }
 
   let(:test_page) { "http://#{server_host}:#{server.port}/js-client/test.html" }
   let(:name) { "James\u1f61d" }

spec/integration/ruby_server_nodejs_client_spec.rb

@@ -36,7 +36,7 @@
 
   # Use Capybara::Server directly instead of creating a session that would use WebDriver
   let(:server) { Capybara::Server.new(rack_app).tap(&:boot) }
-  let(:server_host) { CapybaraServerHelper.server_host_for_client(server) }
+  let(:server_host) { '127.0.0.1' }
   let(:server_url) { "http://#{server_host}:#{server.port}" }
   let(:name) { "James\u1f61d" }
 

spec/integration/ruby_server_ruby_client_spec.rb

@@ -21,7 +21,7 @@
   # Use Capybara::Server directly instead of creating a session that would use WebDriver
   let(:server) { Capybara::Server.new(rack_app).tap(&:boot) }
 
-  let(:server_host) { CapybaraServerHelper.server_host_for_client(server) }
+  let(:server_host) { '127.0.0.1' }
   let(:client_url) { "http://#{basic_username}:#{basic_password}@#{server_host}:#{server.port}" }
   let(:client) do
     GRPCWeb::Client.new(

spec/support/capybara.rb

@@ -50,21 +50,9 @@
   config.server = :webrick
   config.always_include_port = true
 
-  if ENV['CI'] || ENV['DOCKER']
-    # In Docker/CI, bind to all interfaces
-    # Use the container's hostname which docker compose --use-aliases makes resolvable
-    config.server_host = '0.0.0.0'
-    config.default_driver = :selenium_chrome_standalone
-    config.javascript_driver = :selenium_chrome_standalone
-  else
-    config.server_host = IPSocket.getaddress(Socket.gethostname)
-    config.app_host = "http://#{Socket.gethostname}"
-    if ENV['CHROME']
-      config.javascript_driver = :selenium_chrome
-    else
-      config.javascript_driver = :selenium_chrome_headless
-    end
-  end
+  config.server_host = '0.0.0.0'
+  config.default_driver = :selenium_chrome_standalone
+  config.javascript_driver = :selenium_chrome_standalone
 
   puts "Capybara is using the #{config.javascript_driver} driver for javascript tests"
 end
@@ -73,27 +61,6 @@
 # https://github.com/puzzle/skills/issues/800
 Selenium::WebDriver.logger.ignore(:clear_local_storage, :clear_session_storage)
 
-# Helper module for getting the correct server host in different environments
-module CapybaraServerHelper
-  def self.server_host_for_client(server)
-    # In Docker, use localhost since client runs in same container as server
-    if ENV['CI'] || ENV['DOCKER']
-      '127.0.0.1'
-    else
-      server.host
-    end
-  end
-
-  def self.server_host_for_browser(_server)
-    # In Docker, use the container's hostname since docker compose --use-aliases makes it resolvable
-    if ENV['CI'] || ENV['DOCKER']
-      Socket.gethostname
-    else
-      Capybara.server_host
-    end
-  end
-end
-
 RSpec.configure do |config|
   config.before(type: :feature) { Capybara.app = TestGRPCWebApp.build }