curate 0.1.0 → 0.1.1

data/spec/features/error_handlers_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe_options = {type: :feature}
+if ENV['JS']
+  describe_options[:js] = true
+end
+
+describe 'error behavior', describe_options do
+  before(:each) do
+    @initial_consider_all_requests_local = Rails.configuration.consider_all_requests_local
+    Rails.configuration.consider_all_requests_local = false
+  end
+  after(:each) do
+    Rails.configuration.consider_all_requests_local = @initial_consider_all_requests_local
+  end
+
+  it 'handles non-existent noid for common objects with 404' do
+    visit('/show/invalid_noid')
+    expect(page).to have_content('The page you are looking for may have been removed, had its name changed, or is temporarily unavailable.')
+    expect(page).to have_content("Not Found")
+  end
+
+  let(:curation_concern_type) { :mock_curation_concern }
+  let(:user) { FactoryGirl.create(:user) }
+  let(:visibility) { AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED }
+  let(:curation_concern) {
+    FactoryGirl.create_curation_concern(
+      curation_concern_type, user, {visibility: visibility}
+    )
+  }
+  it 'handles unauthorized pages'do
+    visit("/concern/#{curation_concern_type.to_s.pluralize}/#{curation_concern.to_param}")
+    expect(page).to have_content("Unauthorized")
+    expect(page).to have_content("You are not authorized to access the page.")
+  end
+end

data/spec/routing/classify_concerns_routing_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe 'classify_concerns routing' do
+  it "routes GET /classify_concerns/new" do
+    expect(
+      get: "/classify_concerns/new"
+    ).to route_to(controller: "classify_concerns",action: "new")
+  end
+
+  it "routes POST /classify_concerns" do
+    expect(
+      post: "/classify_concerns"
+    ).to route_to(controller: "classify_concerns", action: "create")
+  end
+end

data/spec/routing/common_objects_routing_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe '/purl routing' do
+  it "routes GET /purl/:id" do
+    param_id = "12a34b56c"
+    expect(get: "/show/#{param_id}").to(
+      route_to(controller: "common_objects", action: "show", id: param_id)
+    )
+  end
+end

data/spec/routing/downloads_routing_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe 'downloads routing' do
+  let(:noid) { '1a2b3c' }
+
+  it "routes GET /downloads/:id" do
+    expect(
+      get: "/downloads/#{noid}"
+    ).to(
+      route_to(
+        controller: "downloads",
+        action: "show",
+        id: noid
+      )
+    )
+  end
+end

data/spec/routing/generic_files_routing_spec.rb

@@ -0,0 +1,79 @@
+require 'spec_helper'
+
+describe 'generic files routing' do
+  let(:parent_id) { '1a2b3c' }
+  let(:child_id) { '1a2b3c4d5e' }
+
+  it "routes GET /concern/related_files/:id" do
+    expect(
+      get: "/concern/generic_files/#{child_id}"
+    ).to(
+      route_to(
+        controller: "curation_concern/generic_files",
+        action: "show",
+        id: child_id
+      )
+    )
+  end
+
+  it "routes GET /concern/related_files/:id/edit" do
+    expect(
+      get: "/concern/generic_files/#{child_id}/edit"
+    ).to(
+      route_to(
+        controller: "curation_concern/generic_files",
+        action: "edit",
+        id: child_id
+      )
+    )
+  end
+
+  it "routes GET /concern/related_files/:id" do
+    expect(
+      put: "/concern/generic_files/#{child_id}"
+    ).to(
+      route_to(
+        controller: "curation_concern/generic_files",
+        action: "update",
+        id: child_id
+      )
+    )
+  end
+
+  it "routes GET /concern/container/:parent_id/related_files/new" do
+    expect(
+      get: "/concern/container/#{parent_id}/generic_files/new"
+    ).to(
+      route_to(
+        controller: "curation_concern/generic_files",
+        action: "new",
+        parent_id: parent_id
+      )
+    )
+  end
+
+  it "routes POST /concern/container/:parent_id/related_files" do
+    expect(
+      post: "/concern/container/#{parent_id}/generic_files"
+    ).to(
+      route_to(
+        controller: "curation_concern/generic_files",
+        action: "create",
+        parent_id: parent_id
+      )
+    )
+  end
+
+  it "routes DELETE /concern/container/:parent_id/related_files" do
+    expect(
+      delete: "/concern/generic_files/#{child_id}"
+    ).to(
+      route_to(
+        controller: "curation_concern/generic_files",
+        action: "destroy",
+        id: child_id
+      )
+    )
+  end
+
+end

