#48 - add logger;

Author: timgluz

.DS_Store

@@ -1,5 +1,6 @@
 *.rbc
 *.sassc
+*.log
 .sass-cache
 capybara-*.html
 .rspec

.gitignore

@@ -44,6 +44,9 @@ flag :timeout, :default_value => 180
 desc 'force open timeout (in seconds)'
 flag :open_timeout, :default_value => 20
 
+desc 'set log level'
+flag :log_level, :default_value => 'debug'
+
 pre do |global_options, command, args|
   check_config_file
   $global_options = init_environment

bin/veye

@@ -2,6 +2,7 @@
 # you just need to require this one file in your bin file
 require 'rest-client'
 require 'yaml'
+require 'logger'
 
 require 'veye/helpers/format_helpers.rb'
 
@@ -30,7 +31,7 @@ def config_exists?
 
 def check_config_file
   unless config_exists?
-    p format(
+    printf(
       "%s: %s\n",
       'config file doesnt exist.'.color(:red),
       'Use `veye initconfig` to initialize settings file.'
@@ -57,7 +58,7 @@ def init_environment
 def check_api_key(global_opts)
   result = false
   if global_opts[:api_key].nil? || global_opts[:api_key].match("add your api key")
-    print format(
+    printf(
       "%s: %s\n",
       'Warning: API key is missing.'.color(:yellow),
       'You cant access private data.'
@@ -74,9 +75,31 @@ def save_configs
   File.open(filepath, 'w') do |f|
     f.puts $global_options.to_yaml
   end
-  print format(
+  printf(
     "%s: %s",
     'Success'.color(:green),
     "new settings are saved into file: `#{filepath}`"
   )
 end
+
+
+#-- initialize logger
+module Veye
+  class << self
+    attr_writer :logger
+
+    def logger
+      @logger ||= Logger.new('veye.log').tap do |log|
+        log.level = case $global_options.fetch(:log_level, 'debug')
+                    when 'info'   then Logger::INFO
+                    when 'debug'  then Logger::DEBUG 
+                    when 'warn'   then Logger::INFO
+                    when 'error'  then Logger::ERROR
+                    when 'fatal'  then Logger::FATAL
+                    else Logger::DEBUG
+                    end
+        log.progname = self.name
+      end
+    end
+  end
+end

lib/veye.rb

@@ -36,6 +36,7 @@ def self.get_list(api_key, org_name, team_name = nil)
       end
 
       def self.upload(api_key, filename, org_name = nil, team_name = nil, temporary = false, public =true, name = nil)
+        Veye.logger.info("Uploading `#{filename}` under #{org_name}, temporary? #{temporary}, organization? #{public} ")
         project_api = Resource.new(RESOURCE_PATH)
         file_path = check_file(filename)
         return if file_path.nil?

lib/veye/api/project.rb

@@ -22,6 +22,7 @@ def self.show_results(output_formats, results, options = {}, paging = nil)
   def self.valid_response?(response, msg)
     if response.nil? || response.success != true
       printf "#{msg.to_s.color(:red)}: #{response.data}\n"
+      Veye.logger.error "API returned error - #{response.code}, #{response.data}"
       return false
     end
 

lib/veye/base_executor.rb

@@ -24,6 +24,7 @@ class Check < BaseExecutor
       }
 
       def self.get_list(api_key, org_name = 'private', team_name = nil, options)
+        Veye.logger.info "Fetching a list of project for #{org_name}, team:#{team_name}"
         results = Veye::API::Project.get_list(api_key, org_name, team_name)
         valid_response?(results, 'Can not read list of projects.')
         show_results(@output_formats, results.data, options)