etcdv3 0.11.4 → 0.11.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 18d76403d57c3e56b75e603931345900a4583b2e94d773b33ca04aa1547e304d
-  data.tar.gz: f1d1f9336f62f18b32fb9d3f055e0043bddb6076022b922b097c0104a7467c8b
+  metadata.gz: 03fa84076fc6d31e895bc80d55abb4501398a0be305cefa1f492b5e217e52277
+  data.tar.gz: 020aa912711d848ceb60a842a2d04bc1d5d3aacbf2714a380782cefd37c3b9c4
 SHA512:
-  metadata.gz: 9633a47e982b7777ccda7e5472a663a661e4cf6368689cfbbd8d06a6ca862be47e7c3a75d075919b645611879513d4c88c7f59686b2a0eac15cc160e3ea7520f
-  data.tar.gz: 5a330de810e02ebdec73c279e33b7c2ece0c2aa7c4c877da89a2b285976ce68578ca20b3c51206d9a14c9fccbf18ef6310254010053eaefa59b06b898e1c195a
+  metadata.gz: 7541f20326b9577fa7b595e371a0e94713b0b0d83ced62eaae120b5fe77c853625020425258ed941f49b345d40025f7c9d8398ba7bbb136e666f13d1b64e181b
+  data.tar.gz: 3e1226dcf9a6f7a3fcfce148148fcdcabe3f07bf92f24335a7a624b8af3e3b07176162a8f52f606b80dc9df864574ce8e430b95c456a54e79444f6466d3b37b4

data/lib/etcdv3/lock.rb

@@ -10,12 +10,12 @@ class Etcdv3
 
     def lock(name, lease_id, timeout: nil)
       request = V3lockpb::LockRequest.new(name: name, lease: lease_id)
-      @stub.lock(request, deadline: deadline(timeout))
+      @stub.lock(request, metadata: @metadata, deadline: deadline(timeout))
     end
 
     def unlock(key, timeout: nil)
       request = V3lockpb::UnlockRequest.new(key: key)
-      @stub.unlock(request, deadline: deadline(timeout))
+      @stub.unlock(request, metadata: @metadata, deadline: deadline(timeout))
     end
 
     private

data/lib/etcdv3/namespace/lock.rb

@@ -13,14 +13,14 @@ class Etcdv3::Namespace
     def lock(name, lease_id, timeout: nil)
       name = prepend_prefix(@namespace, name)
       request = V3lockpb::LockRequest.new(name: name, lease: lease_id)
-      resp = @stub.lock(request, deadline: deadline(timeout))
+      resp = @stub.lock(request, metadata: @metadata, deadline: deadline(timeout))
       strip_prefix_from_lock(@namespace, resp)
     end
 
     def unlock(key, timeout: nil)
       key = prepend_prefix(@namespace, key)
       request = V3lockpb::UnlockRequest.new(key: key)
-      @stub.unlock(request, deadline: deadline(timeout))
+      @stub.unlock(request, metadata: @metadata, deadline: deadline(timeout))
     end
 
     private

data/lib/etcdv3/version.rb

@@ -1,3 +1,3 @@
 class Etcdv3
-  VERSION = '0.11.4'.freeze
+  VERSION = '0.11.5'.freeze
 end

data/spec/etcdv3/lock_spec.rb

@@ -10,14 +10,28 @@ unless $instance.version < Gem::Version.new("3.2.0")
     #it_should_behave_like "a method with a GRPC timeout", described_class, :lock, :lock, 'foo'
 
     describe '#lock' do
-      let(:lease_id) { lease_stub.lease_grant(10)['ID'] }
-      subject { stub.lock('foo', lease_id) }
-      it { is_expected.to be_an_instance_of(V3lockpb::LockResponse) }
+      it 'returns a response' do
+        lease_id = lease_stub.lease_grant(10)['ID']
+
+        expect(stub.lock('example1', lease_id)).to be_an_instance_of(V3lockpb::LockResponse)
+      end
+
+      it 'passes metadata correctly' do
+        lease_id = lease_stub.lease_grant(10)['ID']
+        stub = expect_metadata_passthrough(described_class, :lock, :lock)
+        stub.lock('example2', lease_id)
+      end
     end
 
     describe '#unlock' do
-      subject { stub.unlock('foo') }
-      it { is_expected.to be_an_instance_of(V3lockpb::UnlockResponse) }
+      it 'returns a response' do
+        expect(stub.unlock('example3')).to be_an_instance_of(V3lockpb::UnlockResponse)
+      end
+
+      it 'passes metadata correctly' do
+        stub = expect_metadata_passthrough(described_class, :unlock, :unlock)
+        stub.unlock('example4')
+      end
     end
   end
 end

data/spec/etcdv3/namespace/lock_spec.rb

@@ -2,22 +2,41 @@ require 'spec_helper'
 
 # Locking is not implemented in etcd v3.1.X
 unless $instance.version < Gem::Version.new("3.2.0")
-  describe Etcdv3::Lock do
+  describe Etcdv3::Namespace::Lock do
     let(:stub) { local_namespace_stub(Etcdv3::Namespace::Lock, 1, '/namespace/') }
     let(:lease_stub) { local_stub(Etcdv3::Lease, 1) }
 
-    it_should_behave_like "a method with a GRPC timeout", described_class, :unlock, :unlock, 'foo'
+    # NOTE: this was running duplicate tests against Etcdv3::Lock before, but it
+    # doesn't work with Etcdv3::Namespace::Lock
+    #
+    # it_should_behave_like "a method with a GRPC timeout", described_class, :unlock, :unlock, 'foo'
+
     # it_should_behave_like "a method with a GRPC timeout", described_class, :lock, :lock, 'foo'
 
     describe '#lock' do
