request-log-analyzer 1.4.0 → 1.5.0

data/spec/integration/scout_spec.rb

@@ -0,0 +1,152 @@
+require File.dirname(__FILE__) + '/../spec_helper.rb'
+
+def capture_stdout_and_stderr_with_warnings_on
+  $stdout, $stderr, warnings, $VERBOSE =
+    StringIO.new, StringIO.new, $VERBOSE, true
+  yield
+  return $stdout.string, $stderr.string
+ensure
+  $stdout, $stderr, $VERBOSE = STDOUT, STDERR, warnings
+end
+
+describe RequestLogAnalyzer, 'when using the rla API like the scout plugin' do
+  
+  before(:each) do
+    # prepare a place to capture the output
+    sio = StringIO.new
+    
+    # place an IO object where I want RequestLogAnalyzer to read from
+    open(log_fixture(:rails_1x)) do |log|
+      completed_count = 0
+      log.each do |line|
+        completed_count += 1 if line =~ /\ACompleted\b/
+        break if completed_count == 2  # skipping first two requests
+      end
+      
+      # trigger the log parse
+      @stdout, @stderr = capture_stdout_and_stderr_with_warnings_on do
+        RequestLogAnalyzer::Controller.build(
+          :output       => EmbeddedHTML,
+          :file         => sio,
+          :after        => Time.local(2008, 8, 14, 21, 16, 31),  # after 3rd req
+          :source_files => log,
+          :format        => RequestLogAnalyzer::FileFormat::Rails
+        ).run!
+      end
+    end
+    
+    # read the resulting output
+    @analysis = sio.string
+  end
+  
+  it "should generate an analysis" do
+    @analysis.should_not be_empty
+  end
+  
+  it "should generate customized output using the passed Class" do
+    credit = %r{<p>Powered by request-log-analyzer v\d+(?:\.\d+)+</p>\z}
+    @analysis.should match(credit)
+  end
+  
+  it "should skip requests before :after Time" do
+    @analysis.should_not include("PeopleController#show")
+  end
+  
+  it "should include requests after IO#pos and :after Time" do
+    @analysis.should include("PeopleController#picture")
+  end
+  
+  it "should skip requests before IO#pos" do
+    @analysis.should_not include("PeopleController#index")
+  end
+  
+  it "should not print to $stdout" do
+    @stdout.should be_empty
+  end
+  
+  it "should not print to $stderr (with warnings on)" do
+    @stderr.should be_empty
+  end
+
+end
+
+# Helpers
+class EmbeddedHTML < RequestLogAnalyzer::Output::Base
+  def print(str)
+    @io << str
+  end
+  alias_method :<<, :print
+
+  def puts(str = "")
+    @io << "#{str}<br/>\n"
+  end
+
+  def title(title)
+    @io.puts(tag(:h2, title))
+  end
+
+  def line(*font)  
+    @io.puts(tag(:hr))
+  end
+
+  def link(text, url = nil)
+    url = text if url.nil?
+    tag(:a, text, :href => url)
+  end
+
+  def table(*columns, &block)
+    rows = Array.new
+    yield(rows)
+
+    @io << tag(:table, :cellspacing => 0) do |content|
+      if table_has_header?(columns)
+        content << tag(:tr) do
+          columns.map { |col| tag(:th, col[:title]) }.join("\n")
+        end
+      end
+
+      odd = false
+      rows.each do |row|
+        odd = !odd
+        content << tag(:tr) do
+          if odd
+            row.map { |cell| tag(:td, cell, :class => "alt") }.join("\n") 
+          else
+            row.map { |cell| tag(:td, cell) }.join("\n") 
+          end
+        end
+      end
+    end
+  end
+
+  def header
+  end
+
+  def footer
+    @io << tag(:hr) << tag(:p, "Powered by request-log-analyzer v#{RequestLogAnalyzer::VERSION}")
+  end
+
+  private
+
+  def tag(tag, content = nil, attributes = nil)
+    if block_given?
+      attributes = content.nil? ? "" : " " + content.map { |(key, value)| "#{key}=\"#{value}\"" }.join(" ")
+      content_string = ""
+      content = yield(content_string)
+      content = content_string unless content_string.empty? 
+      "<#{tag}#{attributes}>#{content}</#{tag}>"
+    else
+      attributes = attributes.nil? ? "" : " " + attributes.map { |(key, value)| "#{key}=\"#{value}\"" }.join(" ")
+      if content.nil?
+        "<#{tag}#{attributes} />"
+      else
+        if content.class == Float
+          "<#{tag}#{attributes}><div class='color_bar' style=\"width:#{(content*200).floor}px;\"/></#{tag}>"
+        else
+          "<#{tag}#{attributes}>#{content}</#{tag}>"
+        end
+      end
+    end
+  end  
+end
+

