curation_concerns 0.3.0 → 0.4.0

data/spec/models/upload_set_spec.rb

@@ -1,28 +1,26 @@
 require 'spec_helper'
 
 describe UploadSet do
-  let(:user) { FactoryGirl.find_or_create(:jill) }
-  let(:upload_set) { described_class.create(title: ["test collection"], creator: [user.user_key]) }
+  let(:user) { create(:user) }
+  let(:upload_set) { described_class.create(title: ["test collection"]) }
   subject { upload_set }
 
   it "has dc metadata" do
-    expect(subject.creator).to eq [user.user_key]
     expect(subject.title).to eq ["test collection"]
   end
 
-  it "responds to .file_sets" do
-    expect(subject).to respond_to(:file_sets)
+  it "responds to #works" do
+    expect(subject).to respond_to(:works)
   end
 
   it "supports to_solr" do
     expect(subject.to_solr).to_not be_nil
     expect(subject.to_solr["upload_set__title_t"]).to be_nil
-    expect(subject.to_solr["upload_set__creator_t"]).to be_nil
   end
 
   describe "find_or_create" do
     describe "when the object exists" do
-      let!(:upload_set) { described_class.create(title: ["test collection"], creator: [user.user_key]) }
+      let!(:upload_set) { described_class.create(title: ["test collection"]) }
       it "finds upload_set instead of creating" do
         expect(described_class).to_not receive(:create)
         described_class.find_or_create(subject.id)

data/spec/presenters/curation_concerns/collection_presenter_spec.rb

@@ -2,10 +2,14 @@ require 'spec_helper'
 
 describe CurationConcerns::CollectionPresenter do
   let(:collection) { Collection.new(id: 'adc12v', description: 'a nice collection', title: 'A clever title') }
+  let(:work) { FactoryGirl.build(:work, title: ['unimaginitive title']) }
   let(:solr_document) { SolrDocument.new(collection.to_solr) }
   let(:ability) { double }
   let(:presenter) { described_class.new(solr_document, ability) }
 
+  # Mock bytes so collection does not have to be saved.
+  before { allow(collection).to receive(:bytes).and_return(0) }
+
   describe '#title' do
     subject { presenter.title }
     it { is_expected.to eq 'A clever title' }
@@ -15,4 +19,20 @@ describe CurationConcerns::CollectionPresenter do
     subject { presenter.to_key }
     it { is_expected.to eq ['adc12v'] }
   end
+
+  describe "#size" do
+    subject { presenter.size }
+    it { is_expected.to eq '0 Bytes' }
+  end
+
+  describe "#total_items" do
+    subject { presenter.total_items }
+    context "empty collection" do
+      it { is_expected.to eq 0 }
+    end
+    context "collection with work" do
+      before { collection.members << work }
+      it { is_expected.to eq 1 }
+    end
+  end
 end

data/spec/presenters/curation_concerns/file_set_presenter_spec.rb

@@ -43,10 +43,17 @@ describe CurationConcerns::FileSetPresenter do
     it { is_expected.to eq 'File Set' }
   end
 
-  describe "date_uploaded" do
+  describe "properties delegated to solr_document" do
+    let(:solr_properties) do
+      ["date_uploaded", "depositor", "tags", "title_or_label",
+       "contributor", "creator", "title", "description", "publisher",
+       "subject", "language", "rights"]
+    end
     it "delegates to the solr_document" do
-      expect(solr_document).to receive(:date_uploaded)
-      presenter.date_uploaded
+      solr_properties.each do |property|
+        expect(solr_document).to receive(property.to_sym)
+        presenter.send(property)
+      end
     end
   end
 

data/spec/presenters/curation_concerns/work_show_presenter_spec.rb

@@ -2,10 +2,15 @@ require 'spec_helper'
 
 describe CurationConcerns::WorkShowPresenter do
   let(:solr_document) { SolrDocument.new(attributes) }
+  let(:date_value) { Date.today }
+  let(:date_index) { date_value.to_s }
   let(:attributes) do
     { "title_tesim" => ["foo bar"],
       "human_readable_type_tesim" => ["Generic Work"],
-      "has_model_ssim" => ["GenericWork"] }
+      "has_model_ssim" => ["GenericWork"],
+      "date_created_dtsi" => date_index,
+      "date_modified_dtsi" => date_index,
+      "date_uploaded_dtsi" => date_index }
   end
 
   let(:ability) { nil }
