tracco 0.0.14 → 0.0.15

data/spec/tracking/effort_tracking_spec.rb

@@ -1,114 +1,116 @@
 require 'spec_helper'
 
-module Tracking
-  describe EffortTracking do
+module Tracco
+  module Tracking
+    describe EffortTracking do
 
-    describe "#effort" do
+      describe "#effort" do
 
-      %w{pietrodibello michelepangrazzi alessandrodescovi michelevincenzi}.each do |username|
-        let(username.to_sym) { Member.new(username: username) }
-      end
-
-      before(:each) do
-        Trello::Member.stub(:find).and_return(Member.new(username: "any"))
-      end
-
-      it "is nil when the notification does not contain an estimate" do
-        with(unrecognized_notification) { |tracking| tracking.effort.should be_nil }
-      end
+        %w{pietrodibello michelepangrazzi alessandrodescovi michelevincenzi}.each do |username|
+          let(username.to_sym) { Member.new(username: username) }
+        end
 
-      it "does not parse effort in minutes (e.g. +30m)" do
-        with_message("@trackinguser +30m") { |tracking| tracking.effort.should be_nil }
-      end
+        before(:each) do
+          Trello::Member.stub(:find).and_return(Member.new(username: "any"))
+        end
 
-      it "is the hour-based effort when the notification contains an effort in hours" do
-        Trello::Member.should_receive(:find).with("michelepangrazzi").and_return(michelepangrazzi)
+        it "is nil when the notification does not contain an estimate" do
+          with(unrecognized_notification) { |tracking| tracking.effort.should be_nil }
+        end
 
-        raw_data = create_notification(data: { 'text' => "@trackinguser +2h" },
-                                       date: "2012-10-28T21:06:14.801Z",
-                                       member_creator: stub(username: "michelepangrazzi"))
+        it "does not parse effort in minutes (e.g. +30m)" do
+          with_message("@trackinguser +30m") { |tracking| tracking.effort.should be_nil }
+        end
 
-        Tracking::Factory.build_from(raw_data).effort.should == Effort.new(amount: 2.0, date: Date.parse('2012-10-28'), members: [michelepangrazzi])
-      end
+        it "is the hour-based effort when the notification contains an effort in hours" do
+          Trello::Member.should_receive(:find).with("michelepangrazzi").and_return(michelepangrazzi)
 
-      it "converts the effort in hours when the notification contains an effort in days" do
-        with_message("@trackinguser +1.5d") { |t| t.effort.amount.should == 8+4 }
-        with_message("@trackinguser +1.5g") { |t| t.effort.amount.should == 8+4 }
-      end
+          raw_data = create_notification(data: { 'text' => "@trackinguser +2h" },
+                                         date: "2012-10-28T21:06:14.801Z",
+                                         member_creator: stub(username: "michelepangrazzi"))
 
-      it "converts the effort in hours when the notification contains an effort in pomodori" do
-        with_message("@trackinguser +10p") { |t| t.effort.amount.should == 5}
-      end
+          Tracking::Factory.build_from(raw_data).effort.should == Effort.new(amount: 2.0, date: Date.parse('2012-10-28'), members: [michelepangrazzi])
+        end
 
-      it "fetch the effort from a complex effort message" do
-        with_message "@trackinguser ho speso +2h e spero che stavolta possiamo rilasciarla" do |tracking|
-          tracking.effort.amount.should == 2.0
+        it "converts the effort in hours when the notification contains an effort in days" do
+          with_message("@trackinguser +1.5d") { |t| t.effort.amount.should == 8+4 }
+          with_message("@trackinguser +1.5g") { |t| t.effort.amount.should == 8+4 }
         end
-      end
 
-      it "fetch the effort even when beween square brackets" do
-        with_message "@trackinguser [+0.5h]" do |tracking|
-          tracking.effort.amount.should == 0.5
+        it "converts the effort in hours when the notification contains an effort in pomodori" do
+          with_message("@trackinguser +10p") { |t| t.effort.amount.should == 5}
         end
-      end
 
-      it "computes the effort considering all the mentioned team mates in the message" do
-        with_message "@trackinguser +2h assieme a @michelepangrazzi e @alessandrodescovi" do |tracking|
-          tracking.effort.amount.should == 2.0 * 3
+        it "fetch the effort from a complex effort message" do
+          with_message "@trackinguser ho speso +2h e spero che stavolta possiamo rilasciarla" do |tracking|
+            tracking.effort.amount.should == 2.0
+          end
         end
-      end
 
