Run Formatter (izelnakri#122)

Author: maennchen

lib/paper_trail.ex

@@ -38,7 +38,18 @@ defmodule PaperTrail do
   @doc """
   Inserts a record to the database with a related version insertion in one transaction
   """
-  def insert(changeset, options \\ [origin: nil, meta: nil, originator: nil, prefix: nil, model_key: :model, version_key: :version, ecto_options: []]) do
+  def insert(
+        changeset,
+        options \\ [
+          origin: nil,
+          meta: nil,
+          originator: nil,
+          prefix: nil,
+          model_key: :model,
+          version_key: :version,
+          ecto_options: []
+        ]
+      ) do
     PaperTrail.Multi.new()
     |> PaperTrail.Multi.insert(changeset, options)
     |> PaperTrail.Multi.commit()
@@ -47,7 +58,18 @@ defmodule PaperTrail do
   @doc """
   Same as insert/2 but returns only the model struct or raises if the changeset is invalid.
   """
-  def insert!(changeset, options \\ [origin: nil, meta: nil, originator: nil, prefix: nil, model_key: :model, version_key: :version, ecto_options: []]) do
+  def insert!(
+        changeset,
+        options \\ [
+          origin: nil,
+          meta: nil,
+          originator: nil,
+          prefix: nil,
+          model_key: :model,
+          version_key: :version,
+          ecto_options: []
+        ]
+      ) do
     repo = RepoClient.repo()
     ecto_options = options[:ecto_options] || []
 

lib/paper_trail/multi.ex

@@ -31,7 +31,7 @@ defmodule PaperTrail.Multi do
           originator: nil,
           prefix: nil,
           model_key: :model,
-          version_key: :version, 
+          version_key: :version,
           ecto_options: []
         ]
       ) do

mix.exs

@@ -29,7 +29,6 @@ defmodule PaperTrail.Mixfile do
     [
       {:ecto, ">= 3.4.6"},
       {:ecto_sql, ">= 3.4.5"},
-
       {:ex_doc, ">= 0.0.0", only: :dev, runtime: false},
       {:jason, ">= 1.2.0", only: [:dev, :test]},
       {:postgrex, ">= 0.0.0", only: [:dev, :test]}

test/paper_trail/bang_functions_simple_mode_test.exs

@@ -87,12 +87,17 @@ defmodule PaperTrailTest.SimpleModeBangFunctions do
 
   test "PaperTrail.insert!/2 passes ecto options through (e.g. upsert options)" do
     user = create_user()
-    _result = create_company_with_version(@create_company_params, [originator: user])
+    _result = create_company_with_version(@create_company_params, originator: user)
 
     new_create_company_params = @create_company_params |> Map.replace!(:city, "Barcelona")
 
-    ecto_options =  [on_conflict: {:replace_all_except, ~w{name}a}, conflict_target: :name]
-    result = create_company_with_version(new_create_company_params, [originator: user, ecto_options: ecto_options])
+    ecto_options = [on_conflict: {:replace_all_except, ~w{name}a}, conflict_target: :name]
+
+    result =
+      create_company_with_version(new_create_company_params,
+        originator: user,
+        ecto_options: ecto_options
+      )
 
     assert Company.count() == 1
     assert Version.count() == 2

test/paper_trail/bang_functions_strict_mode_test.exs

@@ -124,12 +124,17 @@ defmodule PaperTrailTest.StrictModeBangFunctions do
 
   test "PaperTrail.insert!/2 passes ecto options through (e.g. upsert options)" do
     user = create_user()
-    _result = create_company_with_version(@create_company_params, [originator: user])
+    _result = create_company_with_version(@create_company_params, originator: user)
 
     new_create_company_params = @create_company_params |> Map.replace!(:city, "Barcelona")
 
-    ecto_options =  [on_conflict: {:replace_all_except, ~w{name}a}, conflict_target: :name]
-    result = create_company_with_version(new_create_company_params, [originator: user, ecto_options: ecto_options])
+    ecto_options = [on_conflict: {:replace_all_except, ~w{name}a}, conflict_target: :name]
+
+    result =
+      create_company_with_version(new_create_company_params,
+        originator: user,
+        ecto_options: ecto_options
+      )
 
     assert Company.count() == 1
     assert Version.count() == 2
@@ -1046,5 +1051,4 @@ defmodule PaperTrailTest.StrictModeBangFunctions do
   defp convert_to_string_map(map) do
     map |> Jason.encode!() |> Jason.decode!()
   end
-
 end

test/paper_trail/base_test.exs

@@ -99,17 +99,25 @@ defmodule PaperTrailTest do
 
   test "PaperTrail.insert/2 passes ecto options through (e.g. upsert options)" do
     user = create_user()
-    {:ok, _result} = create_company_with_version(@create_company_params, [originator: user])
+    {:ok, _result} = create_company_with_version(@create_company_params, originator: user)
 
     new_create_company_params = @create_company_params |> Map.replace!(:city, "Barcelona")
 
-    ecto_options =  [on_conflict: {:replace_all_except, ~w{name}a}, conflict_target: :name]
-    {:ok, result} = create_company_with_version(new_create_company_params, [originator: user, ecto_options: ecto_options])
+    ecto_options = [on_conflict: {:replace_all_except, ~w{name}a}, conflict_target: :name]
+
+    {:ok, result} =
+      create_company_with_version(new_create_company_params,
+        originator: user,
+        ecto_options: ecto_options
+      )
 
     assert Company.count() == 1
     assert Version.count() == 2
 
-    assert Map.take(serialize(result[:model]), [:name, :city]) == %{name: "Acme LLC", city: "Barcelona"}
+    assert Map.take(serialize(result[:model]), [:name, :city]) == %{
+             name: "Acme LLC",
+             city: "Barcelona"
+           }
   end
 
   test "updating a company with originator creates a correct company version" do

test/paper_trail/strict_mode_test.exs

@@ -131,17 +131,25 @@ defmodule PaperTrailStrictModeTest do
 
   test "PaperTrail.insert/2 passes ecto options through (e.g. upsert options)" do
     user = create_user()
-    {:ok, _result} = create_company_with_version(@create_company_params, [originator: user])
+    {:ok, _result} = create_company_with_version(@create_company_params, originator: user)
 
     new_create_company_params = @create_company_params |> Map.replace!(:city, "Barcelona")
 
-    ecto_options =  [on_conflict: {:replace_all_except, ~w{name}a}, conflict_target: :name]
-    {:ok, result} = create_company_with_version(new_create_company_params, [originator: user, ecto_options: ecto_options])
+    ecto_options = [on_conflict: {:replace_all_except, ~w{name}a}, conflict_target: :name]
+
+    {:ok, result} =
+      create_company_with_version(new_create_company_params,
+        originator: user,
+        ecto_options: ecto_options
+      )
 
     assert Company.count() == 1
     assert Version.count() == 2
 
-    assert Map.take(serialize(result[:model]), [:name, :city]) == %{name: "Acme LLC", city: "Barcelona"}
+    assert Map.take(serialize(result[:model]), [:name, :city]) == %{
+             name: "Acme LLC",
+             city: "Barcelona"
+           }
   end
 
   test "updating a company creates a company version with correct item_changes" do

test/support/simple_models.exs

@@ -1,4 +1,3 @@
-
 defmodule LocationType do
   use Ecto.Type