@internetarchive/collection-browser 2.2.3-alpha.6 → 2.3.0

package/package.json

@@ -3,7 +3,7 @@
   "description": "The Internet Archive Collection Browser.",
   "license": "AGPL-3.0-only",
   "author": "Internet Archive",
-  "version": "2.2.3-alpha.6",
+  "version": "2.3.0",
   "main": "dist/index.js",
   "module": "dist/index.js",
   "scripts": {

package/src/collection-browser.ts

@@ -1153,8 +1153,6 @@ export class CollectionBrowser
   }
 
   updated(changed: PropertyValues) {
-    console.log('updated', '\n' + [...changed.entries()].map(([k,v]) => `${String(k)}: ${v} -> ${this[k as keyof this]}`).join('\n'));
-
     if (changed.has('placeholderType') && this.placeholderType === null) {
       if (!this.leftColIntersectionObserver) {
         this.setupLeftColumnScrollListeners();
@@ -1617,7 +1615,6 @@ export class CollectionBrowser
 
   private restoreState() {
     const restorationState = this.restorationStateHandler.getRestorationState();
-    console.log('restoring state:', restorationState);
     this.displayMode = restorationState.displayMode;
     if (restorationState.searchType != null)
       this.searchType = restorationState.searchType;
@@ -1651,7 +1648,6 @@ export class CollectionBrowser
       selectedTitleFilter: this.selectedTitleFilter ?? undefined,
       selectedCreatorFilter: this.selectedCreatorFilter ?? undefined,
     };
-    console.log('persisting state:', restorationState);
     this.restorationStateHandler.persistState(
       restorationState,
       this.dataSourceInstallInProgress

package/src/data-source/collection-browser-data-source.ts

@@ -186,7 +186,6 @@ export class CollectionBrowserDataSource
   }
 
   hostUpdate(): void {
-    console.log('host updated');
     // This reactive controller hook is run whenever the host component (collection-browser) performs an update.
     // We check whether the host's state has changed in a way which should trigger a reset & new results fetch.
 
@@ -379,7 +378,6 @@ export class CollectionBrowserDataSource
   async handleQueryChange(): Promise<void> {
     // Don't react to the change if fetches are suppressed for this data source
     if (this.suppressFetches) return;
-    console.log('resetting data source & handling query change');
 
     this.reset();
 
@@ -403,7 +401,6 @@ export class CollectionBrowserDataSource
    * @inheritdoc
    */
   async handleFacetReadinessChange(ready: boolean): Promise<void> {
-    console.log('handling facet readiness change:', ready);
     const facetsBecameReady = !this.facetsReadyToLoad && ready;
     this.facetsReadyToLoad = ready;
 
@@ -595,7 +592,6 @@ export class CollectionBrowserDataSource
    * Sets the state for whether the facets for a query is loading
    */
   private setFacetsLoading(loading: boolean): void {
-    console.log('setFacetsLoading', loading);
     this.facetsLoading = loading;
     if (this.activeOnHost) {
       this.host.setFacetsLoading(loading);
@@ -924,7 +920,6 @@ export class CollectionBrowserDataSource
     if (this.fetchesInProgress.has(facetFetchQueryKey)) return;
     this.fetchesInProgress.add(facetFetchQueryKey);
 
-    console.log('fetching facets', this.filterMap);
     this.setFacetsLoading(true);
 
     const sortParams = this.host.sortParam ? [this.host.sortParam] : [];

package/test/collection-browser.test.ts

@@ -1659,10 +1659,33 @@ describe('Collection Browser', () => {
     ).to.equal(1);
   });
 
+  it('shows dropdown accordion in facet sidebar when opt-in strategy is specified', async () => {
+    const searchService = new MockSearchService();
+    const el = await fixture<CollectionBrowser>(
+      html`<collection-browser
+        .searchService=${searchService}
+        facetLoadStrategy=${'opt-in'}
+      >
+      </collection-browser>`
+    );
+
+    el.baseQuery = 'foo';
+    await el.updateComplete;
+    await el.initialSearchComplete;
+
+    const facetsDropdown = el.shadowRoot?.querySelector(
+      '.desktop-facets-dropdown'
+    );
+    expect(facetsDropdown).to.exist;
+  });
+
   it('shows temporarily unavailable message when facets suppressed', async () => {
     const searchService = new MockSearchService();
     const el = await fixture<CollectionBrowser>(
-      html`<collection-browser .searchService=${searchService} suppressFacets>
+      html`<collection-browser
+        .searchService=${searchService}
+        facetLoadStrategy=${'off'}
+      >
       </collection-browser>`
     );