cozy-sharing 33.3.4 → 33.4.1

package/dist/components/FederatedFolder/FederatedFolderModal.js

@@ -2,8 +2,9 @@ import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
 import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import _regeneratorRuntime from "@babel/runtime/regenerator";
 import PropTypes from 'prop-types';
-import React, { useCallback, useState } from 'react';
+import React, { useCallback, useEffect, useState } from 'react';
 import { useClient } from 'cozy-client';
+import minilog from 'cozy-minilog';
 import Button from 'cozy-ui/transpiled/react/Buttons';
 import { FixedDialog } from 'cozy-ui/transpiled/react/CozyDialogs';
 import Typography from 'cozy-ui/transpiled/react/Typography';
@@ -39,6 +40,7 @@ import AntivirusAlert from '../AntivirusAlert';
 import { default as DumbShareByEmail } from '../ShareByEmail';
 import ShareByLink from '../ShareByLink';
 import WhoHasAccess from '../WhoHasAccess';
+var log = minilog('FederatedFolderModal');
 
 var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
   var _sharedDriveSharing$a;
@@ -57,6 +59,8 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       getSharingById = _useSharingContext.getSharingById,
       getSharingLink = _useSharingContext.getSharingLink,
       getFederatedShareLink = _useSharingContext.getFederatedShareLink,
+      getDocumentPermissions = _useSharingContext.getDocumentPermissions,
+      fetchSharedDriveSharingLinks = _useSharingContext.fetchSharedDriveSharingLinks,
       getRecipients = _useSharingContext.getRecipients,
       hasSharedChild = _useSharingContext.hasSharedChild,
       hasSharedParent = _useSharingContext.hasSharedParent,
@@ -86,15 +90,25 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       isSending = _useState2[0],
       setIsSending = _useState2[1];
 
-  var _useState3 = useState(null),
+  var _useState3 = useState(false),
       _useState4 = _slicedToArray(_useState3, 2),
-      frozenDoc = _useState4[0],
-      setFrozenDoc = _useState4[1];
+      isFetchingSharingLinks = _useState4[0],
+      setIsFetchingSharingLinks = _useState4[1];
 
   var _useState5 = useState(null),
       _useState6 = _slicedToArray(_useState5, 2),
-      frozenRecipients = _useState6[0],
-      setFrozenRecipients = _useState6[1];
+      fetchedSharingLinksDocumentId = _useState6[0],
+      setFetchedSharingLinksDocumentId = _useState6[1];
+
+  var _useState7 = useState(null),
+      _useState8 = _slicedToArray(_useState7, 2),
+      frozenDoc = _useState8[0],
+      setFrozenDoc = _useState8[1];
+
+  var _useState9 = useState(null),
+      _useState10 = _slicedToArray(_useState9, 2),
+      frozenRecipients = _useState10[0],
+      setFrozenRecipients = _useState10[1];
 
   var handleCloseRequest = useCallback(function () {
     if (pendingRecipients.length === 0) {
@@ -209,8 +223,57 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
     };
   }();
 
-  var existingRecipients = existingDocument ? getRecipients(existingDocument._id) : [];
+  var documentId = (existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument._id) || (existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.id);
+  var existingRecipients = documentId ? getRecipients(documentId) : [];
+  var documentPermissions = documentId ? getDocumentPermissions(documentId) : [];
   var displayedLink = existingDocument !== null && existingDocument !== void 0 && existingDocument.driveId ? getFederatedShareLink(existingDocument) : getSharingLink(existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument._id);
+  useEffect(function () {
+    if (!(existingDocument !== null && existingDocument !== void 0 && existingDocument.driveId)) return;
+    if (!documentId) return;
+    if (!fetchSharedDriveSharingLinks) return;
+    if (documentPermissions.length > 0) return;
+    if (isFetchingSharingLinks) return;
+    if (fetchedSharingLinksDocumentId === documentId) return;
+
+    var fetchSharingLinks = /*#__PURE__*/function () {
+      var _ref3 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2() {
+        return _regeneratorRuntime.wrap(function _callee2$(_context2) {
+          while (1) switch (_context2.prev = _context2.next) {
+            case 0:
+              setIsFetchingSharingLinks(true);
+              _context2.prev = 1;
+              _context2.next = 4;
+              return fetchSharedDriveSharingLinks(existingDocument);
+
+            case 4:
+              _context2.next = 9;
+              break;
+
+            case 6:
+              _context2.prev = 6;
+              _context2.t0 = _context2["catch"](1);
+              log.error('Failed to fetch shared drive sharing links', _context2.t0);
+
+            case 9:
+              _context2.prev = 9;
+              setFetchedSharingLinksDocumentId(documentId);
+              setIsFetchingSharingLinks(false);
+              return _context2.finish(9);
+
+            case 13:
+            case "end":
+              return _context2.stop();
+          }
+        }, _callee2, null, [[1, 6, 9, 13]]);
+      }));
+
+      return function fetchSharingLinks() {
+        return _ref3.apply(this, arguments);
+      };
+    }();
+
+    fetchSharingLinks();
+  }, [documentId, existingDocument, documentPermissions.length, fetchedSharingLinksDocumentId, fetchSharedDriveSharingLinks, isFetchingSharingLinks]);
   var modalTitle = t('FederatedFolder.shareTitle', {
     name: folderName
   });

