blacklight-spotlight 0.0.2 → 0.0.3

data/spec/helpers/spotlight/admin_title_helper_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe Spotlight::AdminTitleHelper do
+  before do
+    helper.stub(application_name: "Application")
+  end
+
+  describe "#page_title" do
+    it "should set the @page_title ivar" do
+      helper.page_title("Section", "Title")
+      title = helper.instance_variable_get(:@page_title)
+      expect(title).to eq "Section - Title  --  Application"
+    end
+
+    it "should render the section title and the page title" do
+      title = helper.page_title("Section", "Title")
+      expect(title).to have_selector "h1", text: "Section"
+      expect(title).to have_selector "h1 small", text: "Title"
+      
+    end
+  end
+
+  describe "#curation_page_title" do
+    it "should render a page title in the curation section" do
+      title = helper.curation_page_title "Some title"
+      expect(title).to have_selector "h1", text: "Curation"
+      expect(title).to have_selector "h1 small", text: "Some title"
+    end
+  end
+
+  describe "#administration_page_title" do
+    it "should render a page title in the administration section" do
+      title = helper.administration_page_title "Some title"
+      expect(title).to have_selector "h1", text: "Administration"
+      expect(title).to have_selector "h1 small", text: "Some title"
+    end
+  end
+
+  describe "#header_with_count" do
+    it "should merge the title with a count label" do
+      val = helper.header_with_count "some title", 5
+      expect(val).to match /some title /
+      expect(val).to have_selector "span.label", text: 5
+    end
+  end
+
+end

data/spec/helpers/spotlight/application_helper_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe Spotlight::ApplicationHelper do
+  describe "#url_to_tag_facet" do
+    before do
+      helper.stub(current_exhibit: Spotlight::Exhibit.default)
+      helper.stub(blacklight_config: Blacklight::Configuration.new)
+
+      # controller provided helper.
+      helper.stub(:search_action_url) do |*args|
+        spotlight.exhibit_catalog_index_path(helper.current_exhibit, *args)
+      end
+    end
+
+    it "should be a url for a search with the given tag facet" do
+      Spotlight::SolrDocument.stub(solr_field_for_tagger: :exhibit_tags)
+      expect(helper.url_to_tag_facet "tag_value").to eq spotlight.exhibit_catalog_index_path(exhibit_id: 1, f: { exhibit_tags: ['tag_value']})
+    end
+  end
+end

data/spec/helpers/spotlight/crud_link_helpers_spec.rb

