curation_concerns 0.7.0 → 0.8.0

data/spec/javascripts/helpers/test_responses.js

@@ -0,0 +1,12 @@
+var TestResponses = {
+  file_manager_member: {
+    success: {
+      status: 200,
+      responseText: '{}'
+    },
+    failure: {
+      status: 500,
+      responseText: '{}'
+    }
+  }
+};

data/spec/javascripts/jasmine_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+require 'rake'
+
+# Run the jasmine tests by running the jasmine:ci rake command and parses the output for failures.
+# The spec will fail if any jasmine tests fails.
+describe "Jasmine" do
+  it "expects all jasmine tests to pass" do
+    load_rake_environment ["#{jasmine_path}/lib/jasmine/tasks/jasmine.rake"]
+    jasmine_out = run_task 'jasmine:ci'
+    if jasmine_out.include? "0 failures"
+      js_specs_count = Dir['spec/javascripts/**/*_spec.js*'].count + Dir['spec/javascripts/**/*_spec.coffee*'].count
+      puts "#{jasmine_out.match(/\n(.+) specs?/)[1]} jasmine specs run (in #{js_specs_count} jasmine test files)"
+    else
+      puts "\n\n************************  Jasmine Output *************"
+      puts jasmine_out
+      puts "************************  Jasmine Output *************\n\n"
+    end
+    expect(jasmine_out).to include "0 failures"
+    expect(jasmine_out).to_not include "\n0 specs"
+  end
+end
+
+def jasmine_path
+  Gem.loaded_specs['jasmine'].full_gem_path
+end

data/spec/javascripts/save_manager_spec.coffee

@@ -0,0 +1,84 @@
+describe "FileManager Save Button", ->
+  save_manager = null
+  handler = null
+  deferred_result = null
+  beforeEach () ->
+    loadFixtures('save_button.html')
+    save_manager = new SaveManager
+    deferred_result = $.Deferred()
+    handler = {
+      persist: () =>
+        deferred_result
+    }
+  describe "#push_changed", ->
+    it "marks that object as changed", ->
+      save_manager.push_changed(handler)
+
+      expect(save_manager.is_changed).toEqual(true)
+    it "enables the save button", ->
+      save_manager.push_changed(handler)
+
+      expect($("button.disabled").length).toEqual(0)
+    it "doesn't add it twice", ->
+      save_manager.push_changed(handler)
+      save_manager.push_changed(handler)
+
+      expect(save_manager.elements).toEqual([handler])
+  describe "#mark_unchanged", ->
+    describe "when it was never marked changed", ->
+      it "no-ops", ->
+        save_manager.mark_unchanged(handler)
+
+        expect(save_manager.elements).toEqual([])
+    describe "when it was previously marked changed", ->
+      it "removes it", ->
+        save_manager.push_changed(handler)
+        save_manager.mark_unchanged(handler)
+
+        expect(save_manager.is_changed).toEqual(false)
+      it "disables the save button", ->
+        save_manager.push_changed(handler)
+        save_manager.mark_unchanged(handler)
+
+        expect($("button.disabled").length).toEqual(1)
+  describe "#persist", ->
+    it "is called by clicking the save button", ->
+      spyOn(save_manager, "persist").and.callThrough()
+      save_manager.push_changed(handler)
+
+      $("button").click()
+      expect(save_manager.persist).toHaveBeenCalled()
+    it "sets the text to be saving...", ->
+      save_manager.push_changed(handler)
+      save_manager.persist()
+
+      expect($("button").hasClass("disabled")).toEqual(true)
+      expect($("button").text()).toEqual("Saving...")
+    it "calls persist on each registered handler", ->
+      spyOn(handler, "persist").and.callThrough()
+      save_manager.push_changed(handler)
+      save_manager.persist()
+
+      expect(handler.persist).toHaveBeenCalled()
+    describe "when the changed elements are done persisting", ->
+      it "resets everything", ->
+        save_manager.push_changed(handler)
+        save_manager.persist()
+
+        # Resolve the promise - like an Ajax request.
+        deferred_result.resolve()
+
+        expect($("button").hasClass("disabled")).toEqual(true)
+        expect($("button").text()).toEqual("Save")
+        expect(save_manager.is_changed).toEqual(false)
+    describe "when the changed elements have failed to persist", ->
+      it "fixes the text but keeps the button enabled", ->
+        save_manager.push_changed(handler)
+        save_manager.persist()
+
+        # Reject the promise.
+        deferred_result.reject()
+
+        expect($("button").hasClass("disabled")).toEqual(false)
+        expect($("button").text()).toEqual("Save")
+        expect(save_manager.is_changed).toEqual(true)

