Changeset 4451

Timestamp:

06/16/06 03:22:09 (2 years ago)

Author:

rick

Message:

add passing test to make sure unhandled requests don't load unnecessary classes. Closed #5408. [nkriege@hotmail.com]

Files:

• trunk/actionpack/test/controller/routing_test.rb (modified) (2 diffs)

trunk/actionpack/test/controller/routing_test.rb

@@ -843 +843 @@
-
-  def test_simple_build_query_string
-@route.build_query_string(:x => '1', :y => '2'
+order_query_string(@route.build_query_string(:x => '1', :y => '2')
-  end
-
-  def test_convert_ints_build_query_string
-@route.build_query_string(:x => 1, :y => 2
+order_query_string(@route.build_query_string(:x => 1, :y => 2)
-  end
-
-  def test_escape_spaces_build_query_string
-@route.build_query_string(:x => 'hello world', :y => 'goodbye world'
+order_query_string(@route.build_query_string(:x => 'hello world', :y => 'goodbye world')
-  end
-
-  def test_expand_array_build_query_string
-@route.build_query_string(:x => [1, 2]
+order_query_string(@route.build_query_string(:x => [1, 2])
-  end
-
-  def test_escape_spaces_build_query_string_selected_keys
-
-
+
+
+
+
+
+
+
-end
-
@@ -1302 +1307 @@
-  end
-
+  def test_routing_traversal_does_not_load_extra_classes
+    assert !Object.const_defined?("Profiler__"), "Profiler should not be loaded"
+    set.draw do |map|
+      map.connect '/profile', :controller => 'profile'
+    end
+
+    request.path = '/profile'
+
+    set.recognize(request) rescue nil
+    
+    assert !Object.const_defined?("Profiler__"), "Profiler should not be loaded"
+  end
-
-  def test_recognize_with_conditions_and_format