From 6287c109d3362985d352160fc0fe674ed899d6d2 Mon Sep 17 00:00:00 2001 From: Akira Matsuda Date: Tue, 3 Jan 2023 11:57:14 +0900 Subject: [PATCH] Don't report broken absolute path when app_root is nil When running tests that does not initialize rails (e.g. `rake test` under active_model), it used to just cut off the leading "/" from absolte paths and report something like rails test Users/a_matsuda/rails/activemodel/test/cases/api_test.rb:40 that cannot be executed. --- railties/lib/rails/test_unit/reporter.rb | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/railties/lib/rails/test_unit/reporter.rb b/railties/lib/rails/test_unit/reporter.rb index f36685da4b..9f28cdca30 100644 --- a/railties/lib/rails/test_unit/reporter.rb +++ b/railties/lib/rails/test_unit/reporter.rb @@ -52,7 +52,11 @@ def filtered_results end def relative_path_for(file) - file.sub(/^#{app_root}\/?/, "") + if app_root + file.sub(/^#{app_root}\/?/, "") + else + file + end end private