data/spec/lib/helpers.rb

@@ -10,6 +10,11 @@ module RequestLogAnalyzer::Spec::Helpers
     File.dirname(__FILE__) + "/../fixtures/#{name}.#{extention}"
   end
 
+  # Creates a log file given some lines
+  def log_stream(*lines)
+    StringIO.new(lines.join("\n") + "\n")
+  end
+
   # Request loopback
   def request(fields, format = testing_format)
     if fields.kind_of?(Array)
@@ -42,6 +47,26 @@ module RequestLogAnalyzer::Spec::Helpers
 
   # 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"
+    File.expand_path("#{File.dirname(__FILE__)}/../../tmp/spec.#{file_type}.tmp")
+  end
+  
+  # Check if a given string can be found in the given file
+  # Returns the line number if found, nil otherwise
+  def find_string_in_file(string, file, options = {})
+    return nil unless File::exists?(file)
+    
+    line_counter = 0
+
+    File.open( file ) do |io|
+      io.each {|line|
+        line_counter += 1
+        line.chomp!
+
+        p line if options[:debug]
+        return line_counter if line.include? string
+      }
+    end
+    
+    return nil
   end
 end

data/spec/lib/mocks.rb

@@ -70,5 +70,7 @@ module RequestLogAnalyzer::Spec::Mocks
     return connection
   end
 
-
+  def request_counter
+    @request_counter ||= mock('aggregator to count request')
+  end
 end

data/spec/unit/file_format/apache_format_spec.rb

@@ -51,146 +51,153 @@ describe RequestLogAnalyzer::FileFormat::Apache do
 
   context '"Common" access log parsing' do
     before(:all) do
-      @file_format = RequestLogAnalyzer::FileFormat.load(:apache, :common)
-      @log_parser = RequestLogAnalyzer::Source::LogParser.new(@file_format)
-      @sample_1 = '1.129.119.13 - - [08/Sep/2009:07:54:09 -0400] "GET /profile/18543424 HTTP/1.0" 200 8223'
-      @sample_2 = '1.82.235.29 - - [08/Sep/2009:07:54:05 -0400] "GET /gallery/fresh?page=23&per_page=16 HTTP/1.1" 200 23414'
+      @file_format     = RequestLogAnalyzer::FileFormat.load(:apache, :common)
+      @log_parser      = RequestLogAnalyzer::Source::LogParser.new(@file_format)
+      @sample_1        = '1.129.119.13 - - [08/Sep/2009:07:54:09 -0400] "GET /profile/18543424 HTTP/1.0" 200 8223'
+      @sample_2        = '1.82.235.29 - - [08/Sep/2009:07:54:05 -0400] "GET /gallery/fresh?page=23&per_page=16 HTTP/1.1" 200 23414'
+      @nonsense_sample = 'addasdsasadadssadasd'
     end
 
     it "should have a valid language definitions" do
       @file_format.should be_valid
     end
 
-    it "should parse a valid access log line" do
-      @file_format.line_definitions[:access].matches(@sample_1).should be_kind_of(Hash)
-    end
-
     it "should not parse a valid access log line" do
