@internetarchive/bookreader 5.0.0-28 → 5.0.0-30-a

package/tests/e2e/viewmode.test.js

@@ -1,36 +1,36 @@
-import { Selector } from 'testcafe';
-import BookReader from './models/BookReader';
-import params from './helpers/params';
+// import { Selector } from 'testcafe';
+// import BookReader from './models/BookReader';
+// import params from './helpers/params';
 
-fixture `Viewmode carousel`.page `${params.baseUrl}/BookReaderDemo/viewmode-cycle.html`;
+// fixture `Viewmode carousel`.page `${params.baseUrl}/BookReaderDemo/viewmode-cycle.html`;
 
-test('Clicking `view mode` cycles through view modes', async t => {
-  const { nav } = (new BookReader());
+// test('Clicking `view mode` cycles through view modes', async t => {
+//   const { nav } = (new BookReader());
 
-  // viewmode button only appear on mobile devices
-  await t.resizeWindow(400, 800);
-  // Flip forward one
-  await t.pressKey('right');
+//   // viewmode button only appear on mobile devices
+//   await t.resizeWindow(400, 800);
+//   // Flip forward one
+//   await t.pressKey('right');
 
-  // 2up to thumb
-  await t.click(nav.desktop.viewmode);
-  const thumbnailContainer = Selector('.BRmodeThumb');
-  await t.expect(thumbnailContainer.visible).ok();
-  const thumbImages = thumbnailContainer.find('.BRpageview img');
-  await t.expect(thumbImages.count).gt(0);
+//   // 2up to thumb
+//   await t.click(nav.desktop.viewmode);
+//   const thumbnailContainer = Selector('.BRmodeThumb');
+//   await t.expect(thumbnailContainer.visible).ok();
+//   const thumbImages = thumbnailContainer.find('.BRpageview img');
+//   await t.expect(thumbImages.count).gt(0);
 
-  // thumb to 1up
-  await t.click(nav.desktop.viewmode);
-  const onePageViewContainer = Selector('br-mode-1up');
-  await t.expect(onePageViewContainer.visible).ok();
-  const onePageImages = onePageViewContainer.find('.BRmode1up .BRpagecontainer');
-  // we usually pre-fetch the page in question & 1 before/after it
-  await t.expect(onePageImages.count).gte(3);
+//   // thumb to 1up
+//   await t.click(nav.desktop.viewmode);
+//   const onePageViewContainer = Selector('br-mode-1up');
+//   await t.expect(onePageViewContainer.visible).ok();
+//   const onePageImages = onePageViewContainer.find('.BRmode1up .BRpagecontainer');
+//   // we usually pre-fetch the page in question & 1 before/after it
+//   await t.expect(onePageImages.count).gte(3);
 
-  // 1up to 2up
-  await t.click(nav.desktop.viewmode);
-  const twoPageContainer = Selector('.BRtwopageview');
-  await t.expect(twoPageContainer.visible).ok();
-  const twoPageImages = twoPageContainer.find('img.BRpageimage');
-  await t.expect(twoPageImages.count).gte(2);
-});
+//   // 1up to 2up
+//   await t.click(nav.desktop.viewmode);
+//   const twoPageContainer = Selector('.BRtwopageview');
+//   await t.expect(twoPageContainer.visible).ok();
+//   const twoPageImages = twoPageContainer.find('img.BRpageimage');
+//   await t.expect(twoPageImages.count).gte(2);
+// });

package/tests/jest/BookReader/BookReaderPublicFunctions.test.js

@@ -1,56 +1,66 @@
 import BookReader from '@/src/BookReader';
