Skip to content

Commit e0c312b

Browse files
author
José Valim
committed
Add a test case ensuring managers are not set eagerly
1 parent 523b7f1 commit e0c312b

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

lib/mix/test/mix/deps_test.exs

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,21 @@ defmodule Mix.DepsTest do
6767
Mix.Project.pop
6868
end
6969

70+
test "does not set the manager before the dependency was fetched" do
71+
# It is important to not eagerly set the manager because the dependency
72+
# needs to be fetched (i.e. available in the filesystem) in order to get
73+
# the proper manager.
74+
Mix.Project.push DepsApp
75+
76+
{ _, true } =
77+
Mix.Deps.unfetched(false, fn dep, acc ->
78+
assert nil?(dep.manager)
79+
{ dep, acc or true }
80+
end)
81+
after
82+
Mix.Project.pop
83+
end
84+
7085
defmodule ConvergedDepsApp do
7186
def project do
7287
[

0 commit comments

Comments
 (0)