@internetarchive/bookreader 5.0.0-7 → 5.0.0-70

package/tests/{plugins → jest/plugins/url}/plugin.url.test.js

@@ -1,6 +1,6 @@
-
-import BookReader from '../../src/BookReader.js';
-import '../../src/plugins/plugin.url.js';
+import BookReader from '@/src/BookReader.js';
+import '@/src/plugins/url/plugin.url.js';
+import sinon from 'sinon';
 
 let br;
 beforeAll(() => {
@@ -10,6 +10,7 @@ beforeAll(() => {
 
 afterEach(() => {
   jest.clearAllMocks();
+  sinon.restore();
 });
 
 describe('Plugin: URL controller', () => {
@@ -26,10 +27,10 @@ describe('Plugin: URL controller', () => {
     expect(defaultOptions).toHaveProperty('updateWindowTitle');
     expect(defaultOptions).toHaveProperty('defaults');
     expect(defaultOptions).toHaveProperty('bookId');
-  })
+  });
 
   test('has a whitelist of params that it tracks', () => {
-    ['page', 'search', 'mode', 'region', 'highlight']
+    ['page', 'search', 'mode', 'region', 'highlight'];
     const { defaultOptions: { urlTrackedParams } } = BookReader;
     expect(Array.isArray(urlTrackedParams)).toEqual(true);
     expect(urlTrackedParams.find(param => param === 'page')).toBeTruthy();
@@ -37,7 +38,8 @@ describe('Plugin: URL controller', () => {
     expect(urlTrackedParams.find(param => param === 'mode')).toBeTruthy();
     expect(urlTrackedParams.find(param => param === 'region')).toBeTruthy();
     expect(urlTrackedParams.find(param => param === 'highlight')).toBeTruthy();
-  })
+    expect(urlTrackedParams.find(param => param === 'view')).toBeTruthy();
+  });
 
   test('initializes polling for url changes if using hash', () => {
     BookReader.prototype.urlStartLocationPolling = jest.fn();
@@ -60,7 +62,7 @@ describe('Plugin: URL controller', () => {
     br.urlUpdateFragment();
 
     expect(window.history.replaceState).toHaveBeenCalled();
-  })
+  });
 
   test('updates URL when mode changes', () => {
     window.history.replaceState = jest.fn();
@@ -76,7 +78,7 @@ describe('Plugin: URL controller', () => {
     br.urlUpdateFragment();
 
     expect(window.history.replaceState).toHaveBeenCalled();
-  })
+  });
 
   test('updates URL when search changes', () => {
     window.history.replaceState = jest.fn();
@@ -93,7 +95,44 @@ describe('Plugin: URL controller', () => {
     br.urlUpdateFragment();
 
     expect(window.history.replaceState).toHaveBeenCalled();
-  })
+  });
+
+  test('updates URL when view changes', () => {
+    window.history.replaceState = jest.fn();
+    BookReader.prototype.currentIndex = jest.fn(() => 1);
+    BookReader.prototype.urlReadFragment = jest.fn(() => '');
+    BookReader.prototype.paramsFromCurrent = jest.fn(() => ({
+      index: 1,
+      mode: 2,
+      view: 'theater'
+    }));
+    BookReader.prototype.search = jest.fn();
+    br.options.urlMode = 'history';
+    br.init();
+    br.urlUpdateFragment();
+
+    expect(window.history.replaceState).toHaveBeenCalled();
+  });
+
+  test('switches to hashMode if replaceState errors', () => {
+    window.history.replaceState = jest.fn(() => {
+      throw new Error('foo');
+    });
+    BookReader.prototype.currentIndex = jest.fn(() => 1);
+    BookReader.prototype.urlReadFragment = jest.fn(() => '');
+    BookReader.prototype.paramsFromCurrent = jest.fn(() => ({
+      index: 1,
+      mode: 2,
+      view: 'theater'
+    }));
+    BookReader.prototype.search = jest.fn();
+    br.options.urlMode = 'history';
+    br.init();
+    br.urlUpdateFragment();
+
+    expect(window.history.replaceState).toHaveBeenCalled();
+    expect(br.options.urlMode).toEqual('hash');
+  });
 
   test('does not update URL when search in query string', () => {
     window.history.replaceState = jest.fn();
@@ -112,18 +151,18 @@ describe('Plugin: URL controller', () => {
     br.urlUpdateFragment();
 
     expect(window.history.replaceState).toHaveBeenCalledTimes(0);
-  })
+  });
 
   test('only q= param is selected from url query params', () => {
-    const INTIAL_URL = "http://127.0.0.1:8080/BookReaderDemo/demo-internetarchive.html?ocaid=adventuresofoli00dick&q=foo"
+    const INTIAL_URL = "http://127.0.0.1:8080/BookReaderDemo/demo-internetarchive.html?ocaid=adventuresofoli00dick&q=foo";
     const result = br.urlParamsFiltersOnlySearch(INTIAL_URL);
     expect(result).toBe("?q=foo");
-  })
+  });
 
   test('only q= param is selected from url query params with special character', () => {
-    const INTIAL_URL = "http://127.0.0.1:8080/BookReaderDemo/demo-internetarchive.html?ocaid=adventuresofoli00dick&q=foo%24%24"
+    const INTIAL_URL = "http://127.0.0.1:8080/BookReaderDemo/demo-internetarchive.html?ocaid=adventuresofoli00dick&q=foo%24%24";
     const result = br.urlParamsFiltersOnlySearch(INTIAL_URL);
     expect(result).toBe("?q=foo%24%24");
-  })
+  });
 });
 

