happy-commander 0.0.6

data/lib/commander/trello.rb

@@ -0,0 +1,63 @@
+require 'trello'
+
+
+class Trello::Card
+  # Monkeypatching..
+  def assignees
+    @trello = Commander::TrelloConnection.new
+    member_ids.map{|id| @trello.find_member_by_id(id)}
+  end
+end
+
+
+module Commander
+  # Trello Module
+  class TrelloConnection
+
+    attr_accessor :board
+
+    # Loads the Configuration for Trello Auth
+    def initialize
+      configure_trello
+    end
+
+    # Adds a member to a certain card
+    def add_commander_to_card(commander, card)
+      card.add_member(commander) if commander
+    end
+
+    # Comments on a certain card
+    def comment_on_card(commander, card)
+      card.add_comment(commander) if commander
+    end
+
+    # Finds a member by its username
+    def find_member_by_username(username)
+      @board.members.find { |m| m.username == username }
+    end
+
+    # Lists all members who are assigned on a certain card
+    def list_of_assigned_members(card)
+      card.assignees.map(&:username)
+    end
+
+    # Finds a member by its id
+    def find_member_by_id(id)
+      @board.members.find { |m| m.id == id }
+    end
+
+    # Finds a card by its id
+    def find_card_by_id(id)
+      @board.cards.find{|c| c.short_id == id.to_i}
+    end
+
+    # Configures Trello
+    def configure_trello
+      Trello.configure do |config|
+        config.developer_public_key = Commander::CONFIG['consumerkey']
+        config.member_token = Commander::CONFIG['oauthtoken']
+      end
+      @board = Trello::Board.find(Commander::CONFIG['board_id'])
+    end
+  end
+end

data/lib/commander/vacations.rb

@@ -0,0 +1,38 @@
+require 'net/telnet'
+module Commander
+
+  class Vacations
+
+    def self.find_vacations(username)
+      vacations = []
+      tn = Net::Telnet.new('Host' => 'present.suse.de', 'Port' => 9874, 'Binmode' => false)
+      collect = false
+      tn.cmd(username) do |data|
+        data.split("\n").each do |l|
+          collect = true if l =~ /^Absence/
+          next unless collect
+          if l[0,1] == "-"
+            collect = false
+            next
+          end
+          dates = []
+          l.split(" ").each do |date|
+            unless date =~ /#{Time.now.year}/
+              next
+            end
+            dates.push(date)
+          end
+          case dates.size
+            when 1
+              vacations.push("#{dates[0]}")
+            when 2
+              vacations.push("#{dates[0]} - #{dates[1]}")
+            else
+          end
+        end
+      end
+      tn.close
+      vacations
+    end
+  end
+end

data/lib/commander/version.rb

@@ -0,0 +1,4 @@
+# Version Control
+module Commander
+  VERSION = '0.0.6'
+end

data/spec/client_spec.rb

