trackinator 0.0.13 → 0.0.14

data/bin/trackinate

@@ -13,6 +13,7 @@ opts = Trollop::options do
   opt :port, "YouTrack port", :type => :int, :default => 80, :short => "r"
   opt :path_prefix, "YouTrack path prefix (e.g. '/youtrack/')", :type => :string, :default => "/", :short =>"e"
   opt :create_rc, "Create a .trackinatorrc file in your home dir", :default => false, :short => "c"
+  opt :dry_run, "Try it out but don't actually import", :default => false, :short => "d"
 end
 
 unless opts[:create_rc]
@@ -36,7 +37,9 @@ Trollop::die :google_password, "is required" if opts[:google_password].nil?
 Trollop::die :host, "is required" if opts[:host].nil?
 
 you_track = Trackinator::YouTrack.new opts
-google = Trackinator::Google.new opts
+you_track.login(opts[:youtrack_username], opts[:youtrack_password])
+google = Trackinator::Google.new(GData::Client::Spreadsheets.new)
+google.login(opts)
 
 Trollop::die "Check your YouTrack credentials and try again" unless you_track.is_logged_in?
 Trollop::die "Check your Google credentials and try again" unless google.is_logged_in?
@@ -63,6 +66,7 @@ end
 
 if ARGV.length == 1
   importer = Trackinator::Importer.new you_track, google
+  importer.dry_run = opts[:dry_run]
   issues = importer.import(ARGV[0])
 
   if issues.length > 0
@@ -70,4 +74,6 @@ if ARGV.length == 1
       p issue
     end
   end
+
+  puts "done..."
 end

data/lib/trackinator/google.rb

@@ -1,10 +1,13 @@
 module Trackinator
   class Google
-    def initialize opts
-      @client = GData::Client::Spreadsheets.new
+    def initialize client
+      @client = client
+    end
+
+    def login opts
       begin
         @token = @client.clientlogin(opts[:google_username], opts[:google_password])
-      rescue
+      rescue Exception => e
         @token = nil
       end
     end
@@ -16,9 +19,7 @@ module Trackinator
     def get_tickets file_name
       puts "reading document..."
 
-      spreadsheet_feed = @client.get("http://spreadsheets.google.com/feeds/worksheets/#{get_spreadsheet_key(file_name)}/private/full").to_xml
-      spreadsheet_list_data = @client.get(spreadsheet_feed.elements[1, 'entry'].elements[1, 'content'].attributes['src']).to_xml
-
+      spreadsheet_list_data = get_spreadsheet_list_data file_name
       tickets = []
 
       spreadsheet_list_data.elements.each('entry') do |entry|
@@ -30,6 +31,12 @@ module Trackinator
 
     private
 
+    def get_spreadsheet_list_data file_name
+      spreadsheet_feed = @client.get("http://spreadsheets.google.com/feeds/worksheets/#{get_spreadsheet_key(file_name)}/private/full").to_xml
+      spreadsheet_list_url = spreadsheet_feed.elements[1, 'entry'].elements[1, 'content'].attributes['src']
+      @client.get(spreadsheet_list_url).to_xml
+    end
+
     def get_spreadsheet_key file_name
       doc_feed = @client.get("http://spreadsheets.google.com/feeds/spreadsheets/private/full").to_xml
 

data/lib/trackinator/importer.rb

@@ -8,6 +8,8 @@ require 'trackinator'
 module Trackinator
   class Importer
 
+    attr_accessor :dry_run
+
     def initialize you_track, google
       @you_track = you_track
       @google = google
@@ -18,7 +20,7 @@ module Trackinator
 
       issues = validate_tickets(ticket_data)
 
-      if issues.length == 0
+      if issues.length == 0 && !@dry_run
         puts "importing..."
 
         ticket_data.each do |entry|
@@ -27,8 +29,6 @@ module Trackinator
         end
       end
 
