trollolo 0.0.5 → 0.0.6

data/spec/unit/burndown_chart_spec.rb

@@ -3,7 +3,6 @@ require_relative 'spec_helper'
 include GivenFilesystemSpecHelpers
 
 describe BurndownChart do
-
   before(:each) do
     @settings = dummy_settings
     @burndown_data = BurndownData.new(@settings)
@@ -58,94 +57,172 @@ describe BurndownChart do
           "tasks_extra" =>
           {
             "done" => 2
+          },
+          "story_points_unplanned" =>
+          {
+            "total" => 3,
+            "open" => 1
+          },
+          "tasks_unplanned" =>
+          {
+            "total" => 2,
+            "open" => 1
           }
         }
       ]
     end
 
-    it "creates first data entry" do
-      @burndown_data.story_points.open = 16
-      @burndown_data.story_points.done = 7
-      @burndown_data.tasks.open = 10
-      @burndown_data.tasks.done = 11
-      @burndown_data.date_time = DateTime.parse("2014-05-30")
+    it "returns sprint number" do
+      expect(@chart.sprint).to eq 1
+    end
+
+    describe "#add_data" do
+      it "creates first data entry" do
+        @burndown_data.story_points.open = 16
+        @burndown_data.story_points.done = 7
+        @burndown_data.tasks.open = 10
+        @burndown_data.tasks.done = 11
+        @burndown_data.date_time = DateTime.parse("2014-05-30")
 
-      @chart.add_data(@burndown_data)
+        @chart.add_data(@burndown_data)
 
