fix: parse

Author: uvlad7

ext/json_scanner/json_scanner.c

@@ -753,7 +753,7 @@ VALUE scan(int argc, VALUE *argv, VALUE self)
   yajl_status stat;
   scan_ctx *ctx;
   int free_ctx = true;
-  VALUE err_msg = Qnil, bytes_consumed, result, roots_info_result = Qundef;
+  VALUE err_msg = Qnil, bytes_consumed = Qnil, result, roots_info_result = Qundef;
   // Turned out callbacks can't raise exceptions
   // VALUE callback_err;
 #if RUBY_API_VERSION_MAJOR > 2 || (RUBY_API_VERSION_MAJOR == 2 && RUBY_API_VERSION_MINOR >= 7)

lib/json_scanner.rb

@@ -13,6 +13,8 @@ class Error < StandardError; end
   ALLOWED_OPTS = %i[verbose_error allow_comments dont_validate_strings allow_multiple_values
                     allow_trailing_garbage allow_partial_values symbolize_path_keys].freeze
   private_constant :ALLOWED_OPTS
+  STUB = :stub
+  private_constant :STUB
 
   def self.parse(json_str, config_or_path_ary, **opts)
     # with_path and with_roots_info is set here
@@ -22,22 +24,29 @@ def self.parse(json_str, config_or_path_ary, **opts)
 
     results, roots = scan(json_str, config_or_path_ary, **opts, with_path: true, with_roots_info: true)
 
+    res = process_results(json_str, results, roots, opts[:symbolize_path_keys])
+
+    opts[:allow_multiple_values] ? res : res.first
+  end
+
+  def self.process_results(json_str, results, roots, symbolize_names)
     # stubs are symbols, so they can be distinguished from real values
     res = roots.map(&:first)
     # results for different path matchers can overlap, in that case we will simply parse more than one time,
     # but there shouln't be any surprises in the behavior
     results.each do |result|
-      process_result(res, result, roots, json_str, opts[:symbolize_path_keys])
+      process_result(res, result, roots, json_str, symbolize_names)
     end
-
-    opts[:allow_multiple_values] ? res : res.first
+    res
   end
 
+  private_class_method :process_results
+
   def self.process_result(res, result, roots, json_str, symbolize_names)
     current_root_index = 0
     next_root = roots[1]
     result.each do |path, (begin_pos, end_pos, _type)|
-      if next_root && begin_pos >= next_root[1]
+      while next_root && begin_pos >= next_root[1]
         current_root_index += 1
         next_root = roots[current_root_index + 1]
       end

spec/json_scanner_spec.rb

@@ -320,17 +320,17 @@
     end
   end
 
-  # it "extracts values" do
-  #   json_str = 'null false {"a": 1, "b": [0,1], "c": 42}  4.2 {"b": [1,2,3]} {} 1 []'
-  #   json_selector = [["a"], ["b", 2]]
-  #   expect(
-  #     described_class.parse(json_str, json_selector, allow_multiple_values: true),
-  #   ).to eq(
-  #     [
-  #       :null, :boolean, { "a" => 1 }, :number, { "b" => [nil, nil, 3] }, :object, :number, :array,
-  #     ]
-  #   )
-  # end
+  it "extracts values" do
+    json_str = 'null false {"a": 1, "b": [0,1], "c": 42}  4.2 {"b": [1,2,3]} {} 1 []'
+    json_selector = [["a"], ["b", 2]]
+    expect(
+      described_class.parse(json_str, json_selector, allow_multiple_values: true),
+    ).to eq(
+      [
+        :null, :boolean, { "a" => 1 }, :number, { "b" => [nil, nil, 3] }, :object, :number, :array,
+      ],
+    )
+  end
 
   describe described_class::Config do
     it "saves state" do