request-log-analyzer 1.3.7 → 1.4.0

data/spec/lib/mocks.rb

@@ -1,24 +1,24 @@
 module RequestLogAnalyzer::Spec::Mocks
-  
+
   def mock_source
     source = mock('RequestLogAnalyzer::Source::Base')
     source.stub!(:file_format).and_return(testing_format)
     source.stub!(:parsed_requests).and_return(2)
-    source.stub!(:skipped_requests).and_return(1)    
+    source.stub!(:skipped_requests).and_return(1)
     source.stub!(:parse_lines).and_return(10)
-    
+
     source.stub!(:warning=)
     source.stub!(:progress=)
     source.stub!(:source_changes=)
 
     source.stub!(:prepare)
     source.stub!(:finalize)
-        
+
     source.stub!(:each_request).and_return do |block|
       block.call(testing_format.request(:field => 'value1'))
       block.call(testing_format.request(:field => 'value2'))
     end
-    
+
     return source
   end
 
@@ -29,24 +29,25 @@ module RequestLogAnalyzer::Spec::Mocks
     mio.stub!(:write)
     return mio
   end
-  
+
   def mock_output
     output = mock('RequestLogAnalyzer::Output::Base')
     output.stub!(:header)
-    output.stub!(:footer)   
+    output.stub!(:footer)
     output.stub!(:puts)
     output.stub!(:<<)
     output.stub!(:colorize).and_return("Fancy text")
     output.stub!(:link)
     output.stub!(:title)
     output.stub!(:line)
-    output.stub!(:with_style)    
+    output.stub!(:with_style)
     output.stub!(:table).and_yield([])
     output.stub!(:io).and_return(mock_io)
-    
+    output.stub!(:options).and_return({})
+    output.stub!(:slice_results).and_return { |a| a } 
     return output
   end
-  
+
   def mock_database(*stubs)
     database = mock('RequestLogAnalyzer::Database')
     database.stub!(:connect)
@@ -55,11 +56,11 @@ module RequestLogAnalyzer::Spec::Mocks
     stubs.each { |s| database.stub!(s)}
     return database
   end
-  
+
   def mock_connection
     table_creator = mock('ActiveRecord table creator')
     table_creator.stub!(:column)
-    
+
     connection = mock('ActiveRecord::Base.connection')
     connection.stub!(:add_index)
     connection.stub!(:remove_index)
@@ -68,6 +69,6 @@ module RequestLogAnalyzer::Spec::Mocks
     connection.stub!(:table_creator).and_return(table_creator)
     return connection
   end
-  
-  
+
+
 end

data/spec/lib/testing_format.rb

@@ -5,42 +5,42 @@ class TestingFormat < RequestLogAnalyzer::FileFormat::Base
     line.header = true
     line.teaser = /processing /
     line.regexp = /processing request (\d+)/
-    line.captures = [{ :name => :request_no, :type => :integer }]    
+    line.captures = [{ :name => :request_no, :type => :integer }]
   end
-  
+
   format_definition.test do |line|
     line.teaser = /testing /
     line.regexp = /testing is (\w+)(?: in (\d+\.\d+)ms)?/
     line.captures = [{ :name => :test_capture, :type => :test_type },
                      { :name => :duration, :type => :duration, :unit => :msec }]
   end
-  
+
   format_definition.eval do |line|
     line.regexp = /evaluation (\{.*\})/
     line.captures = [{ :name => :evaluated, :type => :eval, :provides => { :greating => :string, :what => :string } }]
-  end 
-  
+  end
+
   format_definition.last do |line|
     line.footer = true
     line.teaser = /finishing /
     line.regexp = /finishing request (\d+)/
     line.captures = [{ :name => :request_no, :type => :integer }]
   end
-  
+
   format_definition.combined do |line|
     line.header = true
     line.footer = true
     line.regexp = /this is a header and footer line/
   end
-  
+
   report do |analyze|
     analyze.frequency :test_capture, :title => 'What is testing exactly?'
   end
-  
+
   class Request < RequestLogAnalyzer::Request
     def convert_test_type(value, definition)
       "Testing is #{value}"
     end
   end
-  
+
 end

data/spec/spec_helper.rb

@@ -9,16 +9,16 @@ module RequestLogAnalyzer::Spec
 end
 
 # Include all files in the spec_helper directory
