trello_lead_time 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 904205f708050d8b62075e053346c1fd68d55df8
-  data.tar.gz: 973dc0bcbd6b5d5870b32b740dce5f1c71b098c1
+  metadata.gz: 99edb0b33c6e54cfeaaf2b33a9f1a30a9207ef3c
+  data.tar.gz: 53abfe5d13488621fb6aa181bc824c9b8cb3b71b
 SHA512:
-  metadata.gz: 59c7f5637a70509d785cf64ac2469383b57efcff616beb14588d23f030103e58674545520626ff7ea6cc8b85a9ec0ca887411b245e9ee9fc81bb3c2b2ff2ab80
-  data.tar.gz: 791bd8306d02dc97906a83015721bb4b7aa22c1cab8ecab0b2f051d5df832dfcd3a8e0909083a6b7aef0f5d6b35537d3bea7b8d7e6d893eeebce9e0647ef155f
+  metadata.gz: 1ab0c95cb5dd91bb05b48f0e85bdb81a517b01c1032ee777ec35bd3f73962b159b75230931ec2aec5c8de8f65c7f04b831a8f3bb3c4cc62e46288575afa7b680
+  data.tar.gz: 6458a52dbc862e23a4dcd4ffe196259ada4f352acb791da5b33f7fd694cd88d92ba74d247daf4386e74247713e0767de076922d97f91e0a7b8cae8634adedca0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    trello_lead_time (0.1.2)
+    trello_lead_time (0.1.3)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -50,6 +50,10 @@ Now, configure how you will calculate the metrics.
     queue_time_lists     = ['Next Up']
     cycle_time_lists     = ['In Progress', 'Testing']
 
+Lead time analysis can be performed on a per-label basis. Create an array of labels and for which you want to get the total lead time of cards with those labels.
+
+    finance_type_labels        = ['Feature', 'Bug']
+
 When analyzing the timeline of each Trello card, the gem has to find out when it was moved into a "Done" state. This usually is the event when the card is placed in the one of the `source_lists`, e.g. "Live (4/8)". 
 
 You must provide a regular expression that will be compared to the names of lists a Trello card was in (at some point of its life) to identify when it was marked done.
@@ -59,11 +63,12 @@ You must provide a regular expression that will be compared to the names of list
 Now that all those variables are set, you can configure the gem!
 
     TrelloLeadTime.configure do |cfg|
-      cfg.organization_name = organization_name
-      cfg.set_trello_key_and_token(developer_public_key, member_token)
-      cfg.queue_time_lists = queue_time_lists
-      cfg.cycle_time_lists = cycle_time_lists
+      cfg.organization_name          = organization_name
+      cfg.queue_time_lists           = queue_time_lists
+      cfg.cycle_time_lists           = cycle_time_lists
+      cfg.finance_type_labels        = finance_type_labels
       cfg.list_name_matcher_for_done = /^Live/
+      cfg.set_trello_key_and_token(developer_public_key, member_token)
     end
 
 Easy right? Now let's put it to use!
@@ -78,7 +83,7 @@ Easy right? Now let's put it to use!
       totals   = board.totals(source_list)
       averages = board.averages(source_list)
 
-      puts "Using cards in list: #{source_list}"
+      puts "Overall metrics for: #{source_list}"
       puts "\tAverage Card Age:   #{TrelloLeadTime::TimeHumanizer.humanize_seconds(averages[:age][:overall])}"
       puts "\tAverage Lead Time:  #{TrelloLeadTime::TimeHumanizer.humanize_seconds(averages[:lead_time][:overall])}"
       puts "\tAverage Queue Time: #{TrelloLeadTime::TimeHumanizer.humanize_seconds(averages[:queue_time][:overall])}"
@@ -89,6 +94,11 @@ Easy right? Now let's put it to use!
       puts "\tTotal Queue Time:   #{TrelloLeadTime::TimeHumanizer.humanize_seconds(totals[:queue_time][:overall])}"
       puts "\tTotal Cycle Time:   #{TrelloLeadTime::TimeHumanizer.humanize_seconds(totals[:lead_time][:overall])}"
       puts ""
