restify 1.15.1 → 2.0.0

data/spec/restify/timeout_spec.rb

@@ -8,15 +8,15 @@ describe Restify::Timeout do
   describe '#timeout!' do
     context 'before having timed out' do
       it 'do nothing' do
-        expect { timer.timeout! }.to_not raise_error
+        expect { timer.timeout! }.not_to raise_error
       end
     end
 
     context 'after having timed out' do
       it 'calls given block' do
-        expect { timer.timeout! }.to_not raise_error
+        expect { timer.timeout! }.not_to raise_error
         sleep timer.send(:wait_interval)
-        expect { timer.timeout! }.to raise_error ::Restify::Timeout::Error
+        expect { timer.timeout! }.to raise_error Restify::Timeout::Error
       end
     end
   end
@@ -25,13 +25,13 @@ describe Restify::Timeout do
     it 'calls block on IVar timeout' do
       expect do
         timer.wait_on!(Concurrent::IVar.new)
-      end.to raise_error ::Restify::Timeout::Error
+      end.to raise_error Restify::Timeout::Error
     end
 
     it 'calls block on Promise timeout' do
       expect do
         timer.wait_on!(Restify::Promise.new)
-      end.to raise_error ::Restify::Timeout::Error
+      end.to raise_error Restify::Timeout::Error
     end
 
     it 'does nothing on successful IVar' do
@@ -40,7 +40,7 @@ describe Restify::Timeout do
           Thread.new { ivar.set :success }
           expect(timer.wait_on!(ivar)).to eq :success
         end
-      end.to_not raise_error
+      end.not_to raise_error
     end
 
     it 'does nothing on successful Promise' do
@@ -48,7 +48,7 @@ describe Restify::Timeout do
         Restify::Promise.fulfilled(:success).tap do |p|
           expect(timer.wait_on!(p)).to eq :success
         end
-      end.to_not raise_error
+      end.not_to raise_error
     end
   end
 end

data/spec/restify_spec.rb

@@ -97,7 +97,7 @@ describe Restify do
     end
 
     context 'within threads' do
-      it 'should consume the API' do
+      it 'consumes the API' do
         # Let's get all users
 
         # First request the entry resource usually the
@@ -132,21 +132,21 @@ describe Restify do
           # Because we forgot to send a "name" the server complains
           # with an error code that will lead to a raised error.
 
-          expect(e.status).to eq :unprocessable_entity
+          expect(e.status).to eq :unprocessable_content
           expect(e.code).to eq 422
           expect(e.errors).to eq 'name' => ["can't be blank"]
         end
 
         # Let's try again.
-        created_user = users_relation.post(name: 'John Smith').value!
+        created_user = users_relation.post({name: 'John Smith'}).value!
 
         # The server returns a 201 Created response with the created
         # resource.
         expect(created_user.response.status).to eq :created
         expect(created_user.response.code).to eq 201
 
-        expect(created_user).to have_key :name
-        expect(created_user.name).to eq 'John Smith'
+        expect(created_user).to have_key 'name'
+        expect(created_user['name']).to eq 'John Smith'
 
         # Let's follow the "Location" header.
         followed_resource = created_user.follow.get.value!
@@ -154,8 +154,8 @@ describe Restify do
         expect(followed_resource.response.status).to eq :ok
         expect(followed_resource.response.code).to eq 200
 
-        expect(followed_resource).to have_key :name
-        expect(followed_resource.name).to eq 'John Smith'
+        expect(followed_resource).to have_key 'name'
+        expect(followed_resource['name']).to eq 'John Smith'
 
         # Now we will fetch a list of all users.
         users = users_relation.get.value!
@@ -168,80 +168,19 @@ describe Restify do
 
         # We have all our attributes and relations here as defined in the
         # responses from the server.
-        expect(user).to have_key :name
-        expect(user[:name]).to eq 'John Smith'
+        expect(user).to have_key 'name'
+        expect(user['name']).to eq 'John Smith'
         expect(user).to have_relation :self
         expect(user).to have_relation :blurb
 
         # Let's get the blurb.
         blurb = user.rel(:blurb).get.value!
 
-        expect(blurb).to have_key :title
-        expect(blurb).to have_key :image
+        expect(blurb).to have_key 'title'
+        expect(blurb).to have_key 'image'
 
