trollolo 0.0.3 → 0.0.4

data/spec/unit/burndown_data_spec.rb

@@ -0,0 +1,118 @@
+require_relative 'spec_helper'
+
+include GivenFilesystemSpecHelpers
+
+describe BurndownData do
+  
+  before(:each) do
+    @burndown = BurndownData.new(dummy_settings)
+    @burndown.board_id = "53186e8391ef8671265eba9d"
+    full_board_mock
+  end
+
+  describe BurndownData::Result do
+    it "calculates total" do
+      r = described_class.new
+      r.open = 7
+      r.done = 4
+      
+      expect(r.total).to eq 11
+    end
+  end
+
+  describe "setters" do
+    it "sets open story points" do
+      @burndown.story_points.open = 13
+      expect(@burndown.story_points.open).to eq 13
+    end
+    
+    it "sets open tasks" do
+      @burndown.tasks.open = 42
+      expect(@burndown.tasks.open).to eq 42
+    end
+  end
+
+  describe "#fetch" do
+
+    before do
+      @burndown.fetch
+    end
+
+    it "returns story points" do
+      expect( @burndown.story_points.total ).to eq 16
+      expect( @burndown.story_points.open ).to eq 13
+      expect( @burndown.story_points.done ).to eq 3
+    end
+
+    it "returns extra story points" do
+      expect( @burndown.extra_story_points.total ).to eq 8
+      expect( @burndown.extra_story_points.open ).to eq 8
+      expect( @burndown.extra_story_points.done ).to eq 0
+    end
+
+    it "returns tasks" do
+      expect( @burndown.tasks.total ).to eq 13
+      expect( @burndown.tasks.open ).to eq 9
+      expect( @burndown.tasks.done ).to eq 4
+    end
+    
+    it "returns extra tasks" do
+      expect( @burndown.extra_tasks.total ).to eq 1
+      expect( @burndown.extra_tasks.open ).to eq 1
+      expect( @burndown.extra_tasks.done ).to eq 0
+    end
+
+    it "returns meta data" do
+      expect( @burndown.meta ).to eq({
+        "sprint" => 10,
+        "total_days" => 18,
+        "weekend_lines" => [1.5, 6.5, 11.5, 16.5]
+      })
+    end
+
+    it "saves date and time" do
+      expected_date_time = DateTime.parse("2015-01-12T13:57:16+01:00")
+      allow(DateTime).to receive(:now).and_return(expected_date_time)
+      @burndown.fetch
+      expect(@burndown.date_time).to eq(expected_date_time)
+    end
+  end
+
+  describe '#to_hash' do
+
+    it 'converts to hash' do
+      @burndown.story_points.open = 1
+      @burndown.story_points.done = 2
+      @burndown.tasks.open = 3
+      @burndown.tasks.done = 4
+      @burndown.extra_story_points.open = 5
+      @burndown.extra_story_points.done = 6
+      @burndown.extra_tasks.open = 7
+      @burndown.extra_tasks.done = 8
+      @burndown.date_time = DateTime.parse('20150115')
+
+      expected_hash = {
+        'date' => '2015-01-15',
+        'updated_at' => '2015-01-15T00:00:00+00:00',
+        'story_points' => {
+          'total' => 3,
+          'open' => 1
+        },
+        'tasks' => {
+          'total' => 7,
+          'open' => 3
+        },
+        'story_points_extra' => {
+          'done' => 6
+        },
+        'tasks_extra' => {
+          'done' => 8
+        }
+      }
+
+      expect(@burndown.to_hash).to eq(expected_hash)
+    end
+    
+  end
+
+end

data/spec/unit/card_spec.rb

@@ -0,0 +1,79 @@
+require_relative "spec_helper"
+
+describe Card do
+
+  describe "parses name" do
+    before(:each) do
+      allow_any_instance_of(Card).to receive(:init_data)
+      @card = Card.new(double, double)
+    end
+
+    it "extracts single digit story point value from card name" do
+      allow(@card).to receive(:name).and_return("(3) P1: Refactor cards")
+      expect(@card.story_points).to eq(3)
+    end
+
+    it "extracts double digit story point value from card name" do
+      allow(@card).to receive(:name).and_return "(13) P1: Refactor cards"
+      expect(@card.story_points).to eq(13)
+    end
+
+    it "extracts fractional story point value from card name" do
+      allow(@card).to receive(:name).and_return "(0.5) P1: Refactor cards"
+      expect(@card.story_points).to eq(0.5)
+    end
+
+    it "extracts story points when value is not at beginning of card name" do
+      allow(@card).to receive(:name).and_return "P01: (3) Refactor cards"
+      expect(@card.story_points).to eq(3)
+    end
+  end
+
+  describe "#parse_yaml_from_description" do
+    it "parses description only having YAML" do
+      description = <<EOT
+```yaml
+total_days: 18
+weekend_lines:
+  - 1.5
+  - 6.5
+```
+EOT
+      meta = Card.parse_yaml_from_description(description)
+      expect(meta["total_days"]).to eq(18)
+      expect(meta["weekend_lines"]).to eq([1.5, 6.5])
+    end
+
+    it "parses description only having unmarked YAML" do
+      description = <<EOT
+```
+total_days: 18
+weekend_lines:
+  - 1.5
+  - 6.5
+```
+EOT
+      meta = Card.parse_yaml_from_description(description)
+      expect(meta["total_days"]).to eq(18)
+      expect(meta["weekend_lines"]).to eq([1.5, 6.5])
+    end
+
+    it "parses description having YAML and text" do
+      description = <<EOT
+This is some text
+
+```yaml
+total_days: 18
+weekend_lines:
+  - 1.5
+  - 6.5
+```
+
+And more text.
+EOT
+      meta = Card.parse_yaml_from_description(description)
+      expect(meta["total_days"]).to eq(18)
+      expect(meta["weekend_lines"]).to eq([1.5, 6.5])
+    end
+  end
+end

