cozy-sharing 28.5.0 → 28.6.0

package/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [28.6.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.5.0...cozy-sharing@28.6.0) (2026-03-11)
+
+### Features
+
+- Add ability to change member permission type in sharing ([76d0dce](https://github.com/cozy/cozy-libs/commit/76d0dce150d8ea7ac7cb12c6e00b340756b6512a))
+- Hide link text in link recipient ([209857f](https://github.com/cozy/cozy-libs/commit/209857fcf01b71e584e49ef8ab3da4a897d408ed))
+- Remove link prop from LinkRecipient component ([7bbf831](https://github.com/cozy/cozy-libs/commit/7bbf831594d7c33357ade79665f4308fd35e2b8e))
+
 # [28.5.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.4.4...cozy-sharing@28.5.0) (2026-03-10)
 
 ### Bug Fixes

package/dist/SharingProvider.js

@@ -463,48 +463,143 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       };
     }());
 
-    _defineProperty(_this2, "shareByLink", /*#__PURE__*/function () {
-      var _ref12 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(document, options) {
-        var resp;
+    /**
+     * Update a sharing member's read_only field to change permission type
+     * @param {string} sharingId - The ID of the sharing
+     * @param {number} memberIndex - The index of the member in the sharing members array
+     * @param {string} newType - 'one-way' for read-only, 'two-way' for read-write
+     */
+    _defineProperty(_this2, "updateSharingMemberType", /*#__PURE__*/function () {
+      var _ref12 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(sharingId, memberIndex, newType) {
+        var sharing, member, matchedContacts, contact, readOnly;
         return _regeneratorRuntime.wrap(function _callee10$(_context10) {
           while (1) switch (_context10.prev = _context10.next) {
             case 0:
-              if (!document.driveId) {
+              sharing = _getSharingById(_this2.state, sharingId);
+
+              if (sharing) {
+                _context10.next = 3;
+                break;
+              }
+
+              throw new Error('Sharing not found');
+
+            case 3:
+              member = sharing.attributes.members[memberIndex];
+
+              if (member) {
                 _context10.next = 6;
                 break;
               }
 
-              _context10.next = 3;
+              throw new Error('Member not found');
+
+            case 6:
+              _context10.next = 8;
+              return _this2.props.client.collection('io.cozy.contacts').find({
+                email: {
+                  $elemMatch: {
+                    address: member.email
+                  }
+                },
+                _id: {
+                  $gt: null
+                }
+              }, {
+                indexedFields: ['_id']
+              });
+
+            case 8:
+              matchedContacts = _context10.sent;
+
+              if (matchedContacts.data.length) {
+                _context10.next = 11;
+                break;
+              }
+
+              throw new Error("Contact not found for member ".concat(member.email));
+
+            case 11:
+              contact = matchedContacts.data[0]; // Revoke (API call only, no state dispatch to avoid UI flickering)
+
+              _context10.next = 14;
+              return _this2.sharingCol.revokeRecipient(sharing, memberIndex);
+
+            case 14:
+              // Re-add with the new permission type
+              readOnly = newType === 'one-way';
+              _context10.prev = 15;
+              _context10.next = 18;
+              return _this2.addRecipients({
+                document: sharing,
+                recipients: readOnly ? [] : [contact],
+                readOnlyRecipients: readOnly ? [contact] : []
+              });
+
+            case 18:
+              _context10.next = 24;
+              break;
+
+            case 20:
+              _context10.prev = 20;
+              _context10.t0 = _context10["catch"](15);
+              log.error("Failed to re-add member ".concat(member.email, " after revocation. The member has been revoked but could not be re-added with the new permission type."), _context10.t0);
+              throw _context10.t0;
+
+            case 24:
+            case "end":
+              return _context10.stop();
+          }
+        }, _callee10, null, [[15, 20]]);
+      }));
+
+      return function (_x16, _x17, _x18) {
+        return _ref12.apply(this, arguments);
+      };
+    }());
+
+    _defineProperty(_this2, "shareByLink", /*#__PURE__*/function () {
+      var _ref13 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(document, options) {
+        var resp;
+        return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+          while (1) switch (_context11.prev = _context11.next) {
+            case 0:
+              if (!document.driveId) {
+                _context11.next = 6;
+                break;
+              }
+
+              _context11.next = 3;
               return _this2.createSharedDriveSharingLink(document, options);
 
             case 3:
-              _context10.t0 = _context10.sent;
-              _context10.next = 9;
+              _context11.t0 = _context11.sent;
+              _context11.next = 9;
               break;
 
             case 6:
-              _context10.next = 8;
+              _context11.next = 8;
               return _this2.permissionCol.createSharingLink(document, options);
 
             case 8:
-              _context10.t0 = _context10.sent;
+              _context11.t0 = _context11.sent;
 
             case 9:
-              resp = _context10.t0;
+              resp = _context11.t0;
 
               _this2.dispatch(addSharingLink(resp.data));
 
-              return _context10.abrupt("return", resp);
+              return _context11.abrupt("return", resp);
 
             case 12:
             case "end":
-              return _context10.stop();
+              return _context11.stop();
           }
-        }, _callee10);
+        }, _callee11);
       }));
 
-      return function (_x16, _x17) {
-        return _ref12.apply(this, arguments);
+      return function (_x19, _x20) {
+        return _ref13.apply(this, arguments);
       };
     }());
 