package/tests/jest/setup.js

@@ -0,0 +1,3 @@
+process.on("unhandledRejection", (error) => {
+  throw error;
+});

package/tests/{util → jest/util}/browserSniffing.test.js

@@ -1,6 +1,6 @@
 import {
   isChrome, isFirefox, isSafari,
-} from '../../src/util/browserSniffing.js';
+} from '@/src/util/browserSniffing.js';
 
 const TESTS = [
   {

package/tests/jest/util/docCookies.test.js

@@ -0,0 +1,24 @@
+import * as docCookies from '@/src/util/docCookies.js';
+
+afterEach(() => {
+  jest.clearAllMocks();
+});
+
+describe('Helper function: set and get cookie item', () => {
+  test('set cookie item value', () => {
+    expect(docCookies.setItem('test-cookie', 'jack-sparow')).toBeTruthy();
+  });
+
+  test('get cookie item value', () => {
+    expect(docCookies.getItem('test-cookie')).toEqual('jack-sparow');
+  });
+});
+
+describe('isCookiesBlocked', () => {
+  test('return true if cookies are blocked', () => {
+    expect(docCookies.areCookiesBlocked({ get cookie() { throw new Error(); }})).toBeTruthy();
+  });
+  test('return false if cookies are not blocked', () => {
+    expect(docCookies.areCookiesBlocked({ cookie: 'blah' })).toBeFalsy();
+  });
+});

package/tests/{util → jest/util}/strings.test.js

@@ -1,4 +1,4 @@
-import { APPLY_FILTERS, applyVariables } from '../../src/util/strings.js';
+import { APPLY_FILTERS, applyVariables } from '@/src/util/strings.js';
 
 describe('applyVariables', () => {
   test('null cases', () => {

package/tests/{utils.js → jest/utils.js}

@@ -31,6 +31,44 @@ export function eventTargetMixin() {
   };
 }
 
+/** @implements {EventTarget} */
+export class EventTargetSpy {
+  /** @type {Record<string, Function[]} */
+  _listeners = {}
+  get _totalListenerCount() {
+    return Object.values(this._listeners)
+      .map(a => a.length)
+      .reduce((a, b) => a + b, 0);
+  }
+
+  addEventListener(evName, listener) {
+    const listeners = this._listeners[evName] || [];
+    listeners.push(listener);
+    this._listeners[evName] = listeners;
+  }
+
+  dispatchEvent(evName, ev) {
+    const listeners = this._listeners[evName] || [];
+    listeners.forEach(fn => fn(ev));
+  }
+
+  removeEventListener(evName, listener) {
+    const listeners = this._listeners[evName] || [];
+    this._listeners[evName] = listeners.filter(fn => fn != listener);
+  }
+
+  /**
+   * @param {EventTarget} realEventTarget
+   */
+  static wrap(realEventTarget) {
+    const spy = new EventTargetSpy();
+    realEventTarget.addEventListener = spy.addEventListener.bind(spy);
+    realEventTarget.removeEventListener = spy.removeEventListener.bind(spy);
+    realEventTarget.dispatchEvent = spy.dispatchEvent.bind(spy);
+    return spy;
+  }
+}
+
 /**
  * Lazy deep copy function; only supports objects (no classes)
  * @template T

package/webpack.config.js

@@ -14,7 +14,11 @@ const shared = {
 
   module: {
     rules: [
-      { test: /\.js$/, exclude: /node_modules/, loader: "babel-loader" }
+      {
+        test: /\.js$/,
+        exclude: /node_modules[/\\](?!(lit-element|lit-html|lit|@lit)[/\\]).*/,
+        loader: "babel-loader",
+      }
     ]
   },
 
@@ -31,6 +35,9 @@ module.exports = [
 
     // Output file -> srcfile
     entry: {
+      // Polyfill bundles
+      'webcomponents-bundle.js': { import: '@webcomponents/webcomponentsjs/webcomponents-bundle.js' },
+
       // BookReader
       'BookReader.js': './src/BookReader.js',
 
@@ -39,15 +46,13 @@ module.exports = [
       'plugins/plugin.autoplay.js': { import: './src/plugins/plugin.autoplay.js', dependOn: 'BookReader.js' },
       'plugins/plugin.chapters.js': { import: './src/plugins/plugin.chapters.js', dependOn: 'BookReader.js' },
       'plugins/plugin.iframe.js': { import: './src/plugins/plugin.iframe.js', dependOn: 'BookReader.js' },
-      'plugins/plugin.menu_toggle.js': { import: './src/plugins/menu_toggle/plugin.menu_toggle.js', dependOn: 'BookReader.js' },
-      'plugins/plugin.mobile_nav.js': { import: './src/plugins/plugin.mobile_nav.js', dependOn: 'BookReader.js' },
       'plugins/plugin.resume.js': { import: './src/plugins/plugin.resume.js', dependOn: 'BookReader.js' },
       'plugins/plugin.search.js': { import: './src/plugins/search/plugin.search.js', dependOn: 'BookReader.js' },
       'plugins/plugin.text_selection.js': { import: './src/plugins/plugin.text_selection.js', dependOn: 'BookReader.js' },
       'plugins/plugin.tts.js': { import: './src/plugins/tts/plugin.tts.js', dependOn: 'BookReader.js' },
-      'plugins/plugin.url.js': { import: './src/plugins/plugin.url.js', dependOn: 'BookReader.js' },
+      'plugins/plugin.url.js': { import: './src/plugins/url/plugin.url.js', dependOn: 'BookReader.js' },
       'plugins/plugin.vendor-fullscreen.js': { import: './src/plugins/plugin.vendor-fullscreen.js', dependOn: 'BookReader.js' },
-      'bookreader-component-bundle.js': { import: './src/BookReaderComponent/BookReaderComponent.js', dependOn: 'BookReader.js' }
+      'ia-bookreader-bundle.js': { import: './src/ia-bookreader/ia-bookreader.js', dependOn: 'BookReader.js' }
     },
 
     externals: {
@@ -80,7 +85,7 @@ module.exports = [
     ...shared,
 
     entry: {
-      'jquery-1.10.1.js': { import: './src/jquery-wrapper.js' },
+      'jquery-3.js': { import: './src/jquery-wrapper.js' },
     },
   },
 ];

package/.babelrc

@@ -1,12 +0,0 @@
-{
-    "presets": [
-        [
-            "@babel/preset-env",
-            {
-                "targets": "> 2%, ie 11, edge 14, samsung > 9, OperaMini all, UCAndroid > 12",
-                "useBuiltIns": "usage",
-                "corejs": 3
-            }
-        ]
-    ]
-}

package/.dependabot/config.yml

@@ -1,6 +0,0 @@
-version: 1
-
-update_configs:
-  - package_manager: "javascript"
-    directory: "/"
-    update_schedule: "monthly"

package/.testcaferc.json

@@ -1,5 +0,0 @@
-{
-  "appCommand": "npm run serve",
-  "browsers": ["chrome:headless", "firefox:headless"],
-  "src": ["tests/e2e/**/*test.js"]
-}