curation_concerns 1.1.2 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 835147e30b02f4bd8176f008f6581a72852a03f0
-  data.tar.gz: 653102b81aed097f57c153baaea3c1f85f4e3f1f
+  metadata.gz: 832c5d5ac39f97b63124797e7ddd1b4f44b50314
+  data.tar.gz: dc3f9767e7770d2df1cd944a1b7fa0ae8319c0bb
 SHA512:
-  metadata.gz: 7a6827a24f8fc84168c0d53fdfadc4745b08707d29a9ef0b6abaf628ccb8e8ae6177473068e1a723bcfc4b342d49e9323e0e6a354defffe2a7ed14d094581a5a
-  data.tar.gz: 20707bc6b4305fbc5df1cdf03216f5f8b7634aa442ec141b42b67bacf77e51da16bb58efe3804ae334dba22af31a48be982e5a7e479f53e81bb856df6c1ff106
+  metadata.gz: 4d4493a39a865be80e9a9b63b9b6aeea62a4d5080fc989916b798c731b03b2a8e60845f2ab84fdad3adfa12f51838aa3495731da5df8efebd516f331b705f3a1
+  data.tar.gz: bc3ae415927e33076d79012f495676f92d08f6b42fa3add0795f7172d91e96cecc7fdd41e29d47c59a149e2618ac77ce27296941cad4a798c74d2390d4e9eec2

data/Gemfile

@@ -12,10 +12,10 @@ group :development, :test do
 end
 
 # BEGIN ENGINE_CART BLOCK
-# engine_cart: 0.8.0
-# engine_cart stanza: 0.8.0
+# engine_cart: 0.10.0
+# engine_cart stanza: 0.10.0
 # the below comes from engine_cart, a gem used to test this Rails engine gem in the context of a Rails app.
-file = File.expand_path("Gemfile", ENV['ENGINE_CART_DESTINATION'] || ENV['RAILS_ROOT'] || File.expand_path(".internal_test_app", File.dirname(__FILE__)))
+file = File.expand_path('Gemfile', ENV['ENGINE_CART_DESTINATION'] || ENV['RAILS_ROOT'] || File.expand_path('.internal_test_app', File.dirname(__FILE__)))
 if File.exist?(file)
   begin
     eval_gemfile file
@@ -25,13 +25,24 @@ if File.exist?(file)
   end
 else
   Bundler.ui.warn "[EngineCart] Unable to find test application dependencies in #{file}, using placeholder dependencies"
-  gem 'rails', ENV['RAILS_VERSION'] if ENV['RAILS_VERSION']
 
-  if ENV['RAILS_VERSION'].nil? || ENV['RAILS_VERSION'] =~ /^4.2/
-    gem 'responders', "~> 2.0"
-    gem 'sass-rails', ">= 5.0"
-  else
-    gem 'sass-rails', "< 5.0"
+  if ENV['RAILS_VERSION']
+    if ENV['RAILS_VERSION'] == 'edge'
+      gem 'rails', github: 'rails/rails'
+      ENV['ENGINE_CART_RAILS_OPTIONS'] = '--edge --skip-turbolinks'
+    else
+      gem 'rails', ENV['RAILS_VERSION']
+    end
+  end
+
+  case ENV['RAILS_VERSION']
+  when /^4.2/
+    gem 'responders', '~> 2.0'
+    gem 'sass-rails', '>= 5.0'
+    gem 'coffee-rails', '~> 4.1.0'
+  when /^4.[01]/
+    gem 'sass-rails', '< 5.0'
   end
 end
 # END ENGINE_CART BLOCK
+

data/app/actors/curation_concerns/actors/apply_order_actor.rb

@@ -11,7 +11,7 @@ module CurationConcerns
 
         def sync_members(ordered_member_ids)
           return true if ordered_member_ids.nil?
-          existing_members_ids = curation_concern.member_ids
+          existing_members_ids = curation_concern.ordered_member_ids
           (existing_members_ids - ordered_member_ids).each do |old_id|
             work = ::ActiveFedora::Base.find(old_id)
             curation_concern.ordered_members.delete(work)

data/app/assets/javascripts/curation_concerns/application.js

@@ -11,5 +11,6 @@
 //= require bootstrap/affix
 //
 //= require jquery-ui/sortable