@@ -26,6 +31,13 @@ describe CurationConcerns::WorkShowPresenter do
     it { is_expected.to be_kind_of ActiveModel::Name }
   end
 
+  [:date_created, :date_modified, :date_uploaded].each do |date_field|
+    describe "##{date_field}" do
+      subject { presenter.send date_field }
+      it { is_expected.to eq date_value.to_formatted_s(:standard) }
+    end
+  end
+
   describe "#permission_badge" do
     it "calls the PermissionBadge object" do
       expect_any_instance_of(CurationConcerns::PermissionBadge).to receive(:render)

data/spec/renderers/curation_concerns/attribute_renderer_spec.rb

@@ -1,14 +1,47 @@
 require 'spec_helper'
 
 describe CurationConcerns::AttributeRenderer do
-  let(:renderer) { described_class.new(:name, ['Bob', 'Jessica']) }
-
+  let(:field) { :name }
+  let(:renderer) { described_class.new(field, ['Bob', 'Jessica']) }
+  let(:yml_path) { File.join(File.dirname(__FILE__), '..', '..', 'fixtures', 'locales', '*.{rb,yml}') }
+  before do
+    I18n.load_path += Dir[File.join(yml_path)]
+    I18n.reload!
+  end
+  after do
+    I18n.load_path -= Dir[File.join(yml_path)]
+    I18n.reload!
+  end
   describe "#attribute_to_html" do
-    subject { renderer.render }
-
-    it { is_expected.to eq "<tr><th>Name</th>\n" \
-       "<td><ul class='tabular'><li class=\"attribute name\">Bob</li>\n" \
-       "<li class=\"attribute name\">Jessica</li>\n" \
-       "</ul></td></tr>" }
+    subject { Nokogiri::HTML(renderer.render) }
+    let(:expected) { Nokogiri::HTML(tr_content) }
+    context 'without microdata enabled' do
+      before do
+        CurationConcerns.config.display_microdata = false
+      end
+      let(:tr_content) {
+        "<tr><th>Name</th>\n" \
+         "<td><ul class='tabular'><li class=\"attribute name\"><span>Bob</span></li>\n" \
+         "<li class=\"attribute name\"><span>Jessica</span></li>\n" \
+         "</ul></td></tr>"
+      }
+      it { expect(renderer).not_to be_microdata(field) }
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+    context 'with microdata enabled' do
+      before do
+        CurationConcerns.config.display_microdata = true
+      end
+      let(:tr_content) {
+        "<tr><th>Name</th>\n" \
+         "<td><ul class='tabular'><li class=\"attribute name\" itemscope itemtype=\"http://schema.org/Person\" itemprop=\"name\">" \
+         "<span itemprop=\"firstName\">Bob</span></li>\n" \
+         "<li class=\"attribute name\" itemscope itemtype=\"http://schema.org/Person\" itemprop=\"name\">" \
+         "<span itemprop=\"firstName\">Jessica</span></li>\n" \
+         "</ul></td></tr>"
+      }
+      it { expect(renderer).to be_microdata(field) }
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
   end
 end

data/spec/routing/curation_concerns/routes_spec.rb

@@ -50,12 +50,12 @@ module CurationConcerns
       it 'routes to #show' do
         expect(curation_concerns_generic_work_path(generic_work)).to eq "/concern/generic_works/#{generic_work.id}"
         expect(get("/concern/generic_works/#{generic_work.id}")).to route_to(controller: 'curation_concerns/generic_works', action: 'show', id: generic_work.id)
-        expect(url_for([:curation_concerns, generic_work, only_path: true])).to eq "/concern/generic_works/#{generic_work.id}"
+        expect(url_for([generic_work, only_path: true])).to eq "/concern/generic_works/#{generic_work.id}"
       end
       it 'routes to #edit' do
         expect(edit_curation_concerns_generic_work_path(generic_work)).to eq "/concern/generic_works/#{generic_work.id}/edit"
         expect(get("/concern/generic_works/#{generic_work.id}/edit")).to route_to(controller: 'curation_concerns/generic_works', action: 'edit', id: generic_work.id)