-      it "tracks all the team mates which spent that effort on the card" do
-        %w{pietrodibello michelepangrazzi alessandrodescovi}.each do |username|
-          Trello::Member.should_receive(:find).with(username).and_return(self.send(username))
+        it "fetch the effort even when beween square brackets" do
+          with_message "@trackinguser [+0.5h]" do |tracking|
+            tracking.effort.amount.should == 0.5
+          end
         end
 
-        notification = create_notification(data: { 'text' => "@trackinguser +2h assieme a @michelepangrazzi e @alessandrodescovi" },
-                                           member_creator: stub(username: "pietrodibello"))
-        with notification do |tracking|
-          tracking.effort.members.should == [michelepangrazzi, alessandrodescovi, pietrodibello]
+        it "computes the effort considering all the mentioned team mates in the message" do
+          with_message "@trackinguser +2h assieme a @michelepangrazzi e @alessandrodescovi" do |tracking|
+            tracking.effort.amount.should == 2.0 * 3
+          end
         end
-      end
 
-      it "tracks the effort only on the team members listed between round brackets" do
-        %w{michelevincenzi alessandrodescovi}.each do |username|
-          Trello::Member.should_receive(:find).with(username).and_return(self.send(username))
+        it "tracks all the team mates which spent that effort on the card" do
+          %w{pietrodibello michelepangrazzi alessandrodescovi}.each do |username|
+            Trello::Member.should_receive(:find).with(username).and_return(self.send(username))
+          end
+
+          notification = create_notification(data: { 'text' => "@trackinguser +2h assieme a @michelepangrazzi e @alessandrodescovi" },
+                                             member_creator: stub(username: "pietrodibello"))
+          with notification do |tracking|
+            tracking.effort.members.should == [michelepangrazzi, alessandrodescovi, pietrodibello]
+          end
         end
 
-        notification = create_notification(data: { 'text' => "@trackinguser +3p (@alessandrodescovi @michelevincenzi)" },
-                                           member_creator: stub(username: "pietrodibello"))
+        it "tracks the effort only on the team members listed between round brackets" do
+          %w{michelevincenzi alessandrodescovi}.each do |username|
+            Trello::Member.should_receive(:find).with(username).and_return(self.send(username))
+          end
+
+          notification = create_notification(data: { 'text' => "@trackinguser +3p (@alessandrodescovi @michelevincenzi)" },
+                                             member_creator: stub(username: "pietrodibello"))
 
-        with notification do |tracking|
-          tracking.effort.members.should == [alessandrodescovi, michelevincenzi]
-          tracking.effort.amount.should == 1.5 * 2
+          with notification do |tracking|
+            tracking.effort.members.should == [alessandrodescovi, michelevincenzi]
+            tracking.effort.amount.should == 1.5 * 2
+          end
         end
-      end
 
-      it "tracks the effort with the date given in the notification text, not the actual notification date" do
-        raw_data = create_notification(data: { 'text' => "@trackinguser 22.11.2012 +6p" }, date: "2012-09-19T12:46:13.713Z")
+        it "tracks the effort with the date given in the notification text, not the actual notification date" do
+          raw_data = create_notification(data: { 'text' => "@trackinguser 22.11.2012 +6p" }, date: "2012-09-19T12:46:13.713Z")
 
-        tracking = Tracking::Factory.build_from(raw_data)
+          tracking = Tracking::Factory.build_from(raw_data)
 
-        tracking.effort.date.should == Date.parse('2012-11-22')
-      end
+          tracking.effort.date.should == Date.parse('2012-11-22')
+        end
 
-      it "tracks the effort to yesterday when the keyword 'yesterday' is present before the effort amount" do
-        raw_data = create_notification(data: { 'text' => "@trackinguser yesterday +6p" }, date: "2012-09-19T12:46:13.713Z")
+        it "tracks the effort to yesterday when the keyword 'yesterday' is present before the effort amount" do
+          raw_data = create_notification(data: { 'text' => "@trackinguser yesterday +6p" }, date: "2012-09-19T12:46:13.713Z")
 
-        tracking = Tracking::Factory.build_from(raw_data)
+          tracking = Tracking::Factory.build_from(raw_data)
 
-        tracking.effort.date.should == Date.parse('2012-09-18')
-      end
+          tracking.effort.date.should == Date.parse('2012-09-18')
+        end
 
