dor-workflow-client 7.2.0 → 7.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 29157476de970bf333e7ca643d49b79ef4e03cf0bf9e18aab8277767492aab3f
-  data.tar.gz: c25c23a411be47575be4c3d91e912c4d5b48b698423d302d8263a7ca0f1170f7
+  metadata.gz: 7e57b379b07c024d6add2d3746c30f042ee10cecef9e5373b85cf697d5538a35
+  data.tar.gz: ae9ba93b8d724464772962044363428ba045716682b35011087b88af6dc5aa05
 SHA512:
-  metadata.gz: 32b74f669d2ec51717389c34e6ab113f1271ba05b84f3a9ac5a2e5e95da562ada8d2e7cb85ed78a8d4c80e14862e7658b15bf18b9933f0be4223933d80dcc0c2
-  data.tar.gz: af1f2bbbb802ebd84ccb239ea62e29b8bfdfb3a0947d0729a9e16caea8594b9bc224750085674b28c7448ff47f4fd8525830b8627ad5dab2f31f0c861d8dafcd
+  metadata.gz: 14d6da0f6d3075f8fa335bcc550419a9534974ab76cab28bd2adf39a75ba1eabf75231c207bca89fee8251ee3f7be5549c7378ee68579a7647b0fd261927aff8
+  data.tar.gz: 4331412a8b8b4d6a87f072b79835c377909d7636f7519e8937e91ae844e59ef4ec592aa68bb7205b870b9b98a4e3c33cd4c85cc5c8428b14368cce3a560d54b5

data/.gitignore

@@ -15,5 +15,6 @@ spec/reports
 test/tmp
 test/version_tmp
 tmp
+workflow_service.log*
 .ruby-version
 .ruby-gemset

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    dor-workflow-client (7.2.0)
+    dor-workflow-client (7.4.0)
       activesupport (>= 3.2.1, < 8)
       deprecation (>= 0.99.0)
       faraday (~> 2.0)
@@ -42,29 +42,29 @@ GEM
       faraday-net_http (>= 2.0, < 3.2)
     faraday-net_http (3.1.0)
       net-http
-    faraday-retry (2.2.0)
+    faraday-retry (2.2.1)
       faraday (~> 2.0)
     hashdiff (1.1.0)
     i18n (1.14.4)
       concurrent-ruby (~> 1.0)
-    json (2.7.1)
+    json (2.7.2)
     language_server-protocol (3.17.0.3)
-    mini_portile2 (2.8.5)
+    mini_portile2 (2.8.6)
     minitest (5.22.3)
     mutex_m (0.2.0)
     net-http (0.4.1)
       uri
-    nokogiri (1.16.3)
+    nokogiri (1.16.4)
       mini_portile2 (~> 2.8.2)
       racc (~> 1.4)
     parallel (1.24.0)
-    parser (3.3.0.5)
+    parser (3.3.1.0)
       ast (~> 2.4.1)
       racc
-    public_suffix (5.0.4)
+    public_suffix (5.0.5)
     racc (1.7.3)
     rainbow (3.1.1)
-    rake (13.1.0)
+    rake (13.2.1)
     regexp_parser (2.9.0)
     rexml (3.2.6)
     rspec (3.13.0)
@@ -82,7 +82,7 @@ GEM
     rspec-support (3.13.1)
     rspec_junit_formatter (0.6.0)
       rspec-core (>= 2, < 4, != 2.12.0)
-    rubocop (1.62.1)
+    rubocop (1.63.4)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)
@@ -101,12 +101,12 @@ GEM
       rubocop (~> 1.41)
     rubocop-rake (0.6.0)
       rubocop (~> 1.0)
-    rubocop-rspec (2.28.0)
+    rubocop-rspec (2.29.1)
       rubocop (~> 1.40)
       rubocop-capybara (~> 2.17)
       rubocop-factory_bot (~> 2.22)
       rubocop-rspec_rails (~> 2.28)
-    rubocop-rspec_rails (2.28.2)
+    rubocop-rspec_rails (2.28.3)
       rubocop (~> 1.40)
     ruby-progressbar (1.13.0)
     simplecov (0.22.0)

data/README.md

