hyrax 3.3.0 → 3.4.2

data/hyrax.gemspec

@@ -26,7 +26,7 @@ SUMMARY
 
   spec.required_ruby_version = '>= 2.5'
 
-  # Note: rails does not follow sem-ver conventions, it's
+  # NOTE: rails does not follow sem-ver conventions, it's
   # minor version releases can include breaking changes; see
   # http://guides.rubyonrails.org/maintenance_policy.html
   spec.add_dependency 'rails', '~> 5.0'
@@ -58,6 +58,7 @@ SUMMARY
   spec.add_dependency 'iiif_manifest', '>= 0.3', '< 2.0'
   spec.add_dependency 'jquery-datatables-rails', '~> 3.4'
   spec.add_dependency 'jquery-ui-rails', '~> 6.0'
+  spec.add_dependency 'json-ld', '< 3.2'
   spec.add_dependency 'json-schema' # for Arkivo
   # Pin more tightly because 0.x gems are potentially unstable
   spec.add_dependency 'kaminari_route_prefix', '~> 0.1.1'
@@ -70,6 +71,7 @@ SUMMARY
   spec.add_dependency 'oauth2', '~> 1.2'
   spec.add_dependency 'posix-spawn'
   spec.add_dependency 'power_converter', '~> 0.1', '>= 0.1.2'
+  spec.add_dependency 'psych', '~> 3.3'
   spec.add_dependency 'qa', '~> 5.5', '>= 5.5.1' # questioning_authority
   spec.add_dependency 'rails_autolink', '~> 1.1'
   spec.add_dependency 'rdf-rdfxml' # controlled vocabulary importer
@@ -82,7 +84,7 @@ SUMMARY
   spec.add_dependency 'samvera-nesting_indexer', '~> 2.0'
   spec.add_dependency 'select2-rails', '~> 3.5'
   spec.add_dependency 'signet'
-  spec.add_dependency 'tinymce-rails'
+  spec.add_dependency 'tinymce-rails', '~> 5.10'
   spec.add_dependency 'valkyrie', '~> 2', '>= 2.1.1'
 
   spec.add_development_dependency "capybara", '~> 3.29'
@@ -106,7 +108,7 @@ SUMMARY
   spec.add_development_dependency 'i18n_yaml_sorter'
   spec.add_development_dependency 'rails-controller-testing', '~> 1'
   # the hyrax style guide is based on `bixby`. see `.rubocop.yml`
-  spec.add_development_dependency 'bixby', '~> 3.0', ">= 3.0.2"
+  spec.add_development_dependency 'bixby', '~> 5.0', '>= 5.0.2' # bixby 5 briefly dropped Ruby 2.5
   spec.add_development_dependency 'shoulda-callback-matchers', '~> 1.1.1'
   spec.add_development_dependency 'shoulda-matchers', '~> 3.1'
   spec.add_development_dependency 'webdrivers', '~> 4.4'

data/lib/generators/hyrax/collection_resource/USAGE

@@ -0,0 +1,20 @@
+Description:
+  This generator creates the necessary files for a Hyrax::PcdmCollection.
+
+Example:
+  Usage: rails generate hyrax:collection_resource MyCollection [with_basic_metadata]
+
+  Argument:
+    with_basic_metadata (optional) - if present, the collection resource will include basic metadata
+
+  This will create:
+    app/models/my_collection.rb
+    app/forms/my_collection_form.rb
+    app/indexers/my_collection_indexer.rb
+    config/metadata/my_collection.yaml
+    spec/models/my_collection_spec.rb
+    spec/forms/my_collection_form_spec.rb
+    spec/indexers/my_collection_indexer_spec.rb
+
+  This will also:
+    set Hyrax.config.collection_model = "MyCollection"

data/lib/generators/hyrax/collection_resource/collection_resource_generator.rb