@@ -0,0 +1,121 @@
+require 'spec_helper'
+
+describe Commander::Client do
+
+  subject { Commander::Client }
+
+  before do
+    allow_any_instance_of(Commander::Runner).to receive(:set_commander) { true }
+  end
+
+  describe '#execute' do
+    context 'with auto options' do
+      let(:cli) { subject.new(%w{-a}) }
+
+      it 'should run with default settings' do
+        expect(Commander::Runner).to receive(:new).with(cli.options).and_return Commander::Runner.new(cli.options)
+        expect_any_instance_of(Commander::Runner).to receive(:set_commander)
+        cli.execute!
+      end
+    end
+
+    context 'with force options' do
+      let(:cli) { subject.new(%w{-f, name}) }
+
+      it 'should run with force settings' do
+        expect(Commander::Runner).to receive(:new).with(cli.options).and_return Commander::Runner.new(cli.options)
+        expect_any_instance_of(Commander::Runner).to receive(:set_commander)
+        cli.execute!
+      end
+    end
+
+    context 'with list options' do
+      let(:cli) { subject.new(%w{-l}) }
+
+      it 'should run with list settings' do
+        expect(Commander::Runner).to receive(:new).with(cli.options).and_return Commander::Runner.new(cli.options)
+        expect_any_instance_of(Commander::Runner).to receive(:list_all_members)
+        cli.execute!
+      end
+    end
+
+    context 'with status options' do
+      let(:cli) { subject.new(%w{-s, name}) }
+
+      it 'should run with force settings' do
+        expect(Commander::Runner).to receive(:new).with(cli.options).and_return Commander::Runner.new(cli.options)
+        expect_any_instance_of(Commander::Runner).to receive(:show_status)
+        cli.execute!
+      end
+    end
+
+    context 'with vacation options' do
+      let(:cli) { subject.new(%w{-v, name, true}) }
+
+      it 'should run with vacation settings' do
+        expect(Commander::Runner).to receive(:new).with(cli.options).and_return Commander::Runner.new(cli.options)
+        expect_any_instance_of(Commander::Runner).to receive(:set_vacation_flag)
+        cli.execute!
+      end
+    end
+
+    context 'with setup options' do
+      let(:cli) { subject.new(%w{-x}) }
+
+      it 'should run with setup settings' do
+        expect(Commander::Setup).to receive(:configure)
+        cli.execute!
+      end
+    end
+
+    context 'with no options' do
+      let(:cli) { subject.new(%w{}) }
+
+      it 'should exit cleanly when no arg is given' do
+        expect_any_instance_of(subject).to receive(:exit)
+        cli.execute!
+      end
+    end
+  end
+
+  describe '?extract_options' do
+    it 'sets status options' do
+      cli = subject.new(%w{-s Joshua})
+      expect(cli.options[:status]).to eq 'Joshua'
+    end
+
+    it 'sets force options' do
+      cli = subject.new(%w{-f Joshua})
+      expect(cli.options[:force]).to eq 'Joshua'
+    end
+
+    it 'sets vacation options' do
+      cli = subject.new(%w{-v Joshua,true})
+      expect(cli.options[:vacation]).to eq ['Joshua', 'true']
+    end
+
+    it 'sets list options' do
+      cli = subject.new(['-l'])
+      expect(cli.options[:list]).to eq true
+    end
+
+    it 'sets auto options' do
+      cli = subject.new(['-a'])
+      expect(cli.options[:auto]).to eq true
+    end
+
+    it 'sets help options' do
+      argv = %w{-h}
+      expect_any_instance_of(subject).to receive(:puts)
+      expect_any_instance_of(subject).to receive(:exit)
+      subject.new(argv)
+    end
+
+    it 'gets version options' do
+      argv = %w{-u}
+      expect_any_instance_of(subject).to receive(:puts).with(Commander::VERSION)
+      expect_any_instance_of(subject).to receive(:exit)
+      subject.new(argv)
+    end
+  end
+end

data/spec/fixtures/trello.txt

@@ -0,0 +1 @@
+card_id: 15

data/spec/fixtures/users.txt

@@ -0,0 +1,12 @@
+{"Thomas"=>
+{:times_commander=>3, :trello_name=>"thomasschmidt", :date=>2014-07-11 11:23:21 +0200, :vacation=>false, :tel_name=>"Thomas Schmidt", :vacations=>["2014-07-25 - 2014-07-27", "2014-08-01 - 2014-08-03", "2014-08-16 - 2014-08-31", "2014-10-03 - 2014-10-05", "2014-12-24 - 2014-12-28"]},
+ "Vlad"=>
+ {:times_commander=>3, :trello_name=>"vlewin", :date=>2014-07-11 11:12:08 +0200, :vacation=>false, :tel_name=>"Vladislav Lewin", :vacations=>["2014-08-23 - 2014-09-14", "2014-10-03 - 2014-10-05", "2014-12-24 - 2014-12-28"]},
+  "Dominik"=>
+  {:times_commander=>0, :trello_name=>"b4mboo", :date=>2014-07-10 13:23:19 +0200, :vacation=>true, :tel_name=>"Dominik Bamberger", :vacations=>["2014-07-04 - 2014-07-06", "2014-07-08 - 2014-07-13", "2014-10-03 - 2014-10-05", "2014-12-24 - 2014-12-28"]},
+   "Joshua"=>
+   {:times_commander=>4, :trello_name=>"jschmid", :date=>2014-07-11 11:24:46 +0200, :vacation=>false, :tel_name=>"Joshua Schmid", :vacations=>["2014-07-19 - 2014-07-23", "2014-07-26 - 2014-07-28", "2014-10-03 - 2014-10-05", "2014-12-24 - 2014-12-28"]},
+   "Artem"=>
+   {:times_commander=>3, :trello_name=>"artemchernikov", :date=>2014-07-11 11:09:16 +0200, :vacation=>false, :tel_name=>"Artem Chernikov", :vacations=>["2014-10-03 - 2014-10-05", "2014-12-24 - 2014-12-28"]},
+    "Will"=>
+    {:times_commander=>3, :trello_name=>"wstephenson", :date=>2014-07-11 11:08:59 +0200, :vacation=>false, :tel_name=>"Will Stephenson", :vacations=>["2014-07-05 - 2014-07-07", "2014-10-03 - 2014-10-05", "2014-12-24 - 2014-12-28"]}}