package/dist/components/FederatedFolder/FederatedFolderModal.spec.js

@@ -23,6 +23,8 @@ var mockGetRecipients = jest.fn().mockReturnValue([]);
 var mockGetSharingById = jest.fn();
 var mockHasSharedChild = jest.fn();
 var mockHasSharedParent = jest.fn();
+var mockFetchSharedDriveSharingLinks = jest.fn();
+var mockGetDocumentPermissions = jest.fn().mockReturnValue([]);
 var mockShowAlert = jest.fn();
 var mockOnClose = jest.fn();
 jest.mock('../../hooks/useSharingContext', function () {
@@ -34,7 +36,8 @@ jest.mock('../../hooks/useSharingContext', function () {
         revoke: mockRevoke,
         getSharingLink: mockGetSharingLink,
         getFederatedShareLink: mockGetFederatedShareLink,
-        getDocumentPermissions: jest.fn().mockReturnValue([]),
+        getDocumentPermissions: mockGetDocumentPermissions,
+        fetchSharedDriveSharingLinks: mockFetchSharedDriveSharingLinks,
         getOwner: jest.fn(),
         getRecipients: mockGetRecipients,
         getSharingById: mockGetSharingById,
@@ -85,7 +88,8 @@ var createSharingContextValue = function createSharingContextValue() {
     hasWriteAccess: jest.fn(),
     getRecipients: jest.fn(),
     getSharingLink: mockGetSharingLink,
-    getDocumentPermissions: jest.fn().mockReturnValue([]),
+    getDocumentPermissions: mockGetDocumentPermissions,
+    fetchSharedDriveSharingLinks: mockFetchSharedDriveSharingLinks,
     share: mockShare
   };
 };
@@ -118,6 +122,8 @@ describe('FederatedFolderModal', function () {
   var sharingContextValue;
   beforeEach(function () {
     jest.clearAllMocks();
+    mockGetDocumentPermissions.mockReturnValue([]);
+    mockFetchSharedDriveSharingLinks.mockResolvedValue([]);
     mockGetSharingLink.mockReturnValue('https://example.com/share/abc123');
     mockGetFederatedShareLink.mockReturnValue('https://example.com/share/federated-abc123');
     mockGetRecipients.mockReturnValue([]);
@@ -326,26 +332,169 @@ describe('FederatedFolderModal', function () {
       }, _callee7);
     })));
   });
+  describe('shared drive link fetch on mount', function () {
+    it('should fetch shared drive sharing links when document has a driveId and no permissions yet', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8() {
+      return _regeneratorRuntime.wrap(function _callee8$(_context8) {
+        while (1) switch (_context8.prev = _context8.next) {
+          case 0:
+            mockGetDocumentPermissions.mockReturnValue([]);
+            setup({
+              document: _objectSpread(_objectSpread({}, mockDocument), {}, {
+                driveId: 'federated-folder-id'
+              })
+            });
+            _context8.next = 4;
+            return waitFor(function () {
+              expect(mockFetchSharedDriveSharingLinks).toHaveBeenCalledTimes(1);
+            });
+
+          case 4:
+            expect(mockFetchSharedDriveSharingLinks).toHaveBeenCalledWith(expect.objectContaining({
+              _id: mockDocument._id,
+              driveId: 'federated-folder-id'
+            }));
+
+          case 5:
+          case "end":
+            return _context8.stop();
+        }
+      }, _callee8);
+    })));
+    it('should use document.id when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9() {
+      var document;
+      return _regeneratorRuntime.wrap(function _callee9$(_context9) {
+        while (1) switch (_context9.prev = _context9.next) {
+          case 0:
+            document = {
+              id: 'folder-id-only',
+              name: 'My Test Folder',
+              path: '/test/folder',
+              driveId: 'federated-folder-id'
+            };
+            setup({
+              document: document
+            });
+            _context9.next = 4;
+            return waitFor(function () {
+              expect(mockFetchSharedDriveSharingLinks).toHaveBeenCalledWith(document);
+            });
+
+          case 4:
+            expect(mockGetDocumentPermissions).toHaveBeenCalledWith('folder-id-only');
+
+          case 5:
+          case "end":
+            return _context9.stop();
+        }
+      }, _callee9);
+    })));
+    it('should not fetch shared drive sharing links when document has no driveId', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
+      var _setup8, findByText;
+
+      return _regeneratorRuntime.wrap(function _callee10$(_context10) {
+        while (1) switch (_context10.prev = _context10.next) {
+          case 0:
+            _setup8 = setup({
+              document: mockDocument
+            }), findByText = _setup8.findByText;
+            _context10.next = 3;
+            return findByText('Copy link');
+
+          case 3:
+            expect(mockFetchSharedDriveSharingLinks).not.toHaveBeenCalled();
+
+          case 4:
+          case "end":
+            return _context10.stop();
+        }
+      }, _callee10);
+    })));
+    it('should not refetch after one attempt when permissions stay empty', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
+      var document, _setup9, rerender;
+
+      return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+        while (1) switch (_context11.prev = _context11.next) {
+          case 0:
+            document = _objectSpread(_objectSpread({}, mockDocument), {}, {
+              driveId: 'federated-folder-id'
+            });
+            _setup9 = setup({
+              document: document
+            }), rerender = _setup9.rerender;
+            _context11.next = 4;
+            return waitFor(function () {
+              expect(mockFetchSharedDriveSharingLinks).toHaveBeenCalledTimes(1);
+            });
+
+          case 4:
+            rerender( /*#__PURE__*/React.createElement(AppLike, {
+              client: client,
+              sharingContextValue: sharingContextValue
+            }, /*#__PURE__*/React.createElement(FederatedFolderModal, {
+              document: document,
+              onClose: mockOnClose
+            })));
+            expect(mockFetchSharedDriveSharingLinks).toHaveBeenCalledTimes(1);
+
+          case 6:
+          case "end":
+            return _context11.stop();
+        }
+      }, _callee11);
+    })));
+    it('should not fetch shared drive sharing links when permissions are already loaded', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
+      var _setup10, findByText;
+
+      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+        while (1) switch (_context12.prev = _context12.next) {
+          case 0:
+            mockGetDocumentPermissions.mockReturnValue([{
+              id: 'perm-1',
+              type: 'io.cozy.permissions',
+              attributes: {
+                shortcodes: {
+                  code: 'abc'
+                }
+              }
+            }]);
+            _setup10 = setup({
+              document: _objectSpread(_objectSpread({}, mockDocument), {}, {
+                driveId: 'federated-folder-id'
+              })
+            }), findByText = _setup10.findByText;
+            _context12.next = 4;
+            return findByText('Copy link');
+
+          case 4:
+            expect(mockFetchSharedDriveSharingLinks).not.toHaveBeenCalled();
+
+          case 5:
+          case "end":
+            return _context12.stop();
+        }
+      }, _callee12);
+    })));
+  });
   describe('onSend callback', function () {
     var pendingRecipient = {
       name: 'Alice',
       email: 'alice@example.com'
     };
-    it('should call onClose and skip share when there are no pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8() {
-      var _setup8, findByText, sendButton;
+    it('should call onClose and skip share when there are no pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
+      var _setup11, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee8$(_context8) {
-        while (1) switch (_context8.prev = _context8.next) {
+      return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+        while (1) switch (_context13.prev = _context13.next) {
           case 0:
             // pendingRecipients defaults to [] via beforeEach
-            _setup8 = setup(), findByText = _setup8.findByText;
-            _context8.next = 3;
+            _setup11 = setup(), findByText = _setup11.findByText;
+            _context13.next = 3;
             return findByText('Done');
 
           case 3:
-            sendButton = _context8.sent;
+            sendButton = _context13.sent;
             fireEvent.click(sendButton);
-            _context8.next = 7;
+            _context13.next = 7;
             return waitFor(function () {
               expect(mockOnClose).toHaveBeenCalled();
               expect(mockShare).not.toHaveBeenCalled();
@@ -353,15 +502,15 @@ describe('FederatedFolderModal', function () {
 
           case 7:
           case "end":
-            return _context8.stop();
+            return _context13.stop();
         }
-      }, _callee8);
+      }, _callee13);
     })));
-    it('should call share with correct parameters', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9() {
-      var _setup9, findByText, sendButton;
+    it('should call share with correct parameters', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
+      var _setup12, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee9$(_context9) {
-        while (1) switch (_context9.prev = _context9.next) {
+      return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+        while (1) switch (_context14.prev = _context14.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -371,14 +520,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockResolvedValueOnce({});
-            _setup9 = setup(), findByText = _setup9.findByText;
-            _context9.next = 6;
+            _setup12 = setup(), findByText = _setup12.findByText;
+            _context14.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context9.sent;
+            sendButton = _context14.sent;
             fireEvent.click(sendButton);
-            _context9.next = 10;
+            _context14.next = 10;
             return waitFor(function () {
               expect(mockShare).toHaveBeenCalledWith({
                 description: 'My Test Folder',
@@ -392,15 +541,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context9.stop();
+            return _context14.stop();
         }
-      }, _callee9);
+      }, _callee14);
     })));
-    it('should show success alert and close modal on successful share', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
-      var _setup10, findByText, sendButton;
+    it('should show success alert and close modal on successful share', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
+      var _setup13, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee10$(_context10) {
-        while (1) switch (_context10.prev = _context10.next) {
+      return _regeneratorRuntime.wrap(function _callee15$(_context15) {
+        while (1) switch (_context15.prev = _context15.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -410,14 +559,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockResolvedValueOnce({});
-            _setup10 = setup(), findByText = _setup10.findByText;
-            _context10.next = 6;
+            _setup13 = setup(), findByText = _setup13.findByText;
+            _context15.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context10.sent;
+            sendButton = _context15.sent;
             fireEvent.click(sendButton);
-            _context10.next = 10;
+            _context15.next = 10;
             return waitFor(function () {
               expect(mockShowAlert).toHaveBeenCalledWith({
                 message: 'Folder has been shared',
@@ -429,15 +578,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context10.stop();
+            return _context15.stop();
         }
-      }, _callee10);
+      }, _callee15);
     })));
-    it('should show error alert when share fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
-      var _setup11, findByText, sendButton;
+    it('should show error alert when share fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
+      var _setup14, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee11$(_context11) {
-        while (1) switch (_context11.prev = _context11.next) {
+      return _regeneratorRuntime.wrap(function _callee16$(_context16) {
+        while (1) switch (_context16.prev = _context16.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -447,14 +596,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockRejectedValueOnce(new Error('Share failed'));
-            _setup11 = setup(), findByText = _setup11.findByText;
-            _context11.next = 6;
+            _setup14 = setup(), findByText = _setup14.findByText;
+            _context16.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context11.sent;
+            sendButton = _context16.sent;
             fireEvent.click(sendButton);
-            _context11.next = 10;
+            _context16.next = 10;
             return waitFor(function () {
               expect(mockShowAlert).toHaveBeenCalledWith({
                 message: 'Error while sharing folder',
@@ -466,15 +615,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context11.stop();
+            return _context16.stop();
         }
-      }, _callee11);
+      }, _callee16);
     })));
-    it('should show progress icon on Done button while share is in progress', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
-      var resolveShare, _setup12, findByText, getByRole, queryByRole, sendButton;
+    it('should show progress icon on Done button while share is in progress', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
+      var resolveShare, _setup15, findByText, getByRole, queryByRole, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee13$(_context13) {
-        while (1) switch (_context13.prev = _context13.next) {
+      return _regeneratorRuntime.wrap(function _callee18$(_context18) {
+        while (1) switch (_context18.prev = _context18.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -486,14 +635,14 @@ describe('FederatedFolderModal', function () {
             mockShare.mockReturnValueOnce(new Promise(function (resolve) {
               resolveShare = resolve;
             }));
-            _setup12 = setup(), findByText = _setup12.findByText, getByRole = _setup12.getByRole, queryByRole = _setup12.queryByRole;
-            _context13.next = 6;
+            _setup15 = setup(), findByText = _setup15.findByText, getByRole = _setup15.getByRole, queryByRole = _setup15.queryByRole;
+            _context18.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context13.sent;
+            sendButton = _context18.sent;
             fireEvent.click(sendButton);
-            _context13.next = 10;
+            _context18.next = 10;
             return waitFor(function () {
               expect(getByRole('button', {
                 name: 'Done'
@@ -502,56 +651,56 @@ describe('FederatedFolderModal', function () {
             });
 
           case 10:
-            _context13.next = 12;
-            return act( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
-              return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-                while (1) switch (_context12.prev = _context12.next) {
+            _context18.next = 12;
+            return act( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
+              return _regeneratorRuntime.wrap(function _callee17$(_context17) {
+                while (1) switch (_context17.prev = _context17.next) {
                   case 0:
                     resolveShare({});
 
                   case 1:
                   case "end":
-                    return _context12.stop();
+                    return _context17.stop();
                 }
-              }, _callee12);
+              }, _callee17);
             })));
 
           case 12:
           case "end":
-            return _context13.stop();
+            return _context18.stop();
         }
-      }, _callee13);
+      }, _callee18);
     })));
   });
   describe('onClose callback', function () {
-    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
-      var _setup13, findByRole, closeButton;
+    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
+      var _setup16, findByRole, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee14$(_context14) {
-        while (1) switch (_context14.prev = _context14.next) {
+      return _regeneratorRuntime.wrap(function _callee19$(_context19) {
+        while (1) switch (_context19.prev = _context19.next) {
           case 0:
-            _setup13 = setup(), findByRole = _setup13.findByRole;
-            _context14.next = 3;
+            _setup16 = setup(), findByRole = _setup16.findByRole;
+            _context19.next = 3;
             return findByRole('button', {
               name: /close/i
             });
 
           case 3:
-            closeButton = _context14.sent;
+            closeButton = _context19.sent;
             fireEvent.click(closeButton);
             expect(mockOnClose).toHaveBeenCalled();
 
           case 6:
           case "end":
-            return _context14.stop();
+            return _context19.stop();
         }
-      }, _callee14);
+      }, _callee19);
     })));
-    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
-      var _setup14, getByRole, findByRole, getByText, closeButton;
+    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
+      var _setup17, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee15$(_context15) {
-        while (1) switch (_context15.prev = _context15.next) {
+      return _regeneratorRuntime.wrap(function _callee20$(_context20) {
+        while (1) switch (_context20.prev = _context20.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -562,14 +711,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup14 = setup(), getByRole = _setup14.getByRole, findByRole = _setup14.findByRole, getByText = _setup14.getByText;
-            _context15.next = 4;
+            _setup17 = setup(), getByRole = _setup17.getByRole, findByRole = _setup17.findByRole, getByText = _setup17.getByText;
+            _context20.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context15.sent;
+            closeButton = _context20.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             expect(mockOnClose).not.toHaveBeenCalled();
@@ -580,15 +729,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context15.stop();
+            return _context20.stop();
         }
-      }, _callee15);
+      }, _callee20);
     })));
-    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
-      var _setup15, getByRole, findByRole, getByText, closeButton;
+    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
+      var _setup18, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee16$(_context16) {
-        while (1) switch (_context16.prev = _context16.next) {
+      return _regeneratorRuntime.wrap(function _callee21$(_context21) {
+        while (1) switch (_context21.prev = _context21.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -599,14 +748,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup15 = setup(), getByRole = _setup15.getByRole, findByRole = _setup15.findByRole, getByText = _setup15.getByText;
-            _context16.next = 4;
+            _setup18 = setup(), getByRole = _setup18.getByRole, findByRole = _setup18.findByRole, getByText = _setup18.getByText;
+            _context21.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context16.sent;
+            closeButton = _context21.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             fireEvent.click(getByRole('button', {
@@ -616,9 +765,9 @@ describe('FederatedFolderModal', function () {
 
           case 9:
           case "end":
-            return _context16.stop();
+            return _context21.stop();
         }
-      }, _callee16);
+      }, _callee21);
     })));
   });
 });