data/spec/javascripts/support/jasmine.yml

@@ -0,0 +1,136 @@
+# src_files
+#
+# Return an array of filepaths relative to src_dir to include before jasmine specs.
+# Default: []
+#
+# EXAMPLE:
+#
+# src_files:
+#   - lib/source1.js
+#   - lib/source2.js
+#   - dist/**/*.js
+#
+src_files:
+  - assets/application.js
+  - assets/jquery.js
+
+# stylesheets
+#
+# Return an array of stylesheet filepaths relative to src_dir to include before jasmine specs.
+# Default: []
+#
+# EXAMPLE:
+#
+# stylesheets:
+#   - css/style.css
+#   - stylesheets/*.css
+#
+stylesheets:
+  - assets/application.css
+
+# helpers
+#
+# Return an array of filepaths relative to spec_dir to include before jasmine specs.
+# Default: ["helpers/**/*.js"]
+#
+# EXAMPLE:
+#
+# helpers:
+#   - helpers/**/*.js
+#
+helpers:
+  - 'helpers/**/*.js'
+
+# spec_files
+#
+# Return an array of filepaths relative to spec_dir to include.
+# Default: ["**/*[sS]pec.js"]
+#
+# EXAMPLE:
+#
+# spec_files:
+#   - **/*[sS]pec.js
+#
+spec_files:
+  - '**/*[sS]pec.js'
+  - '**/*[sS]pec.coffee'
+
+# src_dir
+#
+# Source directory path. Your src_files must be returned relative to this path. Will use root if left blank.
+# Default: project root
+#
+# EXAMPLE:
+#
+# src_dir: public
+#
+src_dir:
+
+# spec_dir
+#
+# Spec directory path. Your spec_files must be returned relative to this path.
+# Default: spec/javascripts
+#
+# EXAMPLE:
+#
+# spec_dir: spec/javascripts
+#
+spec_dir:
+
+# spec_helper
+#
+# Ruby file that Jasmine server will require before starting.
+# Returned relative to your root path
+# Default spec/javascripts/support/jasmine_helper.rb
+#
+# EXAMPLE:
+#
+# spec_helper: spec/javascripts/support/jasmine_helper.rb
+#
+spec_helper: spec/javascripts/support/jasmine_helper.rb
+
+# boot_dir
+#
+# Boot directory path. Your boot_files must be returned relative to this path.
+# Default: Built in boot file
+#
+# EXAMPLE:
+#
+# boot_dir: spec/javascripts/support/boot
+#
+boot_dir:
+
+# boot_files
+#
+# Return an array of filepaths relative to boot_dir to include in order to boot Jasmine
+# Default: Built in boot file
+#
+# EXAMPLE
+#
+# boot_files:
+#   - '**/*.js'
+#
+boot_files:
+
+# rack_options
+#
+# Extra options to be passed to the rack server
+# by default, Port and AccessLog are passed.
+#
+# This is an advanced options, and left empty by default
+#
+# EXAMPLE
+#
+# rack_options:
+#   server: 'thin'
+
+# random
+#
+# Run specs in semi-random order.
+# Default: false
+#
+# EXAMPLE:
+#
+# random: true
+#
+random:

data/spec/javascripts/support/jasmine_helper.rb

@@ -0,0 +1,15 @@
+# Use this file to set/override Jasmine configuration options
+# You can remove it if you don't need it.
+# This file is loaded *after* jasmine.yml is interpreted.
+#
+# Example: using a different boot file.
+# Jasmine.configure do |config|
+#   config.boot_dir = '/absolute/path/to/boot_dir'
+#   config.boot_files = lambda { ['/absolute/path/to/boot_dir/file.js'] }
+# end
+#
+# Example: prevent PhantomJS auto install, uses PhantomJS already on your path.
+# Jasmine.configure do |config|
+#   config.prevent_phantom_js_auto_install = true
+# end
+#

