fix:mirador_crash

__tests__/src/components/ManifestListItem.test.js

@@ -1,6 +1,6 @@
 import { render, screen } from '@tests/utils/test-utils';
 import userEvent from '@testing-library/user-event';
-
+import { getManifestoInstance } from '../../../src/state/selectors';
 import { ManifestListItem } from '../../../src/components/ManifestListItem';
 
 /** */
@@ -46,6 +46,17 @@ describe('ManifestListItem', () => {
     expect(screen.getByText('The resource cannot be added:')).toBeInTheDocument();
     expect(screen.getByText('http://example.com')).toBeInTheDocument();
   });
+
+  it('renders an error message when fetched manifest is empty', () => {
+    const state = { manifests: { x: { json: {} } } };
+    const manifesto = getManifestoInstance(state, { manifestId: 'x' });
+
+    createWrapper({ error: !manifesto });
+
+    expect(screen.getByText('The resource cannot be added:')).toBeInTheDocument();
+    expect(screen.getByText('http://example.com')).toBeInTheDocument();
+  });
+
   it('updates and adds window when button clicked', async () => {
     const user = userEvent.setup();
     const addWindow = vi.fn();

__tests__/src/selectors/manifests.test.js

@@ -1,4 +1,6 @@
 import { Utils } from 'manifesto.js';
+
+import collection from '../../fixtures/version-2/collection.json';
 import manifestFixture001 from '../../fixtures/version-2/001.json';
 import manifestFixture002 from '../../fixtures/version-2/002.json';
 import manifestFixture019 from '../../fixtures/version-2/019.json';
@@ -7,6 +9,7 @@ import manifestFixturev3001 from '../../fixtures/version-3/001.json';
 import manifestFixtureWithAProvider from '../../fixtures/version-3/with_a_provider.json';
 import manifestFixtureFg165hz3589 from '../../fixtures/version-2/fg165hz3589.json';
 import manifestFixtureRelated from '../../fixtures/version-2/related.json';
+
 import {
   getManifestoInstance,
   getManifestLocale,
@@ -59,6 +62,14 @@ describe('getManifestoInstance', () => {
     const received = getManifestoInstance(state, { manifestId: 'x' });
     expect(received.id).toEqual('http://iiif.io/api/presentation/2.1/example/fixtures/19/manifest.json');
   });
+  it('creates a manifesto collection instance and patches getSequences', () => {
+    const state = { manifests: { x: { json: collection } } };
+
+    const received = getManifestoInstance(state, { manifestId: 'x' });
+    expect(typeof received.getSequences).toBe('function');
+
+    expect(received.getSequences()).toEqual([]);
+  });
   it('is cached based off of input props', () => {
     const state = { manifests: { x: { json: manifestFixture019 } } };
     const received = getManifestoInstance(state, { manifestId: 'x' });

src/containers/ManifestListItem.js

@@ -23,7 +23,7 @@ const mapStateToProps = (state, { manifestId, provider }) => {
     : getCanvases(state, { manifestId }).length;
   return {
     active: getWindowManifests(state).includes(manifestId),
-    error: manifest.error,
+    error: manifest.error || !manifesto,
     isCollection,
     isFetching: manifest.isFetching,
     isMultipart: isCollection

src/state/selectors/manifests.js

@@ -12,11 +12,14 @@ function createManifestoInstance(json, locale) {
   if (!json) return undefined;
   // Use JSON stringify/parse to create a deep copy and prevent Manifesto from mutating the json
   const manifestoObject = Utils.parseManifest(JSON.parse(JSON.stringify(json)), locale ? { locale } : undefined);
-  // Local patching of Manifesto so that when its a Collection, it behaves similarly
-  if (typeof manifestoObject.getSequences != 'function') {
-    manifestoObject.getSequences = () => [];
+  if (manifestoObject) {
+    // Local patching of Manifesto so that when its a Collection, it behaves similarly
+    if (typeof manifestoObject.getSequences != 'function') {
+      manifestoObject.getSequences = () => [];
+    }
+    return manifestoObject;
   }
-  return manifestoObject;
+  return undefined;
 }
 
 /** */