Closes #684. Fix bulk actions in users tab.

Includes test for added bulk_modify route.
MarkUsProject · Mar 12, 2012 · 873b368 · 873b368
1 parent 4b91366
commit 873b368
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/app/views/students/index.html.erb b/app/views/students/index.html.erb
@@ -47,7 +47,7 @@ document.observe("dom:loaded", function(){
 <div class="colsLeftHeavy">
 
   <div class="wrapLeft">
-   <%= form_tag :url => {:action => "bulk_modify"},
+   <%= form_tag bulk_modify_students_path(),
  	        :before => "before_form_submit();",
 	        :complete => "after_form_submit();",
 	        :remote => true do %>

diff --git a/config/routes.rb b/config/routes.rb
@@ -258,6 +258,7 @@
 
   resources :students do
     collection do
+      post 'bulk_modify'
       post 'populate'
       get 'manage'
       get 'download_student_list'

diff --git a/test/functional/students_controller_test.rb b/test/functional/students_controller_test.rb
@@ -94,6 +94,11 @@ def setup
         @section = Section.make
       end
 
+      should "recognize action to bulk modify for a student" do
+        assert_recognizes( {:action => "bulk_modify", :controller => "students"},
+               {:path => "students/bulk_modify", :method => "post"} )
+      end
+
       should "be able to edit a student" do
         get_as @admin,
                :edit,