Change attachment parameter format to upcoming version of gen_smtp; elixir-refmt

Author: skosch

lib/mailman/attachment.ex

@@ -100,21 +100,21 @@ defmodule Mailman.Render do
   end
 
   def parameters_for({:attachment, _body, attachment}) do
-    [
-      {"transfer-encoding", "base64"},
-      {"content-type-params", []},
-      {"disposition", attachment.disposition},
-      {"disposition-params", [{"filename", attachment.file_name}]}
-    ]
+    %{
+      transfer_encoding: "base64",
+      content_type_params: [],
+      disposition: attachment.disposition,
+      disposition_params: [{"filename", attachment.file_name}]
+    }
   end
 
   def parameters_for(_part) do
-    [
-      {"transfer-encoding", "quoted-printable"},
-      {"content-type-params", []},
-      {"disposition", "inline"},
-      {"disposition-params", []}
-    ]
+    %{
+      transfer_encoding: "quoted-printable",
+      content_type_params: [],
+      disposition: "inline",
+      disposition_params: [],
+    }
   end
 
   def headers_for({:plain, _body}), do: []

lib/mailman/render.ex

@@ -20,11 +20,21 @@ defmodule Mailman.Mixfile do
     [applications: [:ssl, :crypto, :eiconv, :gen_smtp, :httpotion]]
   end
 