@@ -0,0 +1,147 @@
+require 'spec_helper'
+
+module Spotlight
+  describe CrudLinkHelpers do
+    let(:some_model) { Spotlight::FeaturePage.create! exhibit: Spotlight::Exhibit.default }
+    describe "#cancel_link" do
+
+      it "should be a model-specific cancel link" do
+        helper.should_receive(:action_default_value).with(:my_model, :cancel).and_return "Cancel"
+        expect(helper.cancel_link(:my_model, "#")).to have_link "Cancel", href: "#"
+      end
+    end
+
+    describe "#view_link" do
+      before do
+        helper.stub action_default_value: "View"
+      end
+
+      it "should be a model-specific view link" do
+        helper.should_receive(:action_default_value).with(some_model, :view).and_return "View"
+        expect(helper.view_link(some_model)).to have_link "View", href: spotlight.feature_page_path(some_model)
+      end
+
+      it "should accept an explicit link" do
+        expect(helper.view_link(some_model, '#')).to have_link "View", href: "#"
+      end
+
+      it "should accept link_to options" do
+        expect(helper.view_link(some_model, '#', class: 'btn')).to have_selector 'a.btn'
+      end
+    end
+
+    describe "#exhibit_create_link" do
+      let(:current_exhibit) { Spotlight::Exhibit.default }
+      let(:some_model) { Spotlight::FeaturePage.new }
+      before do
+        helper.stub(current_exhibit: current_exhibit)
+        helper.stub action_default_value: "Create"
+      end
+
+      it "should be a model-specific view link" do
+        helper.should_receive(:action_default_value).with(some_model).and_return "Create"
+        expect(helper.exhibit_create_link(some_model)).to have_link "Create", href: spotlight.new_exhibit_feature_page_path(current_exhibit)
+      end
+
+      it "should accept an explicit link" do
+        expect(helper.exhibit_create_link(some_model, '#')).to have_link "Create", href: "#"
+      end
+
+      it "should accept link_to options" do
+        expect(helper.exhibit_create_link(some_model, '#', class: 'btn')).to have_selector 'a.btn'
+      end
+    end
+
+    describe "#edit_link" do
+      before do
+        helper.stub action_default_value: "Edit"
+      end
+
+      it "should be a model-specific view link" do
+        helper.should_receive(:action_default_value).with(some_model).and_return "Edit"
+        expect(helper.edit_link(some_model)).to have_link "Edit", href: spotlight.edit_feature_page_path(some_model)
+      end
+
+      it "should accept an explicit link" do
+        expect(helper.edit_link(some_model, '#')).to have_link "Edit", href: "#"
+      end
+
+      it "should accept link_to options" do
+        expect(helper.edit_link(some_model, '#', class: 'btn')).to have_selector 'a.btn'
+      end
+    end
+
+    describe "#exhibit_edit_link" do
+      let(:current_exhibit) { Spotlight::Exhibit.default }
+      let(:some_model) { ::SolrDocument.new  id: 1 }
+      before do
+        helper.stub(current_exhibit: current_exhibit)
+        helper.stub action_default_value: "Edit"
+      end
+
+      it "should be a model-specific edit link" do
+        helper.should_receive(:action_default_value).with(some_model).and_return "Edit"
+        expect(helper.exhibit_edit_link(some_model)).to have_link "Edit", href: spotlight.edit_exhibit_solr_document_path(current_exhibit, some_model)
+      end
+
+      it "should accept an explicit link" do
+        expect(helper.exhibit_edit_link(some_model, '#')).to have_link "Edit", href: "#"
+      end
+
+      it "should accept link_to options" do
+        expect(helper.exhibit_edit_link(some_model, '#', class: 'btn')).to have_selector 'a.btn'
+      end
+    end
+
+    describe "#delete_link" do
+      before do
+        helper.stub action_default_value: "Delete"
+      end
+
+      it "should be a model-specific view link" do
+        helper.should_receive(:action_default_value).with(some_model, :destroy).and_return "Delete"
+        expect(helper.delete_link(some_model)).to have_link "Delete", href: spotlight.feature_page_path(some_model)
+      end
+
+      it "should accept an explicit link" do
+        expect(helper.delete_link(some_model, '#')).to have_link "Delete", href: "#"
+      end
+
+      it "should accept link_to options" do
+        expect(helper.delete_link(some_model, '#', class: 'btn')).to have_selector 'a.btn'
+      end
+    end
+
+    describe "#action_label" do
+      it "should return the label for an action on a model" do
+        some_model = double
+        helper.should_receive(:action_default_value).with(some_model, :action).and_return "xyz"
+        expect(helper.action_label(some_model, :action)).to eq "xyz"
+      end
+    end
+
+    describe "#action_default_value" do
+      it "should attempt i18n lookups for models" do
+        I18n.should_receive(:t).with(:'helpers.action.spotlight/feature_page.edit', model: some_model.class.model_name.human, default: [:'helpers.action.edit', 'Edit Feature page'])
+        expect(helper.send(:action_default_value, some_model))
+      end
+
+      it "should attempt i18n lookups for unpersisted models" do
+        some_model = Spotlight::FeaturePage.new
+        I18n.should_receive(:t).with(:'helpers.action.spotlight/feature_page.create', model: some_model.class.model_name.human, default: [:'helpers.action.create', 'Create Feature page'])
+        expect(helper.send(:action_default_value, some_model))
+      end
+
+      it "should attempt i18n lookups for models with an explicit action" do
+        I18n.should_receive(:t).with(:'helpers.action.spotlight/feature_page.custom_action', model: some_model.class.model_name.human, default: [:'helpers.action.custom_action', 'Custom action Feature page'])
+        expect(helper.send(:action_default_value, some_model, :custom_action))
+      end
+
+      it "should attempt i18n lookups for symbols" do
+        I18n.should_receive(:t).with(:'helpers.action.my_thing.custom_action', model: :my_thing, default: [:'helpers.action.custom_action', 'Custom action my_thing'])
+        expect(helper.send(:action_default_value, :my_thing, :custom_action))
+
+      end
+    end
+  end
+end

