dor-workflow-client 3.23.1 → 4.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2db7637402c0dd629a9d7ced145e20558b24717e56f7bc3fcd3adabbfca4d162
-  data.tar.gz: 7d4b3a6e7903ebb67a7786d1e10e7210020dee185674028507570e3ca271b796
+  metadata.gz: 77fd46c14ada1f0509732073801d7ea699624a79a41ca1cb7c8c47ba6acb7f55
+  data.tar.gz: fe4871b54d1df828b1cca4740aeafefdb1d1eee5af55f2040d23a0762a0cc56a
 SHA512:
-  metadata.gz: 14374bf91330e5b7280984ffec6a9547731b8a65a9dbe251d2496ed6c94e732efdddfaeab342ae077658e26b4db24af0c3f7fc59ca8c77ddf7fb3143e6c26453
-  data.tar.gz: ded3fd3fb5f3cbd7a3a3accbfa010e96cfd30df3773c799176a5645aede13fc5cbdf8171445b1bf60805980292014dd83c4255ad62fe83fb9a0440c78c5b5338
+  metadata.gz: e9404ff0f029c86849316f780d3e16cc2af0838313adf9034c7e4e78a779848a7c3cf7e3d7a773376907cde6bcbcecc894115de7386035240ec83294112405d7
+  data.tar.gz: 88b7444740059ecc1124793b69a1744ae0cf0518e0b5ac06f31690c1b569cc695cc6a322c2916ba1a382586b57a3c0a5ffdd2888893fdd077202bd98f6096f7a

data/.circleci/config.yml

@@ -1,35 +1,10 @@
 version: 2.1
-
-jobs:
+orbs:
+  ruby-rails: sul-dlss/ruby-rails@2.0.0
+workflows:
   build:
-    docker:
-      - image: circleci/ruby:2.7.1
-        environment:
-          CC_TEST_REPORTER_ID: 3853bc688a88a4de1a5568eaa8163d20db8637fe988540a935aff028190cff6a
-          RAILS_ENV: test
-    steps:
-      - checkout
-      - run:
-          name: Install Bundler
-          command: gem install bundler
-      - run:
-          name: Which bundler?
-          command: bundle -v
-      - run:
-          name: Bundle Install
-          command: bundle check || bundle install
-      - run:
-          name: Lint using rubocop
-          command: bundle exec rubocop
-      - run:
-          name: Setup Code Climate test-reporter
-          command: |
-            curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
-            chmod +x ./cc-test-reporter
-            ./cc-test-reporter before-build
-      - run:
-          name: rspec
-          command: bundle exec rspec
-      - run:
-          name: upload test coverage report to Code Climate
-          command: ./cc-test-reporter after-build --coverage-input-type simplecov --exit-code $?
+    jobs:
+      - ruby-rails/lint:
+          name: lint
+      - ruby-rails/test-gem:
+          name: test

data/.github/pull_request_template.md

@@ -1,12 +1,10 @@
-## Why was this change made?
+## Why was this change made? 🤔
 
 
 
-## How was this change tested?
+## How was this change tested? 🤨
 
