cozy-sharing 40.1.2 → 40.1.4

package/dist/SharingProvider.spec.js

@@ -876,6 +876,124 @@ describe('shared drive recipient revocation', function () {
     }, _callee17);
   })));
 });
+describe('revokeSelf', function () {
+  var document = {
+    id: 'received-folder',
+    _id: 'received-folder'
+  };
+  var sharing = {
+    id: 'received-sharing',
+    type: 'io.cozy.sharings',
+    attributes: {
+      owner: false,
+      drive: true,
+      rules: [{
+        values: [document.id]
+      }],
+      members: []
+    }
+  };
+  var recipients = [{
+    sharingId: sharing.id,
+    memberIndex: 1
+  }];
+  var forbidden = Object.assign(new Error('Forbidden'), {
+    status: 403
+  });
+  var provider;
+  var sharingCol;
+  beforeEach(function () {
+    var client = createMockClient({});
+    sharingCol = {
+      revokeSelf: jest.fn().mockResolvedValue({}),
+      fetchEffectiveRecipients: jest.fn().mockRejectedValue(forbidden)
+    };
+
+    client.collection = function () {
+      return sharingCol;
+    };
+
+    provider = setupProvider(client, {}, reducer(undefined, receiveSharings({
+      sharings: [sharing]
+    })));
+    provider.dispatch(receiveEffectiveRecipients(document.id, recipients));
+    provider.dispatch(receiveEffectiveRecipients('other-folder', recipients));
+  });
+  it.each([true, false])('forgets the sharing and clears its recipients without reloading lost access (drive: %s)', /*#__PURE__*/function () {
+    var _ref20 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18(drive) {
+      return _regeneratorRuntime.wrap(function _callee18$(_context18) {
+        while (1) switch (_context18.prev = _context18.next) {
+          case 0:
+            provider.state.sharings = [_objectSpread(_objectSpread({}, sharing), {}, {
+              attributes: _objectSpread(_objectSpread({}, sharing.attributes), {}, {
+                drive: drive
+              })
+            })];
+            _context18.next = 3;
+            return provider.revokeSelf(document);
+
+          case 3:
+            expect(getDocumentSharing(provider.state, document.id)).toBeNull();
+            expect(getEffectiveRecipients(provider.state, document.id)).toEqual([]);
+            expect(getEffectiveRecipients(provider.state, 'other-folder')).toEqual(recipients);
+            expect(sharingCol.fetchEffectiveRecipients).not.toHaveBeenCalled();
+
+          case 7:
+          case "end":
+            return _context18.stop();
+        }
+      }, _callee18);
+    }));
+
+    return function (_x) {
+      return _ref20.apply(this, arguments);
+    };
+  }());
+  it.each([403, 500])('preserves sharing state and rejects when self-revocation fails (%s)', /*#__PURE__*/function () {
+    var _ref21 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19(status) {
+      var error;
+      return _regeneratorRuntime.wrap(function _callee19$(_context19) {
+        while (1) switch (_context19.prev = _context19.next) {
+          case 0:
+            error = Object.assign(new Error('Revocation failed'), {
+              status: status
+            });
+            sharingCol.revokeSelf.mockRejectedValue(error);
+            _context19.next = 4;
+            return expect(provider.revokeSelf(document)).rejects.toBe(error);
+
+          case 4:
+            expect(getDocumentSharing(provider.state, document.id)).toEqual(sharing);
+            expect(getEffectiveRecipients(provider.state, document.id)).toEqual(recipients);
+
+          case 6:
+          case "end":
+            return _context19.stop();
+        }
+      }, _callee19);
+    }));
+
+    return function (_x2) {
+      return _ref21.apply(this, arguments);
+    };
+  }());
+  it('still rejects forbidden recipient refreshes outside self-revocation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
+    return _regeneratorRuntime.wrap(function _callee20$(_context20) {
+      while (1) switch (_context20.prev = _context20.next) {
+        case 0:
+          _context20.next = 2;
+          return expect(provider.fetchEffectiveRecipients(document.id)).rejects.toBe(forbidden);
+
+        case 2:
+          expect(getEffectiveRecipients(provider.state, document.id)).toEqual(recipients);
+
+        case 3:
+        case "end":
+          return _context20.stop();
+      }
+    }, _callee20);
+  })));
+});
 describe('share flow reused by PermissionTypeMenu upgrade', function () {
   var PARENT_SHARING = {
     _id: 'parent_sharing',
@@ -988,13 +1106,13 @@ describe('share flow reused by PermissionTypeMenu upgrade', function () {
     return provider;
   };
 
-  it('creates a child sharing when the child has no direct sharing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
+  it('creates a child sharing when the child has no direct sharing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
     var provider;
-    return _regeneratorRuntime.wrap(function _callee18$(_context18) {
-      while (1) switch (_context18.prev = _context18.next) {
+    return _regeneratorRuntime.wrap(function _callee21$(_context21) {
+      while (1) switch (_context21.prev = _context21.next) {
         case 0:
           provider = setupWithParent();
-          _context18.next = 3;
+          _context21.next = 3;
           return provider.share({
             document: childDocument,
             recipients: [bobContact],
@@ -1020,17 +1138,17 @@ describe('share flow reused by PermissionTypeMenu upgrade', function () {
 
         case 7:
         case "end":
-          return _context18.stop();
+          return _context21.stop();
       }
-    }, _callee18);
+    }, _callee21);
   })));
-  it('refreshes the child effective recipients as two-way and leaves the parent unchanged', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
+  it('refreshes the child effective recipients as two-way and leaves the parent unchanged', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
     var provider, effectiveRecipients, parentSharing;
-    return _regeneratorRuntime.wrap(function _callee19$(_context19) {
-      while (1) switch (_context19.prev = _context19.next) {
+    return _regeneratorRuntime.wrap(function _callee22$(_context22) {
+      while (1) switch (_context22.prev = _context22.next) {
         case 0:
           provider = setupWithParent();
-          _context19.next = 3;
+          _context22.next = 3;
           return provider.share({
             document: childDocument,
             recipients: [bobContact],
@@ -1058,14 +1176,14 @@ describe('share flow reused by PermissionTypeMenu upgrade', function () {
 
         case 15:
         case "end":
-          return _context19.stop();
+          return _context22.stop();
       }
-    }, _callee19);
+    }, _callee22);
   })));
-  it('adds the recipient to an existing child sharing and preserves the other member', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
+  it('adds the recipient to an existing child sharing and preserves the other member', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
     var otherMember, childSharing, updatedSharing, provider, childSharingInState, parentSharing;
-    return _regeneratorRuntime.wrap(function _callee20$(_context20) {
-      while (1) switch (_context20.prev = _context20.next) {
+    return _regeneratorRuntime.wrap(function _callee23$(_context23) {
+      while (1) switch (_context23.prev = _context23.next) {
         case 0:
           otherMember = {
             status: 'ready',
@@ -1092,7 +1210,7 @@ describe('share flow reused by PermissionTypeMenu upgrade', function () {
           provider.sharingCol.addRecipients = jest.fn().mockResolvedValue({
             data: updatedSharing
           });
-          _context20.next = 7;
+          _context23.next = 7;
           return provider.share({
             document: childDocument,
             recipients: [bobContact],
@@ -1123,18 +1241,18 @@ describe('share flow reused by PermissionTypeMenu upgrade', function () {
 
         case 16:
         case "end":
-          return _context20.stop();
+          return _context23.stop();
       }
-    }, _callee20);
+    }, _callee23);
   })));
-  it('leaves the parent unchanged and does not refresh when the child sharing creation fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
+  it('leaves the parent unchanged and does not refresh when the child sharing creation fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
     var provider, parentSharing;
-    return _regeneratorRuntime.wrap(function _callee21$(_context21) {
-      while (1) switch (_context21.prev = _context21.next) {
+    return _regeneratorRuntime.wrap(function _callee24$(_context24) {
+      while (1) switch (_context24.prev = _context24.next) {
         case 0:
           provider = setupWithParent();
           provider.sharingCol.create = jest.fn().mockRejectedValue(new Error('Network error'));
-          _context21.next = 4;
+          _context24.next = 4;
           return expect(provider.share({
             document: childDocument,
             recipients: [bobContact],
@@ -1155,9 +1273,9 @@ describe('share flow reused by PermissionTypeMenu upgrade', function () {
 
         case 11:
         case "end":
-          return _context21.stop();
+          return _context24.stop();
       }
-    }, _callee21);
+    }, _callee24);
   })));
 });
 describe('fetchSharedDriveSharingLinks', function () {
@@ -1183,10 +1301,10 @@ describe('fetchSharedDriveSharingLinks', function () {
     id: 'file_in_drive',
     driveId: 'drive_123'
   };
-  it('fetches shared drive sharing links by file id', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
+  it('fetches shared drive sharing links by file id', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee25() {
     var mockFindLinksByIds, mockClient, provider, result, permissions;
-    return _regeneratorRuntime.wrap(function _callee22$(_context22) {
-      while (1) switch (_context22.prev = _context22.next) {
+    return _regeneratorRuntime.wrap(function _callee25$(_context25) {
+      while (1) switch (_context25.prev = _context25.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: [PERM_DRIVE_FILE]
@@ -1196,11 +1314,11 @@ describe('fetchSharedDriveSharingLinks', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context22.next = 6;
+          _context25.next = 6;
           return provider.fetchSharedDriveSharingLinks(driveFile);
 
         case 6:
-          result = _context22.sent;
+          result = _context25.sent;
           expect(mockClient.collection).toHaveBeenCalledWith('io.cozy.permissions', {
             driveId: 'drive_123'
           });
@@ -1213,14 +1331,14 @@ describe('fetchSharedDriveSharingLinks', function () {
 
         case 14:
         case "end":
-          return _context22.stop();
+          return _context25.stop();
       }
-    }, _callee22);
+    }, _callee25);
   })));
-  it('returns an empty array and skips the API call for non-shared-drive documents', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
+  it('returns an empty array and skips the API call for non-shared-drive documents', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee26() {
     var mockClient, findLinksByIds, provider, result;
-    return _regeneratorRuntime.wrap(function _callee23$(_context23) {
-      while (1) switch (_context23.prev = _context23.next) {
+    return _regeneratorRuntime.wrap(function _callee26$(_context26) {
+      while (1) switch (_context26.prev = _context26.next) {
         case 0:
           mockClient = createMockClient({});
           findLinksByIds = jest.fn();
@@ -1229,27 +1347,27 @@ describe('fetchSharedDriveSharingLinks', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context23.next = 7;
+          _context26.next = 7;
           return provider.fetchSharedDriveSharingLinks({
             _id: 'regular_file'
           });
 
         case 7:
-          result = _context23.sent;
+          result = _context26.sent;
           expect(result).toEqual([]);
           expect(findLinksByIds).not.toHaveBeenCalled();
           expect(provider.dispatch).not.toHaveBeenCalled();
 
         case 11:
         case "end":
-          return _context23.stop();
+          return _context26.stop();
       }
-    }, _callee23);
+    }, _callee26);
   })));
-  it('uses document.id as a fallback when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
+  it('uses document.id as a fallback when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee27() {
     var mockFindLinksByIds, mockClient, provider;
-    return _regeneratorRuntime.wrap(function _callee24$(_context24) {
-      while (1) switch (_context24.prev = _context24.next) {
+    return _regeneratorRuntime.wrap(function _callee27$(_context27) {
+      while (1) switch (_context27.prev = _context27.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: [PERM_DRIVE_FILE]
@@ -1259,7 +1377,7 @@ describe('fetchSharedDriveSharingLinks', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context24.next = 6;
+          _context27.next = 6;
           return provider.fetchSharedDriveSharingLinks({
             id: 'file_in_drive',
             driveId: 'drive_123'
@@ -1270,14 +1388,14 @@ describe('fetchSharedDriveSharingLinks', function () {
 
         case 7:
         case "end":
-          return _context24.stop();
+          return _context27.stop();
       }
-    }, _callee24);
+    }, _callee27);
   })));
-  it('does not dispatch when the response has no data', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee25() {
+  it('does not dispatch when the response has no data', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee28() {
     var mockFindLinksByIds, mockClient, provider, result;
-    return _regeneratorRuntime.wrap(function _callee25$(_context25) {
-      while (1) switch (_context25.prev = _context25.next) {
+    return _regeneratorRuntime.wrap(function _callee28$(_context28) {
+      while (1) switch (_context28.prev = _context28.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: []
@@ -1288,19 +1406,19 @@ describe('fetchSharedDriveSharingLinks', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context25.next = 7;
+          _context28.next = 7;
           return provider.fetchSharedDriveSharingLinks(driveFile);
 
         case 7:
-          result = _context25.sent;
+          result = _context28.sent;
           expect(result).toEqual([]);
           expect(provider.dispatch).not.toHaveBeenCalled();
 
         case 10:
         case "end":
-          return _context25.stop();
+          return _context28.stop();
       }
-    }, _callee25);
+    }, _callee28);
   })));
 });
 describe('shareByLink shared drive 409 recovery', function () {
@@ -1326,10 +1444,10 @@ describe('shareByLink shared drive 409 recovery', function () {
     id: 'file_in_drive',
     driveId: 'drive_123'
   };
-  it('falls back to fetching existing links when create returns 409', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee26() {
+  it('falls back to fetching existing links when create returns 409', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee29() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider, resp;
-    return _regeneratorRuntime.wrap(function _callee26$(_context26) {
-      while (1) switch (_context26.prev = _context26.next) {
+    return _regeneratorRuntime.wrap(function _callee29$(_context29) {
+      while (1) switch (_context29.prev = _context29.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -1344,27 +1462,27 @@ describe('shareByLink shared drive 409 recovery', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context26.next = 8;
+          _context29.next = 8;
           return provider.shareByLink(driveFile, {
             verbs: ['GET']
           });
 
         case 8:
-          resp = _context26.sent;
+          resp = _context29.sent;
           expect(mockCreateSharingLink).toHaveBeenCalled();
           expect(mockFindLinksByIds).toHaveBeenCalledWith(['file_in_drive']);
           expect(resp.data.id).toBe(PERM_DRIVE_FILE.id);
 
         case 12:
         case "end":
-          return _context26.stop();
+          return _context29.stop();
       }
-    }, _callee26);
+    }, _callee29);
   })));
-  it('does not duplicate existing links when 409 recovery refetches them', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee27() {
+  it('does not duplicate existing links when 409 recovery refetches them', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee30() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider, resp, permissions;
-    return _regeneratorRuntime.wrap(function _callee27$(_context27) {
-      while (1) switch (_context27.prev = _context27.next) {
+    return _regeneratorRuntime.wrap(function _callee30$(_context30) {
+      while (1) switch (_context30.prev = _context30.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -1379,13 +1497,13 @@ describe('shareByLink shared drive 409 recovery', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient, {}, reducer(undefined, addSharingLink(PERM_DRIVE_FILE)));
-          _context27.next = 8;
+          _context30.next = 8;
           return provider.shareByLink(driveFile, {
             verbs: ['GET']
           });
 
         case 8:
-          resp = _context27.sent;
+          resp = _context30.sent;
           permissions = getDocumentPermissions(provider.state, 'file_in_drive');
           expect(resp.data.id).toBe(PERM_DRIVE_FILE.id);
           expect(provider.dispatch).not.toHaveBeenCalled();
@@ -1393,14 +1511,14 @@ describe('shareByLink shared drive 409 recovery', function () {
 
         case 13:
         case "end":
-          return _context27.stop();
+          return _context30.stop();
       }
-    }, _callee27);
+    }, _callee30);
   })));
-  it('rethrows the original error when 409 happens and no link is found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee28() {
+  it('rethrows the original error when 409 happens and no link is found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee31() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider;
-    return _regeneratorRuntime.wrap(function _callee28$(_context28) {
-      while (1) switch (_context28.prev = _context28.next) {
+    return _regeneratorRuntime.wrap(function _callee31$(_context31) {
+      while (1) switch (_context31.prev = _context31.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -1416,16 +1534,16 @@ describe('shareByLink shared drive 409 recovery', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context28.next = 9;
+          _context31.next = 9;
           return expect(provider.shareByLink(driveFile, {
             verbs: ['GET']
           })).rejects.toBe(conflict);
 
         case 9:
         case "end":
-          return _context28.stop();
+          return _context31.stop();
       }
-    }, _callee28);
+    }, _callee31);
   })));
 });
 describe('updateSharingMemberType', function () {
@@ -1468,37 +1586,37 @@ describe('updateSharingMemberType', function () {
       })
     };
   });
-  it('should throw when sharing is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee29() {
-    return _regeneratorRuntime.wrap(function _callee29$(_context29) {
-      while (1) switch (_context29.prev = _context29.next) {
+  it('should throw when sharing is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee32() {
+    return _regeneratorRuntime.wrap(function _callee32$(_context32) {
+      while (1) switch (_context32.prev = _context32.next) {
         case 0:
-          _context29.next = 2;
+          _context32.next = 2;
           return expect(instance.updateSharingMemberType('unknown-id', 1, 'one-way')).rejects.toThrow('Sharing not found');
 
         case 2:
         case "end":
-          return _context29.stop();
+          return _context32.stop();
       }
-    }, _callee29);
+    }, _callee32);
   })));
-  it('should throw when member is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee30() {
-    return _regeneratorRuntime.wrap(function _callee30$(_context30) {
-      while (1) switch (_context30.prev = _context30.next) {
+  it('should throw when member is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee33() {
+    return _regeneratorRuntime.wrap(function _callee33$(_context33) {
+      while (1) switch (_context33.prev = _context33.next) {
         case 0:
-          _context30.next = 2;
+          _context33.next = 2;
           return expect(instance.updateSharingMemberType('sharing-123', 99, 'one-way')).rejects.toThrow('Member not found');
 
         case 2:
         case "end":
-          return _context30.stop();
+          return _context33.stop();
       }
-    }, _callee30);
+    }, _callee33);
   })));
-  it('should call setReadOnly when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee31() {
-    return _regeneratorRuntime.wrap(function _callee31$(_context31) {
-      while (1) switch (_context31.prev = _context31.next) {
+  it('should call setReadOnly when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee34() {
+    return _regeneratorRuntime.wrap(function _callee34$(_context34) {
+      while (1) switch (_context34.prev = _context34.next) {
         case 0:
-          _context31.next = 2;
+          _context34.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
 
         case 2:
@@ -1507,15 +1625,15 @@ describe('updateSharingMemberType', function () {
 
         case 4:
         case "end":
-          return _context31.stop();
+          return _context34.stop();
       }
-    }, _callee31);
+    }, _callee34);
   })));
-  it('should call setReadWrite when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee32() {
-    return _regeneratorRuntime.wrap(function _callee32$(_context32) {
-      while (1) switch (_context32.prev = _context32.next) {
+  it('should call setReadWrite when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee35() {
+    return _regeneratorRuntime.wrap(function _callee35$(_context35) {
+      while (1) switch (_context35.prev = _context35.next) {
         case 0:
-          _context32.next = 2;
+          _context35.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
 
         case 2:
@@ -1524,15 +1642,15 @@ describe('updateSharingMemberType', function () {
 
         case 4:
         case "end":
-          return _context32.stop();
+          return _context35.stop();
       }
-    }, _callee32);
+    }, _callee35);
   })));
-  it('should optimistically dispatch a state update when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee33() {
-    return _regeneratorRuntime.wrap(function _callee33$(_context33) {
-      while (1) switch (_context33.prev = _context33.next) {
+  it('should optimistically dispatch a state update when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee36() {
+    return _regeneratorRuntime.wrap(function _callee36$(_context36) {
+      while (1) switch (_context36.prev = _context36.next) {
         case 0:
-          _context33.next = 2;
+          _context36.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
 
         case 2:
@@ -1549,15 +1667,15 @@ describe('updateSharingMemberType', function () {
 
         case 3:
         case "end":
-          return _context33.stop();
+          return _context36.stop();
       }
-    }, _callee33);
+    }, _callee36);
   })));
-  it('should optimistically dispatch a state update when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee34() {
-    return _regeneratorRuntime.wrap(function _callee34$(_context34) {
-      while (1) switch (_context34.prev = _context34.next) {
+  it('should optimistically dispatch a state update when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee37() {
+    return _regeneratorRuntime.wrap(function _callee37$(_context37) {
+      while (1) switch (_context37.prev = _context37.next) {
         case 0:
-          _context34.next = 2;
+          _context37.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
 
         case 2:
@@ -1574,44 +1692,44 @@ describe('updateSharingMemberType', function () {
 
         case 3:
         case "end":
-          return _context34.stop();
+          return _context37.stop();
       }
-    }, _callee34);
+    }, _callee37);
   })));
-  it('should rethrow error if setReadOnly fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee35() {
-    return _regeneratorRuntime.wrap(function _callee35$(_context35) {
-      while (1) switch (_context35.prev = _context35.next) {
+  it('should rethrow error if setReadOnly fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee38() {
+    return _regeneratorRuntime.wrap(function _callee38$(_context38) {
+      while (1) switch (_context38.prev = _context38.next) {
         case 0:
           instance.sharingCol.setReadOnly.mockRejectedValue(new Error('Network error'));
-          _context35.next = 3;
+          _context38.next = 3;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 3:
         case "end":
-          return _context35.stop();
+          return _context38.stop();
       }
-    }, _callee35);
+    }, _callee38);
   })));
-  it('should rethrow error if setReadWrite fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee36() {
-    return _regeneratorRuntime.wrap(function _callee36$(_context36) {
-      while (1) switch (_context36.prev = _context36.next) {
+  it('should rethrow error if setReadWrite fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee39() {
+    return _regeneratorRuntime.wrap(function _callee39$(_context39) {
+      while (1) switch (_context39.prev = _context39.next) {
         case 0:
           instance.sharingCol.setReadWrite.mockRejectedValue(new Error('Network error'));
-          _context36.next = 3;
+          _context39.next = 3;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'two-way')).rejects.toThrow('Network error');
 
         case 3:
         case "end":
-          return _context36.stop();
+          return _context39.stop();
       }
-    }, _callee36);
+    }, _callee39);
   })));
-  it('should rollback optimistic state update if API call fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee37() {
-    return _regeneratorRuntime.wrap(function _callee37$(_context37) {
-      while (1) switch (_context37.prev = _context37.next) {
+  it('should rollback optimistic state update if API call fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee40() {
+    return _regeneratorRuntime.wrap(function _callee40$(_context40) {
+      while (1) switch (_context40.prev = _context40.next) {
         case 0:
           instance.sharingCol.setReadOnly.mockRejectedValue(new Error('Network error'));
-          _context37.next = 3;
+          _context40.next = 3;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 3:
@@ -1650,14 +1768,14 @@ describe('updateSharingMemberType', function () {
 
         case 7:
         case "end":
-          return _context37.stop();
+          return _context40.stop();
       }
-    }, _callee37);
+    }, _callee40);
   })));
-  it('should preserve realtime changes when rolling back an API failure', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee39() {
+  it('should preserve realtime changes when rolling back an API failure', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee42() {
     var realtimeSharing;
-    return _regeneratorRuntime.wrap(function _callee39$(_context39) {
-      while (1) switch (_context39.prev = _context39.next) {
+    return _regeneratorRuntime.wrap(function _callee42$(_context42) {
+      while (1) switch (_context42.prev = _context42.next) {
         case 0:
           realtimeSharing = _objectSpread(_objectSpread({}, mockSharing), {}, {
             attributes: _objectSpread(_objectSpread({}, mockSharing.attributes), {}, {
@@ -1669,9 +1787,9 @@ describe('updateSharingMemberType', function () {
               })]
             })
           });
-          instance.sharingCol.setReadOnly.mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee38() {
-            return _regeneratorRuntime.wrap(function _callee38$(_context38) {
-              while (1) switch (_context38.prev = _context38.next) {
+          instance.sharingCol.setReadOnly.mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee41() {
+            return _regeneratorRuntime.wrap(function _callee41$(_context41) {
+              while (1) switch (_context41.prev = _context41.next) {
                 case 0:
                   instance.state = _objectSpread(_objectSpread({}, instance.state), {}, {
                     sharings: instance.state.sharings.map(function (sharing) {
@@ -1682,11 +1800,11 @@ describe('updateSharingMemberType', function () {
 
                 case 2:
                 case "end":
-                  return _context38.stop();
+                  return _context41.stop();
               }
-            }, _callee38);
+            }, _callee41);
           })));
-          _context39.next = 4;
+          _context42.next = 4;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 4:
@@ -1715,15 +1833,15 @@ describe('updateSharingMemberType', function () {
 
         case 7:
         case "end":
-          return _context39.stop();
+          return _context42.stop();
       }
-    }, _callee39);
+    }, _callee42);
   })));
-  it('should refetch effective recipients of the given document through its drive', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee40() {
-    return _regeneratorRuntime.wrap(function _callee40$(_context40) {
-      while (1) switch (_context40.prev = _context40.next) {
+  it('should refetch effective recipients of the given document through its drive', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee43() {
+    return _regeneratorRuntime.wrap(function _callee43$(_context43) {
+      while (1) switch (_context43.prev = _context43.next) {
         case 0:
-          _context40.next = 2;
+          _context43.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'one-way', {
             _id: 'nested-folder',
             driveId: 'drive-sharing'
@@ -1736,15 +1854,15 @@ describe('updateSharingMemberType', function () {
 
         case 3:
         case "end":
-          return _context40.stop();
+          return _context43.stop();
       }
-    }, _callee40);
+    }, _callee43);
   })));
-  it('should refetch effective recipients of an owned document locally', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee41() {
-    return _regeneratorRuntime.wrap(function _callee41$(_context41) {
-      while (1) switch (_context41.prev = _context41.next) {
+  it('should refetch effective recipients of an owned document locally', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee44() {
+    return _regeneratorRuntime.wrap(function _callee44$(_context44) {
+      while (1) switch (_context44.prev = _context44.next) {
         case 0:
-          _context41.next = 2;
+          _context44.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'one-way', {
             _id: 'owned-folder'
           });
@@ -1756,13 +1874,13 @@ describe('updateSharingMemberType', function () {
 
         case 3:
         case "end":
-          return _context41.stop();
+          return _context44.stop();
       }
-    }, _callee41);
+    }, _callee44);
   })));
-  it('should not refetch effective recipients without a document', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee42() {
-    return _regeneratorRuntime.wrap(function _callee42$(_context42) {
-      while (1) switch (_context42.prev = _context42.next) {
+  it('should not refetch effective recipients without a document', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee45() {
+    return _regeneratorRuntime.wrap(function _callee45$(_context45) {
+      while (1) switch (_context45.prev = _context45.next) {
         case 0:
           instance.state = _objectSpread(_objectSpread({}, instance.state), {}, {
             byDocId: {
@@ -1772,7 +1890,7 @@ describe('updateSharingMemberType', function () {
               }
             }
           });
-          _context42.next = 3;
+          _context45.next = 3;
           return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
 
         case 3:
@@ -1780,9 +1898,9 @@ describe('updateSharingMemberType', function () {
 
         case 4:
         case "end":
-          return _context42.stop();
+          return _context45.stop();
       }
-    }, _callee42);
+    }, _callee45);
   })));
 });
 describe('hasWriteAccess', function () {