Fix for failing test cases
authormmd-osm <mmd.osm@gmail.com>
Sat, 16 Dec 2017 12:51:02 +0000 (13:51 +0100)
committermmd-osm <mmd.osm@gmail.com>
Sat, 16 Dec 2017 12:51:02 +0000 (13:51 +0100)
test/controllers/changeset_controller_test.rb

index a0f7960c31b7197702ac2927734853e7996c13b3..c0af4021eeca251e5f11453e6bce23b9a93dd7c5 100644 (file)
@@ -2248,6 +2248,13 @@ CHANGESET
       post :subscribe, :params => { :id => changeset.id }
     end
     assert_response :success
+
+    # not closed changeset
+    changeset = create(:changeset)
+    assert_difference "changeset.subscribers.count", 1 do
+      post :subscribe, :params => { :id => changeset.id }
+    end
+    assert_response :success
   end
 
   ##
@@ -2270,12 +2277,6 @@ CHANGESET
     end
     assert_response :not_found
 
-    # not closed changeset
-    changeset = create(:changeset)
-    assert_no_difference "changeset.subscribers.count" do
-      post :subscribe, :params => { :id => changeset.id }
-    end
-    assert_response :conflict
 
     # trying to subscribe when already subscribed
     changeset = create(:changeset, :closed)
@@ -2298,6 +2299,15 @@ CHANGESET
       post :unsubscribe, :params => { :id => changeset.id }
     end
     assert_response :success
+
+    # not closed changeset
+    changeset = create(:changeset)
+    changeset.subscribers.push(user)
+
+    assert_difference "changeset.subscribers.count", -1 do
+      post :unsubscribe, :params => { :id => changeset.id }
+    end
+    assert_response :success
   end
 
   ##
@@ -2318,13 +2328,6 @@ CHANGESET
     end
     assert_response :not_found
 
-    # not closed changeset
-    changeset = create(:changeset)
-    assert_no_difference "changeset.subscribers.count" do
-      post :unsubscribe, :params => { :id => changeset.id }
-    end
-    assert_response :conflict
-
     # trying to unsubscribe when not subscribed
     changeset = create(:changeset, :closed)
     assert_no_difference "changeset.subscribers.count" do