I'm not sure I understand this? Previously the exception message was
duplicated in the app and in the test, and that caused a problem when I ran
the tests, so I just make it common.


On Fri, Dec 20, 2013 at 5:28 PM, Eric Meadows-Jönsson <
notifications@github.com> wrote:

> In lib/mix/lib/mix/tasks/deps.check.ex:
>
> > @@ -58,4 +68,8 @@ defmodule Mix.Tasks.Deps.Check do
> >        end)
> >      end
> >    end
> > +
> > +  # DRY up tests
> > +  def cant_continue_message,
> > +  do: "Can't continue: there were errors in the dependencies"
>
> Isn't it better to just remove the message check on assert_raise rather
> than doing this? Otherwise we are just testing that this function is being
> called.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/elixir-lang/elixir/pull/1951/files#r8514426>
> .
>