Fix Credo Trailing Whitespace (#630)

lib/event_emitter.ex

@@ -2,7 +2,9 @@ defmodule EventEmitter do
   @moduledoc false
   # This module offers telemetry style event emission for testing purposes.
 
-  # If you'd like to emit a message to the event stream, you can call `emit/1` from your implementation code. This is macro, and will not result in any AST injection when not being compiled in the test env.
+  # If you'd like to emit a message to the event stream, you can call `emit/1` from your
+  # implementation code. This is macro, and will not result in any AST injection when not being
+  # compiled in the test env.
 
   # ```elixir
   # defmodule ImplMod do
@@ -11,12 +13,17 @@ defmodule EventEmitter do
   #   def implementation do
   #     # some logic
 
-  #     emit %{name: :implementation, module: __MODULE__, metadata: %{unique_identifier: some_variable}}
+  #     emit %{
+  #       name: :implementation,
+  #       module: __MODULE__,
+  #       metadata: %{unique_identifier: some_variable}
+  #     }
   #   end
   # end
   # ```
 
-  # If you'd like to await on a message emitted by implementation code, you can call `await/3` from your test code after registering a handler for your test process
+  # If you'd like to await on a message emitted by implementation code, you can call `await/3` from
+  # your test code after registering a handler for your test process
 
   # ```elixir
   # defmodule TestMod do

lib/wallaby.ex

@@ -89,9 +89,9 @@ defmodule Wallaby do
   """
   @spec end_session(Session.t()) :: :ok | {:error, reason}
   def end_session(%Session{driver: driver} = session) do
-    with :ok <- SessionStore.demonitor(session),
-         :ok <- driver.end_session(session),
-         do: :ok
+    with :ok <- SessionStore.demonitor(session) do
+      driver.end_session(session)
+    end
   end
 
   @doc false

lib/wallaby/browser.ex

@@ -1,5 +1,4 @@
 defmodule Wallaby.Browser do
-  # credo:disable-for-this-file Credo.Check.Readability.TrailingWhiteSpace
   @moduledoc """
   The Browser module is the entrypoint for interacting with a real browser.
 

lib/wallaby/helpers/key_codes.ex

@@ -53,8 +53,7 @@ defmodule Wallaby.Helpers.KeyCodes do
     unicode =
       keys
       |> Enum.reduce([], fn x, acc -> acc ++ split_strings(x) end)
-      |> Enum.map(&"\"#{code(&1)}\"")
-      |> Enum.join(",")
+      |> Enum.map_join(",", &"\"#{code(&1)}\"")
 
     "{\"value\": [#{unicode}]}"
   end

lib/wallaby/httpclient.ex

@@ -79,9 +79,9 @@ defmodule Wallaby.HTTPClient do
 
       {:ok, %HTTPoison.Response{body: body}} ->
         with {:ok, decoded} <- Jason.decode(body),
-             {:ok, response} <- check_status(decoded),
-             {:ok, validated} <- check_for_response_errors(response),
-             do: {:ok, validated}
+             {:ok, response} <- check_status(decoded) do
+          check_for_response_errors(response)
+        end
     end
   end
 

lib/wallaby/selenium.ex

@@ -181,10 +181,11 @@ defmodule Wallaby.Selenium do
 
   @doc false
   def current_path(%Session{} = session) do
-    with {:ok, url} <- WebdriverClient.current_url(session),
-         uri <- URI.parse(url),
-         {:ok, path} <- Map.fetch(uri, :path),
-         do: {:ok, path}
+    with {:ok, url} <- WebdriverClient.current_url(session) do
+      url
+      |> URI.parse()
+      |> Map.fetch(:path)
+    end
   end
 
   @doc false