@@ -49,6 +49,8 @@ module Burndown if !self.active? self.update_attributes(:closed_at => Time.now) return true + else + self.update_attributes(:closed_at => nil) if !self.closed_at.nil? end results = Lighthouse.get_milestone_tickets(self.name, self.project.remote_id, self.project.token.account, self.project.token.token) lib/burndown/milestone.rb @@ -122,7 +122,16 @@ class MiilestoneTest < Test::Unit::TestCase @existing.closed_at.should_not be_nil end - it "reopenes the milestone if there are open tickets" + it "reopenes the milestone if there are open tickets" do + closed = Milestone.generate(:remote_id => 42, :tickets_count => 20, :open_tickets_count => 0, :name => "ClosedMilestone", :due_on => DateTime.new(2006, 06, 06), :project_id => @activated_project.id, :closed_at => DateTime.new(2006, 07, 07)) + closed.should_not be_active + closed.closed_at.should_not be_nil + + closed.sync_with_lighthouse + + closed.should be_active + closed.closed_at.should be_nil + end end end \ No newline at end of file test/unit/miilestone_test.rb d3ea1230bb9f0cc86b58b0a0f37840a58bb9d744 Kyle Neath kneath@gmail.com http://github.com/kneath/lighthouse_burndown/commit/dacd81975484fc1393cd08e3281c2d650b99bd64 dacd81975484fc1393cd08e3281c2d650b99bd64 2009-05-02T16:51:54-07:00 2009-05-02T16:51:54-07:00 Close and reopen milestones based on open tickets b2bfba57c3ab2c0b9100a541043f8cb8019b886e Kyle Neath kneath@gmail.com