indexeddbshim 19.0.5 → 19.0.7

package/src/IDBObjectStore.js

@@ -205,14 +205,17 @@ IDBObjectStore.__createInstance = function (storeProperties, transaction) {
                         tx.executeSql(sql, sqlValues, function (tx) {
                             // This SQL preserves indexes per https://www.sqlite.org/lang_altertable.html
                             const sql = 'ALTER TABLE ' + util.escapeStoreNameForSQL(oldName) + ' RENAME TO ' + util.escapeStoreNameForSQL(name);
+                            /* c8 ignore next -- Debug-only log; no test sets `CFG.DEBUG` while renaming a store. */
                             if (CFG.DEBUG) { console.log(sql); }
                             tx.executeSql(sql, [], function () {
                                 delete me.__pendingName;
                                 success();
                             });
                         }, function (tx, err) {
+                            /* c8 ignore start -- Defensive: SQL execution error, not reliably reproducible without mocking the SQL layer. */
                             error(err);
                             return false;
+                            /* c8 ignore stop -- see comment above */
                         });
                     });
                 }
@@ -296,6 +299,7 @@ IDBObjectStore.__createObjectStore = function (db, store) {
     }
 
     transaction.__addNonRequestToTransactionQueue(function createObjectStore (tx, args, success, failure) {
+        /* c8 ignore start -- Defensive: SQL execution error, not reliably reproducible without mocking the SQL layer. */
         /**
          * @param {WebSQLTransaction} tx
          * @param {(Error & {code?: number})} [err]
@@ -306,6 +310,7 @@ IDBObjectStore.__createObjectStore = function (db, store) {
             failure(createDOMException('UnknownError', 'Could not create object store "' + storeName + '"', err));
             return false;
         }
+        /* c8 ignore stop -- see comment above */
 
         const escapedStoreNameSQL = util.escapeStoreNameForSQL(storeName);
         // key INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL UNIQUE
@@ -369,6 +374,7 @@ IDBObjectStore.__deleteObjectStore = function (db, store) {
 
     // Remove the object store from WebSQL
     transaction.__addNonRequestToTransactionQueue(function deleteObjectStore (tx, args, success, failure) {
+        /* c8 ignore start -- Defensive: SQL execution error, not reliably reproducible without mocking the SQL layer. */
         /**
          * @param {WebSQLTransaction} tx
          * @param {(Error & {code?: number})} [err]
@@ -379,6 +385,7 @@ IDBObjectStore.__deleteObjectStore = function (db, store) {
             failure(createDOMException('UnknownError', 'Could not delete ObjectStore', err));
             return false;
         }
+        /* c8 ignore stop -- see comment above */
 
         tx.executeSql('SELECT "name" FROM __sys__ WHERE "name" = ?', [util.escapeSQLiteStatement(store.__currentName)], function (tx, data) {
             if (data.rows.length > 0) {
@@ -572,13 +579,15 @@ IDBObjectStore.prototype.__checkIndexConstraints = function (tx, value, excludeK
                 return;
             }
             const indexKeyValue = indexKey.value;
+            /* c8 ignore start -- Defensive: unreachable given the current `Key.js` implementation, where a keyPath evaluation that would resolve to `undefined` always instead reports `invalid`/`failure` (caught above, causing an earlier `return`) rather than succeeding with an `undefined` value. */
             if (indexKeyValue === undefined) {
                 resolve(undefined);
                 return;
             }
+            /* c8 ignore stop -- see comment above */
             const multiCheck = index.multiEntry && Array.isArray(indexKeyValue);
             const fetchArgs = buildFetchIndexDataSQL(true, index, indexKeyValue, 'key', multiCheck);
-            executeFetchIndexData(null, ...fetchArgs, tx, null, function success (key) {
+            executeFetchIndexData(...fetchArgs, tx, null, function success (key) {
                 if (key === undefined || (excludeKey !== undefined && cmp(
                     /** @type {import('./Key.js').Key} */ (key), excludeKey
                 ) === 0)) {
@@ -670,9 +679,11 @@ IDBObjectStore.prototype.__insertData = function (tx, encoded, value, clonedKeyO
              * @returns {void}
              */
             function setIndexInfo (index) {
+                /* c8 ignore start -- Defensive: unreachable given the current `Key.js` implementation, where a keyPath evaluation that would resolve to `undefined` always instead reports `invalid`/`failure` (caught above, causing an earlier `return`) rather than succeeding with an `undefined` value. */
                 if (indexKeyValue === undefined) {
                     return;
                 }
+                /* c8 ignore stop -- see comment above */
                 paramMap[index.__currentName] = /** @type {string} */ (
                     Key.encode(indexKeyValue, index.multiEntry)
                 );
@@ -680,7 +691,7 @@ IDBObjectStore.prototype.__insertData = function (tx, encoded, value, clonedKeyO
             if (index.unique) {
                 const multiCheck = index.multiEntry && Array.isArray(indexKeyValue);
                 const fetchArgs = buildFetchIndexDataSQL(true, index, indexKeyValue, 'key', multiCheck);
-                executeFetchIndexData(null, ...fetchArgs, tx, null, function success (key) {
+                executeFetchIndexData(...fetchArgs, tx, null, function success (key) {
                     if (key === undefined) {
                         setIndexInfo(index);
                         resolve(undefined);
@@ -780,6 +791,7 @@ IDBObjectStore.prototype.add = function (value /* , key */) {
         me.transaction
     ).__createRequest(me);
     const [ky, clonedValue] = me.__validateKeyAndValueAndCloneValue(value, key, false);
+    // eslint-disable-next-line jsdoc/ts-ban-ts-comment -- TS 6/7
     // @ts-ignore -- Private API
     IDBObjectStore.__storingRecordObjectStore(request, me, true, clonedValue, true, ky);
     return request;
@@ -812,6 +824,7 @@ IDBObjectStore.prototype.put = function (value /* , key */) {
         me.transaction
     ).__createRequest(me);
     const [ky, clonedValue] = me.__validateKeyAndValueAndCloneValue(value, key, false);
+    // eslint-disable-next-line jsdoc/ts-ban-ts-comment -- TS 6/7
     // @ts-ignore -- Private API
     IDBObjectStore.__storingRecordObjectStore(request, me, true, clonedValue, false, ky);
     return request;
@@ -836,8 +849,10 @@ IDBObjectStore.prototype.__overwrite = function (tx, key, cb, error) {
         if (CFG.DEBUG) { console.log('Did the row with the', key, 'exist?', data.rowsAffected); }
         cb(tx);
     }, function (tx, err) {
+        /* c8 ignore start -- Defensive: SQL execution error, not reliably reproducible without mocking the SQL layer. */
         error(err);
         return false;
+        /* c8 ignore stop -- see comment above */
     });
 };
 
@@ -950,14 +965,18 @@ IDBObjectStore.prototype.__get = function (query, getKey) {
                 ret = getKey
                     ? Key.decode(util.unescapeSQLiteResponse(/** @type {{key: string}} */ (data.rows.item(0)).key), false)
                     : Sca.decode(util.unescapeSQLiteResponse(/** @type {{value: string}} */ (data.rows.item(0)).value));
+                /* c8 ignore start -- Defensive: catches unexpected decode failures (e.g. corrupted stored data); not reliably reproducible via the public API since encode/decode are always paired and never write undecodable data. */
             } catch (e) {
                 // If no result is returned, or error occurs when parsing JSON
                 if (CFG.DEBUG) { console.log(e); }
             }
+            /* c8 ignore stop -- see comment above */
             success(ret);
         }, function (tx, err) {
+            /* c8 ignore start -- Defensive: SQL execution error, not reliably reproducible without mocking the SQL layer. */
             error(err);
             return false;
+            /* c8 ignore stop -- see comment above */
         });
     }, undefined, me);
 };
@@ -1060,8 +1079,10 @@ IDBObjectStore.prototype.delete = function (query) {
             });
             success();
         }, function (tx, err) {
+            /* c8 ignore start -- Defensive: SQL execution error, not reliably reproducible without mocking the SQL layer. */
             error(err);
             return false;
+            /* c8 ignore stop -- see comment above */
         });
     }, undefined, me);
 };
@@ -1091,8 +1112,10 @@ IDBObjectStore.prototype.clear = function () {
             });
             success();
         }, function (tx, err) {
+            /* c8 ignore start -- Defensive: SQL execution error, not reliably reproducible without mocking the SQL layer. */
             error(err);
             return false;
+            /* c8 ignore stop -- see comment above */
         });
     }, undefined, me);
 };

package/src/IDBRecord.js

@@ -43,6 +43,7 @@ IDBRecord.__createInstance = function (key, primaryKey, value) {
      * @this {IDBRecordFull}
      */
     function IDBRecord () {
+        // eslint-disable-next-line jsdoc/ts-ban-ts-comment -- TS 6/7
         // @ts-ignore Should be ok
         this[Symbol.toStringTag] = 'IDBRecord';
         this.__key = key;
@@ -84,6 +85,7 @@ readonlyProperties.forEach((prop) => {
             if (!(this instanceof IDBRecordAlias)) {
                 throw new TypeError('Illegal invocation');
             }
+            // eslint-disable-next-line jsdoc/ts-ban-ts-comment -- TS 6/7
             // @ts-ignore `this` is a real instance past the check above (not an issue in TS7)
             return this['__' + prop];
         }

package/src/IDBRequest.js

@@ -42,8 +42,10 @@ function IDBRequest () {
  * @this {IDBRequestFull}
  */
 IDBRequest.__super = function IDBRequest () {
+    // eslint-disable-next-line jsdoc/ts-ban-ts-comment -- TS 6/7
     // @ts-ignore It's ok
     this[Symbol.toStringTag] = 'IDBRequest';
+    // eslint-disable-next-line jsdoc/ts-ban-ts-comment -- TS 6/7
     // @ts-ignore Part of `ShimEventTarget`
     this.__setOptions({
         legacyOutputDidListenersThrowFlag: true // Event hook for IndexedB
@@ -106,10 +108,9 @@ IDBRequest.prototype[Symbol.toStringTag] = 'IDBRequestPrototype';
  * @returns {import('./IDBTransaction.js').IDBTransactionFull|null|undefined}
  */
 IDBRequest.prototype.__getParent = function () {
-    if (this.toString() === '[object IDBOpenDBRequest]') {
-        return null;
-    }
-    return this.__transaction;
+    return this.toString() === '[object IDBOpenDBRequest]'
+        ? null
+        : this.__transaction;
 };
 
 /* eslint-disable unicorn/no-top-level-side-effects -- Would be good */
@@ -171,8 +172,10 @@ IDBOpenDBRequest.__createInstance = function () {
     function IDBOpenDBRequest () {
         IDBRequest.__super.call(this);
 
+        // eslint-disable-next-line jsdoc/ts-ban-ts-comment -- TS 6/7
         // @ts-ignore It's ok
         this[Symbol.toStringTag] = 'IDBOpenDBRequest';
+        // eslint-disable-next-line jsdoc/ts-ban-ts-comment -- TS 6/7
         // @ts-ignore It's ok
         this.__setOptions({
             legacyOutputDidListenersThrowFlag: true, // Event hook for IndexedB