Tidy up some of the map bugs code

[rails.git] / test / functional / map_bugs_controller_test.rb

diff --git a/test/functional/map_bugs_controller_test.rb b/test/functional/map_bugs_controller_test.rb
index c3335d2b10ec8f257523c4e8fb9edd73a0f09812..9aa155c3eb907502710dcd72db3c65e3ae00889c 100644 (file)
--- a/test/functional/map_bugs_controller_test.rb
+++ b/test/functional/map_bugs_controller_test.rb
@@ -5,34 +5,33 @@ class MapBugsControllerTest < ActionController::TestCase
     
   def test_map_bug_create_success
     assert_difference('MapBug.count') do
     
   def test_map_bug_create_success
     assert_difference('MapBug.count') do

-         assert_difference('MapBugComment.count') do
+      assert_difference('MapBugComment.count') do

         post :add_bug, {:lat => -1.0, :lon => -1.0, :name => "new_tester", :text => "This is a comment"}
         post :add_bug, {:lat => -1.0, :lon => -1.0, :name => "new_tester", :text => "This is a comment"}

-         end
-       end
+      end
+    end

     assert_response :success
     assert_response :success

-       id = @response.body.sub(/ok/,"").to_i
-       get :read, {:id => id, :format => 'json'}
+    id = @response.body.sub(/ok/,"").to_i
+    get :read, {:id => id, :format => 'json'}

     assert_response :success
     assert_response :success

-       js = @response.body
-       assert_match "\"status\":\"open\"", js
-       assert_match "\"comment\":\"This is a comment\"", js
-       assert_match "\"commenter_name\":\"new_tester (a)\"", js
+    js = @response.body
+    assert_match "\"status\":\"open\"", js
+    assert_match "\"comment\":\"This is a comment\"", js
+    assert_match "\"commenter_name\":\"new_tester (a)\"", js

   end
 
   def test_map_bug_comment_create_success
     assert_difference('MapBugComment.count') do
   end
 
   def test_map_bug_comment_create_success
     assert_difference('MapBugComment.count') do

-        post :edit_bug, {:id => 2, :name => "new_tester2", :text => "This is an additional comment"}
+      post :edit_bug, {:id => 2, :name => "new_tester2", :text => "This is an additional comment"}

     end
     assert_response :success
 
     end
     assert_response :success
 

-       get :read, {:id => 2, :format => 'json'}
+    get :read, {:id => 2, :format => 'json'}

     assert_response :success
     assert_response :success

-       js = @response.body
-       assert_match "\"id\":2", js
-       assert_match "\"status\":\"open\"", js
-       assert_match "\"comment\":\"This is an additional comment\"", js
-       assert_match "\"commenter_name\":\"new_tester2 (a)\"", js
-
+    js = @response.body
+    assert_match "\"id\":2", js
+    assert_match "\"status\":\"open\"", js
+    assert_match "\"comment\":\"This is an additional comment\"", js
+    assert_match "\"commenter_name\":\"new_tester2 (a)\"", js

   end
 
   def test_map_bug_read_success
   end
 
   def test_map_bug_read_success


@@ -53,122 +52,118 @@ class MapBugsControllerTest < ActionController::TestCase
   end
 
   def test_map_bug_close_success
   end
 
   def test_map_bug_close_success

-       post :close_bug, {:id => 2}
+    post :close_bug, {:id => 2}

     assert_response :success
 
     assert_response :success
 

-       get :read, {:id => 2, :format => 'json'}
-       js = @response.body
-       assert_match "\"id\":2", js
-       assert_match "\"status\":\"closed\"", js
+    get :read, {:id => 2, :format => 'json'}
+    js = @response.body
+    assert_match "\"id\":2", js
+    assert_match "\"status\":\"closed\"", js

   end
 
   def test_get_bugs_success
   end
 
   def test_get_bugs_success

-       get :get_bugs, {:bbox=>'1,1,1.2,1.2'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'1,1,1.2,1.2'}
+    assert_response :success

 
 

-       get :get_bugs, {:bbox=>'1,1,1.2,1.2', :format => 'rss'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'1,1,1.2,1.2', :format => 'rss'}
+    assert_response :success

 
 

-       get :get_bugs, {:bbox=>'1,1,1.2,1.2', :format => 'json'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'1,1,1.2,1.2', :format => 'json'}
+    assert_response :success

 
 

-       get :get_bugs, {:bbox=>'1,1,1.2,1.2', :format => 'xml'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'1,1,1.2,1.2', :format => 'xml'}
+    assert_response :success

 
 

-       get :get_bugs, {:bbox=>'1,1,1.2,1.2', :format => 'gpx'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'1,1,1.2,1.2', :format => 'gpx'}
+    assert_response :success

   end
 
   def test_get_bugs_large_area_success
   end
 
   def test_get_bugs_large_area_success

