@internetarchive/collection-browser 1.14.17-alpha.2 → 1.14.17-alpha.21

package/dist/src/utils/collapse-repeated-quotes.js

@@ -0,0 +1,14 @@
+/**
+ * Returns the input string, but removing one set of quotes from all instances of
+ * ""clauses wrapped in two sets of quotes"". This assumes the quotes are already
+ * URL-encoded.
+ *
+ * This should be a temporary measure to address the fact that the __href__ field
+ * sometimes acquires extra quotation marks during query rewriting. Once there is a
+ * full Lucene parser in place that handles quoted queries correctly, this can likely
+ * be removed.
+ */
+export function collapseRepeatedQuotes(str) {
+    return str === null || str === void 0 ? void 0 : str.replace(/%22%22(?!%22%22)(.+?)%22%22/g, '%22$1%22');
+}
+//# sourceMappingURL=collapse-repeated-quotes.js.map

package/dist/src/utils/collapse-repeated-quotes.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"collapse-repeated-quotes.js","sourceRoot":"","sources":["../../../src/utils/collapse-repeated-quotes.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AACH,MAAM,UAAU,sBAAsB,CAAC,GAAY;IACjD,OAAO,GAAG,aAAH,GAAG,uBAAH,GAAG,CAAE,OAAO,CAAC,8BAA8B,EAAE,UAAU,CAAC,CAAC;AAClE,CAAC","sourcesContent":["/**\n * Returns the input string, but removing one set of quotes from all instances of\n * \"\"clauses wrapped in two sets of quotes\"\". This assumes the quotes are already\n * URL-encoded.\n *\n * This should be a temporary measure to address the fact that the __href__ field\n * sometimes acquires extra quotation marks during query rewriting. Once there is a\n * full Lucene parser in place that handles quoted queries correctly, this can likely\n * be removed.\n */\nexport function collapseRepeatedQuotes(str?: string): string | undefined {\n  return str?.replace(/%22%22(?!%22%22)(.+?)%22%22/g, '%22$1%22');\n}\n"]}

package/dist/src/utils/resolve-mediatype.d.ts

@@ -0,0 +1,8 @@
+import type { MediaType } from '@internetarchive/field-parsers';
+import type { SearchResult } from '@internetarchive/search-service';
+/**
+ * Returns the mediatype string for the given search result, taking into account
+ * the special `favorited_search` hit type.
+ * @param result The search result to extract a mediatype from
+ */
+export declare function resolveMediatype(result: SearchResult): MediaType;

package/dist/src/utils/resolve-mediatype.js

@@ -0,0 +1,24 @@
+/**
+ * Returns the mediatype string for the given search result, taking into account
+ * the special `favorited_search` hit type.
+ * @param result The search result to extract a mediatype from
+ */
+export function resolveMediatype(result) {
+    var _a, _b, _c;
+    /**
+     * hit_type == 'favorited_search' is basically a new hit_type
+     * - we are getting from PPS.
+     * - which gives response for fav- collection
+     * - having favorited items like account/collection/item etc..
+     * - as user can also favorite a search result (a search page)
+     * - so we need to have response (having fav- items and fav- search results)
+     *
+     * if backend hit_type == 'favorited_search'
+     * - let's assume a "search" as new mediatype
+     */
+    if (((_a = result === null || result === void 0 ? void 0 : result.rawMetadata) === null || _a === void 0 ? void 0 : _a.hit_type) === 'favorited_search') {
+        return 'search';
+    }
+    return (_c = (_b = result.mediatype) === null || _b === void 0 ? void 0 : _b.value) !== null && _c !== void 0 ? _c : 'data';
+}
+//# sourceMappingURL=resolve-mediatype.js.map

package/dist/src/utils/resolve-mediatype.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"resolve-mediatype.js","sourceRoot":"","sources":["../../../src/utils/resolve-mediatype.ts"],"names":[],"mappings":"AAGA;;;;GAIG;AACH,MAAM,UAAU,gBAAgB,CAAC,MAAoB;;IACnD;;;;;;;;;;OAUG;IACH,IAAI,CAAA,MAAA,MAAM,aAAN,MAAM,uBAAN,MAAM,CAAE,WAAW,0CAAE,QAAQ,MAAK,kBAAkB,EAAE;QACxD,OAAO,QAAQ,CAAC;KACjB;IAED,OAAO,MAAA,MAAA,MAAM,CAAC,SAAS,0CAAE,KAAK,mCAAI,MAAM,CAAC;AAC3C,CAAC","sourcesContent":["import type { MediaType } from '@internetarchive/field-parsers';\nimport type { SearchResult } from '@internetarchive/search-service';\n\n/**\n * Returns the mediatype string for the given search result, taking into account\n * the special `favorited_search` hit type.\n * @param result The search result to extract a mediatype from\n */\nexport function resolveMediatype(result: SearchResult): MediaType {\n  /**\n   * hit_type == 'favorited_search' is basically a new hit_type\n   * - we are getting from PPS.\n   * - which gives response for fav- collection\n   * - having favorited items like account/collection/item etc..\n   * - as user can also favorite a search result (a search page)\n   * - so we need to have response (having fav- items and fav- search results)\n   *\n   * if backend hit_type == 'favorited_search'\n   * - let's assume a \"search\" as new mediatype\n   */\n  if (result?.rawMetadata?.hit_type === 'favorited_search') {\n    return 'search';\n  }\n\n  return result.mediatype?.value ?? 'data';\n}\n"]}

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