@@ -531,67 +626,67 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
     });
 
     _defineProperty(_this2, "createSharedDriveSharingLink", /*#__PURE__*/function () {
-      var _ref13 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(document, options) {
+      var _ref14 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12(document, options) {
         var client, drivePermissionCollection, resp;
-        return _regeneratorRuntime.wrap(function _callee11$(_context11) {
-          while (1) switch (_context11.prev = _context11.next) {
+        return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+          while (1) switch (_context12.prev = _context12.next) {
             case 0:
               client = _this2.props.client;
               drivePermissionCollection = client.collection('io.cozy.permissions', {
                 driveId: document.driveId
               });
-              _context11.next = 4;
+              _context12.next = 4;
               return drivePermissionCollection.createSharingLink(document, options);
 
             case 4:
-              resp = _context11.sent;
-              return _context11.abrupt("return", resp);
+              resp = _context12.sent;
+              return _context12.abrupt("return", resp);
 
             case 6:
             case "end":
-              return _context11.stop();
+              return _context12.stop();
           }
-        }, _callee11);
+        }, _callee12);
       }));
 
-      return function (_x18, _x19) {
-        return _ref13.apply(this, arguments);
+      return function (_x21, _x22) {
+        return _ref14.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "updateSharedDrivePermissions", /*#__PURE__*/function () {
-      var _ref14 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12(document, permissionDocument, updatedPermissions, options) {
+      var _ref15 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13(document, permissionDocument, updatedPermissions, options) {
         var client, expiresAt, password, drivePermissionCollection, resp;
-        return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-          while (1) switch (_context12.prev = _context12.next) {
+        return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+          while (1) switch (_context13.prev = _context13.next) {
             case 0:
               client = _this2.props.client;
               expiresAt = options.expiresAt, password = options.password;
               drivePermissionCollection = client.collection('io.cozy.permissions', {
                 driveId: document.driveId
               });
-              _context12.next = 5;
+              _context13.next = 5;
               return drivePermissionCollection.add(permissionDocument, updatedPermissions, {
                 expiresAt: expiresAt,
                 password: password
               });
 
             case 5:
-              resp = _context12.sent;
+              resp = _context13.sent;
 
               _this2.dispatch(updateSharingLink(resp.data));
 
-              return _context12.abrupt("return", resp);
+              return _context13.abrupt("return", resp);
 
             case 8:
             case "end":
-              return _context12.stop();
+              return _context13.stop();
           }
-        }, _callee12);
+        }, _callee13);
       }));
 
-      return function (_x20, _x21, _x22, _x23) {
-        return _ref14.apply(this, arguments);
+      return function (_x23, _x24, _x25, _x26) {
+        return _ref15.apply(this, arguments);
       };
     }());
 
