@internetarchive/collection-browser 0.4.3-alpha.9 → 0.4.4-alpha

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

@@ -43,7 +43,9 @@ describe('Item Tile', () => {
   });
 
   it('should not render with created-by but date element', async () => {
-    const el = await fixture<ItemTile>(html`<item-tile></item-tile>`);
+    const el = await fixture<ItemTile>(
+      html`<item-tile .model=${{ datePublished: new Date() }}></item-tile>`
+    );
 
     el.sortParam = {
       field: 'date',
@@ -59,7 +61,9 @@ describe('Item Tile', () => {
   });
 
   it('should not render with created-by but reviewdate element', async () => {
-    const el = await fixture<ItemTile>(html`<item-tile></item-tile>`);
+    const el = await fixture<ItemTile>(
+      html`<item-tile .model=${{ dateReviewed: new Date() }}></item-tile>`
+    );
 
     el.sortParam = {
       field: 'reviewdate',
@@ -75,7 +79,9 @@ describe('Item Tile', () => {
   });
 
   it('should not render with created-by but publicdate element', async () => {
-    const el = await fixture<ItemTile>(html`<item-tile></item-tile>`);
+    const el = await fixture<ItemTile>(
+      html`<item-tile .model=${{ dateArchived: new Date() }}></item-tile>`
+    );
 
     el.sortParam = {
       field: 'publicdate',
@@ -90,8 +96,28 @@ describe('Item Tile', () => {
     expect(dateSortedBy).to.exist;
   });
 
+  it('should not render date-sorted-by element if date is not provided', async () => {
+    const el = await fixture<ItemTile>(
+      html`<item-tile .model=${{ dateArchived: undefined }}></item-tile>`
+    );
+
+    el.sortParam = {
+      field: 'publicdate',
+      direction: 'desc',
+    };
+    await el.updateComplete;
+
+    const createdBy = el.shadowRoot?.querySelector('.created-by');
+    const dateSortedBy = el.shadowRoot?.querySelector('.date-sorted-by');
+
+    expect(createdBy).to.not.exist;
+    expect(dateSortedBy).to.not.exist;
+  });
+
   it('should render with created-by when sorting not related to date', async () => {
-    const el = await fixture<ItemTile>(html`<item-tile></item-tile>`);
+    const el = await fixture<ItemTile>(
+      html`<item-tile .model=${{ dateAdded: new Date() }}></item-tile>`
+    );
 
     el.sortParam = {
       field: 'addeddate',

package/test/utils/format-unit-size.test.ts

@@ -0,0 +1,21 @@
+import { expect } from '@open-wc/testing';
+import { nothing } from 'lit';
+import { formatUnitSize } from '../../src/utils/format-unit-size';
+
+describe('formatUnitSize', () => {
+  it('returns blank when undefined number', () => {
+    expect(formatUnitSize(undefined, 1)).to.equal(nothing);
+  });
+
+  it('returns collection data is in singular form', () => {
+    expect(formatUnitSize(1025, 1)).to.equal('1 kilobyte');
+  });
+
+  it('return collection size data in kilobytes and plural form', () => {
+    expect(formatUnitSize(15000, 1)).to.equal('14.6 kilobytes');
+  });
+
+  it('return huge collection size data', () => {
+    expect(formatUnitSize(454154654134, 1)).to.equal('423 gigabytes');
+  });
+});

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
+});