-        expect(url_for([:edit, :curation_concerns, generic_work, only_path: true])).to eq "/concern/generic_works/#{generic_work.id}/edit"
+        expect(url_for([:edit, generic_work, only_path: true])).to eq "/concern/generic_works/#{generic_work.id}/edit"
       end
     end
   end

data/spec/routing/route_spec.rb

@@ -47,7 +47,7 @@ describe 'Routes', type: :routing do
     end
   end
 
-  describe 'GenericFile' do
+  describe 'FileSet' do
     it 'routes to create' do
       expect(post: 'concern/container/1/file_sets').to route_to(controller: 'curation_concerns/file_sets', action: 'create', parent_id: '1')
     end

data/spec/services/file_set_audit_service_spec.rb

@@ -53,19 +53,61 @@ describe CurationConcerns::FileSetAuditService do
   end
 
   describe '#human_readable_audit_status' do
+    before do
+      CurationConcerns::VersioningService.create(f.original_file)
+      ChecksumAuditLog.create!(pass: 1, file_set_id: f.id, version: f.original_file.versions.first.uri, file_id: 'original_file')
+    end
+    subject { service.human_readable_audit_status }
+    it { is_expected.to eq 'Some audits have not been run, but the ones run were passing.' }
+  end
+
+  describe '#logged_audit_status' do
+    subject { service.logged_audit_status }
+
+    it "doesn't trigger audits" do
+      expect(service).not_to receive(:audit_file)
+      expect(subject).to eq "Audits have not yet been run on this file."
+    end
+
+    context "when no audit is passing" do
+      before do
+        ChecksumAuditLog.create!(pass: 1, file_set_id: f.id, version: f.original_file.versions.first.label, file_id: 'original_file')
+      end
+
+      it "reports the audit result" do
+        expect(subject).to eq 'passing'
+      end
+    end
+
+    context "when one audit is passing" do
+      before do
+        ChecksumAuditLog.create!(pass: 0, file_set_id: f.id, version: f.original_file.versions.first.label, file_id: 'original_file')
+        ChecksumAuditLog.create!(pass: 1, file_set_id: f.id, version: f.original_file.versions.first.label, file_id: 'original_file')
+      end
+
+      it "reports the audit result" do
+        expect(subject).to eq 'failing'
+      end
+    end
+  end
+
+  describe '#stat_to_string' do
+    subject { service.send(:stat_to_string, val) }
     context 'when audit_stat is 0' do
-      subject { service.human_readable_audit_status 0 }
+      let(:val) { 0 }
       it { is_expected.to eq 'failing' }
     end
 
     context 'when audit_stat is 1' do
-      subject { service.human_readable_audit_status 1 }
+      let(:val) { 1 }
       it { is_expected.to eq 'passing' }
     end
 
     context 'when audit_stat is something else' do
-      subject { service.human_readable_audit_status 'something else' }
-      it { is_expected.to eq 'something else' }
+      let(:val) { 'something else' }
+      it "fails" do
+        expect { subject }.to raise_error ArgumentError, "Unknown status `something else'"
+      end
     end
   end
 end

data/spec/spec_helper.rb

@@ -10,7 +10,10 @@ require 'database_cleaner'
 require 'engine_cart'
 EngineCart.load_application!
 require 'devise'
+require 'mida'
 
+require 'rspec/matchers'
+require 'equivalent-xml/rspec_matchers'
 require 'rspec/its'
 require 'rspec/rails'
 require 'rspec/active_model/mocks'
@@ -29,6 +32,8 @@ if ENV['COVERAGE'] || $in_travis
   SimpleCov.formatter = Coveralls::SimpleCov::Formatter
   SimpleCov.start('rails') do
     add_filter '/spec'
+    add_filter '/lib/generators/curation_concerns/templates'
+    add_filter '/.internal_test_app'
   end
   SimpleCov.command_name('spec')
 end
@@ -79,13 +84,6 @@ RSpec.configure do |config|
   config.deprecation_stream
 end
 
-module FactoryGirl
-  def self.find_or_create(handle, by = :email)
-    tmpl = FactoryGirl.build(handle)
-    tmpl.class.send("find_by_#{by}".to_sym, tmpl.send(by)) || FactoryGirl.create(handle)
-  end
-end
-
 if defined?(ClamAV)
   ClamAV.instance.loaddb
 else