data/spec/fixtures/vacations.txt

@@ -0,0 +1 @@
+[['2014-08-08','2014-08-09'],['2014-08-18','2014-08-20'],['2014-09-08','2014-10-08'],['2014-11-08','2014-12-08']]

data/spec/helpers_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe Commander::Helpers do
+
+  subject { Commander::Helpers }
+
+  describe '.to_boolean' do
+
+    it 'converts string to bool' do
+      expect(subject.to_boolean('true')).to eq true
+    end
+  end
+end

data/spec/runner_spec.rb

@@ -0,0 +1,262 @@
+require 'spec_helper'
+require 'fileutils'
+
+describe Commander::Runner do
+
+  let( :vacations) { File.read(File.join(File.dirname(__FILE__), 'fixtures/vacations.txt')) }
+  let( :config) { File.read(File.join(File.dirname(__FILE__), 'fixtures/trello.txt')) }
+  let( :users) { File.read(File.join(File.dirname(__FILE__), 'fixtures/users.txt')) }
+  subject { Commander::Runner }
+
+  describe '.new' do
+    context 'without options' do
+      it 'initializes new instance of Commander::Runner' do
+        instance = subject.new({})
+        expect(instance.users).not_to be nil
+        expect(instance.options[:force]).to be_nil
+        expect(instance.selected_commander).to be nil
+      end
+    end
+
+    context 'with options' do
+      it 'initializes new instance of Commander::Runner' do
+        instance = subject.new({:vacation=>false, :force=>'a_user', :status=>false, :auto=>true, :list=>false})
+        expect(instance.options[:force]).to be_a_kind_of String
+        expect(instance.users).not_to be nil
+        expect(instance.options[:auto]).to be true
+        expect(instance.selected_commander).to eq 'a_user'
+      end
+    end
+  end
+
+  describe '#set_commander' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>false, :status=>false, :auto=>true, :list=>false}) }
+
+    it 'sets the commander with help of methods' do
+      expect(subject).to receive(:import)
+      expect(subject).to receive(:find_card)
+      expect(subject).to receive(:update_vacations)
+      expect(subject).to receive(:select_commander)
+      expect(subject).to receive(:manipulate_trello)
+      expect(subject).not_to receive(:write_attributes)
+      subject.set_commander
+    end
+  end
+
+  describe '#manipulate_trello' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>false, :status=>false, :auto=>true, :list=>false}) }
+
+    it 'holds trello manipulation methods' do
+      expect(subject).to receive(:comment_on_card)
+      expect(subject).to receive(:delete_assigned_members)
+      expect(subject).to receive(:add_member_to_card)
+      subject.manipulate_trello
+    end
+  end
+
+  describe '#set_commander' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>false}) }
+
+    it 'sets the commander with help of methods' do
+      expect(subject).to receive(:import)
+      expect(subject).to receive(:find_card)
+      expect(subject).to receive(:update_vacations)
+      expect(subject).not_to receive(:select_commander)
+      expect(subject).to receive(:write_attributes)
+      expect(subject).to receive(:manipulate_trello)
+      subject.set_commander
+    end
+  end
+
+  describe '#update_vacations' do
+
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>false}) }
+
+    let( :vacations) { File.read(File.join(File.dirname(__FILE__), 'fixtures/vacations.txt')) }
+
+    it 'updates vacations from tel' do
+      instance_variable_set(:@vacations, vacations)
+      allow(Commander::Vacations).to receive(:find_vacations).with(any_args).and_return vacations
+      expect(subject).to receive_message_chain(:users, :keys, :each)
+      # expect(subject).to receive(:evaluate_vacations)
+      subject.update_vacations
+    end
+
+    it 'can write to file' do
+      expect(subject).to receive(:write_to_file)
+      subject.update_vacations
+    end
+  end
+
+  describe '#set_vacation_flag' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>false}) }
+
+    it 'sets a vacation flag if on vacation' do
+      allow(Commander::Helpers).to receive(:to_boolean).and_return 'true'
+      subject.set_vacation_flag(subject.users.first.first, 'true')
+    end
+  end
+
+  describe '#evaulate_vacations' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>false}) }
+
+    it 'evaluates vacation times' do
+      allow(subject).to receive(:parse_vacations).and_return [1,2,3,4]
+      subject.evaluate_vacations(subject.users.first.first)
+    end
+
+  end
+
+  describe '#show_status' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>false, :status=>'Joshua', :auto=>false, :list=>false}) }
+
+    it 'prints out the status' do
+      expect(STDOUT).to receive(:puts).with("#{subject.options[:status]} was #{subject.users['Joshua'][:times_commander]} times Commanding officer of the week.")
+      expect(STDOUT).to_not receive(:puts).with("#{subject.options[:status]} is currently on vacation.")
+      subject.show_status(subject.options[:status])
+    end
+
+  end
+
+  describe '#wirte_attributes' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'writes standard attributes' do
+      instance_variable_set(:@selected_commander, 'test')
+      instance_variable_set(:@users, 'test')
+      expect(subject.users.first.last[:vacation]).to eq false
+      expect(subject.users.first.last[:times_commander]).to be_a_kind_of Integer
+      expect(subject.users.first.last[:date]).to be_a_kind_of Time
+      expect(subject).to receive(:write_to_file).and_return true
+      subject.write_attributes
+    end
+  end
+
+
+  describe '#delete_assigned_members' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'delete all remaining members on the trello card' do
+      card = Trello::Card.new
+      members = Trello::Member.new
+      subject.instance_variable_set(:@card, card)
+      expect(card).to receive_message_chain(:members, :each) { members }
+      subject.delete_assigned_members
+    end
+  end
+
+
+
+  describe '#list_all_members' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'lists all available members' do
+      # user = subject.users.first.first
+      expect(STDOUT).to receive(:puts).with(String)
+      expect(STDOUT).to receive(:puts).with(String)
+      expect(STDOUT).to receive(:puts).with(String)
+      expect(STDOUT).to receive(:puts).with(String)
+      expect(STDOUT).to receive(:puts).with(String)
+      expect(STDOUT).to receive(:puts).with(String)
+      # how to test 6 times?
+      subject.list_all_members
+    end
+  end
+
+  describe '#find_card' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'find the card from trello board' do
+      card = double('card')
+      trello = double('trello')
+      subject.instance_variable_set(:@trello, trello)
+      allow(subject).to receive(:find_card_by_id).with(Commander::CONFIG[:card_id])
+      expect(trello).to receive(:find_card_by_id).with('card')
+      subject.find_card
+    end
+  end
+
+  describe '#count_up' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'increments the user counter' do
+      instance_variable_set(:@selected_commander, 'Joshua')
+      count = subject.users[@selected_commander][:times_commander]
+      expect(subject.count_up).to eq (count + 1)
+    end
+  end
+
+  describe '#add_member_to_card' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'adds a member to the specified trello card ' do
+      trello = double('trello')
+      card =  double('card')
+      member = double('member')
+      subject.instance_variable_set(:@trello, trello)
+      subject.instance_variable_set(:@card, card)
+      expect(subject).to receive(:find_member).and_return member
+      expect(trello).to receive(:add_commander_to_card).with(member, card)
+      subject.add_member_to_card
+    end
+  end
+
+  describe '#find_member' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'finds the member based on its trello username' do
+      trello = double('trello')
+      subject.instance_variable_set(:@trello, trello)
+      expect(trello).to receive(:find_member_by_username)
+      subject.find_member
+    end
+  end
+
+  describe '#wirte_to_file' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'writes all the stuff to the yaml file' do
+      expect(File).to receive(:open).and_return true
+      File.open
+    end
+  end
+
+  describe '#comment_on_card' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'comments on the card' do
+      trello = double('trello')
+      card = double('card')
+      comment_string = '@jschmid is your commanding officer for the next 7 Days.'
+      subject.instance_variable_set(:@trello, trello)
+      subject.instance_variable_set(:@comment_string, comment_string)
+      subject.instance_variable_set(:@card, card)
+      expect(trello).to receive(:comment_on_card).with(comment_string, card)
+      subject.comment_on_card
+    end
+  end
+
+  describe '#select_commander' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'selects a commander based on date, count and presence' do
+      subject.instance_variable_get(:@selected_commander)
+      subject.select_commander
+    end
+  end
+
+  describe '#import' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>'Joshua', :status=>false, :auto=>false, :list=>true}) }
+
+    it 'imports trello config' do
+      stub_boards_call
+      expect(subject.import).to be_a_kind_of Commander::TrelloConnection
+    end
+  end
+
+  describe '#select_commander' do
+    subject { Commander::Runner.new({:vacation=>false, :force=>false, :status=>false, :auto=>true, :list=>true}) }
+      it 'selects the commander based on yamldata' do
+      end
+  end
+end