Remove generated code from app traces; improve accuracy of framework …

…traces

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@2676 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

activesupport/CHANGELOG

@@ -1,3 +1,8 @@
+* Classify generated routing code as framework code to avoid appearing in application traces. [Nicholas Seckar]
+
+* Show all framework frames in the framework trace. [Nicholas Seckar]
+
+
 *1.2.0* (October 16th, 2005)
 
 * Update Exception extension to show the first few framework frames in an application trace. [Nicholas Seckar] 

activesupport/lib/active_support/core_ext/exception.rb

@@ -3,6 +3,7 @@ class Exception
   alias :clean_message :message
 
   TraceSubstitutions = []
+  FrameworkRegexp = /generated_code|vendor|dispatch|ruby|script\/\w+/
 
   def clean_backtrace
     backtrace.collect do |line|
@@ -16,9 +17,13 @@ def application_backtrace
     before_application_frame = true
 
     clean_backtrace.reject do |line|
-      non_app_frame = !! (line =~ /vendor|dispatch|ruby|script\/\w+/)
+      non_app_frame = !! (line =~ FrameworkRegexp)
       before_application_frame = false unless non_app_frame
       non_app_frame && ! before_application_frame
     end
   end
+
+  def framework_backtrace
+    clean_backtrace.select {|line| line =~ FrameworkRegexp}
+  end
 end

activesupport/test/core_ext/exception_test.rb

@@ -34,5 +34,13 @@ def test_app_backtrace_with_before
     assert_kind_of Exception, e
     assert_equal ['vendor/file.rb some stuff', 'bhal.rb', 'almost all'], e.application_backtrace
   end
+
+  def test_framework_backtrace_with_before
+    Exception::TraceSubstitutions << [/\s*hidden.*/, '']
+    e = get_exception RuntimeError, 'RAWR', ['vendor/file.rb some stuff', 'bhal.rb', ' vendor/file.rb some stuff', 'almost all']
+    assert_kind_of Exception, e
+    assert_equal ['vendor/file.rb some stuff', ' vendor/file.rb some stuff'], e.framework_backtrace
+  end
+
 
 end