feat: Add Ferrum::Network::Exchange#blob?

Author: route

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ### Added
 
+- `Ferrum::Network::Exchange#blob?` determines if the exchange is blob [#516]
 - `Ferrum::Network::Request#headers` are enhanced and supplemented with `Network.requestWillBeSentExtraInfo` [#506]
 - `Ferrum::Page#off` to unsubscribe from CDP events [#455]
 - `Ferrum::Mouse#scroll_by` to be able to scroll by, as alternative to `scroll_to` [#514]
@@ -12,6 +13,7 @@
 
 - Correctly set mouse events buttons property [#509]
 - Prevent 'Hash#[]=': can't add a new key into hash during iteration [#513]
+- `Ferrum::Network::Exchange#finished?` and `Ferrum::Network#wait_for_idle` take into account that request can be a blob [#496]
 
 ### Removed
 

lib/ferrum/network/exchange.rb

@@ -83,7 +83,7 @@ def blocked?
       # @return [Boolean]
       #
       def finished?
-        blocked? || response&.loaded? || !error.nil? || ping? || !!url&.start_with?("blob:")
+        blocked? || response&.loaded? || !error.nil? || ping? || blob?
       end
 
       #
@@ -131,6 +131,15 @@ def ping?
         !!request&.ping?
       end
 
+      #
+      # Determines if the exchange is blob.
+      #
+      # @return [Boolean]
+      #
+      def blob?
+        !!url&.start_with?("blob:")
+      end
+
       #
       # Returns request's URL.
       #

spec/network/exchange_spec.rb

@@ -139,6 +139,7 @@
 
     it "returns true for blob requests" do
       exchange = Ferrum::Network::Exchange.new(page, "1")
+      expect(exchange.blob?).to be false
       expect(exchange.finished?).to be false
 
       exchange.request = Ferrum::Network::Request.new(
@@ -149,6 +150,7 @@
           }
         }
       )
+      expect(exchange.blob?).to be true
       expect(exchange.finished?).to be true
     end
   end