HYDRASIR-404 A Profile Section should always be open/public

1. ProfileSection class overrides visibility to return public access right by default.
2. No Access Rights section is shown on the new/edit profile section form.
3. Updated existing specs to test for both of the above changes.
Change language from Proxy to Delegate

HYDRASIR-409 #close
Adding rescue clause for failed CI task

Prevent the generator from failing if it is unable to remove a test
factory.

HYDRASIR-417 #close

JIRA References:
https://jira.duraspace.org/browse/HYDRASIR-417
https://jira.duraspace.org/browse/HYDRASIR-275
Allow repository manager to see embargoed content

Edited app/helpers/hydramata/solr_helper.rb to not add embargo to
the solr query if the current user is a repository manager.

Added a new test to verify that a repository manager can discover
another user's embargoed work.

Note: Managers could already see the show pages for embargoed
works.  This sub-task only needed to fix discoverability.

HYDRASIR-355 #close
Merge pull request #486 from projecthydra/fix/HYDRASIR-417

Adding rescue clause for failed CI task
Merge pull request #485 from projecthydra/feature/409-change-proxy-to-delegate

Change language from Proxy to Delegate
Merge pull request #484 from projecthydra/feature/HYDRASIR-404

HYDRASIR-404 A Profile Section should always be open/public
Not looking up User in view

Given that we already know the user we are editing, instead refer
to the object that is being edited (i.e. f.object is the user).

In the case of editing your own profile, devise's convention is to
have the route be /users/edit. This means that `params[:id]` is nil.

[skip ci]
Merge pull request #488 from projecthydra/fix/current_user_error

Not looking up User in view
Merge pull request #487 from projecthydra/feature/HYDRASIR-355

Allow repository manager to see embargoed content
Visual adjustments for groups management page. HYDRASIR-403
Reworking contributors to not create people

TL;DR
* We exposed a Contributor attribute in the UI
* This was transformed to the :creator in the datastream
* When we persisted the object any literal for a contributor would be
used to create a Person object

To Run the Conversion from Command Line

$ rails console
> require 'curate/migration_services/runner'
> runner = Curate::MigrationServices::Runner.new(container_namespace: "::Curate::MigrationServices::MigrationContainers::RelatedContributor")
> runner.run

To Run via Capistrano, add the following to your Capfile, then look
at the available cap tasks

require 'curate/migration_services/capistrano'

Details

For some confounding reason, the form element for Contributor was
being persisted to the persisted :creator predicate and not a
:contributor predicate.

The assumption for this transformation is that any triple with the
:creator predicate and an object that is a PID can safely be mapped
from the :creator predicate to the :contributor predicate. The reason
being that we have not exposed a means of appending PIDs to :creator
except via the UI. And we have not exposed the :creator attribute
to the UI.

In the case of Image, we've only ever exposed the :contributor
attribute.

See also: 0d57a81dd2d62191f7e0267def5459f93d42c21b
See also: ndlib/planning#137

JIRA References:
https://jira.duraspace.org/browse/HYDRASIR-275
Merge pull request #489 from projecthydra/feature/HYDRASIR-403

Visual adjustments for groups management page. HYDRASIR-403
Merge pull request #490 from projecthydra/feature/HYDRASIR-275

Reworking contributors to not create people
Added unit tests for embargo. Test coverage now includes testing search and show view for owner, non-logged in user and a user in a group that has access to the work. The tests put a work in embargo and out of embargo."

Added tests to make sure search (Solr query) is correctly enforcing embargo

Added tests to make sure search (Solr query) is correctly enforcing embargo

Tests for making sure groups work correctly with embargo
Merge pull request #494 from projecthydra/feature/embargo-tests

Added unit tests for embargo. Test coverage now includes testing search ...
Use group pid instead of title in solr_helper.rb
Fix merge conflict