active-fedora 13.2.7 → 14.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b2f9eb8fbfa34d943a8aed89f588a58e7a10c6d4fc077c9d0afb4600f38dc771
-  data.tar.gz: e1342febdbba15e85b46a032b05336377fc06233c89c94bc5a48d5c9b2a3f33a
+  metadata.gz: c5ea2bfd359fe8ac2103cc2f22d4bfacb564a5a6fe0964d6586120dbba785035
+  data.tar.gz: 64b28f342a53d4b2176ae4fbcdf502a7acd6e2f533bb68b71d2664e8fd95cf35
 SHA512:
-  metadata.gz: 4fcd0f270caea3bae2a2d4e6c00e1ba6c12d6d61523c16942b4c674a78f8f28490d29e4b62e666e2babfaec44be18a9b581b4e5a4191ed9c95a748bf39d91bf0
-  data.tar.gz: 4375e0c3877dad17e06ec3481ea0801e5e6ab5815d1f752e0a86efb0291334573af3ce068941f71ca48c9a18140d093fedce1b6b12b965085ea9c5d488f4ed1f
+  metadata.gz: 0dd1090c7c74aa2926172c858c32bc223120ffb29b75fa1b099f8a2f881e3cf4ca772c520b54241b6f72f5997aecb5149ce4a218339e8ce0d00ad4f3c42beec9
+  data.tar.gz: bb97c2be3c9b48fc185a1ce528234e1b7d96a497fa957810fd7c4546f087d4b6a4df7c6626e1340f74d2e5bd856a777d759ce98ddbb2a6c59e666c2bb31d398e

data/.circleci/config.yml

@@ -1,6 +1,6 @@
 version: 2.1
 orbs:
-  samvera: samvera/circleci-orb@0
+  samvera: samvera/circleci-orb@1.0
 jobs:
   bundle_lint_test:
     parameters:
@@ -8,7 +8,7 @@ jobs:
         type: string
       bundler_version:
         type: string
-        default: 2.0.1
+        default: 2.3.11
       rails_version:
         type: string
       solr_config_path:
@@ -31,7 +31,7 @@ jobs:
                   echo "$(git branch --all --list master */master)"
               fi
               [[ -z "$(git branch --all --list master */master)" ]]
-              
+
       - samvera/bundle_for_gem:
           ruby_version: << parameters.ruby_version >>
           bundler_version: << parameters.bundler_version >>
@@ -47,31 +47,143 @@ jobs:
 workflows:
   ci:
     jobs:
+      # Ruby 3.2 releases
+      - bundle_lint_test:
+          name: ruby3-2_rails7-0
+          ruby_version: 3.2.0
+          rails_version: 7.0.4.1
+      - bundle_lint_test:
+          name: ruby3-2_rails6-1
+          ruby_version: 3.2.0
+          rails_version: 6.1.7.1
+      - bundle_lint_test:
+          name: ruby3-2_rails6-0
+          ruby_version: 3.2.0
+          rails_version: 6.0.6.1
+      # Ruby 3.1 releases
+      - bundle_lint_test:
+          name: ruby3-1_rails7-0
+          ruby_version: 3.1.3
+          rails_version: 7.0.4.1
+      - bundle_lint_test:
+          name: ruby3-1_rails6-1
+          ruby_version: 3.1.3
+          rails_version: 6.1.7.1
+      - bundle_lint_test:
+          name: ruby3-1_rails6-0
+          ruby_version: 3.1.3
+          rails_version: 6.0.6.1
+      # Ruby 3.0 releases
+      - bundle_lint_test:
+          name: ruby3-0_rails7-0
+          ruby_version: 3.0.5
+          rails_version: 7.0.4.1
+      - bundle_lint_test:
+          name: ruby3-0_rails6-1
+          ruby_version: 3.0.5
+          rails_version: 6.1.7.1
+      - bundle_lint_test:
+          name: ruby3-0_rails6-0
+          ruby_version: 3.0.5
+          rails_version: 6.0.6.1
+      # Ruby 2.7 releases
+      - bundle_lint_test:
+          name: ruby2-7_rails7-0
+          ruby_version: 2.7.7
+          rails_version: 7.0.4.1
+      - bundle_lint_test:
+          name: ruby2-7_rails6-1
+          ruby_version: 2.7.7
+          rails_version: 6.1.7.1
+      - bundle_lint_test:
+          name: ruby2-7_rails6-0
+          ruby_version: 2.7.7
+          rails_version: 6.0.6.1
       - bundle_lint_test:
           name: ruby2-7_rails5-2
