mongo_request_logger 0.2.0 → 0.2.1

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.2.1
+
+* Fix log viewer using the mongo adapter.
+* Fix default options for the mongo adapter to be consistent with the moped adapter.
 
 ## 0.2.0
 

data/lib/mongo_request_logger/adapters/mongo.rb

@@ -9,6 +9,9 @@ module MongoRequestLogger
       def initialize(options = {})
         @authenticated = false
         @configuration = options.with_indifferent_access
+        @configuration['host'] ||= '127.0.0.1'
+        @configuration['port'] ||= '27017'
+        @configuration['collection'] ||= 'server_log'
 
         connect
 
@@ -54,9 +57,9 @@ module MongoRequestLogger
       end
 
       def query(criteria, options={})
-        q = @collection.find(filter.get_mongo_conditions).sort('timestamp', -1)
+        q = @collection.find(criteria).sort('timestamp', -1)
         if options[:limit]
-          q = q.limit(options.limit)
+          q = q.limit(options[:limit])
         end
         q
       end

data/lib/mongo_request_logger/version.rb

@@ -1,3 +1,3 @@
 module MongoRequestLogger
-  VERSION = '0.2.0'
+  VERSION = '0.2.1'
 end

data/spec/shared_examples.rb

@@ -1,8 +1,8 @@
 shared_examples "log" do
 
+  let(:logger) { MongoRequestLogger::Logger.new @adapter, @file }
 
   it "should log without a request" do
-    logger = MongoRequestLogger::Logger.new @adapter, @file
     logger.info "Test message"
     logger.error "Test error"
     logger.close
@@ -11,7 +11,7 @@ shared_examples "log" do
   end
 
   it "should log requests" do
-    logger = MongoRequestLogger::Logger.new @adapter, @file
+
     options = {test_option: "Test value", args: {"one" => "Value"}}
     logger.log_request(options) do
       logger.add_metadata value: "Test"
@@ -36,9 +36,20 @@ shared_examples "log" do
     messages[1].should match /ERROR.*Test error/
   end
 
+  it "should query logs" do
+    logger.log_request(my_number: 5) do
+      logger.info "Test message"
+    end
+
+    logs = @adapter.query({}, limit: 2).to_a
+    logs.count.should == 1
+    log = logs.first
+    log['my_number'].should == 5
+  end
+
   it "should log exceptions" do
     class MyError < StandardError; end
-    logger = MongoRequestLogger::Logger.new @adapter, @file
+
     options = {test_option: "Test value", args: {"one" => "Value"}}
     logger.log_request(options) do
       logger.info "Test message"
@@ -59,8 +70,6 @@ shared_examples "log" do
   it "should log raised errors" do
     class MyError < StandardError; end
 
-    logger = MongoRequestLogger::Logger.new @adapter, @file
-    options = {test_option: "Test value", args: {"one" => "Value"}}
     block = ->{
       logger.log_request do
         logger.info "Test message"
@@ -86,7 +95,6 @@ shared_examples "log" do
       infinite_recursion(n+1)
     end
 
-    logger = MongoRequestLogger::Logger.new @adapter, @file
     block = ->{
       logger.log_request do
         begin

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongo_request_logger
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
   prerelease: 
 platform: ruby
 authors:
@@ -418,7 +418,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1701116920738529892
+      hash: 3718731922470250442
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -427,7 +427,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1701116920738529892
+      hash: 3718731922470250442
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24