worthwhile 0.0.1.alpha → 0.0.1

data/spec/controllers/leases_controller_spec.rb

@@ -0,0 +1,95 @@
+require 'spec_helper'
+
+describe LeasesController do
+  let(:user) { FactoryGirl.create(:user) }
+  let(:a_work) { FactoryGirl.create(:generic_work, user: user) }
+  let(:not_my_work) { FactoryGirl.create(:generic_work) }
+
+  before { sign_in user }
+
+  describe "#index" do
+    context "when I am NOT a repository manager" do
+      it "should redirect" do
+        get :index
+        expect(response).to redirect_to root_path
+      end
+    end
+    context "when I am a repository manager" do
+      before { allow_any_instance_of(User).to receive(:groups).and_return(['admin']) }
+      it "should show me the page" do
+        get :index
+        expect(response).to be_success
+      end
+    end
+  end
+
+  describe "#edit" do
+    context "when I do not have edit permissions for the object" do
+      it "should redirect" do
+        get :edit, id: not_my_work
+        expect(response.status).to eq 401
+        expect(response).to render_template :unauthorized
+      end
+    end
+    context "when I have permission to edit the object" do
+      it "should show me the page" do
+        get :edit, id: a_work
+        expect(response).to be_success
+      end
+    end
+  end
+
+
+
+  describe "#destroy" do
+    context "when I do not have edit permissions for the object" do
+      it "should deny access" do
+        get :destroy, id: not_my_work
+        expect(response.status).to eq 401
+        expect(response).to render_template :unauthorized
+      end
+    end
+
+    context "when I have permission to edit the object" do
+      before do
+        expect(ActiveFedora::Base).to receive(:find).with(a_work.pid).and_return(a_work)
+        a_work.visibility_during_lease = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+        a_work.visibility_after_lease = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
+        a_work.lease_expiration_date = release_date.to_s
+        expect(Sufia.queue).to receive(:push).with(an_instance_of(VisibilityCopyWorker))
+        get :destroy, id: a_work
+      end
+
+      context "with an active lease" do
+        let(:release_date) { Date.today+2 }
+        it "should deactivate the lease without updating visibility and redirect" do
+          expect(a_work.visibility).to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+          expect(response).to redirect_to edit_lease_path(a_work)
+        end
+      end
+
+      context "with an expired lease" do
+        let(:release_date) { Date.today-2 }
+
+        it "should deactivate the lease, update the visibility and redirect" do
+          expect(a_work.visibility).to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
+          expect(response).to redirect_to edit_lease_path(a_work)
+        end
+      end
+    end
+  end
+
+  describe "#update" do
+    context "when I have permission to edit the object" do
+      before do
+        expect(ActiveFedora::Base).to receive(:find).with(a_work.pid).and_return(a_work)
+      end
+      it "should deactivate lease and redirect" do
+        expect(a_work).to receive(:deactivate_lease!)
+        expect(a_work).to receive(:save)
+        patch :update, batch_document_ids: [a_work.pid]
+        expect(response).to redirect_to leases_path
+      end
+    end
+  end
+end

data/spec/controllers/worthwhile/classify_concerns_controller_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Worthwhile::ClassifyConcernsController do
+  routes { Worthwhile::Engine.routes }
+  let(:user) { FactoryGirl.create(:user) }
+
+  describe '#new' do
+    it 'requires authentication' do
+      get :new
+      expect(response).to redirect_to(main_app.user_session_path)
+    end
+    it 'renders when signed in' do
+      sign_in(user)
+      get :new
+      expect(response).to be_successful
+    end
+  end
+
+  describe '#create' do
+    it 'redirect to login page if user is not logged in' do
+      post :create, classify: { curation_concern_type: 'GenericWork' }
+      expect(response).to redirect_to(main_app.user_session_path)
+    end
+
+    it 'requires authentication' do
+      sign_in(user)
+      # Had to stub the actual handling of curation_concern paths since those paths live outside the engine while the path to this controller lives inside the engine.
+      new_curation_concern_generic_work_path = "/stub/path"
+      subject.should_receive(:new_polymorphic_path).with([:curation_concern, GenericWork]).and_return(new_curation_concern_generic_work_path)
+      post :create, classify_concern: { curation_concern_type: 'GenericWork' }
+      expect(response).to redirect_to(new_curation_concern_generic_work_path)
+    end
+
+  end
+end

data/spec/factories/collections_factory.rb