data/spec/routing/route_spec.rb

@@ -35,6 +35,10 @@ describe 'Routes', type: :routing do
     it '*not*s route to index' do
       expect(get: 'concern/generic_works').not_to route_to(controller: 'curation_concerns/generic_works', action: 'index')
     end
+
+    it 'routes to file_manager' do
+      expect(get: 'concern/generic_works/6/file_manager').to route_to(controller: 'curation_concerns/generic_works', action: 'file_manager', id: '6')
+    end
   end
 
   describe 'Permissions' do

data/spec/support/rake_support.rb

@@ -0,0 +1,41 @@
+require 'rake'
+
+module RakeHelper
+  def load_rake_environment(files)
+    @rake = Rake::Application.new
+    Rake.application = @rake
+    Rake::Task.define_task(:environment)
+    files.each { |file| load file }
+  end
+
+  def run_task(task, arg = nil)
+    capture_stdout_stderr do
+      @rake[task].invoke(arg)
+    end
+  end
+
+  # saves original $stdout in variable
+  # set $stdout as local instance of StringIO
+  # yields to code execution
+  # returns the local instance of StringIO
+  # resets $stdout to original value
+  def capture_stdout_stderr
+    out = StringIO.new
+    err = StringIO.new
+    $stdout = out
+    $stderr = err
+    begin
+      yield
+    rescue SystemExit => e
+      puts "error = #{e.inspect}"
+    end
+    return "Output: #{out.string}\n Errors:#{err.string}"
+  ensure
+    $stdout = STDOUT
+    $stdout = STDERR
+  end
+
+  RSpec.configure do |config|
+    config.include RakeHelper
+  end
+end

data/spec/views/catalog/index.html.erb_spec.rb

@@ -45,7 +45,7 @@ describe 'catalog/index.html.erb' do
       render
       page = Capybara::Node::Simple.new(rendered)
       expect(page).to have_link(collection.title)
-      expect(page).to have_content 'List of files deposited'
+      expect(page).to have_content 'List of items deposited'
     end
   end
 
@@ -56,7 +56,7 @@ describe 'catalog/index.html.erb' do
       render
       page = Capybara::Node::Simple.new(rendered)
       expect(page).to have_link(collection.title)
-      expect(page).to have_content 'List of files deposited'
+      expect(page).to have_content 'List of items deposited'
     end
   end
 end

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

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+RSpec.describe "curation_concerns/base/file_manager.html.erb" do
+  let(:members) { [file_set] }
+  let(:file_set) { CurationConcerns::FileSetPresenter.new(solr_doc, nil) }
+  let(:solr_doc) do
+    SolrDocument.new(
+      resource.to_solr.merge(
+        id: "test",
+        title_tesim: "Test",
+        thumbnail_path_ss: "/test/image/path.jpg",
+        label_tesim: "file_name.tif"
+      )
+    )
+  end
+  let(:resource) { FactoryGirl.build(:file_set) }
+
+  let(:parent) { FactoryGirl.build(:generic_work) }
+  let(:parent_solr_doc) do
+    SolrDocument.new(parent.to_solr.merge(id: "resource"), nil)
+  end
+  let(:parent_presenter) do
+    CurationConcerns::WorkShowPresenter.new(parent_solr_doc, nil)
+  end
+
+  let(:blacklight_config) { CatalogController.new.blacklight_config }
+
+  before do
+    allow(parent_presenter).to receive(:file_presenters).and_return([file_set])
+    assign(:presenter, parent_presenter)
+    # Blacklight nonsense
+    allow(view).to receive(:dom_class) { '' }
+    allow(view).to receive(:blacklight_config).and_return(blacklight_config)
+    allow(view).to receive(:blacklight_configuration_context).and_return(CatalogController.new.send(:blacklight_configuration_context))
+    allow(view).to receive(:search_session).and_return({})
+    allow(view).to receive(:current_search_session).and_return(nil)
+    allow(view).to receive(:curation_concern).and_return(parent)
+    render
+  end
+
+  it "has a bulk edit header" do
+    expect(rendered).to include "<h1>#{I18n.t('file_manager.link_text')}</h1>"
+  end
+
+  it "displays each file set's label" do
+    expect(rendered).to have_selector "input[name='file_set[title][]'][type='text'][value='#{file_set}']"
+  end
+
+  it "displays each file set's file name" do
+    expect(rendered).to have_content "file_name.tif"
+  end
+
+  it "has a link to edit each file set" do
+    expect(rendered).to have_selector('a[href="/concern/file_sets/test"]')
+  end
+
+  it "has a link back to parent" do
+    expect(rendered).to have_link "Test title", href: curation_concerns_generic_work_path(id: "resource")
+  end
+
+  it "has thumbnails for each resource" do
+    expect(rendered).to have_selector("img[src='/test/image/path.jpg']")
+  end
+
+  it "renders a form for each member" do
+    expect(rendered).to have_selector("form", count: members.length)
+  end
+
+  it "renders an input for titles" do
+    expect(rendered).to have_selector("input[name='file_set[title][]']")
+  end
+end