-        expect(blurb[:title]).to eq 'Prof. Dr. John Smith'
-        expect(blurb[:image]).to eq 'http://example.org/avatar.png'
-      end
-    end
-
-    context 'within EM-synchrony' do
-      it 'should consume the API' do
-        skip 'Seems to be impossible to detect EM scheduled fibers from within'
-
-        EM.synchrony do
-          root = Restify.new('http://localhost:9292/base').get.value!
-
-          users_relation = root.rel(:users)
-
-          expect(users_relation).to be_a Restify::Relation
-
-          create_user_promise = users_relation.post
-          expect(create_user_promise).to be_a Restify::Promise
-
-          expect { create_user_promise.value! }.to \
-            raise_error(Restify::ClientError) do |e|
-            expect(e.status).to eq :unprocessable_entity
-            expect(e.code).to eq 422
-            expect(e.errors).to eq 'name' => ["can't be blank"]
-          end
-
-          created_user = users_relation.post(name: 'John Smith').value!
-
-          expect(created_user.response.status).to eq :created
-          expect(created_user.response.code).to eq 201
-
-          expect(created_user).to have_key :name
-          expect(created_user.name).to eq 'John Smith'
-
-          followed_resource = created_user.follow.get.value!
-
-          expect(followed_resource.response.status).to eq :ok
-          expect(followed_resource.response.code).to eq 200
-
-          expect(followed_resource).to have_key :name
-          expect(followed_resource.name).to eq 'John Smith'
-
-          users = users_relation.get.value!
-
-          expect(users).to have(2).items
-
-          user = users.first
-
-          expect(user).to have_key :name
-          expect(user[:name]).to eq 'John Smith'
-          expect(user).to have_relation :self
-          expect(user).to have_relation :blurb
-
-          blurb = user.rel(:blurb).get.value!
-
-          expect(blurb).to have_key :title
-          expect(blurb).to have_key :image
-
-          expect(blurb[:title]).to eq 'Prof. Dr. John Smith'
-          expect(blurb[:image]).to eq 'http://example.org/avatar.png'
-
-          EventMachine.stop
-        end
+        expect(blurb['title']).to eq 'Prof. Dr. John Smith'
+        expect(blurb['image']).to eq 'http://example.org/avatar.png'
       end
     end
   end

data/spec/spec_helper.rb

@@ -4,25 +4,21 @@ require 'rspec'
 require 'rspec/collection_matchers'
 
 require 'simplecov'
+require 'simplecov-cobertura'
+
 SimpleCov.start do
   add_filter 'spec'
 end
 
-if ENV['CI']
-  require 'codecov'
-  SimpleCov.formatter = SimpleCov::Formatter::Codecov
-end
+SimpleCov.formatters = [
+  SimpleCov::Formatter::HTMLFormatter,
+  SimpleCov::Formatter::CoberturaFormatter,
+]
 
 require 'restify'
 
 if ENV['ADAPTER']
   case ENV['ADAPTER'].to_s.downcase
-    when 'em'
-      require 'restify/adapter/em'
-      Restify.adapter = Restify::Adapter::EM.new
-    when 'em-pooled'
-      require 'restify/adapter/pooled_em'
-      Restify.adapter = Restify::Adapter::PooledEM.new
     when 'typhoeus'
       require 'restify/adapter/typhoeus'
       Restify.adapter = Restify::Adapter::Typhoeus.new
@@ -40,21 +36,21 @@ RSpec.configure do |config|
   config.order = 'random'
 
   config.before(:suite) do
-    ::Restify::Timeout.default_timeout = 1.0
+    Restify::Timeout.default_timeout = 1.0
   end
 
-  config.before(:each) do |example|
+  config.before do |example|
     next unless (adapter = example.metadata[:adapter])
-    next if Restify.adapter.is_a?(adapter)
+    next if Restify.adapter.class.ancestors.map(&:to_s).include?(adapter)
 
     skip 'Spec not enabled for current adapter'
   end
 
-  config.before(:each) do
-    Ethon.logger = ::Logging.logger[Ethon] if defined?(Ethon)
+  config.before do
+    Ethon.logger = Logging.logger[Ethon] if defined?(Ethon)
 
-    ::Logging.logger.root.level = :debug
-    ::Logging.logger.root.add_appenders ::Logging.appenders.stdout
+    Logging.logger.root.level = :debug
+    Logging.logger.root.add_appenders Logging.appenders.stdout
   end
 
   config.warnings = true

data/spec/support/stub_server.rb

@@ -17,10 +17,10 @@ module Stub
   #
   # If no stub is found a special HTTP 599 error code will be returned.
   class Handler
-    def call(env) # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
+    def call(env)
       signature = WebMock::RequestSignature.new(
         env['REQUEST_METHOD'].downcase,
-        "http://stubserver#{env['REQUEST_URI']}"
+        "http://stubserver#{env['REQUEST_URI']}",
       )
 
       # Extract request headers from rack env. Most header should start with
@@ -99,7 +99,7 @@ RSpec.configure do |config|
     WebMock.disable!(except: %i[net_http])
   end
 
-  config.around(:each) do |example|
+  config.around do |example|
     example.run
     WebMock.reset!
   end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: restify
 version: !ruby/object:Gem::Version
-  version: 1.15.1
+  version: 2.0.0
 platform: ruby
 authors:
 - Jan Graichen
-autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-07-15 00:00:00.000000000 Z
+date: 2025-02-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -53,83 +52,77 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: hashie
+  name: hitimes
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.3'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.3'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: rack
+  name: logging
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: typhoeus
+  name: msgpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.2'
 - !ruby/object:Gem::Dependency
-  name: logging
+  name: opentelemetry-api
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: msgpack
+  name: opentelemetry-common
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: hitimes
+  name: rack
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -143,23 +136,23 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: typhoeus
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: '1.3'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.3'
 description: An experimental hypermedia REST client that uses parallel, keep-alive
   and pipelined requests by default.
 email:
