(#6055) - remove docCount optimization from IDB

Author: nolanlawson

packages/node_modules/pouchdb-adapter-idb/src/allDocs.js

@@ -24,18 +24,25 @@ if (typeof window.Worker === 'function' &&
 
 function runTests() {
 
+  var worker;
+
+  before(function () {
+    worker = new Worker('worker.js');
+    worker.postMessage(['source', sourceFile]);
+  });
+
+  after(function () {
+    worker.terminate();
+  });
+
   function workerPromise(message) {
     return new Promise(function (resolve, reject) {
-      var worker = new Worker('worker.js');
-      worker.addEventListener('error', function (e) {
-        worker.terminate();
+      worker.onerror = function (e) {
         reject(new Error(e.message + ": " + e.filename + ': ' + e.lineno));
-      });
-      worker.addEventListener('message', function (e) {
-        worker.terminate();
+      };
+      worker.onmessage = function (e) {
         resolve(e.data);
-      });
-      worker.postMessage(['source', sourceFile]);
+      };
       worker.postMessage(message);
     });
   }
@@ -79,7 +86,7 @@ function runTests() {
     });
 
     it('add doc with blob attachment', function () {
-      return workerPromise(['allDocs', dbs.name]).then(function (data) {
+      return workerPromise(['postAttachmentThenAllDocs', dbs.name]).then(function (data) {
         data.title.should.equal('lalaa');
       });
     });
@@ -93,5 +100,31 @@ function runTests() {
         blob.size.should.equal(6);
       });
     });
+
+    it('total_rows consistent between worker and main thread', function () {
+      var db = new PouchDB(dbs.name);
+      // both threads agree the count is 0
+      return testUtils.Promise.all([
+        db.allDocs().then(function (res) {
+          res.total_rows.should.equal(0);
+        }),
+        workerPromise(['allDocs', dbs.name]).then(function (res) {
+          res.total_rows.should.equal(0);
+        })
+      ]).then(function () {
+        // post a doc
+        return db.post({});
+      }).then(function () {
+        // both threads agree the count is 1
+        return testUtils.Promise.all([
+          db.allDocs().then(function (res) {
+            res.total_rows.should.equal(1);
+          }),
+          workerPromise(['allDocs', dbs.name]).then(function (res) {
+            res.total_rows.should.equal(1);
+          })
+        ]);
+      });
+    });
   });
 }

packages/node_modules/pouchdb-adapter-idb/src/bulkDocs.js

@@ -1062,6 +1062,33 @@ adapters.forEach(function (adapter) {
       });
     });
 
+    it('test info() after db close', function () {
+      var db = new PouchDB(dbs.name);
+      return db.close().then(function () {
+        return db.info().catch(function (err) {
+          err.message.should.equal('database is closed');
+        });
+      });
+    });
+
+    it('test get() after db close', function () {
+      var db = new PouchDB(dbs.name);
+      return db.close().then(function () {
+        return db.get('foo').catch(function (err) {
+          err.message.should.equal('database is closed');
+        });
+      });
+    });
+
+    it('test close() after db close', function () {
+      var db = new PouchDB(dbs.name);
+      return db.close().then(function () {
+        return db.close().catch(function (err) {
+          err.message.should.equal('database is closed');
+        });
+      });
+    });
+
     if (adapter === 'local') {
       // TODO: this test fails in the http adapter in Chrome
       it('should allow unicode doc ids', function (done) {