cozy-pouch-link 60.24.1 → 60.24.2

package/dist/PouchManager.spec.js

@@ -235,7 +235,7 @@ describe('PouchManager', () => {
     manager.updateSyncInfo('io.cozy.todos')
     await manager.replicateOnce()
     expect(onSync).toHaveBeenCalledWith({
-      'cozy.tools__doctype__io.cozy.todos': [
+      'io.cozy.todos': [
         {
           _id: '1',
           name: 'Make replication work'

package/dist/replicateOnce.js

@@ -176,7 +176,9 @@ var replicateOnce = /*#__PURE__*/function () {
               };
             }()); // Waiting on each replication
 
-            doctypes = Object.keys(pouchManager.pouches);
+            doctypes = Object.keys(pouchManager.pouches).map(function (dbName) {
+              return (0, _utils.getDoctypeFromDatabaseName)(dbName);
+            });
             promises = Object.values(pouchManager.replications);
             _context2.prev = 9;
             _context2.next = 12;

package/dist/replicateOnce.spec.js

@@ -0,0 +1,104 @@
+/* global jest */
+
+jest.mock('./remote', () => ({
+  fetchRemoteLastSequence: jest.fn().mockResolvedValue('0'),
+  isDatabaseNotFoundError: jest.fn().mockReturnValue(false),
+  isDatabaseUnradableError: jest.fn().mockReturnValue(false)
+}))
+
+jest.mock('./startReplication', () => ({
+  startReplication: jest.fn()
+}))
+
+import { replicateOnce } from './replicateOnce'
+import { startReplication } from './startReplication'
+
+describe('replicateOnce', () => {
+  beforeEach(() => {
+    jest.clearAllMocks()
+  })
+
+  it('should call onSync with clean doctype keys, not prefixed pouch dbNames', async () => {
+    const dbName = 'instance__doctype__io.cozy.todos'
+    const docs = [{ _id: 'doc-1', name: 'a' }]
+
+    startReplication.mockResolvedValue(docs)
+
+    const pouchManager = {
+      pouches: { [dbName]: { id: 'fake-pouch' } },
+      replications: {},
+      doctypesReplicationOptions: {},
+      options: {
+        onSync: jest.fn()
+      },
+      storage: {
+        persistDoctypeLastSequence: jest.fn().mockResolvedValue(),
+        getDoctypeLastSequence: jest.fn().mockResolvedValue(''),
+        destroyDoctypeLastSequence: jest.fn().mockResolvedValue()
+      },
+      client: {},
+      isOnline: jest.fn().mockResolvedValue(true),
+      getReplicationURL: jest.fn().mockReturnValue('http://example/db'),
+      getSyncStatus: jest.fn().mockReturnValue('synced'),
+      updateSyncInfo: jest.fn().mockResolvedValue(),
+      checkToWarmupDoctype: jest.fn(),
+      handleReplicationError: jest.fn(),
+      cancelCurrentReplications: jest.fn()
+    }
+
+    await replicateOnce(pouchManager)
+
+    expect(pouchManager.options.onSync).toHaveBeenCalledTimes(1)
+    const payload = pouchManager.options.onSync.mock.calls[0][0]
+
+    // The bug: payload was keyed by the full prefixed dbName
+    // The fix: payload must be keyed by the clean doctype
+    expect(Object.keys(payload)).toEqual(['io.cozy.todos'])
+    expect(Object.keys(payload)).not.toContain(dbName)
+    expect(payload['io.cozy.todos']).toEqual(docs)
+  })
+
+  it('should preserve clean doctype keys for multiple pouches', async () => {
+    const dbNameA = 'instance__doctype__io.cozy.todos'
+    const dbNameB = 'instance__doctype__io.cozy.files'
+    const docsA = [{ _id: 'a1' }]
+    const docsB = [{ _id: 'b1' }]
+
+    startReplication.mockImplementation(pouch => {
+      if (pouch === 'pouchA') return Promise.resolve(docsA)
+      if (pouch === 'pouchB') return Promise.resolve(docsB)
+      return Promise.resolve([])
+    })
+
+    const pouchManager = {
+      pouches: { [dbNameA]: 'pouchA', [dbNameB]: 'pouchB' },
+      replications: {},
+      doctypesReplicationOptions: {},
+      options: { onSync: jest.fn() },
+      storage: {
+        persistDoctypeLastSequence: jest.fn().mockResolvedValue(),
+        getDoctypeLastSequence: jest.fn().mockResolvedValue(''),
+        destroyDoctypeLastSequence: jest.fn().mockResolvedValue()
+      },
+      client: {},
+      isOnline: jest.fn().mockResolvedValue(true),
+      getReplicationURL: jest.fn().mockReturnValue('http://example/db'),
+      getSyncStatus: jest.fn().mockReturnValue('synced'),
+      updateSyncInfo: jest.fn().mockResolvedValue(),
+      checkToWarmupDoctype: jest.fn(),
+      handleReplicationError: jest.fn(),
+      cancelCurrentReplications: jest.fn()
+    }
+
+    await replicateOnce(pouchManager)
+
+    expect(pouchManager.options.onSync).toHaveBeenCalledTimes(1)
+    const payload = pouchManager.options.onSync.mock.calls[0][0]
+    expect(Object.keys(payload).sort()).toEqual([
+      'io.cozy.files',
+      'io.cozy.todos'
+    ])
+    expect(payload['io.cozy.todos']).toEqual(docsA)
+    expect(payload['io.cozy.files']).toEqual(docsB)
+  })
+})

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-pouch-link",
-  "version": "60.24.1",
+  "version": "60.24.2",
   "license": "MIT",
   "main": "dist/index.js",
   "types": "types/index.d.ts",
@@ -41,5 +41,5 @@
     "typecheck": "tsc -p tsconfig.json"
   },
   "sideEffects": false,
-  "gitHead": "97abc13aa043737c338d0e25694320432675b43a"
+  "gitHead": "e621858d1b3d16f79d75bb79d3f183d204f2eabe"
 }