require 'spec_helper'

describe Reviewlette do

  subject { Reviewlette }
  let( :reviewlette ) { subject.new }

  before {
    allow(GithubConnection).to receive(:new).and_return GithubConnection
    allow(TrelloConnection).to receive(:new).and_return TrelloConnection
  }


  describe '.new' do
    it 'sets github and trello connections' do
       expect(GithubConnection).to receive (:new)
       expect(TrelloConnection).to receive (:new)
       subject.new
    end
  end

  describe '.run' do

    it 'iterates over all open pull requests and extracts trello ids from name' do
      expect(GithubConnection).to receive(:unassigned_pull_requests).and_return([{number: 11, title: 'test_issue_12'}])
      expect(TrelloConnection).to receive(:find_card_by_id).with(12)

      reviewlette.run
    end

    it 'adds assignee and reviewer comment on github, adds comment on trello and moves card' do
      card = Trello::Card.new
      expect(GithubConnection).to receive(:unassigned_pull_requests).and_return([{number: 11, title: 'test_issue_12'}])
      expect(TrelloConnection).to receive(:find_card_by_id).with(12).and_return(card)
      expect(reviewlette).to receive(:select_reviewer).and_return({'suse_username' => 'test', 'github_username' => 'testgit'})

      expect(GithubConnection).to receive(:add_assignee).with(11, 'testgit')
      expect(GithubConnection).to receive(:reviewer_comment).with(11, 'testgit', card)
      expect(GithubConnection).to receive(:repo).and_return('SUSE/test')

      expect(TrelloConnection).to receive(:comment_on_card).with("@ will review https://github.com/SUSE/test/issues/11", card)
      expect(TrelloConnection).to receive(:move_card_to_list).with(card, 'In review')

      reviewlette.run
    end

  end

  describe '.select_reviewer' do

    MEMBERS_CONFIG['members'] = [{'suse_username' => 'test1', 'trello_username' => 'trellotest1'},
                                 {'suse_username' => 'test2', 'trello_username' => 'trellotest2'}]

    it 'excludes members on vacation' do
      card = Trello::Card.new
      allow(card).to receive(:members).and_return([])
      expect(Vacations).to receive(:members_on_vacation).and_return(MEMBERS_CONFIG['members'].first['suse_username'])
      expect(reviewlette.select_reviewer(nil, card)).to eq(MEMBERS_CONFIG['members'].last)
    end

    it 'excludes the owner of the trello card' do
      card = Trello::Card.new
      allow(card).to receive_message_chain(:members, :map).and_return(['trellotest1'])
      expect(Vacations).to receive(:members_on_vacation).and_return([])
      expect(reviewlette.select_reviewer(nil, card)).to eq(MEMBERS_CONFIG['members'].last)
    end

  end


end