rafka 0.2.2 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: af039ada3c68e5b6a3478a84942b344bc9a229f0
-  data.tar.gz: 6f0e9cc0c464b83e0661cd868f4c3d94f00217cb
+  metadata.gz: fc549dedd6c527810e287f92bb95f57e001addb4
+  data.tar.gz: c1a4cf7b486392968cf7982b35cccfe81e87536c
 SHA512:
-  metadata.gz: '0817ecf06bf5fbd36852f19d98b317ecf56aa9cf3e052db3bd75e22174c609b3be2541f145dccbc6640e82ebb7247245b75d24561ddfd27e3a1d2c05f4f746a8'
-  data.tar.gz: 40bff0316776a450b8e77b5c4cce5364d9fe93cd237cc7ae9c296e198470908616cdba392dbb5aa9dd093eaeeae3096017614f2037adf98bbdeff77a7406aa18
+  metadata.gz: e2d45712221fca6e7ff0f24a0990678cb93faf26bf7a78713180185fd21e53ae9f159a2a4998acd87c8b78c87fe09475f8c666d1f015e7396aa37f93e025306b
+  data.tar.gz: 0d27ecd0e1de49a8e91680f0d50440c686a71c46bd57e58d2cb4c35440c9dd6795767bded90323c33eefe0fcefaf5d492def24f1c9be3498319f56a4612d9a78

data/CHANGELOG.md

@@ -4,12 +4,17 @@ Breaking changes are prefixed with a "[BREAKING]" label.
 
 ## master (unreleased)
 
+## 0.2.3 (2018-05-07)
+
+### Fixed
+
+- `Consumer#consume` could yield a nil message to the block it received [[0ded948](https://github.com/skroutz/rafka-rb/commit/0ded94821b21d590a6cdf1314f85da56b48a9c40)]
 
 ## 0.2.2 (2018-05-04)
 
 ### Fixed
 
-- SETNAME was incorrectly called more than once in batch mode [[3416d7](https://github.com/skroutz/rafka-rb/commit/3416d7bbd9f9e36b4e4d7f87f1e51ba2f559caf2))
+- SETNAME was incorrectly called more than once in batch mode [[3416d7](https://github.com/skroutz/rafka-rb/commit/3416d7bbd9f9e36b4e4d7f87f1e51ba2f559caf2)]
 
 ## 0.2.1 (2018-05-04)
 

data/lib/rafka/consumer.rb

@@ -76,12 +76,12 @@ module Rafka
       end
 
       raised = false
+      msg = consume_one(timeout)
+
+      return nil if !msg
 
       begin
-        msg = consume_one(timeout)
         yield(msg) if block_given?
-
-        return nil if !msg
       rescue => e
         raised = true
         raise e

data/lib/rafka/version.rb

@@ -1,3 +1,3 @@
 module Rafka
-  VERSION = "0.2.2".freeze
+  VERSION = "0.2.3".freeze
 end

data/test/consumer_test.rb

@@ -42,4 +42,16 @@ class ConsumerTest < Minitest::Test
 
     assert_equal(consumer.send(:prepare_for_commit), {})
   end
+
+  def test_consume_block_no_message
+    cons = Rafka::Consumer.new(group: "foo", topic: "bar")
+
+    def cons.consume_one(t)
+      nil
+    end
+
+    cons.consume do |msg|
+      assert_kind_of Rafka::Message, msg
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rafka
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
 platform: ruby
 authors:
 - Agis Anastasopoulos
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-04 00:00:00.000000000 Z
+date: 2018-05-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis