sufia 4.0.0.rc1 → 4.0.0.rc2

data/spec/models/local_authority_spec.rb

@@ -35,7 +35,7 @@ describe LocalAuthority do
     LocalAuthority.harvest_tsv("geo", @tsv, prefix: 'http://sws.geonames.org/')
     LocalAuthority.count.should == 1
     auth = LocalAuthority.where(name: "geo").first
-    LocalAuthorityEntry.where(local_authority_id: auth.id).first.uri.start_with?('http://sws.geonames.org/').should be_true
+    expect(LocalAuthorityEntry.where(local_authority_id: auth.id).first.uri).to start_with('http://sws.geonames.org/')
     LocalAuthorityEntry.count.should == 149
   end
   describe "when vocabs are harvested" do

data/spec/models/user_spec.rb

@@ -47,15 +47,15 @@ describe User do
       @another_user.delete
     end
     it "should be able to follow another user" do
-      @user.following?(@another_user).should be_true
-      @another_user.following?(@user).should be_false
-      @another_user.followed_by?(@user).should be_true
-      @user.followed_by?(@another_user).should be_false
+      expect(@user).to be_following(@another_user)
+      expect(@another_user).to_not be_following(@user)
+      expect(@another_user).to be_followed_by(@user)
+      expect(@user).to_not be_followed_by(@another_user)
     end
     it "should be able to unfollow another user" do
       @user.stop_following(@another_user)
-      @user.following?(@another_user).should be_false
-      @another_user.followed_by?(@user).should be_false
+      expect(@user).to_not be_following(@another_user)
+      expect(@another_user).to_not be_followed_by(@user)
     end
   end
 
@@ -73,4 +73,31 @@ describe User do
     end
 
   end
+
+  describe "activity streams" do
+    let(:now){DateTime.now.to_i}
+    let(:user) { @user }
+    let(:activities) {
+        [{ action: 'so and so edited their profile', timestamp: now },
+        { action: 'so and so uploaded a file', timestamp: (now - 360 ) }]
+    }
+    let(:file_activities) {
+      [{ action: 'uploaded a file', timestamp: now + 1 }]
+    }
+
+    before do
+      allow(user).to receive(:events).and_return(activities)
+      allow(user).to receive(:profile_events).and_return(file_activities)
+    end
+
+    it "gathers the user's recent activity within the default amount of time" do
+      expect(user.get_all_user_activity).to eq(file_activities.concat(activities))
+    end
+
+    it "gathers the user's recent activity within a given timestamp" do
+      expect(user.get_all_user_activity(now-60)).to eq(file_activities.concat([activities.first]))
+    end
+
+  end
+
 end

data/spec/spec_helper.rb

@@ -1,22 +1,23 @@
-# This file is copied to spec/ when you run 'rails generate rspec:install'
 ENV["RAILS_ENV"] ||= 'test'
 
-require File.expand_path("config/environment", ENV['RAILS_ROOT'] || File.expand_path("../internal", __FILE__))
-
-require 'database_cleaner'
+require 'engine_cart'
+EngineCart.load_application!
 
+require 'mida'
 require 'rspec/rails'
-require 'rspec/autorun'
+require 'rspec/its'
+require 'rspec/active_model/mocks'
+require 'factory_girl_rails'
 require 'capybara/poltergeist'
 Capybara.javascript_driver = :poltergeist
 Capybara.default_wait_time = ENV['TRAVIS'] ? 30 : 15
 require 'capybara/rspec'
 require 'capybara/rails'
 require 'equivalent-xml/rspec_matchers'
+require 'database_cleaner'
 
 require File.expand_path('../support/features', __FILE__)
 
-
 if ENV['COVERAGE']
   require 'simplecov'
   SimpleCov.start 'rails'
@@ -60,9 +61,6 @@ end
 
 Resque.inline = Rails.env.test?
 
-FactoryGirl.definition_file_paths = [File.expand_path("../factories", __FILE__)]
-FactoryGirl.find_definitions
-
 module EngineRoutes
   def self.included(base)
     base.routes { Sufia::Engine.routes }
@@ -91,7 +89,6 @@ RSpec.configure do |config|
     DatabaseCleaner.clean
   end
 
-
   # If true, the base class of anonymous controllers will be inferred
   # automatically. This will be the default behavior in future versions of
   # rspec-rails.

data/spec/{support → test_app_templates}/lib/generators/test_app_generator.rb

@@ -1,13 +1,18 @@
 require 'rails/generators'
 
 class TestAppGenerator < Rails::Generators::Base
-  source_root File.expand_path("../../../../support", __FILE__)
+  source_root "./spec/test_app_templates"
 
-  def run_sufia_generator
-    say_status("warning", "GENERATING SUFIA", :yellow)
-    generate 'sufia', '-f'
+  # if you need to generate any additional configuration
+  # into the test app, this generator will be run immediately
+  # after setting up the application
+
+  def install_engine
+    generate 'sufia:install', '-f'
+  end
+
+  def browse_everything_config
     generate "browse_everything:config"
-    remove_file 'spec/factories/users.rb'
   end
 
   def add_analytics_config
@@ -35,5 +40,5 @@ class TestAppGenerator < Rails::Generators::Base
     gsub_file 'app/assets/javascripts/application.js',
               '//= require_tree .', '//= require sufia'
   end
-
+    
 end

data/spec/views/batch/edit.html.erb_spec.rb

@@ -22,8 +22,14 @@ describe 'batch/edit.html.erb' do
     @page = Capybara::Node::Simple.new(rendered)
   end
 
-  it "should draw modal for license" do
-    expect(@page).to have_selector("div#myModal .modal-dialog .modal-content", count: 1)
+  it "should draw tooltip for visibility" do
+    expect(@page).to have_selector("span#visibility_tooltip", count: 1)
+    expect(@page).to have_selector("a#generic_file_visibility_help", count: 1)
+  end
+
+  it "should draw tooltip for share_with" do
+    expect(@page).to have_selector("span#share_with_tooltip", count: 1)
+    expect(@page).to have_selector("a#generic_file_share_with_help", count: 1)
   end
 
   it "should draw modal for rights" do

data/spec/views/catalog/index.html.erb_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe 'catalog/index.html.erb' do
+  before do
+    allow(view).to receive(:blacklight_config).and_return(CatalogController.blacklight_config)
+    stub_template 'catalog/_search_sidebar.html.erb' => ''
+    stub_template 'catalog/_search_header.html.erb' => ''
+    allow(view).to receive(:render_opensearch_response_metadata).and_return('')
+    allow(view).to receive(:render_grouped_response?).and_return(false)
+    allow(view).to receive(:search_session).and_return({})
+    allow(view).to receive(:current_search_session).and_return(nil)
+    allow(view).to receive(:document_counter_with_offset).and_return(5)
+    params[:view] = 'gallery'
+
+    resp = []
+    assign(:response, resp )
+    allow(resp).to receive(:total_pages).and_return(1)
+    allow(resp).to receive(:current_page).and_return(1)
+    allow(resp).to receive(:limit_value).and_return(10)
+    allow(resp).to receive(:empty?).and_return(false)
+
+    @collection = Collection.new(pid: 'sufia:foo', title: 'collection1')
+    doc = SolrDocument.new(@collection.to_solr.merge('has_model_ssim' => [Collection.to_class_uri]))
+    assign(:document_list, [doc])
+  end
+
+
+
+  it 'appears on page without error' do
+    render
+    expect(rendered).to include(@collection.title)
+  end
+
+end

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

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe 'collections/_form.html.erb' do
+  describe 'when the collection edit form is rendered' do
+    let(:collection) { Collection.new({title: 'the title', description: 'the description',
+                                       creator: 'the creator'})}
+
+    before do
+      controller.request.path_parameters[:id] = 'j12345'
+      collection.initialize_fields
+      assign(:collection, collection)
+    end
+
+    it "should draw the metadata fields for collection" do
+      render
+      expect(rendered).to have_selector("input#collection_title", count: 1)
+      expect(rendered).to have_selector("input#collection_creator", count: 1)
+      expect(rendered).to have_selector("textarea#collection_description", count: 1)
+      expect(rendered).to have_selector("input#collection_contributor", count: 1)
+      expect(rendered).to have_selector("input#collection_tag", count: 1)
+      expect(rendered).to have_selector("input#collection_subject", count: 1)
+      expect(rendered).to have_selector("input#collection_publisher", count: 1)
+      expect(rendered).to have_selector("input#collection_date_created", count: 1)
+      expect(rendered).to have_selector("input#collection_language", count: 1)
+      expect(rendered).to have_selector("input#collection_identifier", count: 1)
+      expect(rendered).to have_selector("input#collection_based_near", count: 1)
+      expect(rendered).to have_selector("input#collection_related_url", count: 1)
+      expect(rendered).to have_selector("select#collection_rights", count: 1)
+      expect(rendered).to have_selector("select#collection_resource_type", count: 1)
+    end
+  end
+end

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

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe 'collections/_show_descriptions.html.erb' do
+  context 'displaying a custom collection' do
+    before do
+      @collection = mock_model(Collection)
+      allow(view).to receive(:blacklight_config).and_return(Blacklight::Configuration.new)
+      allow(@collection).to receive(:date_modified).and_return(["today"])
+      allow(@collection).to receive(:terms_for_display).and_return([:date_modified])
+      allow(@collection).to receive(:members).and_return(["foo","bar"])
+      allow(@collection).to receive(:bytes).and_return(123456678)
+    end
+
+    it "should draw the metadata fields for collection" do
+      render
+      expect(rendered).to have_content 'Date modified'
+      expect(rendered).to include('itemprop="date_modified"')
+      expect(rendered).to have_content 'Total Items'
+      expect(rendered).to have_content '2'
+      expect(rendered).to have_content 'Size'
+      expect(rendered).to have_content '118 MB'
+    end
+  end
+
+end

