request-log-analyzer 1.3.7 → 1.4.0

data/spec/unit/filter/anonymize_filter_spec.rb

@@ -5,7 +5,7 @@ describe RequestLogAnalyzer::Filter::Anonymize, 'anonymize request' do
   before(:each) do
     @filter = RequestLogAnalyzer::Filter::Anonymize.new(testing_format)
   end
-  
+
   it "should anonimize ip" do
     @filter.filter(request(:ip => '123.123.123.123'))[:ip].should_not eql('123.123.123.123')
   end
@@ -17,5 +17,5 @@ describe RequestLogAnalyzer::Filter::Anonymize, 'anonymize request' do
   it "should fuzz durations" do
     @filter.filter(request(:duration => 100))[:duration].should_not eql(100)
   end
-  
+
 end

data/spec/unit/filter/field_filter_spec.rb

@@ -5,22 +5,22 @@ describe RequestLogAnalyzer::Filter::Field, 'string in accept mode' do
   before(:each) do
     @filter = RequestLogAnalyzer::Filter::Field.new(testing_format, :field => :test, :value => 'test', :mode => :select)
   end
-  
+
   it "should reject a request if the field value does not match" do
     @filter.filter(request(:test => 'not test')).should be_nil
   end
-  
+
   it "should reject a request if the field name does not match" do
     @filter.filter(request(:testing => 'test')).should be_nil
   end
 
   it "should accept a request if the both name and value match" do
     @filter.filter(request(:test => 'test')).should_not be_nil
-  end 
-    
+  end
+
   it "should accept a request if the value is not the first value" do
     @filter.filter(request([{:test => 'ignore'}, {:test => 'test'}])).should_not be_nil
-  end  
+  end
 end
 
 describe RequestLogAnalyzer::Filter::Field, 'string in reject mode' do
@@ -28,22 +28,22 @@ describe RequestLogAnalyzer::Filter::Field, 'string in reject mode' do
   before(:each) do
     @filter = RequestLogAnalyzer::Filter::Field.new(testing_format, :field => :test, :value => 'test', :mode => :reject)
   end
-  
+
   it "should accept a request if the field value does not match" do
     @filter.filter(request(:test => 'not test')).should_not be_nil
   end
-  
+
   it "should accept a request if the field name does not match" do
     @filter.filter(request(:testing => 'test')).should_not be_nil
   end
 
   it "should reject a request if the both name and value match" do
     @filter.filter(request(:test => 'test')).should be_nil
-  end 
-    
+  end
+
   it "should reject a request if the value is not the first value" do
     @filter.filter(request([{:test => 'ignore'}, {:test => 'test'}])).should be_nil
-  end  
+  end
 end
 
 describe RequestLogAnalyzer::Filter::Field, 'regexp in accept mode' do
@@ -51,16 +51,16 @@ describe RequestLogAnalyzer::Filter::Field, 'regexp in accept mode' do
   before(:each) do
     @filter = RequestLogAnalyzer::Filter::Field.new(testing_format, :field => :test, :value => '/test/', :mode => :select)
   end
-  
+
   it "should reject a request if the field value does not match" do
     @filter.filter(request(:test => 'a working test')).should_not be_nil
   end
-  
+
   it "should reject a request if the field name does not match" do
     @filter.filter(request(:testing => 'test')).should be_nil
   end
 
   it "should accept a request if the value is not the first value" do
     @filter.filter(request([{:test => 'ignore'}, {:test => 'testing 123'}])).should_not be_nil
-  end  
+  end
 end

data/spec/unit/filter/filter_spec.rb

@@ -5,7 +5,7 @@ describe RequestLogAnalyzer::Filter::Base, 'base filter' do
   before(:each) do
     @filter = RequestLogAnalyzer::Filter::Base.new(testing_format)
   end
-  
+
   it "should return everything" do
     @filter.filter(request(:ip => '123.123.123.123'))[:ip].should eql('123.123.123.123')
   end