@@ -0,0 +1,133 @@
+# frozen_string_literal: true
+require 'rails/generators'
+require 'rails/generators/model_helpers'
+
+class Hyrax::CollectionResourceGenerator < Rails::Generators::NamedBase # rubocop:disable Metrics/ClassLength
+  # ActiveSupport can interpret models as plural which causes
+  # counter-intuitive route paths. Pull in ModelHelpers from
+  # Rails which warns users about pluralization when generating
+  # new models or scaffolds.
+  include Rails::Generators::ModelHelpers
+
+  source_root File.expand_path('../templates', __FILE__)
+
+  argument :with_basic_metadata, type: :string, default: "", banner: 'with_basic_metadata'
+
+  desc 'This generator makes the following changes to your application:
+   1. Creates a collection model and model spec, optionally including basic metadata.
+   2. Creates a collection form and spec.
+   3. Creates a collection indexer and spec.
+   4. Creates a collection metadata config.
+   5. Sets this to be the collection class.
+'
+
+  def self.exit_on_failure?
+    true
+  end
+
+  def validate_name
+    return unless name.strip.casecmp("collection").zero?
+    raise Thor::MalformattedArgumentError,
+          set_color("Error: A collection resource with the name '#{name}' would cause name-space clashes. "\
+                    "Please use a different name.", :red)
+  end
+
+  def banner
+    if revoking?
+      say_status("info", "DESTROYING VALKYRIE COLLECTION MODEL: #{class_name}", :blue)
+    else
+      say_status("info", "GENERATING VALKYRIE COLLECTION MODEL: #{class_name}", :blue)
+    end
+  end
+
+  def create_metadata_config
+    template('collection_metadata.yaml', File.join('config/metadata/', "#{file_name}.yaml"))
+  end
+
+  def create_model
+    filepath = File.join('app/models', "#{file_name}.rb")
+    template('collection.rb.erb', filepath)
+    return unless include_basic_metadata?
+    inject_into_file filepath, before: /include Hyrax::Schema/ do
+      "include Hyrax::Schema(:basic_metadata)\n  "
+    end
+  end
+
+  def create_model_spec
+    return unless rspec_installed?
+    filepath = File.join('spec/models/', "#{file_name}_spec.rb")
+    template('collection_spec.rb.erb', filepath)
+
+    return unless include_basic_metadata?
+    inject_into_file filepath, after: /it_behaves_like 'a Hyrax::PcdmCollection'/ do
+      "\n  it_behaves_like 'a model with basic metadata'"
+    end
+  end
+
+  def create_form
+    filepath = File.join('app/forms/', "#{file_name}_form.rb")
+    template('collection_form.rb.erb', filepath)
+    return unless include_basic_metadata?
+    inject_into_file filepath, before: /include Hyrax::FormFields/ do
+      "include Hyrax::FormFields(:basic_metadata)\n  "
+    end
+  end
+
+  # @todo If shared specs are expanded to test for basic metadata, inject calling that test here.
+  def create_form_spec
+    return unless rspec_installed?
+    template('collection_form_spec.rb.erb', File.join('spec/forms/', "#{file_name}_form_spec.rb"))
+  end
+
+  def create_indexer
+    filepath = File.join('app/indexers/', "#{file_name}_indexer.rb")
+    template('collection_indexer.rb.erb', filepath)
+    return unless include_basic_metadata?
+    inject_into_file filepath, before: /include Hyrax::Indexer/ do
+      "include Hyrax::Indexer(:basic_metadata)\n  "
+    end
+  end
+
+  def create_indexer_spec
+    return unless rspec_installed?
+    filepath = File.join('spec/indexers/', "#{file_name}_indexer_spec.rb")
+    template('collection_indexer_spec.rb.erb', filepath)
+
+    return unless include_basic_metadata?
+    inject_into_file filepath, after: /it_behaves_like 'a Hyrax::Resource indexer'/ do
+      "\n  it_behaves_like 'a Basic metadata indexer'"
+    end
+  end
+
+  # Inserts after the last registered work, or at the top of the config block
+  def set_as_the_collection_class
+    config = 'config/initializers/hyrax.rb'
+    lastmatch = nil
+    in_root do
+      File.open(config).each_line do |line|
+        lastmatch = line if line.match?(/config.collection_model = /)
+      end
+      content = "  # Injected via `rails g hyrax:collection_resource #{class_name}`\n" \
+                "  config.collection_model = '#{class_name}'\n"
+
+      anchor = lastmatch || "Hyrax.config do |config|\n"
+      inject_into_file config, after: anchor do
+        content
+      end
+    end
+  end
+
+  private
+
+  def include_basic_metadata?
+    with_basic_metadata.present? && with_basic_metadata == "with_basic_metadata"
+  end
+
+  def rspec_installed?
+    defined?(RSpec) && defined?(RSpec::Rails)
+  end
+
+  def revoking?
+    behavior == :revoke
+  end
+end

