Add test to verify posts are moved in transaction

mattvanhorn · May 26, 2013 · 1ba1831 · 1ba1831
1 parent b8fbac5
commit 1ba1831
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/spec/models/topic_spec.rb b/spec/models/topic_spec.rb
@@ -204,10 +204,15 @@ def build_topic_with_title(title)
         lambda { topic.move_posts(user, [1003], title: "new testing topic name") }.should raise_error(Discourse::InvalidParameters)
       end
 
-      it "raises an error if no posts were moved" do
-        lambda { topic.move_posts(user, [], title: "new testing topic name") }.should raise_error(Discourse::InvalidParameters)
-      end
+      it "raises an error and does not create a topic if no posts were moved" do
+        Topic.count.tap do |original_topic_count|
+          lambda {
+            topic.move_posts(user, [], title: "new testing topic name")
+          }.should raise_error(Discourse::InvalidParameters)
 
+          expect(Topic.count).to eq original_topic_count
+        end
+      end
     end
 
     context "successfully moved" do