Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make "-a" the default #12

Open
PVince81 opened this issue Apr 21, 2015 · 2 comments
Open

Make "-a" the default #12

PVince81 opened this issue Apr 21, 2015 · 2 comments

Comments

@PVince81
Copy link
Contributor

Many people who run the tests manually forget to add "-a" to make sure to run all test cases.

I suggest to reverse the behavior and always run all the test sets by default, unless specified otherwise.

CC @jnfrmarks

@moscicki
Copy link
Contributor

Thanks for the suggestion. I think this is really a matter of what is your most frequent use. Could I suggest that for convenience you create an alias in your shell of the smash command with all the switches you most frequently use? You may also want to add --keep-going to this list. For non-interactive smash invocation (e.g. some script or cron) I think it is still a better idea to invoke smash directly with all options explicitly set.

@PVince81
Copy link
Contributor Author

It is not for me personally but for any Dev who might want to quickly run tests while developing in core (not test development)
So far one would always expect all suites to run by default like many other frameworks do (like PHPUnit with data providers). Such devs could think the tests ran fine for them and move on and not realize that they only ran the first test case.
So it is mostly about usability and slightly reducing the barrier of entry.
Is it more common for you guys to only run the first suite than all of them ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants