dor-workflow-client 4.1.0 → 5.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 77fd46c14ada1f0509732073801d7ea699624a79a41ca1cb7c8c47ba6acb7f55
-  data.tar.gz: fe4871b54d1df828b1cca4740aeafefdb1d1eee5af55f2040d23a0762a0cc56a
+  metadata.gz: 660ca959342689ef767c9b7d6ce72f90ec5d568227e1acab968c7cf5b2e9bb95
+  data.tar.gz: e5c19e4cf9c2f79edc1cefaa0b9dbe4954866c2f29357fd58425d173662422ad
 SHA512:
-  metadata.gz: e9404ff0f029c86849316f780d3e16cc2af0838313adf9034c7e4e78a779848a7c3cf7e3d7a773376907cde6bcbcecc894115de7386035240ec83294112405d7
-  data.tar.gz: 88b7444740059ecc1124793b69a1744ae0cf0518e0b5ac06f31690c1b569cc695cc6a322c2916ba1a382586b57a3c0a5ffdd2888893fdd077202bd98f6096f7a
+  metadata.gz: d7f985246d0238a872a26f9fcab9a65f7d2795be9b7946dffa4fba5cb2557203184de1ec1e1b4a758405a397c34dc991ef74290e8d5d5d28691b69984c18ddfd
+  data.tar.gz: c458009469d0684198722475fbe01a1dce917272298b18494979ed63957e399ea76830f1c22036ae0e21b46fb7ac31481b1e37e5f715916ee217648cb62526cc

data/.circleci/config.yml

@@ -1,6 +1,6 @@
 version: 2.1
 orbs:
-  ruby-rails: sul-dlss/ruby-rails@2.0.0
+  ruby-rails: sul-dlss/ruby-rails@3.1.2
 workflows:
   build:
     jobs:

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    dor-workflow-client (4.1.0)
+    dor-workflow-client (5.0.0)
       activesupport (>= 3.2.1, < 8)
       deprecation (>= 0.99.0)
       faraday (~> 2.0)
@@ -12,7 +12,7 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (7.0.3)
+    activesupport (7.0.3.1)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
@@ -28,16 +28,19 @@ GEM
       activesupport
     diff-lcs (1.5.0)
     docile (1.4.0)
-    faraday (2.3.0)
+    faraday (2.4.0)
       faraday-net_http (~> 2.0)
       ruby2_keywords (>= 0.0.4)
-    faraday-net_http (2.0.3)
+    faraday-net_http (2.1.0)
     faraday-retry (1.0.3)
     hashdiff (1.0.1)
-    i18n (1.10.0)
+    i18n (1.12.0)
       concurrent-ruby (~> 1.0)
-    minitest (5.16.1)
-    nokogiri (1.13.6-x86_64-darwin)
+    json (2.6.2)
+    mini_portile2 (2.8.0)
+    minitest (5.16.2)
+    nokogiri (1.13.8)
+      mini_portile2 (~> 2.8.0)
       racc (~> 1.4)
     parallel (1.22.1)
     parser (3.1.2.0)
@@ -63,21 +66,22 @@ GEM
     rspec-support (3.11.0)
     rspec_junit_formatter (0.5.1)
       rspec-core (>= 2, < 4, != 2.12.0)
-    rubocop (1.31.0)
+    rubocop (1.32.0)
+      json (~> 2.3)
       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)
+      rubocop-ast (>= 1.19.1, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.18.0)
+    rubocop-ast (1.19.1)
       parser (>= 3.1.1.0)
     rubocop-rake (0.6.0)
       rubocop (~> 1.0)
-    rubocop-rspec (2.11.1)
-      rubocop (~> 1.19)
+    rubocop-rspec (2.12.1)
+      rubocop (~> 1.31)
     ruby-progressbar (1.11.0)
     ruby2_keywords (0.0.5)
     simplecov (0.21.2)
@@ -86,7 +90,7 @@ GEM
       simplecov_json_formatter (~> 0.1)
     simplecov-html (0.12.3)
     simplecov_json_formatter (0.1.4)
-    tzinfo (2.0.4)
+    tzinfo (2.0.5)
       concurrent-ruby (~> 1.0)
     unicode-display_width (2.2.0)
     webmock (3.14.0)
@@ -115,4 +119,4 @@ DEPENDENCIES
   yard
 
 BUNDLED WITH
-   2.2.32
+   2.3.17

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

@@ -14,39 +14,12 @@ module Dor
         # @param [String] repo The repository the object resides in. This parameter is deprecated
         # @param [String] druid object id
         # @param [String] milestone_name the name of the milestone being queried for
-        # @param [Number] version the version to query for
+        # @param [Number] version (nil) the version to query for
         # @param [Boolean] active_only (false) if true, return only lifecycle steps for versions that have all processes complete
         # @return [Time] when the milestone was achieved.  Returns nil if the milestone does not exist
-        #
-        # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
-        def lifecycle(*args)
-          case args.size
-          when 4
-            Deprecation.warn(self, 'you provided 4 args, but lifecycle now takes kwargs')
-            (repo, druid, milestone_name) = args[0..2]
-            version = args[3][:version]
-            active_only = args[3][:active_only] || false
-          when 3
-            Deprecation.warn(self, 'you provided 3 args, but lifecycle now takes kwargs')
-            (repo, druid, milestone_name) = args
-            version = nil
-            active_only = false
-          when 1
-            opts = args.first
-            repo = opts[:repo]
-            druid = opts[:druid]
-            milestone_name = opts[:milestone_name]
-            version = opts[:version]
-            active_only = opts[:active_only] || false
-          else
-            raise ArgumentError, 'wrong number of arguments, must be 1, or 3-5'
-          end
-
-          Deprecation.warn(self, 'passing the repo parameter to lifecycle is no longer necessary. This will raise an error in dor-workflow-client version 4') if repo
-
+        def lifecycle(druid:, milestone_name:, version: nil, active_only: false)
           filter_milestone(query_lifecycle(druid, version: version, active_only: active_only), milestone_name)
         end
-        # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
 
         # Returns the Date for a requested milestone ONLY for the current version.
         # This is slow as the workflow server will query dor-services-app for the version.
@@ -55,50 +28,12 @@ module Dor
         # @param [String] milestone_name the name of the milestone being queried for
         # @param [Number] version the version to query for
         # @return [Time] when the milestone was achieved.  Returns nil if the milestone does not exis
-        #
-        # rubocop:disable Metrics/MethodLength
-        def active_lifecycle(*args)
-          case args.size
-          when 4
-            Deprecation.warn(self, 'you provided 4 args, but active_lifecycle now takes kwargs')
-            (repo, druid, milestone_name) = args[0..2]
-            version = args[3][:version]
-          when 3
-            Deprecation.warn(self, 'you provided 3 args, but active_lifecycle now takes kwargs')
-            (repo, druid, milestone_name) = args
-            version = nil
-          when 1
-            opts = args.first
-            repo = opts[:repo]
-            druid = opts[:druid]
-            milestone_name = opts[:milestone_name]
-            version = opts[:version]
-          else
-            raise ArgumentError, 'wrong number of arguments, must be 1, 3, or 4'
-          end
-
-          Deprecation.warn(self, 'passing the repo parameter to active_lifecycle is no longer necessary. This will raise an error in dor-workflow-client version 4') if repo
-
+        def active_lifecycle(druid:, milestone_name:, version:)
           lifecycle(druid: druid, milestone_name: milestone_name, version: version, active_only: true)
         end
-        # rubocop:enable Metrics/MethodLength
 
         # @return [Array<Hash>]
-        def milestones(*args)
-          case args.size
-          when 2
-            Deprecation.warn(self, 'you provided 2 args, but active_lifecycle now takes kwargs')
-            (repo, druid) = args
-          when 1
-            opts = args.first
-            repo = opts[:repo]
-            druid = opts.fetch(:druid)
-          else
-            raise ArgumentError, 'wrong number of arguments, must be 1-2'
-          end
-
-          Deprecation.warn(self, 'passing the repo parameter to active_lifecycle is no longer necessary. This will raise an error in dor-workflow-client version 4') if repo
-
+        def milestones(druid:)
           doc = query_lifecycle(druid, active_only: false)
           doc.xpath('//lifecycle/milestone').collect do |node|
             { milestone: node.text, at: Time.parse(node['date']), version: node['version'] }

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

@@ -11,59 +11,35 @@ module Dor
 
         # Returns all the distinct laneIds for a given workflow step
         #
-        # @param [String] repo -- deprecated, ignored by workflow service
         # @param [String] workflow name
         # @param [String] process name
         # @return [Array<String>] all of the distinct laneIds.  Array will be empty if no lane ids were found
-        def lane_ids(*args)
-          if args.count == 3
-            Deprecation.warn(
-              self,
-              '`#lane_ids` only takes two args: workflow name, & process/step name. This will raise an exception in Dor::Workflow::Client 4.0.0'
-            )
-            args.shift # ditch the `repo` argument
-          end
-          uri = "workflow_queue/lane_ids?step=#{args.first}:#{args.second}"
+        def lane_ids(workflow, process)
+          uri = "workflow_queue/lane_ids?step=#{workflow}:#{process}"
           doc = Nokogiri::XML(requestor.request(uri))
           doc.xpath('/lanes/lane').map { |n| n['id'] }
         end
 
         # Gets all of the workflow steps that have a status of 'queued' that have a last-updated timestamp older than the number of hours passed in
         #   This will enable re-queueing of jobs that have been lost by the job manager
-        # @param [String] repository -- deprecated, ignored by workflow service
         # @param [Hash] opts optional values for query
         # @option opts [Integer] :hours_ago steps older than this value will be returned by the query.  If not passed in, the service defaults to 0 hours,
         #   meaning you will get all queued workflows
         # @option opts [Integer] :limit sets the maximum number of workflow steps that can be returned.  Defaults to no limit
         # @return [Array[Hash]] each Hash represents a workflow step.  It will have the following keys:
         #  :workflow, :step, :druid, :lane_id
-        def stale_queued_workflows(*args)
-          if args.count == 2
-            Deprecation.warn(
-              self,
-              '`#stale_queued_workflows` only takes one arg: a hash. This will raise an exception in Dor::Workflow::Client 4.0.0'
-            )
-            args.shift # ditch the `repo` argument
-          end
-          uri_string = build_queued_uri(args.first)
+        def stale_queued_workflows(opts)
+          uri_string = build_queued_uri(opts)
           parse_queued_workflows_response requestor.request(uri_string)
         end
 
         # Returns a count of workflow steps that have a status of 'queued' that have a last-updated timestamp older than the number of hours passed in
-        # @param [String] repository -- deprecated, ignored by workflow service
         # @param [Hash] opts optional values for query
         # @option opts [Integer] :hours_ago steps older than this value will be returned by the query.  If not passed in, the service defaults to 0 hours,
         #   meaning you will get all queued workflows
         # @return [Integer] number of stale, queued steps if the :count_only option was set to true
-        def count_stale_queued_workflows(*args)
-          if args.count == 2
-            Deprecation.warn(
-              self,
-              '`#count_stale_queued_workflows` only takes one arg: a hash. This will raise an exception in Dor::Workflow::Client 4.0.0'
-            )
-            args.shift # ditch the `repo` argument
-          end
-          uri_string = "#{build_queued_uri(args.first)}&count-only=true"
+        def count_stale_queued_workflows(opts)
+          uri_string = "#{build_queued_uri(opts)}&count-only=true"
           doc = Nokogiri::XML(requestor.request(uri_string))
           doc.at_xpath('/objects/@count').value.to_i
         end
@@ -102,7 +78,6 @@ module Dor
         #     }
         #
         def objects_for_workstep(completed, waiting, lane_id = 'default', options = {})
-          Deprecation.warn(self, 'the `:default_repository` option in `#objects_for_workstep` is unused and will go away in Dor::Workflow::Client 4.0.0. omit argument to silence.') if options[:default_repository]
           waiting_param = qualify_step(options[:default_workflow], waiting)
           uri_string = "workflow_queue?waiting=#{waiting_param}"
           if completed
@@ -133,15 +108,13 @@ module Dor
         #
         # @param [String] workflow name
         # @param [String] step name
-        # @param [String] repository -- deprecated, ignored by workflow service
         #
         # @return [Hash] hash of results, with key has a druid, and value as the error message
         # @example
         #     client.errored_objects_for_workstep('accessionWF','content-metadata')
         #     => {"druid:qd556jq0580"=>"druid:qd556jq0580 - Item error; caused by
         #        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?
+        def errored_objects_for_workstep(workflow, step)
           resp = requestor.request "workflow_queue?workflow=#{workflow}&error=#{step}"
           Nokogiri::XML(resp).xpath('//object').to_h do |node|
             [node['id'], node['errorMessage']]
@@ -153,11 +126,9 @@ module Dor
         # @param [String] workflow name
         # @param [String] step name
         # @param [String] type
-        # @param [String] repo -- deprecated, ignored by workflow service
         #
         # @return [Hash] hash of results, with key has a druid, and value as the error message
