Merge pull request #18769 from gsamokovarov/exception-wrapper-windows-paths

Show proper traces on Windows for the error pages

1 files changed, 17 insertions, 0 deletions

diff --git a/actionpack/test/dispatch/exception_wrapper_test.rb b/actionpack/test/dispatch/exception_wrapper_test.rb
index d7408164ba..7a29a7ff97 100644
--- a/actionpack/test/dispatch/exception_wrapper_test.rb
+++ b/actionpack/test/dispatch/exception_wrapper_test.rb
@@ -34,6 +34,23 @@ module ActionDispatch
       assert_equal [ code: 'foo', line_number: 42 ], wrapper.source_extracts
     end
 
+    test '#source_extracts works with Windows paths' do
+      exc = TestError.new("c:/path/to/rails/app/controller.rb:27:in 'index':")
+
+      wrapper = ExceptionWrapper.new({}, exc)
+      wrapper.expects(:source_fragment).with('c:/path/to/rails/app/controller.rb', 27).returns('nothing')
+
+      assert_equal [ code: 'nothing', line_number: 27 ], wrapper.source_extracts
+    end
+
+    test '#source_extracts works with non standard backtrace' do
+      exc = TestError.new('invalid')
+
+      wrapper = ExceptionWrapper.new({}, exc)
+      wrapper.expects(:source_fragment).with('invalid', 0).returns('nothing')
+
+      assert_equal [ code: 'nothing', line_number: 0 ], wrapper.source_extracts
+    end
 
     test '#application_trace returns traces only from the application' do
       exception = TestError.new(caller.prepend("lib/file.rb:42:in `index'"))