+//= require clipboard
 //
 //= require curation_concerns/curation_concerns

data/app/assets/javascripts/curation_concerns/curation_concerns.js

@@ -6,6 +6,7 @@
 //= require curation_concerns/file_manager/sorting
 //= require curation_concerns/file_manager/save_manager
 //= require curation_concerns/file_manager/member
+//= require curation_concerns/single_use_links_manager
 //= require curation_concerns/batch_select
 //= require curation_concerns/collections
 

data/app/assets/javascripts/curation_concerns/single_use_links_manager.js

@@ -0,0 +1,53 @@
+(function( $ ){
+  $.fn.singleUseLinks = function( options ) {
+
+    var clipboard = new Clipboard('.copy-single-use-link');
+
+    var manager = {
+      reload_table: function() {
+        var url = $("table.single-use-links tbody").data('url')
+        $.get(url).done(function(data) {
+          $('table.single-use-links tbody').html(data);
+        });
+      },
+
+      create_link: function(caller) {
+        $.post(caller.attr('href')).done(function(data) {
+          manager.reload_table()
+        })
+      },
+
+      delete_link: function(caller) {
+        $.ajax({
+          url: caller.attr('href'),
+          type: 'DELETE',
+          done: caller.parent('td').parent('tr').remove()
+        })
+      }
+    };
+
+    $('.generate-single-use-link').click(function(event) {
+      event.preventDefault()
+      manager.create_link($(this))
+      return false
+    });
+
+    $("table.single-use-links tbody").on('click', '.delete-single-use-link', function(event) {
+      event.preventDefault()
+      manager.delete_link($(this))
+      return false;
+    });
+
+    clipboard.on('success', function(e) {
+      $(e.trigger).tooltip('show');
+      e.clearSelection();
+    }); 
+
+    return manager;
+
+  };
+})( jQuery );
+
+Blacklight.onLoad(function () {
+  $('.single-use-links').singleUseLinks();
+});

data/app/controllers/concerns/curation_concerns/single_use_links_controller_behavior.rb

@@ -25,6 +25,16 @@ module CurationConcerns
       render text: curation_concerns.show_single_use_link_url(@su.downloadKey)
     end
 
+    def index
+      links = SingleUseLink.where(itemId: params[:id])
+      render partial: 'curation_concerns/file_sets/single_use_link_rows', locals: { single_use_links: links }
+    end
+
+    def destroy
+      SingleUseLink.find_by_downloadKey(params[:link_id]).destroy
+      head :ok
+    end
+
     protected
 
       def authorize_user!

data/app/presenters/curation_concerns/file_set_presenter.rb

@@ -33,5 +33,9 @@ module CurationConcerns
     def link_name
       current_ability.can?(:read, id) ? label : 'File'
     end
+
+    def single_use_links
+      @single_use_links ||= SingleUseLink.where(itemId: id)
+    end
   end
 end

data/app/renderers/curation_concerns/renderers/date_attribute_renderer.rb

@@ -0,0 +1,11 @@
+module CurationConcerns
+  module Renderers
+    class DateAttributeRenderer < AttributeRenderer
+      private
+
+        def attribute_value_to_html(value)
+          Date.parse(value).to_formatted_s(:standard)
+        end
+    end
+  end
+end

data/app/renderers/renderers.rb

@@ -1,11 +1,10 @@
 module CurationConcerns
-  # Module that will containing Renderers
+  # Module that will contain Renderer classes
   # @since 0.14.0
-  # Renderers are used to display Ruby objects to users.
-  # Renderers take the Ruby object supplied by a
-  # CurationConcerns::Presenter object Renderers are typically  # used to respond to read requests from the controller.
-  # Renderers may apply some level of HTML or may just supply
-  # human readable values for object attributes.
+  # Renderers are used to display Ruby objects to users and take arguments from the CurationConcerns::Presenter
+  # supplied by the controller.
+  # They are typically used to respond to read requests from the controller, and may apply some level of
+  # HTML, or may just display human readable values for object attributes.
   module Renderers
   end
 end

data/app/views/curation_concerns/base/_attributes.html.erb

@@ -6,8 +6,8 @@
   <tbody>
     <%= render 'attribute_rows', presenter: presenter %>
     <%= presenter.attribute_to_html(:permission_badge, label: 'Visibility') %>