-      @file_format.line_definitions[:access].matches('addasdsasadadssadasd').should be_false
+      @file_format.should_not parse_line(@nonsense_sample)
     end
 
     it "should read the correct values from a valid HTTP/1.0 access log line" do
-      @log_parser.parse_io(StringIO.new(@sample_1)) do |request|
-        request[:remote_host].should  == '1.129.119.13'
-        request[:timestamp].should    == 20090908075409
-        request[:http_status].should  == 200
-        request[:http_method].should  == 'GET'
-        request[:http_version].should == '1.0'
-        request[:bytes_sent].should   == 8223
-        request[:user].should         == nil
-      end
+      @file_format.should parse_line(@sample_1).as(:access).and_capture(
+        :remote_host    => '1.129.119.13',
+        :remote_logname => nil,
+        :user           => nil,
+        :timestamp      => 20090908075409,
+        :http_status    => 200,
+        :http_method    => 'GET',
+        :http_version   => '1.0',
+        :bytes_sent     => 8223)
     end
 
     it "should read the correct values from a valid 200 access log line" do
-      @log_parser.parse_io(StringIO.new(@sample_2)) do |request|
-        request[:remote_host].should  == '1.82.235.29'
-        request[:timestamp].should    == 20090908075405
-        request[:http_status].should  == 200
-        request[:http_method].should  == 'GET'
-        request[:http_version].should == '1.1'
-        request[:bytes_sent].should   == 23414
-        request[:user].should         == nil
+      @file_format.should parse_line(@sample_2).as(:access).and_capture(
+        :remote_host    => '1.82.235.29',
+        :remote_logname => nil,
+        :user           => nil,
+        :timestamp      => 20090908075405,
+        :http_status    => 200,
+        :http_method    => 'GET',
+        :http_version   => '1.1',
+        :bytes_sent     => 23414)
+    end
+
+    it "should parse 10 request from fixture access log without warnings" do
+      request_counter.should_receive(:hit!).exactly(10).times
+      @log_parser.should_not_receive(:warn)
+      
+      @log_parser.parse_file(log_fixture(:apache_common)) do |request| 
+        request_counter.hit! if request.kind_of?(RequestLogAnalyzer::FileFormat::Apache::Request)
       end
     end
-
-    it "should parse 10 request from fixture access log" do
-      counter = mock('counter')
-      counter.should_receive(:hit!).exactly(10).times
-      @log_parser.parse_file(log_fixture(:apache_common)) { counter.hit! }
-    end
   end
 
-  context '"Rack" access log parser' do
-    before(:each) do
-      @file_format = RequestLogAnalyzer::FileFormat.load(:rack)
-      @log_parser  = RequestLogAnalyzer::Source::LogParser.new(@file_format)
-      @sample_1 = '127.0.0.1 - - [16/Sep/2009 06:40:08] "GET /favicon.ico HTTP/1.1" 500 63183 0.0453'
-    end
+  context '"Combined" access log parsing' do
 
-    it "should create a kind of an Apache file format" do
-      @file_format.should be_kind_of(RequestLogAnalyzer::FileFormat::Apache)
+    before(:all) do
+      @file_format = RequestLogAnalyzer::FileFormat.load(:apache, :combined)
+      @log_parser  = RequestLogAnalyzer::Source::LogParser.new(@file_format)
+      @sample_1 = '69.41.0.45 - - [02/Sep/2009:12:02:40 +0200] "GET //phpMyAdmin/ HTTP/1.1" 404 209 "-" "Mozilla/4.0 (compatible; MSIE 6.0; Windows 98)"'
+      @sample_2 = '10.0.1.1 - - [02/Sep/2009:05:08:33 +0200] "GET / HTTP/1.1" 200 30 "-" "Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; en-us) AppleWebKit/531.9 (KHTML, like Gecko) Version/4.0.3 Safari/531.9"'
+      @nonsense_sample = 'addasdsasadadssadasd'
     end
 
     it "should have a valid language definitions" do
       @file_format.should be_valid
     end
 
-    it "should parse a valid access log line" do
-      @file_format.line_definitions[:access].matches(@sample_1).should be_kind_of(Hash)
-    end
-
     it "should not parse a valid access log line" do
-      @file_format.line_definitions[:access].matches('addasdsasadadssadasd').should be_false
+      @file_format.should_not parse_line(@nonsense_sample)
     end
 
     it "should read the correct values from a valid 404 access log line" do
-      @log_parser.parse_io(StringIO.new(@sample_1)) do |request|
-        request[:remote_host].should  == '127.0.0.1'
-        request[:timestamp].should    == 20090916064008
-        request[:http_status].should  == 500
-        request[:http_method].should  == 'GET'
-        request[:http_version].should == '1.1'
-        request[:bytes_sent].should   == 63183
-        request[:user].should         == nil
-        request[:duration].should     == 0.0453
+      @file_format.should parse_line(@sample_1).as(:access).and_capture(
+        :remote_host    => '69.41.0.45',
+        :remote_logname => nil,
+        :user           => nil,
+        :timestamp      => 20090902120240,
+        :http_status    => 404,
+        :http_method    => 'GET',
+        :http_version   => '1.1',
+        :bytes_sent     => 209,
+        :referer        => nil,
+        :user_agent     => 'Mozilla/4.0 (compatible; MSIE 6.0; Windows 98)')
+    end
+
+    it "should read the correct values from a valid 200 access log line" do
+      @file_format.should parse_line(@sample_2).as(:access).and_capture(
+        :remote_host    => '10.0.1.1',
+        :remote_logname => nil,
+        :user           => nil,
+        :timestamp      => 20090902050833,
+        :http_status    => 200,
+        :http_method    => 'GET',
+        :http_version   => '1.1',
+        :bytes_sent     => 30,
+        :referer        => nil,
+        :user_agent     => 'Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; en-us) AppleWebKit/531.9 (KHTML, like Gecko) Version/4.0.3 Safari/531.9')
+    end
+
+    it "should parse 5 request from fixture access log without warnings" do
+      request_counter.should_receive(:hit!).exactly(5).times
+      @log_parser.should_not_receive(:warn)
+      
+      @log_parser.parse_file(log_fixture(:apache_combined)) do |request|
+        request_counter.hit! if request.kind_of?(RequestLogAnalyzer::FileFormat::Apache::Request)
       end
     end
   end
-
-  context '"Combined" access log parsing' do
-
-    before(:all) do
-      @file_format = RequestLogAnalyzer::FileFormat.load(:apache, :combined)
+  
+  context '"Rack" access log parser' do
+    before(:each) do
+      @file_format = RequestLogAnalyzer::FileFormat.load(:rack)
       @log_parser  = RequestLogAnalyzer::Source::LogParser.new(@file_format)
-      @sample_1 = '69.41.0.45 - - [02/Sep/2009:12:02:40 +0200] "GET //phpMyAdmin/ HTTP/1.1" 404 209 "-" "Mozilla/4.0 (compatible; MSIE 6.0; Windows 98)"'
-      @sample_2 = '10.0.1.1 - - [02/Sep/2009:05:08:33 +0200] "GET / HTTP/1.1" 200 30 "-" "Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; en-us) AppleWebKit/531.9 (KHTML, like Gecko) Version/4.0.3 Safari/531.9"'
+      @sample_1 = '127.0.0.1 - - [16/Sep/2009 07:40:08] "GET /favicon.ico HTTP/1.1" 500 63183 0.0453'
+      @sample_2 = '127.0.0.1 - - [01/Oct/2009 07:58:10] "GET / HTTP/1.1" 200 1 0.0045'
+      @nonsense_sample = 'addasdsasadadssadasd'
+    end
+
+    it "should create a kind of an Apache file format" do
+      @file_format.should be_kind_of(RequestLogAnalyzer::FileFormat::Apache)
     end
 
     it "should have a valid language definitions" do
       @file_format.should be_valid
     end
 
