call the cops

app/jobs/migrate_resources_job.rb

@@ -2,27 +2,33 @@
 
 # migrates models from AF to valkyrie
 class MigrateResourcesJob < ApplicationJob
-  attr_accessor :errors
+  attr_writer :errors
   # input [Array>>String] Array of ActiveFedora model names to migrate to valkyrie objects
   # defaults to AdminSet & Collection models if empty
   def perform(models: ['AdminSet', 'Collection'], ids: [])
-    errors = []
     if ids.blank?
       models.each do |model|
         model.constantize.find_each do |item|
-          resource = Hyrax.query_service.find_by(id: item.id)
-          result = MigrateResourceService.new(resource: resource).call
-          errors << result unless result.success?
+          migrate(item.id)
         end
       end
     else
       ids.each do |id|
-        resource = Hyrax.query_service.find_by(id: id)
-        next unless resource.wings? # this resource has already been converted
-        result = MigrateResourceService.new(resource: resource).call
-        errors << result unless result.success?
-      end
+        migrate(id)
+     end
     end
     raise errors.inspect if errors.present?
   end
+
+  def errors
+    @errors ||= []
+  end
+
+  def migrate(id)
+    resource = Hyrax.query_service.find_by(id: id)
+    next unless resource.wings? # this resource has already been converted
+    result = MigrateResourceService.new(resource: resource).call
+    errors << result unless result.success?
+    result
+  end
 end

app/models/hyrax/resource.rb

@@ -151,7 +151,7 @@ def work?
 
     # Its nice to know if a record is still in AF or not
     def wings?
-      self.respond_to?(:head) && self.respond_to?(:tail)
+      respond_to?(:head) && respond_to?(:tail)
     end
 
     def ==(other)

spec/jobs/migrate_resources_job_spec.rb

@@ -11,8 +11,8 @@
     clear_enqueued_jobs
   end
 
-  let(:account)        { create(:account_with_public_schema) }
-  let(:af_file_set)       { create(:file_set, title: ['TestFS']) }
+  let(:account) { create(:account_with_public_schema) }
+  let(:af_file_set) { create(:file_set, title: ['TestFS']) }
 
   let!(:af_admin_set) do
     as = AdminSet.new(title: ['AF Admin Set'])