@internetarchive/collection-browser 1.14.17-beta.5 → 1.14.18-alpha.0

package/test/mocks/mock-search-responses.ts

@@ -6,6 +6,7 @@ import {
   SearchServiceError,
   TextHit,
 } from '@internetarchive/search-service';
+import { WebArchiveHit } from '@internetarchive/search-service/dist/src/models/hit-types/web-archive-hit';
 import { SearchServiceErrorType } from '@internetarchive/search-service/dist/src/search-service-error';
 
 export const getMockSuccessSingleResult: () => Result<
@@ -49,6 +50,68 @@ export const getMockSuccessSingleResult: () => Result<
   },
 });
 
+export const getMockSuccessManyFields: () => Result<
+  SearchResponse,
+  SearchServiceError
+> = () => ({
+  success: {
+    request: {
+      kind: 'hits',
+      clientParameters: {
+        user_query: 'many-fields',
+        sort: [],
+      },
+      backendRequests: {
+        primary: {
+          kind: 'hits',
+          finalized_parameters: {
+            user_query: 'many-fields',
+            sort: [],
+          },
+        },
+      },
+    },
+    rawResponse: {},
+    response: {
+      totalResults: 1,
+      returnedCount: 1,
+      results: [
+        new ItemHit({
+          fields: {
+            identifier: 'foo',
+            avg_rating: 3.5,
+            collection: ['foo', 'bar', 'no-preview', 'loggedin'],
+            collection_files_count: 123,
+            collection_size: 234,
+            creator: ['baz', 'boop'],
+            date: '2010-01-03T01:23:45Z',
+            addeddate: '2010-01-01T01:23:45Z',
+            publicdate: '2010-01-02T01:23:45Z',
+            reviewdate: '2010-01-04T01:23:45Z',
+            description: 'foo bar baz',
+            downloads: 246,
+            issue: 1,
+            item_count: 20,
+            mediatype: 'texts',
+            num_favorites: 12,
+            num_reviews: 23,
+            source: 'foo bar',
+            subject: ['baz', 'quux'],
+            title: 'Foo Bar',
+            volume: 2,
+            week: 50,
+            __href__: 'https://archive.org/details/foo',
+          },
+        }),
+      ],
+    },
+    responseHeader: {
+      succeeded: true,
+      query_time: 0,
+    },
+  },
+});
+
 export const getMockSuccessWithYearHistogramAggs: () => Result<
   SearchResponse,
   SearchServiceError
@@ -408,6 +471,10 @@ export const getMockSuccessWithCollectionAggregations: () => Result<
           ],
         }),
       },
+      collectionTitles: {
+        foo: 'Foo',
+        bar: 'Bar',
+      },
     },
     responseHeader: {
       succeeded: true,
@@ -508,6 +575,40 @@ export const getMockSuccessMultipleResults: () => Result<
   },
 });
 
+export const getMockSuccessNoResults: () => Result<
+  SearchResponse,
+  SearchServiceError
+> = () => ({
+  success: {
+    request: {
+      kind: 'hits',
+      clientParameters: {
+        user_query: 'no-results',
+        sort: [],
+      },
+      backendRequests: {
+        primary: {
+          kind: 'hits',
+          finalized_parameters: {
+            user_query: 'no-results',
+            sort: [],
+          },
+        },
+      },
+    },
+    rawResponse: {},
+    response: {
+      totalResults: 0,
+      returnedCount: 0,
+      results: [],
+    },
+    responseHeader: {
+      succeeded: true,
+      query_time: 0,
+    },
+  },
+});
+
 export const getMockSuccessMultiLineDescription: () => Result<
   SearchResponse,
   SearchServiceError
@@ -776,6 +877,49 @@ export const getMockSuccessWithParentCollections: () => Result<
   },
 });
 
