Skip to content

Commit

Permalink
Merge pull request #3490 from alphagov/hidden-clear-filt
Browse files Browse the repository at this point in the history
All content finder: Implement hidden clearable facet
  • Loading branch information
csutter authored Oct 4, 2024
2 parents f18d11c + 8c66d46 commit f5bedae
Show file tree
Hide file tree
Showing 5 changed files with 28 additions and 1 deletion.
2 changes: 2 additions & 0 deletions app/models/hidden_clearable_facet.rb
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
class HiddenClearableFacet < FilterableFacet
attr_reader :value

def initialize(facet, value_hash)
@value = Array(value_hash)
super(facet)
Expand Down
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
<%# TODO: Hidden clearable facet %>
<% hidden_clearable_facet.value.each do |value| %>
<%= hidden_field_tag "#{hidden_clearable_facet.key}[]", value %>
<% end %>
5 changes: 5 additions & 0 deletions features/all_content_finder.feature
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,11 @@ Feature: All content finder ("site search")
Then I can see results for my search
And I can see how many results there are
Scenario: Making a search with a hidden clearable filter
When I search for "search-term" with a hidden clearable manual filter
And I change my search term to "search-term-updated" and submit
Then my search is still filtered by manual
Scenario: Filtering results
When I search all content for "how to walk silly"
And I open the filter panel
Expand Down
13 changes: 13 additions & 0 deletions features/step_definitions/site_search_steps.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,15 @@
visit "/search/all?q=#{search_term}"
end

When(/^I search for "([^"]*)" with a hidden clearable manual filter$/) do |search_term|
visit "/search/all?q=#{search_term}&manual%5B%5D=how-to-be-a-wizard"
end

When(/^I change my search term to "([^"]*)" and submit$/) do |search_term|
fill_in "Search", with: search_term
click_on "Search"
end

Then("I can see results for my search") do
expect(page).to have_link("West London wobbley walk")
expect(page).to have_link("The Gerry Anderson")
Expand All @@ -11,6 +20,10 @@
expect(page).to have_selector("h2", text: "2 results")
end

Then("my search is still filtered by manual") do
expect(page).to have_link("Remove filter Manual: How to be a Wizard", normalize_ws: true)
end

When("I open the filter panel") do
click_on "Filter and sort"
end
Expand Down
5 changes: 5 additions & 0 deletions features/support/document_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,11 @@ def stub_search_api_request_with_manual_filter_all_content_results
filtered_by_manual_all_content_results_json,
including_v2: true,
)
stub_response(
hash_including("q" => "search-term-updated", "filter_manual" => %w[how-to-be-a-wizard]),
filtered_by_manual_all_content_results_json,
including_v2: true,
)
end

def stub_search_api_request_with_filtered_policy_papers_results
Expand Down

0 comments on commit f5bedae

Please sign in to comment.