ezid-client 1.8.0 → 1.9.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: ab1d0301b2fdef9d87602338269625d3f5f6b32f
-  data.tar.gz: 6f1ffabe1d6e126c2ffe39ff5dc091a528144c03
+SHA256:
+  metadata.gz: 992d846692204d2ee3317ba756f1f70448a1194fb8e2d7cf129dd9b7f270919e
+  data.tar.gz: 3dba7c268214d865b85425cfb18ad3dd5f2e23d1b8a0c21ed14366b4aaae283c
 SHA512:
-  metadata.gz: 726c74c5346c4ca85128b4716ef565c888059fb5f3ee1575b04896ca8ba5bf68212e7a699b68dd051251917a05d86bf0f0f465b75feb1b801e195aef17a9d08b
-  data.tar.gz: 8809f5f2ebd0bfe9a2893738368b3442565c171f81dc08e50333c912ea7c173ecc2a130368c4de451929a09ff4a01ee3127df79c4ab4fce5bb8d3e3b851c6fd5
+  metadata.gz: 161c7c1037a3016c2399b24e3f6fb6addfd91988e3b97149da907bd5461a72b29eb9058d403067b9a166ba6eea40bd58fd7f291a11100a698b93e24a9d228159
+  data.tar.gz: 3538d1792458f2d53ee81868249c3217112994aea0da67e1023eda3526bc98246282b90d87a346287e12f7eadd24874a48f5d521f055ea5a34d1130844832b87

data/.github/workflows/ruby.yml

@@ -0,0 +1,35 @@
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+# This workflow will download a prebuilt Ruby version, install dependencies and run tests with Rake
+# For more information see: https://github.com/marketplace/actions/setup-ruby-jruby-and-truffleruby
+
+name: Ruby
+
+on:
+  push:
+    branches: [ develop ]
+  pull_request:
+    branches: [ develop ]
+
+jobs:
+  test:
+
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: ['2.6', '2.7']
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+    # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
+    # change this to (see https://github.com/ruby/setup-ruby#versioning):
+    # uses: ruby/setup-ruby@v1
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+        bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+    - name: Run tests
+      run: bundle exec rspec ./spec/unit/

data/Rakefile

@@ -5,18 +5,18 @@ desc "Run all specs in spec directory"
 RSpec::Core::RakeTask.new(:spec)
 
 desc "Run the ci build (no integration tests)"
-task :ci do
-  system "rspec . -t ~deprecated -t ~integration"
+RSpec::Core::RakeTask.new(:ci) do |t|
+  t.rspec_opts = "--tag '~deprecated' --tag '~ezid'"
 end
 
 desc "Run tests of deprecated functionality"
-task :deprecated do
-  system "rspec . -t deprecated"
+RSpec::Core::RakeTask.new(:deprecated) do |t|
+  t.rspec_opts = "--tag deprecated"
 end
 
 desc "Run the integration tests"
-task :integration do
-  system "rspec . -t integration"
+RSpec::Core::RakeTask.new(:integration) do |t|
+  t.rspec_opts = "--tag integration"
 end
 
 task default: :spec

data/VERSION

@@ -1 +1 @@
-1.8.0
+1.9.0.rc1

data/ezid-client.gemspec

@@ -22,7 +22,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "hashie", "~> 3.4", ">= 3.4.3"
   spec.add_dependency "nokogiri"
 
-  spec.add_development_dependency "bundler", "~> 1.7"
+  spec.add_development_dependency "bundler"
   spec.add_development_dependency "byebug"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec", "~> 3.4"

data/lib/ezid/error.rb

@@ -8,4 +8,6 @@ module Ezid
   class NotAllowedError < Error; end
 
   class DeletionError < Error; end
+
+  class UnexpectedResponseError < Error; end
 end

data/lib/ezid/requests/request.rb

@@ -1,9 +1,10 @@
-require "delegate"
-require "uri"
-require "net/http"
-require "forwardable"
+require 'delegate'
+require 'uri'
+require 'net/http'
+require 'forwardable'
+require 'date'
 
-require_relative "../responses/response"
+require_relative '../responses/response'
 
 module Ezid
   #
@@ -31,7 +32,7 @@ module Ezid
       end
 
       def short_name
-        name.split("::").last.sub("Request", "")
+        name.split('::').last.sub('Request', '')
       end
     end
 
@@ -42,13 +43,24 @@ module Ezid
     def initialize(client, *args)
       @client = client
       super build_request
-      set_content_type("text/plain", charset: "UTF-8")
+      set_content_type('text/plain', charset: 'UTF-8')
     end
 
     # Executes the request and returns the response
     # @return [Ezid::Response] the response
     def execute
-      response_class.new(get_response_for_request)
+      retries = 0
+      begin
+        response_class.new(get_response_for_request)
+      rescue Net::HTTPServerException, UnexpectedResponseError => e
+        if retries < 2
+          sleep 15
+          retries += 1
+          retry
+        else
+          raise
+        end
+      end
     end
 
     # The request URI
@@ -91,6 +103,7 @@ module Ezid
 
     def get_response_for_request
       connection.start do |conn|
+        self['Accept'] = 'text/plain'
         add_authentication if authentication_required?
         add_metadata if has_metadata?
         conn.request(__getobj__)
@@ -113,7 +126,7 @@ module Ezid
     # Adds authentication data to the request
     def add_authentication
       if session.open?
-        self["Cookie"] = session.cookie
+        self['Cookie'] = session.cookie
       else
         basic_auth(user, password)
       end

data/lib/ezid/responses/response.rb

@@ -14,6 +14,19 @@ module Ezid
     # Error response status
     ERROR = "error".freeze
 
