AdministrativeSet#description should be single-valued

related to #5385.
samvera · Jan 31, 2022 · 8d84eaa · 8d84eaa
1 parent f01616d
commit 8d84eaa
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/app/models/hyrax/administrative_set.rb b/app/models/hyrax/administrative_set.rb
@@ -10,7 +10,7 @@ class AdministrativeSet < Hyrax::Resource
 
     attribute :alternative_title, Valkyrie::Types::Set.of(Valkyrie::Types::String)
     attribute :creator,           Valkyrie::Types::Set.of(Valkyrie::Types::String)
-    attribute :description,       Valkyrie::Types::Set.of(Valkyrie::Types::String)
+    attribute :description,       Valkyrie::Types::String
 
     ##
     # @return [Boolean] true

diff --git a/spec/forms/hyrax/forms/administrative_set_form_spec.rb b/spec/forms/hyrax/forms/administrative_set_form_spec.rb
@@ -17,6 +17,22 @@
     end
   end
 
+  describe '#description' do
+    it 'is a single value' do
+      form.description = 'moomin'
+
+      expect { form.sync }
+        .to change { admin_set.description }
+        .to eq 'moomin'
+    end
+
+    it 'is a single value on repopulate' do
+      admin_set.description = 'moomin'
+
+      expect(form).to have_attributes(description: 'moomin')
+    end
+  end
+
   describe '#member_ids' do
     it 'populates as empty' do
       expect { form.prepopulate! }