@@ -607,28 +702,28 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
      * @return {Array}
      */
     _defineProperty(_this2, "updateDocumentPermissions", /*#__PURE__*/function () {
-      var _ref15 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14(document, options) {
+      var _ref16 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15(document, options) {
         var verbs, expiresAt, password, permissions, responses;
-        return _regeneratorRuntime.wrap(function _callee14$(_context14) {
-          while (1) switch (_context14.prev = _context14.next) {
+        return _regeneratorRuntime.wrap(function _callee15$(_context15) {
+          while (1) switch (_context15.prev = _context15.next) {
             case 0:
               verbs = options.verbs, expiresAt = options.expiresAt, password = options.password;
               permissions = _getDocumentPermissions(_this2.state, document.id);
 
               if (!(!permissions || permissions.length === 0)) {
-                _context14.next = 4;
+                _context15.next = 4;
                 break;
               }
 
-              return _context14.abrupt("return", []);
+              return _context15.abrupt("return", []);
 
             case 4:
-              _context14.next = 6;
+              _context15.next = 6;
               return Promise.all(permissions.map( /*#__PURE__*/function () {
-                var _ref16 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13(permissionDocument) {
+                var _ref17 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14(permissionDocument) {
                   var updatedPermissions, resp;
-                  return _regeneratorRuntime.wrap(function _callee13$(_context13) {
-                    while (1) switch (_context13.prev = _context13.next) {
+                  return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+                    while (1) switch (_context14.prev = _context14.next) {
                       case 0:
                         updatedPermissions = permissionDocument.attributes.permissions;
                         Object.keys(updatedPermissions).forEach(function (permType) {
@@ -636,70 +731,70 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
                         });
 
                         if (!document.driveId) {
-                          _context13.next = 8;
+                          _context14.next = 8;
                           break;
                         }
 
-                        _context13.next = 5;
+                        _context14.next = 5;
                         return _this2.updateSharedDrivePermissions(document, permissionDocument, updatedPermissions, options);
 
                       case 5:
-                        _context13.t0 = _context13.sent;
-                        _context13.next = 11;
+                        _context14.t0 = _context14.sent;
+                        _context14.next = 11;
                         break;
 
                       case 8:
-                        _context13.next = 10;
+                        _context14.next = 10;
                         return _this2.permissionCol.add(permissionDocument, updatedPermissions, {
                           expiresAt: expiresAt,
                           password: password
                         });
 
                       case 10:
-                        _context13.t0 = _context13.sent;
+                        _context14.t0 = _context14.sent;
 
                       case 11:
-                        resp = _context13.t0;
-                        return _context13.abrupt("return", resp);
+                        resp = _context14.t0;
+                        return _context14.abrupt("return", resp);
 
                       case 13:
                       case "end":
-                        return _context13.stop();
+                        return _context14.stop();
                     }
-                  }, _callee13);
+                  }, _callee14);
                 }));
 
-                return function (_x26) {
-                  return _ref16.apply(this, arguments);
+                return function (_x29) {
+                  return _ref17.apply(this, arguments);
                 };
               }()));
 
             case 6:
-              responses = _context14.sent;
-              return _context14.abrupt("return", responses);
+              responses = _context15.sent;
+              return _context15.abrupt("return", responses);
 
             case 8:
             case "end":
-              return _context14.stop();
+              return _context15.stop();
           }
-        }, _callee14);
+        }, _callee15);
       }));
 
-      return function (_x24, _x25) {
-        return _ref15.apply(this, arguments);
+      return function (_x27, _x28) {
+        return _ref16.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "revokeSharingLink", /*#__PURE__*/function () {
-      var _ref17 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15(document) {
+      var _ref18 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16(document) {
         var perms;
-        return _regeneratorRuntime.wrap(function _callee15$(_context15) {
-          while (1) switch (_context15.prev = _context15.next) {
+        return _regeneratorRuntime.wrap(function _callee16$(_context16) {
+          while (1) switch (_context16.prev = _context16.next) {
             case 0:
               // Because some duplicate links have been created in the past, we must ensure
               // we revoke all of them
               perms = _getDocumentPermissions(_this2.state, document.id);
-              _context15.next = 3;
+              _context16.next = 3;
               return Promise.all(perms.map(function (p) {
                 return _this2.permissionCol.destroy(p);
               }));
@@ -709,13 +804,13 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 4:
             case "end":
-              return _context15.stop();
+              return _context16.stop();
           }
-        }, _callee15);
+        }, _callee16);
       }));
 
-      return function (_x27) {
-        return _ref17.apply(this, arguments);
+      return function (_x30) {
+        return _ref18.apply(this, arguments);
       };
     }());
 
@@ -802,7 +897,8 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       revokeAllRecipients: _this2.revokeAllRecipients,
       refresh: _this2.fetchAllSharings,
       hasWriteAccess: _this2.hasWriteAccess,
-      renameSharedDrive: _this2.renameSharedDrive
+      renameSharedDrive: _this2.renameSharedDrive,
+      updateSharingMemberType: _this2.updateSharingMemberType
     };
     _this2.isPublic = (_props$isPublic = props.isPublic) !== null && _props$isPublic !== void 0 ? _props$isPublic : false;
     _this2.realtime = null;
@@ -838,14 +934,14 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
   }, {
     key: "fetchAllSharings",
     value: function () {
-      var _fetchAllSharings = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
+      var _fetchAllSharings = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
         var _this$props, doctype, client, _yield$Promise$all, _yield$Promise$all2, sharings, permissions, apps, sharedDocIds, resp, folderPaths, filePaths;
 
-        return _regeneratorRuntime.wrap(function _callee16$(_context16) {
-          while (1) switch (_context16.prev = _context16.next) {
+        return _regeneratorRuntime.wrap(function _callee17$(_context17) {
+          while (1) switch (_context17.prev = _context17.next) {
             case 0:
               if (!this.isPublic) {
-                _context16.next = 4;
+                _context17.next = 4;
                 break;
               }
 
@@ -854,17 +950,17 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
               this.setState({
                 allLoaded: true
               });
-              return _context16.abrupt("return");
+              return _context17.abrupt("return");
 
             case 4:
               _this$props = this.props, doctype = _this$props.doctype, client = _this$props.client;
-              _context16.next = 7;
+              _context17.next = 7;
               return Promise.all([this.sharingCol.findByDoctype(doctype, {
                 withSharedDocs: false
               }), this.permissionCol.findLinksByDoctype(doctype), client.fetchQueryAndGetFromState(fetchApps())]);
 
             case 7:
-              _yield$Promise$all = _context16.sent;
+              _yield$Promise$all = _context17.sent;
               _yield$Promise$all2 = _slicedToArray(_yield$Promise$all, 3);
               sharings = _yield$Promise$all2[0];
               permissions = _yield$Promise$all2[1];
@@ -879,35 +975,35 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
                 hasLoadedAtLeastOnePage: true
               }); // eslint-disable-next-line promise/catch-or-return
 
-              _context16.next = 16;
+              _context17.next = 16;
               return fetchNextPermissions(permissions, this.dispatch, this.permissionCol);
 
             case 16:
               if (!(doctype === 'io.cozy.files')) {
-                _context16.next = 26;
+                _context17.next = 26;
                 break;
               }
 
               sharedDocIds = getSharedDocIdsBySharings(sharings);
-              _context16.next = 20;
+              _context17.next = 20;
               return client.collection(doctype).all({
                 keys: sharedDocIds
               });
 
             case 20:
-              resp = _context16.sent;
+              resp = _context17.sent;
               folderPaths = resp.data.filter(function (f) {
                 return f.type === 'directory' && !f.trashed;
               }).map(function (f) {
                 return f.path;
               });
-              _context16.next = 24;
+              _context17.next = 24;
               return fetchFilesPaths(client, doctype, resp.data.filter(function (f) {
                 return f.type !== 'directory' && !f.trashed;
               }));
 
             case 24:
-              filePaths = _context16.sent;
+              filePaths = _context17.sent;
               this.dispatch(receivePaths([].concat(_toConsumableArray(folderPaths), _toConsumableArray(filePaths))));
 
             case 26:
@@ -917,9 +1013,9 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 27:
             case "end":
-              return _context16.stop();
+              return _context17.stop();
           }
-        }, _callee16, this);
+        }, _callee17, this);
       }));
 
       function fetchAllSharings() {

package/dist/SharingProvider.spec.js

@@ -1,4 +1,10 @@
+import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
+
+function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
+
+function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { _defineProperty(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
+
 import _regeneratorRuntime from "@babel/runtime/regenerator";
 import { act, render, screen } from '@testing-library/react';
 import React from 'react';
@@ -308,6 +314,152 @@ describe('updateDocumentPermissions', function () {
       }
     }, _callee5);
   })));
