worthwhile 0.0.1.alpha → 0.0.1

data/spec/support/controllers/engine_helpers.rb

@@ -0,0 +1,7 @@
+module Controllers
+  module EngineHelpers
+    def main_app
+      Rails.application.class.routes.url_helpers
+    end
+  end
+end

data/spec/support/curation_concern/factory_helpers.rb

@@ -0,0 +1,14 @@
+module CurationConcern
+  module FactoryHelpers
+    module_function
+    def load_factories_for(context, klass)
+      context.instance_exec(klass) do |curation_concern_class|
+        let(:curation_concern_type_underscore) { curation_concern_class.name.underscore }
+        let(:default_work_factory_name) { curation_concern_type_underscore }
+        let(:default_work_factory_name_with_files) { "#{default_work_factory_name}_with_files".to_sym }
+        let(:private_work_factory_name) { "private_#{curation_concern_type_underscore}".to_sym }
+        let(:public_work_factory_name) { "public_#{curation_concern_type_underscore}".to_sym }
+      end
+    end
+  end
+end

data/spec/support/features.rb

@@ -0,0 +1,19 @@
+require File.expand_path('../features/session_helpers', __FILE__)
+require File.expand_path('../features/fixture_file_upload', __FILE__)
+
+RSpec.configure do |config|
+  config.include Warden::Test::Helpers, type: :feature
+  config.include Features::SessionHelpers, type: :feature
+  config.include Features::FixtureFileUpload
+  
+  config.before(:each, type: :feature) do
+    Warden.test_mode!
+    @old_resque_inline_value = Resque.inline
+    Resque.inline = true
+  end
+
+  config.after(:each, type: :feature) do
+    Warden.test_reset!
+    Resque.inline = @old_resque_inline_value
+  end
+end

data/spec/support/features/fixture_file_upload.rb

@@ -0,0 +1,14 @@
+module Features
+  module FixtureFileUpload
+    # Can't just call this fixture_file_upload due to name conflicts.
+    def worthwhile_fixture_file_upload(path, content_type = 'text/plain', binary = false)
+      Rack::Test::UploadedFile.new(fixture_file_path(path), content_type, binary)
+    end
+    def fixture_file(path)
+      File.open(fixture_file_path(path))
+    end
+    def fixture_file_path(path)      
+      File.join(Worthwhile::Engine.root.to_s, 'spec/fixtures', path)
+    end
+  end
+end

data/spec/support/features/session_helpers.rb

@@ -0,0 +1,41 @@
+module Features
+  module SessionHelpers
+    # Regular login
+    def login_as(user)
+      user.reload # because the user isn't re-queried via Warden
+      super(user, scope: :user, run_callbacks: false)
+    end
+    # Regular logout
+    def logout(user=:user)
+      super(user)
+    end
+
+    # Poltergeist-friendly sign-up
+    # Use this in feature tests
+    def sign_up_with(email, password)
+      Capybara.exact = true
+      visit new_user_registration_path
+      fill_in 'Email', with: email
+      fill_in 'Password', with: password
+      fill_in 'Password confirmation', with: password
+      click_button 'Sign up'
+    end
+
+    # Poltergeist-friendly sign-in
+    # Use this in feature tests
+    def sign_in(who = :user)
+      user = if who.instance_of?(User)
+               who
+             else
+               FactoryGirl.build(:user).tap do |u|
+                 u.save!
+               end
+             end
+      visit new_user_session_path
+      fill_in 'Email', with: user.email
+      fill_in 'Password', with: user.password
+      click_button 'Log in'
+      expect(page).to_not have_text 'Invalid email or password.'
+    end
+  end
+end

data/spec/support/shared/shared_examples_has_dc_metadata.rb

@@ -0,0 +1,17 @@
+shared_examples 'has_dc_metadata' do
+
+  # Single-valued fields
+  it { should have_unique_field(:created) }
+  it { should have_unique_field(:date_uploaded) }
+  it { should have_unique_field(:date_modified) }
+
+  # Multivalued fields
+  it { should have_multivalue_field(:content_format) }
+  it { should have_multivalue_field(:description) }
+  it { should have_multivalue_field(:identifier) }
+  it { should have_multivalue_field(:publisher) }
+  it { should have_multivalue_field(:source) }
+  it { should have_multivalue_field(:language) }
+  it { should have_multivalue_field(:subject) }
+
+end

