zookeeper 1.5.1-java → 1.5.4-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0857aaef4a9dfa6b71ca96fd4aadaa6dcb4d2ad7be916dbd6d0ae994e5fb70d3'
-  data.tar.gz: 518129d5e5fb35b53225766926128272424a7f9c0cec6615880520a8ba948319
+  metadata.gz: f3654bf6a5f1c06004a95c7f5f4928f62218eaf6cf3be3921da2df54c6e989ef
+  data.tar.gz: a2818f836c18ef8e4a8f614d82a1a0d7d5dd171df3c04db0e0227262a4df2a15
 SHA512:
-  metadata.gz: 8694d67c5a1b959fbde2707b9f2f4e1ee3dfd7afb53c9173de6d27b1a15f26531b201398b4e8487bfada8874e79f54737086c63cf87dd4c72c32112458a78dbd
-  data.tar.gz: 59323715e3ec0eb4427b455ed6ade65deeaee46a0bcf0172e58038820053b1e854117f44b882da2d8ae187021f2957032f3f61aa6d5a6c415e8ab85ddc1a2e6d
+  metadata.gz: cbaa54afc1d1dc871293b5f2a252fcc461d3bc4f7613b65cb700ea495246735f0ce77cd8827311e95be683824c9f5e08b22dff76847638e0f154f5f49d1385b6
+  data.tar.gz: 50849ba02332a7cd03b420d44439223ff077d45dc7cc91c2f85981560ef7199388c2cd23b62d91157e92a6cd422224cfb439e8f8364f172adfe4d4c0e55da86e

data/.github/workflows/build.yml

@@ -15,19 +15,26 @@ jobs:
       matrix:
         os:
           - ubuntu-latest
+          - ubuntu-20.04
           - macos-latest
         ruby:
-          - 2.5
-          - 2.6
-          - 2.7
-          - 3.0
-          - jruby
+          - '2.5'
+          - '2.6'
+          - '2.7'
+          - '3.0'
+          - '3.1'
+          - '3.2'
+          - 'jruby'
         gcc:
           - 7
           - latest
         exclude:
           - os: macos-latest
             gcc: 7
+          - os: ubuntu-latest
+            gcc: 7
+          - os: ubuntu-20.04
+            gcc: latest
 
     steps:
       - uses: actions/checkout@v2
@@ -46,6 +53,7 @@ jobs:
         with:
           ruby-version: ${{ matrix.ruby }}
           bundler-cache: true
+          cache-version: 1_${{ matrix.ruby }}_${{ matrix.os }}
 
       - name: Run Tests
         env:

data/.gitmodules

@@ -1,3 +1,3 @@
 [submodule "releaseops"]
 	path = releaseops
-	url = git://github.com/slyphon/releaseops.git
+	url = https://github.com/zk-ruby/releaseops.git

data/CHANGELOG

@@ -1,3 +1,13 @@
+v1.5.4
+  * Ruby 3.2 compatibility (h/t: @cheister & @EricRoos)
+
+v1.5.3
+  * Ruby 3.1 compatibility (h/t: @casperisfine)
+
+v1.5.2 (YANKED)
+  * Ruby 3.1 compatibility and zookeeper client updates.
+  * This release had build issues in the wild and was yanked
+
 v1.5.1
   * Update config.guess to support newer environments (h/t: @malmckay)
 

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/ext/zkrb.c

@@ -203,7 +203,6 @@ inline static int we_are_forked(zkrb_instance_data_t *zk) {
   return rv;
 }
 
-
 static int destroy_zkrb_instance(zkrb_instance_data_t* zk) {
   int rv = ZOK;
 
@@ -245,6 +244,11 @@ static void free_zkrb_instance_data(zkrb_instance_data_t* ptr) {
   destroy_zkrb_instance(ptr);
 }
 
+VALUE alloc_zkrb_instance(VALUE klass) {
+  zkrb_instance_data_t* zk = ZALLOC_N(zkrb_instance_data_t, 1);
+  return Data_Wrap_Struct(klass, NULL, free_zkrb_instance_data, zk);
+}
+
 static void print_zkrb_instance_data(zkrb_instance_data_t* ptr) {
   fprintf(stderr, "zkrb_instance_data (%p) {\n", ptr);
   fprintf(stderr, "      zh = %p\n",           ptr->zh);
@@ -1013,7 +1017,7 @@ static void zkrb_define_methods(void) {
   DEFINE_METHOD(close_handle, 0);
   DEFINE_METHOD(deterministic_conn_order, 1);
   DEFINE_METHOD(is_unrecoverable, 0);
-  DEFINE_METHOD(recv_timeout, 1);
+  DEFINE_METHOD(recv_timeout, 0);
   DEFINE_METHOD(zkrb_state, 0);
   DEFINE_METHOD(sync, 2);
   DEFINE_METHOD(zkrb_iterate_event_loop, 0);
@@ -1063,6 +1067,7 @@ void Init_zookeeper_c() {
 
   /* initialize CZookeeper class */
   CZookeeper = rb_define_class_under(mZookeeper, "CZookeeper", rb_cObject);
+  rb_define_alloc_func(CZookeeper, alloc_zkrb_instance);
   zkrb_define_methods();
 
   ZookeeperClientId = rb_define_class_under(CZookeeper, "ClientId", rb_cObject);

data/lib/zookeeper/version.rb

@@ -1,4 +1,4 @@
 module Zookeeper
-  VERSION = '1.5.1'
+  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