assistant-storefront/spec/services/mailbox/conversation_finder_spec.rb

require 'rails_helper'

RSpec.describe Mailbox::ConversationFinder do
  let(:account) { create(:account) }
  let(:email_channel) { create(:channel_email, email: 'test@example.com', account: account) }
  let(:conversation) { create(:conversation, inbox: email_channel.inbox, account: account) }
  let(:mail) { Mail.new }

  describe '#find' do
    context 'when receiver uuid strategy finds conversation' do
      before do
        conversation.update!(uuid: '12345678-1234-1234-1234-123456789012')
        mail.to = 'reply+12345678-1234-1234-1234-123456789012@example.com'
      end

      it 'returns the conversation' do
        finder = described_class.new(mail)
        expect(finder.find).to eq(conversation)
      end

      it 'logs which strategy succeeded' do
        allow(Rails.logger).to receive(:info)
        finder = described_class.new(mail)
        finder.find

        expect(Rails.logger).to have_received(:info).with('Conversation found via receiver_uuid_strategy strategy')
      end
    end

    context 'when in_reply_to strategy finds conversation' do
      before do
        conversation.update!(uuid: '12345678-1234-1234-1234-123456789012')
        mail.in_reply_to = 'conversation/12345678-1234-1234-1234-123456789012/messages/123@example.com'
      end

      it 'returns the conversation' do
        finder = described_class.new(mail)
        expect(finder.find).to eq(conversation)
      end

      it 'logs which strategy succeeded' do
        allow(Rails.logger).to receive(:info)
        finder = described_class.new(mail)
        finder.find

        expect(Rails.logger).to have_received(:info).with('Conversation found via in_reply_to_strategy strategy')
      end
    end

    context 'when references strategy finds conversation' do
      before do
        conversation.update!(uuid: '12345678-1234-1234-1234-123456789012')
        mail.to = 'test@example.com'
        mail.references = 'conversation/12345678-1234-1234-1234-123456789012/messages/123@example.com'
      end

      it 'returns the conversation' do
        finder = described_class.new(mail)
        expect(finder.find).to eq(conversation)
      end

      it 'logs which strategy succeeded' do
        allow(Rails.logger).to receive(:info)
        finder = described_class.new(mail)
        finder.find

        expect(Rails.logger).to have_received(:info).with('Conversation found via references_strategy strategy')
      end
    end

    context 'when no strategy finds conversation' do
      # With NewConversationStrategy in default strategies, this scenario only happens
      # when using custom strategies that exclude NewConversationStrategy
      let(:finding_strategies) do
        [
          Mailbox::ConversationFinderStrategies::ReceiverUuidStrategy,
          Mailbox::ConversationFinderStrategies::InReplyToStrategy,
          Mailbox::ConversationFinderStrategies::ReferencesStrategy
        ]
      end

      it 'returns nil' do
        finder = described_class.new(mail, strategies: finding_strategies)
        expect(finder.find).to be_nil
      end

      it 'logs that no conversation was found' do
        allow(Rails.logger).to receive(:error)
        finder = described_class.new(mail, strategies: finding_strategies)
        finder.find

        expect(Rails.logger).to have_received(:error).with('No conversation found via any strategy (NewConversationStrategy missing?)')
      end
    end

    context 'with custom strategies' do
      let(:custom_strategy_class) do
        Class.new(Mailbox::ConversationFinderStrategies::BaseStrategy) do
          def find
            # Always return nil for testing
            nil
          end
        end
      end

      it 'uses provided strategies instead of defaults' do
        finder = described_class.new(mail, strategies: [custom_strategy_class])
        expect(finder.find).to be_nil
      end
    end

    context 'with strategy execution order' do
      before do
        conversation.update!(uuid: '12345678-1234-1234-1234-123456789012')

        # Set up mail so all strategies could match
        mail.to = 'reply+12345678-1234-1234-1234-123456789012@example.com'
        mail.in_reply_to = 'conversation/12345678-1234-1234-1234-123456789012/messages/123@example.com'
        mail.references = 'conversation/12345678-1234-1234-1234-123456789012/messages/456@example.com'
      end

      it 'returns conversation from first matching strategy' do
        allow(Rails.logger).to receive(:info)
        finder = described_class.new(mail)
        result = finder.find

        expect(result).to eq(conversation)
        # Should only log the first strategy that succeeded (ReceiverUuidStrategy)
        expect(Rails.logger).to have_received(:info).once.with('Conversation found via receiver_uuid_strategy strategy')
      end
    end
  end
end