+export const getMockSuccessWithWebArchiveHits: () => Result<
+  SearchResponse,
+  SearchServiceError
+> = () => ({
+  success: {
+    request: {
+      kind: 'hits',
+      clientParameters: {
+        user_query: 'web-archive',
+        sort: [],
+      },
+      backendRequests: {
+        primary: {
+          kind: 'hits',
+          finalized_parameters: {
+            user_query: 'web-archive',
+            sort: [],
+          },
+        },
+      },
+    },
+    rawResponse: {},
+    response: {
+      totalResults: 1,
+      returnedCount: 1,
+      results: [
+        new WebArchiveHit({
+          fields: {
+            url: 'https://example.com',
+            capture_dates: ['2010-01-02T03:04:05Z'],
+            __href__:
+              'https://web.archive.org/web/20100102030405/https%3A%2F%2Fexample.com',
+          },
+        }),
+      ],
+    },
+    responseHeader: {
+      succeeded: true,
+      query_time: 0,
+    },
+  },
+});
+
 export const getMockErrorResult: () => Result<
   SearchResponse,
   SearchServiceError

package/test/mocks/mock-search-service.ts

@@ -26,6 +26,9 @@ import {
   getMockSuccessWithConciseDefaultSort,
   getMockSuccessWithDefaultFavSort,
   getMockSuccessWithParentCollections,
+  getMockSuccessManyFields,
+  getMockSuccessNoResults,
+  getMockSuccessWithWebArchiveHits,
 } from './mock-search-responses';
 
 const responses: Record<
@@ -47,6 +50,9 @@ const responses: Record<
   'default-sort-concise': getMockSuccessWithConciseDefaultSort,
   'fav-sort': getMockSuccessWithDefaultFavSort,
   'parent-collections': getMockSuccessWithParentCollections,
+  'web-archive': getMockSuccessWithWebArchiveHits,
+  'many-fields': getMockSuccessManyFields,
+  'no-results': getMockSuccessNoResults,
   error: getMockErrorResult,
   malformed: getMockMalformedResult,
 };

package/test/tile-stats.test.ts

@@ -54,4 +54,108 @@ describe('Tile Stats', () => {
     expect(favoritesStatCount).to.equalSnapshot(2);
     expect(reviewsStatCount).to.equalSnapshot(3);
   });
+
+  it('should render view count for non-account items', async () => {
+    const el = await fixture<TileStats>(
+      html`
+        <tile-stats
+          .mediatype=${'texts'}
+          .viewCount=${4}
+          .favCount=${5}
+          .commentCount=${6}
+        >
+        </tile-stats>
+      `
+    );
+
+    const statsRow = el.shadowRoot?.querySelector('#stats-row');
+
+    const mediatypeStat = statsRow?.children.item(0);
+    // get second column item in stats row
+    const viewStatCount = statsRow?.children
+      .item(1)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+    // get third column item in stats row
+    const favoritesStatCount = statsRow?.children
+      .item(2)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+    // get fourth column item in stats row
+    const reviewsStatCount = statsRow?.children
+      .item(3)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+
+    expect(mediatypeStat).to.exist;
+    expect(viewStatCount).to.match(/Views:\s+4/);
+    expect(favoritesStatCount).to.match(/Favorites:\s+5/);
+    expect(reviewsStatCount).to.match(/Reviews:\s+6/);
+  });
+
+  it('handles missing counts gracefully', async () => {
+    const el = await fixture<TileStats>(
+      html`
+        <tile-stats .mediatype=${'texts'} .favCount=${5} .commentCount=${6}>
+        </tile-stats>
+      `
+    );
+
+    const statsRow = el.shadowRoot?.querySelector('#stats-row');
+
+    const mediatypeStat = statsRow?.children.item(0);
+    // get second column item in stats row
+    const viewStatCount = statsRow?.children
+      .item(1)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+    // get third column item in stats row
+    const favoritesStatCount = statsRow?.children
+      .item(2)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+    // get fourth column item in stats row
+    const reviewsStatCount = statsRow?.children
+      .item(3)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+
+    expect(mediatypeStat).to.exist;
+    expect(viewStatCount).to.match(/Views:\s+0/);
+    expect(favoritesStatCount).to.match(/Favorites:\s+5/);
+    expect(reviewsStatCount).to.match(/Reviews:\s+6/);
+  });
+
+  it('handles missing counts gracefully for accounts', async () => {
+    const el = await fixture<TileStats>(
+      html`
+        <tile-stats .mediatype=${'account'} .favCount=${5} .commentCount=${6}>
+        </tile-stats>
+      `
+    );
+
+    const statsRow = el.shadowRoot?.querySelector('#stats-row');
+
+    const mediatypeStat = statsRow?.children.item(0);
+    // get second column item in stats row
+    const itemStatCount = statsRow?.children
+      .item(1)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+    // get third column item in stats row
+    const favoritesStatCount = statsRow?.children
+      .item(2)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+    // get fourth column item in stats row
+    const reviewsStatCount = statsRow?.children
+      .item(3)
+      ?.querySelector('.status-text')
+      ?.textContent?.trim();
+
+    expect(mediatypeStat).to.exist;
+    expect(itemStatCount).to.match(/Uploads:\s+0/);
+    expect(favoritesStatCount).to.match(/Favorites:\s+5/);
+    expect(reviewsStatCount).to.match(/Reviews:\s+6/);
+  });
 });

