hound icon indicating copy to clipboard operation
hound copied to clipboard

Any API function can return {:error, :closed}

Open Kukunin opened this issue 7 years ago • 1 comments

When web driver closes a connection (because of a crash or something similar), Hound might return {:error, :closed} literally from every helper function.

It's pretty dangerous since it's pretty unexpected, and it can feed wrong data further into an application.

One example is that Hound.Session.active_sessions() |> Enum.count crashes with (Protocol.UndefinedError) protocol Enumerable not implemented for {:error, :closed}, as in #225

Another example was that it silently returned {:error, :closed} from find_element(:css, "img#iImage") |> attribute_value("src"), causing the other part of system fail, because it expected a string.

It's better to raise exception in such cases, since it's unexpected case, and unexpected output from functions

Kukunin avatar Nov 05 '18 21:11 Kukunin

At least, it might be ok to return {:error, :closed} on safe methods, such as search_element. but otherwise I expect it to crash

Kukunin avatar Nov 05 '18 21:11 Kukunin