-      puts "done..."
-
       issues
     end
 
@@ -38,12 +38,17 @@ module Trackinator
       puts "validating..."
       issues = []
 
+      if tickets.length == 0
+        return issues
+      end
+
       project = tickets[0]['project']
 
       issues.concat(@you_track.project_exists?(project))
 
       tickets.each do |ticket|
         issues.concat(@you_track.you_track_fields_defined?(project, ticket.keys.collect! { |key| key.downcase }))
+
         issues.concat(validate_fields(ticket))
         issues.concat(validate_formats(ticket))
       end
@@ -67,12 +72,16 @@ module Trackinator
       issues = []
 
       ticket.keys.each do |key|
-        if Trackinator.const_defined?("#{key.upcase}") && Trackinator.const_get("#{key.upcase}").match(ticket[key].downcase).nil?
-          issues << "Validation Error: Ticket with ID: #{ticket["id"]} has field '#{key}' with invalid format"
-        end
+        validate_format(key, ticket[key])
       end
 
       issues
     end
+
+    def validate_format key, value
+      if Trackinator.const_defined?("#{key.upcase}") && Trackinator.const_get("#{key.upcase}").match(value.downcase).nil?
+        issues << "Validation Error: Ticket with ID: #{ticket["id"]} has field '#{key}' with invalid format"
+      end
+    end
   end
 end

data/lib/trackinator/version.rb

@@ -1,3 +1,3 @@
 module Trackinator
-  VERSION = "0.0.13"
+  VERSION = "0.0.14"
 end

data/lib/trackinator/you_track.rb

@@ -9,8 +9,6 @@ module Trackinator
       @host = opts[:host]
       @port = opts[:port]
       @path_prefix = opts[:path_prefix]
-
-      login opts[:youtrack_username], opts[:youtrack_password]
     end
 
     def is_logged_in?
@@ -34,7 +32,7 @@ module Trackinator
     end
 
     def update_ticket issue_id, data
-      success = set_platform(issue_id, data['platform'])
+      success = set_subsystem(issue_id, data['subsystem'])
       success ? (success = set_summary_and_description(issue_id, data['summary'], data['description'], data['outcome'])) : (return success)
       success ? (success = set_type(issue_id, data['type'])) : (return success)
       success ? (success = set_import_identifier(issue_id, "#{data['project']}-#{data['id']}")) : (return success)
@@ -78,7 +76,7 @@ module Trackinator
     end
 
     def is_issue_exists? data
-      find_response_xml = REXML::Document.new(@connection.get("#{@path_prefix}rest/issue/byproject/#{data['project']}?filter=Import+Identifier:+#{data['project']}-#{data['id']}", { 'Cookie' => @cookie, 'Content-Type' => 'text/plain; charset=utf-8' }).body)
+      find_response_xml = REXML::Document.new(@connection.get("#{@path_prefix}rest/issue/byproject/#{data['project']}?filter=Import+Identifier:+#{data['project']}-#{data['id']}+Subsystem:+#{data['subsystem']}", { 'Cookie' => @cookie, 'Content-Type' => 'text/plain; charset=utf-8' }).body)
       find_response_xml.elements['issues'].length > 0 ? find_response_xml.elements['issues/issue'].attributes['id'] : nil
     end
 
@@ -122,10 +120,10 @@ module Trackinator
       response.header.msg.eql? "OK"
     end
 
-    def set_platform issue_id, platform
-      return true if platform.nil?
+    def set_subsystem issue_id, subsystem
+      return true if subsystem.nil?
 
-      response = @connection.post("#{@path_prefix}rest/issue/#{issue_id}/execute?command=Platform+#{URI.escape(platform)}&disableNotifications=true", nil, { 'Cookie' => @cookie })
+      response = @connection.post("#{@path_prefix}rest/issue/#{issue_id}/execute?command=Subsystem+#{URI.escape(subsystem)}&disableNotifications=true", nil, { 'Cookie' => @cookie })
       response.header.msg.eql? "OK"
     end
 