package/test/tiles/grid/item-tile.test.ts

@@ -374,4 +374,59 @@ describe('Item Tile', () => {
 
     expect(volumeIssueBlock).to.not.exist;
   });
+
+  it('should render web capture date links if present', async () => {
+    const captureDates = [
+      new Date('2010-01-02T12:34:56Z'),
+      new Date('2011-02-03T12:43:21Z'),
+    ];
+
+    const el = await fixture<ItemTile>(html`
+      <item-tile
+        .model=${{
+          identifier: 'foo',
+          title: 'https://example.com/',
+          captureDates,
+        }}
+      ></item-tile>
+    `);
+
+    const captureDatesUl = el.shadowRoot?.querySelector('.capture-dates');
+    expect(captureDatesUl, 'capture dates container').to.exist;
+    expect(captureDatesUl?.children.length).to.equal(2);
+
+    const firstDateLink = captureDatesUl?.children[0]?.querySelector('a[href]');
+    expect(firstDateLink, 'first date link').to.exist;
+    expect(firstDateLink?.getAttribute('href')).to.equal(
+      'https://web.archive.org/web/20100102123456/https%3A%2F%2Fexample.com%2F'
+    );
+    expect(firstDateLink?.textContent?.trim()).to.equal('Jan 02, 2010');
+
+    const secondDateLink =
+      captureDatesUl?.children[1]?.querySelector('a[href]');
+    expect(secondDateLink, 'second date link').to.exist;
+    expect(secondDateLink?.getAttribute('href')).to.equal(
+      'https://web.archive.org/web/20110203124321/https%3A%2F%2Fexample.com%2F'
+    );
+    expect(secondDateLink?.textContent?.trim()).to.equal('Feb 03, 2011');
+  });
+
+  it('should not render web captures if no title is present', async () => {
+    const captureDates = [
+      new Date('2010-01-02T12:34:56Z'),
+      new Date('2011-02-03T12:43:21Z'),
+    ];
+
+    const el = await fixture<ItemTile>(html`
+      <item-tile
+        .model=${{
+          identifier: 'foo',
+          captureDates,
+        }}
+      ></item-tile>
+    `);
+
+    const captureDatesUl = el.shadowRoot?.querySelector('.capture-dates');
+    expect(captureDatesUl).not.to.exist;
+  });
 });

package/test/tiles/hover/hover-pane-controller.test.ts

