wvanbergen-request-log-analyzer 1.2.0 → 1.2.1

data/spec/lib/helper.rb

@@ -2,15 +2,17 @@ module RequestLogAnalyzer::Spec::Helper
   
   include RequestLogAnalyzer::Spec::Mocks
 
-
+  # Create or return a new TestingFormat
   def testing_format
     @testing_format ||= TestingFormat.new
   end
   
-  def log_fixture(name)
-    File.dirname(__FILE__) + "/../fixtures/#{name}.log"
+  # Load a log file from the fixture folder
+  def log_fixture(name, extention = "log")
+    File.dirname(__FILE__) + "/../fixtures/#{name}.#{extention}"
   end
 
+  # Request loopback
   def request(fields, format = testing_format)
     if fields.kind_of?(Array)
       format.request(*fields)
@@ -19,6 +21,8 @@ module RequestLogAnalyzer::Spec::Helper
     end
   end
 
+  # Run a specific command
+  # Used to call request-log-analyzer through binary
   def run(arguments)
     binary = "#{File.dirname(__FILE__)}/../../bin/request-log-analyzer"
     arguments = arguments.join(' ') if arguments.kind_of?(Array)
@@ -30,4 +34,16 @@ module RequestLogAnalyzer::Spec::Helper
     $?.exitstatus.should == 0
     output
   end
+  
+  # Cleanup all temporary files generated by specs
+  def cleanup_temp_files!
+    Dir["#{File.dirname(__FILE__)}/../../tmp/spec.*tmp"].each do |file|
+      File.unlink(file)
+    end
+  end
+  
+  # Return a filename that can be used as temporary file in specs
+  def temp_output_file(file_type)
+    "#{File.dirname(__FILE__)}/../../tmp/spec.#{file_type}.tmp"    
+  end
 end

data/spec/lib/testing_format.rb

@@ -1,3 +1,4 @@
+# Simple log file specification, used to test log parser.
 class TestingFormat < RequestLogAnalyzer::FileFormat::Base
 
   format_definition.first do |line|

data/spec/spec_helper.rb

@@ -11,3 +11,5 @@ end
 require File.dirname(__FILE__) + '/lib/testing_format'
 require File.dirname(__FILE__) + '/lib/mocks'
 require File.dirname(__FILE__) + '/lib/helper'
+
+Dir.mkdir("#{File.dirname(__FILE__)}/../tmp") unless File.exist?("#{File.dirname(__FILE__)}/../tmp")

data/spec/unit/aggregator/summarizer_spec.rb

@@ -24,5 +24,5 @@ describe RequestLogAnalyzer::Aggregator::Summarizer do
   it "not raise exception when creating a report after aggregating no requests" do
     lambda { @summarizer.report(mock_output) }.should_not raise_error
   end
-
+  
 end

data/spec/unit/source/log_parser_spec.rb

@@ -64,4 +64,43 @@ describe RequestLogAnalyzer::Source::LogParser, :warnings do
     @log_parser.should_not_receive(:handle_request)    
     @log_parser.parse_file(log_fixture(:test_order))    
   end  
+end
+
+describe RequestLogAnalyzer::Source::LogParser, :decompression do
+  include RequestLogAnalyzer::Spec::Helper
+  
+  before(:each) do
+    @log_parser = RequestLogAnalyzer::Source::LogParser.new(RequestLogAnalyzer::FileFormat::Rails.new)
+  end
+    
+  it "should parse a rails gzipped log file" do
+    @log_parser.should_receive(:handle_request).once
+    @log_parser.parse_file(log_fixture(:decompression, "log.gz"))
+    @log_parser.parsed_lines.should > 0
+  end
+
+  it "should parse a rails tar gzipped log folder" do
+    @log_parser.should_receive(:handle_request).twice
+    @log_parser.parse_file(log_fixture(:decompression, "tar.gz"))
+    @log_parser.parsed_lines.should > 1
+  end
+
+  it "should parse a rails tar gzipped log folder" do
+    @log_parser.should_receive(:handle_request).twice
+    @log_parser.parse_file(log_fixture(:decompression, "tgz"))
+    @log_parser.parsed_lines.should > 1
+  end
+
+  it "should parse a rails bz2 zipped log file" do
+    @log_parser.should_receive(:handle_request).once
+    @log_parser.parse_file(log_fixture(:decompression, "log.bz2"))
+    @log_parser.parsed_lines.should > 0
+  end
+
+  it "should parse a rails zipped log file" do
+    @log_parser.should_receive(:handle_request).once
+    @log_parser.parse_file(log_fixture(:decompression, "log.zip"))
+    @log_parser.parsed_lines.should > 0
+  end
+  
 end

data/spec/unit/tracker/{tracker_api_test.rb → tracker_api_spec.rb}

