Added another test to ensure new_notes accessor works.

git-svn-id: http://substruct.googlecode.com/svn/trunk/plugins/substruct@163 6a1f77cc-a145-0410-afd8-bd7963fcc59f

app/models/order.rb

@@ -148,6 +148,7 @@ def set_new_notes()
 			new_note << "<span class=\"info\">"
 			new_note << "[#{time}]"
 			new_note << "</span></p>"
+			self.notes ||= ''
 			write_attribute(:notes, self.notes + new_note)
 			self.new_notes = nil
 		end

test/unit/order_test.rb

@@ -648,13 +648,17 @@ def test_get_all_line_items_total
     assert_equal an_order.line_items_total, an_order.order_line_items.collect{ |p| p.unit_price * p.quantity }.sum
   end
 
+
   def test_new_notes_update_attr
-    notes_before = @santa_order.notes.dup
+    # Notes need to be NIL in order to test an edge case error.
+    @santa_order.update_attribute(:notes, nil)
+    # ^^ DONT REMOVE THIS
     @santa_order.update_attributes({
       :new_notes => 'Hello world.'
     })
-    notes_after = @santa_order.reload.notes
-    assert_not_equal notes_before, notes_after
+    @santa_order.reload
+    assert_not_nil @santa_order.notes
+    assert @santa_order.notes.include?("<span class=\"info\">")
   end  
 
   # Test an order to see if the correct total weight will be returned.