trollolo 0.1.1 → 0.2.0

data/spec/integration/create_burndown_spec.rb

@@ -1,18 +1,18 @@
-require_relative "integration_spec_helper"
+require_relative 'integration_spec_helper'
 
 include GivenFilesystemSpecHelpers
 include CliTester
 
 def run_helper(working_dir, sprint_number, extra_args = [])
-  helper_dir = File.expand_path("../../../scripts", __FILE__)
-  args = ["run"]
-  args += ["-v", "#{helper_dir}:/trollolo/helper"]
-  args += ["-v", "#{working_dir}:/trollolo/data"]
-  args += ["-w", "/trollolo/data"]
-  args += ["trollolo-matplotlib"]
-  args += ["/trollolo/helper/create_burndown.py", sprint_number]
+  helper_dir = File.expand_path('../../../scripts', __FILE__)
+  args = ['run']
+  args += ['-v', "#{helper_dir}:/trollolo/helper"]
+  args += ['-v', "#{working_dir}:/trollolo/data"]
+  args += ['-w', '/trollolo/data']
+  args += ['trollolo-matplotlib']
+  args += ['/trollolo/helper/create_burndown.py', sprint_number]
   args += extra_args
-  run_command(cmd: "docker", args: args)
+  run_command(cmd: 'docker', args: args)
 end
 
 def compare_images_for_sprint(sprint_number, extra_args = [])
@@ -21,12 +21,12 @@ def compare_images_for_sprint(sprint_number, extra_args = [])
   end
 
   result = run_helper(@working_dir, sprint_number, extra_args)
-  expect(result).to exit_with_success("")
+  expect(result).to exit_with_success('')
   expect(File.join(@working_dir, "burndown-#{sprint_number}.png")).
     to be_same_image_as("create_burndown_helper/burndown-#{sprint_number}.png")
 end
 
-describe "create_burndown.py" do
+describe 'create_burndown.py' do
   use_given_filesystem(keep_files: true)
 
   before(:all) do
@@ -35,31 +35,31 @@ describe "create_burndown.py" do
     end
   end
 
-  it "creates burndown chart with varying number of total story points and tasks" do
-    compare_images_for_sprint("23")
+  it 'creates burndown chart with varying number of total story points and tasks' do
+    compare_images_for_sprint('23')
   end
 
-  it "creates burndown chart with done tasks at the beginning" do
-    compare_images_for_sprint("31")
+  it 'creates burndown chart with done tasks at the beginning' do
+    compare_images_for_sprint('31')
   end
 
-  it "creates burndown chart of unfinished sprint" do
-    compare_images_for_sprint("35")
+  it 'creates burndown chart of unfinished sprint' do
+    compare_images_for_sprint('35')
   end
 
-  it "creates burndown chart with fast lane and no tasks" do
-    compare_images_for_sprint("08", ["--no-tasks", "--with-fast-lane"])
+  it 'creates burndown chart with fast lane and no tasks' do
+    compare_images_for_sprint('08', ['--no-tasks', '--with-fast-lane'])
   end
 
-  it "creates perfect burndown chart" do
-    compare_images_for_sprint("42")
+  it 'creates perfect burndown chart' do
+    compare_images_for_sprint('42')
   end
 
-  it "creates burndown chart with unplanned cards" do
-    compare_images_for_sprint("56")
+  it 'creates burndown chart with unplanned cards' do
+    compare_images_for_sprint('56')
   end
 
-  it "creates burndown chart with unplanned tasks on day one" do
-    compare_images_for_sprint("01")
+  it 'creates burndown chart with unplanned tasks on day one' do
+    compare_images_for_sprint('01')
   end
 end

data/spec/integration/integration_spec_helper.rb

@@ -2,7 +2,7 @@ require_relative '../../lib/trollolo'
 require 'given_filesystem/spec_helpers'
 require 'cli_tester'
 
-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']

data/spec/integration/wrapper/credentials_input_wrapper

@@ -1,19 +1,15 @@
 #!/usr/bin/env ruby
 
-require "expect"
-require "pty"
+require 'expect'
+require 'pty'
 
 config_path = "/tmp/test-trollolorc-#{rand(100000)}"
 bin_path = File.expand_path('../../../../bin/trollolo', __FILE__)
 
-ENV["TROLLOLO_CONFIG_PATH"] = config_path
+ENV['TROLLOLO_CONFIG_PATH'] = config_path
 
