zookeeper 1.5.3 → 1.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b98186b48e9c47529ac6e321dde55dc4a911b46b8ebfbbd9e8889f2808019cfc
-  data.tar.gz: '099a5d5a645518f173f06a9d6d340fd48d3117f20bfab11780c87bc0027b7bde'
+  metadata.gz: c62ad7f2a9f95282d970313d90b627ebcf8d289c59f08a00ed3a44e787d27040
+  data.tar.gz: af6af88a8bc969ffdad9889a7fa7167158dbb9bff5cf114d0dcadbf07ce0a63a
 SHA512:
-  metadata.gz: b2458ab8902282ae69d7f173aebbd19e0488c3c99896ac99d1ba75bab086af003b967b6ab11ac8adf305a26e21f24a7617ade4f5e5ecdc5f78f14452abff1619
-  data.tar.gz: d1fd0124565f9e62b783c7448c24422ebf615b370308e9e46d72930ba431a56b42b171e7dbfdc47ca61530eee421358f8e722bead38dfe7129d5ae47e4077d7c
+  metadata.gz: 43fd7c61e195c54e5fab825685c2e8470b529e374a72c5f47da5efb48ae863dfe2385e48dd4c2d165f10b953169b405807612876523b6845cc3cd00e27ec51b3
+  data.tar.gz: 73afb9f450c1bcd38b4b998d919ca309606d12167adfd5369ff08d4912b73cc88aaabdd5e1cd1afbce049eaa2ac46fe7b33fdb159985881d51579dfd28eead21

data/.github/workflows/build.yml

@@ -15,6 +15,7 @@ jobs:
       matrix:
         os:
           - ubuntu-latest
+          - ubuntu-20.04
           - macos-latest
         ruby:
           - '2.5'
@@ -22,6 +23,7 @@ jobs:
           - '2.7'
           - '3.0'
           - '3.1'
+          - '3.2'
           - 'jruby'
         gcc:
           - 7
@@ -29,6 +31,10 @@ jobs:
         exclude:
           - os: macos-latest
             gcc: 7
+          - os: ubuntu-latest
+            gcc: 7
+          - os: ubuntu-20.04
+            gcc: latest
 
     steps:
       - uses: actions/checkout@v2

data/CHANGELOG

@@ -1,3 +1,6 @@
+v1.5.4
+  * Ruby 3.2 compatibility (h/t: @cheister & @EricRoos)
+
 v1.5.3
   * Ruby 3.1 compatibility (h/t: @casperisfine)
 

data/Gemfile

@@ -2,10 +2,10 @@ source 'https://rubygems.org'
 
 gemspec
 
-gem 'rake', '~> 0.9.0'
+gem 'rake', '>= 13.0'
 
 group :test do
-  gem "rspec" , "~> 2.11"
+  gem "rspec" , "~> 3.0"
   gem 'eventmachine', '1.0.4'
   gem 'evented-spec', '~> 0.9.0'
   gem 'zk-server', '~> 1.0', :git => 'https://github.com/zk-ruby/zk-server.git'

data/Rakefile

@@ -4,7 +4,7 @@ release_ops_path = File.expand_path('../releaseops/lib', __FILE__)
 # we use a submodule because it doesn't depend on anything else (*cough* bundler)
 # and can be shared across projects
 #
-if File.exists?(release_ops_path)
+if File.exist?(release_ops_path)
   require File.join(release_ops_path, 'releaseops')
 
   # sets up the multi-ruby zk:test_all rake tasks
@@ -17,7 +17,7 @@ if File.exists?(release_ops_path)
   # Define a task to run code coverage tests
   ReleaseOps::TestTasks.define_simplecov_tasks
 
-  # set up yard:server, yard:gems, and yard:clean tasks 
+  # set up yard:server, yard:gems, and yard:clean tasks
   # for doing documentation stuff
   ReleaseOps::YardTasks.define
 

data/ext/Rakefile

@@ -4,9 +4,9 @@ ZKRB_WRAPPER = %w[zkrb_wrapper.c zkrb_wrapper.h]
 
 namespace :zkrb do
   task :clean do
-    if File.exists?('Makefile')
+    if File.exist?('Makefile')
       sh 'make clean'
-      rm 'Makefile' # yep, regenerate this
+      FileUtils.rm('Makefile') # yep, regenerate this
     else
       $stderr.puts "nothing to clean, no Makefile"
     end

