blacklight 8.2.2 → 8.3.0

data/spec/features/facet_missing_spec.rb

@@ -14,12 +14,12 @@ RSpec.describe "Facet missing" do
       visit root_path
 
       within "#facet-subject_geo_ssim" do
-        click_link "[Missing]"
+        click_on "[Missing]"
       end
 
       within "#facet-subject_geo_ssim" do
-        expect(page).to have_selector("span.selected", text: "[Missing")
-        expect(page).to have_selector("span.facet-count.selected", text: "13")
+        expect(page).to have_css("span.selected", text: "[Missing")
+        expect(page).to have_css("span.facet-count.selected", text: "13")
       end
 
       within "#sortAndPerPage" do
@@ -36,14 +36,14 @@ RSpec.describe "Facet missing" do
       visit root_path
 
       within "#facet-subject_geo_ssim" do
-        click_link "[Missing]"
+        click_on "[Missing]"
       end
 
       within "#facet-subject_geo_ssim" do
-        click_link "remove"
+        click_on "remove"
       end
 
-      expect(page).not_to have_link "remove"
+      expect(page).to have_no_link "remove"
       expect(page).to have_content("Welcome!")
     end
   end
@@ -53,14 +53,14 @@ RSpec.describe "Facet missing" do
       visit root_path
 
       within "#facet-subject_geo_ssim" do
-        click_link "[Missing]"
+        click_on "[Missing]"
       end
 
       within ".filter-subject_geo_ssim" do
-        click_link "Remove constraint Region: [Missing]"
+        click_on "Remove constraint Region: [Missing]"
       end
 
-      expect(page).not_to have_link "remove"
+      expect(page).to have_no_link "remove"
       expect(page).to have_content("Welcome!")
     end
   end

data/spec/features/facets_spec.rb

@@ -3,31 +3,31 @@
 RSpec.describe "Facets" do
   it "shows a single facet's values" do
     visit facet_catalog_path("language_ssim")
-    expect(page).to have_selector ".modal-title", text: "Language"
-    expect(page).to have_selector ".facet-select", text: "Tibetan"
+    expect(page).to have_css ".modal-title", text: "Language"
+    expect(page).to have_css ".facet-select", text: "Tibetan"
   end
 
   it "paginates through a facet's values" do
     visit facet_catalog_path("subject_ssim")
-    expect(page).to have_selector '.facet-values li:first', text: "Japanese drama"
+    expect(page).to have_css '.facet-values li:first', text: "Japanese drama"
     expect(page).to have_link "A-Z Sort"
-    expect(page).to have_selector '.sort-options .active', text: "Numerical Sort"
+    expect(page).to have_css '.sort-options .active', text: "Numerical Sort"
     within ".modal-footer" do
       click_on "Next »"
     end
-    expect(page).to have_selector '.facet-values li:first', text: "Jewish law"
+    expect(page).to have_css '.facet-values li:first', text: "Jewish law"
     expect(page).to have_link "« Previous"
   end
 
   it "is able to change the facet sort" do
     visit facet_catalog_path("subject_ssim")
-    expect(page).to have_selector '.facet-values li:first', text: "Japanese drama"
+    expect(page).to have_css '.facet-values li:first', text: "Japanese drama"
     within ".modal-footer" do
       click_on "A-Z Sort"
     end
-    expect(page).to have_selector '.facet-values li:first', text: "Accident insurance"
+    expect(page).to have_css '.facet-values li:first', text: "Accident insurance"
     expect(page).to have_link "Numerical Sort"
-    expect(page).to have_selector '.sort-options .active', text: "A-Z Sort"
+    expect(page).to have_css '.sort-options .active', text: "A-Z Sort"
   end
 
   it "is able to sort more facet window by letter" do
@@ -35,44 +35,44 @@ RSpec.describe "Facets" do
     within ".modal-footer" do
       click_on "A-Z Sort"
     end
-    expect(page).to have_selector '.facet-values li:first', text: "Accident insurance"
+    expect(page).to have_css '.facet-values li:first', text: "Accident insurance"
     expect(page).to have_css '.facet-values li', count: 20
     click_on 'B'
-    expect(page).to have_selector '.facet-values li:first', text: "Buddhism"
+    expect(page).to have_css '.facet-values li:first', text: "Buddhism"
     expect(page).to have_css '.facet-values li', count: 1
     click_on 'T'
-    expect(page).to have_selector '.facet-values li:first', text: "Teaching"
+    expect(page).to have_css '.facet-values li:first', text: "Teaching"
     expect(page).to have_css '.facet-values li', count: 4
     click_on 'All'
-    expect(page).to have_selector '.facet-values li:first', text: "Accident insurance"
+    expect(page).to have_css '.facet-values li:first', text: "Accident insurance"
     expect(page).to have_css '.facet-values li', count: 20
     find(:css, ".facet-pagination.bottom").click_on "Numerical Sort"
-    expect(page).to have_selector '.facet-values li:first', text: "Japanese drama"
+    expect(page).to have_css '.facet-values li:first', text: "Japanese drama"
     expect(page).to have_css '.facet-values li', count: 20
   end
 
-  it 'is able to expand facets when javascript is enabled', js: true do
+  it 'is able to expand facets when javascript is enabled', :js do
     visit root_path
 
     expect(page).to have_css('#facet-format', visible: false)
 
-    page.find('h3.facet-field-heading button', text: 'Format').click
+    click_on 'Format'
 
     sleep(1) # let facet animation finish and wait for it to potentially re-collapse
 
     expect(page).to have_css('#facet-format', visible: true) # assert that it didn't re-collapse
   end
 
-  it 'is able to expand pivot facets when javascript is enabled', js: true do
+  it 'is able to expand pivot facets when javascript is enabled', :js do
     visit root_path
 
-    page.find('h3.facet-field-heading button', text: 'Pivot Field').click
+    click_on 'Pivot Field'
 
     within '#facet-example_pivot_field' do
       expect(page).to have_css('.facet-leaf-node', text: "Book 30", normalize_ws: true)
-      expect(page).not_to have_css('.facet-select', text: 'Tibetan')
+      expect(page).to have_no_css('.facet-select', text: 'Tibetan')
       page.find('.facet-toggle-handle').click
-      click_link 'Tibetan'
+      click_on 'Tibetan'
     end
 
     expect(page).to have_css('.constraint-value', text: 'Format Book')
@@ -83,7 +83,8 @@ RSpec.describe "Facets" do
     it 'changes to the button on button click in Firefox' do
       pending 'Capybara::NotSupportedByDriverError: Capybara::Driver::Base#evaluate_script'
       visit root_path
-      page.find('h3.facet-field-heading button', text: 'Format').click
+      click_on 'Format'
+
       focused_element_data_target = page.evaluate_script("document.activeElement")['data-bs-target']
       expect(focused_element_data_target).to eq '#facet-format'
     end

data/spec/features/modal_spec.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 RSpec.describe 'Modal' do
-  it 'can open and dismiss the email modal', js: true do
+  it 'can open and dismiss the email modal', :js do
     visit solr_document_path('2007020969')
     expect(page).to have_no_selector 'dialog#blacklight-modal'
-    click_link 'Email'
-    expect(page).to have_selector 'dialog#blacklight-modal'
-    click_button '×'
+    click_on 'Email'
+    expect(page).to have_css 'dialog#blacklight-modal'
+    click_on '×'
     expect(page).to have_no_selector 'dialog#blacklight-modal'
   end
 end

data/spec/features/record_view_spec.rb

@@ -15,13 +15,13 @@ RSpec.describe "Record View" do
     expect(page).to have_content "E99.D2 H437 2008"
     tmp_value = Capybara.ignore_hidden_elements
     Capybara.ignore_hidden_elements = false
-    expect(page).to have_selector("link[rel=alternate]")
+    expect(page).to have_css("link[rel=alternate]")
     Capybara.ignore_hidden_elements = tmp_value
   end
 
   it "does not display blank titles" do
     visit solr_document_path('2008305903')
-    expect(page).not_to have_content "More Information:"
+    expect(page).to have_no_content "More Information:"
   end
 
   it "does not display vernacular records" do

data/spec/features/search_context_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 RSpec.describe "Search Results context" do
-  it "passes the current search id through", js: true do
+  it "passes the current search id through", :js do
     search_for ''
     click_on 'Pluvial nectar of blessings'
     search_id = Search.last.id.to_s
@@ -16,11 +16,11 @@ RSpec.describe "Search Results context" do
     expect(prev['data-context-href']).to eq "/catalog/2004310986/track?counter=8&document_id=2004310986&search_id=#{search_id}"
   end
 
-  it "redirects context urls to the original url", js: true do
+  it "redirects context urls to the original url", :js do
     search_for ''
     first('.index_title a').click
     expect(page).to have_content "« Previous | 1 of 30 | Next »"
-    expect(page.current_url).not_to have_content "/track"
+    expect(page.current_url).to have_no_content "/track"
   end
 
   it 'shows "Back to Search" and "Start Over links"' do
@@ -38,12 +38,12 @@ RSpec.describe "Search Results context" do
 
     # Go to the number 9 result separately
     visit '/catalog/2003546302'
-    expect(page).not_to have_selector '.page-links'
-    expect(page).not_to have_content "« Previous | 10 of 30 | Next »"
+    expect(page).to have_no_css '.page-links'
+    expect(page).to have_no_content "« Previous | 10 of 30 | Next »"
   end
 
   context "navigating between search results using context pagination" do
-    it "updates the back to search link with the current search pagination context", js: true do
+    it "updates the back to search link with the current search pagination context", :js do
       search_for ''
       expect(page).to have_content "1 - 10"
       find_all('.index_title a').last.click

data/spec/features/search_crawler_spec.rb

@@ -2,27 +2,27 @@
 
 RSpec.describe "Search History Page" do
   describe "crawler search" do
-    let(:original_proc) { ::CatalogController.blacklight_config.crawler_detector }
+    let(:original_proc) { CatalogController.blacklight_config.crawler_detector }
 
     before do
-      ::CatalogController.blacklight_config.crawler_detector = ->(req) { req.env['HTTP_USER_AGENT'] =~ /Googlebot/ }
+      CatalogController.blacklight_config.crawler_detector = ->(req) { req.env['HTTP_USER_AGENT'] =~ /Googlebot/ }
     end
 
     after do
-      ::CatalogController.blacklight_config.crawler_detector = original_proc
+      CatalogController.blacklight_config.crawler_detector = original_proc
     end
 
     it "remembers human searches" do
       visit root_path
       fill_in "q", with: 'chicken'
-      expect { click_button 'search' }.to change(Search, :count).by(1)
+      expect { click_on 'search' }.to change(Search, :count).by(1)
     end
 
     it "doesn't remember bot searches" do
       page.driver.header('User-Agent', 'Mozilla/5.0 (compatible; Googlebot/2.1; +http://www.google.com/bot.html)')
       visit root_path
       fill_in "q", with: 'chicken'
-      expect { click_button 'search' }.not_to change(Search, :count)
+      expect { click_on 'search' }.not_to change(Search, :count)
     end
   end
 end

data/spec/features/search_filters_spec.rb

@@ -4,138 +4,138 @@ RSpec.describe "Facets" do
   it "works without a search term" do
     visit root_path
     within "#facet-language_ssim" do
-      click_link "Tibetan"
+      click_on "Tibetan"
     end
     within "#sortAndPerPage" do
       expect(page).to have_content "1 - 6 of 6"
     end
 
-    expect(page).to have_selector(".blacklight-language_ssim")
-    expect(page).to have_selector(".blacklight-language_ssim.facet-limit-active")
+    expect(page).to have_css(".blacklight-language_ssim")
+    expect(page).to have_css(".blacklight-language_ssim.facet-limit-active")
 
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "6")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "6")
     end
 
-    click_link "India"
+    click_on "India"
     within ("#sortAndPerPage") do
       expect(page).to have_content "1 - 2 of 2"
     end
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "2")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "2")
     end
     within "#facet-subject_geo_ssim" do
-      expect(page).to have_selector("span.selected", text: "India")
-      expect(page).to have_selector("span.facet-count.selected", text: "2")
+      expect(page).to have_css("span.selected", text: "India")
+      expect(page).to have_css("span.facet-count.selected", text: "2")
     end
   end
 
   it "works in conjunction with a search term" do
     visit root_path
     fill_in "q", with: 'history'
-    click_button 'search'
+    click_on 'search'
     within ("#sortAndPerPage") do
       expect(page).to have_content "1 - 10 of 11"
     end
 
     within "#facet-language_ssim" do
-      click_link "Tibetan"
+      click_on "Tibetan"
     end
     within ("#sortAndPerPage") do
       expect(page).to have_content "1 - 2 of 2"
     end
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "2")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "2")
     end
     within "#appliedParams" do
       expect(page).to have_content "You searched for:"
       expect(page).to have_content "history"
     end
 
-    click_link "2004"
+    click_on "2004"
 
     within ("#sortAndPerPage") do
       expect(page).to have_content "1 entry found"
     end
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "1")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "1")
     end
     within(".blacklight-pub_date_ssim") do
-      expect(page).to have_selector("span.selected", text: "2004")
-      expect(page).to have_selector("span.facet-count.selected", text: "1")
+      expect(page).to have_css("span.selected", text: "2004")
+      expect(page).to have_css("span.facet-count.selected", text: "1")
     end
   end
 
   it "allows removing filters" do
     visit root_path
     within "#facet-language_ssim" do
-      click_link "Tibetan"
+      click_on "Tibetan"
     end
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "6")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "6")
     end
     within "#facet-language_ssim" do
-      click_link 'remove'
+      click_on 'remove'
     end
-    expect(page).not_to have_link 'remove'
+    expect(page).to have_no_link 'remove'
     expect(page).to have_content('Welcome!')
   end
 
   it "retains filters when you change the search term" do
     visit root_path
     fill_in "q", with: 'history'
-    click_button 'search'
+    click_on 'search'
     within "#facet-language_ssim" do
-      click_link 'Tibetan'
+      click_on 'Tibetan'
     end
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "2")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "2")
     end
 
-    click_link '2004'
+    click_on '2004'
 
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "1")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "1")
     end
     within(".blacklight-pub_date_ssim") do
-      expect(page).to have_selector("span.selected", text: "2004")
-      expect(page).to have_selector("span.facet-count.selected", text: "1")
+      expect(page).to have_css("span.selected", text: "2004")
+      expect(page).to have_css("span.facet-count.selected", text: "1")
     end
     fill_in "q", with: 'china'
-    click_button 'search'
+    click_on 'search'
 
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "1")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "1")
     end
     within(".blacklight-pub_date_ssim") do
-      expect(page).to have_selector("span.selected", text: "2004")
-      expect(page).to have_selector("span.facet-count.selected", text: "1")
+      expect(page).to have_css("span.selected", text: "2004")
+      expect(page).to have_css("span.facet-count.selected", text: "1")
     end
   end
 
   it "retains the filters when we change sort" do
     visit root_path
     fill_in "q", with: 'history'
-    click_button 'search'
+    click_on 'search'
     within "#facet-language_ssim" do
-      click_link 'Tibetan'
+      click_on 'Tibetan'
     end
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "2")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "2")
     end
-    click_link 'title'
+    click_on 'title'
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "2")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "2")
     end
     within "#appliedParams" do
       expect(page).to have_content "You searched for:"
@@ -146,20 +146,20 @@ RSpec.describe "Facets" do
   it "retains the filters when we change per page number" do
     visit root_path
     fill_in "q", with: 'history'
-    click_button 'search'
+    click_on 'search'
     within "#facet-language_ssim" do
-      click_link 'Tibetan'
+      click_on 'Tibetan'
     end
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "2")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "2")
     end
     within '#per_page-dropdown' do
-      click_link '20'
+      click_on '20'
     end
     within "#facet-language_ssim" do
-      expect(page).to have_selector("span.selected", text: "Tibetan")
-      expect(page).to have_selector("span.facet-count.selected", text: "2")
+      expect(page).to have_css("span.selected", text: "Tibetan")
+      expect(page).to have_css("span.facet-count.selected", text: "2")
     end
     within "#appliedParams" do
       expect(page).to have_content "You searched for:"
@@ -167,46 +167,46 @@ RSpec.describe "Facets" do
     end
   end
 
-  it "is collapsed when not selected", js: true do
+  it "is collapsed when not selected", :js do
     visit root_path
 
     within(".blacklight-subject_ssim") do
-      expect(page).not_to have_selector(".card-body", visible: true)
+      expect(page).to have_no_css(".card-body", visible: true)
     end
   end
 
-  it "expands when the heading button is clicked", js: true do
+  it "expands when the heading button is clicked", :js do
     visit root_path
 
     within(".blacklight-subject_ssim") do
-      expect(page).not_to have_selector(".card-body", visible: true)
-      find(".card-header button").click
-      expect(page).to have_selector(".card-body", visible: true)
+      expect(page).to have_no_css(".card-body", visible: true)
+      click_on 'Topic'
+      expect(page).to have_css(".card-body", visible: true)
     end
   end
 
-  it "expands when the button is clicked", js: true do
+  it "expands when the button is clicked", :js do
     visit root_path
 
     within(".blacklight-subject_ssim") do
-      expect(page).not_to have_selector(".card-body", visible: true)
+      expect(page).to have_no_css(".card-body", visible: true)
       find(".card-header").click
-      expect(page).to have_selector(".card-body", visible: true)
+      expect(page).to have_css(".card-body", visible: true)
     end
   end
 
-  it "keeps selected facets expanded on page load", js: true do
+  it "keeps selected facets expanded on page load", :js do
     visit root_path
 
     within(".blacklight-subject_ssim") do
       page.find('h3.facet-field-heading', text: 'Topic').click
-      expect(page).to have_selector(".panel-collapse", visible: true)
+      expect(page).to have_css(".panel-collapse", visible: true)
     end
     within(".blacklight-subject_ssim") do
-      click_link "Japanese drama"
+      click_on "Japanese drama"
     end
     within(".blacklight-subject_ssim") do
-      expect(page).to have_selector(".panel-collapse", visible: true)
+      expect(page).to have_css(".panel-collapse", visible: true)
     end
   end
 end

data/spec/features/search_history_spec.rb

@@ -4,7 +4,7 @@ RSpec.describe "Search History Page" do
   describe "navigating from the homepage" do
     it "has a link to the history page" do
       visit root_path
-      click_link 'History'
+      click_on 'History'
       expect(page).to have_content 'Search History'
       expect(page).to have_content 'You have no search history'
     end
@@ -14,18 +14,18 @@ RSpec.describe "Search History Page" do
     before do
       visit root_path
       fill_in "q", with: 'book'
-      click_button 'search'
-      click_link 'History'
+      click_on 'search'
+      click_on 'History'
     end
 
     it "shows searches" do
       expect(page).to have_content 'Your recent searches'
       expect(page).to have_content 'book'
-      expect(page).not_to have_content 'dang'
+      expect(page).to have_no_content 'dang'
       visit root_path
       fill_in "q", with: 'dang'
-      click_button 'search'
-      click_link 'History'
+      click_on 'search'
+      click_on 'History'
       expect(page).to have_content 'book'
       expect(page).to have_content 'dang'
     end
@@ -35,18 +35,18 @@ RSpec.describe "Search History Page" do
     before do
       visit root_path
       fill_in "q", with: 'book'
-      click_button 'search'
+      click_on 'search'
       fill_in "q", with: 'dang'
-      click_button 'search'
-      click_link 'History'
+      click_on 'search'
+      click_on 'History'
     end
 
     it "is able to clear the history" do
-      click_link "Clear Search History"
+      click_on "Clear Search History"
       expect(page).to have_content 'Cleared your search history.'
       expect(page).to have_content 'You have no search history'
-      expect(page).not_to have_content 'book'
-      expect(page).not_to have_content 'dang'
+      expect(page).to have_no_content 'book'
+      expect(page).to have_no_content 'dang'
     end
   end
 end