-      let(:lease_id) { lease_stub.lease_grant(10)['ID'] }
-      subject { stub.lock('foo', lease_id) }
-      it { is_expected.to be_an_instance_of(V3lockpb::LockResponse) }
+      it 'returns a response' do
+        lease_id = lease_stub.lease_grant(10)['ID']
+        expect(stub.lock('example1', lease_id)).to(
+          be_an_instance_of(V3lockpb::LockResponse)
+        )
+      end
+
+      it 'passes metadata correctly' do
+        lease_id = lease_stub.lease_grant(10)['ID']
+        stub = expect_metadata_passthrough_namespace(described_class, :lock, :lock, '/namespace/')
+        stub.lock('example2', lease_id)
+      end
     end
 
     describe '#unlock' do
-      subject { stub.unlock('foo') }
-      it { is_expected.to be_an_instance_of(V3lockpb::UnlockResponse) }
+      it 'returns a response' do
+        expect(stub.unlock('example3')).to be_an_instance_of(V3lockpb::UnlockResponse)
+      end
+
+      it 'passes metadata correctly' do
+        stub = expect_metadata_passthrough_namespace(described_class, :unlock, :unlock, '/namespace/')
+        stub.unlock('example4')
+      end
     end
   end
 end

data/spec/helpers/connections.rb

@@ -21,10 +21,18 @@ module Helpers
       interface.new(local_url, :this_channel_is_insecure, timeout, {})
     end
 
+    def local_stub_with_metadata(interface, timeout: nil, metadata: {})
+      interface.new(local_url, :this_channel_is_insecure, timeout, metadata)
+    end
+
     def local_namespace_stub(interface, timeout=nil, namespace)
       interface.new(local_url, :this_channel_is_insecure, timeout, namespace, {})
     end
 
+    def local_namespace_stub_with_metadata(interface, timeout: nil, namespace:, metadata: {})
+      interface.new(local_url, :this_channel_is_insecure, timeout, namespace, metadata)
+    end
+
     def local_url
       "127.0.0.1:#{port}"
     end

data/spec/helpers/metadata_passthrough.rb

@@ -0,0 +1,21 @@
+module Helpers
+  module MetadataPassthrough
+    include Connections
+
+    def expect_metadata_passthrough(stub_class, method_name, expectation_target)
+      metadata = { user: "foo", password: "bar" }
+      handler = local_stub_with_metadata(stub_class, metadata: metadata, timeout: 1)
+      inner_stub = handler.instance_variable_get("@stub")
+      expect(inner_stub).to receive(expectation_target).with(anything, hash_including(metadata: metadata)).and_call_original
+      return handler
+    end
+
+    def expect_metadata_passthrough_namespace(stub_class, method_name, expectation_target, namespace)
+      metadata = { user: "foo", password: "bar" }
+      handler = local_namespace_stub_with_metadata(stub_class, metadata: metadata, timeout: 1, namespace: namespace)
+      inner_stub = handler.instance_variable_get("@stub")
+      expect(inner_stub).to receive(expectation_target).with(anything, hash_including(metadata: metadata)).and_call_original
+      return handler
+    end
+  end
+end

data/spec/helpers/test_instance.rb

@@ -3,10 +3,12 @@ require 'tmpdir'
 require 'socket'
 require 'timeout'
 require 'helpers/connections'
+require 'helpers/metadata_passthrough'
 
 module Helpers
   class TestInstance
     include Helpers::Connections
+    include Helpers::MetadataPassthrough
 
     class InvalidVersionException < StandardError; end
     class PortInUseException < StandardError; end

data/spec/spec_helper.rb

@@ -3,20 +3,22 @@ $LOAD_PATH.unshift File.expand_path('./helpers', __FILE__)
 $LOAD_PATH.unshift File.expand_path('./namespace', __FILE__)
 
 
-# require 'simplecov'
-# require 'codecov'
-# SimpleCov.start
-# SimpleCov.formatter = SimpleCov::Formatter::Codecov
+require 'simplecov'
+require 'codecov'
+SimpleCov.start
+SimpleCov.formatter = SimpleCov::Formatter::Codecov
 
 require 'etcdv3'
 require 'helpers/test_instance'
 require 'helpers/connections'
+require 'helpers/metadata_passthrough'
 require 'helpers/shared_examples_for_timeout'
 
 $instance = Helpers::TestInstance.new
 
 RSpec.configure do |config|
   config.include(Helpers::Connections)
+  config.include(Helpers::MetadataPassthrough)
 
   config.expect_with :rspec do |expectations|
     expectations.include_chain_clauses_in_custom_matcher_descriptions = true

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: etcdv3
 version: !ruby/object:Gem::Version
-  version: 0.11.4
+  version: 0.11.5
 platform: ruby
 authors:
 - Shaun Davis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-09-23 00:00:00.000000000 Z
+date: 2021-12-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: grpc
@@ -126,6 +126,7 @@ files:
 - spec/etcdv3/watch_spec.rb
 - spec/etcdv3_spec.rb
 - spec/helpers/connections.rb
+- spec/helpers/metadata_passthrough.rb
 - spec/helpers/shared_examples_for_timeout.rb
 - spec/helpers/test_instance.rb
 - spec/spec_helper.rb
@@ -148,7 +149,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.20
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: A Etcd client library for Version 3
@@ -166,6 +167,7 @@ test_files:
 - spec/etcdv3/watch_spec.rb
 - spec/etcdv3_spec.rb
 - spec/helpers/connections.rb
+- spec/helpers/metadata_passthrough.rb
 - spec/helpers/shared_examples_for_timeout.rb
 - spec/helpers/test_instance.rb
 - spec/spec_helper.rb