blacklight-spotlight 0.4.1 → 0.5.0

data/spec/features/javascript/preview_block_spec.rb

@@ -1,30 +1,29 @@
-require "spec_helper"
+require 'spec_helper'
 
-feature "Block preview" do
-  let(:exhibit) { FactoryGirl.create(:exhibit) }
+feature 'Block preview' do
+  let(:exhibit) { FactoryGirl.create(:default_exhibit) }
   let(:exhibit_curator) { FactoryGirl.create(:exhibit_curator, exhibit: exhibit) }
   let(:feature_page) do
     FactoryGirl.create(
       :feature_page,
-      title: "FeaturePage1",
+      title: 'FeaturePage1',
       exhibit: exhibit
     )
   end
 
-  before do 
+  before do
     login_as exhibit_curator
     visit spotlight.edit_exhibit_feature_page_path(exhibit, feature_page)
   end
 
-  scenario "should allow you to preview a widget", js: true do
-
+  scenario 'should allow you to preview a widget', js: true do
     add_widget 'solr_documents'
-    fill_in_typeahead_field with: "dq287tq6352"
+    fill_in_typeahead_field with: 'dq287tq6352'
 
     # display the title as the primary caption
     within('.primary-caption') do
-      check("Primary caption")
-      select("Title", from: 'primary-caption-field')
+      check('Primary caption')
+      select('Title', from: 'primary-caption-field')
     end
 
     # Preview page
@@ -32,8 +31,8 @@ feature "Block preview" do
     # verify that the page was previewed
     expect(page).to have_css('.preview')
     # verify that the item + image widget is displaying an image from the document.
-    within(:css, ".preview") do
-      expect(page).to have_css "img"
+    within(:css, '.preview') do
+      expect(page).to have_css 'img'
       expect(page).to have_content "L'AMERIQUE"
     end
   end

data/spec/features/javascript/rule_block_spec.rb

@@ -1,22 +1,22 @@
-require "spec_helper"
+require 'spec_helper'
 
-describe "Horizontal rule block", type: :feature, js: true do
-  let(:exhibit) { FactoryGirl.create(:exhibit) }
+describe 'Horizontal rule block', type: :feature, js: true do
+  let(:exhibit) { FactoryGirl.create(:default_exhibit) }
   let(:exhibit_curator) { FactoryGirl.create(:exhibit_curator, exhibit: exhibit) }
   let!(:feature_page) { FactoryGirl.create(:feature_page, exhibit: exhibit) }
   before { login_as exhibit_curator }
 
-  it "should allow the user to select which image in a multi image object to display" do
-    exhibit.home_page.content = "[]"
+  it 'allows the user to select which image in a multi image object to display' do
+    exhibit.home_page.content = '[]'
     exhibit.home_page.save
 
     visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
-    click_link "Edit"
+    click_link 'Edit'
 
     add_widget 'rule'
 
     save_page
 
-    expect(page).to have_css("hr")
+    expect(page).to have_css('hr')
   end
 end

data/spec/features/javascript/search_config_admin_spec.rb