data/spec/routing/help_requests_routing_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe 'help requests routing' do
+
+  it "routes GET /help_requests/new" do
+    expect(
+      get: "/help_requests/new"
+    ).to route_to(controller: "help_requests", action: "new")
+  end
+
+end

data/spec/support/mock_curation_concern.rb

@@ -1,29 +1,31 @@
 class MockCurationConcern < ActiveFedora::Base
   class MetadataDatastream < ActiveFedora::NtriplesRDFDatastream
-  map_predicates do |map|
-    map.title(in: RDF::DC) do |index|
-      index.as :searchable, :displayable
+    map_predicates do |map|
+      map.title(in: RDF::DC) do |index|
+        index.as :searchable, :displayable
+      end
+      map.created(in: RDF::DC)
+      map.creator(in: RDF::DC) do |index|
+        index.as :searchable, :facetable, :displayable
+      end
+      map.date_uploaded(to: "dateSubmitted", in: RDF::DC) do |index|
+        index.type :date
+        index.as :searchable, :displayable, :sortable
+      end
+      map.date_modified(to: "modified", in: RDF::DC) do |index|
+        index.type :date
+        index.as :searchable, :displayable, :sortable
+      end
+      map.part(:to => "hasPart", in: RDF::DC)
+      map.archived_object_type({in: RDF::DC, to: 'type'}) do |index|
+        index.as :searchable, :displayable, :facetable
+      end
+      map.identifier({in: RDF::DC})
     end
-    map.created(in: RDF::DC)
-    map.creator(in: RDF::DC) do |index|
-      index.as :searchable, :facetable, :displayable
-    end
-    map.date_uploaded(to: "dateSubmitted", in: RDF::DC) do |index|
-      index.type :date
-      index.as :searchable, :displayable, :sortable
-    end
-    map.date_modified(to: "modified", in: RDF::DC) do |index|
-      index.type :date
-      index.as :searchable, :displayable, :sortable
-    end
-    map.part(:to => "hasPart", in: RDF::DC)
-    map.identifier({in: RDF::DC})
   end
-end
-  include Hydra::ModelMixins::CommonMetadata
-  include Sufia::ModelMethods
-  include Sufia::Noid
-  include Sufia::GenericFile::Permissions
+
+  include CurationConcern::Model
+  include CurationConcern::WithGenericFiles
   include CurationConcern::Embargoable
   include CurationConcern::WithAccessRight
 
@@ -39,23 +41,9 @@ end
       :date_uploaded,
       :date_modified,
       :creator,
-      :identifier,
+      :identifier
     ],
     unique: true
   )
 
-  has_many :generic_files, property: :is_part_of
-
-  after_destroy :after_destroy_cleanup
-  def after_destroy_cleanup
-    generic_files.each(&:destroy)
-  end
-
-  def human_readable_type
-    self.class.to_s.demodulize.titleize
-  end
-  def to_param
-    pid.split(':').last
-  end
-
-end
+end

