cozy-sharing 33.1.2 → 33.2.0

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [33.2.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@33.1.2...cozy-sharing@33.2.0) (2026-06-09)
+
+### Features
+
+- **cozy-sharing:** Show progress icon on done button during share ([885a715](https://github.com/cozy/cozy-libs/commit/885a71568a96354e3d962296a205efb05881a5d1))
+
 ## [33.1.2](https://github.com/cozy/cozy-libs/compare/cozy-sharing@33.1.1...cozy-sharing@33.1.2) (2026-06-08)
 
 ### Bug Fixes

package/dist/components/FederatedFolder/FederatedFolderModal.js

@@ -307,7 +307,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
     }), canShareByEmail && /*#__PURE__*/React.createElement(Button, {
       variant: "primary",
       label: t('FederatedFolder.share'),
-      disabled: isSending,
+      busy: isSending,
       onClick: onSend
     }))
   });

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

@@ -7,7 +7,7 @@ function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (O
 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 { fireEvent, render, waitFor } from '@testing-library/react';
+import { act, fireEvent, render, waitFor } from '@testing-library/react';
 import React from 'react';
 import { createMockClient } from 'cozy-client';
 import { FederatedFolderModal } from './FederatedFolderModal';
@@ -384,36 +384,88 @@ describe('FederatedFolderModal', function () {
         }
       }, _callee9);
     })));
+    it('should show progress icon on Done button while share is in progress', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
+      var resolveShare, _setup10, findByText, getByRole, queryByRole, sendButton;
+
+      return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+        while (1) switch (_context11.prev = _context11.next) {
+          case 0:
+            usePendingRecipients.mockReturnValue({
+              pendingRecipients: [pendingRecipient],
+              setPendingRecipients: jest.fn(),
+              selectedOption: 'readWrite',
+              setSelectedOption: jest.fn()
+            });
+            getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
+            mockShare.mockReturnValueOnce(new Promise(function (resolve) {
+              resolveShare = resolve;
+            }));
+            _setup10 = setup(), findByText = _setup10.findByText, getByRole = _setup10.getByRole, queryByRole = _setup10.queryByRole;
+            _context11.next = 6;
+            return findByText('Done');
+
+          case 6:
+            sendButton = _context11.sent;
+            fireEvent.click(sendButton);
+            _context11.next = 10;
+            return waitFor(function () {
+              expect(getByRole('button', {
+                name: 'Done'
+              })).toBeDisabled();
+              expect(queryByRole('progressbar')).not.toBeNull();
+            });
+
+          case 10:
+            _context11.next = 12;
+            return act( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
+              return _regeneratorRuntime.wrap(function _callee10$(_context10) {
+                while (1) switch (_context10.prev = _context10.next) {
+                  case 0:
+                    resolveShare({});
+
+                  case 1:
+                  case "end":
+                    return _context10.stop();
+                }
+              }, _callee10);
+            })));
+
+          case 12:
+          case "end":
+            return _context11.stop();
+        }
+      }, _callee11);
+    })));
   });
   describe('onClose callback', function () {
-    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
-      var _setup10, findByRole, closeButton;
+    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
+      var _setup11, findByRole, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee10$(_context10) {
-        while (1) switch (_context10.prev = _context10.next) {
+      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+        while (1) switch (_context12.prev = _context12.next) {
           case 0:
-            _setup10 = setup(), findByRole = _setup10.findByRole;
-            _context10.next = 3;
+            _setup11 = setup(), findByRole = _setup11.findByRole;
+            _context12.next = 3;
             return findByRole('button', {
               name: /close/i
             });
 
           case 3:
-            closeButton = _context10.sent;
+            closeButton = _context12.sent;
             fireEvent.click(closeButton);
             expect(mockOnClose).toHaveBeenCalled();
 
           case 6:
           case "end":
-            return _context10.stop();
+            return _context12.stop();
         }
-      }, _callee10);
+      }, _callee12);
     })));
-    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
-      var _setup11, getByRole, findByRole, getByText, closeButton;
+    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
+      var _setup12, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee11$(_context11) {
-        while (1) switch (_context11.prev = _context11.next) {
+      return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+        while (1) switch (_context13.prev = _context13.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -424,14 +476,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup11 = setup(), getByRole = _setup11.getByRole, findByRole = _setup11.findByRole, getByText = _setup11.getByText;
-            _context11.next = 4;
+            _setup12 = setup(), getByRole = _setup12.getByRole, findByRole = _setup12.findByRole, getByText = _setup12.getByText;
+            _context13.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context11.sent;
+            closeButton = _context13.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             expect(mockOnClose).not.toHaveBeenCalled();
@@ -442,15 +494,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context11.stop();
+            return _context13.stop();
         }
-      }, _callee11);
+      }, _callee13);
     })));
-    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
-      var _setup12, getByRole, findByRole, getByText, closeButton;
+    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
+      var _setup13, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-        while (1) switch (_context12.prev = _context12.next) {
+      return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+        while (1) switch (_context14.prev = _context14.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -461,14 +513,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup12 = setup(), getByRole = _setup12.getByRole, findByRole = _setup12.findByRole, getByText = _setup12.getByText;
-            _context12.next = 4;
+            _setup13 = setup(), getByRole = _setup13.getByRole, findByRole = _setup13.findByRole, getByText = _setup13.getByText;
+            _context14.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context12.sent;
+            closeButton = _context14.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             fireEvent.click(getByRole('button', {
@@ -478,9 +530,9 @@ describe('FederatedFolderModal', function () {
 
           case 9:
           case "end":
-            return _context12.stop();
+            return _context14.stop();
         }
-      }, _callee12);
+      }, _callee14);
     })));
   });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "33.1.2",
+  "version": "33.2.0",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -83,5 +83,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "4f63c28ae3cd8cac22487e6640006de4f5c31cfc"
+  "gitHead": "aeaf0cb5bb862ce8908d78862ca79e8e749982f2"
 }

package/src/components/FederatedFolder/FederatedFolderModal.jsx

@@ -269,7 +269,7 @@ const FederatedFolderModalContent = ({
             <Button
               variant="primary"
               label={t('FederatedFolder.share')}
-              disabled={isSending}
+              busy={isSending}
               onClick={onSend}
             />
           )}

package/src/components/FederatedFolder/FederatedFolderModal.spec.jsx

@@ -1,4 +1,4 @@
-import { fireEvent, render, waitFor } from '@testing-library/react'
+import { act, fireEvent, render, waitFor } from '@testing-library/react'
 import React from 'react'
 
 import { createMockClient } from 'cozy-client'
@@ -257,6 +257,36 @@ describe('FederatedFolderModal', () => {
         expect(mockOnClose).not.toHaveBeenCalled()
       })
     })
+
+    it('should show progress icon on Done button while share is in progress', async () => {
+      usePendingRecipients.mockReturnValue({
+        pendingRecipients: [pendingRecipient],
+        setPendingRecipients: jest.fn(),
+        selectedOption: 'readWrite',
+        setSelectedOption: jest.fn()
+      })
+      getOrCreateFromArray.mockResolvedValue([pendingRecipient])
+      let resolveShare
+      mockShare.mockReturnValueOnce(
+        new Promise(resolve => {
+          resolveShare = resolve
+        })
+      )
+      const { findByText, getByRole, queryByRole } = setup()
+
+      const sendButton = await findByText('Done')
+
+      fireEvent.click(sendButton)
+
+      await waitFor(() => {
+        expect(getByRole('button', { name: 'Done' })).toBeDisabled()
+        expect(queryByRole('progressbar')).not.toBeNull()
+      })
+
+      await act(async () => {
+        resolveShare({})
+      })
+    })
   })
 
   describe('onClose callback', () => {