-Dir[File.dirname(__FILE__) + "/lib/**/*.rb"].each do |file| 
-  require file 
+Dir[File.dirname(__FILE__) + "/lib/**/*.rb"].each do |file|
+  require file
 end
 
 Dir.mkdir("#{File.dirname(__FILE__)}/../tmp") unless File.exist?("#{File.dirname(__FILE__)}/../tmp")
 
 Spec::Runner.configure do |config|
-  config.include RequestLogAnalyzer::Spec::Matchers  
+  config.include RequestLogAnalyzer::Spec::Matchers
   config.include RequestLogAnalyzer::Spec::Mocks
-  config.include RequestLogAnalyzer::Spec::Helpers  
-  
-  config.extend RequestLogAnalyzer::Spec::Macros  
+  config.include RequestLogAnalyzer::Spec::Helpers
+
+  config.extend RequestLogAnalyzer::Spec::Macros
 end

data/spec/unit/aggregator/database_inserter_spec.rb

@@ -5,27 +5,27 @@ describe RequestLogAnalyzer::Aggregator::DatabaseInserter do
   before(:all) do
     @log_parser = RequestLogAnalyzer::Source::LogParser.new(testing_format)
   end
-  
+
   # The prepare method is called before the parsing starts. It should establish a connection
   # to a database that is suitable for inserting requests later on.
   describe '#prepare' do
-  
+
     before(:each) do
       @database = mock_database(:create_database_schema!, :drop_database_schema!, :file_format=)
       @database_inserter = RequestLogAnalyzer::Aggregator::DatabaseInserter.new(@log_parser)
       RequestLogAnalyzer::Database.stub!(:new).and_return(@database)
     end
-  
+
     it 'should establish the database connection' do
       RequestLogAnalyzer::Database.should_receive(:new).and_return(@database)
       @database_inserter.prepare
     end
-    
+
     it "should set the file_format" do
       @database.should_receive(:file_format=).with(testing_format)
       @database_inserter.prepare
     end
-  
+
     it 'should create the database schema during preparation' do
       @database.should_receive(:create_database_schema!)
       @database_inserter.prepare
@@ -52,28 +52,28 @@ describe RequestLogAnalyzer::Aggregator::DatabaseInserter do
         @database_inserter.prepare
 
         @incomplete_request = testing_format.request( {:line_type => :first, :request_no => 564})
