-
Notifications
You must be signed in to change notification settings - Fork 26
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
Support new Rails 5 syntax #34
Comments
would be awesome to infer it from the project, not sure how we would be able to do that tho |
The main concern I have with both of these is that they're slow—too slow to run before each test run. But they might be fine for setting some sort of project level config, something like |
@leppert interesting. we need to keep in mind this should also work outside rails, for ruby only projects. |
Just bumped into this as well using minitest-emacs in spacemacs. Following along. :-) |
Setting |
Indeed. This config is newer than this issue, and solves the original problem. I still think it could be nice to auto-detect a rails project and version to see if |
Beginning with Rails 5.0.0, testing is expected to be run via the
rails test
command rather thanrake test
, withrake test
being fully deprecated at some point in the future. The effect of this today is that paths and test names passed viarake
appear to be ignored, instead running the entire suite.Not sure if the best solution here is to add a new
(defcustom use-rails-5-syntax)
or try to infer the appropriate syntax based on the project.More on the new syntax can be found here. One upside is that the new runner accepts line numbers for running individual tests, which should obviate the need to regex for the test name.
The text was updated successfully, but these errors were encountered: