UIREC-292: fix update item status on unreceive piece

src/TitleDetails/AddPieceModal/AddPieceModal.js

@@ -32,10 +32,14 @@ import {
   PIECE_STATUS,
   handleKeyCommand,
   useModalToggle,
+  useShowCallout,
 } from '@folio/stripes-acq-components';
 
 import { HOLDINGS_API } from '../../common/constants';
-import { getClaimingIntervalFromDate } from '../../common/utils';
+import {
+  getClaimingIntervalFromDate,
+  unreceivePieces,
+} from '../../common/utils';
 import {
   PIECE_MODAL_ACCORDION,
   PIECE_MODAL_ACCORDION_LABELS,
@@ -51,23 +55,24 @@ import { PieceFields } from './PieceFields';
 import { ReceivingStatusChangeLog } from './ReceivingStatusChangeLog';
 
 const AddPieceModal = ({
+  canDeletePiece,
   close,
   createInventoryValues,
   deletePiece,
-  canDeletePiece,
   form,
-  initialValues,
+  getHoldingsItemsAndPieces,
   handleSubmit,
   hasValidationErrors,
-  pristine,
+  initialValues,
   instanceId,
   locationIds,
   locations,
+  mutator,
   onCheckIn,
   pieceFormatOptions,
-  values: formValues,
   poLine,
-  getHoldingsItemsAndPieces,
+  pristine,
+  values: formValues,
 }) => {
   const {
     batch,
@@ -107,6 +112,7 @@ const AddPieceModal = ({
   const intl = useIntl();
   const accordionStatusRef = useRef();
   const modalLabel = intl.formatMessage({ id: labelId });
+  const showCallout = useShowCallout();
 
   const initialHoldingId = useMemo(() => getState().initialValues?.holdingId, [getState]);
 
@@ -179,6 +185,25 @@ const AddPieceModal = ({
     onSave();
   }, [change, onSave]);
 
+  const onUnreceivePiece = useCallback(() => {
+    const currentPiece = {
+      ...formValues,
+      checked: true,
+    };
+
+    return unreceivePieces([currentPiece], mutator.unreceive)
+      .then(() => {
+        onStatusChange(PIECE_STATUS.expected);
+      })
+      .catch(() => {
+        showCallout({
+          type: 'error',
+          messageId: 'ui-receiving.title.actions.unreceive.error',
+        });
+      });
+  },
+  [formValues, mutator, onStatusChange, showCallout]);
+
   const onClaimDelay = useCallback(({ claimingDate }) => {
     change('claimingInterval', getClaimingIntervalFromDate(claimingDate));
     onStatusChange(PIECE_STATUS.claimDelayed);
@@ -219,6 +244,7 @@ const AddPieceModal = ({
       onClaimSend={toggleClaimSendModal}
       onDelete={toggleDeleteConfirmation}
       onReceive={onReceive}
+      onUnreceivePiece={onUnreceivePiece}
       onSave={onSave}
       onStatusChange={onStatusChange}
       status={receivingStatus}
@@ -384,6 +410,7 @@ AddPieceModal.propTypes = {
   getHoldingsItemsAndPieces: PropTypes.func.isRequired,
   initialValues: PropTypes.object.isRequired,
   pristine: PropTypes.bool.isRequired,
+  mutator: PropTypes.object.isRequired,
 };
 
 AddPieceModal.defaultProps = {

src/TitleDetails/AddPieceModal/AddPieceModal.test.js

@@ -22,6 +22,10 @@ jest.mock('@folio/stripes-acq-components', () => {
 });
 jest.mock('../../common/components/LineLocationsView/LineLocationsView',
   () => jest.fn().mockReturnValue('LineLocationsView'));
+jest.mock('../../common/utils', () => ({
+  ...jest.requireActual('../../common/utils'),
+  unreceivePieces: jest.fn(() => Promise.resolve()),
+}));
 jest.mock('../hooks', () => ({
   ...jest.requireActual('../hooks'),
   usePieceStatusChangeLog: jest.fn(),
@@ -262,6 +266,42 @@ describe('AddPieceModal', () => {
     expect(defaultProps.onSubmit).toHaveBeenCalled();
   });
 
+  it('should unreceive piece', async () => {
+    const onChange = jest.fn();
+
+    renderAddPieceModal({
+      form: {
+        ...defaultProps.form,
+        change: onChange,
+      },
+      hasValidationErrors: false,
+      initialValues: {
+        'id': 'cd3fd1e7-c195-4d8e-af75-525e1039d643',
+        'format': 'Other',
+        'poLineId': 'a92ae36c-e093-4daf-b234-b4c6dc33a258',
+        'titleId': '03329fea-1b5d-43ab-b955-20bcd9ba530d',
+        'holdingId': '60c67dc5-b646-425e-bf08-a8bf2d0681fb',
+        'isCreateAnother': false,
+        'isCreateItem': false,
+        receivingStatus: PIECE_STATUS.received,
+        receivedDate: new Date().toISOString(),
+      },
+      mutator: {
+        unreceive: {
+          POST: jest.fn(),
+        },
+      },
+    });
+
+    await user.click(screen.getByTestId('dropdown-trigger-button'));
+    const unReceiveButton = await screen.findByTestId('unReceive-piece-button');
+
+    expect(unReceiveButton).toBeInTheDocument();
+    await user.click(unReceiveButton);
+
+    expect(defaultProps.onSubmit).toHaveBeenCalled();
+  });
+
   describe('Actions', () => {
     const initialValues = {
       format: PIECE_FORMAT.other,

src/TitleDetails/AddPieceModal/AddPieceModalContainer.js

@@ -22,6 +22,7 @@ const AddPieceModalContainer = ({
   onSubmit,
   poLine,
   getHoldingsItemsAndPieces,
+  mutator,
 }) => {
   const createInventoryValues = useMemo(
     () => ({
@@ -72,6 +73,7 @@ const AddPieceModalContainer = ({
       pieceFormatOptions={pieceFormatOptions}
       poLine={poLine}
       getHoldingsItemsAndPieces={getHoldingsItemsAndPieces}
+      mutator={mutator}
     />
   );
 };
@@ -88,6 +90,7 @@ AddPieceModalContainer.propTypes = {
   onSubmit: PropTypes.func.isRequired,
   poLine: PropTypes.object.isRequired,
   getHoldingsItemsAndPieces: PropTypes.func.isRequired,
+  mutator: PropTypes.object.isRequired,
 };
 
 export default AddPieceModalContainer;

src/TitleDetails/AddPieceModal/ModalActionButtons/ModalActionButtons.js

@@ -25,6 +25,7 @@ export const ModalActionButtons = ({
   onReceive,
   onSave,
   onStatusChange,
+  onUnreceivePiece,
   status,
 }) => {
   const actionMenu = getPieceActionMenu({
@@ -37,6 +38,7 @@ export const ModalActionButtons = ({
     onDelete,
     onReceive,
     onStatusChange,
+    onUnreceivePiece,
     status,
   });
   const saveButtonLabelId = 'ui-receiving.piece.actions.saveAndClose';
@@ -95,6 +97,7 @@ ModalActionButtons.propTypes = {
   onReceive: PropTypes.func.isRequired,
   onSave: PropTypes.func.isRequired,
   onStatusChange: PropTypes.func.isRequired,
+  onUnreceivePiece: PropTypes.func.isRequired,
   status: PropTypes.string,
 };
 

src/TitleDetails/AddPieceModal/ModalActionButtons/constants.js

@@ -53,6 +53,7 @@ export const PIECE_ACTIONS = ({
   onStatusChange,
   onDelete,
   onReceive,
+  onUnreceivePiece,
 }) => ({
   delayClaim: (
     <Button
@@ -131,7 +132,7 @@ export const PIECE_ACTIONS = ({
       disabled={actionsDisabled[PIECE_ACTION_NAMES.unReceive]}
       buttonStyle="dropdownItem"
       data-testid="unReceive-piece-button"
-      onClick={() => onStatusChange(PIECE_STATUS.expected)}
+      onClick={onUnreceivePiece}
     >
       <Icon icon="cancel">
         <FormattedMessage id="ui-receiving.piece.action.button.unReceive" />

src/TitleDetails/AddPieceModal/ModalActionButtons/utils.test.js

@@ -59,14 +59,14 @@ describe('getPieceActionMenus', () => {
   });
 
   describe('unReceive action', () => {
-    it('should `onStatusChange` be called with `Expected` status value', () => {
-      const onStatusChange = jest.fn();
-      const result = getPieceActionMenu({ status: received, onStatusChange });
+    it('should `onUnreceivePiece` be called with `Expected` status value', () => {
+      const onUnreceivePiece = jest.fn();
+      const result = getPieceActionMenu({ status: received, onUnreceivePiece });
       const receiveButton = result.find(i => i.props['data-testid'] === 'unReceive-piece-button');
 
       receiveButton.props.onClick();
 
-      expect(onStatusChange).toHaveBeenCalledWith(PIECE_STATUS.expected);
+      expect(onUnreceivePiece).toHaveBeenCalledWith();
     });
   });
 

src/TitleDetails/TitleDetails.js

@@ -105,6 +105,7 @@ const TitleDetails = ({
   vendorsMap,
   getHoldingsItemsAndPieces,
   getPieceValues,
+  mutator,
 }) => {
   const intl = useIntl();
   const stripes = useStripes();
@@ -601,6 +602,7 @@ const TitleDetails = ({
             onCheckIn={onQuickReceive}
             onSubmit={onSave}
             poLine={poLine}
+            mutator={mutator}
             getHoldingsItemsAndPieces={getHoldingsItemsAndPieces}
           />
         )}
@@ -638,6 +640,7 @@ TitleDetails.propTypes = {
   vendorsMap: PropTypes.object.isRequired,
   getHoldingsItemsAndPieces: PropTypes.func.isRequired,
   getPieceValues: PropTypes.func.isRequired,
+  mutator: PropTypes.object.isRequired,
 };
 
 export default withRouter(TitleDetails);

src/TitleDetails/TitleDetailsContainer.js

@@ -21,6 +21,7 @@
 } from '@folio/stripes-acq-components';
 
 import {
+  receivingResource,
   titleResource,
 } from '../common/resources';
 import {
@@ -106,7 +107,7 @@
         items: itemsInHolding,
       }))
       .catch(() => ({}));
   }, []);
 
   useEffect(
     () => {
@@ -293,6 +294,7 @@
       vendorsMap={vendorsMap}
       getHoldingsItemsAndPieces={getHoldingsItemsAndPieces}
       getPieceValues={getPieceById(mutator.orderPieces)}
+      mutator={mutator}
     />
   );
 };
@@ -320,6 +322,7 @@
     ...locationsManifest,
     fetch: false,
   },
+  unreceive: receivingResource,
   vendors: {
     ...organizationsManifest,
     fetch: false,