add delete and clear request handlers

Author: aguspe

rb/lib/selenium/webdriver/bidi/network.rb

@@ -26,7 +26,7 @@ class Network
           response_started: 'network.responseStarted',
           response_completed: 'network.responseCompleted',
           auth_required: 'network.authRequired',
-          FETCH_ERROR: 'network.fetchError'
+          fetch_error: 'network.fetchError'
         }.freeze
 
         PHASES = {
@@ -76,11 +76,7 @@ def on(event, &)
           event = EVENTS[event] if event.is_a?(Symbol)
           @bidi.add_callback(event, &)
         end
-      end
-
-      # Network
-    end
-
-    # BiDi
+      end # Network
+    end # BiDi
   end # WebDriver
 end # Selenium

rb/lib/selenium/webdriver/common/network.rb

@@ -60,6 +60,16 @@ def add_request_handler
 
         request_id
       end
+
+      def remove_request_handler(id)
+        intercept = @request_callbacks[id]
+        @network.remove_intercept(intercept['intercept'])
+        @request_callbacks.delete(id)
+      end
+
+      def clear_request_handlers
+        @request_callbacks.each_key { |id| remove_request_handler(id) }
+      end
     end # Network
   end # WebDriver
 end # Selenium

rb/sig/lib/selenium/webdriver/common/network.rbs

@@ -15,7 +15,11 @@ module Selenium
 
       def clear_authentication_handlers: -> Hash[nil, nil]
 
+      def clear_request_handlers: -> untyped
+
       def remove_authentication_handler: (String id) -> nil
+
+      def remove_request_handler: -> untyped
     end
   end
 end

rb/spec/integration/selenium/webdriver/network_spec.rb

@@ -61,6 +61,25 @@ module WebDriver
           expect(network.request_callbacks.count).to be 1
         end
       end
+
+      it 'removes a request handler' do
+        reset_driver!(web_socket_url: true) do |driver|
+          network = described_class.new(driver)
+          id = network.add_request_handler
+          network.remove_request_handler(id)
+          expect(network.request_callbacks.count).to be 0
+        end
+      end
+
+      it 'clears all request handlers' do
+        reset_driver!(web_socket_url: true) do |driver|
+          network = described_class.new(driver)
+          network.add_request_handler
+          network.add_request_handler
+          network.clear_request_handlers
+          expect(network.request_callbacks.count).to be 0
+        end
+      end
     end
   end
 end