data/spec/validators/future_date_validator_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+class Validatable
+  include ActiveModel::Validations
+  attr_accessor :embargo_release_date
+  validates :embargo_release_date, future_date: true
+end
+
+describe FutureDateValidator do
+
+  subject { Validatable.new }
+
+  before { subject.embargo_release_date = embargo_release_date }
+
+  context 'with today as embargo release date' do
+    let(:embargo_release_date) { Date.today.to_s    }
+    it { should have(1).error_on(:embargo_release_date) }
+  end
+
+  context 'with past date as embargo release date' do
+    let(:embargo_release_date) { (Date.today - 2).to_s  }
+    it { should have(1).error_on(:embargo_release_date) }
+  end
+
+  context 'invalid date as embargo release date' do
+    let(:embargo_release_date) { "invalid_ date" }
+    it { should have(1).error_on(:embargo_release_date) }
+  end
+
+  context 'future date as embargo release date' do
+    let(:embargo_release_date) { (Date.today + 2).to_s    }
+    it { should have(:no).error_on(:embargo_release_date) }
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: curate
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-11 00:00:00.000000000 Z
+date: 2013-04-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -395,6 +395,22 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: capybara
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A data curation Ruby on Rails engine built on Hydra and Sufia
 email:
 - jeremy.n.friesen@gmail.com
@@ -402,6 +418,33 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- app/assets/images/blacklight/bg.png
+- app/assets/images/blacklight/border.png
+- app/assets/images/blacklight/bul_sq_gry.gif
+- app/assets/images/blacklight/checkmark.gif
+- app/assets/images/blacklight/logo.png
+- app/assets/images/blacklight/magnifying_glass.gif
+- app/assets/images/blacklight/remove.gif
+- app/assets/images/blacklight/separator.gif
+- app/assets/images/blacklight/start_over.gif
+- app/assets/javascripts/curate.js
+- app/assets/javascripts/help_modal.js
+- app/assets/javascripts/manage_repeating_fields.js
+- app/assets/javascripts/toggle_details.js
+- app/assets/stylesheets/blacklight.css.scss
+- app/assets/stylesheets/curate.css.scss
+- app/assets/stylesheets/downloads.css.scss
+- app/assets/stylesheets/global-variables.scss
+- app/assets/stylesheets/help_requests.css.scss
+- app/assets/stylesheets/layout/positioning.css.scss
+- app/assets/stylesheets/modules/action_bar.css.scss
+- app/assets/stylesheets/modules/emphatic_action_area.css.scss
+- app/assets/stylesheets/modules/forms.css.scss
+- app/assets/stylesheets/modules/multi_value_fields.css.scss
+- app/assets/stylesheets/modules/site_search.css.scss
+- app/assets/stylesheets/modules.scss
+- app/assets/stylesheets/style/theme.css.scss
+- app/assets/stylesheets/style/typography.css.scss
 - app/controllers/application_controller.rb
 - app/controllers/catalog_controller.rb
 - app/controllers/classify_concerns_controller.rb
@@ -433,7 +476,9 @@ files:
 - app/models/solr_document.rb
 - app/repository_datastreams/file_content_datastream.rb
 - app/repository_models/curation_concern/embargoable.rb
+- app/repository_models/curation_concern/model.rb
 - app/repository_models/curation_concern/with_access_right.rb
+- app/repository_models/curation_concern/with_generic_files.rb
 - app/repository_models/generic_file.rb
 - app/services/anti_virus_scanner.rb
 - app/services/curation_concern/base_actor.rb
@@ -558,8 +603,18 @@ files:
 - app/workers/doi_worker.rb
 - config/routes.rb
 - lib/curate/engine.rb
+- lib/curate/railtie.rb
 - lib/curate/version.rb
 - lib/curate.rb
+- lib/generators/curate/curation_concern/curation_concern_generator.rb
+- lib/generators/curate/curation_concern/templates/curation_concern.rb.erb
+- lib/generators/curate/curation_concern/templates/curation_concern_factory.rb.erb
+- lib/generators/curate/curation_concern/templates/curation_concern_serializer.rb.erb
+- lib/generators/curate/curation_concern/templates/curation_concern_serializer_spec.rb.erb
+- lib/generators/curate/curation_concern/templates/curation_concern_spec.rb.erb
+- lib/generators/curate/curation_concern/templates/metadata_datastream.rb.erb
+- lib/generators/curate/curation_concern/templates/metadata_datastream_spec.rb.erb
+- lib/generators/curate/curation_concern/USAGE
 - lib/tasks/curate_tasks.rake
 - LICENSE
 - Rakefile