+    def initialize(http_response)
+      super
+
+      unless __getobj__.code =~ /2\d\d/
+        raise Error, "HTTP response error: %s %s" %
+                     [ __getobj__.code, __getobj__.message ]
+      end
+
+      unless status_line =~ /^(#{SUCCESS}|#{ERROR}): /
+        raise UnexpectedResponseError, __getobj__.body
+      end
+    end
+
     # The response status -- "success" or "error"
     # @return [String] the status
     def status

data/spec/integration/batch_download_spec.rb

@@ -1,7 +1,7 @@
 require 'tempfile'
 
 module Ezid
-  RSpec.describe BatchDownload, integration: true do
+  RSpec.describe BatchDownload, ezid: true do
 
     subject do
       a_week_ago = (Time.now - (7*24*60*60)).to_i
@@ -9,8 +9,8 @@ module Ezid
     end
 
     specify {
-      expect(subject.download_url).to match(/\Ahttp:\/\/ezid\.cdlib\.org\/download\/\w+\.zip\z/)
-      expect(subject.url).to match(/\Ahttp:\/\/ezid\.cdlib\.org\/download\/\w+\.zip\z/)
+      expect(subject.download_url).to match(/\Ahttps:\/\/ezid\.cdlib\.org\/download\/\w+\.zip\z/)
+      expect(subject.url).to match(/\Ahttps:\/\/ezid\.cdlib\.org\/download\/\w+\.zip\z/)
       Dir.mktmpdir do |tmpdir|
         expect(subject.file(path: tmpdir))
           .to match(/\A#{tmpdir}\/\w+\.zip\z/)

data/spec/integration/client_spec.rb

@@ -1,7 +1,7 @@
 require "time"
 
 module Ezid
-  RSpec.describe Client, integration: true do
+  RSpec.describe Client, ezid: true do
 
     shared_examples "an EZID client" do |client|
       it "should mint and modify" do

data/spec/integration/identifier_spec.rb

@@ -1,5 +1,5 @@
 module Ezid
-  RSpec.describe Identifier, integration: true do
+  RSpec.describe Identifier, ezid: true do
 
     before {
       @identifier = described_class.mint(TEST_ARK_SHOULDER, target: "http://example.com")
@@ -38,7 +38,8 @@ module Ezid
       end
       describe "delete" do
         subject { described_class.mint(TEST_ARK_SHOULDER, status: "reserved") }
-        it "deletes the identifier" do
+        # Getting 400 Bad Request response - DCS 3/22/21
+        xit "deletes the identifier" do
           subject.delete
           expect(subject).to be_deleted
           expect { described_class.find(subject.id) }.to raise_error(IdentifierNotFoundError)

data/spec/spec_helper.rb

@@ -15,6 +15,8 @@
 #
 # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 
+require "byebug"
+
 require "rspec/its"
 
 require "ezid/test_helper"

data/spec/unit/client_spec.rb

@@ -1,7 +1,15 @@
 module Ezid
   RSpec.describe Client do
 
+    let(:http_response) { double }
+
+    before do
+      allow(http_response).to receive(:value) { nil }
+      allow(http_response).to receive(:code) { '200' }
+    end
+
     describe "initialization without a block" do
+      let(:http_response) { double }
       it "should not login" do
         expect_any_instance_of(described_class).not_to receive(:login)
         described_class.new
@@ -151,9 +159,33 @@ EOS
     end
 
     describe "error handling" do
-      let(:http_response) { double(body: "error: bad request - no such identifier") }
-      it "should raise an exception" do
-        expect { subject.get_identifier_metadata("invalid") }.to raise_error(Error)
+      let(:stub_response) { GetIdentifierMetadataResponse.new(http_response) }
+      before do
+        allow(GetIdentifierMetadataRequest).to receive(:execute).with(subject, "invalid") { stub_response }
+      end
+
+      describe "HTTP error response" do
+        before do
+          allow(http_response).to receive(:code) { '500' }
+          allow(http_response).to receive(:message) { 'Internal Server Error' }
+        end
+        it "should raise an exception" do
+          expect { subject.get_identifier_metadata("invalid") }.to raise_error(Error)
+        end
+      end
+
+      describe "EZID API error response" do
+        let(:http_response) { double(body: "error: bad request - no such identifier") }
+        it "should raise an exception" do
+          expect { subject.get_identifier_metadata("invalid") }.to raise_error(Error)
+        end
+      end
+
+      describe "Unexpected response" do
+        let(:http_response) { double(body: "<html>\n<head><title>Ouch!</title></head>\n<body>Help!</body>\n</html>") }
+        it "should raise an exception" do
+          expect { subject.get_identifier_metadata("invalid") }.to raise_error(UnexpectedResponseError)
+        end
       end
     end
   end

data/spec/unit/metadata_spec.rb

@@ -1,3 +1,5 @@
+require 'time'
+
 module Ezid
   RSpec.describe Metadata do
     describe "initializer" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ezid-client
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.9.0.rc1
 platform: ruby
 authors:
 - David Chandek-Stark
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-06-29 00:00:00.000000000 Z
+date: 2021-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hashie
@@ -48,16 +48,16 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: byebug
   requirement: !ruby/object:Gem::Requirement
@@ -121,9 +121,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -185,7 +185,7 @@ homepage: https://github.com/duke-libraries/ezid-client
 licenses:
 - BSD-3-Clause
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -196,13 +196,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.12
-signing_key: 
+rubygems_version: 3.0.8
+signing_key:
 specification_version: 4
 summary: Ruby client for EZID API Version 2
 test_files:

data/.travis.yml

@@ -1,9 +0,0 @@
-sudo: false
-language: ruby
-rvm:
-  - 2.3.1
-  - 2.2
-  - 2.1
-cache:
-  - bundler
-script: "bundle exec rake ci"