data/lib/generators/hyrax/collection_resource/templates/collection.rb.erb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+class <%= class_name %> < Hyrax::PcdmCollection
+  # @note Do not directly update `basic_metadata.yaml`.  It is also used by works.
+  #
+  # To change metadata for collections
+  # * extend by adding fields to `/config/metadata/<%= file_name %>.yaml`
+  # * remove all basic metadata
+  #   * if you generated `with_basic_metadata` and now don't want any basic metadata,
+  #       comment out or delete the schema include statement for `:basic_metadata`
+  #   * update form and indexer classes to also remove the `:basic_metadata` schema include
+  # * remove some basic metadata
+  #   * comment out or delete the schema include statement for `:basic_metadata`
+  #   * update form and indexer classes to also remove the `:basic_metadata` schema include
+  #   * copy fields you want to keep from `/config/metadata/basic_metadata.yaml`
+  #       to `/config/metadata/<%= file_name %>.yaml`
+  # * override some basic metadata
+  #   * fields are not allowed to repeat, so to override any basic metadata, you
+  #       must first comment out or delete the schema include statement for `:basic_metadata`
+  #   * update form and indexer classes to also remove the `:basic_metadata` schema include
+  #   * copy all fields you want to keep unchanged or to override from `/config/metadata/basic_metadata.yaml`
+  #       to `/config/metadata/<%= file_name %>.yaml`
+  #   * update them in `config/metadata/<%= file_name %>.yaml to have the desired
+  #       characteristics
+  #
+  # Alternative:
+  # * comment out or delete schema include statements
+  # * add Valkyrie attributes to this class
+  # * update form and indexer to process the attributes
+  #
+  include Hyrax::Schema(:<%= file_name %>)
+end

data/lib/generators/hyrax/collection_resource/templates/collection_form.rb.erb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+class <%= class_name %>Form < Hyrax::Forms::PcdmCollectionForm
+  include Hyrax::FormFields(:<%= file_name %>)
+end

data/lib/generators/hyrax/collection_resource/templates/collection_form_spec.rb.erb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+require 'rails_helper'
+require 'valkyrie/specs/shared_specs'
+
+RSpec.describe <%= class_name %>Form do
+  let(:change_set) { described_class.new(resource) }
+  let(:resource)   { <%= class_name %>.new }
+
+  it_behaves_like 'a Valkyrie::ChangeSet'
+end

data/lib/generators/hyrax/collection_resource/templates/collection_indexer.rb.erb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+class <%= class_name %>Indexer < Hyrax::PcdmCollectionIndexer
+  include Hyrax::Indexer(:<%= file_name %>)
+end

data/lib/generators/hyrax/collection_resource/templates/collection_indexer_spec.rb.erb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+require 'rails_helper'
+require 'hyrax/specs/shared_specs/indexers'
+
+RSpec.describe <%= class_name %>Indexer do
+  let(:indexer_class) { described_class }
+  let(:resource)      { <%= class_name %>.new }
+
+  it_behaves_like 'a Hyrax::Resource indexer'
+end

data/lib/generators/hyrax/collection_resource/templates/collection_metadata.yaml

@@ -0,0 +1,22 @@
+# Simple yaml config-driven schema which is used to define model attributes,
+# index key names, and form properties.
+#
+# Attributes must have a type but all other configuration options are optional.
+#
+# attributes:
+#   attribute_name:
+#     type: string
+#     multiple: false
+#     index_keys:
+#       - "attribute_name_sim"
+#     form:
+#       required: true
+#       primary: true
+#       multiple: false
+#
+# @see config/metadata/basic_metadata.yaml for an example configuration
+#
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+
+attributes: {}

data/lib/generators/hyrax/collection_resource/templates/collection_spec.rb.erb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+require 'rails_helper'
+require 'hyrax/specs/shared_specs/hydra_works'
+
+RSpec.describe <%= class_name %> do
+  subject(:collection) { described_class.new }
+
+  it_behaves_like 'a Hyrax::PcdmCollection'
+end

data/lib/generators/hyrax/install_generator.rb

