Now throwing warnings for unknown modules

bryanjos · bryanjos · commit 5eeb1d8d2317 · 2017-08-10T12:26:24.000-05:00
diff --git a/lib/elixir_script/passes/find_used_modules.ex b/lib/elixir_script/passes/find_used_modules.ex
@@ -1,6 +1,7 @@
 defmodule ElixirScript.FindUsedModules do
   @moduledoc false
   alias ElixirScript.State, as: ModuleState
+  require Logger
 
   @doc """
   Takes a list of entry modules and finds modules they use.
@@ -20,6 +21,10 @@ defmodule ElixirScript.FindUsedModules do
         walk_module(module, info, pid)
       {:ok, module, implementations} ->
         walk_protocol(module, implementations, pid)
+      {:error, "Unknown module"} ->
+        Logger.warn fn() ->
+          "#{inspect module} is missing or unavailable"
+        end
       {:error, error} ->
         raise ElixirScript.CompileError, "An error occurred while compiling #{inspect module}: #{error}"
     end
diff --git a/test/compiler_test.exs b/test/compiler_test.exs
@@ -11,12 +11,6 @@ defmodule ElixirScript.Compiler.Test do
     assert is_binary(result)
   end
 
-  test "Error on unknown module" do
-    assert_raise ElixirScript.CompileError, fn ->
-      ElixirScript.Compiler.compile(SomeModule)
-    end
-  end
-
   test "Output" do
     result = ElixirScript.Compiler.compile(Atom, [])
     assert result =~ "export default Elixir"
