From c9b6e9cd95f83c6086a361172a60aab0ff0e2fa9 Mon Sep 17 00:00:00 2001 From: Jamie Shark <5520141+jamieshark@users.noreply.github.com> Date: Thu, 29 Aug 2024 14:08:36 -0500 Subject: [PATCH] Add filter param to playground instance, rename test for clarity. --- previews/primer/alpha/select_panel_preview.rb | 5 ++-- .../eventually_local_fetch.html.erb | 3 ++- test/system/alpha/select_panel_test.rb | 25 ++++++++----------- 3 files changed, 16 insertions(+), 17 deletions(-) diff --git a/previews/primer/alpha/select_panel_preview.rb b/previews/primer/alpha/select_panel_preview.rb index 1d7933cd6f..786c264079 100644 --- a/previews/primer/alpha/select_panel_preview.rb +++ b/previews/primer/alpha/select_panel_preview.rb @@ -77,8 +77,9 @@ def local_fetch(open_on_load: false) # # @snapshot interactive # @param open_on_load toggle - def eventually_local_fetch(open_on_load: false) - render_with_template(locals: { open_on_load: open_on_load }) + # @param show_filter toggle + def eventually_local_fetch(open_on_load: false, show_filter: true) + render_with_template(locals: { open_on_load: open_on_load, show_filter: show_filter }) end # @label Remote fetch diff --git a/previews/primer/alpha/select_panel_preview/eventually_local_fetch.html.erb b/previews/primer/alpha/select_panel_preview/eventually_local_fetch.html.erb index 605d22aa05..0935268a61 100644 --- a/previews/primer/alpha/select_panel_preview/eventually_local_fetch.html.erb +++ b/previews/primer/alpha/select_panel_preview/eventually_local_fetch.html.erb @@ -5,7 +5,8 @@ src: select_panel_items_path, select_variant: :multiple, fetch_strategy: :eventually_local, - open_on_load: open_on_load + open_on_load: open_on_load, + show_filter: show_filter, )) do |panel| %> <% panel.with_show_button { "Sci-fi equipment" } %> <% panel.with_footer(show_divider: true) do %> diff --git a/test/system/alpha/select_panel_test.rb b/test/system/alpha/select_panel_test.rb index 41c19021db..9ce82caffc 100644 --- a/test/system/alpha/select_panel_test.rb +++ b/test/system/alpha/select_panel_test.rb @@ -877,6 +877,17 @@ def test_ev_loc_initial_failure refute_selector "[data-target='select-panel.bannerErrorElement']" end + def test_ev_loc_items_load_without_filter + visit_preview(:eventually_local_fetch, show_filter: false) + + wait_for_items_to_load do + click_on_invoker_button + end + + # items should render without error + assert_selector "select-panel ul li" + end + ########## REMOTE TESTS ############ def test_remote_items_load @@ -952,20 +963,6 @@ def test_remote_filter_failure refute_selector "[data-target='select-panel.fragmentErrorElement']" end - def test_filter_input_gate - visit_preview(:playground, show_filter: false) - - click_on_invoker_button - refute_selector "li[data-item-id=item1] [aria-disabled=true]" - - evaluate_multiline_script(<<~JS) - const panel = document.querySelector('select-panel') - panel.disableItem(panel.getItemById('item1')) - JS - - assert_selector "li[data-item-id=item1] [aria-disabled=true]" - end - ########## TAB INDEX TESTS ############ def test_tab_indices