data/spec/unit/filter/timespan_filter_spec.rb

@@ -5,54 +5,54 @@ describe RequestLogAnalyzer::Filter::Timespan, 'both before and after'  do
   before(:each) do
     @filter = RequestLogAnalyzer::Filter::Timespan.new(testing_format, :after => DateTime.parse('2009-01-01'), :before => DateTime.parse('2009-02-02'))
   end
-  
+
   it "should reject a request before the after date" do
     @filter.filter(request(:timestamp => 20081212000000)).should be_nil
   end
-  
+
   it "should reject a request after the before date" do
     @filter.filter(request(:timestamp => 20090303000000)).should be_nil
   end
-  
+
   it "should accept a request between the after and before dates" do
     @filter.filter(request(:timestamp => 20090102000000)).should_not be_nil
   end
 end
 
 describe RequestLogAnalyzer::Filter::Timespan, 'only before'  do
-  
+
   before(:each) do
-    @filter = RequestLogAnalyzer::Filter::Timespan.new(testing_format, :before => DateTime.parse('2009-02-02'))    
+    @filter = RequestLogAnalyzer::Filter::Timespan.new(testing_format, :before => DateTime.parse('2009-02-02'))
   end
-  
+
   it "should accept a request before the after date" do
     @filter.filter(request(:timestamp => 20081212000000)).should_not be_nil
   end
-  
+
   it "should reject a request after the before date" do
     @filter.filter(request(:timestamp => 20090303000000)).should be_nil
   end
-  
+
   it "should accept a request between the after and before dates" do
     @filter.filter(request(:timestamp => 20090102000000)).should_not be_nil
-  end   
+  end
 end
 
-describe RequestLogAnalyzer::Filter::Timespan, 'only after'  do  
-  
+describe RequestLogAnalyzer::Filter::Timespan, 'only after'  do
+
   before(:each) do
     @filter = RequestLogAnalyzer::Filter::Timespan.new(testing_format, :after => DateTime.parse('2009-01-01'))
   end
-  
+
   it "should reject a request before the after date" do
     @filter.filter(request(:timestamp => 20081212000000)).should be_nil
   end
-  
+
   it "should accept a request after the before date" do
     @filter.filter(request(:timestamp => 20090303000000)).should_not be_nil
   end
-  
+
   it "should accept a request between the after and before dates" do
     @filter.filter(request(:timestamp => 20090102000000)).should_not be_nil
-  end  
+  end
 end

data/spec/unit/mailer_spec.rb

@@ -0,0 +1,30 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe RequestLogAnalyzer::Mailer, 'mailer' do
+
+  it "should store printed data" do
+    @mailer = RequestLogAnalyzer::Mailer.new('alfa@beta.com', 'localhost', :debug => true)
+
+    @mailer << 'test1'
+    @mailer.puts 'test2'
+    
+    @mailer.data.should eql(['test1', 'test2'])
+  end
+
+  it "should send mail" do
+    @mailer = RequestLogAnalyzer::Mailer.new('alfa@beta.com', 'localhost', :debug => true)
+
+    @mailer << 'test1'
+    @mailer.puts 'test2'
+
+    mail = @mailer.mail
+    
+    mail[0].should include("contact@railsdoctors.com")
+    mail[0].should include("test1")
+    mail[0].should include("test2")
+
+    mail[1].should include("contact@railsdoctors.com")
+    mail[2].should include("alfa@beta.com")
+  end
+
+end

data/spec/unit/{source/request_spec.rb → request_spec.rb}

@@ -1,49 +1,49 @@
-require File.dirname(__FILE__) + '/../../spec_helper'
+require File.dirname(__FILE__) + '/../spec_helper'
 
 describe RequestLogAnalyzer::Request do
-  
+
   before(:each) do
     @request = testing_format.request
-  end  
-  
+  end
+
   it "should be empty without any captured lines in it" do
     @request.should be_empty
   end
-  
+
   context :incomplete do
