refactored the create method to actually produce meaningful error messages when something goes wrong. added tests for corner cases

1 files changed, 16 insertions, 0 deletions

diff --git a/test/functional/nodes_controller_test.rb b/test/functional/nodes_controller_test.rb
index 08e50fd..801f2c0 100644
--- a/test/functional/nodes_controller_test.rb
+++ b/test/functional/nodes_controller_test.rb
@@ -70,6 +70,22 @@ class NodesControllerTest < ActionController::TestCase
     assert_equal 1, Node.last.level
   end
   
+  test "creating a top_level node without a title should not work" do
+    login_as :quentin
+    
+    assert_no_difference "Node.count" do
+      post(:create, :kind => "top_level")
+    end
+  end
+  
+  test "creating a generic node without a parent_id should not work" do
+    login_as :quentin
+    
+    assert_no_difference "Node.count" do
+      post(:create, :kind => "generic")
+    end
+  end
+  
   def test_editing_a_node
     login_as :quentin