fix: handle new response from chromedriver

lib/wallaby/httpclient.ex

@@ -53,7 +53,7 @@ defmodule Wallaby.HTTPClient do
   defp make_request(method, url, body, retry_count, retry_reasons) do
     method
     |> HTTPoison.request(url, body, headers(), request_opts())
-    |> handle_response
+    |> handle_response()
     |> case do
       {:error, :httpoison, error} ->
         :timer.sleep(jitter())
@@ -102,8 +102,9 @@ defmodule Wallaby.HTTPClient do
           {:ok, response}
           | {:error, web_driver_error_reason}
           | no_return
-  # credo:disable-for-next-line Credo.Check.Refactor.CyclomaticComplexity
   defp check_for_response_errors(response) do
+    response = coerce_json_message(response)
+
     case Map.get(response, "value") do
       %{"class" => "org.openqa.selenium.StaleElementReferenceException"} ->
         {:error, :stale_reference}
@@ -120,6 +121,9 @@ defmodule Wallaby.HTTPClient do
       } ->
         {:error, :stale_reference}
 
+      %{"message" => %{"value" => "An invalid or illegal selector was specified"}} ->
+        {:error, :invalid_selector}
+
       %{"message" => "invalid selector" <> _} ->
         {:error, :invalid_selector}
 
@@ -158,4 +162,24 @@ defmodule Wallaby.HTTPClient do
   end
 
   defp jitter, do: :rand.uniform(@max_jitter)
+
+  defp coerce_json_message(%{"value" => %{"message" => message} = value} = response) do
+    value =
+      case Regex.named_captures(~r/(?<type>.*): (?<payload>{.*})\n.*/, message) do
+        %{"payload" => payload, "type" => type} ->
+          %{
+            "message" => Jason.decode!(payload),
+            "type" => type
+          }
+
+        _ ->
+          value
+      end
+
+    put_in(response["value"], value)
+  end
+
+  defp coerce_json_message(response) do
+    response
+  end
 end