@@ -35,6 +35,15 @@ module Hyrax
   18. Generates RIIIF image server implementation
          """
 
+    # browse-everything 1.2.0 is not compatible with Ruby 2.5
+    def pin_browse_everything
+      return unless Gem::Version.new(RUBY_VERSION) < Gem::Version.new('2.6')
+      gem 'browse-everything', '~> 1.1.2'
+      Bundler.with_clean_env do
+        run "bundle update browse-everything"
+      end
+    end
+
     def run_required_generators
       say_status('info', '[Hyrax] GENERATING BLACKLIGHT', :blue)
       generate 'blacklight:install --devise'

data/lib/hyrax/administrative_set_name.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module Hyrax
+  ##
+  # A custom name for Valkyrie AdministrativeSet objects. Route keys are mapped to `admin_set`
+  # not be the same as the model name.
+  class AdministrativeSetName < Name
+    def initialize(klass, namespace = nil, name = nil)
+      super
+      @human              = 'AdminSet'
+      @i18n_key           = :admin_set
+      @param_key          = 'admin_set'
+      @plural             = 'admin_sets'
+      @route_key          = 'admin_sets'
+      @singular_route_key = 'admin_set'
+    end
+  end
+end

data/lib/hyrax/collection_name.rb

@@ -14,6 +14,8 @@ module Hyrax
       @plural             = 'collections'
       @route_key          = 'collections'
       @singular_route_key = 'collection'
+      @collection         = 'collections'
+      @element            = 'collection'
     end
   end
 end

data/lib/hyrax/configuration.rb

@@ -427,6 +427,18 @@ module Hyrax
       @branding_storage_adapter = Valkyrie::StorageAdapter.find(adapter.to_sym)
     end
 
+    ##
+    # @return [Valkyrie::StorageAdapter]
+    def derivatives_storage_adapter
+      @derivatives_storage_adapter ||= Valkyrie::StorageAdapter.find(:derivatives_disk)
+    end
+
+    ##
+    # @param [#to_sym] adapter
+    def derivatives_storage_adapter=(adapter)
+      @derivatives_storage_adapter = Valkyrie::StorageAdapter.find(adapter.to_sym)
+    end
+
     ##
     # @return [#save, #save_all, #delete, #wipe!] an indexing adapter
     def index_adapter
@@ -449,6 +461,7 @@ module Hyrax
     ##
     # @return [Boolean] whether to use experimental valkyrie storage features
     def use_valkyrie?
+      return true if disable_wings # always return true if wings is disabled
       ActiveModel::Type::Boolean.new.cast(ENV.fetch('HYRAX_VALKYRIE', false))
     end
     # @!endgroup
@@ -495,6 +508,15 @@ module Hyrax
         Hyrax::Characterization::ValkyrieCharacterizationService
     end
 
+    ##
+    # @!attribute [w] characterization_proxy
+    #   Which FileSet file to use for mime type resolution
+    #   @ see Hyrax::FileSetTypeService
+    attr_writer :characterization_proxy
+    def characterization_proxy
+      @characterization_proxy ||= :original_file
+    end
+
     # Attributes for the lock manager which ensures a single process/thread is mutating a ore:Aggregation at once.
     # @!attribute [w] lock_retry_count
     #   How many times to retry to acquire the lock before raising UnableToAcquireLockError

data/lib/hyrax/controlled_vocabularies/location.rb

@@ -4,10 +4,17 @@ module Hyrax
     class Location < ActiveTriples::Resource
       configure rdf_label: ::RDF::Vocab::GEONAMES.name
 
+      include ResourceLabelCaching
+
       # Return a tuple of url & label
       def solrize
-        return [rdf_subject.to_s] if rdf_label.first.to_s.blank? || rdf_label.first.to_s == rdf_subject.to_s
-        [rdf_subject.to_s, { label: "#{rdf_label.first}$#{rdf_subject}" }]
+        label = full_label || rdf_label.first.to_s
+        return [rdf_subject.to_s] if label.blank? || label == rdf_subject.to_s
+        [rdf_subject.to_s, { label: "#{label}$#{rdf_subject}" }]
+      end
+
+      def full_label
+        Hyrax::LocationService.new.full_label(rdf_subject.to_s)
       end
     end
   end

data/lib/hyrax/controlled_vocabularies/resource_label_caching.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+module Hyrax
+  module ControlledVocabularies
+    ##
+    # adds caching to the {#rdf_label} method.
+    #
+    # for systems that check the {#rdf_label}and use {#fetch} to get upstream
+    # data if it is not present, this can be used to avoid making round trips
+    # to an authoritative web source.
+    #
+    # @see Hyrax::DeepIndexingService
+    module ResourceLabelCaching
+      CACHE_KEY_PREFIX = "hy_label-v1-"
+
+      ##
+      # @note uses the Rails cache to avoid repeated lookups.
+      # @see ActiveTriples::Resource#rdf_label
+      def rdf_label
+        # only cache if this rdf source is represented by a URI;
+        # i.e. don't cache for blank nodes
+        return super unless uri?
+
+        Rails.cache.fetch(cache_key) { super }
+      end
+
+      ##
+      # @note adds behavior to clear the cache whenever a manual fetch of data
+      #   is performed.
+      # @see ActiveTriples::Resource#fetch
+      def fetch(*)
+        Rails.cache.delete(cache_key)
+        super
+      end
+
+      private
+
+      def cache_key
+        "#{CACHE_KEY_PREFIX}#{to_uri.canonicalize.pname}"
+      end
+    end
+  end
+end

data/lib/hyrax/controlled_vocabularies.rb

@@ -5,6 +5,7 @@ module Hyrax
 
     eager_autoload do
       autoload :Location
+      autoload :ResourceLabelCaching
     end
   end
 end

data/lib/hyrax/publisher.rb

@@ -94,18 +94,49 @@ module Hyrax
     # @macro a_registered_event
     register_event('batch.created')
 
+    # @since 3.4.0
+    # @macro a_registered_event
+    register_event('collection.deleted')
+
     # @since 3.0.0
     # @macro a_registered_event
+    #   @note this event SHOULD be published whevener the metadata is saved
+    #     for a PCDM Collection. the payload for each published event MUST
+    #     include an `:collection` (the updated Collection) AND a `:user` (the
+    #     {::User} responsible for the update). the event SHOULD NOT be
+    #     published for changes that only impact membership properties
+    #     (`#member_of_ids`, `#member_of_collection_ids`, and `#member_ids`)
     register_event('collection.metadata.updated')
 
     # @since 3.0.0
     # @macro a_registered_event
+    #   @note this event SHOULD be published whevener the membership is changed
+    #     for a PCDM Collection. this includes changes to the Collection's
+    #     `#member_ids` attribute, as well as inverse membership changes via
+    #     another Collection/Object's `#member_of_ids` or
+    #     `#member_of_collection_ids` attribute. the event payload MUST include
+    #     either a `:collection` OR a `:collection_id` (the Collection OR its
+    #     unique id), AND a `:user` (the ::User responsible for the update).
     register_event('collection.membership.updated')
 
+    # @since 3.5.0
+    # @macro a_registered_event
+    register_event('file.characterized')
+
     # @since 3.3.0
     # @macro a_registered_event
     register_event('file.downloaded')
 
+    # @since 3.4.0
+    # @macro a_registered_event
+    #   @note this event SHOULD be published whenever a {Hyrax::FileMetadata}
+    #     object is updated; i.e. whenever the metadata directly associated with
+    #     a file changes. the event payload MUST include a `:metadata` (the
+    #     {Hyrax::FileMetadata} that has been updated) AND a `:user`. note that
+    #     the `:user` responsible for {FileMetadata} changes may frequently be
+    #     a system user.
+    register_event('file.metadata.updated')
+
     # @since 3.0.0
     # @macro a_registered_event
     register_event('file.set.audited')
@@ -138,8 +169,26 @@ module Hyrax
     # @macro a_registered_event
     register_event('object.acl.updated')
 
+    # @since 3.4.0
+    # @macro a_registered_event
+    #   @note this event SHOULD be published whevener the membership is changed
+    #     for a PCDM Object (including a Hydra Works FileSet). this includes
+    #     changes to the Object's `#member_ids` attribute, as well as inverse
+    #     membership changes via another Object's `#member_of_ids` attribute.
+    #     the event payload MUST include either an `:object` OR an `:object_id`
+    #     (the Object OR its unique id), AND a `:user` (the ::User responsible
+    #     for the update).
+    register_event('object.membership.updated')
+
     # @since 3.0.0
     # @macro a_registered_event
+    #   @note this event SHOULD be published whevener the metadata is saved
+    #     for a PCDM Object (including a Hydra Works FileSet). the payload for
+    #     each published event MUST include an `:object` (the updated Object),
+    #     AND a `:user` (the ::User responsible for the update). the event
+    #     SHOULD NOT be published for changes that only impact membership
+    #     properties (`#member_of_ids`, `#member_of_collection_ids`, and
+    #     `#member_ids`)
     register_event('object.metadata.updated')
 
     # @since 3.2.0

data/lib/hyrax/schema.rb

@@ -4,24 +4,27 @@ module Hyrax
   ##
   # @api public
   #
-  # Builds a schema module for a Valkyrie resource. The schema itself is
-  # resolved by schema loader, which must respond to
-  # `#attributes_for(schema: :name)` with a hash from attribute names to
-  # `Dry::Type` types.
+  # Builds a schema module for a +Valkyrie::Resource+. The schema itself is
+  # resolved by a schema loader instance, which must implement
+  # {SimpleSchemaLoader#attributes_for}, with a hash from attribute names to
+  # +Dry::Type+ types.
   #
   # For the default schema loader, configuration is loaded from
-  # `config/metadata/[name]}.yaml`. Custom schema loaders can be provided
-  # for other types.
+  # +config/metadata/{name}.yaml+. A custom schema loader can be provided as
+  # +:schema_loader+ to
+  # resolve the schema in other ways.
   #
-  # @note `Valkyrie::Resources`/`Hyrax::Resources` are not required to use this
-  # interface, and may define custom attributes using the base
-  # `Valkyrie::Resource.attribute` interface. This mechanism is provided to
-  # allow schemas to be defined in a unified way that don't require programmer
-  # intervention ("configurable schemas"). While the default usage defines
-  # schemas in application configuration, they could also be held in repository
-  # storage, an external schema service, etc... by using a custom schema loader.
+  # @note +Valkyrie::Resource+ and {Hyrax::Resource} classes are not required to
+  #   use this interface, and may define custom attributes using the base
+  #   +Valkyrie::Resource.attribute+ interface. This mechanism is provided to
+  #   allow schemas to be defined in a unified way that don't require programmer
+  #   intervention ("configurable schemas"). While the default schema loader derives
+  #   schemas from configuration files, alternate implementations could provide
+  #   schema definitions pulled from repository storage, an external schema service,
+  #   etc...
   #
   # @param [Symbol] schema_name
+  # @param [#attributes_for] schema_loader
   #
   # @return [Module] a module that, when included, applies a schema to a
   #   `Valkyire::Resource`

data/lib/hyrax/specs/capybara.rb

@@ -21,7 +21,7 @@ if ENV['IN_DOCKER'].present? || ENV['HUB_URL'].present?
   args = %w[disable-gpu no-sandbox whitelisted-ips window-size=1400,1400]
   args.push('headless') if ActiveModel::Type::Boolean.new.cast(ENV['CHROME_HEADLESS_MODE'])
 
-  capabilities = Selenium::WebDriver::Remote::Capabilities.chrome(chromeOptions: { args: args })
+  capabilities = Selenium::WebDriver::Remote::Capabilities.chrome("goog:chromeOptions" => { args: args })
 
   Capybara.register_driver :selenium_chrome_headless_sandboxless do |app|
     driver = Capybara::Selenium::Driver.new(app,

data/lib/hyrax/specs/shared_specs/hydra_works.rb

@@ -135,7 +135,6 @@ RSpec.shared_examples 'a Hyrax::PcdmCollection' do
 
   it_behaves_like 'a Hyrax::Resource'
   it_behaves_like 'a model with core metadata'
-  it_behaves_like 'a model with basic metadata'
   it_behaves_like 'has members'
 
   describe '#collection_type_gid' do
@@ -172,10 +171,12 @@ RSpec.shared_examples 'a Hyrax::AdministrativeSet' do
       .to contain_exactly('lorem ipsum')
   end
 
-  it 'has an #title' do
-    expect { admin_set.title = ['Moomin'] }
-      .to change { admin_set.title }
-      .to contain_exactly('Moomin')
+  describe '#collection_type_gid' do
+    let(:gid) { Hyrax::CollectionType.find_or_create_admin_set_type.to_global_id }
+
+    it 'has a GlobalID for a collection type' do
+      expect(admin_set.collection_type_gid).to eq gid
+    end
   end
 end
 
@@ -256,6 +257,7 @@ RSpec.shared_examples 'a Hyrax::FileSet' do
 
   it_behaves_like 'a Hyrax::Resource'
   it_behaves_like 'a model with core metadata'
+  it_behaves_like 'a model with basic metadata'
 
   it { is_expected.not_to be_collection }
   it { is_expected.not_to be_file }
@@ -287,6 +289,10 @@ RSpec.shared_examples 'a Hyrax::FileSet' do
         expect(fileset.file_ids).to eq file_ids
       end
 
+      it 'has a representative_id' do
+        expect(fileset.representative_id).to eq fileset.id
+      end
+
       it 'can query files' do
         expect(query_service.custom_queries.find_files(file_set: fileset)).to eq files
       end