@@ -19,15 +19,47 @@ client = Dor::Workflow::Client.new(url: 'https://test-server.edu/workflow/')
 
 Consumers of recent versions of the [dor-services](https://github.com/sul-dlss/dor-services) gem can access the configured `Dor::Workflow::Client` object via `Dor::Config`.
 
+## Console
+
+During development, you can test the gem locally on your laptop, hitting a local instance of workflow-server-rails via the console:
+
+```ruby
+bin/console
+
+client = Dor::Workflow::Client.new(url: 'http://localhost:3000')
+client.create_workflow_by_name('druid:bc123df4567', 'accessionWF', version: '1', context: { 'requireOCR' => true})
+
+client.workflows('druid:bc123df4567')
+ => ["accessionWF"]
+
+client.workflow(pid: 'druid:bc123df4567', workflow_name: 'accessionWF')
+=> #<Dor::Workflow::Response::Workflow:0x0000000105c8b440
+
+client.process(pid: 'druid:bc123df4567', workflow_name: 'accessionWF', process: 'start-accession').context
+ => {"requireOCR"=>true}
+
+client.all_workflows(pid: 'druid:bc123df4567')
+=> #<Dor::Workflow::Response::Workflows:0x00000001055d29a0>.....
+```
+
 ## API
 [Rubydoc](https://www.rubydoc.info/github/sul-dlss/dor-workflow-client/main)
 
+### Workflow Variables
+
+If a workflow or workflows for a particular object require data to be persisted and available between steps, workflow variables can be set.  These are per object/version pair and thus available to any step in any workflow for a given version of an object once set.  Pass in a context variable as a Hash as shown in the example below.  The context will be returned as a hash when fetching workflows data for an object.
+
 ### Example usage
 Create a workflow
 ```
 client.create_workflow_by_name('druid:bc123df4567', 'etdSubmitWF', version: '1')
 ```
 
+Create a workflow and send in context
+```
+client.create_workflow_by_name('druid:bc123df4567', 'etdSubmitWF', version: '1', context: { foo: 'bar'} )
+```
+
 Update a workflow step's status
 ```ruby
 client.update_status(druid: 'druid:bc123df4567',
@@ -36,6 +68,24 @@ client.update_status(druid: 'druid:bc123df4567',
                      status: 'completed')
 ```
 
+Fetch information about a workflow:
+```ruby
+client.workflow(pid: 'druid:bc123df4567', workflow_name: 'etdSubmitWF')
+ => #<Dor::Workflow::Response::Workflow:0x000000010cb28588
+```
+
+Fetch information about a workflow step:
+```ruby
+client.process(pid: 'druid:bc123df4567', workflow_name: 'etdSubmitWF', process: 'registrar-approval')
+ => #<Dor::Workflow::Response::Process:0x000000010c505098
+```
+
+Fetch version context about a workflow step:
+```ruby
+client.process(pid: 'druid:bc123df4567', workflow_name: 'etdSubmitWF', process: 'registrar-approval').context
+ => {"foo"=>"bar"}
+```
+
 Show "milestones" for an object
 ```ruby
 client.milestones(druid: 'druid:gv054hp4128')

data/bin/console

@@ -0,0 +1,8 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+require 'bundler/setup'
+require 'dor/workflow/client'
+
+require 'irb'
+IRB.start(__FILE__)

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

@@ -11,7 +11,6 @@ module Dor
 
         # Returns the Date for a requested milestone from workflow lifecycle
         #
-        # @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 (nil) the version to query for

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

@@ -3,7 +3,7 @@
 module Dor
   module Workflow
     class Client
-      VERSION = '7.2.0'
+      VERSION = '7.4.0'
     end
   end
 end

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

@@ -14,7 +14,6 @@ module Dor
         # - completes the versioningWF:submit-version and versioningWF:start-accession steps
         # - initiates accesssionWF
         #
-        # @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
         def close_version(druid:, version:, create_accession_wf: true)

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

@@ -17,13 +17,15 @@ module Dor
         # @param [String] workflow_name The name of the workflow you want to create. This must correspond with a workflow
         # name that is known by the workflow service.
         # @param [String] lane_id adds laneId attribute to all process elements in the wf_xml workflow xml.  Defaults to a value of 'default'
+        # @param [Hash] context optional context to be included in the workflow (same for all processes for a given druid/version pair)
         # @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:, lane_id: 'default')
+        def create_workflow_by_name(druid, workflow_name, version:, lane_id: 'default', context: nil)
           params = { 'lane-id' => lane_id, 'version' => version }
-          requestor.request "objects/#{druid}/workflows/#{workflow_name}", 'post', '',
-                            content_type: 'application/xml',
+          body = context ? { 'context' => context }.to_json : ''
+          requestor.request "objects/#{druid}/workflows/#{workflow_name}", 'post', body,
+                            content_type: 'application/json',
                             params: params
           true
         end
@@ -31,7 +33,6 @@ module Dor
         # 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
@@ -61,7 +62,6 @@ module Dor
 
         #
         # 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".
         # @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

data/lib/dor/workflow/client.rb

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

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

@@ -51,6 +51,13 @@ module Dor
           @attributes[:laneId].presence
         end
 
+        # @return [Hash] the context for the process (or empty hash if none present)
+        def context
+          return {} unless @attributes[:context].present?
+
+          JSON.parse(@attributes[:context])
+        end
+
         delegate :pid, :workflow_name, to: :parent
 
         private

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

@@ -28,8 +28,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 ? node.attributes.to_h { |k, v| [k.to_sym, v.value] } : {}
-          Process.new(parent: self, **attributes)
+          to_process(node)
         end
 
         def empty?
@@ -39,13 +38,24 @@ module Dor
         # Check if all processes are skipped or complete for the provided version.
         # @param [Integer] version the version we are checking for.
         def complete_for?(version:)
-          ng_xml.xpath("/workflow/process[@version=#{version}]/@status").map(&:value).all? { |p| %w[skipped completed].include?(p) }
+          # ng_xml.xpath("/workflow/process[@version=#{version}]/@status").map(&:value).all? { |p| %w[skipped completed].include?(p) }
+          incomplete_processes_for(version: version).empty?
         end
 
         def complete?
           complete_for?(version: version)
         end
 
+        def incomplete_processes_for(version:)
+          process_nodes = ng_xml.xpath("/workflow/process[@version=#{version}]")
+          incomplete_process_nodes = process_nodes.reject { |process_node| %w[skipped completed].include?(process_node.attr('status')) }
+          incomplete_process_nodes.map { |process_node| to_process(process_node) }
+        end
+
+        def incomplete_processes
+          incomplete_processes_for(version: version)
+        end
+
         attr_reader :xml
 
         private
@@ -66,6 +76,11 @@ module Dor
         def ng_xml
           @ng_xml ||= Nokogiri::XML(@xml)
         end
+
+        def to_process(node)
+          attributes = node ? node.attributes.to_h { |k, v| [k.to_sym, v.value] } : {}
+          Process.new(parent: self, **attributes)
+        end
       end
     end
   end

data/spec/dor/workflow/client/workflow_routes_spec.rb

@@ -169,8 +169,48 @@ RSpec.describe Dor::Workflow::Client::WorkflowRoutes do
   end
 
   describe '#create_workflow_by_name' do
-    it 'need to write these specs', skip: 'need to write specs' do
-      # something
+    let(:mock_requestor) { instance_double(Dor::Workflow::Client::Requestor, request: nil) }
+
+    context 'with default lane_id and no context' do
+      subject(:create_workflow_by_name) do
+        routes.create_workflow_by_name('druid:mw971zk1113', 'accessionWF', version: '1')
+      end
+
+      it 'sends a create request with default lane_id and without any context param' do
+        create_workflow_by_name
+        expect(mock_requestor).to have_received(:request)
+          .with('objects/druid:mw971zk1113/workflows/accessionWF', 'post', '',
+                { content_type: 'application/json',
+                  params: { 'lane-id' => 'default', 'version' => '1' } })
+      end
+    end
+
+    context 'with a custom lane_id' do
+      subject(:create_workflow_by_name) do
+        routes.create_workflow_by_name('druid:mw971zk1113', 'accessionWF', version: '1', lane_id: 'hamburgers')
+      end
+
+      it 'sends a create request without any context param' do
+        create_workflow_by_name
+        expect(mock_requestor).to have_received(:request)
+          .with('objects/druid:mw971zk1113/workflows/accessionWF', 'post', '',
+                { content_type: 'application/json',
+                  params: { 'lane-id' => 'hamburgers', 'version' => '1' } })
+      end
+    end
+
+    context 'when context is sent' do
+      subject(:create_workflow_by_name) do
+        routes.create_workflow_by_name('druid:mw971zk1113', 'accessionWF', version: '1', lane_id: 'default', context: { foo: 'bar' })
+      end
+
+      it 'sends a create request with the context in the body' do
+        create_workflow_by_name
+        expect(mock_requestor).to have_received(:request)
+          .with('objects/druid:mw971zk1113/workflows/accessionWF', 'post', '{"context":{"foo":"bar"}}',
+                { content_type: 'application/json',
+                  params: { 'lane-id' => 'default', 'version' => '1' } })
+      end
     end
   end
 end

data/spec/dor/workflow/client_spec.rb

@@ -225,6 +225,29 @@ RSpec.describe Dor::Workflow::Client do
     end
   end
 
+  describe '#all_workflows' do
+    let(:xml) do
+      <<~XML
+        <workflows objectId="druid:123">
+          <workflow repository="dor" objectId="druid:123" id="accessionWF">
+            <process laneId="default" lifecycle="submitted" elapsed="0.0" attempts="1" datetime="2013-02-18T15:08:10-0800" status="completed" name="start-accession"/>
+          </workflow>
+        </workflows>
+      XML
+    end
+    let(:stubs) do
+      Faraday::Adapter::Test::Stubs.new do |stub|
+        stub.get("/objects/#{@druid}/workflows") do |_env|
+          [200, {}, xml]
+        end
+      end
+    end
+
+    it 'returns the workflow details associated with druid' do
+      expect(client.all_workflows(pid: @druid).xml).to eq(xml)
+    end
+  end
+
   describe '#lifecycle' do
     let(:stubs) do
       Faraday::Adapter::Test::Stubs.new do |stub|

data/spec/dor/workflow/response/process_spec.rb

@@ -62,4 +62,32 @@ RSpec.describe Dor::Workflow::Response::Process do
 
     it { is_expected.to eq 'default' }
   end
+
+  describe '#context' do
+    subject { instance.context }
+
+    context 'when context exists' do
+      let(:xml) do
+        <<~XML
+          <workflow repository="dor" objectId="druid:mw971zk1113" id="assemblyWF">
+            <process name="start-assembly" laneId="default" context="{&quot;requireOCR&quot;:true}">
+          </workflow>
+        XML
+      end
+
+      it { is_expected.to eq({ 'requireOCR' => true }) }
+    end
+
+    context 'when no context exists' do
+      let(:xml) do
+        <<~XML
+          <workflow repository="dor" objectId="druid:mw971zk1113" id="assemblyWF">
+            <process name="start-assembly" laneId="default" context="">
+          </workflow>
+        XML
+      end
+
+      it { is_expected.to eq({}) }
+    end
+  end
 end

data/spec/dor/workflow/response/workflow_spec.rb

@@ -198,4 +198,62 @@ RSpec.describe Dor::Workflow::Response::Workflow do
       end
     end
   end
+
+  describe '#incomplete_processes' do
+    subject(:processes) { instance.incomplete_processes }
+
+    context 'when all steps are complete' do
+      let(:xml) do
+        <<~XML
+          <workflow repository="dor" objectId="druid:mw971zk1113" id="assemblyWF">
+            <process version="1" laneId="default" elapsed="0.0" attempts="1" datetime="2013-02-18T14:40:25-0800" status="completed" name="start-assembly"/>
+            <process version="1" laneId="default" elapsed="0.509" attempts="1" datetime="2013-02-18T14:42:24-0800" status="completed" name="jp2-create"/>
+          </workflow>
+        XML
+      end
+
+      it { is_expected.to be_empty }
+    end
+
+    context 'when some steps are not complete' do
+      let(:xml) do
+        <<~XML
+          <workflow repository="dor" objectId="druid:mw971zk1113" id="assemblyWF">
+            <process version="1" laneId="default" elapsed="0.0" attempts="1" datetime="2013-02-18T14:40:25-0800" status="completed" name="start-assembly"/>
+            <process version="1" laneId="default" elapsed="0.509" attempts="1" datetime="2013-02-18T14:42:24-0800" status="waiting" name="jp2-create"/>
+          </workflow>
+        XML
+      end
+
+      it 'returns the incomplete processes' do
+        expect(processes.size).to eq 1
+        expect(processes.first.name).to eq 'jp2-create'
+      end
+    end
+  end
+
+  describe '#incomplete_processes_for' do
+    let(:xml) do
+      <<~XML
+        <workflow repository="dor" objectId="druid:mw971zk1113" id="assemblyWF">
+          <process version="1" laneId="default" elapsed="0.0" attempts="1" datetime="2013-02-18T14:40:25-0800" status="completed" name="start-assembly"/>
+          <process version="1" laneId="default" elapsed="0.509" attempts="1" datetime="2013-02-18T14:42:24-0800" status="waiting" name="jp2-create"/>
+          <process version="2" laneId="default" elapsed="0.0" attempts="1" datetime="2013-02-18T14:40:25-0800" status="completed" name="start-assembly"/>
+          <process version="2" laneId="default" elapsed="0.509" attempts="1" datetime="2013-02-18T14:42:24-0800" status="completed" name="jp2-create"/>
+        </workflow>
+      XML
+    end
+
+    context 'when all steps are complete' do
+      it 'returns empty' do
+        expect(instance.incomplete_processes_for(version: 2)).to be_empty
+      end
+    end
+
+    context 'when some steps are not complete' do
+      it 'returns false' do
+        expect(instance.incomplete_processes_for(version: 1).size).to eq 1
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dor-workflow-client
 version: !ruby/object:Gem::Version
-  version: 7.2.0
+  version: 7.4.0
 platform: ruby
 authors:
 - Willy Mene
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-04-02 00:00:00.000000000 Z
+date: 2024-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -120,6 +120,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- bin/console
 - dor-workflow-client.gemspec
 - lib/dor/missing_workflow_exception.rb
 - lib/dor/workflow/client.rb
@@ -168,7 +169,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
+rubygems_version: 3.5.9
 signing_key:
 specification_version: 4
 summary: Provides convenience methods to work with the DOR Workflow Service