Merge pull request #985 from pulibrary/all_collection_dont_embed_mani…

…fests

Don't embed manifests in all collections manifest.

app/services/sparse_member_collection_manifest.rb

@@ -4,8 +4,12 @@ def apply(manifest)
     manifest['collections'] += [self.manifest]
   end
 
-  def child_manifest_factory
-    OnlyPropertyManifestBuilder
+  def manifest_builders
+    nil
+  end
+
+  def manifest_builder_class
+    IIIF::Presentation::Collection.new
   end
 
   def sequence_builder

spec/services/all_collections_manifest_builder_spec.rb

@@ -10,7 +10,7 @@
       expect(manifest_json["collections"].length).to eq 1
       expect(manifest_json["collections"].first["metadata"]).not_to be_blank
     end
-    it "embeds manifests" do
+    it "doesn't embed manifests" do
       c = FactoryGirl.create(:collection)
       s = FactoryGirl.create(:scanned_resource_with_multi_volume_work)
       s2 = FactoryGirl.create(:scanned_resource)
@@ -21,7 +21,7 @@
       s2.save!
 
       expect(manifest_json["collections"].length).to eq 1
-      expect(manifest_json["collections"].first["manifests"]).not_to be_blank
+      expect(manifest_json["collections"].first["manifests"]).to be_blank
     end
     it "doesn't populate manifests" do
       FactoryGirl.create(:collection)