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

Add more possible ENV variables for tests #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

clayallsopp
Copy link
Contributor

Adds some parity to Rails + RSpec options

adds some parity to Rails + RSpec options
@colinta
Copy link
Member

colinta commented Sep 3, 2013

What's the benefit of having these different names?

@clayallsopp
Copy link
Contributor Author

it's annoying when switching from Rails to remember that RubyMotion's spec task has a different ENV variable for specifying specific tests

@clayallsopp
Copy link
Contributor Author

and not just Rails - I'm unaware of any other Ruby testing lib which uses files as a filter variable

@miry
Copy link

miry commented Sep 12, 2013

and not just Rails - I'm unaware of any other Ruby testing lib which uses files as a filter variable

👍

@alloy
Copy link
Member

alloy commented Jun 25, 2014

I can see that being helpful. Could you please point me to the docs for both env vars? I'm unable to easily find the canonical ones.

@clayallsopp
Copy link
Contributor Author

Rails/Rake: http://ruby-doc.org/stdlib-2.0.0/libdoc/rake/rdoc/Rake/TestTask.html

If rake is invoked with a “TEST=filename” command line option, then the list of test files will be overridden to include only the filename specified on the command line.

Rspec: https://github.com/rspec/rspec-core/blob/918d2059b3b14ef670e5bec3de7ba83925fdf559/lib/rspec/core/rake_task.rb#L112

@alloy
Copy link
Member

alloy commented Jun 27, 2014

Perfect, thanks!

@otzy007
Copy link

otzy007 commented Apr 6, 2015

👍 Do we add this?

@kuldeepaggarwal
Copy link

And I think we can deprecate files option.

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

Successfully merging this pull request may close these issues.

6 participants