+      puts "\tFinance type breakdown (total lead time per label):"
+      totals[:lead_time][:finance_types].each do |label, value|
+          puts "\t\t#{label}: #{TrelloLeadTime::TimeHumanizer.humanize_seconds(value)}"
+      end
+      puts ""
     end
 
 For each of the source lists, the cards are analyzed and aggregate (average and total) metrics are displayed. This takes some time because the Trello API must be hit for each card being analyzed. Therefore, the longer the lists in Trello, the longer it takes to process.

data/lib/trello_lead_time/board.rb

@@ -30,6 +30,12 @@ module TrelloLeadTime
       response[:queue_time][:overall] = list.total_queue_time
       response[:cycle_time][:overall] = list.total_cycle_time
       response[:age][:overall] = list.total_age
+
+      breakdown_by_labels = list.breakdown_by_labels(Config.finance_type_labels)
+      response[:lead_time][:finance_types] = breakdown_by_labels[:total][:lead_time]
+      response[:queue_time][:finance_types] = breakdown_by_labels[:total][:queue_time]
+      response[:cycle_time][:finance_types] = breakdown_by_labels[:total][:cycle_time]
+      response[:age][:finance_types] = breakdown_by_labels[:total][:age]
       response
     end
 
@@ -42,6 +48,12 @@ module TrelloLeadTime
       response[:queue_time][:overall] = list.average_queue_time
       response[:cycle_time][:overall] = list.average_cycle_time
       response[:age][:overall] = list.average_age
+
+      breakdown_by_labels = list.breakdown_by_labels(Config.finance_type_labels)
+      response[:lead_time][:finance_types] = breakdown_by_labels[:average][:lead_time]
+      response[:queue_time][:finance_types] = breakdown_by_labels[:average][:queue_time]
+      response[:cycle_time][:finance_types] = breakdown_by_labels[:average][:cycle_time]
+      response[:age][:finance_types] = breakdown_by_labels[:average][:age]
       response
     end
 

data/lib/trello_lead_time/card.rb

@@ -48,7 +48,16 @@ module TrelloLeadTime
     end
 
     def hash_tags
