`test` task is never run when using `mix do` and space

[vic]

Bug

The test task is never run if followed by space

Environment

• Elixir & Erlang versions (elixir -v): Elixir 1.4.0-rc.1 (16a14c1)
• Operating system: OSX Sierra

Current behavior

When using mix do to execute two tasks test and clean and an space character is placed before the comma, tests are never run and the shell just hangs.

mix new foo
cd foo
env MIX_ENV=test mix do test , clean
Compiling 1 file (.ex)
Generated foo app
# *hangs forever tests are never run*

Expected behavior

Expected test task to run.
Expected mix do test , clean to be equivalent to mix do test, clean

Any other task works correctly even when followed by space, for example, this works:

env MIX_ENV=test mix do clean , test

[vic]

oh, btw I could provide a pull-request if this is confirmed to be a bug by the elixir team.

[josevalim]

Thank you. I believe this is a bug. Here is the quickest way to reproduce it: mix test "". There is something causing it to go infinite looping.

[josevalim]

@vic btw, we generally appreciate pull requests but this seems to be a rather annoying bug so I will go ahead and fix it asap. It is not we are infinite looping but rather we are loading all filenames in the operating system. :)

[josevalim]

@vic I went ahead and fixed one of the bugs, the one that makes mix test "" fail. However, mix do test , clean still fails but now saying:

Test patterns did not match any file:

If you would like to send a pull request for this one, it will be welcome. Keep in mind we tried to generally solve this issue in the past, trying this commit ea00314 and later reverting it bd58e06 (see PR linked in the issue).

Maybe a simple workaround is to ignore the left side of splitting on "," if the left is empty.

[vic]

we are loading all filenames in the operating system. :)
😮

@josevalim thanks for fixing that!

Sure, I'll try to fix the remaining issue and will send a PR.