curate 0.0.1 → 0.1.0

data/app/views/welcome/index.html.erb

@@ -0,0 +1,21 @@
+<% content_for :page_title, construct_page_title('Welcome') %>
+<% content_for :page_class, 'home' %>
+<% content_for :page_header do %>
+
+<h2>What is <%=t('sufia.product_name')%>?</h2>
+
+<% end %>
+
+<% content_for :left_column_class, 'span5' %>
+<% content_for :left_column do %>
+
+<%= render '/catalog/home_text' %>
+
+<% end %>
+
+<% content_for :right_column_class, 'span7 emphatic-action-area' %>
+<% content_for :right_column do %>
+
+  <%= link_to 'Get Started', new_classify_concern_path, :class => "btn btn-large btn-primary" %>
+
+<% end %>

data/app/workers/characterize_job.rb

@@ -0,0 +1,4 @@
+require Sufia::Engine.root.join('lib/sufia/jobs/characterize_job')
+
+# class CharacterizeJob
+# end

data/app/workers/doi_worker.rb

@@ -0,0 +1,16 @@
+class DoiWorker
+  def queue_name
+    :doi
+  end
+
+  attr_accessor :generic_file_id
+
+  def initialize(generic_file_id)
+    self.generic_file_id = generic_file_id
+  end
+
+  def run
+    mint_doi = MintDoi.new(generic_file_id)
+    mint_doi.create_or_retrieve_doi
+  end
+end

data/config/routes.rb

@@ -1,2 +1,37 @@
 Rails.application.routes.draw do
+  # devise_for :users
+
+  mount_roboto
+
+  Blacklight.add_routes(self)
+
+  resources 'dashboard', :only=>:index do
+    collection do
+      get 'page/:page', :action => :index
+      get 'facet/:id',  :action => :facet, :as => :dashboard_facet
+      get 'related/:id',:action => :get_related_file, :as => :related_file
+    end
+  end
+  resources :downloads, only: [:show]
+
+  namespace :curation_concern, path: :concern do
+    resources(
+      :generic_files,
+      only: [:new, :create],
+      path: 'container/:parent_id/generic_files'
+    )
+    resources(
+      :generic_files,
+      only: [:show, :edit, :update, :destroy]
+    )
+  end
+
+  resources :terms_of_service_agreements, only: [:new, :create]
+  resources :help_requests, only: [:new, :create]
+  resources :classify_concerns, only: [:new, :create]
+
+  match "show/:id" => "common_objects#show", via: :get, as: "common_object"
+  match "show/stub/:id" => "common_objects#show_stub_information", via: :get, as: "common_object_stub_information"
+  root to: 'welcome#index'
+
 end

data/lib/curate.rb

@@ -1,4 +1,10 @@
+require 'devise'
+require 'warden'
+require 'sufia'
 require "curate/engine"
+require 'rails'
+require 'simple_form'
+require 'roboto'
 
 module Curate
 end

data/lib/curate/engine.rb

@@ -1,4 +1,50 @@
 module Curate
   class Engine < ::Rails::Engine
+    def paths
+      @curate_paths ||= begin
+        super_paths = super
+        super_paths.add "app/repository_datastreams", eager_load: true
+        super_paths.add "app/repository_models", eager_load: true
+        super_paths.add "app/repository_models/curation_concern", eager_load: true
+        super_paths.add "app/services", eager_load: true
+        super_paths.add "app/validators", eager_load: true
+        super_paths.add "app/workers", eager_load: true
+        super_paths.add "app/inputs", eager_load: true
+        super_paths.add "app/mailers", eager_load: true
+        super_paths
+      end
+    end
+
+    config.action_dispatch.rescue_responses["ActionController::RoutingError"] = :not_found
+    config.action_dispatch.rescue_responses["ActiveFedora::ObjectNotFoundError"] = :not_found
+    config.action_dispatch.rescue_responses["ActiveFedora::ActiveObjectNotFoundError"] = :not_found
+    config.action_dispatch.rescue_responses["Hydra::AccessDenied"] = :unauthorized
+    config.action_dispatch.rescue_responses["CanCan::AccessDenied"] = :unauthorized
+    config.action_dispatch.rescue_responses["Rubydora::RecordNotFound"] = :not_found
+
+    initializer "curate" do |app|
+      require File.expand_path('../../../app/repository_models/generic_file', __FILE__)
+      require File.expand_path('../../../app/models/solr_document', __FILE__)
+      # require File.expand_path('../../../app/workers/characterize_job', __FILE__)
+
+      module WithAntiVirusHandler
+        def run
+          super
+        rescue AntiVirusScanner::VirusDetected => e
+          GenericFile.find(generic_file_id).destroy
+          raise e
+        end
+      end
+      CharacterizeJob.send(:include, WithAntiVirusHandler)
+
+      require File.expand_path('../../../app/controllers/application_controller', __FILE__)
+      require File.expand_path('../../../app/controllers/downloads_controller', __FILE__)
+      require File.expand_path('../../../app/controllers/errors_controller', __FILE__)
+      require File.expand_path('../../../app/controllers/catalog_controller', __FILE__)
+      require File.expand_path('../../../app/controllers/dashboard_controller', __FILE__)
+
+      require File.expand_path('../../../app/helpers/blacklight_helper', __FILE__)
+
+    end
   end
 end

