gitreport 0.0.7 → 0.0.8

data/VERSION

@@ -1 +1 @@
-0.0.7
+0.0.8

data/bin/gitreport

@@ -5,7 +5,7 @@ require "rubygems"
 require 'trollop'
 require "gitreport"
 
-SUB_COMMANDS = %w(activate deactivate commit sync history)
+SUB_COMMANDS = %w(activate deactivate commit sync import)
 
 def parse_options
 
@@ -15,7 +15,7 @@ gitreport is a commandline tool to synchronize your git repo's with gitreport.co
 
 Usage: gitreport <subcommand> [options]
 
-<subcommand> must be one of activate, deactivate, commit, sync, history
+<subcommand> must be one of activate, deactivate, commit, sync, import
 
 Examples:
 
@@ -23,7 +23,7 @@ gitreport activate   - register your project with gitreport.com
 gitreport deactivate - unregister your project from gitreport.com
 gitreport commit     - manually transfer your last and stored (offline) commits to gitreport.com
 gitreport sync       - manually transfer stored (offline) commits to gitreport.com
-gitreport history    - import all the commits of a project to gitreport.com
+gitreport import     - import all the commits of a project to gitreport.com
 
 gitreport allows the folling [options]:
     EOS
@@ -48,7 +48,7 @@ gitreport allows the folling [options]:
       GitReport::Sender.send! :last_and_stored
     when "sync"
       GitReport::Sender.send! :stored
-    when "history"
+    when "import"
       GitReport::BatchSender.send! :history
     else
       Trollop::die "unknown subcommand #{cmd.inspect}"

data/gitreport.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "gitreport"
-  s.version = "0.0.7"
+  s.version = "0.0.8"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Jan Roesner"]
-  s.date = "2011-10-27"
+  s.date = "2011-10-28"
   s.description = "gitreport keeps track of your projects. It collects info about commited and pushed data, submits it to our servers and provides a gorgous frontend to examine, discover and extract the data that you need to generate the payment recipes for your customers. No longer searching or `what did I commit when and where`..."
   s.email = "jan@roesner.it"
   s.executables = ["gitreport"]
@@ -31,6 +31,8 @@ Gem::Specification.new do |s|
     "lib/commit.rb",
     "lib/configuration.rb",
     "lib/current_branch.rb",
+    "lib/g_logger.rb",
+    "lib/generic_sender.rb",
     "lib/git.rb",
     "lib/git/author.rb",
     "lib/git/base.rb",
@@ -53,7 +55,6 @@ Gem::Specification.new do |s|
     "lib/history.rb",
     "lib/hook.rb",
     "lib/log.rb",
-    "lib/logger.rb",
     "lib/project.rb",
     "lib/sender.rb",
     "lib/storage.rb",

data/lib/batch_sender.rb

@@ -1,6 +1,6 @@
 module GitReport
 
-  class BatchSender
+  class BatchSender < GenericSender
 
     # send the given commits chunked
     def self.send! option = nil
@@ -36,6 +36,7 @@ module GitReport
 
     # sends the commit batch data to the server
     def self.send_data! batch, options = nil
+      grlog(1, 'send_data started')
       begin
         response = Net::HTTP.Proxy(configuration.proxy_host, configuration.proxy_port).start(configuration.host, configuration.port) do |http|
           request = Net::HTTP::Post.new(request_path options)
@@ -45,40 +46,11 @@ module GitReport
           http.read_timeout = configuration.timeout
           http.request request unless GitReport.global_opts[:dry_run]
         end
+        grlog(1, response ? "send_data responded with #{response.code}" : "send_data had no response")
         raise GitReport::ServerError unless (response.code == "200" or response.code == "401") unless GitReport.global_opts[:dry_run]
       rescue Exception => e
-        if e.is_a?(GitReport::ServerError)
-          puts "A server error occured during data transfer."
-          if GitReport.global_opts[:trace]
-            puts "Exception: #{e}\n"
-            puts "Message: #{JSON.parse(response.body)["message"]}\n"
-          else
-            puts "Run with --trace to get more info."
-          end
-          grlog(0 ,'send_data! - A server error occured during data transfer.')
-          grlog(0, "send_data! - Exception: #{e}")
-          grlog(0, "send_data! - Message: #{JSON.parse(response.body)["message"]}") if response
-          exit
-        else
-          puts "A client error occured during data transfer."
-          if GitReport.global_opts[:trace]
-            puts "Exception: #{e}\n"
-            e.backtrace.each do |line|
-              puts "#{line}\n"
-            end
-          else
-            puts "Run with --trace to get more info."
-          end
-          grlog(0, "send_data! - A client error occures during data transfer.")
-          grlog(0, "send_data! - Exception: #{e}")
-          grlog(0, "send_data! - Backtrace:")
-          e.backtrace.each do |line|
-            grlog(0, line)
-          end
-          exit
-        end
-
-        return false
+        communicate e, response
+        exit
       end
 
       true