-      comments.select { |comment| comment =~ /^#(.+)$/ } || []
+      comments_with_tags = comments.select { |comment| comment =~ /^#(.+)$/ } || []
+      tags = []
+      comments_with_tags.each do |comment|
+        tags.concat(comment.split.find_all { |word| /^#.+/.match(word) })
+      end
+      tags
+    end
+
+    def labels
+      @_labels ||= @trello_card.labels.map { |label| OpenStruct.new({name: label.name, color: label.color}) }
     end
 
     private
@@ -57,10 +66,6 @@ module TrelloLeadTime
       lists.inject(0) { |sum, list_name| sum + @timeline.seconds_in_list(list_name) }
     end
 
-    def labels
-      @_labels ||= @trello_card.labels
-    end
-
     def comments
       @_comments ||= @trello_card.actions(filter:"commentCard").map { |comment| comment.data['text'] }
     end

data/lib/trello_lead_time/config.rb

@@ -5,7 +5,7 @@ module TrelloLeadTime
   module Config
     extend self
 
-    attr_accessor :organization_name, :queue_time_lists, :cycle_time_lists, :list_name_matcher_for_done
+    attr_accessor :organization_name, :queue_time_lists, :cycle_time_lists, :finance_type_labels, :list_name_matcher_for_done
 
     def configure
       reset!
@@ -29,6 +29,11 @@ module TrelloLeadTime
       @cycle_time_lists
     end
 
+    def finance_type_labels
+      @finance_type_labels = %w{CapEx OpEx} if !@finance_type_labels
+      @finance_type_labels
+    end
+
     def list_name_matcher_for_done
       @list_name_matcher_for_done = /^Done/i if !@list_name_matcher_for_done
       @list_name_matcher_for_done

data/lib/trello_lead_time/list.rb

@@ -52,6 +52,34 @@ module TrelloLeadTime
       average(times)
     end
 
+    def breakdown_by_labels(labels)
+      times = default_breakdown_by_labels(labels)
+
+      done_or_closed_cards.each do |card|
+        matches = labels & card.labels.map(&:name)
+        matches.each do |label|
+          times[:total][:lead_time][label] += card.lead_time
+          times[:total][:queue_time][label] += card.queue_time
+          times[:total][:cycle_time][label] += card.cycle_time
+          times[:total][:age][label] += card.age_in_seconds
+
+          times[:average][:lead_time][label].push card.lead_time
+          times[:average][:queue_time][label].push card.queue_time
+          times[:average][:cycle_time][label].push card.cycle_time
+          times[:average][:age][label].push card.age_in_seconds
+        end
+      end
+
+      labels.each do |label|
+        times[:average][:lead_time][label] = average(times[:average][:lead_time][label])
+        times[:average][:queue_time][label] = average(times[:average][:queue_time][label])
+        times[:average][:cycle_time][label] = average(times[:average][:cycle_time][label])
+        times[:average][:age][label] = average(times[:average][:age][label])
+      end
+
+      times
+    end
+
     private
 
     def done_or_closed_cards
@@ -73,5 +101,25 @@ module TrelloLeadTime
       end
       avg
     end
+
+    def default_breakdown_by_labels(labels)
+      times = {
+        total: { lead_time: {}, queue_time: {}, cycle_time: {}, age: {} },
+        average: { lead_time: {}, queue_time: {}, cycle_time: {}, age: {} }
+      }
+      labels.each do |label|
+        times[:total][:lead_time][label]  = 0
+        times[:total][:queue_time][label] = 0
+        times[:total][:cycle_time][label] = 0
+        times[:total][:age][label]        = 0
+
+        times[:average][:lead_time][label]  = []
+        times[:average][:queue_time][label] = []
+        times[:average][:cycle_time][label] = []
+        times[:average][:age][label]        = []
+      end
+      times
+    end
+
   end
 end

data/lib/trello_lead_time/version.rb

@@ -1,3 +1,3 @@
 module TrelloLeadTime
-  VERSION = '0.1.2'
+  VERSION = '0.1.3'
 end

data/sample.rb

@@ -7,6 +7,7 @@ board_url                  = 'https://trello.com/b/nC8QJJoZ/trello-development'
 source_lists               = ['Live (4/8)', 'Live (3/17)', 'Live (3/3)', 'Live (2/11)', 'Live (1/14)']
 queue_time_lists           = ['Next Up']
 cycle_time_lists           = ['In Progress', 'Testing']
+finance_type_labels        = ['Feature', 'Bug']
 list_name_matcher_for_done = /^Live/
 
 TrelloLeadTime.configure do |cfg|
@@ -14,6 +15,7 @@ TrelloLeadTime.configure do |cfg|
   cfg.set_trello_key_and_token(developer_public_key, member_token)
   cfg.queue_time_lists = queue_time_lists
   cfg.cycle_time_lists = cycle_time_lists
+  cfg.finance_type_labels = finance_type_labels
   cfg.list_name_matcher_for_done = list_name_matcher_for_done
 end
 
@@ -28,7 +30,7 @@ source_lists.each do |source_list|
   totals   = board.totals(source_list)
   averages = board.averages(source_list)
 
-  puts "Using cards in list: #{source_list}"
+  puts "Overall metrics for: #{source_list}"
   puts "\tAverage Card Age:   #{TrelloLeadTime::TimeHumanizer.humanize_seconds(averages[:age][:overall])}"
   puts "\tAverage Lead Time:  #{TrelloLeadTime::TimeHumanizer.humanize_seconds(averages[:lead_time][:overall])}"
   puts "\tAverage Queue Time: #{TrelloLeadTime::TimeHumanizer.humanize_seconds(averages[:queue_time][:overall])}"
@@ -39,4 +41,9 @@ source_lists.each do |source_list|
   puts "\tTotal Queue Time:   #{TrelloLeadTime::TimeHumanizer.humanize_seconds(totals[:queue_time][:overall])}"
   puts "\tTotal Cycle Time:   #{TrelloLeadTime::TimeHumanizer.humanize_seconds(totals[:lead_time][:overall])}"
   puts ""
+  puts "\tFinance type breakdown (total lead time per label):"
+  totals[:lead_time][:finance_types].each do |label, value|
+    puts "\t\t#{label}: #{TrelloLeadTime::TimeHumanizer.humanize_seconds(value)}"
+  end
+  puts ""
 end

data/spec/fixtures/comments.json

@@ -41,7 +41,7 @@
                         "name": "some card name",
                         "id": "83716489"
                   },
-                  "text": "#myhashtag"
+                  "text": "#tag1 #tag2 #tag3"
             },
             "type": "commentCard",
             "date": "2014-05-12T14:14:59.223Z",

data/spec/fixtures/labels.1111.json

@@ -0,0 +1 @@
+[{"color":"green","name":"CapEx"}]

data/spec/fixtures/labels.2222.json

@@ -0,0 +1 @@
+[{"color":"purple","name":"OpEx"}]

data/spec/fixtures/labels.3333.json

@@ -0,0 +1 @@
+[{"color":"green","name":"CapEx"}]

data/spec/fixtures/labels.4444.json

@@ -0,0 +1 @@
+[{"color":"orange","name":"iOS"},{"color":"yellow","name":"Needs definition"}]

data/spec/lib/trello_lead_time/board_spec.rb

@@ -39,6 +39,15 @@ describe TrelloLeadTime::Board do
     }
   }
 