-    it "should parse a valid access log line" do
-      @file_format.line_definitions[:access].matches(@sample_1).should be_kind_of(Hash)
+    it "should parse a valid access log line with status 500" do
+      @file_format.should parse_line(@sample_1).as(:access).and_capture(
+        :remote_host => '127.0.0.1', :timestamp => 20090916074008, :user => nil,
+        :http_status => 500,         :http_method => 'GET',        :http_version => '1.1',
+        :duration => 0.0453,         :bytes_sent => 63183,         :remote_logname => nil)
     end
 
-    it "should not parse a valid access log line" do
-      @file_format.line_definitions[:access].matches('addasdsasadadssadasd').should be_false
+    it "should parse a valid access log line with status 200" do
+      @file_format.should parse_line(@sample_2).as(:access).and_capture(
+        :remote_host => '127.0.0.1', :timestamp => 20091001075810, :user => nil,
+        :http_status => 200,         :http_method => 'GET',        :http_version => '1.1',
+        :duration => 0.0045,         :bytes_sent => 1,             :remote_logname => nil)
     end
 
-    it "should read the correct values from a valid 404 access log line" do
-      @log_parser.parse_io(StringIO.new(@sample_1)) do |request|
-        request[:remote_host].should  == '69.41.0.45'
-        request[:timestamp].should    == 20090902120240
-        request[:http_status].should  == 404
-        request[:http_method].should  == 'GET'
-        request[:http_version].should == '1.1'
-        request[:bytes_sent].should   == 209
-        request[:referer].should      == nil
-        request[:user].should         == nil
-        request[:user_agent].should   == 'Mozilla/4.0 (compatible; MSIE 6.0; Windows 98)'
-      end
+    it "should not parse an invalid access log line" do
+      @file_format.should_not parse_line(@nonsense_sample)
     end
 
-    it "should read the correct values from a valid 200 access log line" do
-      @log_parser.parse_io(StringIO.new(@sample_2)) do |request|
-        request[:remote_host].should  == '10.0.1.1'
-        request[:timestamp].should    == 20090902050833
-        request[:http_status].should  == 200
-        request[:http_method].should  == 'GET'
-        request[:http_version].should == '1.1'
-        request[:bytes_sent].should   == 30
-        request[:referer].should      == nil
-        request[:user].should         == nil
-        request[:user_agent].should   == 'Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; en-us) AppleWebKit/531.9 (KHTML, like Gecko) Version/4.0.3 Safari/531.9'
+    it "should parse 2 Apache requests from a sample without warnings" do
+      request_counter.should_receive(:hit!).twice
+      @log_parser.should_not_receive(:warn)
+      
+      @log_parser.parse_io(log_stream(@sample_1, @nonsense_sample, @sample_2)) do |request|
+        request_counter.hit! if request.kind_of?(RequestLogAnalyzer::FileFormat::Apache::Request)
       end
     end
-
-    it "should parse 5 request from fixture access log" do
-      counter = mock('counter')
-      counter.should_receive(:hit!).exactly(5).times
-      @log_parser.parse_file(log_fixture(:apache_combined)) { counter.hit! }
-    end
   end
 end
-

data/spec/unit/file_format/format_autodetection_spec.rb

@@ -0,0 +1,35 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe RequestLogAnalyzer::FileFormat do
+
+  describe '.autodetect' do
+    it "should autodetect a Merb log" do
+      file_format = RequestLogAnalyzer::FileFormat.autodetect(log_fixture(:merb))
+      file_format.should be_kind_of(RequestLogAnalyzer::FileFormat::Merb)
+    end
+    
+    it "should autodetect a MySQL slow query log" do
+      file_format = RequestLogAnalyzer::FileFormat.autodetect(log_fixture(:mysql_slow_query))
+      file_format.should be_kind_of(RequestLogAnalyzer::FileFormat::Mysql)
+    end
+    
+    it "should autodetect a Rails 1.x log" do
+      file_format = RequestLogAnalyzer::FileFormat.autodetect(log_fixture(:rails_1x))
+      file_format.should be_kind_of(RequestLogAnalyzer::FileFormat::Rails)
+    end
+
+    it "should autodetect a Rails 2.x log" do
+      file_format = RequestLogAnalyzer::FileFormat.autodetect(log_fixture(:rails_22))
+      file_format.should be_kind_of(RequestLogAnalyzer::FileFormat::Rails)
+    end
+
+    it "should autodetect an Apache access log" do
+      file_format = RequestLogAnalyzer::FileFormat.autodetect(log_fixture(:apache_common))
+      file_format.should be_kind_of(RequestLogAnalyzer::FileFormat::Apache)
+    end
+    
+    it "should not find any file format with a bogus file" do
+      RequestLogAnalyzer::FileFormat.autodetect(log_fixture(:test_order)).should be_nil
+    end
+  end
+end