-    <%= presenter.attribute_to_html(:embargo_release_date) %>
-    <%= presenter.attribute_to_html(:lease_expiration_date) %>
+    <%= presenter.attribute_to_html(:embargo_release_date, render_as: :date) %>
+    <%= presenter.attribute_to_html(:lease_expiration_date, render_as: :date) %>
     <%= presenter.attribute_to_html(:rights, render_as: :rights) %>
   </tbody>
 </table>

data/app/views/curation_concerns/file_sets/_single_use_link_rows.html.erb

@@ -0,0 +1,18 @@
+<% single_use_links.each_with_index do |link, index| %>
+  <% type = link.path =~ /downloads/ ? 'Download' : 'Show' %>
+  <% url_helper = link.path =~ /downloads/ ? 'download_single_use_link_url' : 'show_single_use_link_url' %>
+  <tr>
+    <td><%= type %></td>
+    <td><%= link.downloadKey %></td>
+    <td><%= link.expires %></td>
+    <td>
+      <button class="btn btn-xs btn-default copy-single-use-link"
+              data-clipboard-text="<%= curation_concerns.send(url_helper, link.downloadKey) %>"
+              data-toggle="tooltip" data-placement="bottom" title="Copied!">
+        Copy to Clipboard
+      </button>
+      <%= link_to "Delete", curation_concerns.delete_single_use_link_path(params[:id], link),
+          class: 'btn btn-xs btn-danger delete-single-use-link' %>
+    </td>
+  </tr>
+<% end %>

data/app/views/curation_concerns/file_sets/_single_use_links.html.erb

@@ -0,0 +1,20 @@
+<table class="table table-striped <%= dom_class(presenter) %> single-use-links">
+  <caption class="table-heading"><h2>Single-Use Links</h2></caption>
+  <thead>
+    <tr><th>Link</th><th>Key</th><th>Expires</th><th>Actions</th></tr>
+  </thead>
+  <tbody data-url="<%= curation_concerns.generated_single_use_links_path(presenter) %>">
+    <% if presenter.single_use_links.empty? %>
+      <tr><td>No links have been generated</td></tr>
+    <% else %>
+      <%= render 'single_use_link_rows', single_use_links: presenter.single_use_links %>
+    <% end %>
+  </tbody>
+</table>
+
+<div class="form_actions">
+  <%= link_to "Generate Download Link", curation_concerns.generate_download_single_use_link_path(presenter),
+      class: 'btn btn-default generate-single-use-link' %>
+  <%= link_to "Generate Show Link", curation_concerns.generate_show_single_use_link_path(presenter),
+      class: 'btn btn-default generate-single-use-link' %>
+</div>

data/app/views/curation_concerns/file_sets/show.html.erb

@@ -5,3 +5,4 @@
 <%= media_display @presenter %>
 <%= render "attributes", presenter: @presenter %>
 <%= render "show_actions", presenter: @presenter, parent: parent %>
+<%= render "single_use_links", presenter: @presenter %>

data/config/routes.rb

@@ -3,6 +3,8 @@ CurationConcerns::Engine.routes.draw do
   get 'single_use_link/download/:id' => 'single_use_links_viewer#download', as: :download_single_use_link
   post 'single_use_link/generate_download/:id' => 'single_use_links#create_download', as: :generate_download_single_use_link
   post 'single_use_link/generate_show/:id' => 'single_use_links#create_show', as: :generate_show_single_use_link
+  get 'single_use_link/generated/:id' => 'single_use_links#index', as: :generated_single_use_links
+  delete 'single_use_link/:id/delete/:link_id' => 'single_use_links#destroy', as: :delete_single_use_link
 
   # mount BrowseEverything::Engine => '/remote_files/browse'
   resources :classify_concerns, only: [:new, :create]

data/curation_concerns.gemspec

@@ -39,6 +39,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'rdf-vocab', '~> 0'
   spec.add_dependency 'awesome_nested_set', '~> 3.0'
   spec.add_dependency 'browse-everything', '~> 0.10'
+  spec.add_dependency 'clipboard-rails', '~> 1.5'
 
   spec.add_development_dependency 'solr_wrapper', '~> 0.13', '>= 0.13.1'
   spec.add_development_dependency 'fcrepo_wrapper', '~> 0.1'

