scientist-softserv / adventist_knapsack

Apache License 2.0
2 stars 0 forks source link

🧹 Rename image_form_spec to reflect decoration #37

Closed jeremyf closed 12 months ago

jeremyf commented 12 months ago

Prior to this commit the Knapsace image_form_spec had a test for terms that was not part of the Hyku. We introduce these differences through our decoration in ./app/forms/hyrax/image_form_decorator.rb

To reflect the existence of the decorator and not have a duplicate file that future us would need to synchronize with Hyku, I've moved the file and winnowed it down to reflect the specific changes.

I also needed to add three new properties to the spec as it reflects upstream changes in Hyku (e.g. access_right alternative_title rights_notes)

Diff of filein Hyku and Knapsack ``` ❯ diff spec/forms/hyrax/image_form_spec.rb hyrax-webapp/spec/forms/hyrax/image_form_spec.rb 27,44d26 < describe '.terms' do < it 'returns an array of inherited and custom terms' do < expect(described_class.terms.sort).to eq( < %i[ < title creator contributor description keyword abstract < license rights_statement publisher date_created < subject language identifier based_near related_url < representative_id thumbnail_id rendering_ids files < visibility_during_embargo embargo_release_date visibility_after_embargo < visibility_during_lease lease_expiration_date visibility_after_lease < visibility ordered_member_ids source in_works_ids member_of_collection_ids < admin_set_id resource_type extent aark_id part_of place_of_publication < date_issued alt bibliographic_citation remote_url < ].sort < ) < end < end < ```

Related to: