diff --git a/app/assets/javascripts/toggleableResults.js b/app/assets/javascripts/toggleableResults.js index f5ffdd8..89570ba 100644 --- a/app/assets/javascripts/toggleableResults.js +++ b/app/assets/javascripts/toggleableResults.js @@ -2,6 +2,7 @@ function toggleableResults(searcher) { var toggleableResults = $('#' + searcher + ' [data-behavior="toggleable-results"]'); var threshold = toggleableResults.data('toggleThreshold'); var toggleButton = $('#' + searcher + '-results-toggle-button'); + // if there are 10 search results and the threshold is 3, listItemsBeyondThreshold will contain 7 DOM elements var listItemsBeyondThreshold = toggleableResults.find('li:nth-child(n+' + (threshold + 1) + ')'); listItemsBeyondThreshold.toggle(); diff --git a/spec/features/exhibits_spec.rb b/spec/features/exhibits_spec.rb deleted file mode 100644 index 7c408dc..0000000 --- a/spec/features/exhibits_spec.rb +++ /dev/null @@ -1,51 +0,0 @@ -# frozen_string_literal: true - -require 'rails_helper' - -feature 'Exhibits', js: true do - let(:results) do - 10.times.map do |i| - AbstractSearchService::Result.new.tap do |result| - result.title = "Exhibit title #{i}" - result.link = "https://www.example.com/#{i}" - result.description = "The description for Exhibit #{i}" - end - end - end - - before do - response = instance_double( - AbstractSearchService::Response, - results: results, - total: results.length, - highlighted_facet_values: [], - additional_facet_details: nil - ) - allow_any_instance_of(AbstractSearchService).to receive(:search) - .and_return response - visit quick_search_path - end - - scenario 'is present on index page' do - fill_in 'params-q', with: 'Photography' - click_button 'Search' - - within('#exhibits') do - expect(page).to have_css('ol li', count: 10, visible: :all) - expect(page).to have_css('ol li', count: 3, visible: :visible) - expect(page).to have_css('ol li', count: 7, visible: :hidden) - - click_button 'Show 10 exhibits' - expect(page).to have_css('ol li', count: 10, visible: :visible) - expect(page).to have_css('ol li', count: 0, visible: :hidden) - - click_button 'Show fewer' - expect(page).to have_css('ol li', count: 10, visible: :all) - expect(page).to have_css('ol li', count: 3, visible: :visible) - expect(page).to have_css('ol li', count: 7, visible: :hidden) - - expect(page).to have_button('Show 10 exhibits') - expect(page).not_to have_button('Show fewer') - end - end -end