data/tasks/jasmine.rake

@@ -0,0 +1,18 @@
+require 'engine_cart'
+require 'jasmine'
+def jasmine_path
+  Gem.loaded_specs['jasmine'].full_gem_path
+end
+import "#{jasmine_path}/lib/jasmine/tasks/jasmine.rake"
+namespace :curation_concerns do
+  task :jasmine do
+    EngineCart.load_application!
+    Rake::Task["jasmine"].invoke
+  end
+  namespace :jasmine do
+    task :ci do
+      EngineCart.load_application!
+      Rake::Task["jasmine:ci"].invoke
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: curation_concerns
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Matt Zumwalt
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-10 00:00:00.000000000 Z
+date: 2016-02-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hydra-head
@@ -88,14 +88,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: 0.8.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: 0.8.0
 - !ruby/object:Gem::Dependency
   name: hydra-editor
   requirement: !ruby/object:Gem::Requirement
@@ -138,6 +138,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: sprockets-es6
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: solr_wrapper
   requirement: !ruby/object:Gem::Requirement
@@ -362,6 +376,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.3.4
+- !ruby/object:Gem::Dependency
+  name: jasmine
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 'A Rails Engine that allows an application to CRUD CurationConcern objects
   (a.k.a. "Works") '
 email:
@@ -393,6 +421,11 @@ files:
 - app/assets/javascripts/curation_concerns/curation_concerns.js
 - app/assets/javascripts/curation_concerns/embargoes.js
 - app/assets/javascripts/curation_concerns/facet_mine.js
+- app/assets/javascripts/curation_concerns/file_manager/affix.es6
+- app/assets/javascripts/curation_concerns/file_manager/list_toggle.es6
+- app/assets/javascripts/curation_concerns/file_manager/member.es6
+- 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/uploader.js
 - app/assets/javascripts/modernizr.js
@@ -413,6 +446,7 @@ files:
 - app/assets/stylesheets/curation_concerns/modules/classify_work.scss
 - app/assets/stylesheets/curation_concerns/modules/embargoes.scss
 - app/assets/stylesheets/curation_concerns/modules/emphatic_action_area.scss
+- app/assets/stylesheets/curation_concerns/modules/file_manager.scss
 - app/assets/stylesheets/curation_concerns/modules/forms.scss
 - app/assets/stylesheets/curation_concerns/modules/icons.scss
 - app/assets/stylesheets/curation_concerns/modules/multi_value_fields.scss
@@ -539,6 +573,11 @@ files:
 - app/views/curation_concerns/base/_attribute_rows.html.erb
 - app/views/curation_concerns/base/_attributes.html.erb
 - app/views/curation_concerns/base/_collections.html.erb
+- app/views/curation_concerns/base/_file_manager_actions.html.erb
+- app/views/curation_concerns/base/_file_manager_attributes.html.erb
+- app/views/curation_concerns/base/_file_manager_extra_tools.html.erb
+- app/views/curation_concerns/base/_file_manager_member.html.erb
+- app/views/curation_concerns/base/_file_manager_thumbnail.html.erb
 - app/views/curation_concerns/base/_form.html.erb
 - app/views/curation_concerns/base/_form_additional_information.html.erb
 - app/views/curation_concerns/base/_form_descriptive_fields.erb
@@ -563,6 +602,7 @@ files:
 - app/views/curation_concerns/base/_versioning.html.erb
 - app/views/curation_concerns/base/_visibility.html.erb
 - app/views/curation_concerns/base/edit.html.erb
