curation_concerns 1.6.3 → 1.7.0.beta1

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

@@ -20,4 +20,15 @@ class TestAppGenerator < Rails::Generators::Base
     remove_file 'spec/controllers/curation_concerns/generic_works_controller_spec.rb'
     remove_file 'spec/actors/curation_concerns/generic_work_actor_spec.rb'
   end
+
+  def copy_fixture_data
+    generate 'curation_concerns:sample_data', '-f'
+  end
+
+  def enable_i18n_translation_errors
+    gsub_file "config/environments/development.rb",
+              "# config.action_view.raise_on_missing_translations = true", "config.action_view.raise_on_missing_translations = true"
+    gsub_file "config/environments/test.rb",
+              "# config.action_view.raise_on_missing_translations = true", "config.action_view.raise_on_missing_translations = true"
+  end
 end

data/spec/views/curation_concerns/admin/_admin_menu.html.erb_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+RSpec.describe "admin_menu.html.erb" do
+  let(:configuration) do
+    {
+      menu: {
+        index: {}
+      }
+    }
+  end
+  before do
+    allow(view).to receive(:action_name).and_return(:index)
+    render partial: "curation_concerns/admin/admin_menu", locals: { configuration: configuration }
+  end
+  context "when there's an item in the index menu" do
+    it "makes a link to the right action" do
+      expect(rendered).to have_link "Admin Dashboard", href: curation_concerns.admin_index_path
+    end
+    it "marks the current page as active" do
+      expect(rendered).to have_selector "li.active > a"
+    end
+  end
+end

data/spec/views/curation_concerns/admin/_resource_stats.html.erb_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe 'curation_concerns/admin/_resource_stats.html.erb', type: :view do
+  let(:resource_stats) do
+    instance_double(CurationConcerns::ResourceStatisticsSource,
+                    open_concerns_count: 25,
+                    authenticated_concerns_count: 300,
+                    restricted_concerns_count: 777,
+                    expired_embargo_now_authenticated_concerns_count: 66,
+                    expired_embargo_now_open_concerns_count: 77,
+                    active_embargo_now_authenticated_concerns_count: 88,
+                    active_embargo_now_restricted_concerns_count: 99,
+                    expired_lease_now_authenticated_concerns_count: 6666,
+                    expired_lease_now_restricted_concerns_count: 7777,
+                    active_lease_now_authenticated_concerns_count: 8888,
+                    active_lease_now_open_concerns_count: 9999)
+  end
+
+  it "renders without error" do
+    render partial: "curation_concerns/admin/resource_stats", locals: { resource_stats: resource_stats }
+    expect(rendered).to have_content("Open Access25")
+    expect(rendered).to have_content("Institution Name300")
+    expect(rendered).to have_content("Private777")
+    expect(rendered).to have_content("Embargo (Expired, Authenticated)66")
+    expect(rendered).to have_content("Embargo (Expired, Open)77")
+    expect(rendered).to have_content("Embargo (Active, Authenticated)88")
+    expect(rendered).to have_content("Embargo (Active, Restricted)99")
+    expect(rendered).to have_content("Lease (Active, Authenticated)8888")
+    expect(rendered).to have_content("Lease (Active, Open)9999")
+    expect(rendered).to have_content("Lease (Expired, Authenticated)6666")
+    expect(rendered).to have_content("Lease (Expired, Restricted)7777")
+  end
+end

data/spec/views/curation_concerns/admin/_total_objects_charts.html.erb_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe 'curation_concerns/admin/_total_objects_charts.html.erb', type: :view do
+  before do
+    allow(view).to receive(:action_name).and_return(:index)
+    assign(:resource_statistics, resource_stats)
+    stub_template 'curation_concerns/admin/widgets/_pie.html.erb' => 'Mine 1'
+  end
+  let(:resource_stats) do
+    instance_double(CurationConcerns::ResourceStatisticsSource,
+                    open_concerns_count: 25,
+                    authenticated_concerns_count: 300,
+                    restricted_concerns_count: 777,
+                    expired_embargo_now_authenticated_concerns_count: 66,
+                    expired_embargo_now_open_concerns_count: 77,
+                    active_embargo_now_authenticated_concerns_count: 88,
+                    active_embargo_now_restricted_concerns_count: 99,
+                    expired_lease_now_authenticated_concerns_count: 6666,
+                    expired_lease_now_restricted_concerns_count: 7777,
+                    active_lease_now_authenticated_concerns_count: 8888,
+                    active_lease_now_open_concerns_count: 9999)
+  end
+
+  it "renders without error" do
+    render
+    expect(rendered).to have_content("Mine 1")
+  end
+end

data/spec/views/curation_concerns/admin/index.html.erb_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe 'curation_concerns/admin/index.html.erb', type: :view do
+  before do
+    assign(:configuration, configuration)
+    allow(view).to receive(:action_name).and_return(:index)
+    stub_template 'curation_concerns/admin/_my_view_1.html.erb' => 'Mine 1'
+    stub_template 'curation_concerns/admin/_my_view_2.html.erb' => 'Another One'
+  end
+  let(:configuration) do
+    { actions: {
+      index: {
+        partials: [
+          "my_view_1",
+          "my_view_2"
+        ]
+      }
+    } }
+  end
+
+  it "renders all the partials" do
+    render
+    expect(rendered).to have_content("Mine 1")
+    expect(rendered).to have_content("Another One")
+  end
+end

