From f9576c8598c4d44a3bd8adf36d62f5eb9b8c29a3 Mon Sep 17 00:00:00 2001
From: Nathaniel Waisbrot <code@waisbrot.net>
Date: Thu, 4 Aug 2016 14:36:46 -0400
Subject: Handle `escriptize` when the specified app is missing

When rebar.config contains a `escript_main_app` option, but the
specified app doesn't exist in the build directory, print an error.
---
 src/rebar_prv_escriptize.erl    |  8 ++++++--
 test/rebar_escriptize_SUITE.erl | 26 +++++++++++++++++++++++++-
 2 files changed, 31 insertions(+), 3 deletions(-)

diff --git a/src/rebar_prv_escriptize.erl b/src/rebar_prv_escriptize.erl
index d8704f6..6e10947 100644
--- a/src/rebar_prv_escriptize.erl
+++ b/src/rebar_prv_escriptize.erl
@@ -72,8 +72,12 @@ do(State) ->
             end;
         Name ->
             AllApps = rebar_state:all_deps(State)++rebar_state:project_apps(State),
-            {ok, AppInfo} = rebar_app_utils:find(ec_cnv:to_binary(Name), AllApps),
-            escriptize(State, AppInfo)
+            case rebar_app_utils:find(ec_cnv:to_binary(Name), AllApps) of
+                {ok, AppInfo} ->
+                    escriptize(State, AppInfo);
+                _ ->
+                    ?PRV_ERROR({bad_name, Name})
+            end
     end.
 
 escriptize(State0, App) ->
diff --git a/test/rebar_escriptize_SUITE.erl b/test/rebar_escriptize_SUITE.erl
index 1817d6b..139d5cd 100644
--- a/test/rebar_escriptize_SUITE.erl
+++ b/test/rebar_escriptize_SUITE.erl
@@ -5,6 +5,8 @@
          end_per_suite/1,
          init_per_testcase/2,
          all/0,
+         escriptize_with_name/1,
+         escriptize_with_bad_name/1,
          build_and_clean_app/1]).
 
 -include_lib("common_test/include/ct.hrl").
@@ -24,7 +26,11 @@ init_per_testcase(_, Config) ->
     rebar_test_utils:init_rebar_state(Config).
 
 all() ->
-    [build_and_clean_app].
+    [
+     build_and_clean_app,
+     escriptize_with_name,
+     escriptize_with_bad_name
+    ].
 
 %% Test escriptize builds and runs the app's escript
 build_and_clean_app(Config) ->
@@ -35,3 +41,21 @@ build_and_clean_app(Config) ->
     rebar_test_utils:create_app(AppDir, Name, Vsn, [kernel, stdlib]),
     rebar_test_utils:run_and_check(Config, [], ["escriptize"],
                                    {ok, [{app, Name, valid}]}).
+
+escriptize_with_name(Config) ->
+    AppDir = ?config(apps, Config),
+
+    Name = rebar_test_utils:create_random_name("app1_"),
+    Vsn = rebar_test_utils:create_random_vsn(),
+    rebar_test_utils:create_app(AppDir, Name, Vsn, [kernel, stdlib]),
+    rebar_test_utils:run_and_check(Config, [{escript_main_app, Name}], ["escriptize"],
+                                   {ok, [{app, Name, valid}]}).
+
+escriptize_with_bad_name(Config) ->
+    AppDir = ?config(apps, Config),
+
+    Name = rebar_test_utils:create_random_name("app1_"),
+    Vsn = rebar_test_utils:create_random_vsn(),
+    rebar_test_utils:create_app(AppDir, Name, Vsn, [kernel, stdlib]),
+    rebar_test_utils:run_and_check(Config, [{escript_main_app, boogers}], ["escriptize"],
+                                   {error,{rebar_prv_escriptize, {bad_name, boogers}}}).
-- 
cgit v1.1