-          ruby_version: 2.7.0
-          rails_version: 5.2.4
+          ruby_version: 2.7.7
+          rails_version: 5.2.8.1
+      # Ruby 2.6 releases
+      - bundle_lint_test:
+          name: ruby2-6_rails6-0
+          ruby_version: 2.6.10
+          rails_version: 6.0.6.1
       - bundle_lint_test:
           name: ruby2-6_rails5-2
-          ruby_version: 2.6.5
-          rails_version: 5.2.4
+          ruby_version: 2.6.10
+          rails_version: 5.2.8.1
+
+  nightly:
+    triggers:
+      - schedule:
+          cron: "0 0 * * *"
+          filters:
+            branches:
+              only:
+                - main
+    jobs:
+      # Ruby 3.2 releases
+      - bundle_lint_test:
+          name: ruby3-2_rails7-0
+          ruby_version: 3.2.0
+          rails_version: 7.0.4.1
+      - bundle_lint_test:
+          name: ruby3-2_rails6-1
+          ruby_version: 3.2.0
+          rails_version: 6.1.7.1
+      - bundle_lint_test:
+          name: ruby3-2_rails6-0
+          ruby_version: 3.2.0
+          rails_version: 6.0.6.1
+      # Ruby 3.1 releases
+      - bundle_lint_test:
+          name: ruby3-1_rails7-0
+          ruby_version: 3.1.3
+          rails_version: 7.0.4.1
+      - bundle_lint_test:
+          name: ruby3-1_rails6-1
+          ruby_version: 3.1.3
+          rails_version: 6.1.7.1
       - bundle_lint_test:
-          name: ruby2-5_rails5-2
-          ruby_version: 2.5.7
-          rails_version: 5.2.4
+          name: ruby3-1_rails6-0
+          ruby_version: 3.1.3
+          rails_version: 6.0.6.1
+      # Ruby 3.0 releases
       - bundle_lint_test:
-          name: ruby2-4_rails5-2
-          ruby_version: 2.4.9
-          rails_version: 5.2.4
+          name: ruby3-0_rails7-0
+          ruby_version: 3.0.5
+          rails_version: 7.0.4.1
+      - bundle_lint_test:
+          name: ruby3-0_rails6-1
+          ruby_version: 3.0.5
+          rails_version: 6.1.7.1
+      - bundle_lint_test:
+          name: ruby3-0_rails6-0
+          ruby_version: 3.0.5
+          rails_version: 6.0.6.1
+      # Ruby 2.7 releases
+      - bundle_lint_test:
+          name: ruby2-7_rails7-0
+          ruby_version: 2.7.7
+          rails_version: 7.0.4.1
+      - bundle_lint_test:
+          name: ruby2-7_rails6-1
+          ruby_version: 2.7.7
+          rails_version: 6.1.7.1
       - bundle_lint_test:
           name: ruby2-7_rails6-0
-          ruby_version: 2.7.0
-          rails_version: 6.0.2
+          ruby_version: 2.7.7
+          rails_version: 6.0.6.1
+      - bundle_lint_test:
+          name: ruby2-7_rails5-2
+          ruby_version: 2.7.7
+          rails_version: 5.2.8.1
+      # Ruby 2.6 releases
       - bundle_lint_test:
           name: ruby2-6_rails6-0
-          ruby_version: 2.6.5
-          rails_version: 6.0.2
+          ruby_version: 2.6.10
+          rails_version: 6.0.6.1
       - bundle_lint_test:
