ARGO-342: whitespace cleanup
ARGO-342: more style cleanup
ARGO-342: more style stuff
ARGO-342: more style stuff
ARGO-342: more style stuff
minor cleanup of unneeded stub calls
Use Archive format FOXML, not Migrate format
Provide example for fedora REST API download of test data
Use supported rspec syntax (expect, not should)

This prevents the warning:
  Using `should` from rspec-expectations' old `:should` syntax without explicitly enabling the syntax is deprecated.
  Use the new `:expect` syntax or explicitly enable `:should` instead.

There is still a similar warning from rspec-mocks about .stub(), but I
think that will evaporate if we clean up the Gemfile.

Rspec is moving away from .should because it requires monkey-patching
the base Object class, which can produce extremely phantom errors.
Test was passing anyway, which means it is not pending

Possible copy/paste error.
Pure whitespace format correction
ARGO-342: centralize shared logic between source_id and set_tags (into upd_values_for_druids)
ARGO-342: fix row creation in upd_values_for_druids and its callers to use a common name for update data field (and fix hardcoded reference to one caller's old name for the field).
ARGO-318: minor indentation cleanup
ARGO-342: formatting and variable declaration/naming cleanup, make line parsing more robust for upd_values_for_druids (used by source id and tag setting)
ARGO-342: more minor formatting cleanup
ARGO-342: clarify/fix instructional text for source id and tag editing
Upstream already overrides to_solr with same processing

Only the order of operations was different.  Code otherwise the same.
Looks like something that made it into a common gem but never made it
out of the original.
Bump up some versions
Remove extra space