data/spec/support/shared/shared_examples_is_a_curation_concern_model.rb

@@ -0,0 +1,40 @@
+shared_examples 'is_a_curation_concern_model' do
+  CurationConcern::FactoryHelpers.load_factories_for(self, described_class)
+
+  it 'is registered for classification' do
+    expect(Worthwhile.configuration.registered_curation_concern_types).to include(described_class.name)
+  end
+
+  context 'behavior' do
+    subject { FactoryGirl.build(default_work_factory_name) }
+    it 'can be cast as an RDF object' do
+      expect(subject.as_rdf_object).to be_kind_of RDF::URI
+    end
+  end
+
+  context 'collectibility' do
+    subject { FactoryGirl.build(default_work_factory_name) }
+    let(:collection) { double }
+    it '#can_be_member_of_collection?' do
+      expect(subject.can_be_member_of_collection?(collection)).to eq(true)
+    end
+  end
+
+  context 'its test support factories', factory_verification: true do
+    it {
+      expect {
+        FactoryGirl.create(default_work_factory_name)
+      }.to_not raise_error
+    }
+    it {
+      expect {
+        FactoryGirl.create(private_work_factory_name)
+      }.to_not raise_error
+    }
+    it {
+      expect {
+        FactoryGirl.create(public_work_factory_name)
+      }.to_not raise_error
+    }
+  end
+end

data/spec/support/shared/shared_examples_is_embargoable.rb

@@ -0,0 +1,8 @@
+shared_examples 'is_embargoable' do
+
+  it "has an embargo_release_date attribute" do
+    expect(subject).to respond_to(:embargo_release_date)
+    expect(subject).to respond_to(:embargo_release_date=)
+  end
+
+end

data/spec/support/shared/shared_examples_with_access_rights.rb

@@ -0,0 +1,63 @@
+shared_examples 'with_access_rights' do
+
+  def prepare_subject_for_access_rights_visibility_test!
+    # I am doing this because the actual persistence of the objects requires
+    # so much more and I don't know for certain if it has happened.
+    allow(subject).to receive(:persisted?).and_return(true)
+  end
+
+  it "has an under_embargo?" do
+    expect {
+      subject.under_embargo?
+    }.to_not raise_error
+  end
+
+  it "has a visibility attribute" do
+    expect(subject).to respond_to(:visibility)
+    expect(subject).to respond_to(:visibility=)
+  end
+
+  describe 'open access' do
+    it "has an open_access?" do
+      expect {
+        subject.open_access?
+      }.to_not raise_error
+    end
+
+    it 'sets visibility' do
+      prepare_subject_for_access_rights_visibility_test!
+      subject.visibility = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+      expect(subject).to be_open_access
+    end
+  end
+
+  describe 'authenticated access' do
+    it 'sets visibility' do
+      prepare_subject_for_access_rights_visibility_test!
+      subject.visibility = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
+      expect(subject).to be_authenticated_only_access
+    end
+
+    it "has an authenticated_only_access?" do
+      expect {
+        subject.authenticated_only_access?
+      }.to_not raise_error
+    end
+  end
+
+  describe 'private access' do
+    it 'sets visibility' do
+      prepare_subject_for_access_rights_visibility_test!
+      subject.visibility = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
+      expect(subject).to be_private_access
+    end
+
+
+    it "has an private_access?" do
+      expect {
+        subject.private_access?
+      }.to_not raise_error
+    end
+  end
+
+end

data/spec/test_app_templates/lib/generators/test_app_generator.rb

@@ -3,39 +3,18 @@ require 'rails/generators'
 class TestAppGenerator < Rails::Generators::Base
   source_root "../../spec/test_app_templates"
 