data/spec/helpers/spotlight/pages_helper_spec.rb

@@ -5,7 +5,12 @@ module Spotlight
     let(:blacklight_config) { Blacklight::Configuration.new { |config| config.show.title_field = :abc } }
     let(:titled_document)   { ::SolrDocument.new( :abc => "value" ) }
     let(:untitled_document) { ::SolrDocument.new( :id  => "1234"  ) }
-    before(:each) { helper.stub(:blacklight_config => blacklight_config) }
+    let!(:current_exhibit) { Spotlight::Exhibit.default }
+    let!(:home_page) { current_exhibit.home_page }
+
+    before(:each) do
+      helper.stub(:blacklight_config => blacklight_config)
+    end
 
     describe "has_title?" do
       it "should return true if the title is not the same as the ID" do

data/spec/helpers/spotlight/roles_helper_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe Spotlight::RolesHelper do
+  it "should be a list of options" do
+    expect(helper.roles_for_select).to eq({"Admin"=>"admin", "Curator"=>"curator"})
+  end
+end

data/spec/internal/Gemfile

@@ -1,7 +1,7 @@
 source 'https://rubygems.org'
 
 # Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
-gem 'rails', '4.0.2'
+gem 'rails', '4.0.3'
 
 # Use sqlite3 as the database for Active Record
 gem 'sqlite3'

data/spec/internal/Gemfile.lock

@@ -1,6 +1,6 @@
 GIT
   remote: git://github.com/sul-dlss/sir-trevor-rails.git
-  revision: 33de17d54122c6ed3fb0fcc25b46ca50979d0bcb
+  revision: 5350ffe1536dcd28725b22cc97e86e3042a19145
   specs:
     sir-trevor-rails (0.2.4)
       activesupport (>= 3.0.7)
@@ -18,6 +18,7 @@ PATH
       blacklight (~> 5.1)
       blacklight-gallery
       bootstrap_form
+      breadcrumbs_on_rails (~> 2.3.0)
       cancan
       carrierwave
       mail_form
@@ -28,25 +29,25 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    actionmailer (4.0.2)
-      actionpack (= 4.0.2)
+    actionmailer (4.0.3)
+      actionpack (= 4.0.3)
       mail (~> 2.5.4)
-    actionpack (4.0.2)
-      activesupport (= 4.0.2)
+    actionpack (4.0.3)
+      activesupport (= 4.0.3)
       builder (~> 3.1.0)
       erubis (~> 2.7.0)
       rack (~> 1.5.2)
       rack-test (~> 0.6.2)
-    activemodel (4.0.2)
-      activesupport (= 4.0.2)
+    activemodel (4.0.3)
+      activesupport (= 4.0.3)
       builder (~> 3.1.0)
-    activerecord (4.0.2)
-      activemodel (= 4.0.2)
+    activerecord (4.0.3)
+      activemodel (= 4.0.3)
       activerecord-deprecated_finders (~> 1.0.2)
-      activesupport (= 4.0.2)
+      activesupport (= 4.0.3)
       arel (~> 4.0.0)
     activerecord-deprecated_finders (1.0.3)
-    activesupport (4.0.2)
+    activesupport (4.0.3)
       i18n (~> 0.6, >= 0.6.4)
       minitest (~> 4.2)
       multi_json (~> 1.3)
@@ -72,7 +73,11 @@ GEM
     bootstrap-sass (3.1.1.0)
       sass (~> 3.2)
     bootstrap_form (2.0.1)
+    breadcrumbs_on_rails (2.3.0)
     builder (3.1.4)
+    byebug (2.5.0)
+      columnize (~> 0.3.6)
+      debugger-linecache (~> 1.2.0)
     cancan (1.6.10)
     capybara (2.2.1)
       mime-types (>= 1.16)
@@ -92,16 +97,18 @@ GEM
       coffee-script-source
       execjs
     coffee-script-source (1.7.0)
+    columnize (0.3.6)
     database_cleaner (1.2.0)
+    debugger-linecache (1.2.0)
     deprecation (0.1.0)
       activesupport
-    devise (3.2.2)
+    devise (3.2.3)
       bcrypt-ruby (~> 3.0)
       orm_adapter (~> 0.1)
       railties (>= 3.2.6, < 5)
       thread_safe (~> 0.1)
       warden (~> 1.2.3)
-    devise-guests (0.3.2)
+    devise-guests (0.3.3)
       devise
     diff-lcs (1.2.5)
     erubis (2.7.0)
@@ -140,21 +147,21 @@ GEM
       cliver (~> 0.3.1)
       multi_json (~> 1.0)
       websocket-driver (>= 0.2.0)
-    polyglot (0.3.3)
+    polyglot (0.3.4)
     rack (1.5.2)
     rack-test (0.6.2)
       rack (>= 1.0)
-    rails (4.0.2)
-      actionmailer (= 4.0.2)
-      actionpack (= 4.0.2)
-      activerecord (= 4.0.2)
-      activesupport (= 4.0.2)
+    rails (4.0.3)
+      actionmailer (= 4.0.3)
+      actionpack (= 4.0.3)
+      activerecord (= 4.0.3)
+      activesupport (= 4.0.3)
       bundler (>= 1.3.0, < 2.0)
-      railties (= 4.0.2)
+      railties (= 4.0.3)
       sprockets-rails (~> 2.0.0)
-    railties (4.0.2)
-      actionpack (= 4.0.2)
-      activesupport (= 4.0.2)
+    railties (4.0.3)
+      actionpack (= 4.0.3)
+      activesupport (= 4.0.3)
       rake (>= 0.8.7)
       thor (>= 0.18.1, < 2.0)
     rake (10.1.1)
@@ -166,7 +173,7 @@ GEM
     rspec-core (2.14.7)
     rspec-expectations (2.14.5)
       diff-lcs (>= 1.1.3, < 2.0)
-    rspec-mocks (2.14.5)
+    rspec-mocks (2.14.6)
     rspec-rails (2.14.1)
       actionpack (>= 3.0)
       activemodel (>= 3.0)
@@ -183,7 +190,7 @@ GEM
     sdoc (0.4.0)
       json (~> 1.8)
       rdoc (~> 4.0, < 5.0)
-    sprockets (2.10.1)
+    sprockets (2.11.0)
       hike (~> 1.2)
       multi_json (~> 1.0)
       rack (~> 1.0)
@@ -225,6 +232,7 @@ PLATFORMS
 DEPENDENCIES
   blacklight (~> 5.1)
   blacklight-spotlight!
+  byebug
   coffee-rails (~> 4.0.0)
   database_cleaner
   devise
@@ -233,7 +241,7 @@ DEPENDENCIES
   jbuilder (~> 1.2)
   jquery-rails
   poltergeist
-  rails (= 4.0.2)
+  rails (= 4.0.3)
   rspec-rails
   sass-rails (~> 4.0.0)
   sdoc

data/spec/internal/app/assets/stylesheets/spotlight.css.scss

@@ -1,3 +1,3 @@
-/*
- *= require spotlight/application
- */
+@import 'spotlight/variables_bootstrap';
+@import 'bootstrap';
+@import 'spotlight/application';

data/spec/internal/app/controllers/catalog_controller.rb

@@ -50,12 +50,12 @@ class CatalogController < ApplicationController
     #
     # :show may be set to false if you don't want the facet to be drawn in the 
     # facet bar
-    config.add_facet_field 'genre_sim', :label => 'Genre', :limit => true 
+    config.add_facet_field 'genre_ssim', :label => 'Genre', :limit => true 
     config.add_facet_field 'personal_name_ssm', :label => 'Personal Names', :limit => true 
     config.add_facet_field 'corporate_name_ssm', :label => 'Corporate Names', :limit => true
-    config.add_facet_field 'subject_geographic_sim', :label => 'Geographic' 
-    config.add_facet_field 'subject_temporal_sim', :label => 'Era'  
-    config.add_facet_field 'language_sim', :label => 'Language'  
+    config.add_facet_field 'subject_geographic_ssim', :label => 'Geographic' 
+    config.add_facet_field 'subject_temporal_ssim', :label => 'Era'  
+    config.add_facet_field 'language_ssim', :label => 'Language'  
 
 
     # Have BL send all facet field names to Solr, which has been the default
@@ -85,8 +85,9 @@ class CatalogController < ApplicationController
     config.add_show_field 'personal_name_ssm', :label => 'Personal Names'
     config.add_show_field 'corporate_name_ssm', :label => 'Corporate Names'
 
+    config.add_sort_field 'score desc, sort_title_ssi asc', :label => 'Relevance' 
   end
 
 
 
-end 
+end 

data/spec/internal/app/models/solr_document.rb

@@ -3,6 +3,7 @@ class SolrDocument
 
   include Blacklight::Solr::Document
   include Spotlight::SolrDocument
+  include Spotlight::SolrDocument::AtomicUpdates
 
 
   # self.unique_key = 'id'

data/spec/internal/config/initializers/devise.rb

@@ -4,7 +4,7 @@ Devise.setup do |config|
   # The secret key used by Devise. Devise uses this key to generate
   # random tokens. Changing this key will render invalid all existing
   # confirmation, reset password and unlock tokens in the database.
-  config.secret_key = '9572006cf92ed8411cf48997a510a37f3e08fbf62778d8708d72bf2481556f80f17ee86c9e286b829ac19a1a30270ac5a48de06127aed6040145c9b8112751b7'
+  # config.secret_key = 'e12bd397a2b3029e255d89c8ed747d052483c054c10f9371e75084796a074c6e51bdfd65aea60e75f2523cdb2fe40b8010928cbf4dc462f6c175e6741aaf3bd5'
 
   # ==> Mailer Configuration
   # Configure the e-mail address which will be shown in Devise::Mailer,
@@ -95,14 +95,14 @@ Devise.setup do |config|
   config.stretches = Rails.env.test? ? 1 : 10
 
   # Setup a pepper to generate the encrypted password.
-  # config.pepper = '3733c18589e39a338aad3c9963a857a5c55f1b5b673e235ff21e97c9dc35d4d73ac7e7f345a0b557b0a7b891f65539f530bcb1a6c5096475e856f489207d7c20'
+  # config.pepper = '1dff04b8adc1aae2aa50e34cb7bb4462b509ce0b87627bdc5e0422fabbc3cc1497debcea031c4da66213b3a8b03e1d096e1487f132a644b758c3ad3ee8b73304'
 
   # ==> Configuration for :confirmable
   # A period that the user is allowed to access the website even without
-  # confirming his account. For instance, if set to 2.days, the user will be
-  # able to access the website for two days without confirming his account,
+  # confirming their account. For instance, if set to 2.days, the user will be
+  # able to access the website for two days without confirming their account,
   # access will be blocked just in the third day. Default is 0.days, meaning
-  # the user cannot access the website without confirming his account.
+  # the user cannot access the website without confirming their account.
   # config.allow_unconfirmed_access_for = 2.days
 
   # A period that the user is allowed to confirm their account before their
@@ -134,7 +134,7 @@ Devise.setup do |config|
   # config.rememberable_options = {}
 
   # ==> Configuration for :validatable
-  # Range for password length. Default is 8..128.
+  # Range for password length.
   config.password_length = 8..128
 
   # Email regex used to validate email formats. It simply asserts that