redpomo-reloaded 0.0.13

data/spec/fixtures/close_results.txt

@@ -0,0 +1,2 @@
+Make screenshot #838 @cantiere
+Another task @home

data/spec/fixtures/config.yml

@@ -0,0 +1,16 @@
+cache: false
+trackers:
+  cantiere:
+    url: "https://project.cantierecreativo.net"
+    token: "CANTIERE_TOKEN"
+    default_project_id: "cc"
+    closed_status_id: 5
+    default_priority_id: 4
+    priority_ids: [ 16, 6, 5 ]
+  welaika:
+    url: "http://code.welaika.com"
+    token: "WELAIKA_TOKEN"
+    default_project_id: "welaika"
+    closed_status_id: 5
+    default_priority_id: 4
+    priority_ids: [ 7, 6, 5 ]

data/spec/fixtures/printer_output.txt

@@ -0,0 +1,16 @@
++----------+------------+---------+----------------------------+----------+-------+-------+
+|                               Wednesday 05/02/12 - 5 mins                               |
++----------+------------+---------+----------------------------+----------+-------+-------+
+| Context  | Project    | Issue # | Description                | Duration | From  | To    |
++----------+------------+---------+----------------------------+----------+-------+-------+
+| cantiere | giunti-web |         | Layout homepage e contatti | 5 mins   | 08:10 | 08:15 |
++----------+------------+---------+----------------------------+----------+-------+-------+
+
++---------+---------+---------+-------------+----------+-------+-------+
+|                       Friday 05/04/12 - 5 mins                       |
++---------+---------+---------+-------------+----------+-------+-------+
+| Context | Project | Issue # | Description | Duration | From  | To    |
++---------+---------+---------+-------------+----------+-------+-------+
+| welaika |         | 3293    | Ricette     | 5 mins   | 15:04 | 15:09 |
++---------+---------+---------+-------------+----------+-------+-------+
+

data/spec/fixtures/proper_timelog.csv

@@ -0,0 +1,4 @@
+esportazione #1036 +giunti-web @cantiere,2012-05-15 23:10:44 +0000,10
+Dialog Partner aggiuntivi #1016 @cantiere,2012-05-16 08:42:51 +0000,25
+Dialog Partner aggiuntivi #1016 @cantiere,2012-05-16 09:21:35 +0000,25
+Scorrimento barra orizzontale #1059 @cantiere,2012-05-16 10:01:09 +0000,25

data/spec/fixtures/pull_results.txt

@@ -0,0 +1,20 @@
+Another task @home
+mailchimp #1006 +flor @cantiere
+gestione utenti #987 +giunti-web @cantiere
+Documentazione server su Wiki #986 +giunti-web @cantiere
+Form richiesta kit #985 +giunti-web @cantiere
+Invia libro ad un amico #976 +giunti-web @cantiere
+2012-04-04 Immagini promo per chrome #859 +zazit @cantiere
+(C) 2012-04-04 Correzioni frase e logo zazit #858 +zazit @cantiere
+(B) 2012-04-02 icone per safari #839 +zazit @cantiere
+(B) 2012-04-02 icona per chrome market #837 +zazit @cantiere
+Rel Canonical e rel author #820 +railsyard @cantiere
+DSL backend per la gestione dei model #731 +railsyard @cantiere
+paradosso zazit #728 +zazit @cantiere
+(B) New Users iOS #3316 +dashboard-fiat @welaika
+Orlando Locator #3296 +olasagasti @welaika
+Il Paniere #3295 +olasagasti @welaika
+Scheda Prodotto #3294 +olasagasti @welaika
+Ricette #3293 +olasagasti @welaika
+Some users have mentioned that the intranet wiki does not support 'table' markup. Can that be updated to work? #3286 +orchid @welaika
+Issue with the blog emails #3269 +orchid @welaika

data/spec/fixtures/tasks.txt

@@ -0,0 +1,3 @@
+Make screenshot #838 @cantiere
+Fix #3290 @welaika
+Another task @home

data/spec/fixtures/timelog.csv

@@ -0,0 +1,6 @@
+Export data created by Pomodoro on 2012-05-04 16:12:17 +0000
+
+Description, When, Duration, externalInterruptions, internalInterruptions
+
+Layout homepage e contatti +giunti-web @cantiere, 2012-05-02 08:10:08 +0000, 5, 0, 0
+Ricette #3293 @welaika, 2012-05-04 15:04:57 +0000, 5, 0, 0