@@ -0,0 +1,21 @@
+FactoryGirl.define do
+  factory :collection do
+    ignore do
+      user {FactoryGirl.create(:user)}
+    end
+    sequence(:title) {|n| "Title #{n}"}
+    before(:create) { |work, evaluator|
+      work.apply_depositor_metadata(evaluator.user.user_key)
+    }
+
+    factory :public_collection do
+      visibility Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+    end
+
+    factory :private_collection do
+      visibility Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
+    end
+  end
+
+end
+

data/spec/factories/create_curation_concern.rb

@@ -0,0 +1,3 @@
+def FactoryGirl.create_curation_concern(factory_name, user, override_attributes = {})
+  FactoryGirl.create(factory_name, override_attributes.merge(user: user))
+end

data/spec/factories/generic_files.rb

@@ -0,0 +1,18 @@
+FactoryGirl.define do
+  factory :generic_file, class: Worthwhile::GenericFile do
+    factory :file_with_work do
+      ignore do
+        user { FactoryGirl.create(:user) }
+        content nil
+      end
+      batch { FactoryGirl.create(:generic_work, user: user) }
+
+      after(:build) do |file, evaluator|
+        file.apply_depositor_metadata(evaluator.user.user_key)
+        if evaluator.content
+          file.add_file(evaluator.content.read, 'content', evaluator.content.original_filename)
+        end
+      end
+    end
+  end
+end

data/spec/factories/generic_works.rb

@@ -0,0 +1,37 @@
+FactoryGirl.define do
+  factory :work, aliases: [:generic_work, :private_generic_work], class: GenericWork do
+    ignore do
+      user { FactoryGirl.create(:user) }
+    end
+
+    title ["Test title"]
+    visibility Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
+
+    before(:create) do |work, evaluator|
+      work.apply_depositor_metadata(evaluator.user.user_key)
+    end
+
+
+    factory :public_generic_work do
+      visibility Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+    end
+
+    factory :work_with_files do
+      after(:build) { |work, _| 2.times { work.generic_files << FactoryGirl.build(:generic_file) }}
+    end
+
+    factory :embargoed_work do
+      after(:build) { |work, _| work.apply_embargo(Date.tomorrow.to_s, Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE, Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC) }
+    end
+
+    factory :embargoed_work_with_files do
+      after(:build) { |work, _| 2.times { work.generic_files << FactoryGirl.build(:generic_file) }}
+      after(:build) { |work, _| work.apply_embargo(Date.tomorrow.to_s, Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE, Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC) }
+    end
+
+    factory :leased_work_with_files do
+      after(:build) { |work, _| 2.times { work.generic_files << FactoryGirl.build(:generic_file) }}
+      after(:build) { |work, _| work.apply_lease(Date.tomorrow.to_s, Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC, Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE) }
+    end
+  end
+end

data/spec/factories/linked_resources_factory.rb

@@ -0,0 +1,18 @@
+FactoryGirl.define do
+  factory :linked_resource, class: Worthwhile::LinkedResource  do
+    ignore do
+      user {FactoryGirl.create(:user)}
+    end
+    visibility Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
+    url 'http://www.youtube.com/watch?v=oHg5SJYRHA0'
+    batch { FactoryGirl.create(:generic_work, user: user) }
+    before(:create) { |file, evaluator|
+      file.apply_depositor_metadata(evaluator.user.user_key)
+    }
+
+    factory :private_linked_resource do
+      visibility Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
+    end
+
+  end
+end

data/spec/factories/users.rb

@@ -0,0 +1,15 @@
+FactoryGirl.define do
+  factory :user do
+    sequence(:email) {|n| "email-#{srand}@test.com" }
+    password 'a password'
+    password_confirmation 'a password'
+    factory :admin do
+      after(:build) do |user|
+        def user.groups
+          ["admin"]
+        end
+      end
+    end
+  end
+
+end

data/spec/features/add_external_link_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe "Adding a work with external links" do
+  let(:user) { FactoryGirl.create(:user) }
+  before do
+    sign_in user
+    visit '/'
+  end
+
+  it "adds external links" do
+    click_link 'New Generic Work'
+    fill_in 'External link', with: 'http://dp.la'
+    fill_in 'Title', with: 'DPLA link'
+    check 'I have read and accept the contributor license agreement'
+    click_button 'Create Generic work'
+    expect(page).to have_link 'http://dp.la'
+
+    click_link 'Add an External Link'
+    fill_in 'External link', with: 'http://loc.gov'
+    click_button 'Add External Link to Generic Work'
+    expect(page).to have_link 'http://dp.la'
+    expect(page).to have_link 'http://loc.gov'
+  end
+
+end