+  let(:labels_json) {
+    {
+      "1111" => File.read(File.expand_path("../../../fixtures/labels.1111.json", __FILE__)),
+      "2222" => File.read(File.expand_path("../../../fixtures/labels.2222.json", __FILE__)),
+      "3333" => File.read(File.expand_path("../../../fixtures/labels.3333.json", __FILE__)),
+      "4444" => File.read(File.expand_path("../../../fixtures/labels.4444.json", __FILE__))
+    }
+  }
+
   subject { TrelloLeadTime::Board.from_url(board_url) }
 
   describe ".totals" do
@@ -63,6 +72,26 @@ describe TrelloLeadTime::Board do
       stub_all_requests
       totals[:age][:overall].should == 4512370
     end
+
+    it "should have a lead time breakdown by finance type" do
+      stub_all_requests
+      totals[:lead_time][:finance_types].should == {"CapEx" => 1664755, "OpEx" => 474615}
+    end
+
+    it "should have a queue time breakdown by finance type" do
+      stub_all_requests
+      totals[:queue_time][:finance_types].should == {"CapEx" => 864055, "OpEx" => 177300}
+    end
+
+    it "should have a cycle time breakdown by finance type" do
+      stub_all_requests
+      totals[:cycle_time][:finance_types].should == {"CapEx" => 800700, "OpEx" => 297315}
+    end
+
+    it "should have an age breakdown by finance type" do
+      stub_all_requests
+      totals[:age][:finance_types].should == {"CapEx" => 1664755, "OpEx" => 474615}
+    end
   end
 
   describe ".averages" do
@@ -87,6 +116,26 @@ describe TrelloLeadTime::Board do
       stub_all_requests
       averages[:age][:overall].should == 1128093
     end
+
+    it "should have a lead time breakdown by finance type" do
+      stub_all_requests
+      averages[:lead_time][:finance_types].should == {"CapEx" => 832378, "OpEx" => 474615}
+    end
+
+    it "should have a queue time breakdown by finance type" do
+      stub_all_requests
+      averages[:queue_time][:finance_types].should == {"CapEx" => 432028, "OpEx" => 177300}
+    end
+
+    it "should have a cycle time breakdown by finance type" do
+      stub_all_requests
+      averages[:cycle_time][:finance_types].should == {"CapEx" => 400350, "OpEx" => 297315}
+    end
+
+    it "should have an age breakdown by finance type" do
+      stub_all_requests
+      averages[:age][:finance_types].should == {"CapEx" => 832378, "OpEx" => 474615}
+    end
   end
 
   private
