Add tests for featured tag removal (#34888)

spec/lib/activitypub/activity/remove_spec.rb

@@ -4,29 +4,60 @@ require 'rails_helper'
 
 RSpec.describe ActivityPub::Activity::Remove do
   let(:sender) { Fabricate(:account, featured_collection_url: 'https://example.com/featured') }
-  let(:status) { Fabricate(:status, account: sender) }
-
-  let(:json) do
-    {
-      '@context': 'https://www.w3.org/ns/activitystreams',
-      id: 'foo',
-      type: 'Add',
-      actor: ActivityPub::TagManager.instance.uri_for(sender),
-      object: ActivityPub::TagManager.instance.uri_for(status),
-      target: sender.featured_collection_url,
-    }.with_indifferent_access
-  end
 
   describe '#perform' do
     subject { described_class.new(json, sender) }
 
-    before do
+    context 'when removing a pinned status' do
-      StatusPin.create!(account: sender, status: status)
+      let(:status) { Fabricate(:status, account: sender) }
-      subject.perform
+
+      let(:json) do
+        {
+          '@context': 'https://www.w3.org/ns/activitystreams',
+          id: 'foo',
+          type: 'Remove',
+          actor: ActivityPub::TagManager.instance.uri_for(sender),
+          object: ActivityPub::TagManager.instance.uri_for(status),
+          target: sender.featured_collection_url,
+        }.deep_stringify_keys
+      end
+
+      before do
+        StatusPin.create!(account: sender, status: status)
+      end
+
+      it 'removes a pin' do
+        expect { subject.perform }
+          .to change { sender.pinned?(status) }.to(false)
+      end
     end
 
-    it 'removes a pin' do
+    context 'when removing a featured tag' do
-      expect(sender.pinned?(status)).to be false
+      let(:tag) { Fabricate(:tag) }
+
+      let(:json) do
+        {
+          '@context': 'https://www.w3.org/ns/activitystreams',
+          id: 'foo',
+          type: 'Remove',
+          actor: ActivityPub::TagManager.instance.uri_for(sender),
+          object: {
+            type: 'Hashtag',
+            name: "##{tag.display_name}",
+            href: "https://example.com/tags/#{tag.name}",
+          },
+          target: sender.featured_collection_url,
+        }.deep_stringify_keys
+      end
+
+      before do
+        sender.featured_tags.find_or_create_by!(name: tag.name)
+      end
+
+      it 'removes a pin' do
+        expect { subject.perform }
+          .to change { sender.featured_tags.exists?(tag: tag) }.to(false)
+      end
     end
   end
 end