jsonpath 0.8.11 → 0.8.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5cdb674e56726cbd684e5d375ac5c215fa1a1e05
-  data.tar.gz: 3303030517f082f2aa1fb86fd10e239a5e5348f5
+  metadata.gz: 9338212edfe2ba8589ef826b3443fb7dcfdc3112
+  data.tar.gz: b5e09641428a77416aa455ec0450f0730abe86b4
 SHA512:
-  metadata.gz: f25145b3223bbe6e2d7814616b355ac9ccb4d4b2a56378c7214cadf92c1d49d71fc2c6acce881933139863ec8dddfcdc854f68381280f32bb4d7063143818d3b
-  data.tar.gz: f30b7dbb89ba91cee2a77efb8b6ddc4c25eb499161123222006cd51157955f52eab7d48aad315330072d5b6aa4181da6bd1554db7b56410be5c54b8ebfe55243
+  metadata.gz: ce08bd8da4e3240d613d589400d32219563c76afb376e46967258afb6000e00a924e8480ee2f934da4696ef2b34eb52f1409a5b8ecc0b80ed523d6f44dc990b5
+  data.tar.gz: ccbbce4a9e02ad37c2f70ffa16a40a6667d3df00a57be835493589b3c0903efd0d8105e9e56032b2d232235b6d59955dff6281e02f718ffd528abeb8871a8b1c

data/lib/jsonpath/enumerable.rb

@@ -111,8 +111,7 @@ class JsonPath
       return nil unless @_current_node
 
       identifiers = /@?((?<!\d)\.(?!\d)(\w+))+/.match(exp)
-      unless identifiers.nil? ||
-             @_current_node.methods.include?(identifiers[2].to_sym)
+      unless identifiers.nil? || @_current_node.methods.include?(identifiers[2].to_sym)
         exp_to_eval = exp.dup
         exp_to_eval[identifiers[0]] = identifiers[0].split('.').map do |el|
           el == '@' ? '@' : "['#{el}']"

data/lib/jsonpath/parser.rb

@@ -45,12 +45,15 @@ class JsonPath
           raise "Could not process symbol: #{t}"
         end
       end
+
       el = dig(elements, @_current_node)
-      return false unless el
+      return false if el.nil?
       return true if operator.nil? && el
 
       el = Float(el) rescue el
       operand = Float(operand) rescue operand
+      operand = false if operand == 'false' && el == false
+
       el.send(operator.strip, operand)
     end
 

data/lib/jsonpath/version.rb

@@ -1,3 +1,3 @@
 class JsonPath
-  VERSION = '0.8.11'.freeze
+  VERSION = '0.8.12'.freeze
 end

data/test/test_jsonpath.rb

@@ -1,6 +1,7 @@
 require 'minitest/autorun'
 require 'phocus'
 require 'jsonpath'
+require 'json'
 
 class TestJsonpath < MiniTest::Unit::TestCase
   def setup
@@ -368,6 +369,20 @@ class TestJsonpath < MiniTest::Unit::TestCase
     assert_equal [{'number'=>'(492) 080-3961'}], JsonPath.new("$.data[?(@.number == '(492) 080-3961')]").on(data)
   end
 
+
+  def test_boolean_parameter_value
+    data = {
+      'data' => [{
+        'isTrue' => true,
+        'name' => 'testname1',
+      },{
+        'isTrue' => false,
+        'name' => 'testname2',
+        }]
+    }
+    assert_equal [{"isTrue"=>true, "name"=>"testname1"}], JsonPath.new("$.data[?(@.isTrue)]").on(data)
+  end
+
   def test_regex
     assert_equal [], JsonPath.new('$..book[?(@.author =~ /herman/)]').on(@object)
     assert_equal [
@@ -379,6 +394,84 @@ class TestJsonpath < MiniTest::Unit::TestCase
     assert_equal ["asdf", "asdf2"], JsonPath.new("$.store.book..tags[?(@ =~ /asdf/)]").on(@object)
   end
 
+  def test_regression_1
+    json = {
+      ok: true,
+      channels: [
+        {
+          id: 'C09C5GYHF',
+          name: 'general'
+        },
+        {
+          id: 'C09C598QL',
+          name: 'random'
+        }
+      ]
+    }.to_json
+
+    assert_equal 'C09C5GYHF', JsonPath.on(json, "$..channels[?(@.name == 'general')].id")[0]
+  end
+
+  def test_regression_2
+    json = {
+      ok: true,
+      channels: [
+        {
+          id: 'C09C5GYHF',
+          name: 'general',
+          is_archived: false
+        },
+        {
+          id: 'C09C598QL',
+          name: 'random',
+          is_archived: true
+        }
+      ]
+    }.to_json
+
+    assert_equal 'C09C5GYHF', JsonPath.on(json, "$..channels[?(@.is_archived == false)].id")[0]
+  end
+
+  def test_regression_3
+    json = {
+      ok: true,
+      channels: [
+        {
+          id: 'C09C5GYHF',
+          name: 'general',
+          is_archived: false
+        },
+        {
+          id: 'C09C598QL',
+          name: 'random',
+          is_archived: true
+        }
+      ]
+    }.to_json
+
+    assert_equal 'C09C598QL', JsonPath.on(json, "$..channels[?(@.is_archived)].id")[0]
+  end
+
+  def test_regression_5
+    json = {
+      ok: true,
+      channels: [
+        {
+          id: 'C09C5GYHF',
+          name: 'general',
+          is_archived: 'false'
+        },
+        {
+          id: 'C09C598QL',
+          name: 'random',
+          is_archived: true
+        }
+      ]
+    }.to_json
+
+    assert_equal 'C09C5GYHF', JsonPath.on(json, "$..channels[?(@.is_archived == 'false')].id")[0]
+  end
+
   def example_object
     { 'store' => {
       'book' => [

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jsonpath
 version: !ruby/object:Gem::Version
-  version: 0.8.11
+  version: 0.8.12
 platform: ruby
 authors:
 - Joshua Hull
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-02 00:00:00.000000000 Z
+date: 2018-05-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json