diff --git a/app/assets/stylesheets/events.css b/app/assets/stylesheets/events.css index 6a52349fe..1d7f2264c 100644 --- a/app/assets/stylesheets/events.css +++ b/app/assets/stylesheets/events.css @@ -122,10 +122,6 @@ text-align: center; } - .events__popup { - margin-block-start: var(--block-space-half); - } - /* Event /* ------------------------------------------------------------------------ */ diff --git a/app/assets/stylesheets/filters.css b/app/assets/stylesheets/filters.css index 7701e72c9..45be28002 100644 --- a/app/assets/stylesheets/filters.css +++ b/app/assets/stylesheets/filters.css @@ -1,8 +1,11 @@ @layer components { .filters { + view-transition-name: "filters"; + z-index: var(--z-popup); + #header:has(&) { position: relative; - z-index: 1; + z-index: var(--z-popup); } .btn { @@ -94,19 +97,8 @@ } } - .panel:is(.filter__popup) { - --panel-size: 100ch; - --panel-padding: var(--block-space); - - inline-size: auto !important; - min-inline-size: var(--panel-size); - max-block-size: calc(90dvh - (2 * var(--block-space-double))); - max-inline-size: calc(100dvw - (2 * var(--block-space-double))); - z-index: 1; - } - .quick-filter { - &:has(input:not(.default-value):checked) { + &:not(.default-value):has(.checked) { .input--select { --input-background: var(--color-selected); } diff --git a/app/assets/stylesheets/popup.css b/app/assets/stylesheets/popup.css index 0d1c27807..addd87f2d 100644 --- a/app/assets/stylesheets/popup.css +++ b/app/assets/stylesheets/popup.css @@ -94,7 +94,7 @@ } &:has(input:checked) { - --btn-background: var(--color-ink-lightest); + --btn-background: transparent; --btn-color: var(--color-ink); .icon { diff --git a/app/controllers/concerns/filter_scoped.rb b/app/controllers/concerns/filter_scoped.rb index 074dd967b..3d5f1fc91 100644 --- a/app/controllers/concerns/filter_scoped.rb +++ b/app/controllers/concerns/filter_scoped.rb @@ -9,6 +9,11 @@ module FilterScoped DEFAULT_PARAMS = { indexed_by: "latest" } def set_filter - @filter = Current.user.filters.from_params params.reverse_merge(**DEFAULT_PARAMS).permit(*Filter::PERMITTED_PARAMS) + @expand_all = params[:expand_all] + if params[:filter_id].present? + @filter = Current.user.filters.find(params[:filter_id]) + else + @filter = Current.user.filters.from_params params.reverse_merge(**DEFAULT_PARAMS).permit(*Filter::PERMITTED_PARAMS) + end end end diff --git a/app/controllers/filters_controller.rb b/app/controllers/filters_controller.rb index ba34c1622..cbb44963b 100644 --- a/app/controllers/filters_controller.rb +++ b/app/controllers/filters_controller.rb @@ -3,16 +3,17 @@ class FiltersController < ApplicationController def create @filter = Current.user.filters.remember filter_params - redirect_to cards_path(@filter.as_params) + redirect_to cards_path(filter_id: @filter.id) end def destroy + filter_params = @filter.as_params @filter.destroy! if request.referer == root_url redirect_to root_path else - redirect_to cards_path(@filter.as_params) + redirect_to cards_path(filter_params) end end diff --git a/app/helpers/filters_helper.rb b/app/helpers/filters_helper.rb index 7b5f5e51f..5d512218d 100644 --- a/app/helpers/filters_helper.rb +++ b/app/helpers/filters_helper.rb @@ -21,6 +21,14 @@ module FiltersHelper hidden_field_tag name, value, id: nil end + def filter_selected_collections_sentence(filter) + if filter.collections.any? + filter.collections.collect { "#{it.name}" }.uniq.sort.to_sentence + else + tag.strong "All collections" + end + end + def filter_selected_collections_label(filter) selected_collections = if filter.collections.any? filter.collections.collect { "#{it.name}" }.uniq.sort.to_sentence diff --git a/app/models/filter/summarized.rb b/app/models/filter/summarized.rb index c08607f32..e56a2dd10 100644 --- a/app/models/filter/summarized.rb +++ b/app/models/filter/summarized.rb @@ -1,16 +1,18 @@ module Filter::Summarized def summary - [ index_summary, tag_summary, assignee_summary, creator_summary, stage_summary, terms_summary ].compact.to_sentence + " #{collection_summary}" + [ index_summary, tag_summary, assignee_summary, creator_summary, stage_summary, terms_summary ].compact.to_sentence end private def index_summary - indexed_by.humanize + unless indexed_by.latest? + indexed_by.humanize + end end def tag_summary if tags.any? - "tagged #{tags.map(&:hashtag).to_choice_sentence}" + "#{tags.map(&:hashtag).to_choice_sentence}" end end @@ -28,12 +30,6 @@ module Filter::Summarized end end - def collection_summary - if collections.any? - "in #{collections.pluck(:name).to_choice_sentence}" - end - end - def terms_summary if terms.any? "matching #{terms.map { |term| %Q("#{term}") }.to_sentence}" diff --git a/app/views/cards/index/_closure_filter.html.erb b/app/views/cards/index/_closure_filter.html.erb index 258e20d4a..da5b5bf22 100644 --- a/app/views/cards/index/_closure_filter.html.erb +++ b/app/views/cards/index/_closure_filter.html.erb @@ -3,7 +3,7 @@ Recently closed - Closed with reason… diff --git a/app/views/cards/index/_workflow_filter.html.erb b/app/views/cards/index/_workflow_filter.html.erb index 49f2cc31b..d453917d5 100644 --- a/app/views/cards/index/_workflow_filter.html.erb +++ b/app/views/cards/index/_workflow_filter.html.erb @@ -7,7 +7,7 @@ <% end %> - In stage(s)… diff --git a/app/views/events/_filter.html.erb b/app/views/events/_filter.html.erb index 6a916d170..71fdecb0c 100644 --- a/app/views/events/_filter.html.erb +++ b/app/views/events/_filter.html.erb @@ -20,7 +20,7 @@ Fizzy <% end %> - <%= tag.dialog class: "fizzy-dialog filter events__popup popup popup--animated panel flex-column align-start fill-white", + <%= tag.dialog class: "fizzy-dialog filter margin-block-start-half popup popup--animated panel flex-column align-start fill-white", data: { action: "turbo:before-cache@document->dialog#close keydown->navigable-list#navigate filter:changed->navigable-list#reset toggle->filter#filter", controller: "filter navigable-list", @@ -32,12 +32,12 @@ <%= render "events/filter/header" %> - Collections <%= form_with url: events_path, method: :get, class: "popup__list", data: { controller: "form" } do |form| %> <% if Current.user.collections.many? %> diff --git a/app/views/events/filter/_custom_collections.html.erb b/app/views/events/filter/_custom_collections.html.erb index 26dad7da2..0ee1af1df 100644 --- a/app/views/events/filter/_custom_collections.html.erb +++ b/app/views/events/filter/_custom_collections.html.erb @@ -1,11 +1,22 @@ -
- <% @filters.each do |filter| %> -
- <%= link_to cards_path(**filter.as_params), class: "btn popup__new popup__item" do %> - <%= icon_tag "filter" %> - <%= filter.summary %> - › - <% end %> +Saved filters +
+ <%= link_to cards_path(expand_all: true), class: "btn popup__new popup__item" do %> + <%= icon_tag "bookmark" %> +
+ Create a new filter
+ › <% end %>
+<% @filters.each do |filter| %> +
+ <%= link_to cards_path(filter_id: filter.id), class: "btn popup__new popup__item" do %> + <%= icon_tag "bookmark" %> +
+ <%= filter_selected_collections_sentence(filter).html_safe %> + <%= filter.summary %> +
+ › + <% end %> +
+<% end %> diff --git a/app/views/filters/_assignees.html.erb b/app/views/filters/_assignees.html.erb index 2b4d684ee..1a1e33926 100644 --- a/app/views/filters/_assignees.html.erb +++ b/app/views/filters/_assignees.html.erb @@ -1,53 +1,85 @@ -<% if filter.assignees.any? %> +<% if filter.assignees.any? || @expand_all %>
- - Assigned to… - <%= form_with url: cards_path, method: :get, class: "popup__list", - data: { controller: "form" } do |form| %> - <% filter.as_params.except(:assignee_ids, :assignment_status).each do |key, value| %> - <%= filter_hidden_field_tag key, value %> - <% end %> + <%= tag.dialog class: "margin-block-start-half popup panel flex-column align-start gap-half fill-white shadow txt-small", data: { + action: "turbo:before-cache@document->dialog#close keydown->navigable-list#navigate filter:changed->navigable-list#reset toggle->filter#filter", + aria: { label: "Assigned to…", aria_description: "Assigned to…" }, + controller: "filter navigable-list", + dialog_target: "dialog", + navigable_list_focus_on_selection_value: false, + navigable_list_actionable_items_value: true } do %> + Assigned to… - <%= link_to cards_path(filter.as_params.except(:assignee_ids, :assignment_status)), class: "btn popup__item" do %> - Clear all - <% end %> - -
- <%= form.check_box "assignment_status", { - checked: filter.assignment_status.unassigned?, - data: { action: "change->form#submit" }, - include_hidden: false, - }, "unassigned" %> - - <%= form.label :assignment_status, "No one", class: "overflow-ellipsis" %> - <%= icon_tag "check", class: "checked flex-item-justify-end" %> -
- - <% User.active.order(:name).each do |user| %> -
- <%= form.check_box "assignee_ids[]", { - checked: filter.assignees.include?(user), - data: { action: "change->form#submit" }, - include_hidden: false, - }, user.id %> - - <%= form.label "assignee_ids[]", user.name, for: dom_id(user, :filter), class: "overflow-ellipsis" %> - <%= icon_tag "check", class: "checked flex-item-justify-end" %> -
- <% end %> + <% if User.active.many? %> + <%= text_field_tag :search, nil, placeholder: "Filter…", class: "input input--transparent txt-small", autofocus: true, + type: "search", autocorrect: "off", autocomplete: "off", data: { "1p-ignore": "true", filter_target: "input", action: "input->filter#filter" } %> <% end %> - + +
    +
  • + <%= form_with url: cards_path, method: :get, class: "full-width", data: { controller: "form" } do |form| %> + <% filter.as_params.except(:assignee_ids, :assignment_status).each do |key, value| %> + <%= filter_hidden_field_tag key, value %> + <% end %> + + <% unless filter.assignment_status.unassigned? %> + <%= hidden_field_tag :assignment_status, "unassigned" %> + <% end %> + + <% if @expand_all %> + <%= hidden_field_tag :expand_all, @expand_all %> + <% end %> + + <%= form.button type: "submit", class: "btn popup__item min-width full-width unpad-inline" do %> + No one + <% if filter.assignment_status.unassigned? %> + <%= icon_tag "check", class: "checked flex-item-justify-end" %> + <% end %> + <% end %> + <% end %> +
    • + <% User.active.order(:name).each do |user| %> +
  • + <%= form_with url: cards_path, method: :get, class: "full-width", data: { controller: "form" } do |form| %> + <% filter.as_params.except(:assignee_ids, :assignment_status).each do |key, value| %> + <%= filter_hidden_field_tag key, value %> + <% end %> + + <% filter.assignees.each do |existing_assignee| %> + <% unless existing_assignee == user %> + <%= hidden_field_tag "assignee_ids[]", existing_assignee.id %> + <% end %> + <% end %> + + <% unless filter.assignees.include?(user) %> + <%= hidden_field_tag "assignee_ids[]", user.id %> + <% end %> + + <% if @expand_all %> + <%= hidden_field_tag :expand_all, @expand_all %> + <% end %> + + <%= form.button type: "submit", class: "btn popup__item min-width full-width unpad-inline" do %> + <%= user.name %> + <% if filter.assignees.include?(user) %> + <%= icon_tag "check", class: "checked flex-item-justify-end" %> + <% end %> + <% end %> + <% end %> +
    • + <% end %> +
+ <% end %>
<% end %> diff --git a/app/views/filters/_creators.html.erb b/app/views/filters/_creators.html.erb index 5c9e48b83..9ba8954e7 100644 --- a/app/views/filters/_creators.html.erb +++ b/app/views/filters/_creators.html.erb @@ -1,56 +1,62 @@ -<% if filter.creators.any? %> +<% if filter.creators.any? || @expand_all %>
- - Added by… - <%= form_with url: cards_path, method: :get, class: "popup__list", - data: { controller: "form" } do |form| %> - <% filter.as_params.except(:creator_ids).each do |key, value| %> - <%= filter_hidden_field_tag key, value %> - <% end %> + <%= tag.dialog class: "margin-block-start-half popup panel flex-column align-start gap-half fill-white shadow txt-small", data: { + action: "turbo:before-cache@document->dialog#close keydown->navigable-list#navigate filter:changed->navigable-list#reset toggle->filter#filter", + aria: { label: "Added by…", aria_description: "Added by…" }, + controller: "filter navigable-list", + dialog_target: "dialog", + navigable_list_focus_on_selection_value: false, + navigable_list_actionable_items_value: true } do %> + Added by… - <%= link_to cards_path(filter.as_params.except(:creator_ids)), class: "btn popup__item" do %> - Clear all - <% end %> - -
- <%= form.check_box :creator_ids, { - multiple: true, - checked: filter.creators.include?(Current.user), - data: { action: "change->form#submit" }, - include_hidden: false, - }, Current.user.id %> - - <%= form.label :creator_ids, "Me", for: form.field_id(:creator_ids, Current.user.id), class: "overflow-ellipsis" %> - <%= icon_tag "check", class: "checked flex-item-justify-end" %> -
- - <% User.active.without(Current.user).order(:name).each do |user| %> -
- <%= form.check_box :creator_ids, { - multiple: true, - checked: filter.creators.include?(user), - data: { action: "change->form#submit" }, - include_hidden: false, - }, user.id %> - - <%= form.label :creator_ids, user.name, for: form.field_id(:creator_ids, user.id), class: "overflow-ellipsis" %> - <%= icon_tag "check", class: "checked flex-item-justify-end" %> -
- <% end %> + <% if User.active.many? %> + <%= text_field_tag :search, nil, placeholder: "Filter…", class: "input input--transparent txt-small", autofocus: true, + type: "search", autocorrect: "off", autocomplete: "off", data: { "1p-ignore": "true", filter_target: "input", action: "input->filter#filter" } %> <% end %> - + +
    + <% User.active.order(:name).each do |user| %> +
  • + <%= form_with url: cards_path, method: :get, class: "full-width", data: { controller: "form" } do |form| %> + <% filter.as_params.except(:creator_ids).each do |key, value| %> + <%= filter_hidden_field_tag key, value %> + <% end %> + + <% filter.creators.each do |existing_creators| %> + <% unless existing_creators == user %> + <%= hidden_field_tag "creator_ids[]", existing_creators.id %> + <% end %> + <% end %> + + <% unless filter.creators.include?(user) %> + <%= hidden_field_tag "creator_ids[]", user.id %> + <% end %> + + <% if @expand_all %> + <%= hidden_field_tag :expand_all, @expand_all %> + <% end %> + + <%= form.button type: "submit", class: "btn popup__item min-width full-width unpad-inline" do %> + <%= user.name %> + <% if filter.creators.include?(user) %> + <%= icon_tag "check", class: "checked flex-item-justify-end" %> + <% end %> + <% end %> + <% end %> +
    • + <% end %> +
+ <% end %>
<% end %> diff --git a/app/views/filters/_dialog.html.erb b/app/views/filters/_dialog.html.erb index 6c67335d5..fd9d91b94 100644 --- a/app/views/filters/_dialog.html.erb +++ b/app/views/filters/_dialog.html.erb @@ -1,5 +1,5 @@ <%# Don't delete this file just yet, it's coming back soon. %> - +
diff --git a/app/views/filters/_indexed_by.html.erb b/app/views/filters/_indexed_by.html.erb index bc87053b5..37a22388f 100644 --- a/app/views/filters/_indexed_by.html.erb +++ b/app/views/filters/_indexed_by.html.erb @@ -1,32 +1,44 @@ -<% if !filter.indexed_by.latest? %> -
- - - Sort by… - <%= form_with url: cards_path, method: :get, class: "popup__list", - data: { controller: "form" } do |form| %> - <% filter.as_params.except(:indexed_by).each do |key, value| %> - <%= filter_hidden_field_tag key, value %> - <% end %> - + <%= tag.dialog class: "margin-block-start-half popup panel flex-column align-start gap-half fill-white shadow txt-small", data: { + action: "turbo:before-cache@document->dialog#close keydown->navigable-list#navigate", + aria: { label: "Sort by…", aria_description: "Sort by…" }, + controller: "navigable-list", + dialog_target: "dialog", + navigable_list_focus_on_selection_value: false, + navigable_list_actionable_items_value: true } do %> + Sort by… + +
    <% Filter::INDEXES.each do |index| %> -
    - <%= form.radio_button :indexed_by, index, - checked: filter.indexed_by == index, - data: { action: "change->form#submit" } %> +
  • + <%= form_with url: cards_path, method: :get, class: "full-width", data: { controller: "form" } do |form| %> + <% filter.as_params.except(:indexed_by).each do |key, value| %> + <%= filter_hidden_field_tag key, value %> + <% end %> - <%= form.label :indexed_by, index.humanize, value: index, class: "overflow-ellipsis" %> - <%= icon_tag "check", class: "checked flex-item-justify-end" %> -
    • + <%= hidden_field_tag :indexed_by, index %> + + <% if @expand_all %> + <%= hidden_field_tag :expand_all, @expand_all %> + <% end %> + + <%= form.button type: "submit", class: "btn popup__item min-width full-width unpad-inline" do %> + <%= index.humanize %> + <% if filter.indexed_by == index %> + <%= icon_tag "check", class: "checked flex-item-justify-end" %> + <% end %> + <% end %> + <% end %> + <% end %> - <% end %> -
+ + <% end %>
<% end %> diff --git a/app/views/filters/_settings.html.erb b/app/views/filters/_settings.html.erb index ecc39e45d..b7d6666d6 100644 --- a/app/views/filters/_settings.html.erb +++ b/app/views/filters/_settings.html.erb @@ -1,17 +1,46 @@
+ <% if any_filters?(filter) %> + <% if @expand_all %> + <%= link_to cards_path(filter.as_params), class: "btn txt-x-small btn--reversed", aria: { selected: true} do %> + <%= icon_tag "filter" %> + Edit filters + <% end %> + <% else %> + <%= link_to cards_path(filter.as_params.merge(expand_all: true)), class: "btn txt-x-small" do %> + <%= icon_tag "filter" %> + Edit filters + <% end %> + <% end %> + <% end %> + + <%= render "filters/indexed_by", filter: filter %> <%= render "filters/tags", filter: filter %> <%= render "filters/assignees", filter: filter %> <%= render "filters/creators", filter: filter %> - <%= render "filters/stages", filter: filter %> + <%# FIXME LATER: this needs to be able to allow selection of unique stages across all active workflows %> + <%#= render "filters/stages", filter: filter %> <%= render "filters/cards", filter: filter %> - <%= render "filters/indexed_by", filter: filter %> + <% filter.terms.each do |term| %> <%= filter_chip_tag %Q("#{term}"), filter.as_params_without(:terms, term) %> <% end %> <% if any_filters?(filter) %> + <% if filter.persisted? %> + <%= button_to filter_path(filter), method: :delete, class: "btn txt-x-small btn--reversed", form_class: "inline" do %> + <%= icon_tag "bookmark" %> + Remove saved filter + <% end %> + <% else %> + <% filter_params = filter.as_params.transform_values { |v| v.respond_to?(:to_str) ? v.to_str : v } %> + <%= button_to filters_path, method: :post, params: filter_params, class: "btn txt-x-small", form_class: "inline" do %> + <%= icon_tag "bookmark" %> + Save filter + <% end %> + <% end %> + <%= link_to cards_path(filter.as_params.except(:assignee_ids, :assignment_status, :card_ids, :creator_ids, :stage_ids, :tag_ids, :terms, :indexed_by)), class: "btn btn--remove txt-x-small" do %> <%= icon_tag "close" %> Clear all diff --git a/app/views/filters/_stages.html.erb b/app/views/filters/_stages.html.erb index db44cbd32..dffd2c5f9 100644 --- a/app/views/filters/_stages.html.erb +++ b/app/views/filters/_stages.html.erb @@ -1,4 +1,4 @@ -<% if filter.stages.any? %> +<% if filter.stages.any? || @expand_all %>
- In stage… @@ -20,6 +20,10 @@ <%= filter_hidden_field_tag key, value %> <% end %> + <% if @expand_all %> + <%= hidden_field_tag :expand_all, @expand_all %> + <% end %> + <% filter.stages.first.workflow.stages.each do |stage| %>
<%= form.check_box :stage_ids, { diff --git a/app/views/filters/_tags.html.erb b/app/views/filters/_tags.html.erb index b3b964784..665d1f048 100644 --- a/app/views/filters/_tags.html.erb +++ b/app/views/filters/_tags.html.erb @@ -1,40 +1,61 @@ -<% if filter.tags.any? %> +<% if filter.tags.any?|| @expand_all %>
- - Tagged… - <%= form_with url: cards_path, method: :get, class: "popup__list", - data: { controller: "form" } do |form| %> - <% filter.as_params.except(:tag_ids).each do |key, value| %> - <%= filter_hidden_field_tag key, value %> - <% end %> + <%= tag.dialog class: "margin-block-start-half popup panel flex-column align-start gap-half fill-white shadow txt-small", data: { + action: "turbo:before-cache@document->dialog#close keydown->navigable-list#navigate filter:changed->navigable-list#reset toggle->filter#filter", + aria: { label: "Tagged…", aria_description: "Tagged…" }, + controller: "filter navigable-list", + dialog_target: "dialog", + navigable_list_focus_on_selection_value: false, + navigable_list_actionable_items_value: true } do %> + Tagged… - <%= link_to cards_path(filter.as_params.except(:tag_ids)), class: "btn popup__item" do %> - Clear all - <% end %> - - <% Tag.order(:title).each do |tag| %> -
- <%= form.check_box "tag_ids[]", { - checked: filter.tags.include?(tag), - data: { action: "change->form#submit" }, - include_hidden: false, - }, tag.id %> - - <%= form.label "tag_ids[]", tag.hashtag, for: dom_id(tag, :filter), class: "overflow-ellipsis" %> - <%= icon_tag "check", class: "checked flex-item-justify-end" %> -
- <% end %> + <% if Tag.many? %> + <%= text_field_tag :search, nil, placeholder: "Filter…", class: "input input--transparent txt-small", autofocus: true, + type: "search", autocorrect: "off", autocomplete: "off", data: { "1p-ignore": "true", filter_target: "input", action: "input->filter#filter" } %> <% end %> - + +
    + <% Tag.order(:title).each do |tag| %> +
  • + <%= form_with url: cards_path, method: :get, class: "full-width", data: { controller: "form" } do |form| %> + <% filter.as_params.except(:tag_ids).each do |key, value| %> + <%= filter_hidden_field_tag key, value %> + <% end %> + + <% filter.tags.each do |existing_tag| %> + <% unless existing_tag == tag %> + <%= hidden_field_tag "tag_ids[]", existing_tag.id %> + <% end %> + <% end %> + + <% unless filter.tags.include?(tag) %> + <%= hidden_field_tag "tag_ids[]", tag.id %> + <% end %> + + <% if @expand_all %> + <%= hidden_field_tag :expand_all, @expand_all %> + <% end %> + + <%= form.button type: "submit", class: "btn popup__item min-width full-width unpad-inline" do %> + <%= tag.hashtag %> + <% if filter.tags.include?(tag) %> + <%= icon_tag "check", class: "checked flex-item-justify-end" %> + <% end %> + <% end %> + <% end %> +
    • + <% end %> +
+ <% end %>
<% end %> diff --git a/test/controllers/filters_controller_test.rb b/test/controllers/filters_controller_test.rb index ecf0481d5..ef50cc5a7 100644 --- a/test/controllers/filters_controller_test.rb +++ b/test/controllers/filters_controller_test.rb @@ -14,7 +14,7 @@ class FiltersControllerTest < ActionDispatch::IntegrationTest assignee_ids: [ users(:jz).id ], collection_ids: [ collections(:writebook).id ] } end - assert_redirected_to cards_path(Filter.last.as_params) + assert_redirected_to cards_path(filter_id: Filter.last.id) filter = Filter.last assert_predicate filter.indexed_by, :closed? @@ -25,9 +25,12 @@ class FiltersControllerTest < ActionDispatch::IntegrationTest end test "destroy" do + filter = filters(:jz_assignments) + expected_params = filter.as_params + assert_difference "users(:david).filters.count", -1 do - delete filter_path(filters(:jz_assignments)) + delete filter_path(filter) end - assert_redirected_to cards_path(filters(:jz_assignments).as_params) + assert_redirected_to cards_path(expected_params) end end diff --git a/test/models/filter_test.rb b/test/models/filter_test.rb index 6dec75089..4ebe17434 100644 --- a/test/models/filter_test.rb +++ b/test/models/filter_test.rb @@ -114,16 +114,16 @@ class FilterTest < ActiveSupport::TestCase end test "summary" do - assert_equal "Newest, tagged #mobile, and assigned to JZ ", filters(:jz_assignments).summary + assert_equal "Newest, #mobile, and assigned to JZ", filters(:jz_assignments).summary filters(:jz_assignments).update!(stages: workflow_stages(:qa_triage, :qa_in_progress)) - assert_equal "Newest, tagged #mobile, assigned to JZ, and staged in Triage or In progress ", filters(:jz_assignments).summary + assert_equal "Newest, #mobile, assigned to JZ, and staged in Triage or In progress", filters(:jz_assignments).summary filters(:jz_assignments).update!(stages: [], assignees: [], tags: [], collections: [ collections(:writebook) ]) - assert_equal "Newest in Writebook", filters(:jz_assignments).summary + assert_equal "Newest", filters(:jz_assignments).summary filters(:jz_assignments).update!(indexed_by: "stalled") - assert_equal "Stalled in Writebook", filters(:jz_assignments).summary + assert_equal "Stalled", filters(:jz_assignments).summary end test "get a clone with some changed params" do