data/spec/views/collections/_sort_and_per_page.html.erb_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe 'collections/_sort_and_per_page.html.erb' do
+  let(:collection) { double }
+  let(:response) { double(response: { 'numFound' => 3 }) }
+
+  before do
+    allow(view).to receive(:sort_fields).and_return(['title_sort', 'date_sort'])
+    allow(view).to receive(:document_index_views).and_return(list: Blacklight::Configuration::ViewConfig.new)
+    assign(:response, response)
+  end
+
+  context "when the action is edit" do
+    before do
+      controller.action_name = "edit"
+    end
+    it "renders the form with a button" do
+      expect(view).to receive(:button_for_remove_selected_from_collection).with(collection)
+      render 'collections/sort_and_per_page', collection: collection
+    end
+  end
+
+  context "when the action is show" do
+    before do
+      controller.action_name = "show"
+    end
+    it "renders the form without a button" do
+      expect(view).not_to receive(:button_for_remove_selected_from_collection)
+      render 'collections/sort_and_per_page', collection: collection
+    end
+  end
+end

data/spec/views/curation_concerns/base/_attributes.html.erb_spec.rb

@@ -4,14 +4,17 @@ describe 'curation_concerns/base/_attributes.html.erb' do
   let(:creator)     { 'Bilbo' }
   let(:contributor) { 'Frodo' }
   let(:subject)     { 'history' }
+  let(:description) { ['Lorem ipsum lorem ipsum. http://my.link.com'] }
 
   let(:solr_document) { SolrDocument.new(subject_tesim: subject,
                                          contributor_tesim: contributor,
-                                         creator_tesim: creator) }
+                                         creator_tesim: creator,
+                                         description_tesim: description) }
   let(:ability) { nil }
   let(:presenter) do
     CurationConcerns::WorkShowPresenter.new(solr_document, ability)
   end
+  let(:doc) { Nokogiri::HTML(rendered) }
 
   before do
     allow(view).to receive(:dom_class) { '' }
@@ -25,4 +28,8 @@ describe 'curation_concerns/base/_attributes.html.erb' do
     expect(rendered).to have_link(contributor, href: catalog_index_path(search_field: 'contributor', q: contributor))
     expect(rendered).to have_link(subject, href: catalog_index_path(search_field: 'subject', q: subject))
   end
+  it 'shows links in the description' do
+    a1 = doc.xpath("//li[@class='attribute description']/span/a").text
+    expect(a1).to start_with 'http://my.link.com'
+  end
 end

data/spec/views/curation_concerns/base/show.html.erb_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe 'curation_concerns/base/show.html.erb' do
   let(:object_profile) { ["{\"id\":\"999\"}"] }
-  let(:contributor) { 'Frodo' }
+  let(:contributor) { ['Frodo', 'Sam'] }
   let(:creator)     { 'Bilbo' }
   let(:solr_document) {
     SolrDocument.new(
@@ -12,6 +12,7 @@ describe 'curation_concerns/base/show.html.erb' do
       human_readable_type_tesim: ['Generic Work'],
       contributor_tesim: contributor,
       creator_tesim: creator,
+      language_tesim: 'Hobbish',
       rights_tesim: ['http://creativecommons.org/licenses/by/3.0/us/']
     )
   }
@@ -31,7 +32,7 @@ describe 'curation_concerns/base/show.html.erb' do
 
     it 'draws the page' do
       expect(rendered).to have_link 'Attribution 3.0 United States', href: 'http://creativecommons.org/licenses/by/3.0/us/'
-      expect(rendered).to have_link 'Edit This Generic Work', href: edit_polymorphic_path([:curation_concerns, presenter])
+      expect(rendered).to have_link 'Edit This Generic Work', href: edit_polymorphic_path(presenter)
     end
   end
 
@@ -46,4 +47,37 @@ describe 'curation_concerns/base/show.html.erb' do
       expect(rendered).not_to have_content('Edit this Generic Work')
     end
   end
