Improve syntax error on invalid calls

Closes #1847

Author: José Valim

lib/elixir/src/elixir_translator.erl

@@ -425,8 +425,12 @@ translate_each({ { '.', _, [Expr] }, Meta, Args }, S) when is_list(Args) ->
 
 %% Invalid calls
 
-translate_each({ Invalid, Meta, Args }, S) when is_list(Meta) and is_list(Args) ->
-  syntax_error(Meta, S#elixir_scope.file, "unexpected parenthesis after ~ts",
+translate_each({ { '.', _, [Invalid, _] }, Meta, Args }, S) when is_list(Meta) and is_list(Args) ->
+  syntax_error(Meta, S#elixir_scope.file, "invalid remote call on ~ts",
+    ['Elixir.Macro':to_string(Invalid)]);
+
+translate_each({ _, Meta, Args } = Invalid, S) when is_list(Meta) and is_list(Args) ->
+  syntax_error(Meta, S#elixir_scope.file, "invalid call ~ts",
     ['Elixir.Macro':to_string(Invalid)]);
 
 translate_each({ _, _, _ } = Tuple, S) ->

lib/elixir/test/elixir/kernel/errors_test.exs

@@ -202,10 +202,14 @@ defmodule Kernel.ErrorsTest do
       'quote 1'
   end
 
-  test :invalid_parens do
+  test :invalid_calls do
     assert_compile_fail SyntaxError,
-      "nofile:1: unexpected parenthesis after foo(1)",
+      "nofile:1: invalid call foo(1)(2)",
       'foo(1)(2)'
+
+    assert_compile_fail SyntaxError,
+      "nofile:1: invalid remote call on 1",
+      '1.foo'
   end
 
   test :unhandled_stab do