data/lib/curation_concerns/version.rb

@@ -1,3 +1,3 @@
 module CurationConcerns
-  VERSION = "1.1.2".freeze
+  VERSION = "1.2.0".freeze
 end

data/lib/curation_concerns.rb

@@ -4,6 +4,7 @@ require 'curation_concerns/configuration'
 require 'curation_concerns/collections'
 require 'blacklight_advanced_search'
 require 'kaminari_route_prefix'
+require 'clipboard/rails'
 
 module CurationConcerns
 end

data/spec/controllers/curation_concerns/single_use_links_controller_spec.rb

@@ -2,78 +2,94 @@ require 'spec_helper'
 
 describe CurationConcerns::SingleUseLinksController, type: :controller do
   routes { CurationConcerns::Engine.routes }
-  let(:user) { create(:user) }
 
-  let(:file) do
-    FileSet.create do |file|
-      file.apply_depositor_metadata(user)
-    end
-  end
+  let(:user) { create(:user) }
+  let(:file) { create(:file_set, user: user) }
 
   describe "logged in user with edit permission" do
     let(:hash) { "some-dummy-sha2-hash" }
 
-    before do
-      sign_in user
-      allow(DateTime).to receive(:now).and_return(DateTime.now)
-      expect(Digest::SHA2).to receive(:new).and_return(hash)
+    before { sign_in user }
+
+    context "POST create" do
+      before do
+        allow(DateTime).to receive(:now).and_return(DateTime.now)
+        expect(Digest::SHA2).to receive(:new).and_return(hash)
+      end
+
+      describe "creating a single-use download link" do
+        it "returns a link for downloading" do
+          post 'create_download', id: file
+          expect(response).to be_success
+          expect(response.body).to eq download_single_use_link_url(hash)
+        end
+      end
+
+      describe "creating a single-use show link" do
+        it "returns a link for showing" do
+          post 'create_show', id: file
+          expect(response).to be_success
+          expect(response.body).to eq show_single_use_link_url(hash)
+        end
+      end
     end
 
-    describe "GET 'download'" do
-      it "and_return http success" do
-        post 'create_download', id: file
-        expect(response).to be_success
-        expect(response.body).to eq download_single_use_link_url(hash)
+    context "GET index" do
+      describe "viewing existing links" do
+        before { get :index, id: file }
+        subject { response }
+        it { is_expected.to be_success }
       end
     end
 
-    describe "GET 'show'" do
-      it "and_return http success" do
-        post 'create_show', id: file
+    context "DELETE destroy" do
+      let!(:link) { create(:download_link) }
+      it "deletes the link" do
+        expect { delete :destroy, id: file, link_id: link }.to change { SingleUseLink.count }.by(-1)
         expect(response).to be_success
-        expect(response.body).to eq show_single_use_link_url(hash)
       end
     end
   end
 
   describe "logged in user without edit permission" do
-    before do
-      @other_user = create(:user)
-      file.read_users << @other_user
-      file.save!
-      sign_in @other_user
-      file.read_users << @other_user
-      file.save!
+    let(:other_user) { create(:user) }
+    let(:file) { create(:file_set, user: user, read_users: [other_user]) }
+
+    before { sign_in other_user }
+    subject { response }
+
+    describe "creating a single-use download link" do
+      before { post 'create_download', id: file }
+      it { is_expected.not_to be_success }
     end
 
-    describe "GET 'download'" do
-      it "and_return http success" do
-        post 'create_download', id: file
-        expect(response).not_to be_success
-      end
+    describe "creating a single-use show link" do
+      before { post 'create_show', id: file }
+      it { is_expected.not_to be_success }
     end
 
-    describe "GET 'show'" do
-      it "and_return http success" do
-        post 'create_show', id: file
-        expect(response).not_to be_success
-      end
+    describe "viewing existing links" do
+      before { get :index, id: file }
+      it { is_expected.not_to be_success }
     end
   end
 
   describe "unknown user" do
-    describe "GET 'download'" do
-      it "and_return http failure" do
-        post 'create_download', id: file
-        expect(response).not_to be_success
-      end
+    subject { response }
+
+    describe "creating a single-use download link" do
+      before { post 'create_download', id: file }
+      it { is_expected.not_to be_success }
     end
 
