Default to undefined repo_organization when key does not exist (#66)

Author: starbelly

src/hex_repo.erl

@@ -159,7 +159,9 @@ tarball_url(Config, Name, Version) ->
 build_url(#{repo_url := URI, repo_organization := Org}, Path) when is_binary(Org) ->
     <<URI/binary, "/repos/", Org/binary, "/", Path/binary>>;
 build_url(#{repo_url := URI, repo_organization := undefined}, Path) ->
-    <<URI/binary, "/", Path/binary>>.
+    <<URI/binary, "/", Path/binary>>;
+build_url(Config, Path) ->
+    build_url(Config#{repo_organization => undefined}, Path).
 
 tarball_filename(Name, Version) ->
     <<Name/binary, "-", Version/binary, ".tar">>.

test/hex_repo_SUITE.erl

@@ -17,7 +17,7 @@ suite() ->
     [{require, {ssl_certs, test_pub}}].
 
 all() ->
-    [get_names_test, get_versions_test, get_package_test, get_tarball_test].
+    [get_names_test, get_versions_test, get_package_test, get_tarball_test, repo_org_not_set].
 
 get_names_test(_Config) ->
     {ok, {200, _, Packages}} = hex_repo:get_names(?CONFIG),
@@ -47,3 +47,12 @@ get_tarball_test(_Config) ->
 
     {ok, {403, _, _}} = hex_repo:get_tarball(?CONFIG, <<"ecto">>, <<"9.9.9">>),
     ok.
+
+repo_org_not_set(_Config) ->
+    Config = maps:remove(repo_organization, ?CONFIG),
+    {ok, {200, _, Releases}} = hex_repo:get_package(Config, <<"ecto">>),
+    [#{version := <<"1.0.0">>}] =
+        lists:filter(fun(#{version := Version}) -> Version == <<"1.0.0">> end, Releases),
+
+    {ok, {403, _, _}} = hex_repo:get_package(Config, <<"nonexisting">>),
+    ok.