data/spec/views/dashboard/index_spec.rb

@@ -11,6 +11,8 @@ describe "dashboard/index.html.erb" do
     allow(@user).to receive(:all_following).and_return(["magneto"])
     allow(@user).to receive(:followers).and_return(["wolverine","storm"])
     allow(controller).to receive(:current_user).and_return(@user)
+    allow(view).to receive(:number_of_files).and_return("15")
+    allow(view).to receive(:number_of_collections).and_return("3")
     assign(:activity, [])
     assign(:notifications, [])
   end
@@ -56,9 +58,13 @@ describe "dashboard/index.html.erb" do
       expect(@sidebar).to include "Your Statistics"
       expect(@sidebar).to include '<span class="label label-default">1</span>'
       expect(@sidebar).to include '<span class="label label-default">2</span>'
-      expect(@sidebar).to include '<span class="label label-default">0</span>'
+      expect(@sidebar).to include '<span class="label label-default">15</span>'
+      expect(@sidebar).to include '<span class="label label-default">3</span>'
     end
 
+    it "should show the statistics before the profile" do
+      expect(@sidebar).to match /Your Statistics.*Charles Francis Xavier/m
+    end
   end
 
   describe "main" do
@@ -71,19 +77,42 @@ describe "dashboard/index.html.erb" do
           { action: 'so and so edited their profile', timestamp: @now },
           { action: 'so and so uploaded a file', timestamp: (@now - 360 ) }
         ])
-        assign(:notifications, FactoryGirl.create(:user_with_mail).mailbox.inbox)
       end
 
       it "should include recent activities and notifications" do
         render
         expect(rendered).to include "so and so edited their profile"
         expect(rendered).to include "6 minutes ago"
-        expect(rendered).to include "You've got mail"
-        expect(rendered).to include "Sample notification"
       end
 
     end
 
+    context "with notifications" do
+
+      before do
+        assign(:notifications, FactoryGirl.create(:user_with_mail).mailbox.inbox)
+      end
+
+      it "shows a link to all notifications" do
+        render
+        expect(rendered).to include "See all notifications"
+      end
+
+      it "defaults to a limited number of notifications" do
+        render
+        expect(rendered).to include "Single File 9"
+        expect(rendered).to_not include "Single File 2"
+      end
+
+      it "allows showing more notifications" do
+        Sufia.config.max_notifications_for_dashboard = 6
+        render
+        expect(rendered).to include "Single File 1"
+      end
+
+
+    end
+
     context "without activities and notifications" do
 
       it "should include headings for activities and notifications" do