-       get :get_bugs, {:bbox=>'-10,-10,12,12'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'-10,-10,12,12'}
+    assert_response :success

   end
 
   def test_get_bugs_closed_7_success
   end
 
   def test_get_bugs_closed_7_success

-       get :get_bugs, {:bbox=>'1,1,1.2,1.2', :closed => '7'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'1,1,1.2,1.2', :closed => '7'}
+    assert_response :success

   end
 
   def test_get_bugs_closed_0_success
   end
 
   def test_get_bugs_closed_0_success

-       get :get_bugs, {:bbox=>'1,1,1.2,1.2', :closed => '0'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'1,1,1.2,1.2', :closed => '0'}
+    assert_response :success

   end
 
   def test_get_bugs_closed_n1_success
   end
 
   def test_get_bugs_closed_n1_success

-       get :get_bugs, {:bbox=>'1,1,1.2,1.2', :closed => '-1'}
-       assert_response :success
+    get :get_bugs, {:bbox=>'1,1,1.2,1.2', :closed => '-1'}
+    assert_response :success

   end
 
 
   def test_search_success
   end
 
 
   def test_search_success

-       get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1'}
-       assert_response :success
+    get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1'}
+    assert_response :success

 
 

-       get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1', :format => 'xml'}
-       assert_response :success
+    get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1', :format => 'xml'}
+    assert_response :success

 
 

-       get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1', :format => 'json'}
-       assert_response :success
+    get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1', :format => 'json'}
+    assert_response :success

 
 

-       get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1', :format => 'rss'}
-       assert_response :success
+    get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1', :format => 'rss'}
+    assert_response :success

 
 

-       get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1', :format => 'gpx'}
-       assert_response :success
+    get :search, {:bbox=>'1,1,1.2,1.2', :q => 'bug 1', :format => 'gpx'}
+    assert_response :success

   end
 
   def test_rss_success
   end
 
   def test_rss_success

-       get :rss, {:bbox=>'1,1,1.2,1.2'}
-       assert_response :success
+    get :rss, {:bbox=>'1,1,1.2,1.2'}
+    assert_response :success

        
        

-       get :rss
-       assert_response :success
+    get :rss
+    assert_response :success

   end
 
   def test_user_bugs_success
   end
 
   def test_user_bugs_success

-       get :my_bugs, {:display_name=>'test'}
-       assert_response :success
+    get :my_bugs, {:display_name=>'test'}
+    assert_response :success

 
 

-       get :my_bugs, {:display_name=>'pulibc_test2'}
-       assert_response :success
+    get :my_bugs, {:display_name=>'pulibc_test2'}
+    assert_response :success

 
 

-       get :my_bugs, {:display_name=>'non-existent'}
-       assert_response :not_found
-       
+    get :my_bugs, {:display_name=>'non-existent'}
+    assert_response :not_found 

   end
 
   def test_map_bug_comment_create_not_found
     assert_no_difference('MapBugComment.count') do
   end
 
   def test_map_bug_comment_create_not_found
     assert_no_difference('MapBugComment.count') do

-        post :edit_bug, {:id => 12345, :name => "new_tester", :text => "This is an additional comment"}
+      post :edit_bug, {:id => 12345, :name => "new_tester", :text => "This is an additional comment"}

     end
     assert_response :not_found
   end
 
   def test_map_bug_close_not_found
     end
     assert_response :not_found
   end
 
   def test_map_bug_close_not_found

-       post :close_bug, {:id => 12345}
+    post :close_bug, {:id => 12345}

     assert_response :not_found
   end
 
   def test_map_bug_read_not_found
     assert_response :not_found
   end
 
   def test_map_bug_read_not_found

-       get :read, {:id => 12345}
+    get :read, {:id => 12345}

     assert_response :not_found
   end
 
   def test_map_bug_read_gone
     assert_response :not_found
   end
 
   def test_map_bug_read_gone

-       get :read, {:id => 4}
+    get :read, {:id => 4}

     assert_response :gone
   end
 
   def test_map_bug_hidden_comment
     assert_response :gone
   end
 
   def test_map_bug_hidden_comment

-       get :read, {:id => 5, :format => 'json'}
-       assert_response :success
-       js = @response.body
-       assert_match "\"id\":5", js
-       assert_match "\"comment\":\"Valid comment for bug 5\"", js
-       assert_match "\"comment\":\"Another valid comment for bug 5\"", js
-       assert_no_match /\"comment\":\"Spam for bug 5\"/, js
+    get :read, {:id => 5, :format => 'json'}
+    assert_response :success
+    js = @response.body
+    assert_match "\"id\":5", js
+    assert_match "\"comment\":\"Valid comment for bug 5\"", js
+    assert_match "\"comment\":\"Another valid comment for bug 5\"", js
+    assert_no_match /\"comment\":\"Spam for bug 5\"/, js

   end
   end

-  
-
-  

 end
 end