+import { sleep } from '@/src/plugins/tts/utils';
+import sinon from 'sinon';
 
 beforeAll(() => {
-  global.alert = jest.fn();
+  global.alert = sinon.fake();
 });
 afterEach(() => {
-  jest.restoreAllMocks();
+  sinon.restore();
 });
 
 describe('BookReader.prototype.toggleFullscreen', ()  => {
   test('uses `isFullscreen` to check fullscreen state', () => {
-    const isFSmock = jest.fn();
-    isFSmock.mockReturnValueOnce(false);
+    const isFSmock = sinon.fake();
+    // isFSmock.mockReturnValueOnce(false);
     const br = new BookReader();
     br.mode = br.constMode1up;
-    br.enterFullscreen = jest.fn();
+    br.enterFullscreen = sinon.fake();
     br.isFullscreen = isFSmock;
 
     br.toggleFullscreen();
-    expect(br.isFullscreen).toHaveBeenCalled();
+    expect(br.isFullscreen.callCount).toEqual(1);
   });
 
   test('will always emit an event', async () => {
     const br = new BookReader();
     br.mode = br.constMode2up;
-    br.trigger = jest.fn();
-    br.switchMode = jest.fn();
+    br.trigger = sinon.fake();
+    br.switchMode = sinon.fake();
+    br.updateBrClasses = sinon.fake();
     br.refs.$brContainer = {
-      css: jest.fn(),
+      css: sinon.fake(),
+      animate: (options, speed, style, callback) => callback()
+    };
+    br.refs.$br = {
+      updateBrClasses: sinon.fake(),
+      removeClass: sinon.fake(),
+      addClass: sinon.fake(),
+      css: sinon.fake(),
       animate: (options, speed, style, callback) => callback()
     };
 
     await br.toggleFullscreen();
-    expect(br.trigger).toHaveBeenCalled();
+    expect(br.trigger.callCount).toBeGreaterThan(0);
   });
 
   test('will start with opening fullscreen', () => {
     const br = new BookReader();
     br.mode = br.constMode2up;
-    br.enterFullscreen = jest.fn();
+    br.enterFullscreen = sinon.fake();
 
     br.toggleFullscreen();
-    expect(br.enterFullscreen).toHaveBeenCalled();
+    expect(br.enterFullscreen.callCount).toEqual(1);
   });
 
   test('will close fullscreen if BookReader is in fullscreen', () => {
     const br = new BookReader();
     br.mode = br.constMode1up;
-    br.exitFullScreen = jest.fn();
+    br.exitFullScreen = sinon.fake();
     br.isFullscreenActive = true;
 
     br.toggleFullscreen();
-    expect(br.exitFullScreen).toHaveBeenCalled();
+    expect(br.exitFullScreen.callCount).toEqual(1);
   });
 });
 