-    describe "GET 'show'" do
-      it "and_return http failure" do
-        post 'create_show', id: file
-        expect(response).not_to be_success
-      end
+    describe "creating a single-use show link" do
+      before { post 'create_show', id: file }
+      it { is_expected.not_to be_success }
+    end
+
+    describe "viewing existing links" do
+      before { get :index, id: file }
+      it { is_expected.not_to be_success }
     end
   end
 end

data/spec/controllers/curation_concerns/single_use_links_viewer_controller_spec.rb

@@ -31,7 +31,7 @@ describe CurationConcerns::SingleUseLinksViewerController do
     describe "GET 'download'" do
       let(:expected_content) { ActiveFedora::Base.find(file.id).original_file.content }
 
-      it "and_return http success" do
+      it "downloads the file and deletes the link from the database" do
         expect(controller).to receive(:send_file_headers!).with(filename: 'world.png', disposition: 'attachment', type: 'image/png')
         get :download, id: download_link_hash
         expect(response.body).to eq expected_content
@@ -39,10 +39,10 @@ describe CurationConcerns::SingleUseLinksViewerController do
         expect { SingleUseLink.find_by_downloadKey!(download_link_hash) }.to raise_error ActiveRecord::RecordNotFound
       end
 
-      context "and the key is not found" do
+      context "when the key is not found" do
         before { SingleUseLink.find_by_downloadKey!(download_link_hash).destroy }
 
-        it "returns 404 if the key is not present" do
+        it "returns 404" do
           get :download, id: download_link_hash
           expect(response).to render_template("curation_concerns/single_use_links_viewer/single_use_error", "layouts/error")
         end
@@ -50,16 +50,16 @@ describe CurationConcerns::SingleUseLinksViewerController do
     end
 
     describe "GET 'show'" do
-      it "and_return http success" do
+      it "renders the file set's show page and deletes the link from the database" do
         get 'show', id: show_link_hash
         expect(response).to be_success
         expect(assigns[:presenter].id).to eq file.id
         expect { SingleUseLink.find_by_downloadKey!(show_link_hash) }.to raise_error ActiveRecord::RecordNotFound
       end
 
-      context "and the key is not found" do
+      context "when the key is not found" do
         before { SingleUseLink.find_by_downloadKey!(show_link_hash).destroy }
-        it "returns 404 if the key is not present" do
+        it "returns 404" do
           get :show, id: show_link_hash
           expect(response).to render_template("curation_concerns/single_use_links_viewer/single_use_error", "layouts/error")
         end

data/spec/factories/single_use_links.rb

@@ -0,0 +1,13 @@
+FactoryGirl.define do
+  factory :single_use_link do
+    factory :show_link do
+      itemId 'fs-id'
+      path '/concerns/generic_work/1234'
+    end
+
+    factory :download_link do
+      itemId 'fs-id'
+      path '/downloads/1234'
+    end
+  end
+end

data/spec/features/embargo_spec.rb

@@ -23,7 +23,7 @@ feature 'embargo' do
       click_button 'Create Generic work'
 
       # chosen embargo date is on the show page
-      expect(page).to have_content(future_date.to_datetime.iso8601.sub(/\+00:00/, 'Z'))
+      expect(page).to have_content(future_date.to_date.to_formatted_s(:standard))
 
       click_link 'Edit This Generic Work'
       click_link 'Embargo Management Page'
@@ -34,7 +34,7 @@ feature 'embargo' do
       fill_in 'until', with: later_future_date.to_s
 
       click_button 'Update Embargo'
-      expect(page).to have_content(later_future_date.iso8601)
+      expect(page).to have_content(later_future_date.to_date.to_formatted_s(:standard))
     end
   end
 

data/spec/features/lease_spec.rb

@@ -20,7 +20,7 @@ feature 'leases' do
       click_button 'Create Generic work'
 
       # chosen lease date is on the show page
-      expect(page).to have_content(future_date.to_datetime.iso8601.sub(/\+00:00/, 'Z'))
+      expect(page).to have_content(future_date.to_date.to_formatted_s(:standard))
 
       click_link 'Edit This Generic Work'
       click_link 'Lease Management Page'