-  def add_gems
-    gem 'sufia-models', github: 'projecthydra/sufia', branch: 'dce_dev'
-    # gem 'blacklight', ">= 5.4.0.rc1", "<6"
-    # gem "blacklight-gallery", :github => 'projectblacklight/blacklight-gallery'
-    # gem 'sir-trevor-rails', :github => 'sul-dlss/sir-trevor-rails'
-    # gem 'openseadragon', :github => 'sul-dlss/openseadragon-rails'
-    # gem "jettywrapper"
-    Bundler.with_clean_env do
-      run "bundle install"
-    end
-  end
-
-  def run_blacklight_generator
-    say_status("warning", "GENERATING BL", :yellow)
+  # def add_gems
+  #   gem 'sufia-models', github: 'projecthydra/sufia'
+  #   Bundler.with_clean_env do
+  #     run "bundle install"
+  #   end
+  # end
 
-    generate 'blacklight:install', '--devise'
-    say_status("warning", "GENERATING HYDRA-HEAD", :yellow)
-    generate "hydra:head -f"
-    say_status("warning", "GENERATING SUFIA", :yellow)
-    generate "sufia:models:install#{options[:force] ? ' -f' : ''}"
+  def run_generator
+    generate 'worthwhile:install'
   end
 
   def run_migrations
-    # rake "spotlight:install:migrations"
     rake "db:migrate"
   end
-
-  # def add_spotlight_routes_and_assets
-  #   generate 'spotlight:install'
-  # end
-
-  # def add_mailer_defaults
-  #   mail_config = "    config.action_mailer.default_url_options = { host: \"localhost:3000\", from: \"noreply@example.com\" }\n"
-  #   insert_into_file 'config/application.rb', mail_config, after: "< Rails::Application\n"
-  # end
 end

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

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe 'curation_concern/base/_attributes.html.erb' do
+
+  let(:creator)     { 'Bilbo' }
+  let(:contributor) { 'Frodo' }
+  let(:subject)     { 'history' }
+
+  let(:curation_concern) { double(creator: [creator],
+                                  contributor: [contributor],
+                                  subject: [subject]) }
+
+  before do
+    view.stub(:dom_class) { '' }
+    view.stub(:permission_badge_for) { '' }
+
+    render partial: 'attributes', locals: { curation_concern: curation_concern }
+  end 
+
+  it 'has links to search for other objects with the same metadata' do
+    expect(rendered).to have_link(creator, href: catalog_index_path(search_field: 'creator', q: creator))
+    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
+end

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

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe 'curation_concern/base/show.html.erb' do
+
+  let!(:curation_concern) { FactoryGirl.create(:private_generic_work) }
+
+  context "for editors" do
+    it 'has links to edit and add to collections' do
+      allow(view).to receive(:can?).and_return(true)
+      render file: 'curation_concern/base/show', locals: { curation_concern: curation_concern }
+      expect(view.content_for(:second_row)).to have_link("Edit This Generic Work", href: edit_polymorphic_path([:curation_concern, curation_concern]))
+      expect(view.content_for(:second_row)).to have_link("Add to a Collection", href: add_member_form_collections_path(collectible_id:curation_concern.pid))
+    end
+  end
+  context "for non-editors" do
+    it 'does not have links to edit' do
+      render file: 'curation_concern/base/show', locals: { curation_concern: curation_concern }
+      expect(view.content_for(:second_row)).not_to have_content("Edit this Generic Work")
+    end
+    it 'has link to add to a collection' do
+      render file: 'curation_concern/base/show', locals: { curation_concern: curation_concern }
+      expect(view.content_for(:second_row)).to have_link("Add to a Collection", href: add_member_form_collections_path(collectible_id:curation_concern.pid))
+    end
+  end
+end

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