-          name: ruby2-5_rails6.0
-          ruby_version: 2.5.7
-          rails_version: 6.0.2
+          name: ruby2-6_rails5-2
+          ruby_version: 2.6.10
+          rails_version: 5.2.8.1

data/.rubocop.yml

@@ -1,18 +1,13 @@
-require: rubocop-rspec
+inherit_gem:
+  bixby: bixby_default.yml
 
 inherit_from:
 - .rubocop_todo.yml
 
-AllCops:
-  TargetRubyVersion: 2.2
-  DisplayCopNames: true
-  Include:
-    - '**/Rakefile'
-  Exclude:
-    - 'script/**/*'
-    - 'vendor/**/*'
+Style/FrozenStringLiteralComment:
+  Enabled: false
 
-Lint/HandleExceptions:
+Lint/SuppressedException:
   Exclude:
     - 'spec/unit/**/*'
     - 'spec/integration/**/*'
@@ -22,12 +17,19 @@ Lint/HandleExceptions:
 Lint/AssignmentInCondition:
   Enabled: false
 
-Metrics/LineLength:
+Layout/LineLength:
   Enabled: false
 
 Metrics/AbcSize:
   Enabled: false
 
+Metrics/BlockLength:
+  Exclude:
+    - 'lib/active_fedora.rb'
+    - 'lib/active_fedora/nested_attributes.rb'
+    - 'lib/active_fedora/rspec_matchers/**/*'
+    - 'lib/tasks/**/*'
+
 Metrics/BlockNesting:
   Exclude:
     - 'lib/active_fedora/autosave_association.rb'
@@ -173,7 +175,7 @@ Style/Lambda:
     - 'spec/**/*'
 
 Layout/IndentationConsistency:
-  EnforcedStyle: rails
+  EnforcedStyle: indented_internal_methods
 
 Style/CollectionMethods:
   PreferredMethods:
@@ -240,6 +242,15 @@ Rails/Date:
 Rails/TimeZone:
   Enabled: false
 
+Rails/FilePath:
+  Enabled: false
+
+Rails/ActiveRecordAliases:
+  Enabled: false
+
+Rails/SkipsModelValidations:
+  Enabled: false
+
 RSpec/AnyInstance:
   Enabled: false
 
@@ -271,3 +282,6 @@ RSpec/NotToNot:
 
 RSpec/MessageSpies:
   Enabled: false
+
+Security/MarshalLoad:
+  Enabled: false

data/.rubocop_todo.yml

@@ -16,5 +16,5 @@ Style/PercentLiteralDelimiters:
 # Offense count: 1
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
 # URISchemes: http, https
-Metrics/LineLength:
+Layout/LineLength:
   Max: 82

data/CONTRIBUTING.md