+- app/views/curation_concerns/base/file_manager.html.erb
 - app/views/curation_concerns/base/new.html.erb
 - app/views/curation_concerns/base/show.html.erb
 - app/views/curation_concerns/base/show.json.jbuilder
@@ -726,6 +766,17 @@ files:
 - spec/inputs/multifile_input_spec.rb
 - spec/inputs/select_with_help_input_spec.rb
 - spec/inputs/select_with_modal_help_input_spec.rb
+- spec/javascripts/file_manager_member_spec.coffee
+- spec/javascripts/fixtures/.gitkeep
+- spec/javascripts/fixtures/file_manager_member.html
+- spec/javascripts/fixtures/save_button.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/support/jasmine.yml
+- spec/javascripts/support/jasmine_helper.rb
 - spec/jobs/active_fedora_id_based_job_spec.rb
 - spec/jobs/audit_job_spec.rb
 - spec/jobs/characterize_job_spec.rb
@@ -790,6 +841,7 @@ files:
 - spec/support/features/session_helpers.rb
 - spec/support/input_support.rb
 - spec/support/matchers/api_responses.rb
+- spec/support/rake_support.rb
 - spec/support/shared/shared_examples_has_dc_metadata.rb
 - spec/support/shared/shared_examples_is_embargoable.rb
 - spec/support/shared/shared_examples_with_access_rights.rb
@@ -802,6 +854,7 @@ files:
 - spec/views/curation_concerns/base/_attributes.html.erb_spec.rb
 - spec/views/curation_concerns/base/_form_permission.html.erb_spec.rb
 - spec/views/curation_concerns/base/_show_actions.html.erb_spec.rb
+- spec/views/curation_concerns/base/file_manager.html.erb_spec.rb
 - spec/views/curation_concerns/base/show.html.erb_spec.rb
 - spec/views/curation_concerns/base/show.json.jbuilder_spec.rb
 - spec/views/curation_concerns/file_sets/_file_set.html.erb_spec.rb
@@ -815,6 +868,7 @@ files:
 - spec/views/shared/_my_actions.html.erb_spec.rb
 - spec/views/single_use_links/new_download.html.erb_spec.rb
 - spec/views/single_use_links_viewer/show.html.erb_spec.rb
+- tasks/jasmine.rake
 - tasks/release.rake
 - vendor/assets/images/ui-bg_glass_100_fdf5ce_1x400.png
 - vendor/assets/images/ui-bg_highlight-soft_100_eeeeee_1x100.png
@@ -904,6 +958,17 @@ test_files:
 - spec/inputs/multifile_input_spec.rb
 - spec/inputs/select_with_help_input_spec.rb
 - spec/inputs/select_with_modal_help_input_spec.rb
+- spec/javascripts/file_manager_member_spec.coffee
+- spec/javascripts/fixtures/.gitkeep
+- spec/javascripts/fixtures/file_manager_member.html
+- spec/javascripts/fixtures/save_button.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/support/jasmine.yml
+- spec/javascripts/support/jasmine_helper.rb
 - spec/jobs/active_fedora_id_based_job_spec.rb
 - spec/jobs/audit_job_spec.rb
 - spec/jobs/characterize_job_spec.rb
@@ -968,6 +1033,7 @@ test_files:
 - spec/support/features/session_helpers.rb
 - spec/support/input_support.rb
 - spec/support/matchers/api_responses.rb
+- spec/support/rake_support.rb
 - spec/support/shared/shared_examples_has_dc_metadata.rb
 - spec/support/shared/shared_examples_is_embargoable.rb
 - spec/support/shared/shared_examples_with_access_rights.rb
@@ -980,6 +1046,7 @@ test_files:
 - spec/views/curation_concerns/base/_attributes.html.erb_spec.rb
 - spec/views/curation_concerns/base/_form_permission.html.erb_spec.rb
 - spec/views/curation_concerns/base/_show_actions.html.erb_spec.rb
+- spec/views/curation_concerns/base/file_manager.html.erb_spec.rb
 - spec/views/curation_concerns/base/show.html.erb_spec.rb
 - spec/views/curation_concerns/base/show.json.jbuilder_spec.rb
 - spec/views/curation_concerns/file_sets/_file_set.html.erb_spec.rb