-      it "tracks the effort to yesterday when the keyword 'yesterday' is present before the effort amount" do
-        raw_data = create_notification(data: { 'text' => "@trackinguser +6p yesterday" }, date: "2012-09-19T12:46:13.713Z")
+        it "tracks the effort to yesterday when the keyword 'yesterday' is present before the effort amount" do
+          raw_data = create_notification(data: { 'text' => "@trackinguser +6p yesterday" }, date: "2012-09-19T12:46:13.713Z")
 
-        tracking = Tracking::Factory.build_from(raw_data)
+          tracking = Tracking::Factory.build_from(raw_data)
+
+          tracking.effort.date.should == Date.parse('2012-09-18')
+        end
 
-        tracking.effort.date.should == Date.parse('2012-09-18')
       end
 
     end
-
   end
 end

data/spec/tracking/estimate_tracking_spec.rb

@@ -1,44 +1,46 @@
 require 'spec_helper'
 
-module Tracking
-  describe EstimateTracking do
+module Tracco
+  module Tracking
+    describe EstimateTracking do
 
-    describe "#estimate" do
+      describe "#estimate" do
 
-      it "is nil when the notification does not contain an estimate" do
-        Tracking::Factory.build_from(unrecognized_notification).estimate.should be_nil
-      end
+        it "is nil when the notification does not contain an estimate" do
+          Tracking::Factory.build_from(unrecognized_notification).estimate.should be_nil
+        end
 
-      it "is the hour-based estimate when the notification contains an estimate in hours" do
-        raw_data = create_notification(data: { 'text' => "@trackinguser [2h]" }, date: "2012-10-28T21:06:14.801Z")
+        it "is the hour-based estimate when the notification contains an estimate in hours" do
+          raw_data = create_notification(data: { 'text' => "@trackinguser [2h]" }, date: "2012-10-28T21:06:14.801Z")
 
-        Tracking::Factory.build_from(raw_data).estimate.should == Estimate.new(amount: 2.0, date: Date.parse('2012-10-28'))
-      end
+          Tracking::Factory.build_from(raw_data).estimate.should == Estimate.new(amount: 2.0, date: Date.parse('2012-10-28'))
+        end
 
-      it "converts the estimate in hours when the notification contains an estimate in days" do
-        Tracking::Factory.build_from(create_notification(data: { 'text' => "@trackinguser [1.5d]" })).estimate.amount.should == 8+4
-        Tracking::Factory.build_from(create_notification(data: { 'text' => "@trackinguser [1.5g]" })).estimate.amount.should == 8+4
-      end
+        it "converts the estimate in hours when the notification contains an estimate in days" do
+          Tracking::Factory.build_from(create_notification(data: { 'text' => "@trackinguser [1.5d]" })).estimate.amount.should == 8+4
+          Tracking::Factory.build_from(create_notification(data: { 'text' => "@trackinguser [1.5g]" })).estimate.amount.should == 8+4
+        end
 
-      it "converts the estimate in hours when the notification contains an estimate in pomodori" do
-        raw_data = create_notification(data: { 'text' => "@trackinguser [10p]" })
-        Tracking::Factory.build_from(raw_data).estimate.amount.should == 5
-      end
+        it "converts the estimate in hours when the notification contains an estimate in pomodori" do
+          raw_data = create_notification(data: { 'text' => "@trackinguser [10p]" })
+          Tracking::Factory.build_from(raw_data).estimate.amount.should == 5
+        end
 
-      it "fetch the estimate from a complex estimate message" do
-        raw_data = create_notification(data: { 'text' => "@maxmazza Dobbiamo ancora lavorarci.\n@trackinguser ristimo ancora [3h] per il fix" })
-        Tracking::Factory.build_from(raw_data).estimate.amount.should == 3.0
-      end
+        it "fetch the estimate from a complex estimate message" do
+          raw_data = create_notification(data: { 'text' => "@maxmazza Dobbiamo ancora lavorarci.\n@trackinguser ristimo ancora [3h] per il fix" })
+          Tracking::Factory.build_from(raw_data).estimate.amount.should == 3.0
+        end
 
-      it "tracks the effort with the date given in the notification text, not the actual notification date" do
-        raw_data = create_notification( data: { 'text' => "@trackinguser 22.11.2012 [6p]" }, date: "2012-09-19T12:46:13.713Z")
+        it "tracks the effort with the date given in the notification text, not the actual notification date" do
+          raw_data = create_notification( data: { 'text' => "@trackinguser 22.11.2012 [6p]" }, date: "2012-09-19T12:46:13.713Z")
 
-        tracking = Tracking::Factory.build_from(raw_data)
+          tracking = Tracking::Factory.build_from(raw_data)
 
