From: John Dong <jdong@mit.edu>
Date: Tue, 21 Aug 2007 19:12:02 +0000 (-0400)
Subject: Revise functional test of graph controller for retrieving an unknown id number. Curre... 
X-Git-Url: https://projects.mako.cc/source/selectricity/commitdiff_plain/4c5046ffa16ea77da4e4b69517dcb33bc6ec2646?ds=sidebyside;hp=059813befe57bab83dbc84627149f1af08d1c850

Revise functional test of graph controller for retrieving an unknown id number. Currently I think raising RecordNotFound is fine. The alternate would be to generate a record not found PNG, but I don't see the point of doing that either.
---

diff --git a/test/functional/graph_controller_test.rb b/test/functional/graph_controller_test.rb
index e147707..e5c2790 100644
--- a/test/functional/graph_controller_test.rb
+++ b/test/functional/graph_controller_test.rb
@@ -19,15 +19,15 @@ class GraphControllerTest < Test::Unit::TestCase
 
   # TODO Replace this with your actual tests
   def test_show_nonexist_votes_per_interval
-    assert_nothing_raised { get :votes_per_interval, { :id => 42 } }
+    assert_raise(ActiveRecord::RecordNotFound) { get :votes_per_interval, { :id => 42 } }
   end
 
   def test_show_nonexist_borda_bar
-    assert_nothing_raised { get :borda_bar, { :id => 42 } }
+    assert_raise(ActiveRecord::RecordNotFound) { get :borda_bar, { :id => 42 } }
   end
 
   def test_show_nonexist_choices_positions
-    assert_nothing_raised { get :choices_positions, { :id => 42 } }
+    assert_raise(ActiveRecord::RecordNotFound) { get :choices_positions, { :id => 42 } }
   end
   
   def test_show_empty_votes_per_interval