-PTY.spawn("#{bin_path} #{ARGV.join(" ")}") do |trollolo_out, trollolo_in, pid|
-  if !trollolo_out.expect("key:", 2)
-    exit 1
-  end
-  trollolo_in.puts("xxx")
-  if !trollolo_out.expect("token:", 2)
-    exit 1
-  end
+PTY.spawn("#{bin_path} #{ARGV.join(' ')}") do |trollolo_out, trollolo_in, pid|
+  exit 1 unless trollolo_out.expect('key:', 2)
+  trollolo_in.puts('xxx')
+  exit 1 unless trollolo_out.expect('token:', 2)
 end

data/spec/integration/wrapper/empty_config_trollolo_wrapper

@@ -3,8 +3,8 @@
 config_path = "/tmp/test-trollolorc-#{rand(100000)}"
 bin_path = File.expand_path('../../../../bin/trollolo', __FILE__)
 
-ENV["TROLLOLO_CONFIG_PATH"] = config_path
+ENV['TROLLOLO_CONFIG_PATH'] = config_path
 
-cmd = "#{bin_path} #{ARGV.join(" ")}"
+cmd = "#{bin_path} #{ARGV.join(' ')}"
 
 exec(cmd)

data/spec/integration/wrapper/trollolo_wrapper

@@ -3,8 +3,8 @@
 test_config_path = File.expand_path('../../../data/trollolorc', __FILE__)
 bin_path = File.expand_path('../../../../bin/trollolo', __FILE__)
 
-ENV["TROLLOLO_CONFIG_PATH"] = test_config_path
+ENV['TROLLOLO_CONFIG_PATH'] = test_config_path
 
-cmd = "#{bin_path} #{ARGV.join(" ")}"
+cmd = "#{bin_path} #{ARGV.join(' ')}"
 
 exec(cmd)

data/spec/unit/backup_spec.rb

@@ -1,14 +1,14 @@
-require_relative "spec_helper"
+require_relative 'spec_helper'
 
 include GivenFilesystemSpecHelpers
 
 describe Backup do
-  it "sets backup directory" do
+  it 'sets backup directory' do
     backup = Backup.new(dummy_settings)
-    expect(backup.directory).to match File.expand_path("~/.trollolo/backup")
+    expect(backup.directory).to match File.expand_path('~/.trollolo/backup')
   end
 
-  context "custom backup directory" do
+  context 'custom backup directory' do
     use_given_filesystem(keep_files: true)
 
     before(:each) do
@@ -18,22 +18,22 @@ describe Backup do
       @backup.directory = @directory
     end
 
-    it "backups board" do
-      @backup.backup("53186e8391ef8671265eba9d")
-      backup_file = File.join(@directory, "53186e8391ef8671265eba9d", "board.json")
+    it 'backups board' do
+      @backup.backup('53186e8391ef8671265eba9d')
+      backup_file = File.join(@directory, '53186e8391ef8671265eba9d', 'board.json')
       expect(File.exist?(backup_file)).to be true
-      expect(File.read(backup_file)).to eq load_test_file("full-board.json").chomp
+      expect(File.read(backup_file)).to eq load_test_file('full-board.json').chomp
     end
 
-    it "lists backups" do
-      @backup.backup("53186e8391ef8671265eba9d")
-      expect(@backup.list).to eq ["53186e8391ef8671265eba9d"]
+    it 'lists backups' do
+      @backup.backup('53186e8391ef8671265eba9d')
+      expect(@backup.list).to eq ['53186e8391ef8671265eba9d']
     end
 
-    it "shows backup" do
+    it 'shows backup' do
       output_capturer = StringIO.new