-        tracking.estimate.date.should == Date.parse('2012-11-22')
-      end
+          tracking.estimate.date.should == Date.parse('2012-11-22')
+        end
 
+      end
     end
-  end
 
+  end
 end

data/spec/tracking/factory_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+module Tracco
+  module Tracking
+    describe Factory do
+
+      context "unknown tracking format" do
+        it "builds an invalid tracking instance" do
+          Tracking::Factory.build_from(unrecognized_notification).class.should == Tracking::InvalidTracking
+
+          with_message("@trackinguser +30m") { |tracking| tracking.class.should == Tracking::InvalidTracking }
+        end
+      end
+
+      TIME_MEASUREMENTS.each_key do |time_measurement|
+
+        context "estimate tracking notification in #{time_measurement}" do
+          it "builds an estimate tracking instance" do
+            Tracking::Factory.build_from(create_estimate(time_measurement)).class.should == Tracking::EstimateTracking
+          end
+        end
+
+        context "effort tracking notification in #{time_measurement}" do
+          it "builds an effort tracking instance" do
+            Tracking::Factory.build_from(create_effort(time_measurement)).class.should == Tracking::EffortTracking
+          end
+        end
+
+      end
+
+      context "card done tracking notification" do
+        it "builds a card done tracking instance" do
+          with_message("@trackinguser DONE") { |tracking| tracking.class.should == Tracking::CardDoneTracking }
+        end
+      end
+
+    end
+  end
+end

data/spec/trello_tracker_spec.rb

@@ -1,26 +1,28 @@
 require 'spec_helper'
 
-describe TrelloTracker do
+module Tracco
+  describe TrelloTracker do
 
-  before(:each) do
-    @original = ENV["tracker_username"]
-    ENV["tracker_username"] = "my_tracker"
-  end
-  
-  after(:each) do
-    ENV["tracker_username"] = @original
-  end
-  
-  it "force the trello tracker username in the constructor" do    
-    tracker = TrelloTracker.new(tracker_username: "any_other_tracker")
+    before(:each) do
+      @original = ENV["tracker_username"]
+      ENV["tracker_username"] = "my_tracker"
+    end
 
-    tracker.tracker_username.should == "any_other_tracker"
-  end
+    after(:each) do
+      ENV["tracker_username"] = @original
+    end
 
-  it "takes the tracker username from the ENV var tracker_username" do
-    tracker = TrelloTracker.new
+    it "force the trello tracker username in the constructor" do
+      tracker = TrelloTracker.new(tracker_username: "any_other_tracker")
 
-    tracker.tracker_username.should == "my_tracker"
-  end
+      tracker.tracker_username.should == "any_other_tracker"
+    end
 
+    it "takes the tracker username from the ENV var tracker_username" do
+      tracker = TrelloTracker.new
+
+      tracker.tracker_username.should == "my_tracker"
+    end
+
+  end
 end

data/tracco.gemspec

@@ -3,7 +3,7 @@ require File.expand_path('../lib/tracco/version', __FILE__)
 
 Gem::Specification.new do |gem|
   gem.name              = "tracco"
-  gem.version           = TrelloEffortTracker::VERSION
+  gem.version           = Tracco::VERSION
   gem.platform          = Gem::Platform::RUBY
 
   gem.description       = "Tracco is a Trello effort tracker: the purpose of Tracco is to extract and track estimates and actual efforts out of the cards on your Trello boards."

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tracco
 version: !ruby/object:Gem::Version
-  version: 0.0.14
+  version: 0.0.15
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-27 00:00:00.000000000 Z
+date: 2013-03-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-trello
@@ -262,12 +262,12 @@ files:
 - lib/tasks/rspec.rake
 - lib/tasks/tasks.rake
 - lib/tracco.rb
-- lib/tracco/effort.rb
-- lib/tracco/estimate.rb
 - lib/tracco/google_docs_exporter.rb
-- lib/tracco/member.rb
+- lib/tracco/models/effort.rb
+- lib/tracco/models/estimate.rb
+- lib/tracco/models/member.rb
+- lib/tracco/models/tracked_card.rb
 - lib/tracco/mongoid_helper.rb
-- lib/tracco/tracked_card.rb
 - lib/tracco/tracking/base.rb
 - lib/tracco/tracking/card_done_tracking.rb
 - lib/tracco/tracking/effort_tracking.rb
@@ -282,23 +282,23 @@ files:
 - script/ci/run_build.sh
 - script/crontab.template
 - script/mate.sh