data/lib/curate/version.rb

@@ -1,3 +1,3 @@
 module Curate
-  VERSION = "0.0.1"
+  VERSION = "0.1.0"
 end

data/spec/abilities/generic_file_abilities_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+require "cancan/matchers"
+
+describe "User" do
+  describe "Abilities" do
+    subject { ability }
+    let(:ability) { Ability.new(current_user) }
+    let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE }
+    let(:mock_curation_concern) {
+      FactoryGirl.create_curation_concern(:mock_curation_concern, creating_user, { visibility: visibility })
+    }
+    let(:generic_file) {
+      FactoryGirl.create_generic_file(mock_curation_concern, creating_user) { |gf|
+        gf.visibility = visibility
+      }
+    }
+    let(:creating_user) { nil }
+    let(:current_user) { nil }
+    let(:user) { FactoryGirl.create(:user) }
+    describe 'without embargo' do
+      describe 'creator of object' do
+        let(:creating_user) { user }
+        let(:current_user) { user }
+        it {
+          should be_able_to(:create, GenericFile.new)
+          should be_able_to(:read, generic_file)
+          should be_able_to(:update, generic_file)
+          should_not be_able_to(:delete, generic_file)
+        }
+      end
+
+      describe 'another authenticated user' do
+        let(:creating_user) { FactoryGirl.create(:user) }
+        let(:current_user) { user }
+        it {
+          should be_able_to(:create, GenericFile.new)
+          should_not be_able_to(:read, generic_file)
+          should_not be_able_to(:update, generic_file)
+          should_not be_able_to(:delete, generic_file)
+        }
+      end
+
+      describe 'a nil user' do
+        let(:creating_user) { FactoryGirl.create(:user) }
+        let(:current_user) { nil }
+        it {
+          should_not be_able_to(:create, GenericFile.new)
+          should_not be_able_to(:read, generic_file)
+          should_not be_able_to(:update, generic_file)
+          should_not be_able_to(:delete, generic_file)
+        }
+      end
+    end
+  end
+end

data/spec/controllers/application_controller_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+
+describe CommonObjectsController do
+
+  let(:user) { FactoryGirl.create(:user) }
+
+  describe 'devise sign_in helper' do
+    it 'should set :current_user on signin' do
+      expect {
+        sign_in(user)
+      }.to change(controller, :current_user).from(nil).to(user)
+    end
+
+    it 'should indicate :user_signed_in?' do
+      expect {
+        sign_in(user)
+      }.to change(controller, :user_signed_in?).from(false).to(true)
+    end
+  end
+
+  describe 'devise sign_out helper' do
+    before(:each) do
+      sign_in(user)
+    end
+    it 'should unset :current_user' do
+      expect {
+        sign_out(user)
+      }.to change(controller, :current_user).from(user).to(nil)
+    end
+
+    it 'should indicate :user_signed_in?' do
+      expect {
+        sign_out(user)
+      }.to change(controller, :user_signed_in?).from(true).to(false)
+    end
+
+  end
+end

