fix: .clear() removed in favour of .reset()

BREAKING CHANGE:

`store.clear()` method has been removed in favor of `store.reset(options)`

Author: Nick Colley

index.js

@@ -61,7 +61,21 @@ function Store (dbName, options) {
 
   var api = merge(
     scoped.bind(null, state, storeApi),
-    storeApi,
+    {
+      db: storeApi.db,
+      add: storeApi.add,
+      find: storeApi.find,
+      findAll: storeApi.findAll,
+      findOrAdd: storeApi.findOrAdd,
+      update: storeApi.update,
+      updateOrAdd: storeApi.updateOrAdd,
+      updateAll: storeApi.updateAll,
+      remove: storeApi.remove,
+      removeAll: storeApi.removeAll,
+      on: storeApi.on,
+      one: storeApi.one,
+      off: storeApi.off
+    },
     {
       hasLocalChanges: hasLocalChanges,
       push: syncWrapper.bind(syncApi, 'push'),
@@ -73,7 +87,7 @@ function Store (dbName, options) {
     }
   )
 
-  api.reset = require('./lib/reset').bind(null, dbName, CustomPouchDB, state, api, emitter, remote, ajaxOptions, PouchDB)
+  api.reset = require('./lib/reset').bind(null, dbName, CustomPouchDB, state, api, storeApi.clear, emitter, remote, ajaxOptions, PouchDB)
 
   subscribeToSyncEvents(syncApi, emitter)
   subscribeToInternalEvents(emitter)

lib/reset.js

@@ -6,7 +6,7 @@ var subscribeToSyncEvents = require('./subscribe-to-sync-events')
 var syncWrapper = require('./sync-wrapper')
 var scoped = require('./scoped/')
 
-function reset (dbName, CustomPouchDB, state, api, emitter, remote, ajaxOptions, PouchDB, options) {
+function reset (dbName, CustomPouchDB, state, api, clear, emitter, remote, ajaxOptions, PouchDB, options) {
   if (options) {
     if (options.name) {
       dbName = options.name
@@ -19,7 +19,7 @@ function reset (dbName, CustomPouchDB, state, api, emitter, remote, ajaxOptions,
     CustomPouchDB = PouchDB.defaults(options)
   }
 
-  return api.clear()
+  return clear()
 
   .then(function () {
     var newDB = new CustomPouchDB(dbName)
@@ -31,7 +31,21 @@ function reset (dbName, CustomPouchDB, state, api, emitter, remote, ajaxOptions,
     return merge(
       api,
       scoped.bind(null, state, storeApi),
-      storeApi,
+      {
+        db: storeApi.db,
+        add: storeApi.add,
+        find: storeApi.find,
+        findAll: storeApi.findAll,
+        findOrAdd: storeApi.findOrAdd,
+        update: storeApi.update,
+        updateOrAdd: storeApi.updateOrAdd,
+        updateAll: storeApi.updateAll,
+        remove: storeApi.remove,
+        removeAll: storeApi.removeAll,
+        on: storeApi.on,
+        one: storeApi.one,
+        off: storeApi.off
+      },
       {
         push: syncWrapper.bind(syncApi, 'push'),
         pull: syncWrapper.bind(syncApi, 'pull'),

tests/specs/constructor.js

@@ -135,7 +135,7 @@ function cleanupDb (store, t) {
 test('Store api exports', function (t) {
   var store = new Store('test-db', merge({remote: 'test-db-remote'}, options))
 
-  t.is(store.clear, 'undefined', 'store doesnt expose clear method')
+  t.is(typeof store.clear, 'undefined', 'store doesnt expose clear method')
 
   t.end()
 })

tests/specs/has-local-changes.js

@@ -49,7 +49,7 @@ test('.hasLocalChanges(filter)', function (t) {
     t.is(store.hasLocalChanges('test1'), true, 'returns "true" with id filter')
     t.is(store.hasLocalChanges({id: 'test2'}), true, 'returns "true" to with object filter')
 
-    return store.clear()
+    return store.reset()
   })
 
   .then(function () {