@@ -122,6 +171,10 @@ describe TrelloLeadTime::Board do
       stub_request(:get, "https://api.trello.com/1/cards/#{card_id}/actions?filter=createCard,updateCard:idList,updateCard:closed&key=#{key}&token=#{token}").
         with(headers).
         to_return(stub_returns(actions_json[card_id]))
+
+      stub_request(:get, "https://api.trello.com/1/cards/#{card_id}/labels?key=#{key}&token=#{token}").
+        with(:headers => {'Accept'=>'*/*; q=0.5, application/xml', 'Accept-Encoding'=>'gzip, deflate', 'User-Agent'=>'Ruby'}).
+        to_return(:status => 200, :body => labels_json[card_id], :headers => {})
     end
   end
 

data/spec/lib/trello_lead_time/card_spec.rb

@@ -42,7 +42,15 @@ describe TrelloLeadTime::Card do
   describe ".hash_tags" do
     it "should have a hash tag" do
       do_stubs(key, token)
-      expect(subject.hash_tags).to eq(['#myhashtag'])
+      expect(subject.hash_tags).to eq(%w{#tag1 #tag2 #tag3})
+    end
+  end
+
+  describe ".labels" do
+    it "should have labels" do
+      do_stubs(key, token)
+      expect(subject.labels.collect(&:name)).to eq(%w{iOS API})
+
     end
   end
 

data/spec/lib/trello_lead_time/config_spec.rb

@@ -7,15 +7,17 @@ describe TrelloLeadTime::Config do
       organization_name: TrelloLeadTime::Config.organization_name,
       queue_time_lists: TrelloLeadTime::Config.queue_time_lists,
       cycle_time_lists: TrelloLeadTime::Config.cycle_time_lists,
+      finance_type_labels: TrelloLeadTime::Config.finance_type_labels,
       list_name_matcher_for_done: TrelloLeadTime::Config.list_name_matcher_for_done
     }
   end
 
   after(:all) do
     TrelloLeadTime.configure do |cfg|
+      cfg.organization_name          = @cfg[:organization_name]
       cfg.queue_time_lists           = @cfg[:queue_time_lists]
       cfg.cycle_time_lists           = @cfg[:cycle_time_lists]
-      cfg.organization_name          = @cfg[:organization_name]
+      cfg.finance_type_labels        = @cfg[:finance_type_labels]
       cfg.list_name_matcher_for_done = @cfg[:list_name_matcher_for_done]
     end
   end
@@ -48,6 +50,20 @@ describe TrelloLeadTime::Config do
     subject.cycle_time_lists.should include("my list")
   end
 
+  it "should have default list for finance type labels" do
+    TrelloLeadTime.configure do |cfg|
+      cfg.finance_type_labels = nil
+    end
+    expect(subject.finance_type_labels).to eq(%w{CapEx OpEx})
+  end
+
+  it "should set finance type labels" do
+    TrelloLeadTime.configure do |cfg|
+      cfg.finance_type_labels = ["my label"]
+    end
+    subject.finance_type_labels.should include("my label")
+  end
+
   it "should have default done matcher" do
     TrelloLeadTime.configure do |cfg|
       cfg.list_name_matcher_for_done = nil

data/spec/lib/trello_lead_time/list_spec.rb

@@ -27,6 +27,19 @@ describe TrelloLeadTime::List do
     }
   }
 
+  let(:labels_json) {
+    {
+      "1111" => File.read(File.expand_path("../../../fixtures/labels.1111.json", __FILE__)),
+      "2222" => File.read(File.expand_path("../../../fixtures/labels.2222.json", __FILE__)),
+      "3333" => File.read(File.expand_path("../../../fixtures/labels.3333.json", __FILE__)),
+      "4444" => File.read(File.expand_path("../../../fixtures/labels.4444.json", __FILE__))
+    }
+  }
+
+  let(:labels) {
+    %w{CapEx OpEx}
+  }
+
   subject { TrelloLeadTime::List.from_trello_list(list) }
 
   it "should have an average age" do
@@ -69,6 +82,54 @@ describe TrelloLeadTime::List do
     subject.total_cycle_time.should == 1815015
   end
 