-        @completed_request  = testing_format.request( {:line_type => :first, :request_no => 564}, 
+        @completed_request  = testing_format.request( {:line_type => :first, :request_no => 564},
                               {:line_type => :test, :test_capture => "awesome"},
-                              {:line_type => :test, :test_capture => "indeed"}, 
-                              {:line_type => :eval, :evaluated => { :greating => 'howdy'}, :greating => 'howdy' }, 
+                              {:line_type => :test, :test_capture => "indeed"},
+                              {:line_type => :eval, :evaluated => { :greating => 'howdy'}, :greating => 'howdy' },
                               {:line_type => :last, :request_no   => 564})
       end
-      
+
       after(:each) do
         @database_inserter.database.send :remove_orm_classes!
       end
 
       it "should insert a record in the request table" do
-        lambda { 
+        lambda {
           @database_inserter.aggregate(@incomplete_request)
         }.should change(RequestLogAnalyzer::Database::Request, :count).from(0).to(1)
       end
-      
+
       it "should insert a record in the first_lines table" do
-        lambda { 
+        lambda {
           @database_inserter.aggregate(@incomplete_request)
         }.should change(@database_inserter.database.get_class(:first), :count).from(0).to(1)
-      end      
+      end
 
       it "should insert records in all relevant line tables" do
         @database_inserter.aggregate(@completed_request)

data/spec/unit/aggregator/summarizer_spec.rb

@@ -6,11 +6,11 @@ describe RequestLogAnalyzer::Aggregator::Summarizer do
     @summarizer = RequestLogAnalyzer::Aggregator::Summarizer.new(mock_source, :output => mock_output)
     @summarizer.prepare
   end
-  
+
   it "not raise exception when creating a report after aggregating multiple requests" do
     @summarizer.aggregate(request(:data => 'bluh1'))
-    @summarizer.aggregate(request(:data => 'bluh2'))    
-    
+    @summarizer.aggregate(request(:data => 'bluh2'))
+
     lambda { @summarizer.report(mock_output) }.should_not raise_error
   end
 
@@ -22,5 +22,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/controller/controller_spec.rb

@@ -17,7 +17,7 @@ describe RequestLogAnalyzer::Controller do
   it "should call aggregators correctly when run" do
     controller  = RequestLogAnalyzer::Controller.new(mock_source, :output => mock_output)
     
-    mock_aggregator = mock('RequestLogAnalyzer::Aggregator::Base')
+    mock_aggregator = mock('RequestLogAnalyzer::Aggregator::Base') 
     mock_aggregator.should_receive(:prepare).once.ordered
     mock_aggregator.should_receive(:aggregate).with(an_instance_of(testing_format.request_class)).twice.ordered
     mock_aggregator.should_receive(:finalize).once.ordered
@@ -38,4 +38,4 @@ describe RequestLogAnalyzer::Controller do
     controller.run!
   end
   
-end
+end

data/spec/unit/controller/log_processor_spec.rb

@@ -7,7 +7,7 @@ describe RequestLogAnalyzer::LogProcessor, 'stripping log files' do
   before(:each) do
     @log_stripper = RequestLogAnalyzer::LogProcessor.new(testing_format, :strip, {})
   end
-  
+
   it "should remove a junk line" do
     @log_stripper.strip_line("junk line\n").should be_empty
   end

data/spec/unit/database/base_class_spec.rb

@@ -1,26 +1,26 @@
 require File.dirname(__FILE__) + '/../../spec_helper'
 
 describe RequestLogAnalyzer::Database::Base do
-  
+
   describe '.subclass_from_line_definition' do
     before(:all) do
       @line_definition = RequestLogAnalyzer::LineDefinition.new(:test, { :regexp   => /Testing (\w+), tries\: (\d+)/,
                           :captures => [{ :name => :what, :type => :string }, { :name => :tries, :type => :integer },
                             { :name => :evaluated, :type => :hash, :provides => {:evaluated_field => :duration} }]})
     end
-    
+
     before(:each) do
       @orm_class = mock('Line ActiveRecord::Base class')
       @orm_class.stub!(:set_table_name)
       @orm_class.stub!(:belongs_to)
       @orm_class.stub!(:serialize)
       @orm_class.stub!(:line_definition=)
-      
+
       Class.stub!(:new).with(RequestLogAnalyzer::Database::Base).and_return(@orm_class)
-      
+
       RequestLogAnalyzer::Database::Request.stub!(:has_many)
       RequestLogAnalyzer::Database::Source.stub!(:has_many)
-      
+
       @database = mock_database
       RequestLogAnalyzer::Database::Base.stub!(:database).and_return(@database)
     end
@@ -29,7 +29,7 @@ describe RequestLogAnalyzer::Database::Base do
       Class.should_receive(:new).with(RequestLogAnalyzer::Database::Base).and_return(@orm_class)
       RequestLogAnalyzer::Database::Base.subclass_from_line_definition(@line_definition)
     end
-    
+
     it "should store the LineDefinition" do
       @orm_class.should_receive(:line_definition=).with(@line_definition)
       RequestLogAnalyzer::Database::Base.subclass_from_line_definition(@line_definition)
@@ -69,26 +69,26 @@ describe RequestLogAnalyzer::Database::Base do
 
   describe '.subclass_from_table' do
     before(:each) do
-      
+
       RequestLogAnalyzer::Database::Request.stub!(:has_many)
-      RequestLogAnalyzer::Database::Source.stub!(:has_many)      
-      
+      RequestLogAnalyzer::Database::Source.stub!(:has_many)
+
       @database = mock_database
       @database.connection.stub!(:table_exists?).and_return(true)
       RequestLogAnalyzer::Database::Base.stub!(:database).and_return(@database)
-      
+
       @klass = mock('ActiveRecord ORM class')
-      @klass.stub!(:column_names).and_return(['id', 'request_id', 'source_id', 'lineno', 'duration']) 
+      @klass.stub!(:column_names).and_return(['id', 'request_id', 'source_id', 'lineno', 'duration'])
       @klass.stub!(:set_table_name)
       @klass.stub!(:belongs_to)
       Class.stub!(:new).with(RequestLogAnalyzer::Database::Base).and_return(@klass)
     end
-    
+
     it "should create a new subclass using the Base class as parent" do
       Class.should_receive(:new).with(RequestLogAnalyzer::Database::Base).and_return(@klass)
       RequestLogAnalyzer::Database::Base.subclass_from_table('completed_lines')
-    end    
-    
+    end
+
     it "should set the table name" do
       @klass.should_receive(:set_table_name).with('completed_lines')
       RequestLogAnalyzer::Database::Base.subclass_from_table('completed_lines')
@@ -98,7 +98,7 @@ describe RequestLogAnalyzer::Database::Base do
       @klass.should_receive(:belongs_to).with(:request)
       RequestLogAnalyzer::Database::Base.subclass_from_table('completed_lines')
     end
-    
+
     it "should create the :has_many relation in the request class" do
       RequestLogAnalyzer::Database::Request.should_receive(:has_many).with(:completed_lines)
       RequestLogAnalyzer::Database::Base.subclass_from_table('completed_lines')
@@ -108,7 +108,7 @@ describe RequestLogAnalyzer::Database::Base do
       @klass.should_receive(:belongs_to).with(:source)
       RequestLogAnalyzer::Database::Base.subclass_from_table('completed_lines')
     end
-    
+
     it "should create the :has_many relation in the request class" do
       RequestLogAnalyzer::Database::Source.should_receive(:has_many).with(:completed_lines)
       RequestLogAnalyzer::Database::Base.subclass_from_table('completed_lines')
@@ -131,7 +131,7 @@ describe RequestLogAnalyzer::Database::Base do
       @klass.stub!(:table_exists?).and_return(false)
     end
 
-    after(:each) do 
+    after(:each) do
       @klass.drop_table!
       @database.remove_orm_classes!
     end
@@ -170,12 +170,12 @@ describe RequestLogAnalyzer::Database::Base do
     it "should create a field of the correct type for every defined capture field" do
       @database.connection.table_creator.should_receive(:column).with(:what, :string)
       @database.connection.table_creator.should_receive(:column).with(:tries, :integer)
-      @database.connection.table_creator.should_receive(:column).with(:evaluated, :text) 
+      @database.connection.table_creator.should_receive(:column).with(:evaluated, :text)
       @klass.create_table!
     end
 
     it "should create a field of the correct type for every provided field" do
-      @database.connection.table_creator.should_receive(:column).with(:evaluated_field, :double) 
+      @database.connection.table_creator.should_receive(:column).with(:evaluated_field, :double)
       @klass.create_table!
     end
   end

data/spec/unit/database/connection_spec.rb

@@ -15,19 +15,19 @@ describe RequestLogAnalyzer::Database::Connection do
 
     it "should parse an URI-based string for MySQL" do
       string = 'mysql://localhost.local/database'
-      RequestLogAnalyzer::Database::Connection.from_string(string).should == 
+      RequestLogAnalyzer::Database::Connection.from_string(string).should ==
               { :adapter => 'mysql', :database => 'database', :host => 'localhost.local' }
     end
 
     it "should parse an URI-based string for MySQL with only username" do
       string = 'mysql://username@localhost.local/database'
-      RequestLogAnalyzer::Database::Connection.from_string(string).should == 
+      RequestLogAnalyzer::Database::Connection.from_string(string).should ==
               { :adapter => 'mysql', :database => 'database', :host => 'localhost.local', :username => 'username' }
     end
 
     it "should parse an URI-based string for MySQL with username and password" do
       string = 'mysql://username:password@localhost.local/database'
-      RequestLogAnalyzer::Database::Connection.from_string(string).should == 
+      RequestLogAnalyzer::Database::Connection.from_string(string).should ==
               { :adapter => 'mysql', :database => 'database', :host => 'localhost.local', :username => 'username', :password => 'password' }
     end
   end

data/spec/unit/database/database_spec.rb

@@ -1,17 +1,17 @@
 require File.dirname(__FILE__) + '/../../spec_helper'
 
 describe RequestLogAnalyzer::Database do
-  
+
   describe '#load_database_schema!' do
-    
+
     context 'for a Rails request database' do
       before(:each) do
         @database = RequestLogAnalyzer::Database.new(log_fixture(:rails, :db))
         @database.load_database_schema!
       end
-    
+
       after(:each) { @database.remove_orm_classes! }
-    
+
       # FileFormat-agnostic classes
       default_orm_class_names.each do |const|
         it "should create the default #{const} constant" do
@@ -22,7 +22,7 @@ describe RequestLogAnalyzer::Database do
           RequestLogAnalyzer::Database.const_get(const).ancestors.should include(ActiveRecord::Base, RequestLogAnalyzer::Database::Base)
         end
       end
-    
+
       # Some Fileformat-specific classes
       ['CompletedLine', 'ProcessingLine'].each do |const|
         it "should create the #{const} constant" do
@@ -32,11 +32,11 @@ describe RequestLogAnalyzer::Database do
         it "should create the #{const} class inheriting from ActiveRecord::Base and RequestLogAnalyzer::Database::Base" do
           Object.const_get(const).ancestors.should include(ActiveRecord::Base, RequestLogAnalyzer::Database::Base)
         end
-      
+
         it "should create a :belongs_to relation from the #{const} class to Request and Source" do
           Object.const_get(const).send(:reflections).should include(:request, :source)
         end
-        
+
         it "should create a :has_many relation from the Request and Source class to the #{const} class" do
           RequestLogAnalyzer::Database::Request.send(:reflections).should include(const.underscore.pluralize.to_sym)
           RequestLogAnalyzer::Database::Source.send(:reflections).should include(const.underscore.pluralize.to_sym)
@@ -44,7 +44,7 @@ describe RequestLogAnalyzer::Database do
       end
     end
   end
-  
+
   describe '#create_database_schema!' do
 
     before(:each) do
@@ -56,11 +56,11 @@ describe RequestLogAnalyzer::Database do
       @mock_class = Class.new(RequestLogAnalyzer::Database::Base)
       @mock_class.stub!(:create_table!)
     end
-    
+
     after(:each) { @database.remove_orm_classes! }
 
     default_orm_class_names.each do |klass|
-      
+
       it "should create a table for the default #{klass} class" do
         @database.connection.should_receive(:create_table).with(klass.underscore.pluralize.to_sym)
         @database.send :create_database_schema!
@@ -73,58 +73,58 @@ describe RequestLogAnalyzer::Database do
     end
 
     testing_format.line_definitions.each do |name, definition|
-      
+
       it "should create the #{(name.to_s + '_line').camelize} class for #{name.inspect} lines" do
         @database.send :create_database_schema!
         Object.const_defined?("#{name}_line".camelize).should be_true
       end
-      
+
       it "should create the #{name.to_s + '_lines'} table for the parsed #{name.inspect} lines" do
         @database.connection.should_receive(:create_table).with("#{name}_lines".to_sym)
         @database.send :create_database_schema!
       end
     end
   end
-  
+
   describe '#load_activerecord_class' do
-    
-    before(:each) do 
+
+    before(:each) do
       @database = RequestLogAnalyzer::Database.new
       @connection = mock_connection
       @database.stub!(:connection).and_return(@connection)
-      
+
       # Mock the has_many method of the defaukt ORM classes
       RequestLogAnalyzer::Database::Request.stub!(:has_many)
       RequestLogAnalyzer::Database::Source.stub!(:has_many)
-      
+
       @mock_class = Class.new(RequestLogAnalyzer::Database::Base)
-      
+
       RequestLogAnalyzer::Database::Base.stub!(:subclass_from_table).and_return(@mock_class)
       RequestLogAnalyzer::Database::Base.stub!(:subclass_from_line_definition).and_return(@mock_class)
     end
-    
+
     after(:each) { @database.remove_orm_classes! }
-    
+
     it "should call :subclass_from_table when a table name is given as string" do
       RequestLogAnalyzer::Database::Base.should_receive(:subclass_from_table).and_return(@mock_class)
       @database.load_activerecord_class('test_lines')
     end
-    
+
     it "should call :subclass_from_table when a table name is given as symbol" do
       RequestLogAnalyzer::Database::Base.should_receive(:subclass_from_table).and_return(@mock_class)
       @database.load_activerecord_class(:test_lines)
     end
-    
+
     it "should call :subclass_from_table when a LineDefinition is given" do
       RequestLogAnalyzer::Database::Base.should_receive(:subclass_from_line_definition).and_return(@mock_class)
       @database.load_activerecord_class(RequestLogAnalyzer::LineDefinition.new(:test))
-    end    
-    
+    end
+
     it "should define the class in the ORM module" do
       @database.load_activerecord_class(:test_lines)
       Object.const_defined?('TestLine').should be_true
     end
-    
+
     it "should add the class to the line_classes array of the database" do
       @database.load_activerecord_class(:test_lines)
       @database.line_classes.should include(TestLine)