data/spec/unit/file_format/mysql_format_spec.rb

@@ -0,0 +1,155 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe RequestLogAnalyzer::FileFormat::Mysql do
+
+  it "should be a valid file format" do
+    RequestLogAnalyzer::FileFormat.load(:mysql).should be_valid
+  end
+
+  describe '#parse_line' do
+    before(:each) do
+      @file_format = RequestLogAnalyzer::FileFormat.load(:mysql)
+    end
+
+    it "should parse a :time line correctly" do
+      line = '# Time: 091112 8:13:56'
+      line = '# Time: 091112 8:13:56'      
+      @file_format.should parse_line(line).as(:time).and_capture(:timestamp => 20091112081356)
+    end
+
+    it "should parse a :user_host line correctly with IP present" do
+      line = '# User@Host: admin[admin] @ db1 [10.0.0.1]'
+      @file_format.should parse_line(line).as(:user_host).and_capture(:user => "admin", :host => 'db1', :ip => '10.0.0.1')
+    end
+
+    it "should parse a :user_host line correctly without a host" do
+      line = '# User@Host: admin[admin] @  [10.0.0.1]'
+      @file_format.should parse_line(line).as(:user_host).and_capture(:user => "admin", :host => '', :ip => '10.0.0.1')
+    end
+
+    it "should parse a :user_host line correctly with IP absent" do
+      line = '# User@Host: root[root] @ localhost []'
+      @file_format.should parse_line(line).as(:user_host).and_capture(:user => "root", :host => 'localhost', :ip => "")
+    end
+
+    it "should parse a :query_statistics line" do
+      line = '# Query_time: 10  Lock_time: 0  Rows_sent: 1191307  Rows_examined: 1191307'
+      @file_format.should parse_line(line).as(:query_statistics).and_capture(:query_time => 10.0,
+          :lock_time => 0.0, :rows_sent => 1191307, :rows_examined => 1191307)
+    end
+
+    it "should parse a :query_statistics line with floating point durations" do
+      line = '# Query_time: 10.00000  Lock_time: 0.00000  Rows_sent: 1191307  Rows_examined: 1191307'
+      @file_format.should parse_line(line).as(:query_statistics).and_capture(:query_time => 10.0,
+          :lock_time => 0.0, :rows_sent => 1191307, :rows_examined => 1191307)
+    end
+    
+    it "should parse a :query_part line" do
+      line = '                               AND clients.index > 0'
+      @file_format.should parse_line(line).as(:query_part).and_capture(:query_fragment => line)
+    end
+    
+    it "should parse a final :query line" do
+      line = 'SELECT /*!40001 SQL_NO_CACHE */ * FROM `events`; '
+      @file_format.should parse_line(line).as(:query).and_capture(:query =>
+        'SELECT /*!:int SQL_NO_CACHE */ * FROM events')
+    end
+    
+    it "should parse a :use_database line" do
+      line = 'use db;'
+      @file_format.should parse_line(line).as(:use_database).and_capture(:database => 'db')
+    end
+    
+    it "should not parse a SET timestamp line" do
+      line = 'SET timestamp=1250651725;'
+      @file_format.should_not parse_line(line)
+    end
+
+    it "should not parse a SET insert_id line" do
+      line = 'SET insert_id=1250651725;'
+      @file_format.should_not parse_line(line)
+    end
+
+    it "should not parse a SET timestamp, insert_id line" do
+      line = 'SET timestamp=1250651725, insert_id=45674;'
+      @file_format.should_not parse_line(line)
+    end
+
+
+  end
+
+  describe '#parse_io' do
+    before(:each) do
+      @log_parser = RequestLogAnalyzer::Source::LogParser.new(RequestLogAnalyzer::FileFormat.load(:mysql))
+    end
+
+    it "should parse a single line query entry correctly" do
+      fixture = <<EOS
+# Time: 091112 18:13:56
+# User@Host: admin[admin] @ db1 [10.0.0.1]
+# Query_time: 10  Lock_time: 0  Rows_sent: 1191307  Rows_examined: 1191307
+SELECT /*!40001 SQL_NO_CACHE */ * FROM `events`;
+EOS
+      @log_parser.parse_io(StringIO.new(fixture)) do |request|
+        request.should be_kind_of(RequestLogAnalyzer::FileFormat::Mysql::Request)
+        request[:query].should == 'SELECT /*!:int SQL_NO_CACHE */ * FROM events'
+      end
+    end
+
+    it "should parse a multiline query entry correctly" do
+      fixture = <<EOS
+# Time: 091112 18:13:56
+# User@Host: admin[admin] @ db1 [10.0.0.1]
+# Query_time: 10  Lock_time: 0  Rows_sent: 1191307  Rows_examined: 1191307
+SELECT * FROM `clients` WHERE (1=1 
+                              AND clients.valid_from < '2009-12-05' AND (clients.valid_to IS NULL or clients.valid_to > '2009-11-20')
+                               AND clients.index > 0
+                              ) AND (clients.deleted_at IS NULL);
+EOS
+      @log_parser.parse_io(StringIO.new(fixture)) do |request|
+        request.should be_kind_of(RequestLogAnalyzer::FileFormat::Mysql::Request)
+        request[:query].should == "SELECT * FROM clients WHERE (:int=:int AND clients.valid_from < :date AND (clients.valid_to IS NULL or clients.valid_to > :date) AND clients.index > :int ) AND (clients.deleted_at IS NULL)"
+      end
+    end
+
+    it "should parse a request without timestamp correctly" do
+        fixture = <<EOS
+# User@Host: admin[admin] @ db1 [10.0.0.1]
+# Query_time: 10  Lock_time: 0  Rows_sent: 1191307  Rows_examined: 1191307
+SELECT /*!40001 SQL_NO_CACHE */ * FROM `events`;
+EOS
+      request_counter.should_receive(:hit!).once
+      @log_parser.should_not_receive(:warn)
+
+      @log_parser.parse_io(StringIO.new(fixture)) do |request|
+        request_counter.hit! if request.kind_of?(RequestLogAnalyzer::FileFormat::Mysql::Request) && request.completed?
+      end
+    end
+
+    it "should parse a query with context information correctly" do
+      fixture = <<EOS
+# Time: 091112 18:13:56
+# User@Host: admin[admin] @ db1 [10.0.0.1]
+# Query_time: 10  Lock_time: 0  Rows_sent: 1191307  Rows_examined: 1191307
+use database_name;
+SET timestamp=4324342342423, insert_id = 224253443;
+SELECT /*!40001 SQL_NO_CACHE */ * FROM `events`;
+EOS
+
+      request_counter.should_receive(:hit!).once
+      @log_parser.should_not_receive(:warn)
+
+      @log_parser.parse_io(StringIO.new(fixture)) do |request|
+        request_counter.hit! if request.kind_of?(RequestLogAnalyzer::FileFormat::Mysql::Request) && request.completed?
+        request[:query].should == 'SELECT /*!:int SQL_NO_CACHE */ * FROM events'
+      end
+
+    end
+
+
+    it "should find 26 completed sloq query entries" do
+      @log_parser.should_receive(:handle_request).exactly(26).times
+      @log_parser.parse_file(log_fixture(:mysql_slow_query))
+    end
+  end
+end