@@ -93,24 +65,11 @@ module GitReport
       }.to_json
     end
 
-    # returns configuration object
-    def self.configuration
-      @@configuration ||= GitReport.configuration
-    end
-
     # returns the request path
     def self.request_path options
       @@path ||= "/v#{configuration.api_version}/projects"
     end
 
-    # returns the default headers
-    def self.headers request
-      request['User-Agent']              = 'gitreport-client-ruby'
-      request['Content-Type']            = 'application/json'
-      request['Accept']                  = 'application/json'
-      request['X-gitreport-Auth-Token']  = configuration.auth_token
-    end
-
   end
 
 end

data/lib/commit.rb

@@ -45,7 +45,7 @@ module GitReport
 
     # returns commits stats in more detail
     def stats
-      @commit.diff(@commit.parent).stats[:total]
+      @commit.parent.diff(@commit).stats[:total]
     rescue
       nil
     end

data/lib/{logger.rb → g_logger.rb}

@@ -1,6 +1,6 @@
 module GitReport
 
-  class Logger
+  class GLogger
 
     # logs the given message in case the given level is less or equal the configured log level
     def self.log level, message
@@ -13,7 +13,7 @@ module GitReport
     # provides method grlog in every class
     class << self.superclass
       def grlog level, message
-        ::GitReport::Logger.log level, Time.now.to_s + " - " + self.to_s + " #" + message
+        ::GitReport::GLogger.log level, Time.now.to_s + " - " + self.to_s + " #" + message
       end
     end
 

data/lib/generic_sender.rb

@@ -0,0 +1,44 @@
+class GenericSender
+
+  # logs and puts the error according to its conditions and configuration
+  # TODO refactor me!
+  def self.communicate exception, response
+    both, cmd, log = [], [], []
+    if exception.is_a?(GitReport::ServerError)
+      @error_type = "server error"
+      @details = JSON.parse(response.body)["message"] rescue response.body
+    else
+      @error_type = "client error"
+      @details = exception.backtrace
+    end
+    both << "A #{@error_type} occured during data transfer."
+    both << "Exception: #{exception}"
+    if GitReport.global_opts[:trace]
+      cmd << @details
+    else
+      cmd << "Run with --trace to get more info."
+    end
+    log << @details
+    puts (both + cmd).join("\n")
+    (both + log).each{ |line| grlog(0, "send_data! #{line}")}
+  end
+
+  # returns local storage
+  def self.storage
+    @@storage ||= GitReport::Storage.new(ENV['HOME'], '.gitreport_storage')
+  end
+
+  # returns configuration object
+  def self.configuration
+    @@configuration ||= GitReport.configuration
+  end
+
+  # returns the default headers
+  def self.headers request
+    request['User-Agent']              = 'gitreport-client-ruby'
+    request['Content-Type']            = 'application/json'
+    request['Accept']                  = 'application/json'
+    request['X-gitreport-Auth-Token']  = configuration.auth_token
+  end
+
+end

data/lib/gitreport.rb

@@ -1,4 +1,4 @@
-require 'logger'
+require 'g_logger'
 require 'git'
 require 'json'
 require 'net/http'
@@ -11,6 +11,7 @@ require 'commit'
 require 'history'
 require 'configuration'
 require 'git_configuration'
+require 'generic_sender'
 require 'sender'
 require 'batch_sender'
 require 'storage'
@@ -19,6 +20,8 @@ require 'supplier'
 
 module GitReport
 
+  @@global_opts ||= nil
+
   class ServerError < StandardError;end
 
   # mattr_reader

data/lib/hook.rb

@@ -54,7 +54,7 @@ module GitReport
 
     # returns true if the hook file already has a hook line in
     def self.line_exists?
-      if file_content.match(/bundle\sexec\sgitreport\scommit\s&/)
+      if file_content.match(/nohup\sbundle\sexec\sgitreport\scommit\s>\s\/dev\/null\s2>\s\/dev\/null\s<\s\/dev\/null\s&/)
         return true
       end
 
@@ -76,7 +76,7 @@ module GitReport
       "#!/bin/sh\n" +
       "# This is a post-commit hook created by gitreport (http://gitreport.com)\n" +
       "#\n" +
