commit 209f14107732dacc484258a7756cdc9c9d477c38
author Ben Rohlfs <brohlfs@google.com> Fri Sep 30 12:25:46 2022 +0200
committer Ben Rohlfs <brohlfs@google.com> Fri Oct 14 10:28:50 2022 +0200
tree 76fac90a07cb19037b1f743d93f659ae3a8f3642
parent 81dcbf28e008f5f5b0e4555385b0691ec746524f

Add a `checksResultsFilter` URL parameter

Just move the logic for maintaining the filter expression from the
checks results panel into the ChangeViewState.

We are also updating the logic of actually expanding sections by
default when a filter is in use. Collapsing the INFO or SUCCESS
sections does not make sense when they contain the results that
match the filter.

Release-Notes: skip
Google-Bug-Id: b/235185477
Change-Id: Ife0823a13ee55b6353d1ea0cec09fa06a0202680

polygerrit-ui/app/elements/core/gr-router/gr-router_test.ts

diff --git a/polygerrit-ui/app/elements/core/gr-router/gr-router_test.ts b/polygerrit-ui/app/elements/core/gr-router/gr-router_test.ts
index 9669ec7..5c5e67d 100644
--- a/polygerrit-ui/app/elements/core/gr-router/gr-router_test.ts
+++ b/polygerrit-ui/app/elements/core/gr-router/gr-router_test.ts
@@ -1150,6 +1150,7 @@
           queryMap.set('select', 'sss');
           queryMap.set('attempt', '1');
           queryMap.set('checksRunsSelected', 'asdf,qwer');
+          queryMap.set('checksResultsFilter', 'asdf.*qwer');
           ctx.querystring = queryMap.toString();
           assertctxToParams(ctx, 'handleChangeRoute', {
             view: GerritView.CHANGE,
@@ -1161,6 +1162,7 @@
             filter: 'fff',
             tab: 'checks',
             checksRunsSelected: ['asdf', 'qwer'],
+            checksResultsFilter: 'asdf.*qwer',
           });
         });
       });