HashNuke / hound

Elixir library for writing integration tests and browser automation
http://hexdocs.pm/hound
MIT License
1.36k stars 145 forks source link

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

Open Kukunin opened 6 years ago

Kukunin commented 6 years ago

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 commented 6 years ago

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