aboutsummaryrefslogtreecommitdiffstats
path: root/railties/lib
diff options
context:
space:
mode:
authorDalibor Nasevic <dalibor.nasevic@gmail.com>2013-02-08 00:53:11 +0100
committerPrem Sichanugrist <s@sikac.hu>2013-03-09 16:03:55 -0500
commitdf85dfa6fa6f7ae9a0b72eb9b9a254d2d5560f38 (patch)
tree6a0782530c35a6010c257394cb17e3bea1903707 /railties/lib
parent1a0c58b2988a24a783b4f9a658ac629922125551 (diff)
downloadrails-df85dfa6fa6f7ae9a0b72eb9b9a254d2d5560f38.tar.gz
rails-df85dfa6fa6f7ae9a0b72eb9b9a254d2d5560f38.tar.bz2
rails-df85dfa6fa6f7ae9a0b72eb9b9a254d2d5560f38.zip
Improve wording for rails test command
Diffstat (limited to 'railties/lib')
-rw-r--r--railties/lib/rails/commands/test_runner.rb34
-rw-r--r--railties/lib/rails/generators/rails/app/templates/test/test_helper.rb4
2 files changed, 19 insertions, 19 deletions
diff --git a/railties/lib/rails/commands/test_runner.rb b/railties/lib/rails/commands/test_runner.rb
index ae901f05f8..92c4d4e593 100644
--- a/railties/lib/rails/commands/test_runner.rb
+++ b/railties/lib/rails/commands/test_runner.rb
@@ -2,13 +2,13 @@ require 'optparse'
require 'minitest/unit'
module Rails
- # Handling the all the logic behind +rails test+ command.
+ # Handles all logic behind +rails test+ command.
class TestRunner
class << self
- # Parse the test suite name from the arguments array and pass in a list
- # of file to a new +TestRunner+ object, then invoke the evaluation. If
- # the argument is not a test suite name, it will be treated as a file
- # name and passed to the +TestRunner+ instance right away.
+ # Creates a new +TestRunner+ object with an array of test files to run
+ # based on the arguments. When no arguments are provided, it runs all test
+ # files. When a suite argument is provided, it runs only the test files in
+ # that suite. Otherwise, it runs the specified test file(s).
def start(files, options = {})
original_fixtures_options = options.delete(:fixtures)
options[:fixtures] = true
@@ -36,18 +36,18 @@ module Rails
end
end
- # Parse arguments and set them as option flags
+ # Parses arguments and sets them as option flags
def parse_arguments(arguments)
options = {}
orig_arguments = arguments.dup
OptionParser.new do |opts|
- opts.banner = "Usage: rails test [path to test file(s) or test suite type]"
+ opts.banner = "Usage: rails test [path to test file(s) or test suite]"
opts.separator ""
- opts.separator "Run single test file, or a test suite, under Rails'"
+ opts.separator "Run a specific test file(s) or a test suite, under Rails'"
opts.separator "environment. If the file name(s) or suit name is omitted,"
- opts.separator "Rails will run all the test suites."
+ opts.separator "Rails will run all tests."
opts.separator ""
opts.separator "Specific options:"
@@ -91,7 +91,7 @@ module Rails
end
end
- # Create a new +TestRunner+ object with a list of test file paths.
+ # Creates a new +TestRunner+ object with a list of test file paths.
def initialize(files, options)
@files = files
Rake::Task['test:prepare'].invoke
@@ -108,25 +108,25 @@ module Rails
MiniTest::Unit.output = SilentUntilSyncStream.new(MiniTest::Unit.output)
end
- # Run the test files by evaluate each of them.
+ # Runs test files by evaluating each of them.
def run
@files.each { |filename| load(filename) }
end
# A null stream object which ignores everything until +sync+ has been set
- # to true. This is only to be used to silence unnecessary output from
- # MiniTest, as MiniTest calls +output.sync = true+ right before output the
- # first test result.
+ # to true. This is only used to silence unnecessary output from MiniTest,
+ # as MiniTest calls +output.sync = true+ right before it outputs the first
+ # test result.
class SilentUntilSyncStream < File
- # Create a +SilentUntilSyncStream+ object by given a stream object that
- # this stream should set +MiniTest::Unit.output+ to after +sync+ has been
+ # Creates a +SilentUntilSyncStream+ object by giving it a target stream
+ # object that will be assigned to +MiniTest::Unit.output+ after +sync+ is
# set to true.
def initialize(target_stream)
@target_stream = target_stream
super(File::NULL, 'w')
end
- # Swap +MiniTest::Unit.output+ to another stream when +sync+ is true.
+ # Swaps +MiniTest::Unit.output+ to another stream when +sync+ is true.
def sync=(sync)
if sync
@target_stream.sync = true
diff --git a/railties/lib/rails/generators/rails/app/templates/test/test_helper.rb b/railties/lib/rails/generators/rails/app/templates/test/test_helper.rb
index 754e99e09f..f0aeeee827 100644
--- a/railties/lib/rails/generators/rails/app/templates/test/test_helper.rb
+++ b/railties/lib/rails/generators/rails/app/templates/test/test_helper.rb
@@ -6,8 +6,8 @@ class ActiveSupport::TestCase
<% unless options[:skip_active_record] -%>
ActiveRecord::Migration.check_pending!
- # Uncomment the `fixtures` line below to setup all fixtures in test/fixtures/*.yml for all tests
- # in alphabetical order.
+ # Uncomment the `fixtures :all` line below to setup all fixtures in test/fixtures/*.yml
+ # for all tests in alphabetical order.
#
# Note: You'll currently still have to declare fixtures explicitly in integration tests
# -- they do not yet inherit this setting