-      "# To remove it issue 'bundle exec unregister' in the projects main directory\n" +
+      "# To remove it issue 'bundle exec deactivate' in the projects main directory\n" +
       "# In case the gitreport gem is not installed anymore, simply remove this hook file\n" +
       "#\n" +
       "# Be aware of other post commit hooks that my be mentioned here!\n"
@@ -84,7 +84,7 @@ module GitReport
 
     # returns the line to activate gitreport via post commit hook
     def self.line
-      "\nbundle exec gitreport commit &\n"
+      "\nnohup bundle exec gitreport commit > /dev/null 2> /dev/null < /dev/null &\n"
     end
 
     # removes the hook
@@ -105,7 +105,7 @@ module GitReport
 
     # returns true if the hook file is ours and was not changed
     def self.hook_file_unchanged?
-      Digest::SHA1.hexdigest(file_content) == "9c69e61ce35b8ce21968343411e6abeb89b237dd"
+      Digest::SHA1.hexdigest(file_content) == "e4032a91bb8e07e09ea637c803d8763e26e165e7"
     end
 
     # removes our hook line from hook file
@@ -115,7 +115,7 @@ module GitReport
 
     # removes our hook line from given content
     def self.clean_up content
-      content.gsub(/\nbundle\sexec\sgitreport\scommit\s&\n/,'')
+      content.gsub(/\nnohup\sbundle\sexec\sgitreport\scommit\s>\s\/dev\/null\s2>\s\/dev\/null\s<\s\/dev\/null\s&\n/,'')
     end
 
   end

data/lib/sender.rb

@@ -1,27 +1,13 @@
 module GitReport
 
-  class Sender
+  class Sender < GenericSender
 
     # sends or saves the commits
     def self.send! options = nil
       begin
         commits = GitReport::Supplier.commits(options)
       rescue Exception => e
-        puts "A client error occured during data transfer."
-        if GitReport.global_opts[:trace]
-          puts "Exception: #{e}\n"
-          e.backtrace.each do |line|
-            puts "#{line}\n"
-          end
-        else
-          puts "Run with --trace to get more info."
-        end
-        grlog(0, "send! - A client error occures during data transfer.")
-        grlog(0, "send! - Exception: #{e}")
-        grlog(0, "send! - Backtrace:")
-        e.backtrace.each do |line|
-          grlog(0, line)
-        end
+        communicate e, nil
         exit
       end
 
@@ -37,6 +23,7 @@ module GitReport
 
     # sends the commit data to the server
     def self.send_data! commit, options = nil
+      grlog(1, 'send_data started')
       begin
         response = Net::HTTP.Proxy(configuration.proxy_host, configuration.proxy_port).start(configuration.host, configuration.port) do |http|
           request = Net::HTTP::Post.new(request_path options)
@@ -46,53 +33,21 @@ module GitReport
           http.read_timeout = configuration.timeout
           http.request request unless GitReport.global_opts[:dry_run]
         end
+        grlog(1, response ? "send_data responded with #{response.code}" : "send_data had no response")
         raise GitReport::ServerError unless (response.code == "201" or response.code == "401") unless GitReport.global_opts[:dry_run]
       rescue Exception => e
-        if e.is_a?(GitReport::ServerError)
-          puts "A server error occured during data transfer."
-          if GitReport.global_opts[:trace]
-            puts "Exception: #{e}\n"
-            puts "Message: #{JSON.parse(response.body)["message"]}\n"
-          else
-            puts "Run with --trace to get more info."
-          end
-        else
-          puts "A client error occured during data transfer."
-          puts "Exception: #{e}\n"
-        end
-        grlog(0 ,'send_data! - A server error occured during data transfer.')
-        grlog(0, "send_data! - Exception: #{e}")
-        grlog(0, "send_data! - Message: #{JSON.parse(response.body)["message"]}") if response
-
+        communicate e, response
         return false
       end
 
       true
     end
 
-    # returns local storage
-    def self.storage
-      @@storage ||= GitReport::Storage.new(ENV['HOME'], '.gitreport_storage')
-    end
-
-    # returns configuration object
-    def self.configuration
-      @@configuration ||= GitReport.configuration
-    end
-
     # returns the request path
     def self.request_path options
       @@path ||= "/v#{configuration.api_version}/commits"
     end
 
-    # returns the default headers
-    def self.headers request
-      request['User-Agent']              = 'gitreport-client-ruby'
-      request['Content-Type']            = 'application/json'
-      request['Accept']                  = 'application/json'
-      request['X-gitreport-Auth-Token']  = configuration.auth_token
-    end
-
   end
 
 end

data/spec/models/commit_spec.rb