@@ -0,0 +1,144 @@
+require 'spec_helper'
+
+feature 'Search Configuration Administration', js: true do
+  let(:exhibit) { FactoryGirl.create(:default_exhibit) }
+  let(:exhibit_curator) { FactoryGirl.create(:exhibit_curator, exhibit: exhibit) }
+  before { login_as exhibit_curator }
+
+  describe 'search fields' do
+    it 'allows the curator to disable all search fields' do
+      visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
+      expect(page).to have_css 'select#search_field'
+
+      click_link exhibit_curator.email
+      within '#user-util-collapse .dropdown' do
+        click_link 'Dashboard'
+      end
+      click_link 'Search'
+      click_link 'Options'
+
+      uncheck 'Enable field-based search'
+
+      click_button 'Save changes'
+
+      expect(page).to have_content('The exhibit was successfully updated.')
+
+      expect(page).not_to have_css 'select#search_field'
+    end
+
+    it 'allows the curator to update search field options' do
+      visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
+      click_link exhibit_curator.email
+      within '#user-util-collapse .dropdown' do
+        click_link 'Dashboard'
+      end
+      click_link 'Search'
+
+      click_link 'Options'
+
+      within('#nested-search-fields') do
+        expect(page).to have_css("#blacklight_configuration_search_fields_title_label[type='hidden']", visible: false)
+        expect(page).not_to have_css("#blacklight_configuration_search_fields_title_label[type='text']")
+        click_link('Title')
+        expect(page).not_to have_css("#blacklight_configuration_search_fields_title_label[type='hidden']")
+        expect(page).to have_css("#blacklight_configuration_search_fields_title_label[type='text']")
+        fill_in 'blacklight_configuration_search_fields_title_label', with: 'My Title Label'
+      end
+
+      click_button 'Save changes'
+
+      expect(page).to have_content('The exhibit was successfully updated.')
+      expect(page).to have_select 'Search in', with_options: ['My Title Label']
+    end
+  end
+
+  describe 'facets' do
+    it 'allows us to update the label with edit-in-place' do
+      input_id = 'blacklight_configuration_facet_fields_genre_ssim_label'
+      visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
+      click_link exhibit_curator.email
+      within '#user-util-collapse .dropdown' do
+        click_link 'Dashboard'
+      end
+      click_link 'Search'
+
+      click_link 'Facets'
+
+      facet = find('.edit-in-place', text: 'Genre')
+      expect(page).not_to have_content('Topic')
+      expect(page).to have_css("input##{input_id}", visible: false)
+
+      facet.click
+
+      expect(page).to have_css("input##{input_id}", visible: true)
+
+      fill_in(input_id, with: 'Topic')
+
+      click_button 'Save changes'
+      click_link 'Facets'
+
+      expect(page).to have_content('The exhibit was successfully updated.')
+
+      expect(page).not_to have_content('Genre')
+      expect(page).to have_content('Topic')
+    end
+  end
+
+  describe 'results' do
+    it 'updates search result options' do
+      visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
+      click_link exhibit_curator.email
+      within '#user-util-collapse .dropdown' do
+        click_link 'Dashboard'
+      end
+      click_link 'Search'
+
+      click_link 'Results'
+
+      uncheck 'List'
+
+      choose '20'
+
+      click_button 'Save changes'
+
+      expect(page).to have_content('The exhibit was successfully updated.')
+
+      click_link 'Results'
+
+      expect(field_labeled('List')).to_not be_checked
+      expect(field_labeled('Gallery')).to be_checked
+
+      expect(field_labeled('20')).to be_checked
+      expect(field_labeled('10')).to_not be_checked
+    end
+    it 'updates Sort field result options' do
+      visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
+      click_link exhibit_curator.email
+      within '#user-util-collapse .dropdown' do
+        click_link 'Dashboard'
+      end
+      click_link 'Search'
+
+      click_link 'Results'
+
+      within('#nested-sort-fields') do
+        expect(page).to have_css("#blacklight_configuration_sort_fields_title_label[type='hidden']", visible: false)
+        expect(page).not_to have_css("#blacklight_configuration_sort_fields_title_label[type='text']")
+        click_link('Title')
+        expect(page).not_to have_css("#blacklight_configuration_sort_fields_title_label[type='hidden']")
+        expect(page).to have_css("#blacklight_configuration_sort_fields_title_label[type='text']")
+        fill_in 'blacklight_configuration_sort_fields_title_label', with: 'My Title Label'
+      end
+
+      click_button 'Save changes'
+
+      expect(page).to have_content('The exhibit was successfully updated.')
+
+      click_link 'Results'
+
+      within('#nested-sort-fields') do
+        expect(page).to have_css('h3', text: 'My Title Label')
+      end
+    end # Sort field
+  end # results tab
+end

data/spec/features/javascript/search_context_spec.rb

@@ -1,26 +1,26 @@
-require "spec_helper"
+require 'spec_helper'
 
-feature "Search contexts" do
-  let(:exhibit) { FactoryGirl.create(:exhibit) }
+feature 'Search contexts' do
+  let(:exhibit) { FactoryGirl.create(:default_exhibit) }
   let(:exhibit_curator) { FactoryGirl.create(:exhibit_curator, exhibit: exhibit) }
   let(:feature_page) do
     FactoryGirl.create(
       :feature_page,
-      title: "FeaturePage1",
+      title: 'FeaturePage1',
       exhibit: exhibit
     )
   end
   before { login_as exhibit_curator }
 
-  scenario "should add context breadcrumbs back to the home page when navigating to an item from the home page", :js => true do
+  scenario 'should add context breadcrumbs back to the home page when navigating to an item from the home page', js: true do
     exhibit.home_page.content = [
       {
-        type: "solr_documents", 
-        data: { 
+        type: 'solr_documents',
+        data: {
           item: {
             dq287tq6352: {
-              id: "dq287tq6352", 
-              display: "true"
+              id: 'dq287tq6352',
+              display: 'true'
             }
           }
         }
@@ -30,56 +30,55 @@ feature "Search contexts" do
     visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
 
     # verify that the item + image widget is displaying an image from the document.
-    within(:css, ".items-block") do
-      expect(page).to have_css(".thumbnail")
-      expect(page).to have_css(".thumbnail a img")
-      expect(page).not_to have_css(".title")
+    within(:css, '.items-block') do
+      expect(page).to have_css('.thumbnail')
+      expect(page).to have_css('.thumbnail a img')
+      expect(page).not_to have_css('.title')
     end
 
-    find('.thumbnail a').trigger("click")
+    find('.thumbnail a').trigger('click')
 
-    expect(page).to have_selector '.breadcrumb a', text: "Home"
+    expect(page).to have_selector '.breadcrumb a', text: 'Home'
   end
 
-  scenario "should add context breadcrumb back to the feature page when navigating to an item from a feature page", :js => true do
+  scenario 'should add context breadcrumb back to the feature page when navigating to an item from a feature page', js: true do
     feature_page.content = [
-        {
-          type: "solr_documents", 
-          data: { 
-            item: {
-              dq287tq6352: {
-                id: "dq287tq6352", 
-                display: "true"
-              }
+      {
+        type: 'solr_documents',
+        data: {
+          item: {
+            dq287tq6352: {
+              id: 'dq287tq6352',
+              display: 'true'
             }
           }
-        }].to_json
+        }
+      }].to_json
     feature_page.save
 
     visit spotlight.exhibit_feature_page_path(exhibit, feature_page)
 
     # verify that the item + image widget is displaying an image from the document.
-    within(:css, ".items-block") do
-      expect(page).to have_css(".thumbnail")
-      expect(page).to have_css(".thumbnail a img")
-      expect(page).not_to have_css(".title")
+    within(:css, '.items-block') do
+      expect(page).to have_css('.thumbnail')
+      expect(page).to have_css('.thumbnail a img')
+      expect(page).not_to have_css('.title')
     end
-    
-    find('.thumbnail a').trigger("click")
 
-    expect(page).to have_selector '.breadcrumb a', text: "Home"
-    expect(page).to have_link "FeaturePage1", href: spotlight.exhibit_feature_page_path(exhibit, feature_page)
+    find('.thumbnail a').trigger('click')
+
+    expect(page).to have_selector '.breadcrumb a', text: 'Home'
+    expect(page).to have_link 'FeaturePage1', href: spotlight.exhibit_feature_page_path(exhibit, feature_page)
   end
 
-  scenario "should add context breadcrumbs back to the browse page when navigating to an item from a browse page", :js => true do
-    search = Spotlight::Search.create! exhibit_id: exhibit.id, title: "Some Saved Search", on_landing_page: true
+  scenario 'should add context breadcrumbs back to the browse page when navigating to an item from a browse page', js: true do
+    search = Spotlight::Search.create! exhibit_id: exhibit.id, title: 'Some Saved Search', published: true
     visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
-    click_link "Browse"
-    click_link "Some Saved Search"
-    click_link "A MAP of AMERICA from the latest and best Observations"
-    expect(page).to have_link "Home"
-    expect(page).to have_link "Browse"
-    expect(page).to have_link "Some Saved Search", href: spotlight.exhibit_browse_path(exhibit, search)
-
+    click_link 'Browse'
+    click_link 'Some Saved Search'
+    click_link 'A MAP of AMERICA from the latest and best Observations'
+    expect(page).to have_link 'Home'
+    expect(page).to have_link 'Browse'
+    expect(page).to have_link 'Some Saved Search', href: spotlight.exhibit_browse_path(exhibit, search)
   end
 end

data/spec/features/main_navigation_spec.rb

@@ -1,52 +1,52 @@
 require 'spec_helper'
 
-describe "Main navigation labels are settable", :type => :feature do
-  let!(:exhibit) { FactoryGirl.create(:exhibit) }
+describe 'Main navigation labels are settable', type: :feature do
+  let!(:exhibit) { FactoryGirl.create(:default_exhibit) }
   let!(:about) { FactoryGirl.create(:about_page, exhibit: exhibit, published: true) }
   before do
     about_nav = exhibit.main_navigations.about
-    about_nav.label = "New About Label"
+    about_nav.label = 'New About Label'
     about_nav.save
     browse_nav = exhibit.main_navigations.browse
-    browse_nav.label = "New Browse Label"
+    browse_nav.label = 'New Browse Label'
     browse_nav.save
     search = exhibit.searches.first
-    search.on_landing_page = true
+    search.published = true
     search.save
     exhibit.reload
   end
-  
-  it "should have the configured about and browse navigation labels" do
+
+  it 'has the configured about and browse navigation labels' do
     visit spotlight.exhibit_path(exhibit)
-    expect(page).to have_css(".navbar-nav li", text: "New About Label")
-    expect(page).to have_css(".navbar-nav li", text: "New Browse Label")
+    expect(page).to have_css('.navbar-nav li', text: 'New About Label')
+    expect(page).to have_css('.navbar-nav li', text: 'New Browse Label')
   end
-  it "should have the configured about page label in the sidebar" do
+  it 'has the configured about page label in the sidebar' do
     visit spotlight.exhibit_about_page_path(exhibit, about)
-    expect(page).to have_css("#sidebar h4", text: "New About Label")
+    expect(page).to have_css('#sidebar h4', text: 'New About Label')
   end
-  it "should have the configured about page label visible in the breadcrumb" do
+  it 'has the configured about page label visible in the breadcrumb' do
     visit spotlight.exhibit_about_page_path(exhibit, about)
-    expect(page).to have_css(".breadcrumb li", text: "New About Label")
+    expect(page).to have_css('.breadcrumb li', text: 'New About Label')
   end
-  it "should have the configured browse page label visible in the breadcrumb of the browse index page" do
+  it 'has the configured browse page label visible in the breadcrumb of the browse index page' do
     visit spotlight.exhibit_browse_index_path(exhibit, exhibit.searches.first)
-    expect(page).to have_content("New Browse Label")
-    expect(page).to have_css(".breadcrumb li", text: "New Browse Label")
+    expect(page).to have_content('New Browse Label')
+    expect(page).to have_css('.breadcrumb li', text: 'New Browse Label')
   end
-  it "should have the configured browse page label visible in the breadcrumb of the browse show page" do
+  it 'has the configured browse page label visible in the breadcrumb of the browse show page' do
     visit spotlight.exhibit_browse_path(exhibit, exhibit.searches.first)
-    expect(page).to have_content("New Browse Label")
-    expect(page).to have_css(".breadcrumb li", text: "New Browse Label")
+    expect(page).to have_content('New Browse Label')
+    expect(page).to have_css('.breadcrumb li', text: 'New Browse Label')
   end
-  it 'should not display any main navigation menu items that are configured to not display' do
+  it 'does not display any main navigation menu items that are configured to not display' do
     about_nav = exhibit.main_navigations.about
     about_nav.display = false
     about_nav.save
     visit spotlight.exhibit_path(exhibit)
-    expect(page).to_not have_css(".navbar-nav li", text: "New About Label")
+    expect(page).to_not have_css('.navbar-nav li', text: 'New About Label')
     about_nav = exhibit.main_navigations.about
     about_nav.display = true
     about_nav.save
   end
-end
+end

data/spec/features/metadata_admin_spec.rb

@@ -1,16 +1,16 @@
-require "spec_helper"
+require 'spec_helper'
 
-describe "Metadata Administration", :type => :feature do
-  let(:exhibit) { FactoryGirl.create(:exhibit) }
+describe 'Metadata Administration', type: :feature do
+  let(:exhibit) { FactoryGirl.create(:default_exhibit) }
   let(:exhibit_curator) { FactoryGirl.create(:exhibit_curator, exhibit: exhibit) }
   before { login_as exhibit_curator }
 
-  describe "edit" do
-    it "should display the metadata edit page" do
-      visit spotlight.exhibit_edit_metadata_path(exhibit)
-      expect(page).to have_css("h1 small", text: "Metadata")
+  describe 'edit' do
+    it 'displays the metadata edit page' do
+      visit spotlight.edit_exhibit_metadata_configuration_path(exhibit)
+      expect(page).to have_css('h1 small', text: 'Metadata')
       within("[data-id='language_ssm']") do
-        expect(page).to have_css("td", text: "Language")
+        expect(page).to have_css('td', text: 'Language')
       end
     end
   end