coffee_trace 0.0.2 → 0.0.3

data/README.md

@@ -40,6 +40,10 @@ Coffee Trace will remove all log statements that it added to files below the cur
 
 ***
 
+For those interested in the source code, I was experimenting with a pattern where class names are not nouns, but the single responsibility of the class. I am not convinced yet that it is a pattern I'd use on a bigger project, but it is quite interesting and seems to help keep classes small and testable.
+
+***
+
 Copyright (c) 2011 Chris Aitchison. See LICENSE for details.
     
 

data/lib/coffee_trace/cli.rb

@@ -1,6 +1,5 @@
 module CoffeeTrace
   class CLI
-
     def initialize insert_logging=CoffeeTrace::InsertLogging.new
       @insert_logging = insert_logging
       @files = Dir.glob "**/*.coffee"
@@ -8,27 +7,27 @@ module CoffeeTrace
 
     def insert_logging
       @files.each do |file|
-	prefix =File.basename file, ".coffee"
-	result = @insert_logging.into File.readlines(file), prefix
-	write_lines file, result 
+        prefix =File.basename file, ".coffee"
+        result = @insert_logging.into File.readlines(file), prefix
+        write_lines file, result 
       end
     end
 
     def remove_logging
       @files.each do |file|
-	lines = @insert_logging.into File.readlines(file)
+        lines = @insert_logging.into File.readlines(file)
         result = lines.select do |line|
-	  /console\.log.*#coffee_trace.*/.match(line).nil?
-	end
-	write_lines file, result
+          /console\.log.*#coffee_trace.*/.match(line).nil?
+        end
+        write_lines file, result
       end
     end
-
+    
+    private
     def write_lines(path, data)
       File.open(path, "wb") do |file|
         file.puts data
       end
     end
-  
   end
 end

data/lib/coffee_trace/insert_log_statement.rb

@@ -1,7 +1,5 @@
-module CoffeeTrace
-  
+module CoffeeTrace  
   class InsertLogStatement
-    
     def for line, prefix=nil
       leading_spaces, method_name, args = parse_function line
       return unless leading_spaces
@@ -13,17 +11,14 @@ module CoffeeTrace
     end
 
     private
-      
     def parse_function line
       no_arg_function_match = line.scan(/^( *)(\w+): * \(? *\)? *[-=]> *$/)
-      return parse_function_with_args(line) if no_arg_function_match.empty?
-      no_arg_function_match[0]
+      no_arg_function_match.empty? ? parse_function_with_args(line) : no_arg_function_match.first
     end
 
     def parse_function_with_args line
       arg_function_match = line.scan(/^( *)(\w+): *\((.*)\) *[-=]> *$/)
-      return if arg_function_match.empty?
-      return arg_function_match[0]
+      arg_function_match.empty? ? nil : arg_function_match.first
     end
   end
 

data/lib/coffee_trace/insert_logging.rb

@@ -9,8 +9,8 @@ module CoffeeTrace
       result = []
       lines.each do |line|
         result.pop if is_coffee_trace_log?(line)  
-	result << line
-	result << (@insert_log_statement.for line, prefix)
+        result << line
+        result << (@insert_log_statement.for line, prefix)
       end
       result.compact
     end 

data/lib/coffee_trace/version.rb

@@ -1,3 +1,3 @@
 module CoffeeTrace
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/insert_log_statement_spec.rb

@@ -31,7 +31,6 @@ describe CoffeeTrace::InsertLogStatement do
     assert "  renderPoi: (poi) =>", "    console.log 'renderPoi', poi #coffee_trace"
   end
 
-
   it "handles trailing spaces" do
     assert "  initialize: -> ", "    console.log 'initialize' #coffee_trace"
   end
@@ -39,6 +38,7 @@ describe CoffeeTrace::InsertLogStatement do
   it "adds a prefix if given" do
     assert_with_prefix "  initialize: -> ","trace_logger","    console.log 'trace_logger-initialize' #coffee_trace"
   end
+  
   it "indents correctly" do
     assert "a: ->", "  console.log 'a' #coffee_trace"
     assert "  a: ->", "    console.log 'a' #coffee_trace"

metadata

@@ -2,7 +2,7 @@
 name: coffee_trace
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors: 
 - Chris Aitchison