data/spec/unit/cli_spec.rb

@@ -0,0 +1,38 @@
+require_relative 'spec_helper'
+
+include GivenFilesystemSpecHelpers
+
+describe Cli do
+
+  use_given_filesystem
+
+  before(:each) do
+    Cli.settings = dummy_settings
+    @cli = Cli.new
+    
+    allow(STDOUT).to receive(:puts)
+  end
+  
+  it "fetches burndown data" do
+    expect_any_instance_of(BurndownData).to receive(:fetch)
+    
+    @cli.fetch_burndown_data
+  end
+  
+  it "fetches burndown data from board-list" do
+    full_board_mock
+    dir = given_directory
+    @cli.options = {"board-list" => "spec/data/board-list.yaml",
+                    "output" => dir}
+    @cli.burndowns
+    expect(File.exist?(File.join(dir,"orange/burndown-data-01.yaml")))
+    expect(File.exist?(File.join(dir,"blue/burndown-data-01.yaml")))
+  end
+
+  it "backups board" do
+    expect_any_instance_of(Backup).to receive(:backup)
+    @cli.options = {"board-id" => "1234"}
+    @cli.backup
+  end
+
+end

data/spec/unit/retrieve_data_spec.rb

@@ -0,0 +1,54 @@
+require_relative 'spec_helper'
+
+describe "retrieve data through Trello API" do
+  before(:each) do
+    full_board_mock
+    trello_wrapper = TrelloWrapper.new(dummy_settings)
+    @board = trello_wrapper.board("53186e8391ef8671265eba9d")
+  end
+
+  describe "board" do
+    it "gets id" do
+      expect(@board.id).to eq("53186e8391ef8671265eba9d")
+    end
+
+    it "gets columns" do
+      columns = @board.columns
+      expect(columns.count).to eq(6)
+      expect(columns[0].name).to eq("Sprint Backlog")
+    end
+
+    it "gets cards" do
+      cards = @board.columns[0].cards
+      expect(cards.count).to eq(6)
+      expect(cards[0].name).to eq("Sprint 3")
+    end
+
+    it "gets checklist item counts" do
+      card = @board.columns[1].cards[0]
+      expect(card.tasks).to eq(2)
+      expect(card.done_tasks).to eq(1)
+    end
+
+    it "gets card labels" do
+      card = @board.columns[0].cards[5]
+      expect(card.card_labels.count).to eq(1)
+      expect(card.card_labels[0]["name"]).to eq("Under waterline")
+    end
+
+    it "gets card description" do
+      card = @board.columns[2].cards[1]
+      expected_desc = <<EOT
+```yaml
+total_days: 18
+weekend_lines:
+  - 1.5
+  - 6.5
+  - 11.5
+  - 16.5
+```
+EOT
+      expect(card.desc).to eq(expected_desc.chomp)
+    end
+  end
+end

data/spec/unit/scrum_board_spec.rb

@@ -0,0 +1,18 @@
+require_relative 'spec_helper'
+
+describe ScrumBoard do
+  describe '#done_column' do
+    before(:each) do
+      @settings = dummy_settings
+
+      board_data = JSON.parse(load_test_file("full-board.json"))
+      @scrum_board = ScrumBoard.new(board_data, @settings)
+    end
+
+    it 'raises error when done column cannot be found' do
+      @settings.done_column_name_regex = /thiscolumndoesntexist/
+
+      expect{@scrum_board.done_column}.to raise_error ScrumBoard::DoneColumnNotFoundError
+    end
+  end
+end

data/spec/{settings_spec.rb → unit/settings_spec.rb}

@@ -6,7 +6,7 @@ describe Settings do
   
   context "given config file" do
     before(:each) do
-      @settings = Settings.new( File.expand_path('../data/trollolorc',__FILE__) )
+      @settings = Settings.new( File.expand_path('../../data/trollolorc',__FILE__) )
     end
     
     it "is not verbose by default" do

data/spec/{spec_helper.rb → unit/spec_helper.rb}

@@ -1,23 +1,15 @@
 require "codeclimate-test-reporter"
 CodeClimate::TestReporter.start