+});
+describe('updateSharingMemberType', function () {
+  var mockContact = {
+    _id: 'contact-1',
+    email: [{
+      address: 'bob@bob.cozy'
+    }]
+  };
+  var mockSharing = {
+    id: 'sharing-123',
+    type: 'io.cozy.sharings',
+    attributes: {
+      members: [{
+        status: 'owner',
+        email: 'owner@cozy.local'
+      }, {
+        status: 'ready',
+        email: 'bob@bob.cozy'
+      }]
+    }
+  };
+  var instance;
+  var mockContactsCollection;
+  beforeEach(function () {
+    mockContactsCollection = {
+      find: jest.fn().mockResolvedValue({
+        data: [mockContact]
+      })
+    };
+    var mockClient = {
+      getStackClient: function getStackClient() {
+        return {
+          uri: 'http://cozy.local'
+        };
+      },
+      collection: jest.fn().mockReturnValue(mockContactsCollection)
+    };
+    instance = new SharingProvider({
+      client: mockClient
+    });
+    instance.state = _objectSpread(_objectSpread({}, instance.state), {}, {
+      sharings: [mockSharing]
+    });
+    instance.sharingCol = {
+      revokeRecipient: jest.fn().mockResolvedValue({})
+    };
+    jest.spyOn(instance, 'addRecipients').mockResolvedValue({});
+  });
+  it('should throw when sharing is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee6() {
+    return _regeneratorRuntime.wrap(function _callee6$(_context6) {
+      while (1) switch (_context6.prev = _context6.next) {
+        case 0:
+          _context6.next = 2;
+          return expect(instance.updateSharingMemberType('unknown-id', 1, 'one-way')).rejects.toThrow('Sharing not found');
+
+        case 2:
+        case "end":
+          return _context6.stop();
+      }
+    }, _callee6);
+  })));
+  it('should throw when member is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7() {
+    return _regeneratorRuntime.wrap(function _callee7$(_context7) {
+      while (1) switch (_context7.prev = _context7.next) {
+        case 0:
+          _context7.next = 2;
+          return expect(instance.updateSharingMemberType('sharing-123', 99, 'one-way')).rejects.toThrow('Member not found');
+
+        case 2:
+        case "end":
+          return _context7.stop();
+      }
+    }, _callee7);
+  })));
+  it('should throw when contact is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8() {
+    return _regeneratorRuntime.wrap(function _callee8$(_context8) {
+      while (1) switch (_context8.prev = _context8.next) {
+        case 0:
+          mockContactsCollection.find.mockResolvedValue({
+            data: []
+          });
+          _context8.next = 3;
+          return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Contact not found');
+
+        case 3:
+        case "end":
+          return _context8.stop();
+      }
+    }, _callee8);
+  })));
+  it('should revoke and re-add with readOnly when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9() {
+    return _regeneratorRuntime.wrap(function _callee9$(_context9) {
+      while (1) switch (_context9.prev = _context9.next) {
+        case 0:
+          _context9.next = 2;
+          return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
+
+        case 2:
+          expect(instance.sharingCol.revokeRecipient).toHaveBeenCalledWith(mockSharing, 1);
+          expect(instance.addRecipients).toHaveBeenCalledWith({
+            document: mockSharing,
+            recipients: [],
+            readOnlyRecipients: [mockContact]
+          });
+
+        case 4:
+        case "end":
+          return _context9.stop();
+      }
+    }, _callee9);
+  })));
+  it('should revoke and re-add as read-write when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
+    return _regeneratorRuntime.wrap(function _callee10$(_context10) {
+      while (1) switch (_context10.prev = _context10.next) {
+        case 0:
+          _context10.next = 2;
+          return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
+
+        case 2:
+          expect(instance.sharingCol.revokeRecipient).toHaveBeenCalledWith(mockSharing, 1);
+          expect(instance.addRecipients).toHaveBeenCalledWith({
+            document: mockSharing,
+            recipients: [mockContact],
+            readOnlyRecipients: []
+          });
+
+        case 4:
+        case "end":
+          return _context10.stop();
+      }
+    }, _callee10);
+  })));
+  it('should rethrow error if addRecipients fails after revocation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
+    return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+      while (1) switch (_context11.prev = _context11.next) {
+        case 0:
+          instance.addRecipients.mockRejectedValue(new Error('Network error'));
+          _context11.next = 3;
+          return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
+
+        case 3:
+        case "end":
+          return _context11.stop();
+      }
+    }, _callee11);
+  })));
 }); // TODO Convert with react-testing-library
 // describe('hasWriteAccess', () => {
 //   it('tells if a doc is writtable', () => {