+
+  describe 'schema.org' do
+    before do
+      assign(:presenter, presenter)
+      allow(view).to receive(:can?).with(:edit, String).and_return(false)
+      allow(view).to receive(:can?).with(:collect, String).and_return(false)
+      render
+    end
+    let(:item) { Mida::Document.new("<html>#{rendered}</html>").items.first }
+    describe 'descriptive metadata' do
+      it 'draws schema.org fields' do
+        # default itemtype to CreativeWork
+        expect(item.type).to eq('http://schema.org/CreativeWork')
+
+        contributors = item.properties['contributor']
+        expect(contributors.count).to eq(2)
+        contributor = contributors.last
+        expect(contributor.type).to eq('http://schema.org/Person')
+        expect(contributor.properties['name'].first).to eq('Sam')
+
+        creators = item.properties['creator']
+        expect(creators.count).to eq(1)
+        creator = creators.first
+        expect(creator.type).to eq('http://schema.org/Person')
+        expect(creator.properties['name'].first).to eq('Bilbo')
+
+        languages = item.properties['inLanguage']
+        expect(languages.count).to eq(1)
+        language = languages.first
+        expect(language).to eq('Hobbish')
+      end
+    end
+  end
 end

data/spec/views/curation_concerns/file_sets/_file_set.html.erb_spec.rb

@@ -3,6 +3,7 @@ require 'spec_helper'
 describe 'curation_concerns/file_sets/_file_set.html.erb' do
   let(:solr_document) { SolrDocument.new(id: '999',
                                          has_model_ssim: ['FileSet'],
+                                         active_fedora_model_ssi: 'FileSet',
                                          thumbnail_path_ss: '/downloads/999?file=thumbnail',
                                          representative_tesim: ["999"],
                                          title_tesim: ["My File"]) }
@@ -28,9 +29,9 @@ describe 'curation_concerns/file_sets/_file_set.html.erb' do
     expect(rendered).to have_selector ".thumbnail img[src='#{download_path(presenter, file: 'thumbnail')}']"
 
     # Action buttons
-    expect(rendered).to have_selector "a[title=\"Edit My File\"][href='#{edit_polymorphic_path([:curation_concerns, presenter])}']", text: 'Edit'
+    expect(rendered).to have_selector "a[title=\"Edit My File\"][href='#{edit_polymorphic_path(presenter)}']", text: 'Edit'
     expect(rendered).to have_selector "a[title=\"Rollback to previous version\"][href='#{versions_curation_concerns_file_set_path(presenter)}']", text: 'Rollback'
-    expect(rendered).to have_selector "a[title=\"Delete My File\"][data-method='delete'][href='#{polymorphic_path([:curation_concerns, presenter])}']", text: 'Delete'
+    expect(rendered).to have_selector "a[title=\"Delete My File\"][data-method='delete'][href='#{polymorphic_path(presenter)}']", text: 'Delete'
     expect(rendered).to have_link('Download')
     expect(rendered).to have_selector "a[title='Download \"My File\"'][href='#{download_path(presenter)}']", text: 'Download'
   end

data/spec/views/curation_concerns/file_sets/show.html.erb_spec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+describe 'curation_concerns/file_sets/show.html.erb', type: :view do
+  before do
+    allow(view).to receive(:parent).and_return(parent)
+  end
+
+  let(:parent) { stub_model(GenericWork) }
+
+  let(:depositor) do
+    stub_model(User,
+               user_key: 'bob',
+               twitter_handle: 'bot4lib')
+  end
+
+  let(:content) do
+    double('content', versions: [], mimeType: 'application/pdf')
+  end
+
+  let(:file_set) do
+    stub_model(FileSet, id: '123',
+                        depositor: depositor.user_key,
+                        audit_stat: 1,
+                        title: ['My Title'],
+                        description: ['Lorem ipsum lorem ipsum. http://my.link.com']
+              )
+  end
+
+  let(:ability) { double }
+  let(:solr_doc) { SolrDocument.new(file_set.to_solr) }
+  let(:presenter) { CurationConcerns::FileSetPresenter.new(solr_doc, ability) }
+
+  before do
+    view.lookup_context.view_paths.push CurationConcerns::Engine.root + 'app/views/curation_concerns/base'
+    allow(view).to receive(:can?).with(:edit, String).and_return(true)
+    assign(:presenter, presenter)
+  end
+
+  describe 'title heading' do
+    before do
+      stub_template 'shared/_title_bar.html.erb' => 'Title Bar'
+      render template: 'curation_concerns/file_sets/show.html.erb', layout: 'layouts/curation_concerns'
+    end
+    it 'shows the title' do
+      expect(rendered).to have_selector 'h1 > small', text: 'My Title'
+    end
+  end
+
+  describe 'attributes' do
+    before { render }
+
+    it 'shows the description' do
+      expect(rendered).to have_selector '.attribute.description', text: 'Lorem ipsum'
+    end
+  end
+end