-- spec/effort_spec.rb
-- spec/estimate_spec.rb
 - spec/factories/effort_factory.rb
 - spec/factories/estimate_factory.rb
 - spec/factories/tracked_card_factory.rb
 - spec/integration/trello_authorization_spec.rb
 - spec/integration/trello_tracker_spec.rb
-- spec/member_spec.rb
+- spec/models/effort_spec.rb
+- spec/models/estimate_spec.rb
+- spec/models/member_spec.rb
+- spec/models/tracked_card_spec.rb
 - spec/patches/trello/card_spec.rb
 - spec/spec_helper.rb
 - spec/support/database_cleaner.rb
 - spec/support/spec_helper_methods.rb
-- spec/tracked_card_spec.rb
 - spec/tracking/card_done_tracking_spec.rb
 - spec/tracking/effort_tracking_spec.rb
 - spec/tracking/estimate_tracking_spec.rb
-- spec/tracking_factory_spec.rb
+- spec/tracking/factory_spec.rb
 - spec/trello_authorize_spec.rb
 - spec/trello_configuration_spec.rb
 - spec/trello_tracker_spec.rb
@@ -318,7 +318,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1097737159014714198
+      hash: -2610639466740555589
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:

data/lib/tracco/effort.rb

@@ -1,37 +0,0 @@
-class Effort
-  include Mongoid::Document
-  include Mongoid::Timestamps
-  include Mongoid::MultiParameterAttributes
-
-  field :amount,  type: BigDecimal
-  field :date,    type: Date
-  field :tracking_notification_id
-  field :muted,   type: Boolean, default: false
-
-  embeds_many :members
-  embedded_in :tracked_card
-
-  default_scope where(muted: false).asc(:date)
-
-  validates_presence_of :amount, :date, :members
-
-  def amount_per_member
-    amount / members.size
-  end
-
-  def include?(member)
-    members.include?(member)
-  end
-
-  def ==(other)
-    return true if other.equal?(self)
-    return false unless other.kind_of?(self.class)
-
-    amount == other.amount && date == other.date && Set.new(members) == Set.new(other.members)
-  end
-
-  def to_s
-    "[#{date}] spent #{amount} hours by #{members.map(&:at_username).join(", ")}"
-  end
-
-end

data/lib/tracco/estimate.rb

@@ -1,26 +0,0 @@
-class Estimate
-  include Mongoid::Document
-  include Mongoid::Timestamps
-  include Mongoid::MultiParameterAttributes
-
-  field :amount,  type: BigDecimal
-  field :date,    type: Date
-  field :tracking_notification_id
-
-  embedded_in :tracked_card
-
-  default_scope asc(:date)
-
-  validates_presence_of :amount, :date
-
-  def ==(other)
-    return true if other.equal?(self)
-    return false unless other.kind_of?(self.class)
-
-    amount == other.amount && date == other.date
-  end
-
-  def to_s
-    "[#{date}] estimated #{amount} hours"
-  end
-end

data/lib/tracco/member.rb

@@ -1,61 +0,0 @@
-class Member
-  include Mongoid::Document
-  include Mongoid::Timestamps
-
-  field :trello_id
-  field :username
-  field :full_name
-  field :avatar_id
-  field :bio
-  field :url
-
-  embedded_in :effort
-
-  validates_presence_of :username
-
-  def self.build_from(trello_member)
-    trello_member_id = trello_member.id
-    trello_member.attributes.delete(:id)
-    new(trello_member.attributes.merge(trello_id: trello_member_id))
-  end
-
-  def at_username
-    "@#{username}"
-  end
-
-  def avatar_url
-    trello_member.avatar_url(size: :small)
-  end
-
-  def effort_spent(from_date=nil)
-    cards = TrackedCard.with_effort_spent_by(username)
-    efforts = cards.map(&:efforts).compact.flatten
-    efforts = efforts.select {|e| e.date >= from_date} if from_date
-    efforts.select { |effort| effort.include?(self) }.inject(0) { |total, effort| total + effort.amount_per_member }
-  end
-  alias_method :effort_spent_since, :effort_spent
-
-  def ==(other)
-    return true if other.equal?(self)
-    return false unless other.kind_of?(self.class)
-
-    username == other.username
-  end
-
-  def eql?(other)
-    return false unless other.instance_of?(self.class)
-    username == other.username
-  end
-
-  def hash
-    username.hash
-  end
-
-  private
-
-  def trello_member
-    @trello_member ||= Trello::Member.new("id" => trello_id, "fullName" => full_name, "username" => username,
-                                           "avatarHash" => avatar_id, "bio" => bio, "url" => url)
-  end
-
-end