@@ -31,7 +31,7 @@ feature 'leases' do
       fill_in 'until', with: later_future_date.to_s
 
       click_button 'Update Lease'
-      expect(page).to have_content(later_future_date.iso8601) # new lease date is displayed in message
+      expect(page).to have_content(later_future_date.to_date.to_formatted_s(:standard)) # new lease date is displayed in message
     end
   end
 

data/spec/javascripts/fixtures/sul_table.html

@@ -0,0 +1,29 @@
+<table class="table table-striped file_set single-use-links">
+  <caption class="table-heading"><h2>Single-Use Links</h2></caption>
+  <thead>
+    <tr><th>Link</th><th>Key</th><th>Expires</th><th>Actions</th></tr>
+  </thead>
+  <tbody data-url="/single_use_link/generated/fs-id">
+    <tr>
+      <td>Download</td>
+      <td>key</td>
+      <td>timestamp</td>
+      <td>
+        <button class="btn btn-xs btn-default copy-single-use-link" 
+                data-clipboard-text="download-link" data-toggle="tooltip" 
+                data-placement="bottom" title="Copied!">
+          Copy to Clipboard
+        </button>
+        <a class="btn btn-xs btn-danger delete-single-use-link" href="/single_use_link/fs-id/delete/key">
+          Delete
+        </a>
+      </td>
+    </tr>
+  </tbody>
+</table>
+
+<div class="form_actions">
+  <a class="btn btn-default generate-single-use-link" href="/single_use_link/generate/fs-id">
+    Generate A Link
+  </a>
+</div>

data/spec/javascripts/single_use_links_spec.coffee

@@ -0,0 +1,51 @@
+describe "Single Use Links manager", ->
+  sul_manager = null
+  beforeEach () ->
+    loadFixtures('sul_table.html')
+    sul_manager = $.fn.singleUseLinks()
+    jasmine.Ajax.install()
+  afterEach () ->
+    jasmine.Ajax.uninstall()  
+
+  describe "#reload_table", ->
+    request = null
+
+    it "replaces the table's content with html data", ->
+      jasmine.Ajax.stubRequest('/single_use_link/generated/fs-id').andReturn({
+        "status": 200, 
+        "contentType": 'text/plain',
+        "responseText": 'updated table contents'
+      });
+
+      sul_manager.reload_table()
+      request = jasmine.Ajax.requests.mostRecent()
+      expect(request.responseText).toEqual("updated table contents")
+
+  describe "#create_link", ->
+    request = null
+
+    it "requests a new link", ->
+      jasmine.Ajax.stubRequest('/single_use_link/generate/fs-id').andReturn({
+        "status": 200, 
+        "contentType": 'text/plain',
+        "responseText": 'created a link'
+      });
+
+      sul_manager.create_link($('.generate-single-use-link'))
+      request = jasmine.Ajax.requests.mostRecent()
+      expect(request.responseText).toEqual("created a link")
+
+  describe "#delete_link", ->
+    request = null
+
+    it "removes the link from the table", ->
+      jasmine.Ajax.stubRequest('/single_use_link/fs-id/delete/key').andReturn({
+        "status": 200, 
+        "contentType": 'text/plain',
+        "responseText": 'deleted a link'
+      });
+
+      sul_manager.delete_link($('.delete-single-use-link'))
+      request = jasmine.Ajax.requests.mostRecent()
+      expect(request.responseText).toEqual("deleted a link")
+      expect($("table.single-use-links tbody").html).not.toContain("<tr>")

data/spec/presenters/curation_concerns/file_set_presenter_spec.rb

@@ -77,4 +77,11 @@ describe CurationConcerns::FileSetPresenter do
       it { is_expected.to eq 'File' }
     end
   end
+
+  describe "single_use_links" do
+    it "returns ActiveRecord::Relation of all single use links for the file set" do
+      expect(SingleUseLink).to receive(:where).with(itemId: presenter.id)
+      presenter.single_use_links
+    end
+  end
 end