data/spec/integration/add_spec.rb

@@ -0,0 +1,29 @@
+require 'tempfile'
+require 'spec_helper'
+
+describe "redpomo add" do
+
+  let(:config_path) { tmp_path('redpomo') }
+
+  it "opens up a template file with REDPOMO_EDITOR as highest priority" do
+    redpomo "add", :env => {"EDITOR" => "echo editor", "VISUAL" => "echo visual", "REDPOMO_EDITOR" => "echo redpomo_editor"}
+    out.should =~ /^redpomo_editor .*issue.*\.textile/
+  end
+
+  it "opens up a template file with VISUAL as 2nd highest priority" do
+    redpomo "add", :env => {"EDITOR" => "echo editor", "VISUAL" => "echo visual", "REDPOMO_EDITOR" => ""}
+    out.should =~ /^visual .*issue.*\.textile/
+  end
+
+  it "opens up a template file with EDITOR as 3rd highest priority" do
+    redpomo "add", :env => {"EDITOR" => "echo editor", "VISUAL" => "", "REDPOMO_EDITOR" => ""}
+    out.should =~ /^editor .*issue.*\.textile/
+  end
+
+  it "complains if no EDITOR is set" do
+    redpomo "add", :env => {"EDITOR" => "", "VISUAL" => "", "REDPOMO_EDITOR" => ""}
+    out.should include "set $EDITOR or $REDPOMO_EDITOR"
+  end
+
+end
+

data/spec/integration/init_spec.rb

@@ -0,0 +1,33 @@
+require 'tempfile'
+require 'spec_helper'
+
+describe "redpomo init" do
+
+  let(:config_path) { tmp_path('redpomo') }
+
+  it "generates a .redpomo config file" do
+    redpomo "init #{config_path}", env: { "EDITOR" => "echo editor" }
+    expect(File.exists?(config_path)).to be true
+  end
+
+  it "opens the config with REDPOMO_EDITOR as highest priority" do
+    redpomo "init #{config_path}", :env => {"EDITOR" => "echo editor", "VISUAL" => "echo visual", "REDPOMO_EDITOR" => "echo redpomo_editor"}
+    out.should == "redpomo_editor #{config_path}"
+  end
+
+  it "opens the config with VISUAL as 2nd highest priority" do
+    redpomo "init #{config_path}", :env => {"EDITOR" => "echo editor", "VISUAL" => "echo visual", "REDPOMO_EDITOR" => ""}
+    out.should == "visual #{config_path}"
+  end
+
+  it "opens the config with EDITOR as 3rd highest priority" do
+    redpomo "init #{config_path}", :env => {"EDITOR" => "echo editor", "VISUAL" => "", "REDPOMO_EDITOR" => ""}
+    out.should == "editor #{config_path}"
+  end
+
+  it "complains if no EDITOR is set" do
+    redpomo "init #{config_path}", :env => {"EDITOR" => "", "VISUAL" => "", "REDPOMO_EDITOR" => ""}
+    out.should include "set $EDITOR or $REDPOMO_EDITOR"
+  end
+
+end

data/spec/lib/redpomo/cli_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+require 'launchy'
+require 'applescript'
+require 'redpomo/cli'
+require 'fileutils'
+
+describe Redpomo::CLI do
+
+  let(:todo_path) {
+    file = File.open(tmp('tasks.txt'), 'w')
+    file.write read_fixture("tasks.txt")
+    file.close
+    file.path
+  }
+
+  let(:config_path) {
+    config = YAML::load_file fixture("config.yml")
+    config["todo"] = todo_path
+    file = File.open(tmp('config.yml'), 'w')
+    file.write config.to_yaml
+    file.close
+    file.path
+  }
+
+  describe "push LOG" do
+    it "pushes the specified timelog to remote trackers" do
+      VCR.use_cassette('cli_push') do
+        cli_redpomo "push #{fixture("timelog.csv")}"
+        out.strip.should == "Pushed 2 time entries"
+      end
+    end
+  end
+
+  describe "push -n LOG" do
+    it "pushes the specified timelog to remote trackers" do
+      VCR.use_cassette('cli_push') do
+        cli_redpomo "push -n #{fixture("timelog.csv")}"
+        out.should == read_fixture("printer_output.txt")
+      end
+    end
+  end
+
+  describe "start ISSUE" do
+    it "starts the specified issue on Pomodoro.app" do
+      AppleScript.expects(:execute).with('tell application "Pomodoro" to start "Make screenshot #838 @cantiere"')
+      cli_redpomo "start 1"
+    end
+  end
+
+  describe "open ISSUE" do
+    it "shows the specified issue on the browser" do
+      Launchy.expects(:open).with('https://project.cantierecreativo.net/issues/838')
+      cli_redpomo "open 1"
+    end
+  end
+
+  describe "close ISSUE" do
+    it "closes issue on remote tracker and marks current task as done" do
+      VCR.use_cassette('cli_close') do
+        cli_redpomo "close 2 -m bar"
+        File.read(todo_path).should == read_fixture("close_results.txt")
+        out.strip.should == 'Issue updated, see it at http://code.welaika.com/issues/3290'
+      end
+    end
+  end
+
+  describe "pull" do
+    it "fetches all the trackers and converts the issues in todo tasks" do
+      VCR.use_cassette('cli_pull') do
+        cli_redpomo "pull"
+        File.read(todo_path).should == read_fixture("pull_results.txt")
+      end
+    end
+  end
+
+  describe "add" do
+    it "creates a new task, adds it the list, and pushes it to the tracker" do
+      VCR.use_cassette('cli_add') do
+        cli_redpomo ["add", "foobar +olasagasti @welaika"]
+        File.read(todo_path).should == read_fixture("add_results.txt")
+        out.strip.should == 'Issue created, see it at http://code.welaika.com/issues/3397'
+      end
+    end
+  end
+
+  after(:each) do
+    File.unlink(todo_path)
+    File.unlink(config_path)
+  end
+
+end

data/spec/lib/redpomo/entry_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+require 'redpomo/entry'
+
+describe Redpomo::Entry do
+
+  describe "#csv_rows" do
+
+    it "parses Pomodoro's classic invalid CSV format" do
+      text = File.read(fixture("timelog.csv"))
+      rows = Redpomo::Entry.csv_rows(text)
+      expect(rows.size).to eq(2)
+    end
+
+    it "parses my Pomodoro fork proper CSV format" do
+      text = File.read(fixture("proper_timelog.csv"))
+      rows = Redpomo::Entry.csv_rows(text)
+      expect(rows.size).to eq(4)
+    end
+
+  end
+
+end
+

data/spec/lib/redpomo/fuzzy_converter_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+require 'redpomo/fuzzy_converter'
+
+describe Redpomo::FuzzyConverter do
+
+  let(:converted_entries) do
+    Redpomo::FuzzyConverter.convert(entries)
+  end
+
+  describe "#convert" do
+
+    context "if there's up to 1.5h of untracked time between entries" do
+      let(:entries) do
+        [
+          Redpomo::Entry.new("foo", DateTime.parse('5th feb 2012 09:00:00'), 25.minutes),
+          Redpomo::Entry.new("bar", DateTime.parse('5th feb 2012 10:00:00'), 25.minutes),
+        ]
+      end
+      it "fills up time between entries" do
+        converted_entries.first.duration.should == 1.hour
+      end
+    end
+
+    context "if there's more than 1.5h of untracked time between entries" do
+      let(:entries) do
+        [
+          Redpomo::Entry.new("foo", DateTime.parse('5th feb 2012 09:00:00'), 25.minutes),
+          Redpomo::Entry.new("bar", DateTime.parse('5th feb 2012 12:00:00'), 25.minutes),
+        ]
+      end
+      it "it adds 1.5 hours between entries" do
+        converted_entries.first.duration.should == (25.minutes + 1.5.hour)
+      end
+    end
+
+    context "on multiple consecutive entries with the same task" do
+      context "in the same day" do
+        let(:entries) do
+          [
+            Redpomo::Entry.new("foo", DateTime.parse('5th feb 2012 09:00:00'), 25.minutes),
+            Redpomo::Entry.new("foo", DateTime.parse('5th feb 2012 09:30:00'), 25.minutes),
+            Redpomo::Entry.new("bar", DateTime.parse('5th feb 2012 12:00:00'), 25.minutes),
+          ]
+        end
+        it "it joins them" do
+          converted_entries.first.duration.should == (30.minutes + 25.minutes + 1.5.hour)
+        end
+      end
+      context "on different days" do
+        let(:entries) do
+          [
+            Redpomo::Entry.new("foo", DateTime.parse('5th feb 2012 09:00:00'), 25.minutes),
+            Redpomo::Entry.new("foo", DateTime.parse('6th feb 2012 09:30:00'), 25.minutes),
+          ]
+        end
+        it "it keeps them separated" do
+          expect(converted_entries.size).to eq(2)
+          converted_entries.first.duration.should == 25.minutes
+          converted_entries.last.duration.should == 25.minutes
+        end
+      end
+    end
+
+  end
+end