data/spec/features/add_file_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe "Add an attached file" do
+  let(:user) { FactoryGirl.create(:user) }
+  let!(:work) { FactoryGirl.create(:work, user: user) }
+
+  before do
+    sign_in user
+
+    # stub out characterization. Travis doesn't have fits installed, and it's not relevant to the test.
+    s2 = double('resque message')
+    expect(CharacterizeJob).to receive(:new).and_return(s2)
+    expect(Sufia.queue).to receive(:push).with(s2).once
+  end
+
+  it "should update the file" do
+    visit "/concern/generic_works/#{work.id}"
+    click_link 'Attach a File'
+
+    attach_file("Upload a file", fixture_file_path('files/image.png'))
+    click_button "Attach to Generic Work"
+
+    within '.related_files' do
+      expect(page).to have_link "image.png"
+    end
+  end
+end
+

data/spec/features/collection_spec.rb

@@ -0,0 +1,238 @@
+require 'spec_helper'
+include Worthwhile::SearchPathsHelper
+
+describe 'collection' do
+  def create_collection(title, description)
+    click_link 'Add a Collection'
+    fill_in('Title', with: title)
+    fill_in('collection_description', with: description)
+    click_button("Create Collection")
+    expect(page).to have_content 'Items in this Collection'
+    expect(page).to have_content title
+    expect(page).to have_content description
+  end
+
+  let(:title1) {"Test Collection 1"}
+  let(:description1) {"Description for collection 1 we are testing."}
+  let(:title2) {"Test Collection 2"}
+  let(:description2) {"Description for collection 2 we are testing."}
+
+  let(:user) { FactoryGirl.create(:user, email: 'user1@example.com') }
+  let(:user_key) { user.user_key }
+
+  before(:all) do
+    @old_resque_inline_value = Resque.inline
+    Resque.inline = true
+
+    @gws = []
+    (0..12).each do |x|
+      @gws[x] =  GenericWork.new.tap do |f|
+        f.title = ["title #{x}"]
+        f.apply_depositor_metadata('user1@example.com')
+        f.save!
+      end
+    end
+    @gw1 = @gws[0]
+    @gw2 = @gws[1]
+  end
+
+  after(:all) do
+    Resque.inline = @old_resque_inline_value
+    Batch.destroy_all
+    GenericWork.destroy_all
+    Collection.destroy_all
+  end
+
+  describe 'create collection' do
+    before do
+      sign_in user
+      visit search_path_for_my_collections
+    end
+    it "should create a collection" do
+      title = "Genealogies of the American West"
+      description = "All about Genealogies of the American West"
+      click_link 'Add a Collection'
+      fill_in('Title', with: title)
+      fill_in('collection_description', with: description)
+      click_button("Create Collection")
+      expect(page).to have_content 'Items in this Collection'
+      expect(page).to have_content title
+      expect(page).to have_content description
+    end
+    it "should create collection from the dashboard and include works" do
+      skip "batch collection operations (add/remove)"
+      create_collection(title2, description2)
+
+      visit search_path_for_my_works
+      first('input#check_all').click
+      click_button "Add to Collection" # opens the modal
+      # since there is only one collection, it's not necessary to choose a radio button
+      click_button "Update Collection"
+      expect(page).to have_content "Items in this Collection"
+      expect(page).to have_selector "ol.catalog li:nth-child(9)" # at least 9 works in this collection
+    end
+  end
+
+  describe 'delete collection' do
+    before (:each) do
+      @collection = Collection.new title:'collection title'
+      @collection.description = 'collection description'
+      @collection.apply_depositor_metadata(user_key)
+      @collection.save
+      sign_in user
+      visit main_app.catalog_index_path(:'f[generic_type_sim][]' => 'Collection', works: 'mine')
+    end
+
+    it "should delete a collection" do
+      page.should have_content(@collection.title)
+      within("#document_#{@collection.noid}") do
+        first(".itemtrash").click
+      end
+      page.should_not have_content(@collection.title)
+      page.should have_content("Collection was successfully deleted.")
+    end
+  end
+
+  describe 'show collection' do
+    before do
+      @collection = Collection.new title: 'collection title'
+      @collection.description = 'collection description'
+      @collection.apply_depositor_metadata(user_key)
+      @collection.members = [@gw1,@gw2]
+      @collection.save
+      sign_in user
+      visit search_path_for_my_collections
+    end
+
+    it "should show a collection with a listing of Descriptive Metadata and catalog-style search results" do
+      page.should have_content(@collection.title)
+      within('#document_'+@collection.noid) do
+        click_link("collection title")
+      end
+      page.should have_content(@collection.title)
+      page.should have_content(@collection.description)
+      # Should have search results / contents listing
+      page.should have_content(@gw1.title.first)
+      page.should have_content(@gw2.title.first)
+      page.should_not have_css(".pager")
+
+      #click_link "Gallery"
+      #expect(page).to have_content(@gw1.title)
+      #expect(page).to have_content(@gw2.title)
+    end
+
+    it "should hide collection descriptive metadata when searching a collection" do
+      page.should have_content(@collection.title)
+      within("#document_#{@collection.noid}") do
+        click_link("collection title")
+      end
+      page.should have_content(@collection.title)
+      page.should have_content(@collection.description)
+      page.should have_content(@gw1.title.first)
+      page.should have_content(@gw2.title.first)
+      fill_in('collection_search', with: @gw1.title.first)
+      click_button('collection_submit')
+      # Should not have Collection Descriptive metadata table
+      page.should_not have_content("Descriptions")
+      page.should have_content(@collection.title)
+      page.should have_content(@collection.description)
+      # Should have search results / contents listing
+      page.should have_content(@gw1.title.first)
+      page.should_not have_content(@gw2.title.first)
+      # Should not have Dashboard content in contents listing
+      page.should_not have_content("Visibility")
+    end
+  end
+
+  describe 'edit collection' do
+    before (:each) do
+      @collection = Collection.new(title: 'Awesome Title')
+      @collection.description = 'collection description'
+      @collection.apply_depositor_metadata(user_key)
+      @collection.members = [@gw1, @gw2]
+      @collection.save
+      sign_in user
+      visit search_path_for_my_collections
+    end
+
+    it "should edit and update collection metadata" do
+      page.should have_content(@collection.title)
+      within("#document_#{@collection.noid}") do
+        click_link('Edit Collection')
+      end
+      page.should have_field('collection_title', with: @collection.title)
+      page.should have_field('collection_description', with: @collection.description)
+      new_title = "Altered Title"
+      new_description = "Completely new Description text."
+      creators = ["Dorje Trollo", "Vajrayogini"]
+      fill_in('Title', with: new_title)
+      fill_in('collection_description', with: new_description)
+      fill_in('Creator', with: creators.first)
+      #within('.form-actions') do
+        click_button('Update Collection')
+      #end
+      page.should_not have_content(@collection.title)
+      page.should_not have_content(@collection.description)
+      page.should have_content(new_title)
+      page.should have_content(new_description)
+      page.should have_content(creators.first)
+    end
+
+    it "should remove a work from a collection" do
+      skip "BUG removing works from a collection"
+      page.should have_content(@collection.title)
+      within("#document_#{@collection.noid}") do
+        click_link('Edit Collection')
+      end
+      page.should have_field('collection_title', with: @collection.title)
+      page.should have_field('collection_description', with: @collection.description)
+      page.should have_content(@gw1.title)
+      page.should have_content(@gw2.title)
+      within("#document_#{@gw1.noid}") do
+        click_button('Remove From Collection')
+      end
+      page.should have_content(@collection.title)
+      page.should have_content(@collection.description)
+      page.should_not have_content(@gw1.title)
+      page.should have_content(@gw2.title)
+    end
+
+    it "should remove all works from a collection" do
+      skip "batch collection operations (add/remove)"
+      page.should have_content(@collection.title)
+      within('#document_'+@collection.noid) do
+        click_link('Edit Collection')
+      end
+      page.should have_field('collection_title', with: @collection.title)
+      page.should have_field('collection_description', with: @collection.description)
+      page.should have_content(@gw1.title)
+      page.should have_content(@gw2.title)
+      first('input#check_all').click
+      click_button('Remove From Collection')
+      page.should have_content(@collection.title)
+      page.should have_content(@collection.description)
+      page.should_not have_content(@gw1.title)
+      page.should_not have_content(@gw2.title)
+    end
+  end
+
+  describe 'show pages of a collection' do
+    before (:each) do
+      @collection = Collection.new title:'collection title'
+      @collection.description = 'collection description'
+      @collection.apply_depositor_metadata(user_key)
+      @collection.members = @gws
+      @collection.save!
+      sign_in user
+      visit search_path_for_my_collections
+    end
+
+    it "should show a collection with a listing of Descriptive Metadata and catalog-style search results" do
+      page.should have_content(@collection.title)
+      within('#document_'+@collection.noid) do
+        click_link("collection title")
+      end
+      page.should have_css(".pager")
+    end
+  end
+end