data/lib/trackinator.rb

@@ -11,6 +11,8 @@ module Trackinator
 
   PRIORITY = /^low$|^normal$|^high$|^show-stopper$/
 
+  SUBSYSTEM = /^android$|^backend$|^ios$/
+
   REQUIRED = %w{
       project
       id

data/spec/trackinator/google_spec.rb

@@ -2,6 +2,65 @@ require File.expand_path(File.dirname(__FILE__)) + '/../spec_helper'
 
 module Trackinator
   describe Google do
-    pending "write it"
+    context "login" do
+      before(:each) do
+        @client = mock(GData::Client::Spreadsheets)
+        @google = Google.new(@client)
+      end
+
+      it "#login should assign a token" do
+        @client.stub!(:clientlogin).and_return(true)
+        @google.login({})
+        @google.is_logged_in?.should == true
+      end
+
+      it "#login should not assign a token" do
+        @client.stub!(:clientlogin).and_raise(Exception.new)
+        @google.login({})
+        @google.is_logged_in?.should == false
+      end
+    end
+
+    context "get_tickets" do
+      before(:each) do
+        @client = mock(GData::Client::Spreadsheets)
+        @google = Google.new(@client)
+        @spreadsheet_list_data = mock(REXML::Document)
+        @spreadsheet_list_data.stub!(:elements).and_return(@spreadsheet_list_data)
+        @google.stub!(:get_spreadsheet_list_data).and_return(@spreadsheet_list_data)
+        @google.stub!(:get_ticket_data).and_return( {"foo" => "bar"} )
+      end
+
+      it "#get_tickets should return an empty array" do
+        @spreadsheet_list_data.stub!(:each).with("entry")
+        tickets = @google.get_tickets("filename")
+        tickets.length.should == 0
+      end
+
+      it "#get_tickets should return an array of 1" do
+        @spreadsheet_list_data.stub!(:each).with("entry").and_yield(%{entry})
+        tickets = @google.get_tickets("filename")
+        tickets.length.should == 1
+        tickets[0]["foo"].should == "bar"
+      end
+    end
+
+    context "get_spreadsheet_list_data" do
+      it "#get_spreadsheet_list_data should do something" do
+        pending
+      end
+    end
+
+    context "get_spreadsheet_key" do
+      it "#get_spreadsheet_key should do something" do
+        pending
+      end
+    end
+
+    context "get_ticket_data" do
+      it "#get_ticket_data should do something" do
+        pending
+      end
+    end
   end
 end

data/spec/trackinator/importer_spec.rb

@@ -2,6 +2,147 @@ require File.expand_path(File.dirname(__FILE__)) + '/../spec_helper'
 
 module Trackinator
   describe Importer do
-    pending "write it"
+    context "import" do
+      before(:each) do
+        @google = mock(Google)
+
+        @you_track = mock(YouTrack)
+        @you_track.stub!(:is_issue_exists?).and_return(nil)
+        @you_track.stub!(:create_ticket).and_return(true)
+
+        @importer = Importer.new @you_track, @google
+        @importer.stub!(:validate_tickets).and_return([])
+      end
+
+      it "#import should return no issues and import" do
+        @google.stub!(:get_tickets).and_return([{"type" => "story"}])
+        $stdout.should_receive(:puts).with("importing...")
+
+        issues = @importer.import "filename"
+        issues.should == []
+      end
+
+      it "#import should return no issues and not import" do
+        @google.stub!(:get_tickets).and_return([])
+        $stdout.should_receive(:puts).with("importing...")
+
+        issues = @importer.import "filename"
+        issues.should == []
+      end
+    end
+
+    context "validate_tickets" do
+      before(:each) do
+        @you_track = mock(YouTrack)
+        @you_track.stub!(:project_exists?).and_return([])
+        @you_track.stub!(:you_track_fields_defined?).and_return([])
+
+        @google = mock(Google)
+
+        @importer = Importer.new @you_track, @google
+      end
+
+      it "#validate_tickets should return no issues and validate" do
+        @importer.stub!(:validate_fields).and_return([])
+        @importer.stub!(:validate_formats).and_return([])
+
+        issues = @importer.send(:validate_tickets, [{}])
+        issues.should == []
+      end
+
+      it "#validate_tickets should return no issues and not validate" do
+        @importer.stub!(:validate_fields).and_return([])
+        @importer.stub!(:validate_formats).and_return([])
+
+        issues = @importer.send(:validate_tickets, [])
+        issues.should == []
+      end
+    end
+
+    context "validate_fields" do
+      before(:each) do
+        @you_track = mock(YouTrack)
+        @google = mock(Google)
+
+        @importer = Importer.new @you_track, @google
+      end
+
+      it "#validate_fields should return no issues and validate feature" do
+        issues = @importer.send(:validate_fields, { "type" => "feature", "project" => "YTTP", "id" => "1.2", "summary" => "A summary", "description" => "A description", "outcome" => "An outcome" })
+        issues.should == []
+      end
+
+      it "#validate_fields should return 1 issue: missing project" do
+        issues = @importer.send(:validate_fields, { "type" => "feature", "id" => "1.2", "summary" => "A summary", "description" => "A description", "outcome" => "An outcome" })
+        issues.size.should == 1
+        issues[0].should == "Validation Error: Ticket with ID: 1.2 is missing required field 'project'"
+      end
+
+      it "#validate_fields should return 1 issue: missing id" do
+        issues = @importer.send(:validate_fields, { "type" => "feature", "project" => "YTTP", "summary" => "A summary", "description" => "A description", "outcome" => "An outcome" })
+        issues.size.should == 1
+        issues[0].should == "Validation Error: Ticket with ID:  is missing required field 'id'"
+      end
+
+      it "#validate_fields should return 1 issue: missing summary" do
+        issues = @importer.send(:validate_fields, { "type" => "feature", "project" => "YTTP", "id" => "1.2", "description" => "A description", "outcome" => "An outcome" })
+        issues.size.should == 1
+        issues[0].should == "Validation Error: Ticket with ID: 1.2 is missing required field 'summary'"
+      end
+
+      it "#validate_fields should return 1 issue: missing description" do
+        issues = @importer.send(:validate_fields, { "type" => "feature", "project" => "YTTP", "id" => "1.2", "summary" => "A summary", "outcome" => "An outcome" })
+        issues.size.should == 1
+        issues[0].should == "Validation Error: Ticket with ID: 1.2 is missing required field 'description'"
+      end
+
+      it "#validate_fields should return 1 issue: missing outcome" do
+        issues = @importer.send(:validate_fields, { "type" => "feature", "project" => "YTTP", "id" => "1.2", "summary" => "A summary", "description" => "A description" })
+        issues.size.should == 1
+        issues[0].should == "Validation Error: Ticket with ID: 1.2 is missing required field 'outcome'"
+      end
+    end
+
+    context "validate_formats" do
+      before(:each) do
+        @you_track = mock(YouTrack)
+        @google = mock(Google)
+
+        @importer = Importer.new @you_track, @google
+      end
+
+      it "#validate_formats should return no issues and validate" do
+        issues = @importer.send(:validate_formats, { "type" => "feature" })
+        issues.should == []
+      end
+
+      it "#validate_formats should return no issues and validate" do
+        issues = @importer.send(:validate_formats, { "type" => "foo" })
+        issues.size.should == 1
+        issues[0].should == "Validation Error: Ticket with ID:  has field 'type' with invalid format"
+      end
+
+      it "#validate_formats should return no issues and validate" do
+        issues = @importer.send(:validate_formats, { "id" => "1.2" })
+        issues.should == []
+      end
+
+      it "#validate_formats should return no issues and validate" do
+        issues = @importer.send(:validate_formats, { "id" => "junk" })
+        issues.size.should == 1
+        issues[0].should == "Validation Error: Ticket with ID: junk has field 'id' with invalid format"
+      end
+
+      it "#validate_formats should return no issues and validate" do
+        issues = @importer.send(:validate_formats, { "priority" => "normal" })
+        issues.should == []
+      end
+
+      it "#validate_formats should return no issues and validate" do
+        issues = @importer.send(:validate_formats, { "priority" => "junk" })
+        issues.size.should == 1
+        issues[0].should == "Validation Error: Ticket with ID:  has field 'priority' with invalid format"
+      end
+    end
   end
 end

data/spec/trackinator/you_track_spec.rb

@@ -2,6 +2,146 @@ require File.expand_path(File.dirname(__FILE__)) + '/../spec_helper'
 
 module Trackinator
   describe YouTrack do
-    pending "write it"
+    context "login" do
+      it "#login should assign a cookie" do
+        pending
+      end
+    end
+
+    context "create_ticket" do
+      it "#create_ticket should return true" do
+        pending
+      end
+
+      it "#create_ticket should return false" do
+        pending
+      end
+    end
+
+    context "update_ticket" do
+      it "#update_ticket should return true" do
+        pending
+      end
+
+      it "#update_ticket should return false" do
+        pending
+      end
+    end
+
+    context "project_exists?" do
+      it "#project_exists? should return no issues" do
+        pending
+      end
+
+      it "#project_exists? should return 1 issue" do
+        pending
+      end
+    end
+
+    context "you_track_fields_defined?" do
+      it "#you_track_fields_defined? should return no issues" do
+        pending
+      end
+
+      it "#you_track_fields_defined? should return some issues" do
+        pending
+      end
+    end
+
+    context "is_issue_exists?" do
+      it "#is_issue_exists? should return nil" do
+        pending
+      end
+
+      it "#is_issue_exists? should return a YouTrack issue id" do
+        pending
+      end
+    end
+
+    context "update_dependencies" do
+      it "#update_dependencies should return true" do
+        pending
+      end
+
+      it "#update_dependencies should return false" do
+        pending
+      end
+    end
+
+    context "create_youtrack_ticket" do
+      it "#create_youtrack_ticket should return true" do
+        pending
+      end
+
+      it "#create_youtrack_ticket should return false" do
+        pending
+      end
+    end
+
+    context "create_dependency" do
+      it "#create_dependency should return true" do
+        pending
+      end
+
+      it "#create_dependency should return false" do
+        pending
+      end
+    end
+
+    context "set_summary_and_description" do
+      it "#set_summary_and_description should return true" do
+        pending
+      end
+
+      it "#set_summary_and_description should return false" do
+        pending
+      end
+    end
+
+    context "set_platform" do
+      it "#set_platform should return true" do
+        pending
+      end
+
+      it "#set_platform should return false" do
+        pending
+      end
+    end
+
+    context "set_type" do
+      it "#set_type should return true" do
+        pending
+      end
+
+      it "#set_type should return false" do
+        pending
+      end
+    end
+
+    context "set_import_identifier" do
+      it "#set_import_identifier should return true" do
+        pending
+      end
+
+      it "#set_import_identifier should return false" do
+        pending
+      end
+    end
+
+    context "set_design_reference" do
+      it "#set_design_reference should return true" do
+        pending
+      end
+
+      it "#set_design_reference should return false" do
+        pending
+      end
+    end
+
+    context "construct_description" do
+      it "#construct_description should render a template" do
+        pending
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: trackinator
 version: !ruby/object:Gem::Version
-  version: 0.0.13
+  version: 0.0.14
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-16 00:00:00.000000000 Z
+date: 2012-09-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gdata_19