codeclimate-poseidon 0.0.13 → 0.0.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0302cc6f1a4201b82c1f76a2254e5fad99c500e5
-  data.tar.gz: 740e24b90c903253b87ea4a7af4baf65ea503b3e
+  metadata.gz: 89451780f4cbf5a82de300d746c4a438bf0bf677
+  data.tar.gz: 873cd07cf3f43dcab11603bb6158a33e105dde7e
 SHA512:
-  metadata.gz: 4e4a6ad856eee78acf6a5ad2ebffb6ea0d4773373823444c959f70cf5c8d044144d8b03256a32691222ddcceeb03ff044c683ef8440e9aab861ba0a9cf943d7a
-  data.tar.gz: 703b698973875c75c00f1b261f0cea3fbdaf601158010ea8858d4f28aaa000435722d05faab81be6cb73f7164a9996664e32a71b8ec2a74125294f5635739fcc
+  metadata.gz: a81fa0935d726c4ab4562b2563cabeaa67712b6fa5e12ce327914a3fdd92fb7b4180ed44cbb1f56a574866e7ddb8561b9165a2f7a53a0e3db333e08459230e23
+  data.tar.gz: e010e73bd02ea768b4ae1170ecd12bbdd3be649afb4fbcea4b1fd25d8977f071820d9557ff43081ee14cc65dd77cb3f57e747a8a02eb2e1fa4e4d5d0c9ece0c0

data/lib/poseidon/cluster_metadata.rb

@@ -55,7 +55,7 @@ module Poseidon
       if broker_id
         @brokers[broker_id]
       else
-        nil
+        raise ::Poseidon::Errors::UnknownTopicOrPartition
       end
     end
 

data/lib/poseidon/version.rb

@@ -1,4 +1,4 @@
 module Poseidon
   # Unstable! API May Change!
-  VERSION = "0.0.13"
+  VERSION = "0.0.14"
 end

data/spec/integration/multiple_brokers/consumer_spec.rb

@@ -23,6 +23,20 @@ RSpec.describe "consuming with multiple brokers", :type => :request do
     expect(brokers.size).to eq(3)
   end
 
+  it "raises a helpful error when there's no matching broker to be found" do
+    get_consumer = ->(partition) {
+      PartitionConsumer.consumer_for_partition(
+        "test_client",
+        ["localhost:9092"],
+        "test", partition,
+        :earliest_offset,
+      )
+    }
+
+    expect(get_consumer.(0)).to be_a PartitionConsumer
+    expect { get_consumer.(42) }.to raise_error(::Poseidon::Errors::UnknownTopicOrPartition)
+  end
+
   it "consumes from all partitions" do
     @p = Producer.new(["localhost:9092","localhost:9093","localhost:9094"], "test",
                      :required_acks => 1)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: codeclimate-poseidon
 version: !ruby/object:Gem::Version
-  version: 0.0.13
+  version: 0.0.14
 platform: ruby
 authors:
 - Bob Potter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-16 00:00:00.000000000 Z
+date: 2017-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -185,7 +185,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: Poseidon is a producer and consumer implementation for Kafka >= 0.8