-      expect( @chart.data["days"].first["story_points"] ).to eq(
-        {
+        expect( @chart.data["days"].first["story_points"] ).to eq(
+          {
+            "total" => 23,
+            "open" => 16
+          } )
+        expect( @chart.data["days"].first["tasks"] ).to eq(
+          {
+            "total" => 21,
+            "open" => 10
+          } )
+      end
+
+      it "doesn't overwrite first data entry" do
+        @burndown_data.story_points.open = 16
+        @burndown_data.story_points.done = 7
+        @burndown_data.tasks.open = 10
+        @burndown_data.tasks.done = 11
+        @burndown_data.date_time = DateTime.parse("2014-05-30")
+
+        @chart.add_data(@burndown_data)
+
+        @burndown_data.story_points.open = 15
+        @burndown_data.story_points.done = 8
+        @burndown_data.tasks.open = 9
+        @burndown_data.tasks.done = 12
+        @burndown_data.date_time = DateTime.parse("2014-05-30")
+
+        @chart.add_data(@burndown_data)
+
+        expect( @chart.data["days"].first["story_points"] ).to eq(
+          {
+            "total" => 23,
+            "open" => 16
+          } )
+        expect( @chart.data["days"].first["tasks"] ).to eq(
+          {
+            "total" => 21,
+            "open" => 10
+          } )
+      end
+
+      it "does overwrite data entries after first one" do
+        @burndown_data.story_points.open = 16
+        @burndown_data.story_points.done = 7
+        @burndown_data.tasks.open = 10
+        @burndown_data.tasks.done = 11
+        @burndown_data.date_time = DateTime.parse("2014-05-30")
+
+        @chart.add_data(@burndown_data)
+
+        @burndown_data.story_points.open = 16
+        @burndown_data.story_points.done = 7
+        @burndown_data.tasks.open = 10
+        @burndown_data.tasks.done = 11
+        @burndown_data.date_time = DateTime.parse("2014-05-31")
+
+        @chart.add_data(@burndown_data)
+
+        @burndown_data.story_points.open = 15
+        @burndown_data.story_points.done = 8
+        @burndown_data.tasks.open = 9
+        @burndown_data.tasks.done = 12
+        @burndown_data.date_time = DateTime.parse("2014-05-31")
+
+        @chart.add_data(@burndown_data)
+
+        expect( @chart.data["days"][1]["story_points"] ).to eq(
+          {
+            "total" => 23,
+            "open" => 15
+          } )
+        expect( @chart.data["days"][1]["tasks"] ).to eq(
+          {
+            "total" => 21,
+            "open" => 9
+          } )
+      end
+
+      it "adds data" do
+        @chart.data["days"] = @raw_data
+
+        @burndown_data.story_points.open = 16
+        @burndown_data.story_points.done = 7
+        @burndown_data.tasks.open = 10
+        @burndown_data.tasks.done = 11
+        @burndown_data.extra_story_points.open = 2
+        @burndown_data.extra_story_points.done = 3
+        @burndown_data.extra_tasks.open = 5
+        @burndown_data.extra_tasks.done = 2
+        @burndown_data.date_time = DateTime.parse("2014-05-30")
+
+        @chart.add_data(@burndown_data)
+
+        expect( @chart.data["days"].count ).to eq 3
+        expect( @chart.data["days"].last["date"] ).to eq ( "2014-05-30" )
+        expect( @chart.data["days"].last["story_points"] ).to eq ( {
           "total" => 23,
           "open" => 16
         } )
-      expect( @chart.data["days"].first["tasks"] ).to eq(
-        {
+        expect( @chart.data["days"].last["tasks"] ).to eq ( {
           "total" => 21,
           "open" => 10
         } )
-    end
-
-    it "returns sprint number" do
-      expect(@chart.sprint).to eq 1
-    end
-
-    it "adds data" do
-      @chart.data["days"] = @raw_data
-
-      @burndown_data.story_points.open = 16
-      @burndown_data.story_points.done = 7
-      @burndown_data.tasks.open = 10
-      @burndown_data.tasks.done = 11
-      @burndown_data.extra_story_points.open = 2
-      @burndown_data.extra_story_points.done = 3
-      @burndown_data.extra_tasks.open = 5
-      @burndown_data.extra_tasks.done = 2
-      @burndown_data.date_time = DateTime.parse("2014-05-30")
-
-      @chart.add_data(@burndown_data)
-
-      expect( @chart.data["days"].count ).to eq 3
-      expect( @chart.data["days"].last["date"] ).to eq ( "2014-05-30" )
-      expect( @chart.data["days"].last["story_points"] ).to eq ( {
-        "total" => 23,
-        "open" => 16
-      } )
-      expect( @chart.data["days"].last["tasks"] ).to eq ( {
-        "total" => 21,
-        "open" => 10
-      } )
-      expect( @chart.data["days"].last["story_points_extra"] ).to eq ( {
-        "done" => 3
-      } )
-      expect( @chart.data["days"].last["tasks_extra"] ).to eq ( {
-        "done" => 2
-      } )
-    end
+        expect( @chart.data["days"].last["story_points_extra"] ).to eq ( {
+          "done" => 3
+        } )
+        expect( @chart.data["days"].last["tasks_extra"] ).to eq ( {
+          "done" => 2
+        } )
+      end
 
-    it "replaces data of same day" do
-      @chart.data["days"] = @raw_data
+      it "replaces data of same day" do
+        @chart.data["days"] = @raw_data
 
-      @burndown_data.story_points.open = 16
-      @burndown_data.story_points.done = 7
-      @burndown_data.tasks.open = 10
-      @burndown_data.tasks.done = 11
-      @burndown_data.date_time = DateTime.parse("2014-05-30")
+        @burndown_data.story_points.open = 16
+        @burndown_data.story_points.done = 7
+        @burndown_data.tasks.open = 10
+        @burndown_data.tasks.done = 11
+        @burndown_data.date_time = DateTime.parse("2014-05-30")
 
-      @chart.add_data(@burndown_data)
+        @chart.add_data(@burndown_data)
 
-      expect( @chart.data["days"].count ).to eq 3
-      expect( @chart.data["days"].last["story_points"] ).to eq ( {
-        "total" => 23,
-        "open" => 16
-      } )
+        expect( @chart.data["days"].count ).to eq 3
+        expect( @chart.data["days"].last["story_points"] ).to eq ( {
+          "total" => 23,
+          "open" => 16
+        } )
 
-      @burndown_data.story_points.done = 8
-      @chart.add_data(@burndown_data)
+        @burndown_data.story_points.done = 8
+        @chart.add_data(@burndown_data)
 
-      expect( @chart.data["days"].count ).to eq 3
-      expect( @chart.data["days"].last["story_points"] ).to eq ( {
-        "total" => 24,
-        "open" => 16
-      } )
+        expect( @chart.data["days"].count ).to eq 3
+        expect( @chart.data["days"].last["story_points"] ).to eq ( {
+          "total" => 24,
+          "open" => 16
+        } )
+      end
     end
 
     describe "#read_data" do
@@ -235,7 +312,6 @@ EOT
         expect(File.read(write_path)).to eq expected_file_content
       end
     end
-
   end
 
   describe "commands" do
@@ -264,14 +340,36 @@ EOT
 
     describe "load_last_sprint" do
       let(:path) { given_directory_from_data("burndown_dir") }
-      it "loads the burndown form the 2nd sprint into data" do
+
+      it "loads the burndown from the 2nd sprint into data" do
         @chart.load_last_sprint(path)
-        expect(@chart.data).to eq({"meta"=>
-                                   {"board_id"=>"53186e8391ef8671265eba9d",
-                                    "sprint"=>2,
-                                    "total_days"=>9,
-                                    "weekend_lines"=>[3.5, 7.5]},
-                                   "days"=>[]})
+        expect(@chart.data).to eq(
+          { "meta" =>
+            { "board_id" => "53186e8391ef8671265eba9d",
+              "sprint" => 2,
+              "total_days" => 9,
+              "weekend_lines" => [3.5, 7.5]
+            },
+            "days" => [
+              { "date" => "2015-08-28",
+                "updated_at" => "2015-08-28T11:04:52+02:00",
+                "story_points" =>
+                  { "total"=>24.0,
+                    "open"=>24.0
+                  },
+                "tasks" =>
+                  { "total" => 43,
+                    "open" => 28
+                  },
+                "story_points_extra" =>
+                  { "done"=>2.0
+                  },
+                "tasks_extra" =>
+                  { "done" => 5
+                  }
+              }
+            ]
+          })
       end
 
       it "returns the path of the last sprint" do
@@ -334,7 +432,6 @@ EOT
   end
 
   describe "reads meta data from the board" do
-
     use_given_filesystem
 
     it "merges meta data from board if present" do
@@ -354,26 +451,21 @@ EOT
   end
 
   describe '.plot' do
-
     it 'sends joined parsed options to python script' do
       allow(described_class).to receive(:process_options).and_return(%w{ --test 1 --no-blah })
       allow(described_class).to receive(:plot_helper).and_return('mescript')
       expect(described_class).to receive(:system).with('python mescript 42 --test 1 --no-blah')
       described_class.plot(42, {foo: 1, bar: 2})
     end
-
   end
 
   describe '.plot_helper' do
-
     it 'expands path to burndown generator' do
       expect(described_class.plot_helper).to include('scripts/create_burndown.py')
     end
-
   end
 
   describe '.process_options' do
-
     it 'builds an array of switches for burndown chart based on input hash' do
       test_hash = { 'no-tasks' => true }
       expect(described_class.send(:process_options, test_hash)).to eq %w{ --no-tasks }
@@ -391,6 +483,5 @@ EOT
       }
       expect(described_class.send(:process_options, test_hash)).to eq ['--no-tasks', '--with-fast-lane', '--output fanagoro', '--verbose']
     end
-
   end
 end

data/spec/unit/burndown_data_spec.rb

@@ -3,7 +3,6 @@ require_relative 'spec_helper'
 include GivenFilesystemSpecHelpers
 
 describe BurndownData do
-  
   before(:each) do
     @burndown = BurndownData.new(dummy_settings)
     @burndown.board_id = "53186e8391ef8671265eba9d"
@@ -15,7 +14,7 @@ describe BurndownData do
       r = described_class.new
       r.open = 7
       r.done = 4
-      
+
       expect(r.total).to eq 11
     end
   end
@@ -25,7 +24,7 @@ describe BurndownData 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
@@ -33,53 +32,76 @@ describe BurndownData do
   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
+    context "with meta data on sprint card" 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 unplanned story points" do
+        expect( @burndown.unplanned_story_points.total ).to eq 3
+        expect( @burndown.unplanned_story_points.open ).to eq 1
+        expect( @burndown.unplanned_story_points.done ).to eq 2
+      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 unplanned tasks" do
+        expect( @burndown.unplanned_tasks.total ).to eq 2
+        expect( @burndown.unplanned_tasks.open ).to eq 1
+        expect( @burndown.unplanned_tasks.done ).to eq 1
+      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
 
-    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
+    context "without meta data on sprint card" do
+      before do
+        allow(Card).to receive(:parse_yaml_from_description).and_return(nil)
+        @burndown.fetch
+      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)
+      it "does not fail" do
+        expect(@burndown.meta).to be(nil)
+      end
     end
   end
 
   describe '#to_hash' do
-
     it 'converts to hash' do
       @burndown.story_points.open = 1
       @burndown.story_points.done = 2
@@ -89,6 +111,10 @@ describe BurndownData do
       @burndown.extra_story_points.done = 6
       @burndown.extra_tasks.open = 7
       @burndown.extra_tasks.done = 8
+      @burndown.unplanned_story_points.open = 1
+      @burndown.unplanned_story_points.done = 2
+      @burndown.unplanned_tasks.open = 1
+      @burndown.unplanned_tasks.done = 1
       @burndown.date_time = DateTime.parse('20150115')
 
       expected_hash = {
@@ -107,12 +133,18 @@ describe BurndownData do
         },
         'tasks_extra' => {
           'done' => 8
+        },
+        'unplanned_story_points' => {
+          'total' => 3,
+          'open' => 1
+        },
+        'unplanned_tasks' => {
+          'total' => 2,
+          'open' => 1
         }
       }
 
       expect(@burndown.to_hash).to eq(expected_hash)
     end
-    
   end
-
 end

data/spec/unit/card_spec.rb

@@ -76,4 +76,79 @@ EOT
       expect(meta["weekend_lines"]).to eq([1.5, 6.5])
     end
   end
+
+  describe "gets raw JSON" do
+    it "for cards" do
+      @settings = dummy_settings
+      full_board_mock
+
+      trello = TrelloWrapper.new(@settings)
+      board = trello.board("53186e8391ef8671265eba9d")
+
+      expected_json = <<EOT
+{
+  "id": "5319bf244cc53afd5afd991f",
+  "checkItemStates": [
+
+  ],
+  "closed": false,
+  "dateLastActivity": "2014-03-07T12:52:07.236Z",
+  "desc": "",
+  "descData": null,
+  "idBoard": "53186e8391ef8671265eba9d",
+  "idList": "53186e8391ef8671265eba9e",
+  "idMembersVoted": [
+
+  ],
+  "idShort": 3,
+  "idAttachmentCover": null,
+  "manualCoverAttachment": false,
+  "idLabels": [
+    "5463b41e74d650d56700f16a"
+  ],
+  "name": "Sprint 3",
+  "pos": 65535,
+  "shortLink": "GRsvY3vZ",
+  "badges": {
+    "votes": 0,
+    "viewingMemberVoted": false,
+    "subscribed": false,
+    "fogbugz": "",
+    "checkItems": 0,
+    "checkItemsChecked": 0,
+    "comments": 0,
+    "attachments": 0,
+    "description": false,
+    "due": null
+  },
+  "due": null,
+  "email": "trello@example.com",
+  "idChecklists": [
+
+  ],
+  "idMembers": [
+
+  ],
+  "labels": [
+    {
+      "id": "5463b41e74d650d56700f16a",
+      "idBoard": "53186e8391ef8671265eba9d",
+      "name": "Sticky",
+      "color": "blue",
+      "uses": 8
+    }
+  ],
+  "shortUrl": "https://trello.com/c/GRsvY3vZ",
+  "subscribed": false,
+  "url": "https://trello.com/c/GRsvY3vZ/3-sprint-3",
+  "checklists": [
+
+  ]
+}
+EOT
+
+      json = board.cards.first.as_json
+      expect(json).to eq(expected_json.chomp)
+    end
+  end
 end

data/spec/unit/cli_spec.rb

@@ -3,22 +3,13 @@ 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
@@ -35,4 +26,74 @@ describe Cli do
     @cli.backup
   end
 
+  it "gets lists" do
+    full_board_mock
+    @cli.options = {"board-id" => "53186e8391ef8671265eba9d"}
+    expected_output = <<EOT
+Sprint Backlog
+Doing
+Done Sprint 10
+Done Sprint 9
+Done Sprint 8
+Legend
+EOT
+    expect {
+      @cli.get_lists
+    }.to output(expected_output).to_stdout
+  end
+
+  it "gets cards" do
+    full_board_mock
+    @cli.options = {"board-id" => "53186e8391ef8671265eba9d"}
+    expected_output = <<EOT
+Sprint 3
+(3) P1: Fill Backlog column
+(5) P4: Read data from Trollolo
+(3) P5: Save read data as reference data
+Waterline
+(8) P6: Celebrate testing board
+(2) P2: Fill Doing column
+(1) Fix emergency
+Burndown chart
+Sprint 10
+(3) P3: Fill Done columns
+(2) Some unplanned work
+Burndown chart
+Sprint 9
+(2) P1: Explain purpose
+(2) P2: Create Scrum columns
+Burndown chart
+Sprint 8
+(1) P1: Create Trello Testing Board
+(5) P2: Add fancy background
+(1) P4: Add legend
+Purpose
+Background image
+EOT
+    expect {
+      @cli.get_cards
+    }.to output(expected_output).to_stdout
+  end
+
+  it "gets checklists" do
+    full_board_mock
+    @cli.options = {"board-id" => "53186e8391ef8671265eba9d"}
+    expected_output = <<EOT
+Tasks
+Tasks
+Tasks
+Tasks
+Tasks
+Feedback
+Tasks
+Tasks
+Tasks
+Tasks
+Tasks
+Tasks
+EOT
+    expect {
+      @cli.get_checklists
+    }.to output(expected_output).to_stdout
+  end
 end