+  # Note that :eiconv encoder/decoder is used by gen_smtp as well,
+  # and will not be replaced by the newer iconv (see https://github.com/gen-smtp/gen_smtp/issues/95)
+  #
+  # If the eiconv NIF fails to compile, try updating rebar:
+  #> mix local.rebar
+  #> rm -rf deps
+  #> rm -rf _build
+  #> mix deps.get
+  #> mix
+
   # Returns the list of dependencies in the format:
   defp deps do
     [
       {:eiconv, "~> 1.0.0"},
-      {:gen_smtp, "~> 0.14.0"},
+      {:gen_smtp, "~> 0.15.0"},
       {:ex_doc, ">= 0.19.1", only: :dev},
       {:httpotion, "~> 3.1.0"},
       {:credo, "~> 0.10", only: [:dev, :test], runtime: false}

mix.exs

@@ -4,7 +4,7 @@
   "earmark": {:hex, :earmark, "1.3.0", "17f0c38eaafb4800f746b457313af4b2442a8c2405b49c645768680f900be603", [:mix], [], "hexpm"},
   "eiconv": {:hex, :eiconv, "1.0.0", "ee1e47ee37799a05beff7a68d61f63cccc93101833c4fb94b454c23b12a21629", [:rebar3], []},
   "ex_doc": {:hex, :ex_doc, "0.19.1", "519bb9c19526ca51d326c060cb1778d4a9056b190086a8c6c115828eaccea6cf", [:mix], [{:earmark, "~> 1.1", [hex: :earmark, repo: "hexpm", optional: false]}, {:makeup_elixir, "~> 0.7", [hex: :makeup_elixir, repo: "hexpm", optional: false]}], "hexpm"},
-  "gen_smtp": {:hex, :gen_smtp, "0.14.0", "39846a03522456077c6429b4badfd1d55e5e7d0fdfb65e935b7c5e38549d9202", [:rebar3], [], "hexpm"},
+  "gen_smtp": {:hex, :gen_smtp, "0.15.0", "9f51960c17769b26833b50df0b96123605a8024738b62db747fece14eb2fbfcc", [:rebar3], [], "hexpm"},
   "httpotion": {:hex, :httpotion, "3.1.0", "14d20d9b0ce4e86e253eb91e4af79e469ad949f57a5d23c0a51b2f86559f6589", [:mix], [{:ibrowse, "~> 4.4", [hex: :ibrowse, optional: false]}]},
   "ibrowse": {:hex, :ibrowse, "4.4.0", "2d923325efe0d2cb09b9c6a047b2835a5eda69d8a47ed6ff8bc03628b764e991", [:rebar3], []},
   "jason": {:hex, :jason, "1.1.1", "d3ccb840dfb06f2f90a6d335b536dd074db748b3e7f5b11ab61d239506585eb2", [:mix], [{:decimal, "~> 1.0", [hex: :decimal, optional: true]}]},

mix.lock

@@ -2,7 +2,7 @@ defmodule AttachmentsTest do
   use ExUnit.Case, async: true
 
   test "#inline returns { :ok, attachment } when file exists" do
-    { :ok, attachment } = Mailman.Attachment.inline("test/data/blank.png")
+    {:ok, attachment} = Mailman.Attachment.inline("test/data/blank.png")
     assert is_map(attachment)
   end
 
@@ -11,7 +11,6 @@ defmodule AttachmentsTest do
     {:error, _} = Mailman.Attachment.inline(file_path)
   end
 
-
   test "Attachment with a different disposition filename" do
     {:ok, attachment} = Mailman.Attachment.inline("test/data/blank.png", "another_name.png")
     assert attachment.file_name == "another_name.png"
@@ -26,14 +25,23 @@ defmodule AttachmentsTest do
   end
 
   test "#mime_types returns the list of 648 types" do
-    assert Enum.count(Mailman.Attachment.mime_types) == 648
+    assert Enum.count(Mailman.Attachment.mime_types()) == 648
   end
 
   test "mime type getter returns proper type" do
-    assert Mailman.Attachment.mime_type_and_subtype_from_extension("image.gif") == {"image", "gif"}
-    assert Mailman.Attachment.mime_type_and_subtype_from_extension("image.png") == {"image", "png"}
-    assert Mailman.Attachment.mime_type_and_subtype_from_extension("invoice.pdf") == {"application", "pdf"}
-    assert Mailman.Attachment.mime_type_and_subtype_from_extension("file.strange") == {"application", "octet-stream"}
-    assert Mailman.Attachment.mime_type_and_subtype_from_extension("settings.mobileconfig") == {"application", "x-apple-aspen-config"}
+    assert Mailman.Attachment.mime_type_and_subtype_from_extension("image.gif") ==
+             {"image", "gif"}
+
+    assert Mailman.Attachment.mime_type_and_subtype_from_extension("image.png") ==
+             {"image", "png"}
+
+    assert Mailman.Attachment.mime_type_and_subtype_from_extension("invoice.pdf") ==
+             {"application", "pdf"}
+
+    assert Mailman.Attachment.mime_type_and_subtype_from_extension("file.strange") ==
+             {"application", "octet-stream"}
+
+    assert Mailman.Attachment.mime_type_and_subtype_from_extension("settings.mobileconfig") ==
+             {"application", "x-apple-aspen-config"}
   end
 end

test/attachments_test.exs

@@ -47,5 +47,4 @@ defmodule Mailman.ConfigTest do
     Application.delete_env(:mailman, :adapter)
     Application.delete_env(:mailman, :some_config)
   end
-
 end

test/config_test.exs

@@ -7,14 +7,14 @@ defmodule DataEncodingTest do
     Litwo! Ojczyzno moja! Ty jesteś jak zdrowie. 
 
     Ile cię stracił. Dziś piękność twą w domu i czuł, 
-że teraz Napoleon, człek mądry a 
-potem najwyższych krajowych zamieszków. Dobra, całe wesoło, lecz w zastępstwie gospodarza, gdy je posłyszał, znikał nagle taż chętka, nie będziesz przy jego poznać nie miał wielką, i rozprawiali, nieco wylotów kontusza nalał 
-węgrzyna i swoją ważność zarazem poznaje. jak światłość miesiąca. Nucąc chwyciła suknie, biegła bardzo szybko, suwała się ramieniu. Przeprosiwszy go powitać. Dawno domu lasami i Obuchowicz Piotrowski, Obolewski, Rożycki, Janowicz, Mirzejewscy, Brochocki i kłopotach, i zdrowie. 
+    że teraz Napoleon, człek mądry a
+    potem najwyższych krajowych zamieszków. Dobra, całe wesoło, lecz w zastępstwie gospodarza, gdy je posłyszał, znikał nagle taż chętka, nie będziesz przy jego poznać nie miał wielką, i rozprawiali, nieco wylotów kontusza nalał
+    węgrzyna i swoją ważność zarazem poznaje. jak światłość miesiąca. Nucąc chwyciła suknie, biegła bardzo szybko, suwała się ramieniu. Przeprosiwszy go powitać. Dawno domu lasami i Obuchowicz Piotrowski, Obolewski, Rożycki, Janowicz, Mirzejewscy, Brochocki i kłopotach, i zdrowie.
 
     Nazywał się nagle, stronnicy Sokół na Francuza. oj, ten Bonapart czarował, no, tak mędrsi fircykom oprzeć się Soplica. wszyscy za pierwszym na świecie jeśli nasza młodzie wyjeżdża za duszę jego upadkiem domy i w klasztorze. 
-Ciszę przerywał ale powiedzieć nie był zacietrzewiony jak refektarz, z nowych gości. W końcu, stawiła przed młodzieżą o ten Bonapart figurka! Bez Suworowa to mówiąc, że serce mu przed ganek zajechał któryś z woźnym 
+    Ciszę przerywał ale powiedzieć nie był zacietrzewiony jak refektarz, z nowych gości. W końcu, stawiła przed młodzieżą o ten Bonapart figurka! Bez Suworowa to mówiąc, że serce mu przed ganek zajechał któryś z woźnym
 
-Protazym ze srebrnymi klamrami trzewiki peruka z których by to mówiąc, że serce niewinne ale nigdzie nie chciał, według nowej.
+    Protazym ze srebrnymi klamrami trzewiki peruka z których by to mówiąc, że serce niewinne ale nigdzie nie chciał, według nowej.
 
     """
   end
@@ -24,31 +24,31 @@ Protazym ze srebrnymi klamrami trzewiki peruka z których by to mówiąc, że se
     Litwo! Ojczyzno moja! Ty jeste=C5=9B jak zdrowie. =
 
     Ile ci=C4=99 straci=C5=82. Dzi=C5=9B pi=C4=99kno=C5=9B=C4=87 tw=C4=85 w=
- domu i czu=C5=82, =
-=C5=BCe teraz Napoleon, cz=C5=82ek m=C4=85dry a =
-potem najwy=C5=BCszych krajowych zamieszk=C3=B3w. Dobra, ca=C5=82e weso=C5=
-=82o, lecz w zast=C4=99pstwie gospodarza, gdy je pos=C5=82ysza=C5=82, znika=
-=C5=82 nagle ta=C5=BC ch=C4=99tka, nie b=C4=99dziesz przy jego pozna=C4=87 =
-nie mia=C5=82 wielk=C4=85, i rozprawiali, nieco wylot=C3=B3w kontusza nala=
-=C5=82 =
-w=C4=99grzyna i swoj=C4=85 wa=C5=BCno=C5=9B=C4=87 zarazem poznaje. jak =C5=
-=9Bwiat=C5=82o=C5=9B=C4=87 miesi=C4=85ca. Nuc=C4=85c chwyci=C5=82a suknie, =
-bieg=C5=82a bardzo szybko, suwa=C5=82a si=C4=99 ramieniu. Przeprosiwszy go =
-powita=C4=87. Dawno domu lasami i Obuchowicz Piotrowski, Obolewski, Ro=C5=BC
-ycki, Janowicz, Mirzejewscy, Brochocki i k=C5=82opotach, i zdrowie. =
+    domu i czu=C5=82, =
+    =C5=BCe teraz Napoleon, cz=C5=82ek m=C4=85dry a =
+    potem najwy=C5=BCszych krajowych zamieszk=C3=B3w. Dobra, ca=C5=82e weso=C5=
+    =82o, lecz w zast=C4=99pstwie gospodarza, gdy je pos=C5=82ysza=C5=82, znika=
+    =C5=82 nagle ta=C5=BC ch=C4=99tka, nie b=C4=99dziesz przy jego pozna=C4=87 =
+    nie mia=C5=82 wielk=C4=85, i rozprawiali, nieco wylot=C3=B3w kontusza nala=
+    =C5=82 =
+    w=C4=99grzyna i swoj=C4=85 wa=C5=BCno=C5=9B=C4=87 zarazem poznaje. jak =C5=
+    =9Bwiat=C5=82o=C5=9B=C4=87 miesi=C4=85ca. Nuc=C4=85c chwyci=C5=82a suknie, =
+    bieg=C5=82a bardzo szybko, suwa=C5=82a si=C4=99 ramieniu. Przeprosiwszy go =
+    powita=C4=87. Dawno domu lasami i Obuchowicz Piotrowski, Obolewski, Ro=C5=BC
+    ycki, Janowicz, Mirzejewscy, Brochocki i k=C5=82opotach, i zdrowie. =
 
     Nazywa=C5=82 si=C4=99 nagle, stronnicy Sok=C3=B3=C5=82 na Francuza. oj,=
- ten Bonapart czarowa=C5=82, no, tak m=C4=99drsi fircykom oprze=C4=87 si=C4=
-=99 Soplica. wszyscy za pierwszym na =C5=9Bwiecie je=C5=9Bli nasza m=C5=82o=
-dzie wyje=C5=BCd=C5=BCa za dusz=C4=99 jego upadkiem domy i w klasztorze. =
-Cisz=C4=99 przerywa=C5=82 ale powiedzie=C4=87 nie by=C5=82 zacietrzewiony j=
-ak refektarz, z nowych go=C5=9Bci. W ko=C5=84cu, stawi=C5=82a przed m=C5=82=
-odzie=C5=BC=C4=85 o ten Bonapart figurka! Bez Suworowa to m=C3=B3wi=C4=85c,=
- =C5=BCe serce mu przed ganek zajecha=C5=82 kt=C3=B3ry=C5=9B z wo=C5=BAnym =
+    ten Bonapart czarowa=C5=82, no, tak m=C4=99drsi fircykom oprze=C4=87 si=C4=
+    =99 Soplica. wszyscy za pierwszym na =C5=9Bwiecie je=C5=9Bli nasza m=C5=82o=
+    dzie wyje=C5=BCd=C5=BCa za dusz=C4=99 jego upadkiem domy i w klasztorze. =
+    Cisz=C4=99 przerywa=C5=82 ale powiedzie=C4=87 nie by=C5=82 zacietrzewiony j=
+    ak refektarz, z nowych go=C5=9Bci. W ko=C5=84cu, stawi=C5=82a przed m=C5=82=
+    odzie=C5=BC=C4=85 o ten Bonapart figurka! Bez Suworowa to m=C3=B3wi=C4=85c,=
+    =C5=BCe serce mu przed ganek zajecha=C5=82 kt=C3=B3ry=C5=9B z wo=C5=BAnym =
 
-Protazym ze srebrnymi klamrami trzewiki peruka z kt=C3=B3rych by to m=C3=B3=
-wi=C4=85c, =C5=BCe serce niewinne ale nigdzie nie chcia=C5=82, wed=C5=82ug =
-nowej.
+    Protazym ze srebrnymi klamrami trzewiki peruka z kt=C3=B3rych by to m=C3=B3=
+    wi=C4=85c, =C5=BCe serce niewinne ale nigdzie nie chcia=C5=82, wed=C5=82ug =
+    nowej.
 
     """
   end
@@ -63,12 +63,11 @@ nowej.
 
   test "#quotes_from/1 returns text with lines not longer than 76 characters" do
     # Enum.each String.split(DataEncoding.quoted_from(testing_string), "\n"), fn(l) ->
-      # assert String.length(l) <= 76
-     #end
+    # assert String.length(l) <= 76
+    # end
   end
 
   test "#quoted_from/1 replaces spaces and tabs at the end of the line properly" do
-     #assert DataEncoding.quoted_from("Lorem ipsum \ndolor sit amet\t\nYYyyyy") == "Lorem ipsum=20\r\ndolor sit amet=09\r\nYYyyyy"
+    # assert DataEncoding.quoted_from("Lorem ipsum \ndolor sit amet\t\nYYyyyy") == "Lorem ipsum=20\r\ndolor sit amet=09\r\nYYyyyy"
   end
-
 end