diff options
author | Fred Hebert <mononcqc@ferd.ca> | 2017-02-24 08:08:46 -0500 |
---|---|---|
committer | Fred Hebert <mononcqc@ferd.ca> | 2017-02-24 08:08:46 -0500 |
commit | ab1f93d6fa712c3b85bce30cab09edb5ddf7f3d9 (patch) | |
tree | 4ca77dcd8bb2529ce4ea9a12651b3c0db7c5fa8b /test | |
parent | 1810ae30abdddc95dc93d96f5c9d0f27182f0e62 (diff) |
Fix default .app.src file for rebar_app_info
The finding of the file was done based on an assumed 'src' path which
may not be correct. This patch instead replaces the value with a lookup
in configured paths and returns the first that matches to an existing
file.
Diffstat (limited to 'test')
-rw-r--r-- | test/rebar_src_dirs_SUITE.erl | 29 |
1 files changed, 23 insertions, 6 deletions
diff --git a/test/rebar_src_dirs_SUITE.erl b/test/rebar_src_dirs_SUITE.erl index 677dbaf..bc22160 100644 --- a/test/rebar_src_dirs_SUITE.erl +++ b/test/rebar_src_dirs_SUITE.erl @@ -16,9 +16,11 @@ build_basic_app/1, build_multi_apps/1, src_dir_takes_precedence_over_extra/1, - src_dir_checkout_dep/1]). + src_dir_checkout_dep/1, + app_src_info/1]). -include_lib("common_test/include/ct.hrl"). +-include_lib("eunit/include/eunit.hrl"). suite() -> []. @@ -41,7 +43,7 @@ all() -> dupe_src_dirs_at_root_and_in_erl_opts, extra_src_dirs_at_root_and_in_erl_opts, build_basic_app, build_multi_apps, src_dir_takes_precedence_over_extra, - src_dir_checkout_dep]. + src_dir_checkout_dep, app_src_info]. src_dirs_at_root(Config) -> AppDir = ?config(apps, Config), @@ -301,9 +303,24 @@ src_dir_checkout_dep(Config) -> Config, RebarConfig, ["compile"], {ok, [{checkout, DepName}, {app, AppName}]} ), + ok. - % {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return), - % ["bar", "baz", "foo", "qux"] = rebar_dir:src_dirs(rebar_state:opts(State), []), - % rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], - % {ok, [{app, Name}]}), +app_src_info(Config) -> + PrivDir = ?config(priv_dir, Config), + AppName1 = rebar_test_utils:create_random_name("app_src_info"), + AppDir1 = filename:join(PrivDir, AppName1), + {ok, Info1} = rebar_app_info:new(AppName1, "1.0.0", AppDir1), + AppSrc1 = filename:join([AppDir1, "src", AppName1 ++ ".app.src"]), + ok = filelib:ensure_dir(AppSrc1), + ok = file:write_file(AppSrc1, "[]."), + ?assertEqual(AppSrc1, rebar_app_info:app_file_src(Info1)), + + AppName2 = rebar_test_utils:create_random_name("app_src_info"), + AppDir2 = filename:join(PrivDir, AppName2), + {ok, Info2Tmp} = rebar_app_info:new(AppName2, "1.0.0", AppDir2), + Info2 = rebar_app_info:set(Info2Tmp, src_dirs, ["foo", "bar", "baz"]), + AppSrc2 = filename:join([AppDir2, "bar", AppName2 ++ ".app.src"]), + ok = filelib:ensure_dir(AppSrc2), + ok = file:write_file(AppSrc2, "[]."), + ?assertEqual(AppSrc2, rebar_app_info:app_file_src(Info2)), ok. |