data/ext/extconf.rb

@@ -58,7 +58,7 @@ Dir.chdir(HERE) do
   else
     puts "Building zkc."
 
-    unless File.exists?('c')
+    unless File.exist?('c')
       safe_sh "tar xzf #{BUNDLE} 2>&1"
       PATCHES.each do |patch|
         safe_sh "patch -p0 < #{patch} 2>&1"
@@ -87,7 +87,7 @@ Dir.chdir("#{HERE}/lib") do
     %w[a la dylib].each do |ext|
       origin_lib_name = "libzookeeper_#{stmt}.#{ext}"
       dest_lib_name = "libzookeeper_#{stmt}_gem.#{ext}"
-      system("cp -f #{origin_lib_name} #{dest_lib_name}") if File.exists?(origin_lib_name)
+      system("cp -f #{origin_lib_name} #{dest_lib_name}") if File.exist?(origin_lib_name)
     end
   end
 end

data/lib/zookeeper/version.rb

@@ -1,4 +1,4 @@
 module Zookeeper
-  VERSION = '1.5.3'
+  VERSION = '1.5.4'
   DRIVER_VERSION = '3.4.5'
 end

data/spec/c_zookeeper_spec.rb

@@ -29,20 +29,20 @@ unless defined?(::JRUBY_VERSION)
       end
 
       it %[should be in connected state within a reasonable amount of time] do
-        wait_until_connected.should be_true
+        expect(wait_until_connected).to be_truthy
       end
 
       describe :after_connected do
         before do
-          wait_until_connected.should be_true
+          expect(wait_until_connected).to be_truthy
         end
 
         it %[should have a connection event after being connected] do
           event = wait_until(10) { @event_queue.pop }
-          event.should be
-          event[:req_id].should == Zookeeper::Constants::ZKRB_GLOBAL_CB_REQ
-          event[:type].should   == Zookeeper::Constants::ZOO_SESSION_EVENT
-          event[:state].should  == Zookeeper::Constants::ZOO_CONNECTED_STATE
+          expect(event).to be
+          expect(event[:req_id]).to eq(Zookeeper::Constants::ZKRB_GLOBAL_CB_REQ)
+          expect(event[:type]).to   eq(Zookeeper::Constants::ZOO_SESSION_EVENT)
+          expect(event[:state]).to  eq(Zookeeper::Constants::ZOO_CONNECTED_STATE)
         end
       end
     end

data/spec/chrooted_connection_spec.rb

@@ -3,7 +3,7 @@ require 'shared/connection_examples'
 
 describe 'Zookeeper chrooted' do
   let(:path) { "/_zkchroottest_" }
-  let(:data) { "underpants" } 
+  let(:data) { "underpants" }
   let(:chroot_path) { '/slyphon-zookeeper-chroot' }
 
   let(:connection_string) { "#{Zookeeper.default_cnx_str}#{chroot_path}" }
@@ -35,8 +35,8 @@ describe 'Zookeeper chrooted' do
 
         it %[should successfully create the path] do
           rv = zk.create(:path => '/', :data => '')
-          rv[:rc].should be_zero
-          rv[:path].should == ''
+          expect(rv[:rc]).to be_zero
+          expect(rv[:path]).to eq('')
         end
       end
     end
@@ -53,8 +53,8 @@ describe 'Zookeeper chrooted' do
 
         it %[should return ZNONODE] do
           rv = zk.create(:path => '/', :data => '')
-          rv[:rc].should_not be_zero
-          rv[:rc].should == Zookeeper::Exceptions::ZNONODE
+          expect(rv[:rc]).not_to be_zero
+          expect(rv[:rc]).to eq(Zookeeper::Exceptions::ZNONODE)
         end
       end
     end
@@ -62,15 +62,15 @@ describe 'Zookeeper chrooted' do
 
 
   describe do
-    before :all do
-      logger.warn "running before :all"
+    before :each do
+      logger.warn "running before :each"
 
       with_open_zk do |z|
         z.create(:path => chroot_path, :data => '')
       end
     end
 
-    after :all do
+    after :each do
       with_open_zk do |z|
         rm_rf(z, chroot_path)
       end

data/spec/compatibilty_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe 'Compatibiliy layer' do
   it %[should raise the correct error when a const is missing] do