data/spec/lib/redpomo/task_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+require 'redpomo/task'
+
+describe Redpomo::Task do
+
+  describe ".orig" do
+    it "returns the original line" do
+      Redpomo::Task.new(stub, "Foo bar #123 +one @two").orig.should == "Foo bar #123 +one @two"
+    end
+  end
+
+  describe ".url" do
+    context "if there's an issue" do
+      it "returns the URL to the issue" do
+        tracker = stub(base_url: "http://foo.bar")
+        task = Redpomo::Task.new(stub, "Foo #123 +project @tracker")
+        task.stubs(:tracker).returns(tracker)
+        task.url.should == "http://foo.bar/issues/123"
+      end
+    end
+    context "if there's a project" do
+      it "returns the URL to the project" do
+        tracker = stub(base_url: "http://foo.bar")
+        task = Redpomo::Task.new(stub, "Foo +project @tracker")
+        task.stubs(:tracker).returns(tracker)
+        task.url.should == "http://foo.bar/projects/project"
+      end
+    end
+    context "else" do
+      it "returns the URL to the tracker default project" do
+        tracker = stub(base_url: "http://foo.bar", default_project: 'bar')
+        task = Redpomo::Task.new(stub, "Foo @tracker")
+        task.stubs(:tracker).returns(tracker)
+        task.url.should == "http://foo.bar/projects/bar"
+      end
+    end
+  end
+
+end

data/spec/lib/redpomo/tracker_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+require 'redpomo/tracker'
+
+describe Redpomo::Tracker do
+
+  subject do
+    Redpomo::Tracker.new(
+      "welaika",
+      url: "http://code.welaika.com",
+      token: "WELAIKA_TOKEN",#"WELAIKA_TOKEN",
+      closed_status_id: "5"
+    )
+  end
+
+  describe ".close_issue!" do
+    it "closes the issue with the specified message" do
+      VCR.use_cassette('close_issue') do
+        lambda {
+          subject.close_issue!("3290", "foobar")
+        }.should_not raise_error
+      end
+    end
+  end
+
+  describe ".issues" do
+    it "creates an Issue for each remote issue" do
+      VCR.use_cassette('issues') do
+        issues = subject.issues
+        expect(issues.size).to eq(7)
+        issues.first.tracker.should == subject
+        issues.first.project_id.should == 'dashboard-fiat'
+        issues.first.issue_id.should == 3316
+        issues.first.due_date.should be_nil
+      end
+    end
+  end
+
+  describe ".push_entry!" do
+    it "creates a TimeEntry" do
+      VCR.use_cassette('push_entry') do
+        task = stub(
+          issue: 3392,
+          text: "Foobar"
+        )
+        entry = stub(
+          to_task: task,
+          datetime: Date.new(2012, 1, 1),
+          duration: 5
+        )
+        result = subject.push_entry!(entry)
+        result["time_entry"]["issue"]["id"].should == 3392
+      end
+    end
+  end
+
+  describe ".create_issue!" do
+    it "creates an Issue" do
+      VCR.use_cassette('create_issue') do
+        issue = stub(
+          subject: "foo",
+          description: "bar",
+          due_date: Date.new(2012, 12, 1),
+          project_id: "olasagasti",
+          priority_id: 5
+        )
+        result = subject.create_issue!(issue)
+        result["issue"]["id"].should be_present
+      end
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,28 @@
+require 'rubygems'
+require 'bundler'
+require 'active_support/core_ext/object'
+
+if ENV['SIMPLECOV']
+  require 'simplecov'
+  SimpleCov.start
+end
+
+Bundler.require(:default, :development)
+
+require 'support/ruby_ext'
+require 'support/cli_helpers'
+require 'support/fixtures'
+
+RSpec.configure do |config|
+  config.mock_framework = :mocha
+  config.include Spec::CLIHelpers
+  config.include Spec::Fixtures
+end
+
+VCR.configure do |c|
+  c.cassette_library_dir = 'spec/fixtures/cassettes'
+  c.hook_into :webmock
+  c.default_cassette_options = {
+    match_requests_on: [:method, :host, :path, :body]
+  }
+end