-
-
-## Which documentation and/or configurations were updated?
+⚡ ⚠ If this change affects the API or other fundamentals of this service, ***run [integration tests](https://github.com/sul-dlss/infrastructure-integration-test)*** that exercise this service and/or test in [stage|qa] environment, in addition to specs. ⚡
 
 
 

data/.gitignore

@@ -4,7 +4,6 @@
 .config
 .yardoc
 .rvmrc
-Gemfile.lock
 InstalledFiles
 _yardoc
 coverage
@@ -18,4 +17,3 @@ test/version_tmp
 tmp
 .ruby-version
 .ruby-gemset
-

data/.rubocop.yml

@@ -1,11 +1,114 @@
 inherit_from: .rubocop_todo.yml
 
 AllCops:
-  TargetRubyVersion: 2.3
+  TargetRubyVersion: 3.0
 # Configuration parameters: AllowURI, URISchemes.
-Metrics/LineLength:
+Layout/LineLength:
   Max: 300
 
 Metrics/BlockLength:
   Exclude:
     - spec/**/*_spec.rb
+
+Gemspec/DeprecatedAttributeAssignment: # new in 1.10
+  Enabled: true
+Gemspec/RequireMFA: # new in 1.23
+  Enabled: true
+Layout/LineEndStringConcatenationIndentation: # new in 1.18
+  Enabled: true
+Layout/SpaceBeforeBrackets: # new in 1.7
+  Enabled: true
+Lint/AmbiguousAssignment: # new in 1.7
+  Enabled: true
+Lint/AmbiguousOperatorPrecedence: # new in 1.21
+  Enabled: true
+Lint/AmbiguousRange: # new in 1.19
+  Enabled: true
+Lint/DeprecatedConstants: # new in 1.8
+  Enabled: true
+Lint/DuplicateBranch: # new in 1.3
+  Enabled: true
+Lint/DuplicateRegexpCharacterClassElement: # new in 1.1
+  Enabled: true
+Lint/EmptyBlock: # new in 1.1
+  Enabled: true
+Lint/EmptyClass: # new in 1.3
+  Enabled: true
+Lint/EmptyInPattern: # new in 1.16
+  Enabled: true
+Lint/IncompatibleIoSelectWithFiberScheduler: # new in 1.21
+  Enabled: true
+Lint/LambdaWithoutLiteralBlock: # new in 1.8
+  Enabled: true
+Lint/NoReturnInBeginEndBlocks: # new in 1.2
+  Enabled: true
+Lint/NumberedParameterAssignment: # new in 1.9
+  Enabled: true
+Lint/OrAssignmentToConstant: # new in 1.9
+  Enabled: true
+Lint/RedundantDirGlobSort: # new in 1.8
+  Enabled: true
+Lint/RequireRelativeSelfPath: # new in 1.22
+  Enabled: true
+Lint/SymbolConversion: # new in 1.9
+  Enabled: true
+Lint/ToEnumArguments: # new in 1.1
+  Enabled: true
+Lint/TripleQuotes: # new in 1.9
+  Enabled: true
+Lint/UnexpectedBlockArity: # new in 1.5
+  Enabled: true
+Lint/UnmodifiedReduceAccumulator: # new in 1.1
+  Enabled: true
+Lint/UselessRuby2Keywords: # new in 1.23
+  Enabled: true
+Naming/BlockForwarding: # new in 1.24
+  Enabled: true
+Security/IoMethods: # new in 1.22
+  Enabled: true
+Style/ArgumentsForwarding: # new in 1.1
+  Enabled: true
+Style/CollectionCompact: # new in 1.2
+  Enabled: true
+Style/DocumentDynamicEvalDefinition: # new in 1.1
+  Enabled: true
+Style/EndlessMethod: # new in 1.8
+  Enabled: true
+Style/FileRead: # new in 1.24
+  Enabled: true
+Style/FileWrite: # new in 1.24
+  Enabled: true
+Style/HashConversion: # new in 1.10
+  Enabled: true
+Style/HashExcept: # new in 1.7
+  Enabled: true
+Style/IfWithBooleanLiteralBranches: # new in 1.9
+  Enabled: true
+Style/InPatternThen: # new in 1.16
+  Enabled: true
+Style/MapToHash: # new in 1.24
+  Enabled: true
+Style/MultilineInPatternThen: # new in 1.16
+  Enabled: true
+Style/NegatedIfElseCondition: # new in 1.2
+  Enabled: true
+Style/NilLambda: # new in 1.3
+  Enabled: true
+Style/NumberedParameters: # new in 1.22
+  Enabled: true
+Style/NumberedParametersLimit: # new in 1.22
+  Enabled: true
+Style/OpenStructUse: # new in 1.23
+  Enabled: true
+Style/QuotedSymbols: # new in 1.16
+  Enabled: true
+Style/RedundantArgument: # new in 1.4
+  Enabled: true
+Style/RedundantSelfAssignmentBranch: # new in 1.19
+  Enabled: true
+Style/SelectByRegexp: # new in 1.22
+  Enabled: true
+Style/StringChars: # new in 1.12
+  Enabled: true
+Style/SwapValues: # new in 1.1
+  Enabled: true

data/.rubocop_todo.yml

@@ -1,25 +1,58 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2020-01-06 12:37:25 -0600 using RuboCop version 0.63.1.
+# on 2022-06-28 04:56:14 UTC using RuboCop version 1.31.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 4
-Metrics/AbcSize:
-  Max: 26
+# Offense count: 1
+# Configuration parameters: Include.
+# Include: **/*.gemspec
+Gemspec/RequiredRubyVersion:
+  Exclude:
+    - 'dor-workflow-client.gemspec'
 
 # Offense count: 1
-Metrics/CyclomaticComplexity:
-  Max: 9
+# Configuration parameters: AllowComments, AllowEmptyLambdas.
+Lint/EmptyBlock:
+  Exclude:
+    - 'spec/workflow/client/workflow_routes_spec.rb'
 
 # Offense count: 1
-# Configuration parameters: CountComments, ExcludedMethods.
+Lint/NoReturnInBeginEndBlocks:
+  Exclude:
+    - 'lib/dor/workflow/client/status.rb'
+
+# Offense count: 5
+# Configuration parameters: IgnoredMethods, CountRepeatedAttributes.
+Metrics/AbcSize:
+  Max: 23
+
+# Offense count: 2
+# Configuration parameters: IgnoredMethods.
+Metrics/CyclomaticComplexity:
+  Max: 8
+
+# Offense count: 2
+# Configuration parameters: CountComments, CountAsOne, ExcludedMethods, IgnoredMethods.
 Metrics/MethodLength:
-  Max: 13
+  Max: 19
 
 # Offense count: 3
-# Configuration parameters: CountKeywordArgs.
+# Configuration parameters: CountKeywordArgs, MaxOptionalParameters.
 Metrics/ParameterLists:
   Max: 8
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+Style/KeywordParametersOrder:
+  Exclude:
+    - 'lib/dor/workflow/client/workflow_routes.rb'
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AllowModifier.
+Style/SoleNestedConditional:
+  Exclude:
+    - 'lib/dor/workflow/client/status.rb'

data/Gemfile

@@ -9,4 +9,5 @@ gem 'activesupport', ENV['RAILS_VERSION'] if ENV['RAILS_VERSION']
 
 group :development, :test do
   gem 'byebug'
+  gem 'rspec_junit_formatter' # For CircleCI
 end

data/Gemfile.lock

@@ -0,0 +1,118 @@
+PATH
+  remote: .
+  specs:
+    dor-workflow-client (4.1.0)
+      activesupport (>= 3.2.1, < 8)
+      deprecation (>= 0.99.0)
+      faraday (~> 2.0)
+      faraday-retry (~> 1.0)
+      nokogiri (~> 1.6)
+      zeitwerk (~> 2.1)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    activesupport (7.0.3)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+    addressable (2.8.0)
+      public_suffix (>= 2.0.2, < 5.0)
+    ast (2.4.2)
+    byebug (11.1.3)
+    concurrent-ruby (1.1.10)
+    crack (0.4.5)
+      rexml
+    deprecation (1.1.0)
+      activesupport
+    diff-lcs (1.5.0)
+    docile (1.4.0)
+    faraday (2.3.0)
+      faraday-net_http (~> 2.0)
+      ruby2_keywords (>= 0.0.4)
+    faraday-net_http (2.0.3)
+    faraday-retry (1.0.3)
+    hashdiff (1.0.1)
+    i18n (1.10.0)
+      concurrent-ruby (~> 1.0)
+    minitest (5.16.1)
+    nokogiri (1.13.6-x86_64-darwin)
+      racc (~> 1.4)
+    parallel (1.22.1)
+    parser (3.1.2.0)
+      ast (~> 2.4.1)
+    public_suffix (4.0.7)
+    racc (1.6.0)
+    rainbow (3.1.1)
+    rake (13.0.6)
+    regexp_parser (2.5.0)
+    rexml (3.2.5)
+    rspec (3.11.0)
+      rspec-core (~> 3.11.0)
+      rspec-expectations (~> 3.11.0)
+      rspec-mocks (~> 3.11.0)
+    rspec-core (3.11.0)
+      rspec-support (~> 3.11.0)
+    rspec-expectations (3.11.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.11.0)
+    rspec-mocks (3.11.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.11.0)
+    rspec-support (3.11.0)
+    rspec_junit_formatter (0.5.1)
+      rspec-core (>= 2, < 4, != 2.12.0)
+    rubocop (1.31.0)
+      parallel (~> 1.10)
+      parser (>= 3.1.0.0)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml (>= 3.2.5, < 4.0)
+      rubocop-ast (>= 1.18.0, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.18.0)
+      parser (>= 3.1.1.0)
+    rubocop-rake (0.6.0)
+      rubocop (~> 1.0)
+    rubocop-rspec (2.11.1)
+      rubocop (~> 1.19)
+    ruby-progressbar (1.11.0)
+    ruby2_keywords (0.0.5)
+    simplecov (0.21.2)
+      docile (~> 1.1)
+      simplecov-html (~> 0.11)
+      simplecov_json_formatter (~> 0.1)
+    simplecov-html (0.12.3)
+    simplecov_json_formatter (0.1.4)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (2.2.0)
+    webmock (3.14.0)
+      addressable (>= 2.8.0)
+      crack (>= 0.3.2)
+      hashdiff (>= 0.4.0, < 2.0.0)
+    webrick (1.7.0)
+    yard (0.9.28)
+      webrick (~> 1.7.0)
+    zeitwerk (2.6.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  byebug
+  dor-workflow-client!
+  rake
+  rspec (~> 3.3)
+  rspec_junit_formatter
+  rubocop (~> 1.24)
+  rubocop-rake
+  rubocop-rspec (~> 2.1)
+  simplecov
+  webmock
+  yard
+
+BUNDLED WITH
+   2.2.32

data/README.md

@@ -11,7 +11,7 @@ https://consul.stanford.edu/display/DOR/DOR+services#DORservices-initializeworkf
 
 ## Usage
 
-You should initialize a `Dor::Workflow::Client` object in your application configuration, i.e. in a bootup or startup method like:
+Initialize a `Dor::Workflow::Client` object in your application configuration, i.e. in a bootup or startup method like:
 
 ```ruby
 client = Dor::Workflow::Client.new(url: 'https://test-server.edu/workflow/')

data/dor-workflow-client.gemspec

@@ -14,20 +14,24 @@ Gem::Specification.new do |gem|
   gem.homepage      = 'https://consul.stanford.edu/display/DOR/DOR+services#DORservices-initializeworkflow'
 
   gem.files         = `git ls-files`.split($INPUT_RECORD_SEPARATOR)
-  gem.test_files    = gem.files.grep(%r{^(spec)/})
   gem.require_paths = ['lib']
+  gem.required_ruby_version = '>= 2.7'
 
-  gem.add_dependency 'activesupport', '>= 3.2.1', '< 7'
+  gem.add_dependency 'activesupport', '>= 3.2.1', '< 8'
   gem.add_dependency 'deprecation', '>= 0.99.0'
-  gem.add_dependency 'faraday', '>= 0.9.2', '< 2.0'
-  gem.add_dependency 'faraday_middleware'
+  gem.add_dependency 'faraday', '~> 2.0'
+  gem.add_dependency 'faraday-retry', '~> 1.0'
+
   gem.add_dependency 'nokogiri', '~> 1.6'
   gem.add_dependency 'zeitwerk', '~> 2.1'
 
   gem.add_development_dependency 'rake'
   gem.add_development_dependency 'rspec', '~> 3.3'
-  gem.add_development_dependency 'rubocop', '~> 0.63.1'
-  gem.add_development_dependency 'simplecov', '~> 0.17.0' # CodeClimate cannot use SimpleCov >= 0.18.0 for generating test coverage
+  gem.add_development_dependency 'rubocop', '~> 1.24'
+  gem.add_development_dependency 'rubocop-rake'
+  gem.add_development_dependency 'rubocop-rspec', '~> 2.1'
+  gem.add_development_dependency 'simplecov'
   gem.add_development_dependency 'webmock'
   gem.add_development_dependency 'yard'
+  gem.metadata['rubygems_mfa_required'] = 'true'
 end

data/lib/dor/workflow/client/connection_factory.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'faraday_middleware'
-
 module Dor
   module Workflow
     class Client
@@ -17,11 +15,9 @@ module Dor
           @logger = logger
         end
 
-        # rubocop:disable Metrics/MethodLength
         def build_connection
           Faraday.new(url: url) do |faraday|
             faraday.use Faraday::Response::RaiseError # raise exceptions on 40x, 50x responses
-            faraday.use FaradayMiddleware::FollowRedirects, limit: 3
             faraday.options.params_encoder = Faraday::FlatParamsEncoder
             if timeout
               faraday.options.timeout = timeout
@@ -40,7 +36,6 @@ module Dor
             faraday.adapter Faraday.default_adapter # Last middleware must be the adapter
           end
         end
-        # rubocop:enable Metrics/MethodLength
 
         def user_agent
           "dor-workflow-client #{VERSION}"
@@ -51,17 +46,17 @@ module Dor
         attr_reader :logger, :timeout, :url
 
         def retriable_methods
-          Faraday::Request::Retry::IDEMPOTENT_METHODS + [:post]
+          Faraday::Retry::Middleware::IDEMPOTENT_METHODS + [:post]
         end
 
         def retriable_exceptions
-          Faraday::Request::Retry::DEFAULT_EXCEPTIONS + [Faraday::ConnectionFailed]
+          Faraday::Retry::Middleware::DEFAULT_EXCEPTIONS + [Faraday::ConnectionFailed]
         end
 
         def retry_block
           lambda do |env, _opts, retries, exception|
             logger.warn "retrying connection (#{retries} remaining) to #{env.url}: (#{exception.class}) " \
-              "#{exception.message} #{env.status}"
+                        "#{exception.message} #{env.status}"
           end
         end
 

data/lib/dor/workflow/client/lifecycle_routes.rb

@@ -84,7 +84,6 @@ module Dor
         # rubocop:enable Metrics/MethodLength
 
         # @return [Array<Hash>]
-        # rubocop:disable Metrics/MethodLength
         def milestones(*args)
           case args.size
           when 2
@@ -105,7 +104,6 @@ module Dor
             { milestone: node.text, at: Time.parse(node['date']), version: node['version'] }
           end
         end
-        # rubocop:enable Metrics/MethodLength
 
         private
 

data/lib/dor/workflow/client/queues.rb

@@ -63,7 +63,7 @@ module Dor
             )
             args.shift # ditch the `repo` argument
           end
-          uri_string = build_queued_uri(args.first) + '&count-only=true'
+          uri_string = "#{build_queued_uri(args.first)}&count-only=true"
           doc = Nokogiri::XML(requestor.request(uri_string))
           doc.at_xpath('/objects/@count').value.to_i
         end
@@ -139,13 +139,13 @@ module Dor
         # @example
         #     client.errored_objects_for_workstep('accessionWF','content-metadata')
         #     => {"druid:qd556jq0580"=>"druid:qd556jq0580 - Item error; caused by
-        #        #<Rubydora::FedoraInvalidRequest: Error modifying datastream contentMetadata for druid:qd556jq0580. See logger for details>"}
+        #        blah blah. See logger for details>"}
         def errored_objects_for_workstep(workflow, step, repository = nil)
           Deprecation.warn(self, 'the third argument to `#errored_objects_for_workstep` is unused and will go away in Dor::Workflow::Client 4.0.0. omit argument to silence.') unless repository.nil?
           resp = requestor.request "workflow_queue?workflow=#{workflow}&error=#{step}"
-          Nokogiri::XML(resp).xpath('//object').map do |node|
+          Nokogiri::XML(resp).xpath('//object').to_h do |node|
             [node['id'], node['errorMessage']]
-          end.to_h
+          end
         end
 
         # Used by preservation robots stats reporter

data/lib/dor/workflow/client/status.rb

@@ -42,7 +42,6 @@ module Dor
         end
 
         # @return [Hash{Symbol => Object}] including :status_code and :status_time
-        # rubocop:disable Metrics/MethodLength
         def info
           @info ||= begin
             # if we have an accessioned milestone, this is the last possible step and should be the status regardless of time stamp
@@ -64,7 +63,6 @@ module Dor
             { status_code: status_code, status_time: status_time }
           end
         end
-        # rubocop:enable Metrics/MethodLength
 
         def status_code
           info.fetch(:status_code)

data/lib/dor/workflow/client/version.rb

@@ -3,7 +3,7 @@
 module Dor
   module Workflow
     class Client
-      VERSION = '3.23.1'
+      VERSION = '4.1.0'
     end
   end
 end

data/lib/dor/workflow/client/workflow_routes.rb

@@ -23,12 +23,12 @@ module Dor
         # @return [Boolean] always true
         #
         def create_workflow(_repo, druid, workflow_name, _wf_xml, opts = {})
-          create_workflow_by_name(druid, workflow_name, opts)
+          create_workflow_by_name(druid, workflow_name, **opts)
         end
         deprecation_deprecate create_workflow: 'use create_workflow_by_name instead'
 
         # Creates a workflow for a given object in the repository.  If this particular workflow for this objects exists,
-        # it will replace the old workflow.  You have the option of creating a datastream or not.
+        # it will replace the old workflow.
         # Returns true on success.  Caller must handle any exceptions.
         #
         # @param [String] druid The id of the object
@@ -124,7 +124,6 @@ module Dor
         # @param [String] workflow The name of the workflow
         # @param [String] process The name of the process step
         # @return [String] status for repo-workflow-process-druid
-        # rubocop:disable Metrics/MethodLength
         # rubocop:disable Metrics/AbcSize
         def workflow_status(*args)
           case args.length
@@ -147,16 +146,14 @@ module Dor
           process = processes.max { |a, b| a.attr('version').to_i <=> b.attr('version').to_i }
           process&.attr('status')
         end
-        # rubocop:enable Metrics/AbcSize
-        # rubocop:enable Metrics/MethodLength
 
+        # rubocop:enable Metrics/AbcSize
         #
         # Retrieves the raw XML for the given workflow
         # @param [String] repo The repository the object resides in.  Currently recoginzes "dor" and "sdr".
         # @param [String] druid The id of the object
         # @param [String] workflow The name of the workflow
         # @return [String] XML of the workflow
-        # rubocop:disable Metrics/MethodLength
         def workflow_xml(*args)
           case args.length
           when 3
@@ -177,8 +174,6 @@ module Dor
         end
         deprecation_deprecate workflow_xml: 'workflow_xml will not be replaced'
 
-        # rubocop:enable Metrics/MethodLength
-
         # Updates the status of one step in a workflow to error.
         # Returns true on success.  Caller must handle any exceptions
         #
@@ -275,19 +270,29 @@ module Dor
         end
 
         # Deletes a workflow from a particular repository and druid. This is only used by Hydrus.
-        # @param [String] repo The repository the object resides in.  The service recoginzes "dor" and "sdr" at the moment
         # @param [String] druid The id of the object to delete the workflow from
         # @param [String] workflow The name of the workflow to be deleted
         # @param [Integer] version The version of the workflow to delete
         # @return [Boolean] always true
-        def delete_workflow(repo, druid, workflow, version: nil)
+        def delete_workflow(*args)
+          case args.length
+          when 3..4
+            Deprecation.warn(self, 'Calling delete_workflow with positional args is deprecated, use kwargs instead')
+            (_repo, druid, workflow, version_hash) = *args
+            version = version_hash && version_hash[:version]
+          when 1
+            opts = args.first
+            druid = opts[:druid]
+            workflow = opts[:workflow]
+            version = opts[:version]
+          end
           qs_args = if version
                       "?version=#{version}"
                     else
                       Deprecation.warn(self, 'Calling delete_workflow without passing version is deprecated and will result in an error in dor-workflow-client 4.0')
                       ''
                     end
-          requestor.request "#{repo}/objects/#{druid}/workflows/#{workflow}#{qs_args}", 'delete'
+          requestor.request "/objects/#{druid}/workflows/#{workflow}#{qs_args}", 'delete'
           true
         end
 
@@ -313,8 +318,8 @@ module Dor
         # @return [String]
         def create_process_xml(params)
           builder = Nokogiri::XML::Builder.new do |xml|
-            attrs = params.reject { |_k, v| v.nil? }
-            attrs = Hash[attrs.map { |k, v| [k.to_s.camelize(:lower), v] }] # camelize all the keys in the attrs hash
+            attrs = params.compact
+            attrs = attrs.transform_keys { |k| k.to_s.camelize(:lower) } # camelize all the keys in the attrs hash
             xml.process(attrs)
           end
           builder.to_xml

data/lib/dor/workflow/client.rb

@@ -5,6 +5,7 @@ require 'active_support/core_ext'
 require 'nokogiri'
 require 'zeitwerk'
 require 'faraday'
+require 'faraday/retry'
 require 'deprecation'
 
 loader = Zeitwerk::Loader.new

data/lib/dor/workflow/response/workflow.rb

@@ -35,7 +35,7 @@ module Dor
         def process_for_recent_version(name:)
           nodes = process_nodes_for(name: name)
           node = nodes.max { |a, b| a.attr('version').to_i <=> b.attr('version').to_i }
-          attributes = node ? Hash[node.attributes.collect { |k, v| [k.to_sym, v.value] }] : {}
+          attributes = node ? node.attributes.to_h { |k, v| [k.to_sym, v.value] } : {}
           Process.new(parent: self, **attributes)
         end
 

data/spec/models/response/workflow_spec.rb

@@ -72,7 +72,7 @@ RSpec.describe Dor::Workflow::Response::Workflow do
     end
   end
 
-  describe '#active?' do
+  describe '#active_for?' do
     subject { instance.active_for?(version: 2) }
 
     context 'when the workflow has not been instantiated for the given version' do

data/spec/workflow/client/workflow_routes_spec.rb

@@ -147,6 +147,19 @@ RSpec.describe Dor::Workflow::Client::WorkflowRoutes do
     end
   end
 
+  describe '#delete_all_workflows' do
+    subject(:delete_all_workflows) do
+      routes.delete_all_workflows(pid: 'druid:mw971zk1113')
+    end
+    let(:mock_requestor) { instance_double(Dor::Workflow::Client::Requestor, request: nil) }
+
+    it 'sends a delete request' do
+      delete_all_workflows
+      expect(mock_requestor).to have_received(:request)
+        .with('objects/druid:mw971zk1113/workflows', 'delete')
+    end
+  end
+
   describe '#all_workflows' do
     let(:xml) do
       <<~XML

data/spec/workflow/client_spec.rb

@@ -729,22 +729,34 @@ RSpec.describe Dor::Workflow::Client do
       end
     end
 
-    context 'without a version' do
-      let(:url) { "/objects/#{@druid}/workflows/accessionWF" }
+    context 'with positional args' do
+      context 'without a version' do
+        let(:url) { "/objects/#{@druid}/workflows/accessionWF" }
 
-      it 'sends a delete request to the workflow service' do
-        expect(Deprecation).to receive(:warn)
-        expect(mock_http_connection).to receive(:delete).with(url).and_call_original
-        client.delete_workflow(nil, @druid, 'accessionWF')
+        it 'sends a delete request to the workflow service' do
+          expect(Deprecation).to receive(:warn).twice
+          expect(mock_http_connection).to receive(:delete).with(url).and_call_original
+          client.delete_workflow(nil, @druid, 'accessionWF')
+        end
+      end
+
+      context 'with a version' do
+        let(:url) { "/objects/#{@druid}/workflows/accessionWF?version=5" }
+
+        it 'sends a delete request to the workflow service' do
+          expect(Deprecation).to receive(:warn)
+          expect(mock_http_connection).to receive(:delete).with(url).and_call_original
+          client.delete_workflow(nil, @druid, 'accessionWF', version: 5)
+        end
       end
     end
 
-    context 'with a version' do
+    context 'with kwargs' do
       let(:url) { "/objects/#{@druid}/workflows/accessionWF?version=5" }
 
       it 'sends a delete request to the workflow service' do
         expect(mock_http_connection).to receive(:delete).with(url).and_call_original
-        client.delete_workflow(nil, @druid, 'accessionWF', version: 5)
+        client.delete_workflow(druid: @druid, workflow: 'accessionWF', version: 5)
       end
     end
   end
@@ -804,7 +816,7 @@ RSpec.describe Dor::Workflow::Client do
 
     let(:stubs) do
       Faraday::Adapter::Test::Stubs.new do |stub|
-        stub.get('workflow_queue/lane_ids?lane_ids?step=dor:accessionWF:shelve') do |_env|
+        stub.get('workflow_queue/lane_ids?step=accessionWF:shelve') do |_env|
           [200, {}, <<-XML]
           <lanes>
             <lane id="lane1"/>

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: dor-workflow-client
 version: !ruby/object:Gem::Version
-  version: 3.23.1
+  version: 4.1.0
 platform: ruby
 authors:
 - Willy Mene
 - Darren Hardy
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-08 00:00:00.000000000 Z
+date: 2022-06-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -20,7 +20,7 @@ dependencies:
         version: 3.2.1
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7'
+        version: '8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -30,7 +30,7 @@ dependencies:
         version: 3.2.1
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7'
+        version: '8'
 - !ruby/object:Gem::Dependency
   name: deprecation
   requirement: !ruby/object:Gem::Requirement
@@ -49,36 +49,30 @@ dependencies:
   name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.9.2
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.9.2
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: faraday_middleware
+  name: faraday-retry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
@@ -141,28 +135,56 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.63.1
+        version: '1.24'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.63.1
+        version: '1.24'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.0
+        version: '2.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.0
+        version: '2.1'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -206,6 +228,7 @@ files:
 - ".rubocop_todo.yml"
 - ".yardopts"
 - Gemfile
+- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -240,8 +263,9 @@ files:
 - spec/workflow/client_spec.rb
 homepage: https://consul.stanford.edu/display/DOR/DOR+services#DORservices-initializeworkflow
 licenses: []
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -249,27 +273,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.32
+signing_key:
 specification_version: 4
 summary: Provides convenience methods to work with the DOR Workflow Service
-test_files:
-- spec/models/response/process_spec.rb
-- spec/models/response/workflow_spec.rb
-- spec/models/response/workflows_spec.rb
-- spec/spec_helper.rb
-- spec/workflow/client/connection_factory_spec.rb
-- spec/workflow/client/lifecycle_routes_spec.rb
-- spec/workflow/client/requestor_spec.rb
-- spec/workflow/client/status_spec.rb
-- spec/workflow/client/version_routes_spec.rb
-- spec/workflow/client/workflow_routes_spec.rb
-- spec/workflow/client/workflow_template_spec.rb
-- spec/workflow/client_spec.rb
+test_files: []