groonga-command-parser 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b2f5dd8bb9f225dd59a1441b1ee4ba18fc7b1d74
-  data.tar.gz: f202a12f7824fbdfd1bdf7dd31fc1542464d5868
+  metadata.gz: 80b66bb089e2eadb42d743db7c71baf82b4d165c
+  data.tar.gz: 60d236172a8301558d9961fe81f15fbfe9f66161
 SHA512:
-  metadata.gz: b056beeb0f6dd1971ba3e2688d0a623e57f88c7d9389df817b81e09583626adf3d596c98434f21e88b6d28c1aded38125dd8e08ba1550eb41a0e818d6ec80050
-  data.tar.gz: 66c6b6d7739e3889f1e205054ed600f830dcd9ecb1816f33a72af916fadd46d5a8037200e31b8536c4405d94d48804aa3ac56df0a3130fae5e014184dcecc01d
+  metadata.gz: 006c544ddaf5d9d64529bed581ecab6c8606ec1229d3c2780b5af2b745d36409dd8210a328c9bb84c3ce5ba288b772e2b0a7140703939df7cff8b5047b77a0a7
+  data.tar.gz: db6660cc03650a070685218ffc7f3350156a4ed273498fa38e67b6e42fa7f7b0e2e4f489da9a151ce9e6518bc2d30ebdee4cfbc85354a11e7bba5c4862eb8099

data/doc/text/news.md

@@ -1,5 +1,11 @@
 # News
 
+## 1.1.1: 2017-10-31
+
+### Fixes
+
+  * Fixed an infinite loop bug on parsing `load`.
+
 ## 1.1.0: 2017-10-30
 
 ### Improvements

data/lib/groonga/command/parser.rb

@@ -358,7 +358,10 @@ module Groonga
           end
         end
         @load_values_parser.on_consumed = lambda do |consumed|
-          @command.original_source << consumed if @loading
+          if @loading
+            @command.original_source << consumed
+            @buffer = @buffer[consumed.bytesize..-1]
+          end
         end
         @load_values_parser.on_end = lambda do |rest|
           loading = @loading

data/lib/groonga/command/parser/version.rb

@@ -17,7 +17,7 @@
 module Groonga
   module Command
     class Parser
-      VERSION = "1.1.0"
+      VERSION = "1.1.1"
     end
   end
 end

data/test/test-parser.rb

@@ -341,7 +341,7 @@ load --table Users
           end
         end
 
-        def test_multiple
+        def test_multiple_bulk
           @parser << <<-COMMANDS
 load --table Users
 [
@@ -387,6 +387,57 @@ load --table Users
           assert_equal(expected_events, @events)
         end
 
+        def test_multiple_line
+          commands = <<-COMMANDS
+load --table Users
+[
+{"_key": "alice", "name": "Alice"}
+]
+
+load --table Users
+[
+{"_key": "bob",   "name": "Bob"}
+]
+          COMMANDS
+          commands.each_line do |line|
+            @parser << line
+          end
+
+          expected_events = []
+
+          expected_events << [:load_start, <<-SOURCE.chomp]
+load --table Users
+          SOURCE
+          value = {"_key" => "alice", "name" => "Alice"}
+          expected_events << [:load_value, <<-SOURCE, value]
+load --table Users
+[
+          SOURCE
+          expected_events << [:load_complete, <<-SOURCE.chomp]
+load --table Users
+[
+{"_key": "alice", "name": "Alice"}
+]
+          SOURCE
+
+          expected_events << [:load_start, <<-SOURCE.chomp]
+load --table Users
+          SOURCE
+          value = {"_key" => "bob", "name" => "Bob"}
+          expected_events << [:load_value, <<-SOURCE, value]
+load --table Users
+[
+          SOURCE
+          expected_events << [:load_complete, <<-SOURCE.chomp]
+load --table Users
+[
+{"_key": "bob",   "name": "Bob"}
+]
+          SOURCE
+
+          assert_equal(expected_events, @events)
+        end
+
         class ErrorTest < self
           def test_location
             message = "record separate comma is missing"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: groonga-command-parser
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Kouhei Sutou
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-30 00:00:00.000000000 Z
+date: 2017-10-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: groonga-command