@@ -0,0 +1,42 @@
+require 'spec_helper'
+
+describe 'shared/_add_content.html.erb' do
+
+  context "for people who can create Works and Collections" do
+    it 'has links to create works and collections' do
+      allow(view).to receive(:current_user).and_return(FactoryGirl.create(:admin))
+      allow(view).to receive(:can?).and_return(true)
+      render partial: 'shared/add_content'
+      Worthwhile.configuration.curation_concerns.each do |curation_concern_type|
+        expect(rendered).to have_link("New #{curation_concern_type.human_readable_type}", href: new_polymorphic_path([:curation_concern, curation_concern_type]))
+      end
+      expect(rendered).to have_link("Add a Collection", href: collections.new_collection_path)
+    end
+  end
+  context "for people who can only create Collections" do
+    it 'has links to add collections but not to add works' do
+      allow(view).to receive(:current_user).and_return(FactoryGirl.create(:admin))
+      allow(view).to receive(:can?).and_return(true)
+      allow(view).to receive(:can_ever_create_works?).and_return(false)
+      render partial: 'shared/add_content'
+      Worthwhile.configuration.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_concern, curation_concern_type]))
+      end
+      expect(rendered).to have_link("Add a Collection", href: collections.new_collection_path)
+    end
+  end
+
+  context "for people who cannot create anything" do
+    it 'does not have links to add works or collections' do
+      allow(view).to receive(:current_user).and_return(FactoryGirl.create(:user))
+      allow(view).to receive(:can?).and_return(false)
+      render partial: 'shared/add_content'
+      expect(rendered).not_to have_text("Add")
+      expect(rendered).not_to have_text("Admin")
+      Worthwhile.configuration.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_concern, curation_concern_type]))
+      end
+      expect(rendered).not_to have_link("Add a Collection", href: collections.new_collection_path)
+    end
+  end
+end

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

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe 'shared/_my_actions.html.erb' do
+  include Worthwhile::SearchPathsHelper
+  context "for admins" do
+    it 'has links to edit and add to collections' do
+      allow(view).to receive(:current_user).and_return(FactoryGirl.create(:user))
+      allow(view).to receive(:can?).and_return(true)
+      render partial: 'shared/my_actions'
+      expect(rendered).to have_link("My Works", href: search_path_for_my_works)
+      expect(rendered).to have_link("My Collections", href: search_path_for_my_collections)
+    end
+  end
+  context "for non-admins" do
+    it 'does not have links to edit' do
+      allow(view).to receive(:current_user).and_return(FactoryGirl.create(:user))
+      allow(view).to receive(:can?).and_return(false)
+      render partial: 'shared/my_actions'
+      expect(rendered).not_to have_text("My Works")
+      expect(rendered).not_to have_text("My Collections")
+    end
+  end
+end

data/spec/workers/visibility_copy_worker_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+
+describe VisibilityCopyWorker do
+
+  describe "an open access work" do
+    let(:work) { FactoryGirl.create(:work_with_files) }
+    subject { VisibilityCopyWorker.new(work.id) }
+
+    it "should have no content at the outset" do
+      expect(work.generic_files.first.visibility).to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
+    end
+
+    it "should copy visibility to its contained files" do
+      work.visibility = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+      work.save
+      subject.run
+      work.reload.generic_files.each do |file|
+        expect(file.visibility).to eq 'open'
+      end
+    end
+  end
+
+  describe "an embargoed work" do
+    let(:work) { FactoryGirl.create(:embargoed_work_with_files) }
+    subject { VisibilityCopyWorker.new(work.id) }
+
+    before do
+      expect(work.visibility).to eq 'restricted'
+      expect(work).to be_under_embargo
+      expect(work.generic_files.first).to_not be_under_embargo
+    end
+
+    context "when run" do
+      before do
+        subject.run
+        work.reload
+      end
+      let(:file) { work.generic_files.first }
+
+      it "should copy visibility to its contained files" do
+        expect(file).to be_under_embargo
+      end
+    end
+  end
+
+  describe "an leased work" do
+    let(:work) { FactoryGirl.create(:leased_work_with_files) }
+    subject { VisibilityCopyWorker.new(work.id) }
+
+    before do
+      expect(work.visibility).to eq 'open'
+      expect(work).to be_active_lease
+      expect(work.generic_files.first).to_not be_active_lease
+    end
+
+    context "when run" do
+      before do
+        subject.run
+        work.reload
+      end
+      let(:file) { work.generic_files.first }
+
+      it "should copy visibility to its contained files" do
+        expect(file).to be_active_lease
+      end
+    end
+  end
+end