data/spec/controllers/classify_concerns_controller_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe ClassifyConcernsController do
+  render_views
+  let(:user) { FactoryGirl.create(:user) }
+
+  describe '#new' do
+    it 'requires authentication' do
+      get :new
+      response.status.should == 302
+      expect(response).to redirect_to(user_session_path)
+    end
+    it 'renders when signed in' do
+      sign_in(user)
+      get :new
+      response.status.should == 200
+    end
+  end
+
+  describe '#create' do
+    let(:user) { FactoryGirl.create(:user) }
+    it 'redirect to login page if user is not logged in' do
+      post :create, classify: { curation_concern_type: 'MockCurationConcern' }
+      response.status.should == 302
+      expect(response).to redirect_to(user_session_path)
+    end
+
+    it 'requires authentication' do
+      sign_in(user)
+      post :create, classify_concern: { curation_concern_type: 'MockCurationConcern' }
+      response.status.should == 302
+      expect(response).to redirect_to(new_curation_concern_mock_curation_concern_path)
+    end
+
+  end
+end

data/spec/controllers/common_objects_controller_spec.rb

@@ -0,0 +1,112 @@
+require 'spec_helper'
+
+describe CommonObjectsController do
+  render_views
+
+  let(:user) { FactoryGirl.create(:user) }
+  let(:another_user) { FactoryGirl.create(:user) }
+  let(:visibility) { nil }
+  let(:curation_concern) {
+    FactoryGirl.create_generic_file(:mock_curation_concern, user) { |gf|
+      gf.visibility = visibility
+    }
+  }
+  describe '#show' do
+    let(:template_for_success) { 'show' }
+    describe '"Open Access" object' do
+      let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC }
+      it 'renders for unauthenticated person' do
+        get :show, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+
+      it 'renders for the creator' do
+        sign_in(user)
+        get :show, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+
+      it 'renders for the another user' do
+        sign_in(another_user)
+        get :show, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+    end
+
+    describe '"Restricted" object' do
+      let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE }
+      it 'redirect to DOI information for unauthenticated person' do
+        get :show, id: curation_concern.to_param
+        response.status.should == 302
+        expect(response).to redirect_to(common_object_stub_information_path(curation_concern))
+      end
+
+      it 'renders for the creator' do
+        sign_in(user)
+        get :show, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+
+      it 'renders for the creator' do
+        sign_in(another_user)
+        get :show, id: curation_concern.to_param
+        response.status.should == 302
+        expect(response).to redirect_to(common_object_stub_information_path(curation_concern))
+      end
+    end
+    describe '"Institution Only" object' do
+      let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED }
+      it 'does not display for unauthenticated person' do
+        get :show, id: curation_concern.to_param
+        response.status.should == 302
+        expect(response).to redirect_to(common_object_stub_information_path(curation_concern))
+      end
+
+      it 'renders for the creator' do
+        sign_in(user)
+        get :show, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+
+      it 'renders for the creator' do
+        sign_in(another_user)
+        get :show, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+    end
+  end
+
+  describe '#show_stub_information' do
+    let(:template_for_success) { 'show_stub_information' }
+    describe '"Open Access" object' do
+      let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC }
+      it 'renders rudimentary DOI information' do
+        get :show_stub_information, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+    end
+    describe '"Restricted" object' do
+      let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE }
+      it 'renders rudimentary DOI information' do
+        get :show_stub_information, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+    end
+    describe '"Institution Only" object' do
+      let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED }
+      it 'renders rudimentary DOI information' do
+        get :show_stub_information, id: curation_concern.to_param
+        response.status.should == 200
+        expect(response).to render_template(template_for_success)
+      end
+    end
+  end
+end

data/spec/controllers/curation_concern/base_controller_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe CurationConcern::BaseController do
+  it 'should have #save_and_add_related_files_submit_value' do
+    subject._helper_methods.should include(:save_and_add_related_files_submit_value)
+  end
+  it 'should have #curation_concern exposed' do
+    subject._helper_methods.should include(:curation_concern)
+  end
+
+  it 'has #contributor_agreement' do
+    subject._helper_methods.should include(:contributor_agreement)
+  end
+end

data/spec/controllers/curation_concern/generic_files_controller_spec.rb