-  
+
     before(:each) do
       @request << { :line_type => :test, :lineno => 1, :test_capture => 'awesome!' }
     end
-  
+
     it "should be single if only one line has been added" do
       @request.should_not be_empty
     end
-  
+
     it "should not be a completed request" do
       @request.should_not be_completed
-    end  
-  
+    end
+
     it "should take the line type of the first line as global line_type" do
       @request.lines[0][:line_type].should == :test
       @request.should =~ :test
     end
-  
+
     it "should return the first field value" do
       @request[:test_capture].should == 'awesome!'
     end
-  
+
     it "should return nil if no such field is present" do
       @request[:nonexisting].should be_nil
     end
   end
-  
+
   context :completed do
-    
+
     before(:each) do
-      @request << { :line_type => :first, :lineno =>  1, :name => 'first line!' }    
-      @request << { :line_type => :test,  :lineno =>  4, :test_capture => 'testing' }        
-      @request << { :line_type => :test,  :lineno =>  7, :test_capture => 'testing some more' }            
+      @request << { :line_type => :first, :lineno =>  1, :name => 'first line!' }
+      @request << { :line_type => :test,  :lineno =>  4, :test_capture => 'testing' }
+      @request << { :line_type => :test,  :lineno =>  7, :test_capture => 'testing some more' }
       @request << { :line_type => :last,  :lineno => 10, :time => 0.03 }
     end
 
@@ -64,29 +64,29 @@ describe RequestLogAnalyzer::Request do
       @request[:time].should == 0.03
     end
 
-    it "should detect the first matching field value" do  
+    it "should detect the first matching field value" do
       @request.first(:test_capture).should == 'testing'
     end
 
-    it "should detect the every matching field value" do  
+    it "should detect the every matching field value" do
       @request.every(:test_capture).should == ['testing', "testing some more"]
     end
 
     it "should set the first_lineno for a request to the lowest lineno encountered" do
       @request.first_lineno.should eql(1)
     end
-    
+
     it "should set the first_lineno for a request if a line with a lower lineno is added" do
-      @request << { :line_type => :test, :lineno =>  0 }          
+      @request << { :line_type => :test, :lineno =>  0 }
       @request.first_lineno.should eql(0)
-    end    
+    end
 
     it "should set the last_lineno for a request to the highest encountered lineno" do
       @request.last_lineno.should eql(10)
     end
-    
+
     it "should not set the last_lineno for a request if a line with a lower lineno is added" do
-      @request << { :line_type => :test, :lineno =>  7 }          
+      @request << { :line_type => :test, :lineno =>  7 }
       @request.last_lineno.should eql(10)
     end
 
@@ -95,17 +95,17 @@ describe RequestLogAnalyzer::Request do
     end
 
     it "should set return a timestamp field if such a field is captured" do
-      @request << { :line_type => :first, :lineno =>  1, :name => 'first line!', :timestamp => Time.now}    
+      @request << { :line_type => :first, :lineno =>  1, :name => 'first line!', :timestamp => Time.now}
       @request.timestamp.should_not be_nil
-    end    
+    end
   end
-  
+
   context 'single line' do
     # combined is both a header and a footer line
     before(:each) { @request << { :line_type => :combined, :lineno => 1 } }
-  
+
     it "should be a completed request if the line is both header and footer" do
       @request.should be_completed
-    end   
+    end
   end
-end  
+end

data/spec/unit/source/log_parser_spec.rb

@@ -1,60 +1,60 @@
 require File.dirname(__FILE__) + '/../../spec_helper'
 
 describe RequestLogAnalyzer::Source::LogParser, :requests do
-  
+
   before(:each) do
     @log_parser = RequestLogAnalyzer::Source::LogParser.new(testing_format)
   end
-  
+
   it "should have multiple line definitions" do
     @log_parser.file_format.line_definitions.length.should >= 2
-  end  
-  
+  end
+
   it "should have a valid language" do
     @log_parser.file_format.should be_valid
   end