data/spec/renderers/curation_concerns/renderers/date_attribute_renderer_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe CurationConcerns::Renderers::DateAttributeRenderer do
+  subject { Nokogiri::HTML(renderer.render) }
+  let(:expected) { Nokogiri::HTML(tr_content) }
+
+  describe "#attribute_to_html" do
+    context 'with embargo release date' do
+      let(:field) { :embargo_release_date }
+      let(:renderer) { described_class.new(field, ['2013-03-14T00:00:00Z']) }
+      let(:tr_content) {%(
+      <tr><th>Embargo release date</th>
+      <td><ul class="tabular">
+      <li class="attribute embargo_release_date">03/14/2013</li>
+      </ul></td></tr>
+      )}
+      it { expect(renderer).not_to be_microdata(field) }
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+
+    context 'with lease expiration date' do
+      let(:field) { :lease_expiration_date }
+      let(:renderer) { described_class.new(field, ['2013-03-14T00:00:00Z']) }
+      let(:tr_content) {%(
+      <tr><th>Lease expiration date</th>
+      <td><ul class="tabular">
+      <li class="attribute lease_expiration_date">03/14/2013</li>
+      </ul></td></tr>
+      )}
+      it { expect(renderer).not_to be_microdata(field) }
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+  end
+end

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

@@ -130,4 +130,37 @@ describe 'curation_concerns/file_sets/show.html.erb', type: :view do
       expect(rendered).to have_selector '.attribute.description', text: 'Lorem ipsum'
     end
   end
+
+  describe 'single use links' do
+    before do
+      allow(presenter).to receive(:single_use_links).and_return(links)
+      controller.params = { id: presenter.id }
+      render
+    end
+
+    context "when links are present" do
+      let(:show_link)     { create(:show_link) }
+      let(:download_link) { create(:download_link) }
+      let(:links)         { [show_link, download_link] }
+
+      it "renders single use links for the file set" do
+        expect(rendered).to have_selector('td', text: 'Show')
+        expect(rendered).to have_selector('td', text: 'Download')
+        expect(rendered).to have_selector('td', text: show_link.downloadKey)
+        expect(rendered).to have_selector('td', text: show_link.expires)
+        expect(rendered).to have_link("Generate Download Link")
+        expect(rendered).to have_link("Generate Show Link")
+      end
+    end
+
+    context "when no links are present" do
+      let(:links) { [] }
+
+      it "renders a table without links" do
+        expect(rendered).to have_selector('td', text: 'No links have been generated')
+        expect(rendered).to have_link("Generate Download Link")
+        expect(rendered).to have_link("Generate Show Link")
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: curation_concerns
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.2.0
 platform: ruby
 authors:
 - Matt Zumwalt
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-12 00:00:00.000000000 Z
+date: 2016-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hydra-head
@@ -318,6 +318,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.10'
+- !ruby/object:Gem::Dependency
+  name: clipboard-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.5'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.5'
 - !ruby/object:Gem::Dependency
   name: solr_wrapper
   requirement: !ruby/object:Gem::Requirement
@@ -659,6 +673,7 @@ files:
 - app/assets/javascripts/curation_concerns/file_manager/save_manager.es6
 - app/assets/javascripts/curation_concerns/file_manager/sorting.es6
 - app/assets/javascripts/curation_concerns/fileupload.js
+- app/assets/javascripts/curation_concerns/single_use_links_manager.js
 - app/assets/javascripts/curation_concerns/uploader.js
 - app/assets/javascripts/modernizr.js
 - app/assets/stylesheets/curation_concerns.scss
@@ -787,6 +802,7 @@ files:
 - app/presenters/curation_concerns/work_show_presenter.rb
 - app/renderers/curation_concerns/renderers/attribute_renderer.rb
 - app/renderers/curation_concerns/renderers/configured_microdata.rb
+- app/renderers/curation_concerns/renderers/date_attribute_renderer.rb
 - app/renderers/curation_concerns/renderers/faceted_attribute_renderer.rb
 - app/renderers/curation_concerns/renderers/linked_attribute_renderer.rb
 - app/renderers/curation_concerns/renderers/rights_attribute_renderer.rb
@@ -932,6 +948,8 @@ files:
 - app/views/curation_concerns/file_sets/_form.html.erb
 - app/views/curation_concerns/file_sets/_rights_modal.html.erb
 - app/views/curation_concerns/file_sets/_show_actions.html.erb