data/spec/views/generic_file/_breadcrumbs.html.erb_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe 'generic_files/_breadcrumbs.html.erb' do
+
+  let(:request) { double("request", referer: referer) }
+  let(:generic_file) { GenericFile.new(title: ["Fake object"]) }
+
+  describe 'when coming from dashboard' do
+    let! (:referer) { "http://...dashboard" }
+    it "should link back to dashboard" do
+      allow(view).to receive(:request).and_return(request)
+      render partial: 'generic_files/breadcrumbs', locals: {include_file: false, generic_file: generic_file }
+      expect(rendered).to have_link(t('sufia.bread_crumb.file_list'), sufia.dashboard_files_path)
+    end
+  end
+  describe 'when coming from files list' do
+    let! (:referer) { "http://...dashboard/files" }
+    it "should link back to files list" do
+      allow(view).to receive(:request).and_return(request)
+      render partial: 'generic_files/breadcrumbs', locals: {include_file: false, generic_file: generic_file }
+      expect(rendered).to have_link(t('sufia.bread_crumb.file_list'), sufia.dashboard_files_path)
+    end
+  end
+  describe 'when coming from collections list' do
+    let! (:referer) { "http://...dashboard/collections" }
+    it "should link back to collections" do
+      allow(view).to receive(:request).and_return(request)
+      render partial: 'generic_files/breadcrumbs', locals: {include_file: false, generic_file: generic_file }
+      expect(rendered).to have_link(t('sufia.bread_crumb.collections_list'), sufia.dashboard_collections_path)
+    end
+  end
+end

data/spec/views/generic_file/show.html.erb_spec.rb

@@ -1,23 +1,250 @@
 require 'spec_helper'
 
 describe 'generic_files/show.html.erb' do
-  let(:generic_file) {
+  let(:depositor) {
+    stub_model(User,
+      user_key: 'bob',
+      twitter_handle: 'bot4lib')
+  }
+
+  let(:content) do
     content = double('content', versions: [], mimeType: 'application/pdf')
+  end
+
+  let(:generic_file) do
     stub_model(GenericFile, noid: '123',
-               depositor: 'bob',
-               audit_stat: 1,
-               content: content)
-  }
+      depositor: depositor.user_key,
+      audit_stat: 1,
+      title: ['My Title'],
+      description: ['Lorem ipsum lorem ipsum.'],
+      tag: ['bacon', 'sausage', 'eggs'],
+      rights: ['http://example.org/rights/1'],
+      based_near: ['Seattle, WA, US'],
+      contributor: ['Tweedledee', 'Tweedledum'],
+      creator: ['Doe, John', 'Doe, Jane'],
+      date_created: ['1984-01-02'],
+      language: ['Quechua'],
+      publisher: ['Random Publishing, Inc.'],
+      subject: ['Biology', 'Physiology', 'Ethnography'],
+      content: content)
+  end
 
   before do
-    allow(controller).to receive(:current_user).and_return(stub_model(User))
+    allow(controller).to receive(:current_user).and_return(depositor)
     allow_any_instance_of(Ability).to receive(:can?).and_return(true)
+    allow(User).to receive(:find_by_user_key).with(generic_file.depositor).and_return(depositor)
+    allow(view).to receive(:blacklight_config).and_return(Blacklight::Configuration.new)
+    allow(view).to receive(:on_the_dashboard?).and_return(false)
     assign(:generic_file, generic_file)
     assign(:events, [])
+    assign(:notify_number, 0)
   end
 