+  it "should have total lead times by label" do
+    stub_all_requests
+    times = subject.breakdown_by_labels(labels)
+    expect(times[:total][:lead_time]).to eq({"CapEx" => 1664755, "OpEx" => 474615})
+  end
+
+  it "should have total queue times by label" do
+    stub_all_requests
+    times = subject.breakdown_by_labels(labels)
+    expect(times[:total][:queue_time]).to eq({"CapEx" => 864055, "OpEx" => 177300})
+  end
+
+  it "should have total cycle times by label" do
+    stub_all_requests
+    times = subject.breakdown_by_labels(labels)
+    expect(times[:total][:cycle_time]).to eq({"CapEx" => 800700, "OpEx" => 297315})
+  end
+
+  it "should have total ages by label" do
+    stub_all_requests
+    times = subject.breakdown_by_labels(labels)
+    expect(times[:total][:age]).to eq({"CapEx" => 1664755, "OpEx" => 474615})
+  end
+
+  it "should have average lead times by label" do
+    stub_all_requests
+    times = subject.breakdown_by_labels(labels)
+    expect(times[:average][:lead_time]).to eq({"CapEx" => 832378, "OpEx" => 474615})
+  end
+
+  it "should have average queue times by label" do
+    stub_all_requests
+    times = subject.breakdown_by_labels(labels)
+    expect(times[:average][:queue_time]).to eq({"CapEx" => 432028, "OpEx" => 177300})
+  end
+
+  it "should have average cycle times by label" do
+    stub_all_requests
+    times = subject.breakdown_by_labels(labels)
+    expect(times[:average][:cycle_time]).to eq({"CapEx" => 400350, "OpEx" => 297315})
+  end
+
+  it "should have average ages by label" do
+    stub_all_requests
+    times = subject.breakdown_by_labels(labels)
+    expect(times[:average][:age]).to eq({"CapEx" => 832378, "OpEx" => 474615})
+  end
+
   private
 
   def stub_all_requests
@@ -87,7 +148,13 @@ describe TrelloLeadTime::List do
       stub_request(:get, "https://api.trello.com/1/cards/#{card_id}/actions?filter=createCard,updateCard:idList,updateCard:closed&key=#{key}&token=#{token}").
         with(headers).
         to_return(stub_returns(actions_json[card_id]))
+
+      stub_request(:get, "https://api.trello.com/1/cards/#{card_id}/labels?key=#{key}&token=#{token}").
+        with(:headers => {'Accept'=>'*/*; q=0.5, application/xml', 'Accept-Encoding'=>'gzip, deflate', 'User-Agent'=>'Ruby'}).
+        to_return(:status => 200, :body => labels_json[card_id], :headers => {})
+
     end
+
   end
 
   def headers

data/spec/spec_helper.rb

@@ -8,6 +8,7 @@ RSpec.configure do |config|
       cfg.set_trello_key_and_token('key', 'token')
       cfg.queue_time_lists = ['Product Backlog']
       cfg.cycle_time_lists = ['Development', 'Acceptance']
+      cfg.finance_type_labels = ['CapEx', 'OpEx']
       cfg.list_name_matcher_for_done = /^Done/i
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: trello_lead_time
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Scott Baldwin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-14 00:00:00.000000000 Z
+date: 2014-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -87,6 +87,10 @@ files:
 - spec/fixtures/card.json
 - spec/fixtures/cards.json
 - spec/fixtures/comments.json
+- spec/fixtures/labels.1111.json
+- spec/fixtures/labels.2222.json
+- spec/fixtures/labels.3333.json
+- spec/fixtures/labels.4444.json
 - spec/fixtures/labels.json
 - spec/fixtures/lists.json
 - spec/fixtures/organization.json
@@ -132,6 +136,10 @@ test_files:
 - spec/fixtures/card.json
 - spec/fixtures/cards.json
 - spec/fixtures/comments.json
+- spec/fixtures/labels.1111.json
+- spec/fixtures/labels.2222.json
+- spec/fixtures/labels.3333.json
+- spec/fixtures/labels.4444.json
 - spec/fixtures/labels.json
 - spec/fixtures/lists.json
 - spec/fixtures/organization.json