-
-require_relative '../lib/trollolo'
-
+require_relative '../../lib/trollolo'
 require 'given_filesystem/spec_helpers'
 require 'webmock/rspec'
-
+require 'byebug'
 WebMock.disable_net_connect!(:allow => "codeclimate.com")
 
-bin_path = File.expand_path( "../../bin/", __FILE__ )
+bin_path = File.expand_path( "../../../bin/", __FILE__ )
 
 if ENV['PATH'] !~ /#{bin_path}/
   ENV['PATH'] = bin_path + File::PATH_SEPARATOR + ENV['PATH']
 end
 
-def load_test_file filename
-  File.read(File.expand_path('../data/' + filename,__FILE__))
-end
-
-def dummy_settings
-  Settings.new( File.expand_path('../data/trollolorc',__FILE__) )
-end
+Dir.glob(::File.expand_path('../support/*.rb', __FILE__)).each { |f| require_relative f }

data/spec/unit/support/test_data_operations.rb

@@ -0,0 +1,7 @@
+def load_test_file(filename)
+  File.read(File.expand_path('../../../data/' + filename, __FILE__))
+end
+
+def dummy_settings
+  Settings.new(File.expand_path('../../../data/trollolorc', __FILE__))
+end

data/spec/unit/support/update_webmock_data

@@ -0,0 +1,17 @@
+#!/usr/bin/env ruby
+
+require_relative "webmocks"
+
+trollolo_bin = File.expand_path("../../../bin/trollolo", __FILE__)
+spec_data_dir = File.expand_path("../../data", __FILE__)
+
+STDERR.puts "Updating web mock data"
+
+webmock_mapping.each do |mapping|
+  url = mapping[:path] + parameters_as_string(mapping)
+  file = mapping[:file]
+
+  cmd = "#{trollolo_bin} get-raw '#{url}' >#{spec_data_dir}/#{file}"
+  puts cmd
+  system cmd
+end

data/spec/unit/support/webmocks.rb

@@ -0,0 +1,52 @@
+def webmock_mapping
+  [
+    {
+      path: 'boards/53186e8391ef8671265eba9d',
+      file: 'board.json'
+    },
+    {
+      path: 'boards/53186e8391ef8671265eba9d/lists',
+      parameters: {
+        "filter" => "open"
+      },
+      file: 'lists.json'
+    },
+    {
+      path: 'boards/53186e8391ef8671265eba9d',
+      parameters: {
+        "cards" => "open",
+        "lists" => "open",
+        "card_checklists" => "all"
+      },
+      file: 'full-board.json'
+    }
+  ]
+end
+
+def parameters_as_string(mapping, parameters = nil)
+  parameters ||= []
+  if mapping[:parameters]
+    mapping[:parameters].each do |key, value|
+      parameters.push("#{key}=#{value}")
+    end
+  end
+  if !parameters.empty?
+    parameters_string = "?" + parameters.join("&")
+  else
+    parameters_string = ""
+  end
+  parameters_string
+end
+
+def mapping_url(mapping, parameters = nil)
+  url = "https://api.trello.com/1/" + mapping[:path]
+  url += parameters_as_string(mapping, parameters)
+end
+
+def full_board_mock
+  webmock_mapping.each do |mapping|
+    url = mapping_url(mapping, [ "key=mykey", "token=mytoken" ])
+    stub_request(:get, url)
+      .to_return(:status => 200, :body => load_test_file(mapping[:file]))
+  end
+end

data/spec/unit/trello_wrapper_spec.rb

@@ -0,0 +1,47 @@
+require_relative 'spec_helper'
+
+describe TrelloWrapper do
+
+  let!(:settings){ double('settings', developer_public_key: "mykey", member_token: "mytoken") }
+  subject { described_class.new(settings) }
+
+  before do
+    stub_request(:get, "https://api.trello.com/1/boards/myboard?cards=open&key=mykey&lists=open&token=mytoken").
+        to_return(:status => 200, :body => load_test_file("board.json"), :headers => {})
+    full_board_mock
+  end
+
+  describe '.new' do
+    it 'populates settings' do
+      expect(subject.instance_variable_get(:@settings)).to be settings
+    end
+
+    it 'init trello configuration' do
+      expect_any_instance_of(described_class).to receive(:init_trello)
+      described_class.new(settings)
+    end
+  end
+
+  describe '#board' do
+    before(:each) do
+      expect(subject).to receive(:retrieve_board_data).with('myboard').and_return(:board)
+    end
+
+    it 'finds board via Trello' do
+      subject.board("myboard")
+    end
+
+    it 'instantiate ScrumBoard with trello board and settings' do
+      expect(ScrumBoard).to receive(:new).with(:board, subject.instance_variable_get(:@settings))
+      subject.board("myboard")
+    end
+
+    it 'returns instance of a ScrumBoard' do
+      expect(subject.board("myboard")).to be_instance_of(ScrumBoard)
+    end
+
+    it 'memoize board object' do
+      expect(subject.board("myboard")).to be subject.board("myboard")
+    end
+  end
+end