Exchange#finished? returns true if no response expected

lib/ferrum/network/exchange.rb

@@ -79,7 +79,7 @@ def blocked?
       # @return [Boolean]
       #
       def finished?
-        blocked? || response&.loaded? || !error.nil?
+        blocked? || response&.loaded? || !error.nil? || !response_expected?
       end
 
       #

spec/network/exchange_spec.rb

@@ -109,6 +109,22 @@
 
       expect(last_exchange.finished?).to be true
     end
+
+    it "returns true if an error occurred" do
+      exchange = Ferrum::Network::Exchange.new(page, "1")
+
+      expect(exchange.finished?).to be false
+      exchange.error = Ferrum::Network::Error.new
+      expect(exchange.finished?).to be true
+    end
+
+    it "returns true for ping requests" do
+      exchange = Ferrum::Network::Exchange.new(page, "1")
+      expect(exchange.finished?).to be false
+
+      exchange.request = Ferrum::Network::Request.new({"type" => "Ping"})
+      expect(exchange.finished?).to be true
+    end
   end
 
   describe "#redirect?" do