@@ -9,6 +9,7 @@ import {
   HoverPaneProviderInterface,
 } from '../../../src/tiles/hover/hover-pane-controller';
 import type { TileHoverPane } from '../../../src/tiles/hover/tile-hover-pane';
+import { TileModel } from '../../../src/models';
 
 @customElement('host-element')
 class HostElement extends LitElement implements HoverPaneProviderInterface {
@@ -33,24 +34,25 @@ class HostElement extends LitElement implements HoverPaneProviderInterface {
   }
 
   getHoverPaneProps(): HoverPaneProperties {
+    const tileModel = new TileModel({});
+    tileModel.checked = false;
+    tileModel.collectionFilesCount = 1;
+    tileModel.collections = ['foo', 'bar'];
+    tileModel.collectionSize = 1;
+    tileModel.commentCount = 1;
+    tileModel.contentWarning = false;
+    tileModel.creators = ['foo', 'bar'];
+    tileModel.favCount = 1;
+    tileModel.identifier = 'foo';
+    tileModel.itemCount = 1;
+    tileModel.loginRequired = false;
+    tileModel.mediatype = 'data';
+    tileModel.subjects = ['foo', 'bar'];
+    tileModel.title = 'foo';
+    tileModel.viewCount = 1;
+
     return {
-      model: {
-        checked: false,
-        collectionFilesCount: 1,
-        collections: ['foo', 'bar'],
-        collectionSize: 1,
-        commentCount: 1,
-        contentWarning: false,
-        creators: ['foo', 'bar'],
-        favCount: 1,
-        identifier: 'foo',
-        itemCount: 1,
-        loginRequired: false,
-        mediatype: 'data',
-        subjects: ['foo', 'bar'],
-        title: 'foo',
-        viewCount: 1,
-      },
+      model: tileModel,
       loggedIn: false,
       sortParam: null,
     };

package/test/tiles/list/tile-list.test.ts

@@ -392,4 +392,59 @@ describe('List Tile', () => {
     expect(creatorBlock).to.exist;
     expect(creatorBlock?.textContent?.trim()).to.equal('Archivist since 2015');
   });
+
+  it('should render web capture date links if present', async () => {
+    const captureDates = [
+      new Date('2010-01-02T12:34:56Z'),
+      new Date('2011-02-03T12:43:21Z'),
+    ];
+
+    const el = await fixture<TileList>(html`
+      <tile-list
+        .model=${{
+          identifier: 'foo',
+          title: 'https://example.com/',
+          captureDates,
+        }}
+      ></tile-list>
+    `);
+
+    const captureDatesUl = el.shadowRoot?.querySelector('.capture-dates');
+    expect(captureDatesUl, 'capture dates container').to.exist;
+    expect(captureDatesUl?.children.length).to.equal(2);
+
+    const firstDateLink = captureDatesUl?.children[0]?.querySelector('a[href]');
+    expect(firstDateLink, 'first date link').to.exist;
+    expect(firstDateLink?.getAttribute('href')).to.equal(
+      'https://web.archive.org/web/20100102123456/https%3A%2F%2Fexample.com%2F'
+    );
+    expect(firstDateLink?.textContent?.trim()).to.equal('Jan 02, 2010');
+
+    const secondDateLink =
+      captureDatesUl?.children[1]?.querySelector('a[href]');
+    expect(secondDateLink, 'second date link').to.exist;
+    expect(secondDateLink?.getAttribute('href')).to.equal(
+      'https://web.archive.org/web/20110203124321/https%3A%2F%2Fexample.com%2F'
+    );
+    expect(secondDateLink?.textContent?.trim()).to.equal('Feb 03, 2011');
+  });
+
+  it('should not render web captures if no title is present', async () => {
+    const captureDates = [
+      new Date('2010-01-02T12:34:56Z'),
+      new Date('2011-02-03T12:43:21Z'),
+    ];
+
+    const el = await fixture<TileList>(html`
+      <tile-list
+        .model=${{
+          identifier: 'foo',
+          captureDates,
+        }}
+      ></tile-list>
+    `);
+
+    const captureDatesUl = el.shadowRoot?.querySelector('.capture-dates');
+    expect(captureDatesUl).not.to.exist;
+  });
 });