-  describe 'analytics' do
+  describe 'schema.org' do
+    describe 'descriptive metadata' do
+      before do
+        render template: 'generic_files/show.html.erb', layout: 'layouts/sufia-one-column'
+        @item = Mida::Document.new(rendered).items.first
+      end
+
+      it 'sets itemtype to CreativeWork' do
+        expect(@item.type).to eq('http://schema.org/CreativeWork')
+      end
+
+      it 'sets title as name' do
+        expect(@item.properties['name'].first).to eq('My Title')
+      end
+
+      it 'sets description' do
+        expect(@item.properties['description'].first).to eq('Lorem ipsum lorem ipsum.')
+      end
+
+      it 'sets tag as keywords' do
+        expect(@item.properties['keywords']).to include('bacon', 'sausage', 'eggs')
+      end
+
+      it 'sets based_near as contentLocation' do
+        based_near = @item.properties['contentLocation'].first
+        expect(based_near.type).to eq('http://schema.org/Place')
+        expect(based_near.properties['name'].first).to eq('Seattle, WA, US')
+      end
+
+      it 'sets contributor' do
+        contributors = @item.properties['contributor']
+        expect(contributors.count).to eq(2)
+        contributor = contributors.first
+        expect(contributor.type).to eq('http://schema.org/Agent')
+        expect(contributor.properties['name'].first).to eq('Tweedledee')
+      end
+
+      it 'sets creator' do
+        creators = @item.properties['creator']
+        expect(creators.count).to eq(2)
+        creator = creators.first
+        expect(creator.type).to eq('http://schema.org/Agent')
+        expect(creator.properties['name'].first).to eq('Doe, John')
+      end
+
+      it 'sets date_created as dateCreated' do
+        expect(@item.properties['dateCreated'].first).to eq('1984-01-02')
+      end
+
+      it 'sets language as inLanguage' do
+        expect(@item.properties['inLanguage'].first).to eq('Quechua')
+      end
+
+      it 'sets publisher' do
+        publisher = @item.properties['publisher'].first
+        expect(publisher.type).to eq('http://schema.org/Organization')
+        expect(publisher.properties['name'].first).to eq('Random Publishing, Inc.')
+      end
+
+      it 'sets subjects' do
+        subjects = @item.properties['about']
+        expect(subjects.count).to eq(3)
+        subject = subjects.first
+        expect(subject.type).to eq('http://schema.org/Thing')
+        expect(subject.properties['name'].first).to eq('Biology')
+      end
+
+      it 'sets depositor as accountablePerson' do
+        depositor = @item.properties['accountablePerson'].first
+        expect(depositor.type).to eq('http://schema.org/Person')
+        expect(depositor.properties['name'].first).to eq('bob')
+      end
+    end
+
+    describe 'resource type-specific itemtypes' do
+      context 'when resource_type is Audio' do
+        it 'sets itemtype to AudioObject' do
+          generic_file.resource_type = ['Audio']
+          render template: 'generic_files/show.html.erb', layout: 'layouts/sufia-one-column'
+          @item = Mida::Document.new(rendered).items.first
+          expect(@item.type).to eq('http://schema.org/AudioObject')
+        end
+      end
+      context 'when resource_type is Conference Proceeding' do
+        it 'sets itemtype to ScholarlyArticle' do
+          generic_file.resource_type = ['Conference Proceeding']
+          render template: 'generic_files/show.html.erb', layout: 'layouts/sufia-one-column'
+          @item = Mida::Document.new(rendered).items.first
+          expect(@item.type).to eq('http://schema.org/ScholarlyArticle')
+        end
+      end
+    end
+  end
+
+  describe 'google scholar' do
+    # NOTE: before(:all) will not work in this context
+    before(:each) do
+      render template: 'generic_files/show.html.erb', layout: 'layouts/sufia-one-column'
+    end
+
+    it 'appears in meta tags' do
+      gscholar_meta_tags = Nokogiri::HTML(rendered).xpath("//meta[contains(@name, 'citation_')]")
+      expect(gscholar_meta_tags.count).to eq(5)
+    end
+
+    it 'displays title' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='citation_title']")
+      expect(tag.attribute('content').value).to eq('My Title')
+    end
+
+    it 'displays authors' do
+      tags = Nokogiri::HTML(rendered).xpath("//meta[@name='citation_author']")
+      expect(tags.first.attribute('content').value).to eq('Doe, John')
+      expect(tags.last.attribute('content').value).to eq('Doe, Jane')
+    end
+
+    it 'displays publication date' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='citation_publication_date']")
+      expect(tag.attribute('content').value).to eq('1984-01-02')
+    end
+
+    it 'displays download URL' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='citation_pdf_url']")
+      expect(tag.attribute('content').value).to eq('http://test.host/downloads/123')
+    end
+  end
 
