Remove Follows::CreateChatChannelJob (#5718) [deploy]

app/jobs/follows/create_chat_channel_job.rb

@@ -1,14 +0,0 @@
-module Follows
-  class CreateChatChannelJob < ApplicationJob
-    queue_as :create_chat_channel_after_follow
-
-    def perform(follow_id)
-      follow = Follow.
-        includes(:follower, :followable).
-        find_by(id: follow_id, follower_type: "User", followable_type: "User")
-      return unless follow&.followable&.following?(follow.follower)
-
-      ChatChannel.create_with_users([follow.followable, follow.follower])
-    end
-  end
-end

spec/jobs/follows/create_chat_channel_job_spec.rb

@@ -1,45 +0,0 @@
-require "rails_helper"
-
-RSpec.describe Follows::CreateChatChannelJob, type: :job do
-  include_examples "#enqueues_job", "create_chat_channel_after_follow", 3
-
-  describe "#perform_now" do
-    context "with follow" do
-      let_it_be(:user) { create(:user) }
-      let_it_be(:user2) { create(:user) }
-      let_it_be(:follow) { create(:follow, follower: user, followable: user2) }
-
-      it "creates a chat channel when mutual followers" do
-        follow2 = create(:follow, follower: user2, followable: user)
-
-        # Follow has an after_create callback that creates a channel between the two users,
-        # so to make sure this test is correct, we delete all channels right after
-        ChatChannelMembership.delete_all
-        ChatChannel.delete_all
-
-        expect do
-          described_class.perform_now(follow2.id)
-        end.to change(ChatChannel, :count).by(1)
-      end
-
-      it "doesn't create a chat channel when the follow is not mutual" do
-        expect do
-          described_class.perform_now(follow.id)
-        end.not_to change(ChatChannel, :count)
-      end
-
-      it "doesn't do anything if follow is not from user to user" do
-        org_follow = create(:follow, follower: user, followable: create(:organization))
-        expect do
-          described_class.perform_now(org_follow.id)
-        end.not_to change(ChatChannel, :count)
-      end
-    end
-
-    context "without follow" do
-      it "does not break" do
-        expect { described_class.perform_now(nil) }.not_to raise_error
-      end
-    end
-  end
-end