@internetarchive/collection-browser 4.1.0 → 4.1.1-alpha-webdev8185.1

package/dist/test/collection-browser.test.js

@@ -71,7 +71,7 @@ describe('Collection Browser', () => {
     });
     it('filterBy creator with analytics', async () => {
         const mockAnalyticsHandler = new MockAnalyticsHandler();
-        const el = await fixture(html `<collection-browser .analyticsHandler=${mockAnalyticsHandler}>
+        const el = await fixture(html `<collection-browser .analyticsHandler=${mockAnalyticsHandler}>
       </collection-browser>`);
         el.searchContext = 'betaSearchService';
         el.selectedSort = 'creator';
@@ -90,7 +90,7 @@ describe('Collection Browser', () => {
     });
     it('filterBy title with analytics', async () => {
         const mockAnalyticsHandler = new MockAnalyticsHandler();
-        const el = await fixture(html `<collection-browser .analyticsHandler=${mockAnalyticsHandler}>
+        const el = await fixture(html `<collection-browser .analyticsHandler=${mockAnalyticsHandler}>
       </collection-browser>`);
         el.searchContext = 'beta-search-service';
         el.selectedSort = 'title';
@@ -119,7 +119,7 @@ describe('Collection Browser', () => {
             collection: {},
             year: {},
         };
-        const el = await fixture(html `<collection-browser .analyticsHandler=${mockAnalyticsHandler}>
+        const el = await fixture(html `<collection-browser .analyticsHandler=${mockAnalyticsHandler}>
       </collection-browser>`);
         el.searchContext = 'search-service';
         el.selectedFacets = mockedSelectedFacets;
@@ -166,7 +166,7 @@ describe('Collection Browser', () => {
             collection: {},
             year: {},
         };
-        const el = await fixture(html `<collection-browser .analyticsHandler=${mockAnalyticsHandler}>
+        const el = await fixture(html `<collection-browser .analyticsHandler=${mockAnalyticsHandler}>
       </collection-browser>`);
         el.searchContext = 'beta-search-service';
         el.selectedFacets = mockedSelectedFacets;
@@ -203,7 +203,7 @@ describe('Collection Browser', () => {
     });
     it('should render with a sort bar, facets, and infinite scroller', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'hello';
         await el.updateComplete;
@@ -217,7 +217,7 @@ describe('Collection Browser', () => {
     });
     it('queries the search service when given a base query', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
@@ -227,7 +227,7 @@ describe('Collection Browser', () => {
     });
     it('queries the search service when given a list of identifiers and no query', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.identifiers = ['foo', 'bar'];
         await el.updateComplete;
@@ -240,7 +240,7 @@ describe('Collection Browser', () => {
     });
     it('queries the search service when given a list of identifiers with a query', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'collection:foo';
         el.identifiers = ['foo', 'bar'];
@@ -255,7 +255,7 @@ describe('Collection Browser', () => {
     });
     it('queries the search service with a metadata search', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html ` <collection-browser .searchService=${searchService}>
+        const el = await fixture(html ` <collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.searchType = SearchType.METADATA;
         await el.updateComplete;
@@ -268,7 +268,7 @@ describe('Collection Browser', () => {
     });
     it('can change search type', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
@@ -279,7 +279,7 @@ describe('Collection Browser', () => {
     });
     it('queries the search service with a fulltext search', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html ` <collection-browser .searchService=${searchService}>
+        const el = await fixture(html ` <collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.searchType = SearchType.FULLTEXT;
         await el.updateComplete;
@@ -292,7 +292,7 @@ describe('Collection Browser', () => {
     });
     it('queries the search service with a radio search', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html ` <collection-browser .searchService=${searchService}>
+        const el = await fixture(html ` <collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.searchType = SearchType.RADIO;
         await el.updateComplete;
@@ -305,7 +305,7 @@ describe('Collection Browser', () => {
     });
     it('queries the search service with a TV search', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html ` <collection-browser .searchService=${searchService}>
+        const el = await fixture(html ` <collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.searchType = SearchType.TV;
         await el.updateComplete;
@@ -344,7 +344,7 @@ describe('Collection Browser', () => {
             collection: {},
             year: {},
         };
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'collection:foo';
         el.selectedFacets = selectedFacets;
@@ -376,7 +376,7 @@ describe('Collection Browser', () => {
         url.searchParams.append('sin', 'TXT');
         window.history.replaceState({}, '', url);
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         expect(el.searchType).to.equal(SearchType.FULLTEXT);
     });
@@ -386,10 +386,10 @@ describe('Collection Browser', () => {
         url.searchParams.append('sin', 'TXT');
         window.history.replaceState({}, '', url);
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        suppressURLSinParam
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        suppressURLSinParam
+      >
       </collection-browser>`);
         url = new URL(window.location.href);
         expect(el.searchType).to.equal(SearchType.DEFAULT);
@@ -401,7 +401,7 @@ describe('Collection Browser', () => {
     });
     it('can construct tile models with many fields present', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'many-fields';
         await el.updateComplete;
@@ -414,10 +414,10 @@ describe('Collection Browser', () => {
     it('emits empty results event when search fetches no results', async () => {
         const searchService = new MockSearchService();
         const emptyResultsSpy = sinon.spy();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        @emptyResults=${emptyResultsSpy}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        @emptyResults=${emptyResultsSpy}
+      >
       </collection-browser>`);
         el.baseQuery = 'no-results';
         await el.updateComplete;
@@ -427,10 +427,10 @@ describe('Collection Browser', () => {
     it('emits searchError event when search results in an error', async () => {
         const searchService = new MockSearchService();
         const searchErrorSpy = sinon.spy();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        @searchError=${searchErrorSpy}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        @searchError=${searchErrorSpy}
+      >
       </collection-browser>`);
         el.baseQuery = 'error';
         await el.updateComplete;
@@ -439,7 +439,7 @@ describe('Collection Browser', () => {
     });
     it('applies loggedin flag to tile models if needed', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'loggedin';
         await el.updateComplete;
@@ -452,7 +452,7 @@ describe('Collection Browser', () => {
     });
     it('applies no-preview flag to tile models if needed', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'no-preview';
         await el.updateComplete;
@@ -465,7 +465,7 @@ describe('Collection Browser', () => {
     });
     it('both loggedin and no-preview flags can be set simultaneously', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'loggedin-no-preview';
         await el.updateComplete;
@@ -479,7 +479,7 @@ describe('Collection Browser', () => {
     });
     it('joins full description array into a single string with line breaks', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         // This query receives an array description like ['line1', 'line2']
         el.baseQuery = 'multi-line-description';
@@ -494,9 +494,9 @@ describe('Collection Browser', () => {
     });
     it('can change search type', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .searchType=${SearchType.METADATA}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .searchType=${SearchType.METADATA}
       ></collection-browser>`);
         el.baseQuery = 'collection:foo';
         el.searchType = SearchType.FULLTEXT;
@@ -507,8 +507,8 @@ describe('Collection Browser', () => {
     });
     it('trims queries of leading/trailing whitespace', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
       ></collection-browser>`);
         el.baseQuery = '   collection:foo   ';
         await el.updateComplete;
@@ -517,8 +517,8 @@ describe('Collection Browser', () => {
     });
     it('shows error message when error response received', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
       ></collection-browser>`);
         el.baseQuery = 'error';
         await el.updateComplete;
@@ -530,8 +530,8 @@ describe('Collection Browser', () => {
     });
     it('shows error message when error response received for a collection', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
       ></collection-browser>`);
         el.withinCollection = 'error';
         await el.updateComplete;
@@ -546,8 +546,8 @@ describe('Collection Browser', () => {
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         window.Sentry = { captureMessage: sentrySpy };
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
       ></collection-browser>`);
         el.baseQuery = 'malformed';
         await el.updateComplete;
@@ -556,7 +556,7 @@ describe('Collection Browser', () => {
     });
     it('adds collection names to cache when present on response', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'collection-titles';
         await el.updateComplete;
@@ -568,7 +568,7 @@ describe('Collection Browser', () => {
     });
     it('adds tv channel aliases to cache when present on response', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'channel-aliases';
         await el.updateComplete;
@@ -582,7 +582,7 @@ describe('Collection Browser', () => {
             asyncResponse: true,
             resultsSpy,
         });
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'with-sort';
         el.selectedSort = SortField.date;
@@ -599,7 +599,7 @@ describe('Collection Browser', () => {
             asyncResponse: true,
             resultsSpy,
         });
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'with-sort';
         el.selectedSort = SortField.date;
@@ -621,7 +621,7 @@ describe('Collection Browser', () => {
             asyncResponse: true,
             resultsSpy,
         });
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'single-result';
         await el.updateComplete;
@@ -642,7 +642,7 @@ describe('Collection Browser', () => {
             asyncResponse: true,
             resultsSpy,
         });
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'with-sort';
         el.selectedSort = SortField.date;
@@ -660,7 +660,7 @@ describe('Collection Browser', () => {
     });
     it('sets sort properties when user changes sort', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         expect(el.selectedSort).to.equal(SortField.default);
         el.baseQuery = 'foo';
@@ -681,7 +681,7 @@ describe('Collection Browser', () => {
     });
     it('sets sort filter properties when user selects title filter', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'first-title';
         el.selectedSort = 'title';
@@ -694,7 +694,7 @@ describe('Collection Browser', () => {
     });
     it('sets sort filter properties when user selects creator filter', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'first-creator';
         el.selectedSort = 'creator';
@@ -716,7 +716,7 @@ describe('Collection Browser', () => {
             subject: {},
             year: {},
         };
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'first-creator';
         el.selectedSort = 'creator';
@@ -743,7 +743,7 @@ describe('Collection Browser', () => {
     });
     it('applies correct TV search filter for commercials', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'tv-fields';
         el.searchType = SearchType.TV;
@@ -758,7 +758,7 @@ describe('Collection Browser', () => {
     });
     it('applies correct TV search filter for fact checks', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'tv-fields';
         el.searchType = SearchType.TV;
@@ -773,7 +773,7 @@ describe('Collection Browser', () => {
     });
     it('applies correct TV search filter for quotes', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'tv-fields';
         el.searchType = SearchType.TV;
@@ -786,7 +786,7 @@ describe('Collection Browser', () => {
     });
     it('resets letter filters when query changes', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'first-creator';
         el.selectedSort = 'creator';
@@ -806,11 +806,11 @@ describe('Collection Browser', () => {
     it('sets date range query when date picker selection changed', async () => {
         const searchService = new MockSearchService();
         const mockAnalyticsHandler = new MockAnalyticsHandler();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .analyticsHandler=${mockAnalyticsHandler}
-        .suppressPlaceholders=${true}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .analyticsHandler=${mockAnalyticsHandler}
+        .suppressPlaceholders=${true}
+      >
       </collection-browser>`);
         el.baseQuery = 'years'; // Includes year_histogram aggregation in response
         el.showHistogramDatePicker = true;
@@ -838,10 +838,10 @@ describe('Collection Browser', () => {
     });
     it('sets date range query when monthly date picker selection changed', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .suppressPlaceholders=${true}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .suppressPlaceholders=${true}
+      >
       </collection-browser>`);
         el.baseQuery = 'months'; // Includes date_histogram aggregation in response
         el.searchType = SearchType.TV;
@@ -871,9 +871,9 @@ describe('Collection Browser', () => {
     it('emits event when results start and end loading', async () => {
         const spy = sinon.spy();
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        @searchResultsLoadingChanged=${spy}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        @searchResultsLoadingChanged=${spy}
       ></collection-browser>`);
         spy.resetHistory();
         el.baseQuery = 'collection:foo';
@@ -886,9 +886,9 @@ describe('Collection Browser', () => {
     });
     it('collapses extra set of quotes around href field', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
       ></collection-browser>`);
         el.baseQuery = 'extra-quoted-href';
         await el.updateComplete;
@@ -898,17 +898,16 @@ describe('Collection Browser', () => {
         // Original href q param starts/ends with %22%22, but should be collapsed to %22 before render
         expect(el.dataSource.getTileModelAt(0)?.href).to.equal('/details/foo?q=%22quoted+query%22');
     });
-    it('sets default sort from collection metadata', async () => {
+    it('sets default sort from externally-provided values', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
+        .withinCollection=${'test-collection'}
+        .defaultSortField=${SortField.title}
+        .defaultSortDirection=${'asc'}
       ></collection-browser>`);
-        el.withinCollection = 'default-sort';
-        await el.updateComplete;
-        await el.initialSearchComplete;
         await el.updateComplete;
-        await aTimeout(50);
         const sortBar = el.shadowRoot?.querySelector('sort-filter-bar');
         expect(sortBar).to.exist;
         expect(sortBar.defaultSortField).to.equal(SortField.title);
@@ -916,17 +915,16 @@ describe('Collection Browser', () => {
         expect(sortBar.selectedSort).to.equal(SortField.default);
         expect(sortBar.sortDirection).to.be.null;
     });
-    it('sets default sort from collection metadata in "-field" format', async () => {
+    it('reflects updated default sort field/direction on sort bar', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
+        .withinCollection=${'test-collection'}
+        .defaultSortField=${SortField.dateadded}
+        .defaultSortDirection=${'desc'}
       ></collection-browser>`);
-        el.withinCollection = 'default-sort-concise';
-        await el.updateComplete;
-        await el.initialSearchComplete;
         await el.updateComplete;
-        await aTimeout(50);
         const sortBar = el.shadowRoot?.querySelector('sort-filter-bar');
         expect(sortBar).to.exist;
         expect(sortBar.defaultSortField).to.equal(SortField.dateadded);
@@ -934,11 +932,12 @@ describe('Collection Browser', () => {
         expect(sortBar.selectedSort).to.equal(SortField.default);
         expect(sortBar.sortDirection).to.be.null;
     });
-    it('falls back to weekly views default sorting on profiles when tab not set', async () => {
-        const el = await fixture(html `<collection-browser
-        .withinProfile=${'@foobar'}
+    it('uses weekly views as default sort when set externally for profiles', async () => {
+        const el = await fixture(html `<collection-browser
+        .withinProfile=${'@foobar'}
+        .defaultSortField=${SortField.weeklyview}
+        .defaultSortDirection=${'desc'}
       ></collection-browser>`);
-        el.applyDefaultProfileSort();
         expect(el.defaultSortParam).to.deep.equal({
             field: 'week',
             direction: 'desc',
@@ -946,9 +945,9 @@ describe('Collection Browser', () => {
     });
     it('uses relevance sort as default when a query is set', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
       ></collection-browser>`);
         el.withinCollection = 'default-sort';
         el.baseQuery = 'default-sort';
@@ -963,17 +962,16 @@ describe('Collection Browser', () => {
         expect(sortBar.selectedSort).to.equal(SortField.default);
         expect(sortBar.sortDirection).to.be.null;
     });
-    it('uses date favorited sort as default when targeting fav- collection', async () => {
+    it('uses date favorited sort as default when set externally for fav- collection', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
+        .withinCollection=${'fav-sort'}
+        .defaultSortField=${SortField.datefavorited}
+        .defaultSortDirection=${'desc'}
       ></collection-browser>`);
-        el.withinCollection = 'fav-sort';
         await el.updateComplete;
-        await el.initialSearchComplete;
-        await el.updateComplete;
-        await aTimeout(50);
         const sortBar = el.shadowRoot?.querySelector('sort-filter-bar');
         expect(sortBar).to.exist;
         expect(sortBar.defaultSortField).to.equal(SortField.datefavorited);
@@ -983,7 +981,7 @@ describe('Collection Browser', () => {
     });
     it('scrolls to page', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         // Infinite scroller won't exist unless there's a base query.
         // First ensure that we don't throw errors when it doesn't exist.
@@ -1004,9 +1002,9 @@ describe('Collection Browser', () => {
     });
     it('shows mobile facets in mobile view', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .mobileBreakpoint=${9999}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .mobileBreakpoint=${9999}
       ></collection-browser>`);
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
@@ -1025,11 +1023,11 @@ describe('Collection Browser', () => {
     it('fires analytics when mobile facets toggled', async () => {
         const searchService = new MockSearchService();
         const analyticsHandler = new MockAnalyticsHandler();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .analyticsHandler=${analyticsHandler}
-        .searchContext=${'foobar-context'}
-        .mobileBreakpoint=${9999}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .analyticsHandler=${analyticsHandler}
+        .searchContext=${'foobar-context'}
+        .mobileBreakpoint=${9999}
       ></collection-browser>`);
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
@@ -1069,9 +1067,9 @@ describe('Collection Browser', () => {
     });
     it('sets parent collections to prop when searching a collection', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .withinCollection=${'fake'}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .withinCollection=${'fake'}
       ></collection-browser>`);
         el.baseQuery = 'parent-collections';
         await el.updateComplete;
@@ -1081,9 +1079,9 @@ describe('Collection Browser', () => {
     });
     it('recognizes TV collections', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .withinCollection=${'TV-FOO'}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .withinCollection=${'TV-FOO'}
       ></collection-browser>`);
         el.baseQuery = 'tv-collection';
         await el.updateComplete;
@@ -1094,9 +1092,9 @@ describe('Collection Browser', () => {
     it('refreshes when certain properties change - with some analytics event sampling', async () => {
         const mockAnalyticsHandler = new MockAnalyticsHandler();
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .analyticsHandler=${mockAnalyticsHandler}
-        .searchService=${searchService}
+        const el = await fixture(html `<collection-browser
+        .analyticsHandler=${mockAnalyticsHandler}
+        .searchService=${searchService}
       ></collection-browser>`);
         const infiniteScrollerRefreshSpy = sinon.spy();
         // Infinite scroller won't exist unless there's a base query
@@ -1141,7 +1139,7 @@ describe('Collection Browser', () => {
     });
     it('query the search service for single result', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'single-result';
         await el.updateComplete;
@@ -1169,7 +1167,7 @@ describe('Collection Browser', () => {
         searchParams.append('and[]', 'year:[2000 TO 2010]');
         window.history.replaceState({}, '', url);
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         await el.initialSearchComplete;
         await el.updateComplete;
@@ -1194,10 +1192,10 @@ describe('Collection Browser', () => {
         searchParams.append('and[]', 'year:[2000 TO 2010]');
         window.history.replaceState({}, '', url);
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .withinCollection=${'foobar'}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .withinCollection=${'foobar'}
+      >
       </collection-browser>`);
         await el.initialSearchComplete;
         await el.updateComplete;
@@ -1221,11 +1219,11 @@ describe('Collection Browser', () => {
         searchParams.append('and[]', 'year:[2000 TO 2010]');
         window.history.replaceState({}, '', url);
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .withinProfile=${'@foobar'}
-        .profileElement=${'uploads'}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .withinProfile=${'@foobar'}
+        .profileElement=${'uploads'}
+      >
       </collection-browser>`);
         await el.initialSearchComplete;
         await el.updateComplete;
@@ -1251,7 +1249,7 @@ describe('Collection Browser', () => {
         searchParams.append('and[]', 'year:[2000 TO 2010]');
         window.history.replaceState({}, '', url);
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         await el.initialSearchComplete;
         await el.updateComplete;
@@ -1278,10 +1276,10 @@ describe('Collection Browser', () => {
         searchParams.append('and[]', 'year:[2000 TO 2010]');
         window.history.replaceState({}, '', url);
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .withinCollection=${'foobar'}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .withinCollection=${'foobar'}
+      >
       </collection-browser>`);
         el.baseQuery = 'bar';
         await el.updateComplete;
@@ -1307,10 +1305,10 @@ describe('Collection Browser', () => {
         searchParams.append('and[]', 'year:[2000 TO 2010]');
         window.history.replaceState({}, '', url);
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .withinCollection=${'foobar'}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .withinCollection=${'foobar'}
+      >
       </collection-browser>`);
         el.withinCollection = 'bar';
         await el.updateComplete;
@@ -1326,11 +1324,11 @@ describe('Collection Browser', () => {
     });
     it('correctly retrieves web archive hits', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .withinProfile=${'@foo'}
-        .profileElement=${'web_archives'}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .withinProfile=${'@foo'}
+        .profileElement=${'web_archives'}
+      >
       </collection-browser>`);
         el.baseQuery = 'web-archive';
         await el.updateComplete;
@@ -1343,10 +1341,10 @@ describe('Collection Browser', () => {
     });
     it('shows dropdown accordion in facet sidebar when opt-in strategy is specified', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        facetLoadStrategy=${'opt-in'}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        facetLoadStrategy=${'opt-in'}
+      >
       </collection-browser>`);
         el.baseQuery = 'foo';
         await el.updateComplete;
@@ -1356,10 +1354,10 @@ describe('Collection Browser', () => {
     });
     it('shows temporarily unavailable message when facets suppressed', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        facetLoadStrategy=${'off'}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        facetLoadStrategy=${'off'}
+      >
       </collection-browser>`);
         el.baseQuery = 'foo';
         await el.updateComplete;
@@ -1370,7 +1368,7 @@ describe('Collection Browser', () => {
     });
     it('shows manage bar interface instead of sort bar when in manage view', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'foo';
         await el.updateComplete;
@@ -1386,11 +1384,11 @@ describe('Collection Browser', () => {
     });
     it('switches to grid display mode when manage view activated', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseQuery=${'foo'}
-        .displayMode=${'list-detail'}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseQuery=${'foo'}
+        .displayMode=${'list-detail'}
+      >
       </collection-browser>`);
         el.isManageView = true;
         await el.updateComplete;
@@ -1398,10 +1396,10 @@ describe('Collection Browser', () => {
     });
     it('can remove all checked tiles', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
+      >
       </collection-browser>`);
         el.baseQuery = 'foo';
         el.pageSize = 1; // To hit the edge case of a page break while offsetting tiles
@@ -1439,10 +1437,10 @@ describe('Collection Browser', () => {
     });
     it('can check/uncheck all tiles', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
+      >
       </collection-browser>`);
         el.baseQuery = 'foo';
         await el.updateComplete;
@@ -1461,10 +1459,10 @@ describe('Collection Browser', () => {
     it('emits event when manage view state changes', async () => {
         const spy = sinon.spy();
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
-        @manageModeChanged=${spy}
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
+        @manageModeChanged=${spy}
       ></collection-browser>`);
         el.isManageView = true;
         await el.updateComplete;
@@ -1478,11 +1476,11 @@ describe('Collection Browser', () => {
     it('emits event when item removal requested', async () => {
         const spy = sinon.spy();
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
-        @itemRemovalRequested=${spy}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
+        @itemRemovalRequested=${spy}
+      >
       </collection-browser>`);
         el.baseQuery = 'foo';
         await el.updateComplete;
@@ -1511,10 +1509,10 @@ describe('Collection Browser', () => {
     });
     it('disables manage view when manage bar cancelled', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        .baseNavigationUrl=${''}
-      >
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .baseNavigationUrl=${''}
+      >
       </collection-browser>`);
         el.baseQuery = 'foo';
         await el.updateComplete;
@@ -1530,7 +1528,7 @@ describe('Collection Browser', () => {
     });
     it('enable/disable manage view delete button when you selectAll/unselectAll', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         el.baseQuery = 'foo';
         await el.updateComplete;
@@ -1557,10 +1555,10 @@ describe('Collection Browser', () => {
     });
     it('shows Blurring checkbox for admin users', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .baseNavigationUrl=${''}
-        .searchService=${searchService}
-      >
+        const el = await fixture(html `<collection-browser
+        .baseNavigationUrl=${''}
+        .searchService=${searchService}
+      >
       </collection-browser>`);
         el.baseQuery = 'archive-org-user-loggedin';
         await el.updateComplete;
@@ -1571,10 +1569,10 @@ describe('Collection Browser', () => {
     });
     it('unchecks Blurring checkbox for admin users with blurring preference off', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .baseNavigationUrl=${''}
-        .searchService=${searchService}
-      >
+        const el = await fixture(html `<collection-browser
+        .baseNavigationUrl=${''}
+        .searchService=${searchService}
+      >
       </collection-browser>`);
         el.baseQuery = 'archive-org-user-loggedin-noblur';
         await el.updateComplete;
@@ -1585,10 +1583,10 @@ describe('Collection Browser', () => {
     });
     it('toggles blur state when Blurring checkbox is toggled', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .baseNavigationUrl=${''}
-        .searchService=${searchService}
-      >
+        const el = await fixture(html `<collection-browser
+        .baseNavigationUrl=${''}
+        .searchService=${searchService}
+      >
       </collection-browser>`);
         el.baseQuery = 'archive-org-user-loggedin';
         await el.updateComplete;
@@ -1603,11 +1601,11 @@ describe('Collection Browser', () => {
     });
     it('applies loans tab properties to sort bar', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .baseNavigationUrl=${''}
-        .searchService=${searchService}
-        .enableSortOptionsSlot=${true}
-      >
+        const el = await fixture(html `<collection-browser
+        .baseNavigationUrl=${''}
+        .searchService=${searchService}
+        .enableSortOptionsSlot=${true}
+      >
       </collection-browser>`);
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
@@ -1621,9 +1619,9 @@ describe('Collection Browser', () => {
     });
     it('can suppress presence of result count', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        suppressResultCount
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        suppressResultCount
       ></collection-browser>`);
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
@@ -1633,9 +1631,9 @@ describe('Collection Browser', () => {
     });
     it('can suppress presence of result tiles', async () => {
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser
-        .searchService=${searchService}
-        suppressResultTiles
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        suppressResultTiles
       ></collection-browser>`);
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
@@ -1648,7 +1646,7 @@ describe('Collection Browser', () => {
             asyncResponse: true,
             resultsSpy,
         });
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
       </collection-browser>`);
         const numberOfPages = 15;
         el.baseQuery = 'jack';