-    lambda { Zookeeper::THISISANINVALIDCONST }.should raise_error(NameError)
+    expect { Zookeeper::THISISANINVALIDCONST }.to raise_error(NameError)
   end
 end
 

data/spec/default_watcher_spec.rb

@@ -4,7 +4,7 @@ describe Zookeeper do
   describe :initialize, 'with watcher block' do
     before do
       @events = []
-      @watch_block = lambda do |hash| 
+      @watch_block = lambda do |hash|
         logger.debug "watch_block: #{hash.inspect}"
         @events << hash
       end
@@ -12,7 +12,7 @@ describe Zookeeper do
       @zk = Zookeeper.new(Zookeeper.default_cnx_str, 10, @watch_block)
 
       wait_until(2) { @zk.connected? }
-      @zk.should be_connected
+      expect(@zk).to be_connected
       logger.debug "connected!"
 
       wait_until(2) { !@events.empty? }
@@ -24,9 +24,9 @@ describe Zookeeper do
     end
 
     it %[should receive initial connection state events] do
-      @events.should_not be_empty
-      @events.length.should == 1
-      @events.first[:state].should == Zookeeper::ZOO_CONNECTED_STATE
+      expect(@events).not_to be_empty
+      expect(@events.length).to eq(1)
+      expect(@events.first[:state]).to eq(Zookeeper::ZOO_CONNECTED_STATE)
     end
 
     it %[should receive disconnection events] do
@@ -34,7 +34,7 @@ describe Zookeeper do
       @events.clear
       @zk.close
       wait_until(2) { !@events.empty? }
-      @events.should_not be_empty
+      expect(@events).not_to be_empty
     end
   end
 end

data/spec/em_spec.rb

@@ -20,7 +20,7 @@ describe 'ZookeeperEM' do
     end
 
     def teardown_and_done
-      @zk.close do 
+      @zk.close do
         logger.debug { "TEST: about to call done" }
         EM.next_tick do
           done
@@ -31,7 +31,7 @@ describe 'ZookeeperEM' do
     describe 'callbacks' do
       it %[should be called on the reactor thread] do
         cb = lambda do |h|
-          EM.reactor_thread?.should be_true
+          expect(EM.reactor_thread?).to be_truthy
           logger.debug { "called back on the reactor thread? #{EM.reactor_thread?}" }
           teardown_and_done
         end
@@ -39,7 +39,7 @@ describe 'ZookeeperEM' do
         setup_zk do
           @zk.on_attached do |*|
             logger.debug { "on_attached called" }
-            rv = @zk.get(:path => '/', :callback => cb) 
+            rv = @zk.get(:path => '/', :callback => cb)
             logger.debug { "rv from @zk.get: #{rv.inspect}" }
           end
         end

data/spec/ext/zookeeper_base_spec.rb

@@ -11,7 +11,7 @@ unless defined?(::JRUBY_VERSION)
     end
 
     it %[should have an original_pid assigned] do
-      @zk.original_pid.should == Process.pid
+      expect(@zk.original_pid).to eq(Process.pid)
     end
   end
 end

data/spec/forked_connection_spec.rb

@@ -112,11 +112,11 @@ unless defined?(::JRUBY_VERSION)
       status = wait_for_child_safely(@pid)
       raise "Child process did not exit, likely hung"  unless status
 
-      status.should_not be_signaled
-      status.should be_success
+      expect(status).not_to be_signaled
+      expect(status).to be_success
 
-      event_waiter_th.join(5).should == event_waiter_th
-      @event.should_not be_nil
+      expect(event_waiter_th.join(5)).to eq(event_waiter_th)
+      expect(@event).not_to be_nil
     end
   end
 end

data/spec/latch_spec.rb

@@ -16,7 +16,7 @@ describe Zookeeper::Latch do
         end
 
         other_latch.await
-        th.join(1).should == th
+        expect(th.join(1)).to eq(th)
       end
     end
   end

data/spec/shared/all_success_return_values.rb

@@ -1,10 +1,10 @@
 shared_examples_for "all success return values" do
   it %[should have a return code of Zookeeper::ZOK] do
-    @rv[:rc].should == Zookeeper::ZOK
+    expect(@rv[:rc]).to eq(Zookeeper::ZOK)
   end
 
   it %[should have a req_id integer] do
-    @rv[:req_id].should be_kind_of(Integer)
+    expect(@rv[:req_id]).to be_kind_of(Integer)
   end
 end