Merge pull request #83 from keathley/allow-users-to-turn-off-js-errors

Fix issue with disabling js-errors

lib/wallaby.ex

@@ -48,7 +48,7 @@ defmodule Wallaby do
   end
 
   def js_errors? do
-    Application.get_env(:wallaby, :js_errors) || true
+    Application.get_env(:wallaby, :js_errors, true)
   end
 
   def phantomjs_path do

test/wallaby/driver/configuration_test.exs

@@ -0,0 +1,14 @@
+defmodule Wallaby.Driver.ConfigurationTest do
+  use Wallaby.SessionCase, async: false
+
+  test "js errors can be disabled", %{session: session, server: server} do
+    Application.put_env(:wallaby, :js_errors, false)
+
+    session
+    |> visit(server.base_url <> "/errors.html")
+    |> click_on("Throw an Error")
+    |> assert
+
+    Application.put_env(:wallaby, :js_errors, nil)
+  end
+end

test/wallaby/driver_test.exs → test/wallaby/driver/js_errors_test.exs

@@ -1,4 +1,4 @@
-defmodule Wallaby.DriverTest do
+defmodule Wallaby.Driver.JSErrorsTest do
   use Wallaby.SessionCase, async: true
 
   import ExUnit.CaptureIO