feat: add skip_all command

[guglielmobartelloni]

#88 Adds the ability to skip all the tests using the "x" command.

[zachallaun]

Two thoughts about this:

1. I think this should use :yellow instead of :red, since it's more akin to s than n.
2. While the issue tracking this feature suggested x (because I was original thinking of it as a sort of "exit"), I'm now wondering if it should use S instead.

Thoughts?

[guglielmobartelloni]

Yes, since it's a "more powerful" skip I think it's better if we use S instead of x and it's more in tune with the yellow color.

[guglielmobartelloni]

I will look like this

[zachallaun]

Would you be willing to take a crack at an integration test for this? You can look at test_integration/skipped_assertion_test.exs for a reference. The integration test file should be similar except that there should be a second test case that would normally update, but shouldn't because it should be skipped.

[zachallaun]

No problem, but how can I run the test itself? I don't know how the integration tests work in this case.

New integration tests are automatically picked up and run. There's no way to run just a single integration test, but you can run only the integration tests with mix test --only integration, and you can get extra information about a specific integration test using the DBG env var. So if you name the test something like skip_all_assertions_test.exs, you could run DBG=skip_all mix test --only integration (the DBG value only needs to be a substring of the test file name).

[guglielmobartelloni]

No problem, but how can I run the test itself? I don't know how the integration tests work in this case.

New integration tests are automatically picked up and run. There's no way to run just a single integration test, but you can run only the integration tests with mix test --only integration, and you can get extra information about a specific integration test using the DBG env var. So if you name the test something like skip_all_assertions_test.exs, you could run DBG=skip_all mix test --only integration (the DBG value only needs to be a substring of the test file name).

Ok, if I understood correctly besed on the skip_all behavior I only need to use a # S comment on top of the first test and the rest I assume that should be skipped. Like in the commit e1527af

[zachallaun]

Just wanted to let you know that this is still on my radar and I'm hoping to do a quick final review, merge, and release later this week. Thanks for being patient!

[guglielmobartelloni]

No worries at all, totally understood that you were busy. Thanks!

[guglielmobartelloni]

I integrated the commented changes with a rebase. The three commits should be updated.

[guglielmobartelloni]

Any updates?