remove webdriver and patch

jwcatprint · jwcatprint · commit adbb23aff78d · 2024-10-14T10:55:40.000-04:00
diff --git a/ruby/hyper-spec/hyper-spec.gemspec b/ruby/hyper-spec/hyper-spec.gemspec
@@ -21,7 +21,6 @@ Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
 
   spec.add_dependency 'actionview'
   spec.add_dependency 'capybara'
-  spec.add_dependency 'chromedriver-helper', '1.2.0'
   spec.add_dependency 'filecache'
   spec.add_dependency 'method_source'
   spec.add_dependency 'opal', ENV['OPAL_VERSION'] || '>= 0.11.0', '< 2.0'
@@ -31,7 +30,6 @@ Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
   spec.add_dependency 'timecop', '~> 0.8.1'
   spec.add_dependency 'uglifier'
   spec.add_dependency 'unparser', '>= 0.4.2'
-  spec.add_dependency 'webdrivers'
 
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'hyper-component', HyperSpec::VERSION
diff --git a/ruby/hyper-spec/lib/hyper-spec.rb b/ruby/hyper-spec/lib/hyper-spec.rb
@@ -26,7 +26,6 @@
 if defined?(Selenium::WebDriver::Firefox)
   require 'selenium/web_driver/firefox/profile'
 end
-require 'selenium-webdriver'
 
 require 'hyper-spec/version'
 
@@ -76,34 +75,8 @@ def self.reset_sessions!
   end
 end
 
-# TODO: figure out why we need this patch - its because we are on an old version
-# of Selenium Webdriver, but why?
 require 'selenium-webdriver'
 
-module Selenium
-  module WebDriver
-    module Chrome
-      module Bridge
-        COMMANDS = remove_const(:COMMANDS).dup
-        COMMANDS[:get_log] = [:post, 'session/:session_id/log']
-        COMMANDS.freeze
-
-        def log(type)
-          data = execute :get_log, {}, type: type.to_s
-
-          Array(data).map do |l|
-            begin
-              LogEntry.new l.fetch('level', 'UNKNOWN'), l.fetch('timestamp'), l.fetch('message')
-            rescue KeyError
-              next
-            end
-          end
-        end
-      end
-    end
-  end
-end
-
 module Capybara
   class << self
     alias old_reset_sessions! reset_sessions!
