Skip to content

Add more possible ENV variables for tests#127

Open
clayallsopp wants to merge 1 commit into
HipByte:masterfrom
clayallsopp:more_test_vars
Open

Add more possible ENV variables for tests#127
clayallsopp wants to merge 1 commit into
HipByte:masterfrom
clayallsopp:more_test_vars

Conversation

@clayallsopp

Copy link
Copy Markdown
Contributor

Adds some parity to Rails + RSpec options

adds some parity to Rails + RSpec options
@colinta

colinta commented Sep 3, 2013

Copy link
Copy Markdown
Member

What's the benefit of having these different names?

@clayallsopp

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

@miry

miry commented Sep 12, 2013

Copy link
Copy Markdown

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

👍

@alloy

alloy commented Jun 25, 2014

Copy link
Copy Markdown
Member

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
Copy Markdown
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

alloy commented Jun 27, 2014

Copy link
Copy Markdown
Member

Perfect, thanks!

@otzy007

otzy007 commented Apr 6, 2015

Copy link
Copy Markdown

👍 Do we add this?

@kuldeepaggarwal

Copy link
Copy Markdown

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