data/spec/views/curation_concerns/admin/widgets/_pie.html.erb_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+RSpec.describe "_pie.html.erb" do
+  let(:data) { { mine: 1, yours: 0 } }
+  before do
+    render "curation_concerns/admin/widgets/pie", data: data, label: 'my_pie'
+  end
+
+  it "makes a div with data and class" do
+    expect(rendered).to have_css "div#my_pie-stats-pie.stats-pie"
+    expect(rendered).to have_selector 'div[data-series]'
+    expect(rendered).to have_selector 'div[data-label]'
+    text = rendered.to_s
+    expect(text).to include 'data-series="{"drilldown":{"series":[]},"series":[{"name":"mine","y":1},{"name":"yours","y":0}]}"'
+    expect(text).to include 'data-label="my_pie_data"'
+  end
+end

data/spec/views/curation_concerns/base/file_manager.html.erb_spec.rb

@@ -31,7 +31,7 @@ RSpec.describe "curation_concerns/base/file_manager.html.erb" do
     SolrDocument.new(parent.to_solr.merge(id: "resource"), nil)
   end
   let(:parent_presenter) do
-    CurationConcerns::WorkShowPresenter.new(parent_solr_doc, nil)
+    CurationConcerns::WorkShowPresenter.new(parent_solr_doc, nil, view)
   end
 
   let(:blacklight_config) { CatalogController.new.blacklight_config }

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

@@ -17,11 +17,16 @@ describe 'curation_concerns/base/show.html.erb' do
     )
   }
 
+  let(:entity) { instance_double(Sipity::Entity, workflow_state_name: 'pending') }
   let(:ability) { nil }
   let(:presenter) do
     CurationConcerns::WorkShowPresenter.new(solr_document, ability)
   end
 
+  before do
+    allow(PowerConverter).to receive(:convert).and_return(entity)
+  end
+
   context 'for editors' do
     before do
       allow(view).to receive(:can?).with(:edit, String).and_return(true)
@@ -30,13 +35,41 @@ describe 'curation_concerns/base/show.html.erb' do
       assign(:presenter, presenter)
       render
     end
+    let(:header_node) do
+      header_content = view.view_flow.content[:page_header]
+      Capybara::Node::Simple.new(header_content)
+    end
 
     it 'draws the page' do
+      expect(header_node).to have_selector '.state.state-pending', text: 'pending'
       expect(rendered).to have_link 'Attribution 3.0 United States', href: 'http://creativecommons.org/licenses/by/3.0/us/'
       expect(rendered).to have_link 'Edit This Generic Work', href: edit_polymorphic_path(presenter)
     end
   end
 
+  context 'with parent_presenter' do
+    before do
+      allow(view).to receive(:can?).with(:edit, String).and_return(true)
+      allow(view).to receive(:can?).with(:collect, String).and_return(true)
+      allow(view).to receive(:collection_options_for_select)
+      assign(:presenter, presenter)
+      assign(:parent_presenter, parent_presenter)
+      allow(parent_presenter).to receive(:to_s).and_return("Parent Work")
+      render
+    end
+    let(:parent_presenter) do
+      CurationConcerns::WorkShowPresenter.new(solr_document, ability)
+    end
+    let(:header_node) do
+      header_content = view.view_flow.content[:page_header]
+      Capybara::Node::Simple.new(header_content)
+    end
+
+    it 'draws the page' do
+      expect(header_node).to have_selector '.breadcrumb', text: 'Parent Work'
+    end
+  end
+
   context 'for non-editors' do
     before do
       assign(:presenter, presenter)

data/spec/views/curation_concerns/permissions/confirm.html.erb_spec.rb

@@ -16,7 +16,7 @@ describe 'curation_concerns/permissions/confirm.html.erb' do
   end
 
   context 'when the work is leased' do
-    let(:curation_concern) { build(:leased_work, embargo_date: '2099-09-26'.to_date) }
+    let(:curation_concern) { build(:leased_work, lease_date: '2099-09-26'.to_date) }
 
     it 'has a message about leases' do
       expect(rendered).to have_content "You've applied a lease to this Generic Work, Test title, changing its visibility to Open Access until September 26th, 2099. Would you like to apply the same lease to all of the files within the Generic Work as well?"

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

@@ -18,6 +18,7 @@ describe 'shared/_my_actions.html.erb' do
       expect(rendered).to have_link("My Collections", href: search_path_for_my_collections)
       expect(rendered).to have_link("Embargos", href: embargoes_path)
       expect(rendered).to have_link("Leases", href: leases_path)
+      expect(rendered).to have_link("Admin Dashboard", href: curation_concerns.admin_index_path)
     end
   end
 

data/tasks/workflow.rake

@@ -0,0 +1,8 @@
+namespace :curation_concerns do
+  namespace :workflow do
+    desc "Load workflow configuration into the database"
+    task load: :environment do
+      CurationConcerns::Workflow::WorkflowImporter.load_workflows
+    end
+  end
+end