mongo-parser-rb 0.0.4 → 0.0.5

data/.gitignore

@@ -15,3 +15,6 @@ spec/reports
 test/tmp
 test/version_tmp
 tmp
+.DS_Store
+.tags
+.tags_sorted_by_file

data/.yardopts

@@ -0,0 +1 @@
+-m markdown lib/**/*.rb - README.md

data/lib/mongo-parser-rb/query.rb

@@ -1,19 +1,34 @@
 module MongoParserRB
   class Query
 
+    # Parse a query, returning an initialized MongoParserRB::Query object.
+    #
+    # @param raw_query [Hash] the hash to parse
+    # @return [MongoParserRB::Query] initialized query object
     def self.parse(raw_query)
       new(raw_query).parse!
     end
 
+    # Use {MongoParserRB::Query.parse}.
+    # @visibility private
     def initialize(raw_query)
       @raw_query = raw_query
     end
 
+    # @visibility private
     def parse!
       @expression_tree = parse_root_expression(@raw_query)
       self
     end
 
+    # Check if a document matches a query.
+    #
+    #     q = MongoParserRB::Query.parse(:x => {:$gt => 3})
+    #     q.matches_document?(:x => 4) => true
+    #     q.matches_document?(:x => 3) => false
+    #
+    # @param document [Hash] The document to check
+    # @return [Boolean]
     def matches_document?(document)
       raise NotParsedError, "Query not parsed (run parse!)" if @expression_tree.nil?
       @expression_tree.evaluate(document)

data/lib/mongo-parser-rb/query/expression.rb

@@ -6,13 +6,13 @@ module MongoParserRB
 
         def conjunction_operators
           @conjunction_operators ||= [
-            :$and, 
+            :$and,
             :$or
           ]
         end
 
         def conjunction_operator?(operator)
-          conjunction_operators.include?(operator) 
+          conjunction_operators.include?(operator)
         end
 
         def negative_equality_operators
@@ -47,7 +47,7 @@ module MongoParserRB
         @operator = operator
 
         if Expression.conjunction_operator? @operator
-          @arguments = args[0] 
+          @arguments = args[0]
         else
           @field = Field.new(args[0])
           @arguments = args[1]
@@ -103,7 +103,6 @@ module MongoParserRB
 
       def evaluate_equality(document)
         value_for_field = @field.value_in_document(document)
-        return false if !value_for_field && !@field.in_document?(document)
 
         case @operator
         when :$eq
@@ -128,7 +127,6 @@ module MongoParserRB
           end
         end
       end
-
     end
   end
-end
+end

data/lib/mongo-parser-rb/version.rb

@@ -1,3 +1,3 @@
 module MongoParserRB
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/test/mongo-parser-rb/query_test.rb

@@ -85,7 +85,7 @@ class QueryTest < MiniTest::Unit::TestCase
 
 
   def test_or
-    query = MongoParserRB::Query.parse(:$or => [ 
+    query = MongoParserRB::Query.parse(:$or => [
       {:string_key => "abc"},
       {:integer_key => {:$gt => 5}}
     ])
@@ -133,6 +133,19 @@ class QueryTest < MiniTest::Unit::TestCase
     query = MongoParserRB::Query.parse(:string_key => nil)
     assert query.matches_document?(:string_key => nil)
     refute query.matches_document?(:string_key => 'hey')
+    assert query.matches_document?(:something_else => "hello")
+    assert query.matches_document?({})
+  end
+
+  def test_ne_with_nil
+    query = MongoParserRB::Query.parse(:string_key => {:$ne => 'cheese'})
+    assert query.matches_document?(:string_key => nil)
+    assert query.matches_document?({})
+  end
+
+  def test_no_key_in_unknown
+    query = MongoParserRB::Query.parse(:string_key => {:$in=>[nil, "", "null", "nil"]})
+    assert query.matches_document?(:something_else => "hello")
   end
 
   def test_regex_eq

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongo-parser-rb
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-27 00:00:00.000000000 Z
+date: 2013-05-04 00:00:00.000000000 Z
 dependencies: []
 description: Parse and evaluate mongo queries in Ruby
 email:
@@ -19,6 +19,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .yardopts
 - Gemfile
 - LICENSE.txt
 - README.md