-      @backup.backup("53186e8391ef8671265eba9d")
-      @backup.show("53186e8391ef8671265eba9d", output: output_capturer )
+      @backup.backup('53186e8391ef8671265eba9d')
+      @backup.show('53186e8391ef8671265eba9d', output: output_capturer )
       expect(output_capturer.string).to eq(<<EOT
 Trollolo Testing Board
   Sprint Backlog

data/spec/unit/burndown_chart_spec.rb

@@ -24,92 +24,88 @@ describe BurndownChart do
     full_board_mock
   end
 
-  describe "initializer" do
-    it "sets initial meta data" do
-      expect(@chart.data["meta"]["sprint"]).to eq 1
-      expect(@chart.data["meta"]["total_days"]).to eq 10
-      expect(@chart.data["meta"]["weekend_lines"]).to eq [3.5, 8.5]
+  describe 'initializer' do
+    it 'sets initial meta data' do
+      expect(@chart.data['meta']['sprint']).to eq 1
+      expect(@chart.data['meta']['total_days']).to eq 10
+      expect(@chart.data['meta']['weekend_lines']).to eq [3.5, 8.5]
     end
   end
 
-  describe "data" do
+  describe 'data' do
     use_given_filesystem
 
     before(:each) do
       @raw_data = [
         {
-          "date" => '2014-04-23',
-          "updated_at" => '2014-04-23T10:00:00+01:00',
-          "story_points" =>
+          'date' => '2014-04-23',
+          'updated_at' => '2014-04-23T10:00:00+01:00',
+          'story_points' =>
           {
-            "total" => 30,
-            "open" => 23
+            'total' => 30,
+            'open' => 23
           },
-          "tasks" =>
+          'tasks' =>
           {
-            "total" => 25,
-            "open" => 21
+            'total' => 25,
+            'open' => 21
           }
         },
         {
-          "date" => '2014-04-24',
-          "updated_at" => '2014-04-24T19:00:00+01:00',
-          "story_points" =>
+          'date' => '2014-04-24',
+          'updated_at' => '2014-04-24T19:00:00+01:00',
+          'story_points' =>
           {
-            "total" => 30,
-            "open" => 21
+            'total' => 30,
+            'open' => 21
           },
-          "tasks" =>
+          'tasks' =>
           {
-            "total" => 26,
-            "open" => 19
+            'total' => 26,
+            'open' => 19
           },
-          "story_points_extra" =>
+          'story_points_extra' =>
           {
-            "done" => 3
+            'done' => 3
           },
-          "tasks_extra" =>
+          'tasks_extra' =>
           {
-            "done" => 2
+            'done' => 2
           },
-          "story_points_unplanned" =>
+          'story_points_unplanned' =>
           {
-            "total" => 3,
-            "open" => 1
+            'total' => 3,
+            'open' => 1
           },
-          "tasks_unplanned" =>
+          'tasks_unplanned' =>
           {
-            "total" => 2,
-            "open" => 1
+            'total' => 2,
+            'open' => 1
           }
         }
       ]
     end
 
-    it "returns sprint number" do
+    it 'returns sprint number' do
       expect(@chart.sprint).to eq 1
     end
 
-    describe "#add_data" do
-      it "creates first data entry" do
+    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")
+        @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
-          } )
+        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
@@ -117,7 +113,7 @@ describe BurndownChart do
         @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.date_time = DateTime.parse('2014-05-30')
 
         @chart.add_data(@burndown_data)
 
@@ -125,28 +121,24 @@ describe BurndownChart do
         @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")
+        @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
-          } )
+        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
+      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")
+        @burndown_data.date_time = DateTime.parse('2014-05-30')
 
         @chart.add_data(@burndown_data)
 
@@ -154,7 +146,7 @@ describe BurndownChart do
         @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")
+        @burndown_data.date_time = DateTime.parse('2014-05-31')
 
         @chart.add_data(@burndown_data)
 
@@ -162,24 +154,20 @@ describe BurndownChart do
         @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")
+        @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
-          } )
+        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
+      it 'adds data' do
+        @chart.data['days'] = @raw_data
 
         @burndown_data.story_points.open = 16
         @burndown_data.story_points.done = 7
@@ -189,83 +177,83 @@ describe BurndownChart do
         @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")
+        @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'].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['tasks'] ).to eq ( {
+          'total' => 21,
+          'open' => 10
         } )
