TSS-35: Rename UserTask to StaffTask (#78)

- Renamed UserTask class to StaffTask
- Renamed UserTaskCreationService to StaffTaskCreationService
- Updated all references in documentation and example code

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Loren Yu <loren@navapbc.com>

Author: devin-ai-integration[bot]

app/models/flex/business_process.rb

@@ -6,9 +6,9 @@ module Flex
   #
   # @example Defining a basic business process in app/business_processes/my_business_process.rb
   #   MyBusinessProcess = Flex::BusinessProcess.define(:my_process, MyCase) do |bp|
-  #     # Define steps - can be UserTask or SystemProcess
+  #     # Define steps - can be StaffTask or SystemProcess
   #     bp.step('collect_info',
-  #       Flex::UserTask.new("Collect Information", TaskCreationService))
+  #       Flex::StaffTask.new("Collect Information", TaskCreationService))
   #
   #     bp.step('process_data',
   #       Flex::SystemProcess.new("Process Data", ->(kase) {
@@ -24,7 +24,7 @@ module Flex
   #   end
   #
   # Steps can be either:
-  # - UserTask: Tasks that require human interaction, created through a TaskCreationService
+  # - StaffTask: Tasks that require human interaction, created through a TaskCreationService
   # - SystemProcess: Automated tasks that run without user intervention, defined with a callable
   #
   # The process automatically listens for events and transitions between steps
@@ -33,7 +33,7 @@ module Flex
   #
   # Event payloads must contain either case_id or application_form_id to identify the case.
   #
-  # @see Flex::UserTask
+  # @see Flex::StaffTask
   # @see Flex::SystemProcess
   #
   # Key Methods:

app/models/flex/user_task.rb app/models/flex/staff_task.rbapp/models/flex/user_task.rb renamed to app/models/flex/staff_task.rb

@@ -1,5 +1,5 @@
 module Flex
-  class UserTask
+  class StaffTask
     include Step
 
     attr_accessor :name, :task_management_service

spec/dummy/app/business_processes/passport_business_process.rb

@@ -1,15 +1,15 @@
 PassportBusinessProcess = Flex::BusinessProcess.define(:passport, PassportCase) do |bp|
   # Define steps
   bp.step('collect_application_info',
-    Flex::UserTask.new("Collect App Info", UserTaskCreationService))
+    Flex::StaffTask.new("Collect App Info", StaffTaskCreationService))
 
   bp.step('verify_identity',
     Flex::SystemProcess.new("Verify Identity", ->(kase) {
       IdentityVerificationService.new(kase).verify_identity
     }))
 
   bp.step('manual_adjudicator_review',
-    Flex::UserTask.new("Manual Adjudicator Review", AdjudicatorTaskCreationService))
+    Flex::StaffTask.new("Manual Adjudicator Review", AdjudicatorTaskCreationService))
 
   bp.step('review_passport_photo',
     Flex::SystemProcess.new("Review Passport Photo", ->(kase) {

spec/dummy/app/business_processes/test_business_process.rb

@@ -1,27 +1,27 @@
 TestBusinessProcess = Flex::BusinessProcess.define(:test, TestCase) do |bp|
   # Define steps
-  bp.step('user_task',
-    Flex::UserTask.new("user_task", UserTaskCreationService))
+  bp.step('staff_task',
+    Flex::StaffTask.new("staff_task", StaffTaskCreationService))
 
   bp.step('system_process',
     Flex::SystemProcess.new("system_process", ->(kase) {
       Flex::EventManager.publish("event2", { case_id: kase.id })
     }))
 
-  bp.step('user_task_2',
-    Flex::UserTask.new("user_task_2", UserTaskCreationService))
+  bp.step('staff_task_2',
+    Flex::StaffTask.new("staff_task_2", StaffTaskCreationService))
 
   bp.step('system_process_2',
     Flex::SystemProcess.new("system_process_2", ->(kase) {
       Flex::EventManager.publish("event4", { case_id: kase.id })
     }))
 
   # Define start step
-  bp.start('user_task')
+  bp.start('staff_task')
 
   # Define transitions
-  bp.transition('user_task', 'event1', 'system_process')
-  bp.transition('system_process', 'event2', 'user_task_2')
-  bp.transition('user_task_2', 'event3', 'system_process_2')
+  bp.transition('staff_task', 'event1', 'system_process')
+  bp.transition('system_process', 'event2', 'staff_task_2')
+  bp.transition('staff_task_2', 'event3', 'system_process_2')
   bp.transition('system_process_2', 'event4', 'end')
 end

…p/services/user_task_creation_service.rb …/services/staff_task_creation_service.rbspec/dummy/app/services/user_task_creation_service.rb renamed to spec/dummy/app/services/staff_task_creation_service.rb spec/dummy/app/services/user_task_creation_service.rb renamed to spec/dummy/app/services/staff_task_creation_service.rb

@@ -1,4 +1,4 @@
-class UserTaskCreationService
+class StaffTaskCreationService
   include Flex::TaskHandlerService
 
   def self.create_task(kase:)

spec/models/flex/business_process_spec.rb

@@ -16,15 +16,15 @@
 
   describe '#handle_event' do
     before do
-      kase.business_process_current_step = 'user_task'
+      kase.business_process_current_step = 'staff_task'
       kase.save!
     end
 
     it 'executes the complete process chain' do
       Flex::EventManager.publish('event1', { case_id: kase.id })
       # system_process automatically publishes event2
       kase.reload
-      expect(kase.business_process_current_step).to eq('user_task_2')
+      expect(kase.business_process_current_step).to eq('staff_task_2')
 
       Flex::EventManager.publish('event3', { case_id: kase.id })
       # system_process_2 automatically publishes event4
@@ -38,15 +38,15 @@
         [ 'event2', 'event3', 'event4' ].each do |event|
           Flex::EventManager.publish(event, { case_id: kase.id })
         end
-        expect(kase.business_process_current_step).to eq('user_task')
+        expect(kase.business_process_current_step).to eq('staff_task')
       end
 
       it 'does not re-execute the current step' do
-        allow(UserTaskCreationService).to receive(:create_task)
+        allow(StaffTaskCreationService).to receive(:create_task)
         [ 'event2', 'event3', 'event4' ].each do |event|
           Flex::EventManager.publish(event, { case_id: kase.id })
         end
-        expect(UserTaskCreationService).not_to have_received(:create_task)
+        expect(StaffTaskCreationService).not_to have_received(:create_task)
       end
     end
   end
@@ -61,20 +61,20 @@
       business_process.stop_listening_for_events
 
       # Try publishing various events
-      kase.business_process_current_step = 'user_task'
+      kase.business_process_current_step = 'staff_task'
       kase.save!
 
       Flex::EventManager.publish('event1', { case_id: kase.id })
       kase.reload
-      expect(kase.business_process_current_step).to eq('user_task') # Should not change
+      expect(kase.business_process_current_step).to eq('staff_task') # Should not change
 
       Flex::EventManager.publish('event2', { case_id: kase.id })
       kase.reload
-      expect(kase.business_process_current_step).to eq('user_task') # Should not change
+      expect(kase.business_process_current_step).to eq('staff_task') # Should not change
 
       Flex::EventManager.publish('event3', { case_id: kase.id })
       kase.reload
-      expect(kase.business_process_current_step).to eq('user_task') # Should not change
+      expect(kase.business_process_current_step).to eq('staff_task') # Should not change
     end
   end
 end