Exit with proper test code on test_erlang, closes #2313

Author: José Valim

Makefile

@@ -149,13 +149,19 @@ release_docs: docs
 
 test: test_erlang test_elixir
 
-test_erlang: compile
+TEST_ERL = lib/elixir/test/erlang
+TEST_EBIN = lib/elixir/test/ebin
+TEST_ERLS = $(addprefix $(TEST_EBIN)/, $(addsuffix .beam, $(basename $(notdir $(wildcard $(TEST_ERL)/*.erl)))))
+
+test_erlang: compile $(TEST_ERLS)
 	@ echo "==> elixir (eunit)"
-	$(Q) mkdir -p lib/elixir/test/ebin
-	$(Q) $(ERLC) -pa lib/elixir/ebin -o lib/elixir/test/ebin lib/elixir/test/erlang/*.erl
-	$(Q) $(ERL) -pa lib/elixir/test/ebin -s test_helper test -s erlang halt;
+	$(Q) $(ERL) -pa $(TEST_EBIN) -s test_helper test;
 	@ echo ""
 
+$(TEST_EBIN)/%.beam: $(TEST_ERL)/%.erl
+	$(Q) mkdir -p $(TEST_EBIN)
+	$(Q) $(ERLC) -o $(TEST_EBIN) $<
+
 test_elixir: test_stdlib test_ex_unit test_doc_test test_mix test_eex test_iex
 
 test_doc_test: compile

lib/elixir/test/erlang/test_helper.erl

@@ -1,19 +1,23 @@
 -module(test_helper).
 -include("elixir.hrl").
 -export([test/0, run_and_remove/2, throw_elixir/1, throw_erlang/1]).
+-define(TESTS, [
+  atom_test,
+  control_test,
+  function_test,
+  match_test,
+  module_test,
+  operators_test,
+  string_test,
+  tokenizer_test
+]).
 
 test() ->
   application:start(elixir),
-  eunit:test([
-    atom_test,
-    control_test,
-    function_test,
-    match_test,
-    module_test,
-    operators_test,
-    string_test,
-    tokenizer_test
-  ]).
+  case eunit:test(?TESTS) of
+    error -> erlang:halt(1);
+    _Res  -> erlang:halt(0)
+  end.
 
 % Execute a piece of code and purge given modules right after
 run_and_remove(Fun, Modules) ->