--include tags about excoveralls

[nathany]

@campezzi told me to do this on Slack:

mix test --include remote --cover

That works, and would work around the issue in #47 if test_coverage: [tool: ExCoveralls] can be configured to generate an HTML report.

[campezzi]

I believe changing "local" to "html" in this line will do what you want. I'll try to take some time over the weekend to make that configurable via the coveralls.json file!

[campezzi]

Maybe running this will work:

MIX_ENV=test mix coveralls.html --include remote

I took a quick look at the code for the excoveralls.html mix task and it seems to forward its arguments to the test task, so this might be enough.

[nathany]

@campezzi Nope.

201 tests, 0 failures, 17 skipped

It would be much more intuitive if excoveralls supported the --include, --exclude, and --only flags.

[parroty]

I'm sorry being late to respond. Does it relate to #50?

[nathany]

It sounds like #50 may fix this, yah.

[leifg]

I just tested it and didn't get this to work (on 0.5.4):

MIX_ENV=test mix coveralls.html --only slow

executes all the tests I have and completely ignores my options.

[nathany]

Yah, it's still not working for me either. To run all the coverage I need to comment out this line from my test_helper because I can't specify --include remote on the command line.

ExUnit.configure(exclude: [:remote])

[nathany]

The patch in #50 doesn't quite work. Args goes from ["--include", "remote"] to [{"--include", nil}]. See invalid here: http://elixir-lang.org/docs/stable/elixir/OptionParser.html#parse/2

I'm working on a fix.

[nathany]

To be honest, it seems like #50 is the reason this doesn't work -- as compared to the previous code in the diff. Fix one thing, break the other, hah.

We could just pass all the args to --cover, but maybe we should be removing the ones that only excoveralls uses.