package/test/tiles/tile-dispatcher.test.ts

@@ -48,6 +48,19 @@ describe('Tile Dispatcher', () => {
     expect(accountTile).to.exist;
   });
 
+  it('should render search-tile for grid mode and search mediatype', async () => {
+    const el = await fixture<TileDispatcher>(html`
+      <tile-dispatcher
+        .tileDisplayMode=${'grid'}
+        .model=${{ mediatype: 'search' }}
+      >
+      </tile-dispatcher>
+    `);
+
+    const searchTile = el.shadowRoot?.querySelector('search-tile');
+    expect(searchTile).to.exist;
+  });
+
   it('should render tile-list for extended list mode', async () => {
     const el = await fixture<TileDispatcher>(html`
       <tile-dispatcher .tileDisplayMode=${'list-detail'} .model=${{}}>

package/tsconfig.json

@@ -1,21 +1,21 @@
-{
-  "compilerOptions": {
-    "target": "es2018",
-    "module": "esnext",
-    "moduleResolution": "node",
-    "noEmitOnError": true,
-    "lib": ["es2017", "dom"],
-    "strict": true,
-    "esModuleInterop": false,
-    "allowSyntheticDefaultImports": true,
-    "experimentalDecorators": true,
-    "importHelpers": true,
-    "outDir": "dist",
-    "sourceMap": true,
-    "inlineSources": true,
-    "rootDir": "./",
-    "declaration": true,
-    "importsNotUsedAsValues": "error"
-  },
-  "include": ["src", "test", "index.ts", "types"],
-}
+{
+  "compilerOptions": {
+    "target": "es2018",
+    "module": "esnext",
+    "moduleResolution": "node",
+    "noEmitOnError": true,
+    "lib": ["es2017", "dom"],
+    "strict": true,
+    "esModuleInterop": false,
+    "allowSyntheticDefaultImports": true,
+    "experimentalDecorators": true,
+    "importHelpers": true,
+    "outDir": "dist",
+    "sourceMap": true,
+    "inlineSources": true,
+    "rootDir": "./",
+    "declaration": true,
+    "importsNotUsedAsValues": "error"
+  },
+  "include": ["src", "test", "index.ts", "types"],
+}

package/web-dev-server.config.mjs

@@ -1,30 +1,30 @@
-// import { hmrPlugin, presets } from '@open-wc/dev-server-hmr';
-
-/** Use Hot Module replacement by adding --hmr to the start command */
-const hmr = process.argv.includes('--hmr');
-
-export default /** @type {import('@web/dev-server').DevServerConfig} */ ({
-  nodeResolve: true,
-  open: '/',
-  watch: !hmr,
-
-  /** Compile JS for older browsers. Requires @web/dev-server-esbuild plugin */
-  // esbuildTarget: 'auto'
-
-  /** Set appIndex to enable SPA routing */
-  // appIndex: 'demo/index.html',
-
-  /** Confgure bare import resolve plugin */
-  // nodeResolve: {
-  //   exportConditions: ['browser', 'development']
-  // },
-
-  plugins: [
-    /** Use Hot Module Replacement by uncommenting. Requires @open-wc/dev-server-hmr plugin */
-    // hmr && hmrPlugin({ exclude: ['**/*/node_modules/**/*'], presets: [presets.litElement] }),
-  ],
-
-  http2: true,
-  sslCert: './local.archive.org.cert',
-  sslKey: './local.archive.org.key',
-});
+// import { hmrPlugin, presets } from '@open-wc/dev-server-hmr';
+
+/** Use Hot Module replacement by adding --hmr to the start command */
+const hmr = process.argv.includes('--hmr');
+
+export default /** @type {import('@web/dev-server').DevServerConfig} */ ({
+  nodeResolve: true,
+  open: '/',
+  watch: !hmr,
+
+  /** Compile JS for older browsers. Requires @web/dev-server-esbuild plugin */
+  // esbuildTarget: 'auto'
+
+  /** Set appIndex to enable SPA routing */
+  // appIndex: 'demo/index.html',
+
+  /** Confgure bare import resolve plugin */
+  // nodeResolve: {
+  //   exportConditions: ['browser', 'development']
+  // },
+
+  plugins: [
+    /** Use Hot Module Replacement by uncommenting. Requires @open-wc/dev-server-hmr plugin */
+    // hmr && hmrPlugin({ exclude: ['**/*/node_modules/**/*'], presets: [presets.litElement] }),
+  ],
+
+  http2: true,
+  sslCert: './local.archive.org.cert',
+  sslKey: './local.archive.org.key',
+});

package/web-test-runner.config.mjs

@@ -1,41 +1,41 @@
-// import { playwrightLauncher } from '@web/test-runner-playwright';
-
-const filteredLogs = ['Running in dev mode', 'lit-html is in dev mode'];
-
-export default /** @type {import("@web/test-runner").TestRunnerConfig} */ ({
-  /** Test files to run */
-  files: 'dist/test/**/*.test.js',
-
-  /** Resolve bare module imports */
-  nodeResolve: {
-    exportConditions: ['browser', 'development'],
-  },
-
-  /** Filter out lit dev mode logs */
-  filterBrowserLogs(log) {
-    for (const arg of log.args) {
-      if (typeof arg === 'string' && filteredLogs.some(l => arg.includes(l))) {
-        return false;
-      }
-    }
-    return true;
-  },
-
-  /** Compile JS for older browsers. Requires @web/dev-server-esbuild plugin */
-  // esbuildTarget: 'auto',
-
-  /** Amount of browsers to run concurrently */
-  // concurrentBrowsers: 2,
-
-  /** Amount of test files per browser to test concurrently */
-  // concurrency: 1,
-
-  /** Browsers to run tests on */
-  // browsers: [
-  //   playwrightLauncher({ product: 'chromium' }),
-  //   playwrightLauncher({ product: 'firefox' }),
-  //   playwrightLauncher({ product: 'webkit' }),
-  // ],
-
-  // See documentation for all available options
-});
+// import { playwrightLauncher } from '@web/test-runner-playwright';
+
+const filteredLogs = ['Running in dev mode', 'lit-html is in dev mode'];
+
+export default /** @type {import("@web/test-runner").TestRunnerConfig} */ ({
+  /** Test files to run */
+  files: 'dist/test/**/*.test.js',
+
+  /** Resolve bare module imports */
+  nodeResolve: {
+    exportConditions: ['browser', 'development'],
+  },
+
+  /** Filter out lit dev mode logs */
+  filterBrowserLogs(log) {
+    for (const arg of log.args) {
+      if (typeof arg === 'string' && filteredLogs.some(l => arg.includes(l))) {
+        return false;
+      }
+    }
+    return true;
+  },
+
+  /** Compile JS for older browsers. Requires @web/dev-server-esbuild plugin */
+  // esbuildTarget: 'auto',
+
+  /** Amount of browsers to run concurrently */
+  // concurrentBrowsers: 2,
+
+  /** Amount of test files per browser to test concurrently */
+  // concurrency: 1,
+
+  /** Browsers to run tests on */
+  // browsers: [
+  //   playwrightLauncher({ product: 'chromium' }),
+  //   playwrightLauncher({ product: 'firefox' }),
+  //   playwrightLauncher({ product: 'webkit' }),
+  // ],
+
+  // See documentation for all available options
+});

package/types/sentry.d.ts

@@ -1 +0,0 @@
-declare module 'Sentry';