@@ -10,7 +10,7 @@ describe RequestLogAnalyzer::Tracker::Base, "API test" do
     @summarizer = RequestLogAnalyzer::Aggregator::Summarizer.new(mock_source)
     @summarizer.trackers << @tracker
   end
-  
+    
   it "should receive :prepare when the summarizer is preparing" do
     @tracker.should_receive(:prepare).once    
     @summarizer.prepare
@@ -42,4 +42,10 @@ describe RequestLogAnalyzer::Tracker::Base, "API test" do
     @summarizer.report(mock_output) 
   end
   
+  it "should receive :to_yaml object when finalizing" do
+    @summarizer.options[:dump] = temp_output_file(:dump)
+    @tracker.should_receive(:to_yaml_object).once
+    @summarizer.to_yaml
+  end
+  
 end

data/tasks/github-gem.rake

@@ -14,8 +14,7 @@ module Rake
     def self.define_tasks!
       gem_task_builder = Rake::GithubGem.new      
       gem_task_builder.register_all_tasks!
-    end
-    
+    end    
 
     def initialize
       reload_gemspec!

data/tasks/request_log_analyzer.rake

@@ -1,10 +1,26 @@
-namespace :log do
-  desc "Analyze the Rails log file using the request-log-analyzer gem." 
-    task :analyze  => :environment do
+namespace :rla do
+  desc "Analyze the Rails log file using the request-log-analyzer gem."
+  task :report  => :environment do
+    puts "Analyzing the Rails log file using the request-log-analyzer gem."
+    puts "  Environment: #{RAILS_ENV}"
+    puts "  Logfile: #{Rails.configuration.log_path}"
+    puts ""
+    IO.popen("request-log-analyzer #{Rails.configuration.log_path}") { |io| $stdout << io.read }
+    
+  end
+
+  namespace :report do
+    desc "Analyze the Rails log file using the request-log-analyzer gem and output an HTML file."
+    task :html  => :environment do
+      output_file = Rails.configuration.log_path + ".html"
+
       puts "Analyzing the Rails log file using the request-log-analyzer gem."
-      puts "Environment: #{RAILS_ENV}"
-      puts "Logfile: #{Rails.configuration.log_path}"
+      puts "  Environment: #{RAILS_ENV}"
+      puts "  Logfile: #{Rails.configuration.log_path}"
+      puts "  Output: #{output_file}"
       puts ""
-      `request-log-analyzer #{Rails.configuration.log_path} -z`
+      IO.popen("request-log-analyzer #{Rails.configuration.log_path} --output HTML --file #{output_file}") { |io| $stdout << io.read }
+    end
   end
-end
+    
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: wvanbergen-request-log-analyzer
 version: !ruby/object:Gem::Version 
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors: 
 - Willem van Bergen
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-06-12 00:00:00 -07:00
+date: 2009-07-15 00:00:00 -07:00
 default_executable: request-log-analyzer
 dependencies: []
 
@@ -23,10 +23,10 @@ extensions: []
 extra_rdoc_files: 
 - README.rdoc
 files: 
-- DESIGN
-- HACKING
+- DESIGN.rdoc
 - LICENSE
 - README.rdoc
+- RELEASE_NOTES.rdoc
 - Rakefile
 - bin
 - bin/request-log-analyzer
@@ -72,6 +72,12 @@ files:
 - lib/request_log_analyzer/tracker/timespan.rb
 - spec
 - spec/fixtures
+- spec/fixtures/decompression.log
+- spec/fixtures/decompression.log.bz2
+- spec/fixtures/decompression.log.gz
+- spec/fixtures/decompression.log.zip
+- spec/fixtures/decompression.tar.gz
+- spec/fixtures/decompression.tgz
 - spec/fixtures/merb.log
 - spec/fixtures/multiple_files_1.log
 - spec/fixtures/multiple_files_2.log
@@ -114,7 +120,7 @@ files:
 - spec/unit/tracker/frequency_tracker_spec.rb
 - spec/unit/tracker/hourly_spread_spec.rb
 - spec/unit/tracker/timespan_tracker_spec.rb
-- spec/unit/tracker/tracker_api_test.rb
+- spec/unit/tracker/tracker_api_spec.rb
 - tasks
 - tasks/github-gem.rake
 - tasks/request_log_analyzer.rake
@@ -169,3 +175,4 @@ test_files:
 - spec/unit/tracker/frequency_tracker_spec.rb
 - spec/unit/tracker/hourly_spread_spec.rb
 - spec/unit/tracker/timespan_tracker_spec.rb
+- spec/unit/tracker/tracker_api_spec.rb

data/HACKING

@@ -1,7 +0,0 @@
-HACKING on request-log-analyzer
--------------------------------
-
-- See DESIGN for the basic internal design of request-log-analyzer
-- See http://wiki.github.com/wvanbergen/request-log-analyzer/development for
-  more information about developing
-- Contact me at my GitHub account for any questions: http://github.com/wvanbergen