data/spec/views/curation_concerns/single_use_links_viewer/show.html.erb_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe 'curation_concerns/single_use_links_viewer/show.html.erb' do
+  let(:ability) { double }
+  let(:model) { stub_model(FileSet, title: ['world.png']) }
+  let(:solr_document) { SolrDocument.new(model.to_solr) }
+  let(:presenter) { CurationConcerns::FileSetPresenter.new(solr_document, ability) }
+  let(:download_link) { '/a_path' }
+  before do
+    assign(:presenter, presenter)
+    assign(:download_link, download_link)
+    view.lookup_context.view_paths.push 'app/views/curation_concerns/base'
+    render
+  end
+
+  it "renders the page" do
+    expect(rendered).to have_selector 'h1', text: 'world.png'
+  end
+end

data/spec/views/shared/_add_content.html.erb_spec.rb

@@ -14,7 +14,7 @@ describe 'shared/_add_content.html.erb' do
 
     it 'has links to create works and collections' do
       CurationConcerns.config.curation_concerns.each do |curation_concern_type|
-        expect(rendered).to have_link("New #{curation_concern_type.human_readable_type}", href: new_polymorphic_path([:curation_concerns, curation_concern_type]))
+        expect(rendered).to have_link("New #{curation_concern_type.human_readable_type}", href: new_polymorphic_path(curation_concern_type))
       end
       expect(rendered).to have_link('Add a Collection', href: collections.new_collection_path)
     end
@@ -30,7 +30,7 @@ describe 'shared/_add_content.html.erb' do
 
     it 'has links to add collections but not to add works' do
       CurationConcerns.config.curation_concerns.each do |curation_concern_type|
-        expect(rendered).not_to have_link("New #{curation_concern_type.human_readable_type}", href: new_polymorphic_path([:curation_concerns, curation_concern_type]))
+        expect(rendered).not_to have_link("New #{curation_concern_type.human_readable_type}", href: new_polymorphic_path(curation_concern_type))
       end
       expect(rendered).to have_link('Add a Collection', href: collections.new_collection_path)
     end
@@ -47,7 +47,7 @@ describe 'shared/_add_content.html.erb' do
       expect(rendered).not_to have_text('Add')
       expect(rendered).not_to have_text('Admin')
       CurationConcerns.config.curation_concerns.each do |curation_concern_type|
-        expect(rendered).not_to have_link("New #{curation_concern_type.human_readable_type}", href: new_polymorphic_path([:curation_concerns, curation_concern_type]))
+        expect(rendered).not_to have_link("New #{curation_concern_type.human_readable_type}", href: new_polymorphic_path(curation_concern_type))
       end
       expect(rendered).not_to have_link('Add a Collection', href: collections.new_collection_path)
     end

data/spec/views/single_use_links/new_download.html.erb_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe 'curation_concerns/single_use_links/new_download.html.erb' do
-  let(:user) { FactoryGirl.find_or_create(:jill) }
+  let(:user) { create(:user) }
 
   let(:f) do
     file = FileSet.create do |gf|

data/spec/views/upload_sets/_metadata.html.erb_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe 'upload_sets/_metadata.html.erb' do
+  let(:user) { build(:user) }
+  let(:ability) { Ability.new(user) }
+  let(:upload_set) { UploadSet.create }
+  let(:form) { CurationConcerns::UploadSetForm.new(upload_set, ability) }
+  let(:work1) { build(:work, id: 'work1', title: ['First work']) }
+  let(:work2) { build(:work, id: 'work2', title: ['Second work']) }
+
+  let(:f) do
+    allow(upload_set).to receive(:works).and_return([work1, work2])
+    view.simple_form_for(form, url: '/update', builder: CurationConcerns::FormBuilder) do |fs_form|
+      return fs_form
+    end
+  end
+
+  before do
+    allow(view).to receive(:f).and_return(f)
+    render
+  end
+
+  it "draws the form" do
+    expect(rendered).to have_text 'Applies to all files just uploaded'
+    expect(rendered).to have_css 'input[name="title[work1][]"][value="First work"]'
+    expect(rendered).to have_css 'input[name="title[work2][]"][value="Second work"]'
+  end
+end