+- app/views/curation_concerns/file_sets/_single_use_link_rows.html.erb
+- app/views/curation_concerns/file_sets/_single_use_links.html.erb
 - app/views/curation_concerns/file_sets/edit.html.erb
 - app/views/curation_concerns/file_sets/jq_upload.json.jbuilder
 - app/views/curation_concerns/file_sets/media_display/_audio.html.erb
@@ -949,6 +967,7 @@ files:
 - app/views/curation_concerns/operations/index.html.erb
 - app/views/curation_concerns/operations/show.html.erb
 - app/views/curation_concerns/permissions/confirm.html.erb
+- app/views/curation_concerns/single_use_links/index.html.erb
 - app/views/curation_concerns/single_use_links_viewer/show.html.erb
 - app/views/curation_concerns/single_use_links_viewer/single_use_error.html.erb
 - app/views/embargoes/_embargo_history.html.erb
@@ -1090,6 +1109,7 @@ files:
 - spec/factories/file_sets.rb
 - spec/factories/generic_works.rb
 - spec/factories/operations.rb
+- spec/factories/single_use_links.rb
 - spec/factories/users.rb
 - spec/features/add_file_spec.rb
 - spec/features/catalog_search_spec.rb
@@ -1120,11 +1140,13 @@ files:
 - spec/javascripts/fixtures/.gitkeep
 - spec/javascripts/fixtures/file_manager_member.html
 - spec/javascripts/fixtures/save_button.html
+- spec/javascripts/fixtures/sul_table.html
 - spec/javascripts/helpers/jasmine-jquery.js
 - spec/javascripts/helpers/mock-ajax.js
 - spec/javascripts/helpers/test_responses.js
 - spec/javascripts/jasmine_spec.rb
 - spec/javascripts/save_manager_spec.coffee
+- spec/javascripts/single_use_links_spec.coffee
 - spec/javascripts/support/jasmine.yml
 - spec/javascripts/support/jasmine_helper.rb
 - spec/jobs/audit_job_spec.rb
@@ -1167,6 +1189,7 @@ files:
 - spec/presenters/embargo_presenter_spec.rb
 - spec/presenters/lease_presenter_spec.rb
 - spec/renderers/curation_concerns/renderers/attribute_renderer_spec.rb
+- spec/renderers/curation_concerns/renderers/date_attribute_renderer_spec.rb
 - spec/renderers/curation_concerns/renderers/faceted_attribute_renderer_spec.rb
 - spec/renderers/curation_concerns/renderers/linked_attribute_renderer_spec.rb
 - spec/renderers/curation_concerns/renderers/rights_attribute_renderer_spec.rb
@@ -1295,6 +1318,7 @@ test_files:
 - spec/factories/file_sets.rb
 - spec/factories/generic_works.rb
 - spec/factories/operations.rb
+- spec/factories/single_use_links.rb
 - spec/factories/users.rb
 - spec/features/add_file_spec.rb
 - spec/features/catalog_search_spec.rb
@@ -1325,11 +1349,13 @@ test_files:
 - spec/javascripts/fixtures/.gitkeep
 - spec/javascripts/fixtures/file_manager_member.html
 - spec/javascripts/fixtures/save_button.html
+- spec/javascripts/fixtures/sul_table.html
 - spec/javascripts/helpers/jasmine-jquery.js
 - spec/javascripts/helpers/mock-ajax.js
 - spec/javascripts/helpers/test_responses.js
 - spec/javascripts/jasmine_spec.rb
 - spec/javascripts/save_manager_spec.coffee
+- spec/javascripts/single_use_links_spec.coffee
 - spec/javascripts/support/jasmine.yml
 - spec/javascripts/support/jasmine_helper.rb
 - spec/jobs/audit_job_spec.rb
@@ -1372,6 +1398,7 @@ test_files:
 - spec/presenters/embargo_presenter_spec.rb
 - spec/presenters/lease_presenter_spec.rb
 - spec/renderers/curation_concerns/renderers/attribute_renderer_spec.rb
+- spec/renderers/curation_concerns/renderers/date_attribute_renderer_spec.rb
 - spec/renderers/curation_concerns/renderers/faceted_attribute_renderer_spec.rb
 - spec/renderers/curation_concerns/renderers/linked_attribute_renderer_spec.rb
 - spec/renderers/curation_concerns/renderers/rights_attribute_renderer_spec.rb