@@ -0,0 +1,156 @@
+require 'spec_helper'
+
+describe CurationConcern::GenericFilesController do
+  render_views
+  let(:user) { FactoryGirl.create(:user) }
+  let(:another_user) { FactoryGirl.create(:user) }
+  let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE }
+  let(:parent) {
+    FactoryGirl.create_curation_concern(:mock_curation_concern, user, {visibility: visibility})
+  }
+  let(:file) { Rack::Test::UploadedFile.new(__FILE__, 'text/plain', false) }
+  let(:generic_file) { FactoryGirl.create_generic_file(parent, user) {|gf|
+      gf.visibility = visibility
+    }
+  }
+  let(:another_user) { FactoryGirl.create(:user) }
+
+  describe '#new' do
+    it 'renders a form if you can edit the parent' do
+      parent
+      sign_in user
+      get(:new, parent_id: parent.to_param)
+      response.should be_successful
+      expect(response).to render_template('new')
+    end
+
+    it 'redirects if you cannot edit the parent' do
+      sign_in(another_user)
+      parent
+      expect {
+        get :new, parent_id: parent.to_param
+      }.to raise_rescue_response_type(:unauthorized)
+    end
+  end
+
+  describe '#create' do
+    let(:actor) { double('actor') }
+    let(:actors_action) { :create! }
+    let(:invalid_exception) {
+      ActiveFedora::RecordInvalid.new(ActiveFedora::Base.new)
+    }
+    let(:failing_actor) {
+      actor.should_receive(actors_action).and_raise(invalid_exception)
+      actor
+    }
+    let(:successful_actor) {
+      actor.should_receive(actors_action).and_return(true)
+      actor
+    }
+
+    it 'redirects to parent when successful' do
+      sign_in(user)
+      parent
+      controller.actor = successful_actor
+
+      post(
+        :create,
+        parent_id: parent.to_param,
+        generic_file: { title: "Title", file: file }
+      )
+
+      expect(response).to(
+        redirect_to(controller.polymorphic_path([:curation_concern, parent]))
+      )
+    end
+
+    it 'renders form when unsuccessful' do
+      sign_in(user)
+      parent
+      controller.actor = failing_actor
+
+      post(
+        :create,
+        parent_id: parent.to_param,
+        generic_file: { title: "Title", file: file }
+      )
+
+      expect(response).to render_template('new')
+      response.status.should == 422
+    end
+
+  end
+
+  describe '#edit' do
+    it 'should be successful' do
+      generic_file
+      sign_in user
+      get :edit, id: generic_file.to_param
+      controller.curation_concern.should be_kind_of(GenericFile)
+      response.should be_successful
+    end
+  end
+
+  describe '#update' do
+    let(:updated_title) { Time.now.to_s }
+    let(:failing_actor) {
+      actor.
+      should_receive(:update!).
+      and_raise(ActiveFedora::RecordInvalid.new(ActiveFedora::Base.new))
+      actor
+    }
+    let(:successful_actor) {
+      actor.should_receive(:update!).and_return(true)
+      actor
+    }
+    let(:actor) { double('actor') }
+    it 'renders form when unsuccessful' do
+      generic_file
+      controller.actor = failing_actor
+      sign_in(user)
+      put :update, id: generic_file.to_param, generic_file: {title: updated_title}
+      expect(response).to render_template('edit')
+      response.status.should == 422
+    end
+
+    it 'redirects to parent when successful' do
+      generic_file
+      sign_in(user)
+      put :update, id: generic_file.to_param, generic_file: {title: updated_title}
+      response.status.should == 302
+      expect(response).to(
+        redirect_to(
+          controller.polymorphic_path([:curation_concern, generic_file])
+        )
+      )
+    end
+  end
+
+  describe '#show' do
+    it 'should be successful if logged in' do
+      generic_file
+      sign_in user
+      get :show, id: generic_file.to_param
+      controller.curation_concern.should be_kind_of(GenericFile)
+      response.should be_successful
+    end
+
+    it 'does not allow another user to view it' do
+      generic_file
+      sign_in another_user
+      expect {
+        get :show, id: generic_file.to_param
+      }.to raise_rescue_response_type(:unauthorized)
+    end
+  end
+
+  describe '#destroy' do
+    it 'should be successful if file exists' do
+      parent = generic_file.batch
+      sign_in(user)
+      delete :destroy, id: generic_file.to_param
+      expect(response.status).to eq(302)
+      expect(response).to redirect_to(controller.polymorphic_path([:curation_concern, parent]))
+    end
+  end
+end