@@ -578,6 +633,7 @@ files:
 - spec/dummy/app/assets/javascripts/application.js
 - spec/dummy/app/assets/stylesheets/application.css
 - spec/dummy/app/controllers/application_controller.rb
+- spec/dummy/app/controllers/curation_concern/mock_curation_concerns_controller.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/models/solr_document.rb
 - spec/dummy/app/models/user.rb
@@ -653,6 +709,7 @@ files:
 - spec/factories/help_requests.rb
 - spec/factories/mock_curation_concerns.rb
 - spec/factories/users.rb
+- spec/features/error_handlers_spec.rb
 - spec/helpers/application_helper_spec.rb
 - spec/helpers/common_objects_helper_spec.rb
 - spec/models/access_right_spec.rb
@@ -667,6 +724,11 @@ files:
 - spec/models/repo_object_spec.rb
 - spec/repository_models/curation_concern/embargoable_spec.rb
 - spec/repository_models/generic_file_spec.rb
+- spec/routing/classify_concerns_routing_spec.rb
+- spec/routing/common_objects_routing_spec.rb
+- spec/routing/downloads_routing_spec.rb
+- spec/routing/generic_files_routing_spec.rb
+- spec/routing/help_requests_routing_spec.rb
 - spec/services/anti_virus_scanner_spec.rb
 - spec/services/curation_concern/base_actor_spec.rb
 - spec/services/curation_concern/generic_file_actor_spec.rb
@@ -677,6 +739,7 @@ files:
 - spec/support/mock_curation_concern.rb
 - spec/support/shared/shared_examples_is_embargoable.rb
 - spec/support/shared/shared_examples_with_access_rights.rb
+- spec/validators/future_date_validator_spec.rb
 - spec/workers/characterize_job_spec.rb
 homepage: https://github.com/ndlib/curate
 licenses: []
@@ -692,7 +755,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1069347292941721312
+      hash: 2061838481868737631
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -701,7 +764,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1069347292941721312
+      hash: 2061838481868737631
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25
@@ -723,6 +786,7 @@ test_files:
 - spec/dummy/app/assets/javascripts/application.js
 - spec/dummy/app/assets/stylesheets/application.css
 - spec/dummy/app/controllers/application_controller.rb
+- spec/dummy/app/controllers/curation_concern/mock_curation_concerns_controller.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/models/solr_document.rb
 - spec/dummy/app/models/user.rb
@@ -798,6 +862,7 @@ test_files:
 - spec/factories/help_requests.rb
 - spec/factories/mock_curation_concerns.rb
 - spec/factories/users.rb
+- spec/features/error_handlers_spec.rb
 - spec/helpers/application_helper_spec.rb
 - spec/helpers/common_objects_helper_spec.rb
 - spec/models/access_right_spec.rb
@@ -812,6 +877,11 @@ test_files:
 - spec/models/repo_object_spec.rb
 - spec/repository_models/curation_concern/embargoable_spec.rb
 - spec/repository_models/generic_file_spec.rb
+- spec/routing/classify_concerns_routing_spec.rb
+- spec/routing/common_objects_routing_spec.rb
+- spec/routing/downloads_routing_spec.rb
+- spec/routing/generic_files_routing_spec.rb
+- spec/routing/help_requests_routing_spec.rb
 - spec/services/anti_virus_scanner_spec.rb
 - spec/services/curation_concern/base_actor_spec.rb
 - spec/services/curation_concern/generic_file_actor_spec.rb
@@ -822,4 +892,5 @@ test_files:
 - spec/support/mock_curation_concern.rb
 - spec/support/shared/shared_examples_is_embargoable.rb
 - spec/support/shared/shared_examples_with_access_rights.rb
+- spec/validators/future_date_validator_spec.rb
 - spec/workers/characterize_job_spec.rb