-- jg@altimos.de
+- jgraichen@altimos.de
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -169,8 +162,7 @@ files:
 - README.md
 - lib/restify.rb
 - lib/restify/adapter/base.rb
-- lib/restify/adapter/em.rb
-- lib/restify/adapter/pooled_em.rb
+- lib/restify/adapter/telemetry.rb
 - lib/restify/adapter/typhoeus.rb
 - lib/restify/cache.rb
 - lib/restify/context.rb
@@ -214,8 +206,8 @@ files:
 homepage: https://github.com/jgraichen/restify
 licenses:
 - LGPL-3.0+
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths:
 - lib
@@ -223,36 +215,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 3.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.6.2
 specification_version: 4
 summary: An experimental hypermedia REST client.
-test_files:
-- spec/restify/cache_spec.rb
-- spec/restify/context_spec.rb
-- spec/restify/error_spec.rb
-- spec/restify/features/head_requests_spec.rb
-- spec/restify/features/request_bodies_spec.rb
-- spec/restify/features/request_errors_spec.rb
-- spec/restify/features/request_headers_spec.rb
-- spec/restify/features/response_errors_spec.rb
-- spec/restify/global_spec.rb
-- spec/restify/link_spec.rb
-- spec/restify/processors/base_spec.rb
-- spec/restify/processors/json_spec.rb
-- spec/restify/processors/msgpack_spec.rb
-- spec/restify/promise_spec.rb
-- spec/restify/registry_spec.rb
-- spec/restify/relation_spec.rb
-- spec/restify/resource_spec.rb
-- spec/restify/timeout_spec.rb
-- spec/restify_spec.rb
-- spec/spec_helper.rb
-- spec/support/stub_server.rb
+test_files: []

data/lib/restify/adapter/em.rb

@@ -1,139 +0,0 @@
-# frozen_string_literal: true
-
-require 'eventmachine'
-require 'em-http-request'
-
-module Restify
-  module Adapter
-    class EM < Base
-      class Connection
-        class << self
-          def open(uri)
-            connections[uri.origin] ||= new uri.origin
-          end
-
-          def connections
-            @connections ||= {}
-          end
-        end
-
-        attr_reader :origin
-
-        def initialize(origin)
-          @origin   = origin
-          @pipeline = true
-        end
-
-        def requests
-          @requests ||= []
-        end
-
-        # rubocop:disable Style/IdenticalConditionalBranches
-        def call(request, writer, retried = false)
-          if requests.empty?
-            requests << [request, writer, retried]
-            process_next
-          else
-            requests << [request, writer, retried]
-          end
-        end
-        # rubocop:enable all
-
-        def connection
-          @connection ||= EventMachine::HttpRequest.new(origin)
-        end
-
-        def pipeline?
-          @pipeline
-        end
-
-        # rubocop:disable Metrics/AbcSize
-        # rubocop:disable Metrics/CyclomaticComplexity
-        # rubocop:disable Metrics/MethodLength
-        # rubocop:disable Metrics/PerceivedComplexity
-        def process_next
-          return if requests.empty?
-
-          request, writer, retried = pipeline? ? requests.shift : requests.first
-          begin
-            req = connection.send request.method.downcase,
-              keepalive: true,
-              redirects: 3,
-              path: request.uri.normalized_path,
-              query: request.uri.normalized_query,
-              body: request.body,
-              head: request.headers
-          rescue Exception => e # rubocop:disable Lint/RescueException
-            writer.reject e
-            requests.shift unless pipeline?
-            return
-          end
-
-          req.callback do
-            requests.shift unless pipeline?
-
-            writer.fulfill Response.new(
-              request,
-              req.last_effective_url,
-              req.response_header.status,
-              req.response_header,
-              req.response
-            )
-
-            if req.response_header['CONNECTION'] == 'close'
-              @connection = nil
-              @pipeline   = false
-            end
-
-            process_next
-          end
-
-          req.errback do
-            requests.shift unless pipeline?
-            @connection = nil
-
-            if pipeline?
-              EventMachine.next_tick do
-                @pipeline = false
-                call request, writer
-              end
-            elsif !retried
-              EventMachine.next_tick { call request, writer }
-            else
-              begin
-                raise "(#{req.response_header.status}) #{req.error}"
-              rescue StandardError => e
-                writer.reject e
-              end
-            end
-          end
-        end
-        # rubocop:enable all
-      end
-
-      def call_native(request, writer)
-        next_tick do
-          Connection.open(request.uri).call(request, writer)
-        end
-      end
-
-      private
-
-      def next_tick(&block)
-        ensure_running
-        EventMachine.next_tick(&block)
-      end
-
-      def ensure_running
-        return if EventMachine.reactor_running?
-
-        Thread.new do
-          EventMachine.run {}
-        rescue StandardError => e
-          puts "#{self.class} -> #{e}\n#{e.backtrace.join("\n")}"
-          raise e
-        end
-      end
-    end
-  end
-end