Detect terminal support before attempting colorization

Fixes #3.

Change-Id: Icd4539fa91d1f12fb44427f38886aeea5ae5d9a4
Reviewed-on: https://gerrit.causes.com/23501
Tested-by: jenkins <jenkins@causes.com>
Reviewed-by: Greg Hurrell <greg@causes.com>

Author: Aiden Scandella

changelog.md

@@ -1,5 +1,6 @@
 0.1.7
 
+* Only colorize output when logging to a TTY
 * Fix crashing --list-templates flag
 
 0.1.6

lib/overcommit/logger.rb

@@ -5,32 +5,44 @@ module Overcommit
   class Logger
     include Singleton
 
+    attr_accessor :output
+
     def partial(*args)
-      print *args
+      out.print *args
     end
 
     def log(*args)
-      puts *args
+      out.puts *args
     end
 
     def bold(str)
-      log "\033[1;37m#{str}\033[0m"
+      color('1;37', str)
     end
 
     def error(str)
-      log "\033[31m#{str}\033[0m"
+      color(31, str)
     end
 
     def success(str)
-      log "\033[32m#{str}\033[0m"
+      color(32, str)
     end
 
     def warning(str)
-      log "\033[33m#{str}\033[0m"
+      color(33, str)
     end
 
     def notice(str)
-      log "\033[1;33m#{str}\033[0m"
+      color('1;33', str)
+    end
+
+    def out
+      self.output ||= $stdout
+    end
+
+  private
+
+    def color(code, str)
+      log(out.isatty ? "\033[#{code}m#{str}\033[0m" : str)
     end
   end
 end

spec/spec_helper.rb

@@ -8,11 +8,4 @@
 
 def exit(*args) ; end
 
-# Silence output to STDOUT
-class Overcommit::Logger
-  def log(*args)
-  end
-
-  def partial(*args)
-  end
-end
+Overcommit::Logger.instance.output = StringIO.new