From 457cc99349972c4cfc52959aac317562c99e47d1 Mon Sep 17 00:00:00 2001
From: Anton Khorev <tony29@yandex.ru>
Date: Sun, 23 Feb 2025 04:54:39 +0300
Subject: [PATCH] Test response body of changeset comment hide/unhide

---
 .../api/changeset_comments_controller_test.rb | 96 +++++++++++++++++++
 1 file changed, 96 insertions(+)

diff --git a/test/controllers/api/changeset_comments_controller_test.rb b/test/controllers/api/changeset_comments_controller_test.rb
index 2a386b3e5..fcb53600e 100644
--- a/test/controllers/api/changeset_comments_controller_test.rb
+++ b/test/controllers/api/changeset_comments_controller_test.rb
@@ -397,6 +397,30 @@ module Api
       post changeset_comment_hide_path(comment), :headers => auth_header
 
       assert_response :success
+      assert_equal "application/xml", response.media_type
+      assert_dom "osm", 1 do
+        assert_dom "> changeset", 1 do
+          assert_dom "> @id", comment.changeset_id.to_s
+          assert_dom "> @comments_count", "0"
+        end
+      end
+
+      assert_not comment.reload.visible
+    end
+
+    def test_hide_with_write_changeset_comments_scope_json
+      comment = create(:changeset_comment)
+      auth_header = bearer_authorization_header create(:moderator_user), :scopes => %w[write_changeset_comments]
+
+      post changeset_comment_hide_path(comment, :format => "json"), :headers => auth_header
+
+      assert_response :success
+      assert_equal "application/json", response.media_type
+      js = ActiveSupport::JSON.decode(@response.body)
+      assert_not_nil js["changeset"]
+      assert_equal comment.changeset_id, js["changeset"]["id"]
+      assert_equal 0, js["changeset"]["comments_count"]
+
       assert_not comment.reload.visible
     end
 
@@ -407,6 +431,30 @@ module Api
       post changeset_comment_hide_path(comment), :headers => auth_header
 
       assert_response :success
+      assert_equal "application/xml", response.media_type
+      assert_dom "osm", 1 do
+        assert_dom "> changeset", 1 do
+          assert_dom "> @id", comment.changeset_id.to_s
+          assert_dom "> @comments_count", "0"
+        end
+      end
+
+      assert_not comment.reload.visible
+    end
+
+    def test_hide_with_write_api_scope_json
+      comment = create(:changeset_comment)
+      auth_header = bearer_authorization_header create(:moderator_user), :scopes => %w[write_api]
+
+      post changeset_comment_hide_path(comment, :format => "json"), :headers => auth_header
+
+      assert_response :success
+      assert_equal "application/json", response.media_type
+      js = ActiveSupport::JSON.decode(@response.body)
+      assert_not_nil js["changeset"]
+      assert_equal comment.changeset_id, js["changeset"]["id"]
+      assert_equal 0, js["changeset"]["comments_count"]
+
       assert_not comment.reload.visible
     end
 
@@ -454,6 +502,30 @@ module Api
       post changeset_comment_unhide_path(comment), :headers => auth_header
 
       assert_response :success
+      assert_equal "application/xml", response.media_type
+      assert_dom "osm", 1 do
+        assert_dom "> changeset", 1 do
+          assert_dom "> @id", comment.changeset_id.to_s
+          assert_dom "> @comments_count", "1"
+        end
+      end
+
+      assert comment.reload.visible
+    end
+
+    def test_unhide_with_write_changeset_comments_scope_json
+      comment = create(:changeset_comment, :visible => false)
+      auth_header = bearer_authorization_header create(:moderator_user), :scopes => %w[write_changeset_comments]
+
+      post changeset_comment_unhide_path(comment, :format => "json"), :headers => auth_header
+
+      assert_response :success
+      assert_equal "application/json", response.media_type
+      js = ActiveSupport::JSON.decode(@response.body)
+      assert_not_nil js["changeset"]
+      assert_equal comment.changeset_id, js["changeset"]["id"]
+      assert_equal 1, js["changeset"]["comments_count"]
+
       assert comment.reload.visible
     end
 
@@ -464,6 +536,30 @@ module Api
       post changeset_comment_unhide_path(comment), :headers => auth_header
 
       assert_response :success
+      assert_equal "application/xml", response.media_type
+      assert_dom "osm", 1 do
+        assert_dom "> changeset", 1 do
+          assert_dom "> @id", comment.changeset_id.to_s
+          assert_dom "> @comments_count", "1"
+        end
+      end
+
+      assert comment.reload.visible
+    end
+
+    def test_unhide_with_write_api_scope_json
+      comment = create(:changeset_comment, :visible => false)
+      auth_header = bearer_authorization_header create(:moderator_user), :scopes => %w[write_api]
+
+      post changeset_comment_unhide_path(comment, :format => "json"), :headers => auth_header
+
+      assert_response :success
+      js = ActiveSupport::JSON.decode(@response.body)
+      assert_equal "application/json", response.media_type
+      assert_not_nil js["changeset"]
+      assert_equal comment.changeset_id, js["changeset"]["id"]
+      assert_equal 1, js["changeset"]["comments_count"]
+
       assert comment.reload.visible
     end
 
-- 
2.39.5