@@ -44,6 +44,10 @@ prevalence of racist language in discussions of predatory
 publishing](https://www.ncbi.nlm.nih.gov/pmc/articles/PMC6148600/) for
 further details.
 
+If you're working on PR for this project, create a feature branch off of `main`.
+
+Please ***do not*** create a branch called `master` for this repository or as part of your pull request; the branch will either need to be removed or renamed before it can be considered for inclusion in the code base and history of our repositories.
+
 ## Contribution Tasks
 
 * Reporting Issues
@@ -67,10 +71,10 @@ further details.
 
 * Fork the repository on GitHub
 * Create a topic branch from where you want to base your work.
-  * This is usually the master branch.
-  * To quickly create a topic branch based on master; `git branch fix/master/my_contribution master`
-  * Then checkout the new branch with `git checkout fix/master/my_contribution`.
-  * Please avoid working directly on the `master` branch.
+  * This is usually the main branch.
+  * To quickly create a topic branch based on main; `git branch fix/main/my_contribution main`
+  * Then checkout the new branch with `git checkout fix/main/my_contribution`.
+  * Please avoid working directly on the `main` branch.
   * You may find the [hub suite of commands](https://github.com/defunkt/hub) helpful
 * Make sure you have added sufficient tests and documentation for your changes.
   * Test functionality with RSpec; Test features / UI with Capybara.
@@ -131,15 +135,15 @@ further details.
 ### Submitting Changes
 
 * Read the article ["Using Pull Requests"](https://help.github.com/articles/using-pull-requests) on GitHub.
-* Make sure your branch is up to date with its parent branch (i.e. master)
-  * `git checkout master`
+* Make sure your branch is up to date with its parent branch (i.e. main)
+  * `git checkout main`
   * `git pull --rebase`
   * `git checkout <your-branch>`
-  * `git rebase master`
+  * `git rebase main`
   * It is a good idea to run your tests again.
 * If you've made more than one commit take a moment to consider whether squashing commits together would help improve their logical grouping.
   * [Detailed Walkthrough of One Pull Request per Commit](http://ndlib.github.io/practices/one-commit-per-pull-request/)
-  * `git rebase --interactive master` ([See Github help](https://help.github.com/articles/interactive-rebase))
+  * `git rebase --interactive main` ([See Github help](https://help.github.com/articles/interactive-rebase))
   * Squashing your branch's changes into one commit is "good form" and helps the person merging your request to see everything that is going on.
 * Push your changes to a topic branch in your fork of the repository.
 * Submit a pull request from your fork to the project.

data/Gemfile

@@ -7,12 +7,18 @@ gemspec path: File.expand_path('..', __FILE__)
 gem 'byebug' unless ENV['TRAVIS']
 gem 'pry-byebug' unless ENV['CI']
 
-gem 'activemodel', ENV['RAILS_VERSION'] if ENV['RAILS_VERSION']
+if ENV['RAILS_VERSION']
+  gem 'activemodel', ENV['RAILS_VERSION']
+  gem 'rails', ENV['RAILS_VERSION']
+else
+  gem 'activemodel', '>= 6.0', '< 8'
+  gem 'rails', '>= 6.0', '< 8'
+end
 
 group :test do
-  gem 'simplecov', require: false
   gem 'coveralls', require: false
   gem 'rspec_junit_formatter'
+  gem 'simplecov', require: false
 end
 
 gem 'jruby-openssl', platform: :jruby

data/README.md

@@ -1,13 +1,14 @@
 # ActiveFedora
 
-Code: [![Version](https://badge.fury.io/rb/active-fedora.png)](http://badge.fury.io/rb/active-fedora)
+Code: [![Samvera Core Component](https://img.shields.io/badge/samvera-core--component-brightgreen)](https://github.com/samvera/maintenance#samvera-core-components)
 [![Build Status](https://circleci.com/gh/samvera/active_fedora.svg?style=svg)](https://circleci.com/gh/samvera/active_fedora)
-[![Coverage Status](https://coveralls.io/repos/github/samvera/active_fedora/badge.svg?branch=master)](https://coveralls.io/github/samvera/active_fedora?branch=master)
+[![Coverage Status](https://coveralls.io/repos/github/samvera/active_fedora/badge.svg?branch=main)](https://coveralls.io/github/samvera/active_fedora?branch=main)
+[![Version](https://img.shields.io/gem/v/active-fedora)](https://rubygems.org/gems/active-fedora)
 
 Docs: [![Contribution Guidelines](http://img.shields.io/badge/CONTRIBUTING-Guidelines-blue.svg)](./CONTRIBUTING.md)
 [![Apache 2.0 License](http://img.shields.io/badge/APACHE2-license-blue.svg)](./LICENSE)
 
-Jump in: [![Slack Status](http://slack.samvera.org/badge.svg)](http://slack.samvera.org/)
+Community Support: [![Samvera Community Slack](https://img.shields.io/badge/samvera-slack-blueviolet)](http://slack.samvera.org/)
 
 # What is ActiveFedora?
 
@@ -101,6 +102,12 @@ is installed, run:
 rake spec
 ```
 
+## Contributing
+
+If you're working on PR for this project, create a feature branch off of `main`.
+
+This repository follows the [Samvera Community Code of Conduct](https://samvera.atlassian.net/wiki/spaces/samvera/pages/405212316/Code+of+Conduct) and [language recommendations](https://github.com/samvera/maintenance/blob/master/templates/CONTRIBUTING.md#language).  Please ***do not*** create a branch called `master` for this repository or as part of your pull request; the branch will either need to be removed or renamed before it can be considered for inclusion in the code base and history of this repository.
+
 # Release Process
 
 The [release process](https://github.com/samvera/active_fedora/wiki/Release-management-process) is documented on the wiki.

data/active-fedora.gemspec

@@ -1,5 +1,6 @@
 # -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "active_fedora/version"
 
 Gem::Specification.new do |s|
@@ -8,39 +9,38 @@ Gem::Specification.new do |s|
   s.platform    = Gem::Platform::RUBY
   s.authors     = ["Matt Zumwalt", "McClain Looney", "Justin Coyne"]
   s.email       = ["samvera-tech@googlegroups.com"]
-  s.homepage    = %q{https://github.com/samvera/active_fedora}
-  s.summary     = %q{A convenience libary for manipulating documents in the Fedora Repository.}
-  s.description = %q{ActiveFedora provides for creating and managing objects in the Fedora Repository Architecture.}
+  s.homepage    = 'https://github.com/samvera/active_fedora'
+  s.summary     = 'A convenience libary for manipulating documents in the Fedora Repository.'
+  s.description = 'ActiveFedora provides for creating and managing objects in the Fedora Repository Architecture.'
   s.license = "Apache-2.0"
-  s.required_ruby_version = '~> 2.4'
+  s.metadata = { "rubygems_mfa_required" => "true" }
+  s.required_ruby_version = '>= 2.6'
 
   s.add_dependency "activemodel", '>= 5.1'
   s.add_dependency "activesupport", '>= 5.1'
   s.add_dependency "active-triples", '>= 0.11.0', '< 2.0.0'
   s.add_dependency "deprecation"
-  s.add_dependency 'faraday', '~> 0.12'
+  s.add_dependency 'faraday', '~> 1.0'
   s.add_dependency 'faraday-encoding', '>= 0.0.5'
   s.add_dependency "ldp", '>= 0.7.0', '< 2'
   s.add_dependency 'rsolr', '>= 1.1.2', '< 3'
   s.add_dependency "ruby-progressbar", '~> 1.0'
 
+  s.add_development_dependency "bixby"
   s.add_development_dependency "equivalent-xml"
   s.add_development_dependency 'fcrepo_wrapper', '~> 0.2'
   s.add_development_dependency "github_changelog_generator"
   s.add_development_dependency "rdoc"
-  s.add_development_dependency "psych", "< 4" # Restricted because 4.0+ do not work with rubocop 0.56.x
   s.add_development_dependency "rails"
   s.add_development_dependency "rake"
   s.add_development_dependency "rspec", "~> 3.5"
   s.add_development_dependency "rspec-its"
-  s.add_development_dependency "rubocop", '~> 0.56.0'
-  s.add_development_dependency "rubocop-rspec", '~> 1.12.0'
   s.add_development_dependency "simplecov", '~> 0.8'
-  s.add_development_dependency "solr_wrapper", "~> 2.0"
+  s.add_development_dependency "solr_wrapper", "~> 4.0"
   s.add_development_dependency "yard"
 
   s.files         = `git ls-files`.split($OUTPUT_RECORD_SEPARATOR).select { |f| File.dirname(f) !~ %r{\A"?spec\/?} }
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
   s.extra_rdoc_files = [
     "LICENSE",
     "README.md"

data/lib/active_fedora/association_hash.rb

@@ -11,11 +11,11 @@ module ActiveFedora
     end
 
     def [](name)
-      association(name).reader if association(name)
+      association(name)&.reader
     end
 
     def []=(name, object)
-      association(name).writer(object) if association(name)
+      association(name)&.writer(object)
     end
 
     def association(name)
@@ -84,6 +84,7 @@ module ActiveFedora
       super
     end
   end
+
   ##
   # Represents the result of merging two association hashes.
   # @note As the keys can come from multiple models, the attributes become
@@ -91,11 +92,13 @@ module ActiveFedora
   class Merged < AssociationHash
     attr_reader :first, :second
 
+    # rubocop:disable Lint/MissingSuper
     def initialize(first, second)
       @first = first
       @base = first.base
       @second = second
     end
+    # rubocop:enable Lint/MissingSuper
 
     def [](name)
       first[name] || second[name]

data/lib/active_fedora/associations/association.rb

@@ -119,7 +119,7 @@ module ActiveFedora
         reset
       end
 
-      def initialize_attributes(record, except_from_scope_attributes = nil) #:nodoc:
+      def initialize_attributes(record, except_from_scope_attributes = nil) # :nodoc:
         except_from_scope_attributes ||= {}
         skip_assign = [reflection.foreign_key].compact
         assigned_keys = record.changed
@@ -141,9 +141,7 @@ module ActiveFedora
           if (reflection.has_one? || reflection.collection?) && !options[:through]
             attributes[reflection.foreign_key] = owner[reflection.active_record_primary_key]
 
-            if reflection.options[:as]
-              attributes[reflection.type] = owner.class.base_class.name
-            end
+            attributes[reflection.type] = owner.class.base_class.name if reflection.options[:as]
           end
 
           attributes

data/lib/active_fedora/associations/association_scope.rb

@@ -1,6 +1,6 @@
 module ActiveFedora
   module Associations
-    class AssociationScope #:nodoc:
+    class AssociationScope # :nodoc:
       def self.scope(association)
         new(association).scope
       end

data/lib/active_fedora/associations/basic_contains_association.rb

@@ -1,8 +1,8 @@
 module ActiveFedora
   module Associations
-    class BasicContainsAssociation < ContainsAssociation #:nodoc:
+    class BasicContainsAssociation < ContainsAssociation # :nodoc:
       def find_target
-        uris = owner.resource.query(predicate: options[:predicate])
+        uris = owner.resource.query({ predicate: options[:predicate] })
                     .map { |r| r.object.to_s }
 
         uris.map { |object_uri| klass.find(klass.uri_to_id(object_uri)) }

data/lib/active_fedora/associations/belongs_to_association.rb

@@ -1,6 +1,6 @@
 module ActiveFedora
   module Associations
-    class BelongsToAssociation < SingularAssociation #:nodoc:
+    class BelongsToAssociation < SingularAssociation # :nodoc:
       def handle_dependency
         target.send(options[:dependent]) if load_target
       end

data/lib/active_fedora/associations/builder/association.rb

@@ -1,5 +1,5 @@
 module ActiveFedora::Associations::Builder
-  class Association #:nodoc:
+  class Association # :nodoc:
     class << self
       attr_accessor :extensions
     end
@@ -36,7 +36,7 @@ module ActiveFedora::Associations::Builder
     def self.build_scope(scope, extension)
       new_scope = scope
 
-      new_scope = proc { instance_exec(&scope) } if scope && scope.arity.zero?
+      new_scope = proc { instance_exec(&scope) } if scope&.arity&.zero?
 
       new_scope = wrap_scope new_scope, extension if extension
 
@@ -114,9 +114,7 @@ module ActiveFedora::Associations::Builder
     end
 
     def self.check_dependent_options(dependent)
-      unless valid_dependent_options.include? dependent
-        raise ArgumentError, "The :dependent option must be one of #{valid_dependent_options}, but is :#{dependent}"
-      end
+      raise ArgumentError, "The :dependent option must be one of #{valid_dependent_options}, but is :#{dependent}" unless valid_dependent_options.include? dependent
     end
 
     def self.add_destroy_callbacks(model, reflection)

data/lib/active_fedora/associations/builder/basic_contains.rb

@@ -1,5 +1,5 @@
 module ActiveFedora::Associations::Builder
-  class BasicContains < CollectionAssociation #:nodoc:
+  class BasicContains < CollectionAssociation # :nodoc:
     def self.macro
       :is_a_container
     end

data/lib/active_fedora/associations/builder/belongs_to.rb

@@ -1,5 +1,5 @@
 module ActiveFedora::Associations::Builder
-  class BelongsTo < SingularAssociation #:nodoc:
+  class BelongsTo < SingularAssociation # :nodoc:
     def self.macro
       :belongs_to
     end
@@ -19,9 +19,7 @@ module ActiveFedora::Associations::Builder
     end
 
     def self.define_validations(model, reflection)
-      if reflection.options.key?(:required)
-        reflection.options[:optional] = !reflection.options.delete(:required)
-      end
+      reflection.options[:optional] = !reflection.options.delete(:required) if reflection.options.key?(:required)
 
       required = if reflection.options[:optional].nil?
                    model.belongs_to_required_by_default
@@ -31,9 +29,7 @@ module ActiveFedora::Associations::Builder
 
       super
 
-      if required
-        model.validates_presence_of reflection.name, message: :required
-      end
+      model.validates_presence_of reflection.name, message: :required if required
     end
   end
 end

data/lib/active_fedora/associations/builder/collection_association.rb

@@ -1,6 +1,6 @@
 require 'active_fedora/associations'
 module ActiveFedora::Associations::Builder
-  class CollectionAssociation < Association #:nodoc:
+  class CollectionAssociation < Association # :nodoc:
     CALLBACKS = [:before_add, :after_add, :before_remove, :after_remove].freeze
 
     def self.valid_options(options)
@@ -15,11 +15,11 @@ module ActiveFedora::Associations::Builder
     end
 
     def self.define_extensions(model, name)
-      if block_given?
-        extension_module_name = "#{model.name.demodulize}#{name.to_s.camelize}AssociationExtension"
-        extension = Module.new(&Proc.new)
-        model.parent.const_set(extension_module_name, extension)
-      end
+      return unless block_given?
+
+      extension_module_name = "#{model.name.demodulize}#{name.to_s.camelize}AssociationExtension"
+      extension = Module.new(&Proc.new)
+      model.parent.const_set(extension_module_name, extension)
     end
 
     def self.define_callback(model, callback_name, name, options)
@@ -44,7 +44,7 @@ module ActiveFedora::Associations::Builder
 
     def self.wrap_scope(scope, mod)
       if scope
-        if scope.arity > 0
+        if scope.arity.positive?
           proc { |owner| instance_exec(owner, &scope).extending(mod) }
         else
           proc { instance_exec(&scope).extending(mod) }

data/lib/active_fedora/associations/builder/directly_contains.rb

@@ -1,5 +1,5 @@
 module ActiveFedora::Associations::Builder
-  class DirectlyContains < CollectionAssociation #:nodoc:
+  class DirectlyContains < CollectionAssociation # :nodoc:
     def self.macro
       :directly_contains
     end
@@ -10,11 +10,9 @@ module ActiveFedora::Associations::Builder
 
     def self.validate_options(options)
       super
-      if !options[:has_member_relation] && !options[:is_member_of_relation]
-        raise ArgumentError, "You must specify a :has_member_relation or :is_member_of_relation predicate for #{name}"
-      elsif !options[:has_member_relation].is_a?(RDF::URI) && !options[:is_member_of_relation].is_a?(RDF::URI)
-        raise ArgumentError, "Predicate must be a kind of RDF::URI"
-      end
+
+      raise ArgumentError, "You must specify a :has_member_relation or :is_member_of_relation predicate for #{name}" if !options[:has_member_relation] && !options[:is_member_of_relation]
+      raise ArgumentError, "Predicate must be a kind of RDF::URI" if !options[:has_member_relation].is_a?(RDF::URI) && !options[:is_member_of_relation].is_a?(RDF::URI)
     end
   end
 end