Uses polymorphic_path([snap, :snapshots]) and self.class.model_name.h…

…uman

lib/seek/research_objects/acts_as_snapshottable.rb

@@ -29,7 +29,7 @@ def is_snapshottable?
       module InstanceMethods
         def create_snapshot
           if self.creators.empty?
-            errors.add(:base, "At least one creator is required. To add, go to Actions -> Manage #{self.class.name}.")
+            errors.add(:base, "At least one creator is required. To add, go to Actions -> Manage #{self.class.model_name.human}.")
             return nil
           end
           Rails.logger.debug("Creating snapshot for: #{self.class.name} #{id}")

test/functional/snapshots_controller_test.rb

@@ -137,11 +137,12 @@ class SnapshotsControllerTest < ActionController::TestCase
                               contributor: user.person, creators: [])
 
     login_as(user)
-    [study, investigation, assay].each do |snap|
+    [investigation, study, assay].each do |snap|
       assert snap.can_manage?(user)
       assert snap.creators.empty?
       type = snap.class.name.underscore
-      request.path = Seek::Util.routes.polymorphic_path(snap)+"/snapshots"
+      # Updating the request.path is needed so that @resource is correctly set, and the snapshots is created for the correct item in the loop
+      request.path = Seek::Util.routes.polymorphic_path([snap, :snapshots])
       # Get preview
       get :new, params: { "#{type}_id": snap.id }
       assert_response :success