Version 2.0.0-prerelease

Author: sobolevn

CONTRIBUTING.md

@@ -1,6 +1,7 @@
 # Contributing to EctoAutoslugField
 
 ## Pull Requests Welcome
+
 1. Fork `ecto_autoslug_field`
 2. Create a topic branch
 3. Make logically-grouped commits with clear commit messages
@@ -11,6 +12,7 @@
 
 Please, make sure that all these commands succeed before pushing anything:
 
+0. `mix format` your code
 1. `mix test`
 2. `mix credo --strict`
 3. `mix dialyzer` (it may take long on the first run)

lib/ecto_autoslug_field/slug_generator.ex

@@ -45,11 +45,14 @@ defmodule EctoAutoslugField.SlugGenerator do
   # Private functions:
 
   defp do_build_slug(source) when is_binary(source) do
-    source |> Slugger.slugify_downcase()
+    source
+    |> Slugger.slugify_downcase()
   end
 
   defp do_build_slug(sources) do
-    sources |> Enum.join("-") |> Slugger.slugify_downcase()
+    sources
+    |> Enum.join("-")
+    |> Slugger.slugify_downcase()
   end
 
   defp do_generate_slug(changeset, sources, opts) do

mix.lock

@@ -2,10 +2,10 @@
   "bunt": {:hex, :bunt, "0.2.0", "951c6e801e8b1d2cbe58ebbd3e616a869061ddadcc4863d0a2182541acae9a38", [:mix], [], "hexpm"},
   "certifi": {:hex, :certifi, "2.5.1", "867ce347f7c7d78563450a18a6a28a8090331e77fa02380b4a21962a65d36ee5", [:rebar3], [{:parse_trans, "~>3.3", [hex: :parse_trans, repo: "hexpm", optional: false]}], "hexpm"},
   "credo": {:hex, :credo, "1.1.0", "e0c07b2fd7e2109495f582430a1bc96b2c71b7d94c59dfad120529f65f19872f", [:mix], [{:bunt, "~> 0.2.0", [hex: :bunt, repo: "hexpm", optional: false]}, {:jason, "~> 1.0", [hex: :jason, repo: "hexpm", optional: false]}], "hexpm"},
-  "decimal": {:hex, :decimal, "1.7.0", "30d6b52c88541f9a66637359ddf85016df9eb266170d53105f02e4a67e00c5aa", [:mix], [], "hexpm"},
+  "decimal": {:hex, :decimal, "1.8.0", "ca462e0d885f09a1c5a342dbd7c1dcf27ea63548c65a65e67334f4b61803822e", [:mix], [], "hexpm"},
   "dialyxir": {:hex, :dialyxir, "0.5.1", "b331b091720fd93e878137add264bac4f644e1ddae07a70bf7062c7862c4b952", [:mix], [], "hexpm"},
   "earmark": {:hex, :earmark, "1.3.2", "b840562ea3d67795ffbb5bd88940b1bed0ed9fa32834915125ea7d02e35888a5", [:mix], [], "hexpm"},
-  "ecto": {:hex, :ecto, "3.0.8", "9eb6a1fcfc593e6619d45ef51afe607f1554c21ca188a1cd48eecc27223567f1", [:mix], [{:decimal, "~> 1.6", [hex: :decimal, repo: "hexpm", optional: false]}, {:jason, "~> 1.0", [hex: :jason, repo: "hexpm", optional: true]}, {:poison, "~> 2.2 or ~> 3.0", [hex: :poison, repo: "hexpm", optional: true]}], "hexpm"},
+  "ecto": {:hex, :ecto, "3.1.7", "fa21d06ef56cdc2fdaa62574e8c3ba34a2751d44ea34c30bc65f0728421043e5", [:mix], [{:decimal, "~> 1.6", [hex: :decimal, repo: "hexpm", optional: false]}, {:jason, "~> 1.0", [hex: :jason, repo: "hexpm", optional: true]}], "hexpm"},
   "ex_doc": {:hex, :ex_doc, "0.20.2", "1bd0dfb0304bade58beb77f20f21ee3558cc3c753743ae0ddbb0fd7ba2912331", [:mix], [{:earmark, "~> 1.3", [hex: :earmark, repo: "hexpm", optional: false]}, {:makeup_elixir, "~> 0.10", [hex: :makeup_elixir, repo: "hexpm", optional: false]}], "hexpm"},
   "excoveralls": {:hex, :excoveralls, "0.11.1", "dd677fbdd49114fdbdbf445540ec735808250d56b011077798316505064edb2c", [:mix], [{:hackney, "~> 1.0", [hex: :hackney, repo: "hexpm", optional: false]}, {:jason, "~> 1.0", [hex: :jason, repo: "hexpm", optional: false]}], "hexpm"},
   "exjsx": {:hex, :exjsx, "4.0.0", "60548841e0212df401e38e63c0078ec57b33e7ea49b032c796ccad8cde794b5c", [:mix], [{:jsx, "~> 2.8.0", [hex: :jsx, repo: "hexpm", optional: false]}], "hexpm"},

test/ecto_autoslug_field_test/slug_generator_test.exs

@@ -11,26 +11,26 @@ defmodule EctoAutoslugField.SlugGeneratorTest do
      %{
        user: User.changeset(%User{}, @valid_attrs),
        opts: [
-         to: :slug,
+         to: :simple_slug,
          slug_builder: &build_slug/2
        ]
      }}
   end
 
   test "maybe_generate_slug with single source", fixture do
     changeset = maybe_generate_slug(fixture.user, :name, fixture.opts)
-    assert changeset.changes.slug == "nikita-sobolev"
+    assert changeset.changes.simple_slug == "nikita-sobolev"
   end
 
   test "maybe_generate_slug with multiple sources", fixture do
     changeset =
       maybe_generate_slug(fixture.user, [:name, :company], fixture.opts)
 
-    assert changeset.changes.slug == "nikita-sobolev-wemake-services"
+    assert changeset.changes.simple_slug == "nikita-sobolev-wemake-services"
   end
 
   test "maybe_generate_slug with 'nil'", fixture do
     changeset = maybe_generate_slug(fixture.user, nil, fixture.opts)
-    refute Map.has_key?(changeset.changes, fixture.opts[:to])
+    assert Map.has_key?(changeset.changes, fixture.opts[:to])
   end
 end