+  describe 'twitter cards' do
+    # NOTE: before(:all) will not work in this context
+    before(:each) do
+      render template: 'generic_files/show.html.erb', layout: 'layouts/sufia-one-column'
+    end
+
+    it 'appears in meta tags' do
+      twitter_meta_tags = Nokogiri::HTML(rendered).xpath("//meta[contains(@name, 'twitter:') or contains(@property, 'og:')]")
+      expect(twitter_meta_tags.count).to eq(13)
+    end
+
+    it 'displays twitter:card' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='twitter:card']")
+      expect(tag.attribute('content').value).to eq('product')
+    end
+
+    it 'displays twitter:site' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='twitter:site']")
+      expect(tag.attribute('content').value).to eq('@HydraSphere')
+    end
+
+    it 'displays twitter:creator' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='twitter:creator']")
+      expect(tag.attribute('content').value).to eq('@bot4lib')
+    end
+
+    it 'displays og:site_name' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@property='og:site_name']")
+      expect(tag.attribute('content').value).to eq('Sufia')
+    end
+
+    it 'displays og:type' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@property='og:type']")
+      expect(tag.attribute('content').value).to eq('object')
+    end
+
+    it 'displays og:title' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@property='og:title']")
+      expect(tag.attribute('content').value).to eq('My Title')
+    end
+
+    it 'displays og:description' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@property='og:description']")
+      expect(tag.attribute('content').value).to eq('Lorem ipsum lorem ipsum.')
+    end
+
+    it 'displays og:image' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@property='og:image']")
+      expect(tag.attribute('content').value).to eq('http://test.host/downloads/123?datastream_id=thumbnail')
+    end
+
+    it 'displays og:url' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@property='og:url']")
+      expect(tag.attribute('content').value).to eq('http://test.host/files/123')
+    end
+
+    it 'displays twitter:data1' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='twitter:data1']")
+      expect(tag.attribute('content').value).to eq('bacon, sausage, eggs')
+    end
+
+    it 'displays twitter:label1' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='twitter:label1']")
+      expect(tag.attribute('content').value).to eq('Keywords')
+    end
+
+    it 'displays twitter:data2' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='twitter:data2']")
+      expect(tag.attribute('content').value).to eq('http://example.org/rights/1')
+    end
+
+    it 'displays twitter:label2' do
+      tag = Nokogiri::HTML(rendered).xpath("//meta[@name='twitter:label2']")
+      expect(tag.attribute('content').value).to eq('Rights')
+    end
+  end
+
+  describe 'analytics' do
     context 'when enabled' do
       before do
         Sufia.config.analytics = true
@@ -44,7 +271,6 @@ describe 'generic_files/show.html.erb' do
   end
 
   describe 'featured' do
-
     context "public file" do
       before do
         allow(generic_file).to receive(:public?).and_return(true)
@@ -69,4 +295,43 @@ describe 'generic_files/show.html.erb' do
       end
     end
   end
+
+  describe 'collections list' do
+    context "when the file is not featured in any collections" do
+
+      it "should display the empty message" do
+        render
+        expect(rendered).to have_text(t('sufia.file.collections_list.empty'))
+      end
+    end
+
+    context "when the file is featured in collections" do
+      let(:collection1) {
+        stub_model(Collection,
+          title: 'collection1',
+          noid: '456')
+      }
+ 
+      before do
+        allow(generic_file).to receive(:collections).and_return([collection1])
+      end
+
+      it "should display the header and titles of collections it belongs to" do
+        render
+        expect(rendered).to have_text(t('sufia.file.collections_list.heading'))
+        expect(rendered).to have_text('collection1')
+      end
+    end
+  end
+
+  describe 'visibility' do
+    let(:expected) do
+      '<span class="label label-danger" title="'+t('sufia.visibility.private')+'">'+t('sufia.visibility.private')+'</span></a>' 
+    end
+    it "should display the visibility badge" do
+      render
+      expect(rendered).to include(expected)
+    end
+  end
+
 end