-  
+
   it "should set the :source for every parsed line" do
     @log_parser.parse_file(log_fixture(:rails_22)) do |request|
       request.lines.all? { |line| line[:source] == log_fixture(:rails_22) }.should be_true
     end
   end
-  
+
   it "should set the :lineno for every parsed line" do
     @log_parser.parse_file(log_fixture(:rails_22)) do |request|
       request.lines.all? { |line| line.has_key?(:lineno) }.should be_true
     end
   end
-  
+
   it "should parse more lines than requests" do
     @log_parser.should_receive(:handle_request).with(an_instance_of(TestingFormat::Request)).twice
     @log_parser.parse_file(log_fixture(:test_language_combined))
-    @log_parser.parsed_lines.should > 2    
+    @log_parser.parsed_lines.should > 2
   end
-  
+
   it "should parse requests spanned over multiple files" do
     @log_parser.should_receive(:handle_request).with(an_instance_of(TestingFormat::Request)).once
     @log_parser.parse_files([log_fixture(:multiple_files_1), log_fixture(:multiple_files_2)])
   end
-  
+
   it "should parse all request values when spanned over multiple files" do
     @log_parser.parse_files([log_fixture(:multiple_files_1), log_fixture(:multiple_files_2)]) do |request|
-      request.lines.should have(4).items      
+      request.lines.should have(4).items
       request[:request_no].should == 1
-      request[:test_capture].should == "Testing is amazing" # Note the custom converter     
+      request[:test_capture].should == "Testing is amazing" # Note the custom converter
     end
-  end  
-  
+  end
+
   it "should parse a stream and find valid requests" do
     io = File.new(log_fixture(:test_file_format), 'r')
     @log_parser.parse_io(io) do |request|
       request.should be_kind_of(RequestLogAnalyzer::Request)
       request.should =~ :test
-      request[:test_capture].should_not be_nil      
+      request[:test_capture].should_not be_nil
     end
     io.close
   end
-  
+
   it "should parse a request that only consists of one line" do
     @log_parser.parse_file(log_fixture(:header_and_footer))
     @log_parser.parsed_requests.should == 2
@@ -62,30 +62,30 @@ describe RequestLogAnalyzer::Source::LogParser, :requests do
 end
 
 describe RequestLogAnalyzer::Source::LogParser, :warnings do
-  
+
   before(:each) do
     @log_parser = RequestLogAnalyzer::Source::LogParser.new(testing_format, :parse_strategy => 'cautious')
   end
-    
+
   it "should warn about teaser matching problems" do
     @log_parser.should_receive(:warn).with(:teaser_check_failed, anything).exactly(5).times
     @log_parser.parse_file(log_fixture(:test_file_format))
   end
-  
+
   it "should warn about unmatching request headers and footers" do
-    @log_parser.should_receive(:warn).with(:unclosed_request, anything).at_least(1).times    
-    @log_parser.should_receive(:warn).with(:no_current_request, anything).at_least(1).times    
-    @log_parser.should_not_receive(:handle_request)    
-    @log_parser.parse_file(log_fixture(:test_order))    
-  end  
+    @log_parser.should_receive(:warn).with(:unclosed_request, anything).at_least(1).times
+    @log_parser.should_receive(:warn).with(:no_current_request, anything).at_least(1).times
+    @log_parser.should_not_receive(:handle_request)
+    @log_parser.parse_file(log_fixture(:test_order))
+  end
 end
 
 describe RequestLogAnalyzer::Source::LogParser, :decompression do
-  
+
   before(:each) do
     @log_parser = RequestLogAnalyzer::Source::LogParser.new(RequestLogAnalyzer::FileFormat::Rails.create)
   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"))
@@ -115,5 +115,5 @@ describe RequestLogAnalyzer::Source::LogParser, :decompression do
     @log_parser.parse_file(log_fixture(:decompression, "log.zip"))
     @log_parser.parsed_lines.should > 0
   end
-  
+
 end