package/dist/components/Recipient/OwnerRecipientDefault.js

@@ -1,7 +1,13 @@
 import React from 'react';
 import { useClient, useQuery, hasQueryBeenLoaded } from 'cozy-client';
 import MemberRecipient from './MemberRecipient';
-import { buildInstanceSettingsQuery } from '../../queries/queries';
+import { buildInstanceSettingsQuery } from '../../queries/queries'; // When the owner is not part of a sharing's members (eg. the folder only has
+// a link), there is no per-sharing avatar to reference. The instance's public
+// avatar still holds the owner's picture. fallback=initials makes the stack
+// serve a generated initials avatar (instead of the app icon) when no picture
+// has been uploaded, matching how avatars are rendered everywhere else.
+
+var OWNER_PUBLIC_AVATAR_PATH = '/public/avatar?fallback=initials';
 
 var OwnerRecipientDefault = function OwnerRecipientDefault() {
   var _instanceSettingsResu;
@@ -13,6 +19,7 @@ var OwnerRecipientDefault = function OwnerRecipientDefault() {
     isOwner: true,
     status: "owner",
     instance: client.options.uri,
+    avatarPath: OWNER_PUBLIC_AVATAR_PATH,
     public_name: instanceSettingsResult === null || instanceSettingsResult === void 0 || (_instanceSettingsResu = instanceSettingsResult.data) === null || _instanceSettingsResu === void 0 || (_instanceSettingsResu = _instanceSettingsResu.attributes) === null || _instanceSettingsResu === void 0 ? void 0 : _instanceSettingsResu.public_name
   });
 };