discourse/plugins/discourse-solved/spec/lib/post_mover_extension_spec.rb

# frozen_string_literal: true

RSpec.describe DiscourseSolved::PostMoverExtension do
  fab!(:admin)
  fab!(:category)
  fab!(:topic) { Fabricate(:topic_with_op, category: category) }
  fab!(:reply) { Fabricate(:post, topic: topic) }
  fab!(:another_reply) { Fabricate(:post, topic: topic) }
  fab!(:destination_topic) { Fabricate(:topic_with_op, category: category) }

  let!(:post_ids) { topic.posts.order(:post_number).pluck(:id) }
  fab!(:solved) { Fabricate(:solved_topic, topic: topic, answer_post: reply) }

  before { SiteSetting.allow_solved_on_all_topics = true }

  context "when moving the solution post" do
    it "transfers the solution to an existing topic" do
      topic.move_posts(admin, post_ids, destination_topic_id: destination_topic.id)

      expect(topic.reload.solved).to be_nil
      expect(destination_topic.reload.topic_answers.first.answer_post_id).to eq(reply.id)
    end

    it "transfers the solution to a new topic" do
      new_topic =
        topic.move_posts(admin, [reply.id], title: "This is a new topic for the moved post")

      expect(topic.reload.solved).to be_nil
      expect(new_topic.reload.topic_answers.first.answer_post_id).to eq(reply.id)
    end

    it "does not transfer the solution when destination topic does not allow solved" do
      SiteSetting.allow_solved_on_all_topics = false

      topic.move_posts(admin, post_ids, destination_topic_id: destination_topic.id)

      expect(topic.reload.solved).to be_nil
      expect(destination_topic.reload.solved).to be_nil
    end

    it "does not transfer the solution when the destination topic already has a solution" do
      Fabricate(:solved_topic, topic: destination_topic, answer_post: destination_topic.first_post)

      topic.move_posts(admin, post_ids, destination_topic_id: destination_topic.id)

      expect(topic.reload.solved).to be_nil
      expect(destination_topic.reload.topic_answers.first.answer_post_id).not_to eq(reply.id)
    end

    it "moving multiple answers to a single topic only keeps the first accepted" do
      Fabricate(:topic_answer, solved_topic: solved, post: another_reply)
      target_topic = Fabricate(:topic_with_op)

      topic.move_posts(admin, [reply.id, another_reply.id], destination_topic_id: target_topic.id)

      expect(topic.reload.solved).to be_nil

      expect(target_topic.reload.topic_answers.count).to eq(1)
      expect(target_topic.topic_answers.first.answer_post_id).to eq(reply.id)
      expect(target_topic.posts.where.not(post_number: 1).pluck(:id)).to contain_exactly(
        reply.id,
        another_reply.id,
      )
      expect(DiscourseSolved::TopicAnswer.find_by(answer_post_id: another_reply.id)).to be_nil
    end

    describe "with multiple solutions enabled" do
      before { SiteSetting.solved_allow_multiple_solutions = true }

      it "transfers one of many solutions to an existing topic" do
        Fabricate(:topic_answer, solved_topic: solved, post: another_reply)
        topic.move_posts(admin, [reply.id], destination_topic_id: destination_topic.id)

        expect(topic.reload.topic_answers.first.answer_post_id).to eq(another_reply.id)
        expect(destination_topic.reload.topic_answers.first.answer_post_id).to eq(reply.id)
      end

      it "transfers one of many solutions to a new topic" do
        Fabricate(:topic_answer, solved_topic: solved, post: another_reply)
        new_topic =
          topic.move_posts(admin, [reply.id], title: "This is a new topic for the moved post")

        expect(topic.reload.topic_answers.first.answer_post_id).to eq(another_reply.id)
        expect(new_topic.reload.topic_answers.first.answer_post_id).to eq(reply.id)
      end

      it "does transfer the solution when the destination topic already has a solution" do
        Fabricate(
          :solved_topic,
          topic: destination_topic,
          answer_post: destination_topic.first_post,
        )

        topic.move_posts(admin, [reply.id], destination_topic_id: destination_topic.id)

        expect(topic.reload.solved).to be_nil
        expect(destination_topic.reload.topic_answers[0].answer_post_id).to eq(
          destination_topic.first_post.id,
        )
        expect(destination_topic.reload.topic_answers[1].answer_post_id).to eq(reply.id)
      end
    end
  end

  context "when not moving the solution post" do
    it "keeps the solution on the original topic" do
      topic.move_posts(admin, [post_ids.first], destination_topic_id: destination_topic.id)

      expect(solved.reload.topic_id).to eq(topic.id)
    end

    it "keeps the solution when moving to a new topic" do
      topic.move_posts(admin, [another_reply.id], title: "This is a new topic for the moved post")

      expect(solved.reload.topic_id).to eq(topic.id)
    end

    it "does not affect the destination topic's solution" do
      Fabricate(:solved_topic, topic: destination_topic, answer_post: destination_topic.first_post)

      topic.move_posts(admin, [another_reply.id], destination_topic_id: destination_topic.id)

      expect(solved.reload.topic_id).to eq(topic.id)
      expect(destination_topic.reload.topic_answers.first.answer_post_id).to eq(
        destination_topic.first_post.id,
      )
    end

    it "does not create a SolvedTopic when moving non-answer posts to a destination that allows answers" do
      topic.move_posts(admin, [post_ids.first], destination_topic_id: destination_topic.id)

      expect(topic.reload.solved).not_to be_nil
      expect(destination_topic.reload.solved).to be_nil
    end
  end
end