-        def count_objects_in_step(workflow, step, type, repo = nil)
-          Deprecation.warn(self, 'the fourth argument to `#count_objects_in_step` is unused and will go away in Dor::Workflow::Client 4.0.0. omit argument to silence.') unless repo.nil?
+        def count_objects_in_step(workflow, step, type)
           resp = requestor.request "workflow_queue?workflow=#{workflow}&#{type}=#{step}"
           extract_object_count(resp)
         end
@@ -166,11 +137,9 @@ module Dor
         #
         # @param [String] workflow name
         # @param [String] step name
-        # @param [String] repository -- deprecated, ignored by workflow service
         #
         # @return [Integer] Number of objects with this repository:workflow:step that have a status of 'error'
-        def count_errored_for_workstep(workflow, step, repository = nil)
-          Deprecation.warn(self, 'the third argument to `#count_errored_for_workstep` is unused and will go away in Dor::Workflow::Client 4.0.0. omit argument to silence.') unless repository.nil?
+        def count_errored_for_workstep(workflow, step)
           count_objects_in_step(workflow, step, 'error')
         end
 
@@ -178,11 +147,9 @@ module Dor
         #
         # @param [String] workflow name
         # @param [String] step name
-        # @param [String] repository -- deprecated, ignored by workflow service
         #
         # @return [Integer] Number of objects with this repository:workflow:step that have a status of 'queued'
-        def count_queued_for_workstep(workflow, step, repository = nil)
-          Deprecation.warn(self, 'the third argument to `#count_queued_for_workstep` is unused and will go away in Dor::Workflow::Client 4.0.0. omit argument to silence.') unless repository.nil?
+        def count_queued_for_workstep(workflow, step)
           count_objects_in_step(workflow, step, 'queued')
         end
 

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

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

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

@@ -17,32 +17,10 @@ module Dor
         # @param [String] repo The repository the object resides in. This parameter is deprecated
         # @param [String] druid The id of the object to delete the workflow from
         # @param [Boolean] create_accession_wf Option to create accessionWF when closing a version.  Defaults to true
-        # rubocop:disable Metrics/MethodLength
-        def close_version(*args)
-          case args.size
-          when 3
-            Deprecation.warn(self, 'you provided 3 args, but close_version now takes kwargs')
-            (repo, druid, create_accession_wf) = args
-          when 2
-            Deprecation.warn(self, 'you provided 2 args, but close_version now takes kwargs')
-            (repo, druid) = args
-            create_accession_wf = true
-          when 1
-            opts = args.first
-            repo = opts[:repo]
-            druid = opts[:druid]
-            version = opts[:version]
-            create_accession_wf = opts.key?(:create_accession_wf) ? opts[:create_accession_wf] : true
-          else
-            raise ArgumentError, 'wrong number of arguments, must be 1-3'
-          end
-
-          Deprecation.warn(self, 'passing the repo parameter to close_version is no longer necessary. This will raise an error in dor-workflow-client version 4') if repo
-
+        def close_version(druid:, version:, create_accession_wf: true)
           requestor.request(construct_url(druid, version, create_accession_wf), 'post', '')
           true
         end
-        # rubocop:enable Metrics/MethodLength
 
         private
 

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

@@ -4,29 +4,11 @@ module Dor
   module Workflow
     class Client
       # Makes requests relating to a workflow
-      # rubocop:disable Metrics/ClassLength
       class WorkflowRoutes
-        extend Deprecation
-
         def initialize(requestor:)
           @requestor = requestor
         end
 
-        # This method is deprecated and calls create_workflow_by_name.
-        #
-        # @param [String] repo Ignored
-        # @param [String] druid The id of the object
-        # @param [String] workflow_name The name of the workflow you want to create
-        # @param [String] wf_xml Ignored
-        # @param [Hash] opts optional params
-        # @option opts [String] :lane_id adds laneId attribute to all process elements in the wf_xml workflow xml.  Defaults to a value of 'default'
-        # @return [Boolean] always true
-        #
-        def create_workflow(_repo, druid, workflow_name, _wf_xml, 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.
         # Returns true on success.  Caller must handle any exceptions.
@@ -38,13 +20,8 @@ module Dor
         # @param [Integer] version specifies the version so that workflow service doesn't need to query dor-services.
         # @return [Boolean] always true
         #
-        def create_workflow_by_name(druid, workflow_name, version: nil, lane_id: 'default')
-          params = { 'lane-id' => lane_id }
-          if version
-            params['version'] = version
-          else
-            Deprecation.warn(self, 'Calling create_workflow_by_name without passing version is deprecated and will result in an error in dor-workflow-client 4.0')
-          end
+        def create_workflow_by_name(druid, workflow_name, version:, lane_id: 'default')
+          params = { 'lane-id' => lane_id, 'version' => version }
           requestor.request "objects/#{druid}/workflows/#{workflow_name}", 'post', '',
                             content_type: 'application/xml',
                             params: params
@@ -82,41 +59,6 @@ module Dor
           Workflow::Response::Update.new(json: response)
         end
 
-        # Updates the status of one step in a workflow.
-        # Returns true on success.  Caller must handle any exceptions
-        #
-        # @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
-        # @param [String] workflow The name of the workflow
-        # @param [String] process The name of the process step
-        # @param [String] status The status that you want to set -- using one of the values in VALID_STATUS
-        # @param [Hash] opts optional values for the workflow step
-        # @option opts [Float] :elapsed The number of seconds it took to complete this step. Can have a decimal.  Is set to 0 if not passed in.
-        # @option opts [String] :lifecycle Bookeeping label for this particular workflow step.  Examples are: 'registered', 'shelved'
-        # @option opts [String] :note Any kind of string annotation that you want to attach to the workflow
-        # @option opts [String] :lane_id Id of processing lane used by the job manager.  Can convey priority or name of an applicaiton specific processing lane (e.g. 'high', 'critical', 'hydrus')
-        # @option opts [String] :current_status Setting this string tells the workflow service to compare the current status to this value.  If the current value does not match this value, the update is not performed
-        # @return [Boolean] always true
-        # Http Call
-        # ==
-        # The method does an HTTP PUT to the base URL.  As an example:
-        #
-        #     PUT "/dor/objects/pid:123/workflows/GoogleScannedWF/convert"
-        #     <process name=\"convert\" status=\"completed\" />"
-        def update_workflow_status(repo, druid, workflow, process, status, opts = {})
-          raise ArgumentError, "Unknown status value #{status}" unless VALID_STATUS.include?(status.downcase)
-
-          opts = { elapsed: 0, lifecycle: nil, note: nil }.merge!(opts)
-          opts[:elapsed] = opts[:elapsed].to_s
-          current_status = opts.delete(:current_status)
-          xml = create_process_xml({ name: process, status: status.downcase }.merge!(opts))
-          uri = "#{repo}/objects/#{druid}/workflows/#{workflow}/#{process}"
-          uri += "?current-status=#{current_status.downcase}" if current_status
-          response = requestor.request(uri, 'put', xml, content_type: 'application/xml')
-          Workflow::Response::Update.new(json: response)
-        end
-        deprecation_deprecate update_workflow_status: 'use update_status instead.'
-
         #
         # Retrieves the process status of the given workflow for the given object identifier
         # @param [String] repo The repository the object resides in.  Currently recoginzes "dor" and "sdr".
@@ -124,20 +66,7 @@ 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/AbcSize
-        def workflow_status(*args)
-          case args.length
-          when 4
-            Deprecation.warn(self, 'Calling workflow_status with positional args is deprecated, use kwargs instead')
-            (_repo, druid, workflow, process) = *args
-          when 1
-            opts = args.first
-            repo = opts[:repo]
-            Deprecation.warn(self, 'Passing `:repo` to workflow_status is deprecated and can be omitted') if repo
-            druid = opts[:druid]
-            workflow = opts[:workflow]
-            process = opts[:process]
-          end
+        def workflow_status(druid:, workflow:, process:)
           workflow_md = fetch_workflow(druid: druid, workflow: workflow)
           doc = Nokogiri::XML(workflow_md)
           raise Dor::WorkflowException, "Unable to parse response:\n#{workflow_md}" if doc.root.nil?
@@ -147,57 +76,6 @@ module Dor
           process&.attr('status')
         end
 
-        # 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
-        def workflow_xml(*args)
-          case args.length
-          when 3
-            Deprecation.warn(self, 'Calling workflow_xml with positional args is deprecated, use kwargs instead')
-            (repo, druid, workflow) = *args
-          when 1
-            opts = args.first
-            repo = opts[:repo]
-            Deprecation.warn(self, 'Passing `:repo` to workflow_xml is deprecated and can be omitted') if repo
-            druid = opts[:druid]
-            workflow = opts[:workflow]
-          end
-
-          raise ArgumentError, 'missing workflow' unless workflow
-          return requestor.request "#{repo}/objects/#{druid}/workflows/#{workflow}" if repo
-
-          fetch_workflow(druid: druid, workflow: workflow)
-        end
-        deprecation_deprecate workflow_xml: 'workflow_xml will not be replaced'
-
-        # Updates the status of one step in a workflow to error.
-        # Returns true on success.  Caller must handle any exceptions
-        #
-        # @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
-        # @param [String] workflow The name of the workflow
-        # @param [String] process The name of the workflow step
-        # @param [String] error_msg The error message.  Ideally, this is a brief message describing the error
-        # @param [Hash] opts optional values for the workflow step
-        # @option opts [String] :error_text A slot to hold more information about the error, like a full stacktrace
-        # @return [Boolean] always true
-        #
-        # Http Call
-        # ==
-        # The method does an HTTP PUT to the base URL.
-        #
-        #     PUT "/objects/pid:123/workflows/GoogleScannedWF/convert"
-        #     <process name=\"convert\" status=\"error\" />"
-        def update_workflow_error_status(_repo, druid, workflow, process, error_msg, opts = {})
-          update_error_status(druid: druid, workflow: workflow, process: process, error_msg: error_msg, error_text: opts[:error_text])
-          true
-        end
-        deprecation_deprecate update_workflow_error_status: 'use update_error_status instead.'
-
         # Updates the status of one step in a workflow to error.
         # Returns true on success.  Caller must handle any exceptions
         #
@@ -240,23 +118,19 @@ module Dor
         # This method only works when this gem is used in a project that is configured to connect to DOR
         #
         # @param [String] pid of druid
-        # @param [String] repo repository for the object
         # @return [Array<String>] list of worklows
         # @example
         #   client.workflows('druid:sr100hp0609')
         #   => ["accessionWF", "assemblyWF", "disseminationWF"]
-        def workflows(pid, repo = nil)
-          Deprecation.warn(self, 'Passing the second argument (repo) to workflows is deprecated and can be omitted') if repo
+        def workflows(pid)
           xml_doc = Nokogiri::XML(fetch_workflow(druid: pid, workflow: ''))
           xml_doc.xpath('//workflow').collect { |workflow| workflow['id'] }
         end
 
-        # @param [String] repo repository of the object
         # @param [String] pid id of object
         # @param [String] workflow_name The name of the workflow
         # @return [Workflow::Response::Workflow]
-        def workflow(repo: nil, pid:, workflow_name:)
-          Deprecation.warn(self, 'passing the repo parameter is deprecated and will be removed in the next major versions') if repo
+        def workflow(pid:, workflow_name:)
           xml = fetch_workflow(druid: pid, workflow: workflow_name)
           Workflow::Response::Workflow.new(xml: xml)
         end
@@ -274,24 +148,8 @@ module Dor
         # @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(*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
+        def delete_workflow(druid:, workflow:, version:)
+          qs_args = "?version=#{version}"
           requestor.request "/objects/#{druid}/workflows/#{workflow}#{qs_args}", 'delete'
           true
         end
@@ -325,7 +183,6 @@ module Dor
           builder.to_xml
         end
       end
-      # rubocop:enable Metrics/ClassLength
     end
   end
 end

data/lib/dor/workflow/client.rb

@@ -40,8 +40,7 @@ module Dor
         @requestor = Requestor.new(connection: connection || ConnectionFactory.build_connection(url, timeout: timeout, logger: logger))
       end
 
-      delegate :create_workflow, :create_workflow_by_name, :update_workflow_status, :workflow_status,
-               :workflow_xml, :update_workflow_error_status, :all_workflows_xml, :workflows,
+      delegate :create_workflow_by_name, :workflow_status, :all_workflows_xml, :workflows,
                :workflow, :process, :delete_workflow, :delete_all_workflows, :update_status, :update_error_status,
                to: :workflow_routes
 

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

@@ -51,8 +51,7 @@ module Dor
           @attributes[:laneId].presence
         end
 
-        delegate :pid, :workflow_name, :repository, to: :parent
-        deprecation_deprecate :repository
+        delegate :pid, :workflow_name, to: :parent
 
         private
 

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

@@ -5,9 +5,6 @@ module Dor
     module Response
       # The response from asking the server about a workflow for an item
       class Workflow
-        extend Deprecation
-        self.deprecation_horizon = '4.x'
-
         def initialize(xml:)
           @xml = xml
         end
@@ -20,11 +17,6 @@ module Dor
           workflow['id']
         end
 
-        def repository
-          workflow['repository']
-        end
-        deprecation_deprecate :repository
-
         # @param [Integer] version the version we are checking for.
         def active_for?(version:)
           result = ng_xml.at_xpath("/workflow/process[@version=#{version}]")