@@ -209,12 +209,13 @@ describe('Collection Browser', () => {
       </collection-browser>`);
         el.baseQuery = 'hello';
         await el.updateComplete;
+        await nextTick();
         const facets = (_a = el.shadowRoot) === null || _a === void 0 ? void 0 : _a.querySelector('collection-facets');
         const sortBar = (_b = el.shadowRoot) === null || _b === void 0 ? void 0 : _b.querySelector('sort-filter-bar');
         const infiniteScroller = (_c = el.shadowRoot) === null || _c === void 0 ? void 0 : _c.querySelector('infinite-scroller');
-        expect(facets).to.exist;
-        expect(sortBar).to.exist;
-        expect(infiniteScroller).to.exist;
+        expect(facets, 'facets').to.exist;
+        expect(sortBar, 'sort bar').to.exist;
+        expect(infiniteScroller, 'infinite scroller').to.exist;
     });
     it('queries the search service when given a base query', async () => {
         var _a, _b, _c;
@@ -332,6 +333,18 @@ describe('Collection Browser', () => {
       </collection-browser>`);
         expect(el.searchType).to.equal(SearchType.FULLTEXT);
     });
+    it('can construct tile models with many fields present', async () => {
+        const searchService = new MockSearchService();
+        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+      </collection-browser>`);
+        el.baseQuery = 'many-fields';
+        await el.updateComplete;
+        await el.initialSearchComplete;
+        const cellTemplate = el.cellForIndex(0);
+        expect(cellTemplate).to.exist;
+        const cell = await fixture(cellTemplate);
+        expect(cell).to.exist;
+    });
     it('applies loggedin flag to tile models if needed', async () => {
         var _a;
         const searchService = new MockSearchService();
@@ -558,9 +571,10 @@ describe('Collection Browser', () => {
         expect(el.selectedSort).to.equal(SortField.default);
         el.baseQuery = 'foo';
         await el.updateComplete;
-        const sortBar = (_a = el.shadowRoot) === null || _a === void 0 ? void 0 : _a.querySelector('sort-filter-bar');
+        await nextTick();
+        const sortBar = (_a = el.shadowRoot) === null || _a === void 0 ? void 0 : _a.querySelector('#content-container sort-filter-bar');
         const sortSelector = (_b = sortBar === null || sortBar === void 0 ? void 0 : sortBar.shadowRoot) === null || _b === void 0 ? void 0 : _b.querySelector('#desktop-sort-selector');
-        expect(sortSelector).to.exist;
+        expect(sortSelector, 'sort bar').to.exist;
         // Click the title sorter
         (_d = (_c = [...sortSelector === null || sortSelector === void 0 ? void 0 : sortSelector.children] // tsc doesn't know children is iterable
             .find(child => { var _a; return ((_a = child.textContent) === null || _a === void 0 ? void 0 : _a.trim()) === 'Title'; })) === null || _c === void 0 ? void 0 : _c.querySelector('button')) === null || _d === void 0 ? void 0 : _d.click();
@@ -654,7 +668,10 @@ describe('Collection Browser', () => {
     it('sets date range query when date picker selection changed', async () => {
         var _a, _b, _c;
         const searchService = new MockSearchService();
-        const el = await fixture(html `<collection-browser .searchService=${searchService}>
+        const el = await fixture(html `<collection-browser
+        .searchService=${searchService}
+        .suppressPlaceholders=${true}
+      >
       </collection-browser>`);
         el.baseQuery = 'years'; // Includes year_histogram aggregation in response
         el.showHistogramDatePicker = true;
@@ -664,7 +681,7 @@ describe('Collection Browser', () => {
         // Wait for the date picker to be rendered (which may take until the next tick)
         await nextTick();
         const histogram = (_b = facets === null || facets === void 0 ? void 0 : facets.shadowRoot) === null || _b === void 0 ? void 0 : _b.querySelector('histogram-date-range');
-        expect(histogram).to.exist;
+        expect(histogram, 'histogram exists').to.exist;
         // Enter a new min date into the date picker
         const minDateInput = (_c = histogram.shadowRoot) === null || _c === void 0 ? void 0 : _c.querySelector('#date-min');
         const pressEnterEvent = new KeyboardEvent('keyup', {
@@ -804,13 +821,14 @@ describe('Collection Browser', () => {
         // infinite scroller does exist.
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
+        await nextTick();
         const infiniteScroller = (_a = el.shadowRoot) === null || _a === void 0 ? void 0 : _a.querySelector('infinite-scroller');
         expect(infiniteScroller).to.exist;
         const oldScrollToCell = infiniteScroller.scrollToCell;
         const spy = sinon.spy();
         infiniteScroller.scrollToCell = spy;
         await el.goToPage(1);
-        expect(spy.callCount).to.equal(1);
+        expect(spy.callCount, 'scroll to page fires once').to.equal(1);
         infiniteScroller.scrollToCell = oldScrollToCell;
     });
     it('shows mobile facets in mobile view', async () => {
@@ -858,6 +876,7 @@ describe('Collection Browser', () => {
         // Infinite scroller won't exist unless there's a base query
         el.baseQuery = 'collection:foo';
         await el.updateComplete;
+        await nextTick();
         const infiniteScroller = (_a = el.shadowRoot) === null || _a === void 0 ? void 0 : _a.querySelector('infinite-scroller');
         infiniteScroller.reload = infiniteScrollerRefreshSpy;
         expect(infiniteScrollerRefreshSpy.called).to.be.false;
@@ -865,33 +884,34 @@ describe('Collection Browser', () => {
         // testing: `loggedIn`
         el.loggedIn = true;
         await el.updateComplete;
-        expect(infiniteScrollerRefreshSpy.called).to.be.true;
-        expect(infiniteScrollerRefreshSpy.callCount).to.equal(1);
+        expect(infiniteScrollerRefreshSpy.called, 'Infinite Scroller Refresh').to.be
+            .true;
+        expect(infiniteScrollerRefreshSpy.callCount, 'Infinite Scroller Refresh call count').to.equal(1);
         el.loggedIn = false;
         await el.updateComplete;
-        expect(infiniteScrollerRefreshSpy.callCount).to.equal(2);
+        expect(infiniteScrollerRefreshSpy.callCount, '2nd Infinite Scroller Refresh').to.equal(2);
         // testing: `displayMode`
         el.displayMode = 'list-compact';
         el.searchContext = 'beta-search';
         await el.updateComplete;
-        expect(infiniteScrollerRefreshSpy.callCount).to.equal(3);
+        expect(infiniteScrollerRefreshSpy.callCount, '3rd Infinite Scroller Refresh').to.equal(3);
         expect(mockAnalyticsHandler.callCategory).to.equal('beta-search');
         expect(mockAnalyticsHandler.callAction).to.equal('displayMode');
         expect(mockAnalyticsHandler.callLabel).to.equal('list-compact');
         el.displayMode = 'list-detail';
         await el.updateComplete;
-        expect(infiniteScrollerRefreshSpy.callCount).to.equal(4);
+        expect(infiniteScrollerRefreshSpy.callCount, '4th Infinite Scroller Refresh').to.equal(4);
         expect(mockAnalyticsHandler.callCategory).to.equal('beta-search');
         expect(mockAnalyticsHandler.callAction).to.equal('displayMode');
         expect(mockAnalyticsHandler.callLabel).to.equal('list-detail');
         // testing: `baseNavigationUrl`
         el.baseNavigationUrl = 'https://funtestsite.com';
         await el.updateComplete;
-        expect(infiniteScrollerRefreshSpy.callCount).to.equal(5);
+        expect(infiniteScrollerRefreshSpy.callCount, '5th Infinite Scroller Refresh').to.equal(5);
         // testing: `baseImageUrl`
         el.baseImageUrl = 'https://funtestsiteforimages.com';
         await el.updateComplete;
-        expect(infiniteScrollerRefreshSpy.callCount).to.equal(6);
+        expect(infiniteScrollerRefreshSpy.callCount, '6th Infinite Scroller Refresh').to.equal(6);
     });
     it('query the search service for single result', async () => {
         var _a, _b;
@@ -1102,7 +1122,7 @@ describe('Collection Browser', () => {
         expect(el.displayMode).to.equal('grid');
     });
     it('can remove all checked tiles', async () => {
-        var _a, _b, _c, _d, _e, _f, _g, _h;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j;
         const searchService = new MockSearchService();
         const el = await fixture(html `<collection-browser
         .searchService=${searchService}
@@ -1136,10 +1156,10 @@ describe('Collection Browser', () => {
         // Remove checked tiles and verify that we only kept the second tile
         el.removeCheckedTiles();
         await el.updateComplete;
-        tiles = (_g = infiniteScroller.shadowRoot) === null || _g === void 0 ? void 0 : _g.querySelectorAll('tile-dispatcher');
+        expect((_g = el === null || el === void 0 ? void 0 : el.dataSource) === null || _g === void 0 ? void 0 : _g.size, 'data source count').to.equal(1);
+        tiles = (_j = (_h = el.shadowRoot) === null || _h === void 0 ? void 0 : _h.querySelector('infinite-scroller').shadowRoot) === null || _j === void 0 ? void 0 : _j.querySelectorAll('tile-dispatcher');
         expect(tiles).to.exist;
-        expect(tiles === null || tiles === void 0 ? void 0 : tiles.length).to.equal(1);
-        expect((_h = tiles[0].model) === null || _h === void 0 ? void 0 : _h.identifier).to.equal('bar');
+        expect(tiles.length, 'tile count after `el.removeCheckedTiles()`').to.equal(1);
     });
     it('can check/uncheck all tiles', async () => {
         const searchService = new MockSearchService();