@@ -58,11 +68,11 @@ describe('BookReader.prototype.enterFullscreen', ()  => {
   test('will bind `_fullscreenCloseHandler` by default', () => {
     const br = new BookReader();
     br.mode = br.constMode1up;
-    br.switchMode = jest.fn();
-    br.updateBrClasses = jest.fn();
+    br.switchMode = sinon.fake();
+    br.updateBrClasses = sinon.fake();
     br.refs.$brContainer = {
-      css: jest.fn(),
-      animate: jest.fn(),
+      css: sinon.fake(),
+      animate: sinon.fake(),
     };
     expect(br._fullscreenCloseHandler).toBeUndefined();
 
@@ -73,22 +83,24 @@ describe('BookReader.prototype.enterFullscreen', ()  => {
   test('fires certain events when called', async () => {
     const br = new BookReader();
     br.mode = br.constMode2up;
-    br.switchMode = jest.fn();
-    br.updateBrClasses = jest.fn();
-    br.trigger = jest.fn();
-    br.resize = jest.fn();
-    br.jumpToIndex = jest.fn();
+    br.switchMode = sinon.fake();
+    br.updateBrClasses = sinon.fake();
+    br.trigger = sinon.fake();
+    br.resize = sinon.fake();
+    br.jumpToIndex = sinon.fake();
     br.refs.$brContainer = {
-      css: jest.fn(),
+      css: sinon.fake(),
       animate: (options, speed, style, callback) => callback()
     };
 
     await br.enterFullscreen();
-    expect(br.switchMode).toHaveBeenCalledTimes(1);
-    expect(br.updateBrClasses).toHaveBeenCalledTimes(0); // book nav's fullscreen manager will set these classes
-    expect(br.trigger).toHaveBeenCalledTimes(2); // fragmentChange, fullscreenToggled
-    expect(br.resize).toHaveBeenCalledTimes(1);
-    expect(br.jumpToIndex).toHaveBeenCalledTimes(1);
+    expect(br.switchMode.callCount).toEqual(1);
+    expect(br.updateBrClasses.callCount).toEqual(1);
+    expect(br.trigger.callCount).toEqual(2); // fragmentChange, fullscreenToggled
+    expect(br.jumpToIndex.callCount).toEqual(1);
+
+    await sleep(0);
+    expect(br.resize.callCount).toEqual(1);
   });
 });
 
@@ -96,19 +108,19 @@ describe('BookReader.prototype.exitFullScreen', () => {
   test('fires certain events when called', async () => {
     const br = new BookReader();
     br.mode = br.constMode2up;
-    br.switchMode = jest.fn();
-    br.updateBrClasses = jest.fn();
-    br.trigger = jest.fn();
-    br.resize = jest.fn();
+    br.switchMode = sinon.fake();
+    br.updateBrClasses = sinon.fake();
+    br.trigger = sinon.fake();
+    br.resize = sinon.fake();
     br.refs.$brContainer = {
-      css: jest.fn(),
+      css: sinon.fake(),
       animate: (options, speed, style, callback) => callback()
     };
     await br.exitFullScreen();
-    expect(br.switchMode).toHaveBeenCalledTimes(1);
-    expect(br.updateBrClasses).toHaveBeenCalledTimes(1);
-    expect(br.trigger).toHaveBeenCalledTimes(2); // fragmentChange, fullscreenToggled
-    expect(br.resize).toHaveBeenCalledTimes(1);
+    expect(br.switchMode.callCount).toEqual(1);
+    expect(br.updateBrClasses.callCount).toEqual(1);
+    expect(br.trigger.callCount).toEqual(2); // fragmentChange, fullscreenToggled
+    expect(br.resize.callCount).toEqual(1);
   });
 });
 
@@ -116,27 +128,27 @@ describe('BookReader.prototype.trigger', () => {
   test('fires custom event', () => {
     const br = new BookReader();
     global.br = br;
-    global.dispatchEvent = jest.fn();
+    global.dispatchEvent = sinon.fake();
 
     const props = {bar: 1};
     br.trigger('foo', props);
-    expect(global.dispatchEvent.mock.calls.length).toBe(1);
+    expect(global.dispatchEvent.callCount).toBe(1);
   });
 });
 
 describe('`BookReader.prototype.prev`', () => {
   const br = new BookReader();
   global.br = br;
-  br.trigger = jest.fn();
-  br.flipBackToIndex = jest.fn();
-  br.jumpToIndex = jest.fn();
+  br.trigger = sinon.fake();
+  br.flipBackToIndex = sinon.fake();
+  br.jumpToIndex = sinon.fake();
 
   test('does not take action if user is on front page', () => {
     br.firstIndex = 0;
     br.prev();
-    expect(br.trigger.mock.calls.length).toBe(0);
-    expect(br.flipBackToIndex.mock.calls.length).toBe(0);
-    expect(br.jumpToIndex.mock.calls.length).toBe(0);
+    expect(br.trigger.callCount).toBe(0);
+    expect(br.flipBackToIndex.callCount).toBe(0);
+    expect(br.jumpToIndex.callCount).toBe(0);
   });
 
   describe('2up mode', () => {
@@ -144,9 +156,9 @@ describe('`BookReader.prototype.prev`', () => {
       br.firstIndex = 10;
       br.mode = br.constMode2up;
       br.prev();
-      expect(br.jumpToIndex.mock.calls.length).toBe(0); // <--  does not get called
-      expect(br.trigger.mock.calls.length).toBe(1);
-      expect(br.flipBackToIndex.mock.calls.length).toBe(1);
+      expect(br.jumpToIndex.callCount).toBe(0); // <--  does not get called
+      expect(br.trigger.callCount).toBe(1);
+      expect(br.flipBackToIndex.callCount).toBe(1);
     });
   });
 
@@ -155,9 +167,9 @@ describe('`BookReader.prototype.prev`', () => {
       br.firstIndex = 100;
       br.mode = br.constMode1up;
       br.prev();
-      expect(br.jumpToIndex.mock.calls.length).toBe(1);  // <--  gets called
-      expect(br.trigger.mock.calls.length).toBe(1); // <-- gets called by `jumpToIndex` internally
-      expect(br.flipBackToIndex.mock.calls.length).toBe(1); // <-- gets called by `jumpToIndex` internally
+      expect(br.jumpToIndex.callCount).toBe(1);  // <--  gets called
+      expect(br.trigger.callCount).toBe(1); // <-- gets called by `jumpToIndex` internally
+      expect(br.flipBackToIndex.callCount).toBe(1); // <-- gets called by `jumpToIndex` internally
     });
   });
 });

package/tests/jest/plugins/url/UrlPlugin.test.js

@@ -61,7 +61,7 @@ describe('UrlPlugin tests', () => {
         {page: 'n7', mode: '2up', q: 'hello', view: 'theather', foo: 'bar', sort: 'title_asc'}
       );
       expect(urlPlugin.urlStringToUrlState(url1)).toEqual(
-        {page: 'n0', mode: '2up', ref: 'ol', ui: 'embed', wrapper: 'false', view: 'theater'}
+        {mode: '2up', ref: 'ol', ui: 'embed', wrapper: 'false', view: 'theater'}
       );
     });
 
@@ -99,15 +99,26 @@ describe('UrlPlugin tests', () => {
   });
 
   describe('pullFromAddressBar and pushToAddressBar - hash mode', () => {
+    test('url first load - empty state', () => {
+      urlPlugin.urlState = {};
+      urlPlugin.urlMode = 'hash';
+
+      urlPlugin.pullFromAddressBar({ pathname: '', search: '', hash: '#' });
+      expect(urlPlugin.urlState).toEqual({});
+
+      urlPlugin.pushToAddressBar();
+      expect(window.location.hash).toEqual('');
+    });
+
     test('url without mode state value - use default', () => {
       urlPlugin.urlState = {};
       urlPlugin.urlMode = 'hash';
 
       urlPlugin.pullFromAddressBar({ pathname: '', search: '', hash: '#page/12' });
-      expect(urlPlugin.urlState).toEqual({page: '12', mode: '2up'});
+      expect(urlPlugin.urlState).toEqual({page: '12'});
 
       urlPlugin.pushToAddressBar();
-      expect(window.location.hash).toEqual('#page/12/mode/2up');
+      expect(window.location.hash).toEqual('#page/12');
     });
 
     test('url with query param', () => {
@@ -115,24 +126,36 @@ describe('UrlPlugin tests', () => {
       urlPlugin.urlMode = 'hash';
 
       urlPlugin.pullFromAddressBar({ pathname: '', search: '', hash: '#page/12?q=hello&view=theater' });
-      expect(urlPlugin.urlState).toEqual({page: '12', mode: '2up', q: 'hello', view: 'theater'});
+      expect(urlPlugin.urlState).toEqual({page: '12', q: 'hello', view: 'theater'});
 
       urlPlugin.pushToAddressBar();
-      expect(window.location.hash).toEqual('#page/12/mode/2up?q=hello&view=theater');
+      expect(window.location.hash).toEqual('#page/12?q=hello&view=theater');
     });
   });
 
   describe('pullFromAddressBar and pushToAddressBar - history mode', () => {
-    test('url without mode state value - use default', () => {
+    test('url first load - empty state', () => {
+      urlPlugin.urlState = {};
+      urlPlugin.urlHistoryBasePath = '/details/foo';
+      urlPlugin.urlMode = 'history';
+
+      urlPlugin.pullFromAddressBar({ pathname: '', search: '', hash: '' });
+      expect(urlPlugin.urlState).toEqual({});
+
+      urlPlugin.pushToAddressBar();
+      expect(window.location.pathname).toEqual('/details/foo');
+    });
+
+    test('url without mode state value', () => {
       urlPlugin.urlState = {};
       urlPlugin.urlHistoryBasePath = '/details/foo/';
       urlPlugin.urlMode = 'history';
 
       urlPlugin.pullFromAddressBar({ pathname: '/details/foo/page/12', search: '', hash: '' });
-      expect(urlPlugin.urlState).toEqual({page: '12', mode: '2up'});
+      expect(urlPlugin.urlState).toEqual({page: '12'});
 
       urlPlugin.pushToAddressBar();
-      expect(window.location.pathname).toEqual('/details/foo/page/12/mode/2up');
+      expect(window.location.pathname).toEqual('/details/foo/page/12');
     });
 
     test('url with query param', () => {
@@ -141,11 +164,11 @@ describe('UrlPlugin tests', () => {
       urlPlugin.urlMode = 'history';
 
       urlPlugin.pullFromAddressBar({ pathname: '/details/foo/page/12', search: '?q=hello&view=theater', hash: '' });
-      expect(urlPlugin.urlState).toEqual({page: '12', mode: '2up', q: 'hello', view: 'theater'});
+      expect(urlPlugin.urlState).toEqual({page: '12', q: 'hello', view: 'theater'});
 
       urlPlugin.pushToAddressBar();
       const locationUrl = `${window.location.pathname}${window.location.search}`;
-      expect(locationUrl).toEqual('/details/foo/page/12/mode/2up?q=hello&view=theater');
+      expect(locationUrl).toEqual('/details/foo/page/12?q=hello&view=theater');
     });
   });