@@ -45,10 +45,10 @@ describe 'GitReport::Commit' do
   describe '#stats' do
     it 'should return the commit stats' do
       stats = GitReport::Commit.new(@project.log.commits.first, @project.identifier).stats
-      stats[:deletions].should  == 1
+      stats[:deletions].should  == 0
       stats[:files].should      == 1
       stats[:lines].should      == 1
-      stats[:insertions].should == 0
+      stats[:insertions].should == 1
     end
   end
 

data/spec/models/hook_spec.rb

@@ -18,7 +18,7 @@ describe 'GitReport::Hook' do
       (File.exists?(hook_file)).should be_true
       content = File.open(hook_file, 'r').read
 
-      content.match(/\nbundle\sexec\sgitreport\scommit\s&\n/).should be_true
+      content.match(/\nnohup\sbundle\sexec\sgitreport\scommit\s>\s\/dev\/null\s2>\s\/dev\/null\s<\s\/dev\/null\s&\n/).should be_true
     end
 
     it 'should insert the gitreport hook into an existing post-commit hook file' do
@@ -30,7 +30,7 @@ describe 'GitReport::Hook' do
 
       GitReport::Hook.set!
       content = File.open(hook_file, 'r').read
-      content.match(/\nbundle\sexec\sgitreport\scommit\s&\n/).should be_true
+      content.match(/\nnohup\sbundle\sexec\sgitreport\scommit\s>\s\/dev\/null\s2>\s\/dev\/null\s<\s\/dev\/null\s&\n/).should be_true
     end
   end
 
@@ -57,7 +57,7 @@ describe 'GitReport::Hook' do
       (File.exists?(hook_file)).should be_true
       content = File.open(hook_file, 'r').read
       content.should match(/\ssome\spreexisting\shook\sfile\n/)
-      content.should_not match(/\nbundle\sexec\sgitreport\scommit\s&\n/)
+      content.should_not match(/\nnohup\sbundle\sexec\sgitreport\scommit\s>\s\/dev\/null\s2>\s\/dev\/null\s<\s\/dev\/null\s&\n/)
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gitreport
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.8
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-10-27 00:00:00.000000000Z
+date: 2011-10-28 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
-  requirement: &70138129848900 !ruby/object:Gem::Requirement
+  requirement: &70284346535700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70138129848900
+  version_requirements: *70284346535700
 - !ruby/object:Gem::Dependency
   name: spork
-  requirement: &70138129848420 !ruby/object:Gem::Requirement
+  requirement: &70284346531340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>'
@@ -32,10 +32,10 @@ dependencies:
         version: 0.9.0.rc
   type: :development
   prerelease: false
-  version_requirements: *70138129848420
+  version_requirements: *70284346531340
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70138129847940 !ruby/object:Gem::Requirement
+  requirement: &70284346525880 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70138129847940
+  version_requirements: *70284346525880
 - !ruby/object:Gem::Dependency
   name: webmock
-  requirement: &70138129847460 !ruby/object:Gem::Requirement
+  requirement: &70284346521540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70138129847460
+  version_requirements: *70284346521540
 - !ruby/object:Gem::Dependency
   name: shoulda
-  requirement: &70138129846980 !ruby/object:Gem::Requirement
+  requirement: &70284346518300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70138129846980
+  version_requirements: *70284346518300
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70138129863620 !ruby/object:Gem::Requirement
+  requirement: &70284346514400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -76,10 +76,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *70138129863620
+  version_requirements: *70284346514400
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &70138129863140 !ruby/object:Gem::Requirement
+  requirement: &70284346510280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -87,10 +87,10 @@ dependencies:
         version: 1.6.4
   type: :development
   prerelease: false
-  version_requirements: *70138129863140
+  version_requirements: *70284346510280
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &70138129862660 !ruby/object:Gem::Requirement
+  requirement: &70284346504540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,7 +98,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70138129862660
+  version_requirements: *70284346504540
 description: gitreport keeps track of your projects. It collects info about commited
   and pushed data, submits it to our servers and provides a gorgous frontend to examine,
   discover and extract the data that you need to generate the payment recipes for
@@ -124,6 +124,8 @@ files:
 - lib/commit.rb
 - lib/configuration.rb
 - lib/current_branch.rb
+- lib/g_logger.rb
+- lib/generic_sender.rb
 - lib/git.rb
 - lib/git/author.rb
 - lib/git/base.rb
@@ -146,7 +148,6 @@ files:
 - lib/history.rb
 - lib/hook.rb
 - lib/log.rb
-- lib/logger.rb
 - lib/project.rb
 - lib/sender.rb
 - lib/storage.rb
@@ -182,7 +183,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2322616335107185957
+      hash: 3923250588470254802
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: