aboutsummaryrefslogtreecommitdiffstats
path: root/railties
diff options
context:
space:
mode:
authoryuuji.yaginuma <yuuji.yaginuma@gmail.com>2015-12-07 15:16:18 +0900
committeryuuji.yaginuma <yuuji.yaginuma@gmail.com>2015-12-10 16:08:37 +0900
commite4e42d0b346065f1fce6caaea669c3e4bc42dfc8 (patch)
tree66e99342a40795b2e5fd9a94b56ab44f549f2789 /railties
parent11fbcf84ac29d6040be1b0988d22054342ed74d4 (diff)
downloadrails-e4e42d0b346065f1fce6caaea669c3e4bc42dfc8.tar.gz
rails-e4e42d0b346065f1fce6caaea669c3e4bc42dfc8.tar.bz2
rails-e4e42d0b346065f1fce6caaea669c3e4bc42dfc8.zip
show relative path the rerun snippet of test runner in rails engine
Since the absolute path is not required to re-run the test, modified so that unnecessary information is not displayed. ```ruby # before bin/rails test /path/to/blorgh/test/integration/navigation_test.rb:5 # after bin/rails test test/integration/navigation_test.rb:5 ```
Diffstat (limited to 'railties')
-rw-r--r--railties/lib/rails/test_unit/reporter.rb6
-rw-r--r--railties/test/generators/plugin_test_helper.rb24
-rw-r--r--railties/test/generators/plugin_test_runner_test.rb25
-rw-r--r--railties/test/generators/test_runner_in_engine_test.rb31
4 files changed, 63 insertions, 23 deletions
diff --git a/railties/lib/rails/test_unit/reporter.rb b/railties/lib/rails/test_unit/reporter.rb
index de4b002e55..00ea32d1b8 100644
--- a/railties/lib/rails/test_unit/reporter.rb
+++ b/railties/lib/rails/test_unit/reporter.rb
@@ -44,7 +44,7 @@ module Rails
end
def relative_path_for(file)
- file.sub(/^#{Rails.root}\/?/, '')
+ file.sub(/^#{app_root}\/?/, '')
end
private
@@ -66,5 +66,9 @@ module Rails
"#{self.executable} #{relative_path_for(assertion_path)}"
end
+
+ def app_root
+ @app_root ||= defined?(ENGINE_ROOT) ? ENGINE_ROOT : Rails.root
+ end
end
end
diff --git a/railties/test/generators/plugin_test_helper.rb b/railties/test/generators/plugin_test_helper.rb
new file mode 100644
index 0000000000..96c1b1d31f
--- /dev/null
+++ b/railties/test/generators/plugin_test_helper.rb
@@ -0,0 +1,24 @@
+require 'abstract_unit'
+require 'tmpdir'
+
+module PluginTestHelper
+ def create_test_file(name, pass: true)
+ plugin_file "test/#{name}_test.rb", <<-RUBY
+ require 'test_helper'
+
+ class #{name.camelize}Test < ActiveSupport::TestCase
+ def test_truth
+ puts "#{name.camelize}Test"
+ assert #{pass}, 'wups!'
+ end
+ end
+ RUBY
+ end
+
+ def plugin_file(path, contents, mode: 'w')
+ FileUtils.mkdir_p File.dirname("#{plugin_path}/#{path}")
+ File.open("#{plugin_path}/#{path}", mode) do |f|
+ f.puts contents
+ end
+ end
+end
diff --git a/railties/test/generators/plugin_test_runner_test.rb b/railties/test/generators/plugin_test_runner_test.rb
index 0444e13865..716728819e 100644
--- a/railties/test/generators/plugin_test_runner_test.rb
+++ b/railties/test/generators/plugin_test_runner_test.rb
@@ -1,7 +1,8 @@
-require 'tmpdir'
-require 'abstract_unit'
+require 'generators/plugin_test_helper'
class PluginTestRunnerTest < ActiveSupport::TestCase
+ include PluginTestHelper
+
def setup
@destination_root = Dir.mktmpdir('bukkits')
Dir.chdir(@destination_root) { `bundle exec rails plugin new bukkits --skip-bundle` }
@@ -100,24 +101,4 @@ class PluginTestRunnerTest < ActiveSupport::TestCase
def run_test_command(arguments)
Dir.chdir(plugin_path) { `bin/test #{arguments}` }
end
-
- def create_test_file(name, pass: true)
- plugin_file "test/#{name}_test.rb", <<-RUBY
- require 'test_helper'
-
- class #{name.camelize}Test < ActiveSupport::TestCase
- def test_truth
- puts "#{name.camelize}Test"
- assert #{pass}, 'wups!'
- end
- end
- RUBY
- end
-
- def plugin_file(path, contents, mode: 'w')
- FileUtils.mkdir_p File.dirname("#{plugin_path}/#{path}")
- File.open("#{plugin_path}/#{path}", mode) do |f|
- f.puts contents
- end
- end
end
diff --git a/railties/test/generators/test_runner_in_engine_test.rb b/railties/test/generators/test_runner_in_engine_test.rb
new file mode 100644
index 0000000000..641c5d0835
--- /dev/null
+++ b/railties/test/generators/test_runner_in_engine_test.rb
@@ -0,0 +1,31 @@
+require 'generators/plugin_test_helper'
+
+class TestRunnerInEngineTest < ActiveSupport::TestCase
+ include PluginTestHelper
+
+ def setup
+ @destination_root = Dir.mktmpdir('bukkits')
+ Dir.chdir(@destination_root) { `bundle exec rails plugin new bukkits --full --skip-bundle` }
+ plugin_file 'test/dummy/db/schema.rb', ''
+ end
+
+ def teardown
+ FileUtils.rm_rf(@destination_root)
+ end
+
+ def test_rerun_snippet_is_relative_path
+ create_test_file 'post', pass: false
+
+ output = run_test_command('test/post_test.rb')
+ assert_match %r{Running:\n\nPostTest\nF\n\nwups!\n\nbin/rails test test/post_test.rb:6}, output
+ end
+
+ private
+ def plugin_path
+ "#{@destination_root}/bukkits"
+ end
+
+ def run_test_command(arguments)
+ Dir.chdir(plugin_path) { `bin/rails test #{arguments}` }
+ end
+end