Merge pull request #38 from antedeguemon/vm-fixes-multiple-input-files

Fixes handling of multiple input files

lib/mix/tasks/recode.ex

@@ -85,8 +85,7 @@ defmodule Mix.Tasks.Recode do
   defp opts!(opts) do
     case OptionParser.parse!(opts, @opts) do
       {opts, []} -> opts
-      {opts, [inputs]} -> Keyword.put(opts, :inputs, inputs)
-      {_opts, args} -> Mix.raise("#{inspect(args)} : Unknown")
+      {opts, inputs} -> Keyword.put(opts, :inputs, inputs)
     end
   end
 

test/mix/tasks/recode_test.exs

@@ -33,23 +33,35 @@ defmodule Mix.Tasks.RecodeTest do
   test "mix recode --config priv/config.exs -" do
     expect(RunnerMock, :run, fn config ->
       assert Keyword.keyword?(config)
-      assert config[:inputs] == "-"
+      assert config[:inputs] == ["-"]
 
       ":test" |> source() |> project()
     end)
 
     assert catch_exit(Tasks.Recode.run(["--config", "priv/config.exs", "-"])) == :normal
   end
 
+  test "mix recode file_1.ex file_2.ex" do
+    expect(RunnerMock, :run, fn config ->
+      assert Keyword.keyword?(config)
+      assert config[:inputs] == ["file_1.ex", "file_2.ex"]
+
+      ":test" |> source() |> project()
+    end)
+
+    assert catch_exit(Tasks.Recode.run(["--config", "priv/config.exs", "file_1.ex", "file_2.ex"])) ==
+             :normal
+  end
+
   test "mix recode raises exception for unknown config file" do
     assert_raise Mix.Error, "Config file not found", fn ->
       Tasks.Recode.run(["--config", "priv/no_config.exs"])
     end
   end
 
   test "mix recode raises exception for unknown arg" do
-    assert_raise Mix.Error, ~s|["inputs", "foo"] : Unknown|, fn ->
-      Tasks.Recode.run(["--config", "priv/config.exs", "inputs", "foo"])
+    assert_raise OptionParser.ParseError, ~r|unknown-arg : Unknown option|, fn ->
+      Tasks.Recode.run(["--config", "priv/config.exs", "--unknown-arg", "inputs", "foo"])
     end
   end