-        expect( @chart.data["days"].last["story_points_extra"] ).to eq ( {
-          "done" => 3
+        expect( @chart.data['days'].last['story_points_extra'] ).to eq ( {
+          'done' => 3
         } )
-        expect( @chart.data["days"].last["tasks_extra"] ).to eq ( {
-          "done" => 2
+        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.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["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)
 
-        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
-      it "reads data" do
+    describe '#read_data' do
+      it 'reads data' do
         @chart.read_data given_file('burndown-data.yaml')
 
-        expect(@chart.data["days"]).to eq @raw_data
+        expect(@chart.data['days']).to eq @raw_data
       end
 
-      it "reads not done columns" do
+      it 'reads not done columns' do
         @chart.read_data given_file('burndown-data.yaml', from: 'burndown-data-with-config.yaml')
-        expect(@settings.not_done_columns).to eq ["Sprint Backlog", "Doing", "QA"]
+        expect(@settings.not_done_columns).to eq ['Sprint Backlog', 'Doing', 'QA']
       end
     end
 
-    describe "#write_data" do
-      it "writes object to disk" do
+    describe '#write_data' do
+      it 'writes object to disk' do
         @chart.sprint = 2
-        @chart.data["meta"]["total_days"] = 9
-        @chart.data["meta"]["weekend_lines"] = [3.5, 7.5]
-        @chart.data["meta"]["board_id"] = "myboardid"
-        @chart.data["days"] = @raw_data
+        @chart.data['meta']['total_days'] = 9
+        @chart.data['meta']['weekend_lines'] = [3.5, 7.5]
+        @chart.data['meta']['board_id'] = 'myboardid'
+        @chart.data['days'] = @raw_data
 
         write_path = given_dummy_file
         @chart.write_data(write_path)
         expect(File.read(write_path)). to eq load_test_file('burndown-data.yaml')
       end
 
-      it "writes all data which was read" do
+      it 'writes all data which was read' do
         read_path = given_file('burndown-data.yaml')
         @chart.read_data(read_path)
 
@@ -278,29 +266,29 @@ describe BurndownChart do
       it "doesn't write extra entries with 0 values" do
         raw_data = [
           {
-            "date" => '2014-04-24',
-            "story_points" =>
+            'date' => '2014-04-24',
+            'story_points' =>
             {
-              "total" => 30,
-              "open" => 21
+              'total' => 30,
+              'open' => 21
             },
-            "tasks" =>
+            'tasks' =>
             {
-              "total" => 26,
-              "open" => 19
+              'total' => 26,
+              'open' => 19
             },
-            "story_points_extra" =>
+            'story_points_extra' =>
             {
-              "done" => 0
+              'done' => 0
             },
-            "tasks_extra" =>
+            'tasks_extra' =>
             {
-              "done" => 0
+              'done' => 0
             }
           }
         ]
-        @chart.data["days"] = raw_data
-        @chart.data["meta"]["board_id"] = "1234"
+        @chart.data['days'] = raw_data
+        @chart.data['meta']['board_id'] = '1234'
 
         write_path = given_dummy_file
         @chart.write_data(write_path)
@@ -343,77 +331,76 @@ EOT
     end
   end
 
-  describe "commands" do
+  describe 'commands' do
     use_given_filesystem(keep_files: true)
 
-    describe "setup" do
-      it "initializes new chart" do
+    describe 'setup' do
+      it 'initializes new chart' do
         path = given_directory
-        @chart.setup(path, "53186e8391ef8671265eba9d")
+        @chart.setup(path, '53186e8391ef8671265eba9d')
 
-        expect(File.exist?(File.join(path, "burndown-data-01.yaml"))).to be true
+        expect(File.exist?(File.join(path, 'burndown-data-01.yaml'))).to be true
 
         chart = BurndownChart.new(@settings)
-        chart.read_data(File.join(path, "burndown-data-01.yaml"))
+        chart.read_data(File.join(path, 'burndown-data-01.yaml'))
 
-        expect(chart.board_id).to eq "53186e8391ef8671265eba9d"
+        expect(chart.board_id).to eq '53186e8391ef8671265eba9d'
       end
     end
 
-    describe "last_sprint" do
-      it "gets the last sprint based on the burndown files" do
-        path = given_directory_from_data("burndown_dir")
+    describe 'last_sprint' do
+      it 'gets the last sprint based on the burndown files' do
+        path = given_directory_from_data('burndown_dir')
         expect(@chart.last_sprint(path)).to eq(2)
       end
     end
 
-    describe "load_sprint" do
-      let(:path) { given_directory_from_data("burndown_dir") }
+    describe 'load_sprint' do
+      let(:path) { given_directory_from_data('burndown_dir') }
 
-      it "loads the burndown from the 2nd sprint into data" do
+      it 'loads the burndown from the 2nd sprint into data' do
         @chart.load_sprint(path)
         expect(@chart.data).to eq(
-          { "meta" =>
-            { "board_id" => "53186e8391ef8671265eba9d",
-              "sprint" => 2,
-              "total_days" => 9,
-              "weekend_lines" => [3.5, 7.5]
+          '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
-                  }
-              }
-            ]
-          })
+          '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
-        expect(@chart.load_sprint(path)).to eq(File.join(path, "burndown-data-02.yaml"))
+      it 'returns the path of the last sprint' do
+        expect(@chart.load_sprint(path)).to eq(File.join(path, 'burndown-data-02.yaml'))
       end
     end
 
-    describe "update" do
-      let(:path) { given_directory_from_data("burndown_dir") }
-      let(:options) { {'output' => path} }
+    describe 'update' do
+      let(:path) { given_directory_from_data('burndown_dir') }
+      let(:options) { {'output' => path, 'board-id' => '7Zar7bNm'} }
       let(:before) { BurndownChart.new(@settings) }
       let(:after) { BurndownChart.new(@settings) }
 
-      it "updates chart with latest data" do
-        updated_at = "2015-01-12T13:57:16+01:00"
+      it 'updates chart with latest data' do
+        updated_at = '2015-01-12T13:57:16+01:00'
         expected_date_time = DateTime.parse(updated_at)
         allow(DateTime).to receive(:now).and_return(expected_date_time)
 
@@ -422,25 +409,30 @@ EOT
         after.read_data(File.join(path, 'burndown-data-02.yaml'))
         expect(after.days.size).to eq before.days.size + 1
 
-        expect(after.days.last["date"]).to eq "2015-01-12"
-        expect(after.days.last["updated_at"]).to eq updated_at
+        expect(after.days.last['date']).to eq '2015-01-12'
+        expect(after.days.last['updated_at']).to eq updated_at
       end
 
-      it "overwrites data on same date" do
+      it 'overwrites data on same date' do
         before.read_data(File.join(path, 'burndown-data-02.yaml'))
         @chart.update(options)
         @chart.update(options)
         after.read_data(File.join(path, 'burndown-data-02.yaml'))
         expect(after.days.size).to eq before.days.size + 1
       end
+
+      it 'uses provided board-id' do
+        @chart.update(options)
+        expect(@chart.board_id).to eq '7Zar7bNm'
+      end
     end
 
-    describe "create_next_sprint" do
-      let(:path) { given_directory_from_data("burndown_dir") }
+    describe 'create_next_sprint' do
+      let(:path) { given_directory_from_data('burndown_dir') }
       let(:chart) { BurndownChart.new(@settings) }
-      let(:next_sprint_file) { File.join(path, "burndown-data-03.yaml") }
+      let(:next_sprint_file) { File.join(path, 'burndown-data-03.yaml') }
 
-      it "create new sprint file" do
+      it 'create new sprint file' do
         expected_file_content = <<EOT
 ---
 meta:
@@ -458,7 +450,7 @@ EOT
         expect(File.read(next_sprint_file)).to eq expected_file_content
       end
 
-      it "create new sprint file with params" do
+      it 'create new sprint file with params' do
         expected_file_content = <<EOT
 ---
 meta:
@@ -472,7 +464,7 @@ meta:
   - 16.5
 days: []
 EOT
-        chart.create_next_sprint(path, { total_days: 17, weekend_lines: [1.5, 6.5, 11.5, 16.5] })
+        chart.create_next_sprint(path, total_days: 17, weekend_lines: [1.5, 6.5, 11.5, 16.5])
 
         expect(File.exist?(next_sprint_file)).to be true
         expect(File.read(next_sprint_file)).to eq expected_file_content
@@ -480,22 +472,22 @@ EOT
     end
   end
 
-  describe "reads meta data from the board" do
+  describe 'reads meta data from the board' do
     use_given_filesystem
 
-    it "merges meta data from board if present" do
+    it 'merges meta data from board if present' do
       chart = BurndownChart.new(@settings)
-      chart.read_data(given_file("burndown-data-10.yaml"))
+      chart.read_data(given_file('burndown-data-10.yaml'))
 
-      expect(chart.data["meta"]["weekend_lines"]).to eq([3.5, 8.5])
+      expect(chart.data['meta']['weekend_lines']).to eq([3.5, 8.5])
 
       burndown = BurndownData.new(@settings)
-      burndown.board_id = "53186e8391ef8671265eba9d"
+      burndown.board_id = '53186e8391ef8671265eba9d'
       burndown.fetch
 
       chart.merge_meta_data_from_board(burndown)
 
-      expect(chart.data["meta"]["weekend_lines"]).to eq([1.5, 6.5, 11.5, 16.5])
+      expect(chart.data['meta']['weekend_lines']).to eq([1.5, 6.5, 11.5, 16.5])
     end
   end
 
@@ -504,7 +496,7 @@ EOT
       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})
+      described_class.plot(42, foo: 1, bar: 2)
     end
   end