From 0eb047d5d57516110f7f5eb76ebdec6f319b1504 Mon Sep 17 00:00:00 2001
From: Priyanka <priyanka_terala@epam.com>
Date: Fri, 24 Nov 2023 13:12:16 +0530
Subject: [PATCH] UICHKIN-398-poppy-dcb - Hide fee/fine action menu items when
 requester is virtual user.

---
 CHANGELOG.md                                  |  4 ++++
 src/CheckIn.js                                |  5 ++++-
 .../FeeFineDetailsButton.js                   |  8 +++++++-
 .../FeeFineDetailsButton.test.js              | 19 ++++++++++++++++++
 src/consts.js                                 |  4 ++++
 src/util.js                                   |  7 ++++++-
 src/util.test.js                              | 20 +++++++++++++++++++
 7 files changed, 64 insertions(+), 3 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index fb3af085..435cfe36 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Change history for ui-checkin
 
+## [9.0.2] IN PROGRESS
+
+* Hide fee/fine action menu items when requester is virtual user. Refs UICHKIN-398.
+
 ## [9.0.1] (https://github.com/folio-org/ui-checkin/tree/v9.0.1) (2023-10-23)
 [Full Changelog](https://github.com/folio-org/ui-checkin/compare/v9.0.0...v9.0.1)
 * Fix circulation timeout issue. Refs UICHKIN-392.
diff --git a/src/CheckIn.js b/src/CheckIn.js
index 37999ab0..d84baad4 100644
--- a/src/CheckIn.js
+++ b/src/CheckIn.js
@@ -39,6 +39,7 @@ import CheckInFooter from './components/CheckInFooter';
 import {
   convertToSlipData,
   getCheckinSettings,
+  isDcbUser,
 } from './util';
 
 import styles from './checkin.css';
@@ -308,6 +309,7 @@ class CheckIn extends React.Component {
     const isCheckInNote = element => element.noteType === 'Check in';
     const checkinNotePresent = get(loan.item, ['circulationNotes'], []).some(isCheckInNote);
     const loanOpenRequest = loan?.staffSlipContext?.request ?? {};
+    const isVirtualUser = isDcbUser(loan?.borrower);
 
     const trigger = ({ getTriggerProps, triggerRef }) => (
       <IconButton
@@ -393,12 +395,13 @@ class CheckIn extends React.Component {
           }
           <IfPermission perm="accounts.collection.get">
             <FeeFineDetailsButton
+              isVirtualUser={isVirtualUser}
               userId={loan.userId}
               itemId={loan.itemId}
               mutator={this.props.mutator}
             />
           </IfPermission>
-          {loan.userId &&
+          {loan.userId && !isVirtualUser &&
             <Button
               role="menuitem"
               buttonStyle="dropdownItem"
diff --git a/src/components/FeeFineDetailsButton/FeeFineDetailsButton.js b/src/components/FeeFineDetailsButton/FeeFineDetailsButton.js
index fbeb352e..919a0d0f 100644
--- a/src/components/FeeFineDetailsButton/FeeFineDetailsButton.js
+++ b/src/components/FeeFineDetailsButton/FeeFineDetailsButton.js
@@ -15,6 +15,7 @@ const FEE_FINES_STATUSES = {
 
 class FeeFineDetailsButton extends React.Component {
   static propTypes = {
+    isVirtualUser: PropTypes.bool,
     userId: PropTypes.string,
     itemId: PropTypes.string,
     mutator: PropTypes.shape({
@@ -116,12 +117,13 @@ class FeeFineDetailsButton extends React.Component {
     const {
       userId,
       itemId,
+      isVirtualUser,
     } = this.props;
     const {
       feeFinesCount,
     } = this.state;
 
-    return !userId || !itemId || !feeFinesCount;
+    return isVirtualUser || !userId || !itemId || !feeFinesCount;
   };
 
   render() {
@@ -148,4 +150,8 @@ class FeeFineDetailsButton extends React.Component {
   }
 }
 
+FeeFineDetailsButton.defaultProps = {
+  isVirtualUser: false,
+};
+
 export default FeeFineDetailsButton;
diff --git a/src/components/FeeFineDetailsButton/FeeFineDetailsButton.test.js b/src/components/FeeFineDetailsButton/FeeFineDetailsButton.test.js
index 20803d14..73e37d29 100644
--- a/src/components/FeeFineDetailsButton/FeeFineDetailsButton.test.js
+++ b/src/components/FeeFineDetailsButton/FeeFineDetailsButton.test.js
@@ -9,6 +9,7 @@ import {
 import { account as accountFixture } from '../../../test/jest/fixtures/account';
 import { loan as loanFixture } from '../../../test/jest/fixtures/loan';
 import FeeFineDetailsButton from './FeeFineDetailsButton';
+import { DCB_USER } from '../../consts';
 
 const mockedQueryUpdate = jest.fn();
 const labelIds = {
@@ -193,4 +194,22 @@ describe('FeeFineDetailsButton', () => {
       });
     });
   });
+
+  describe('when borrower is virtual user', () => {
+    it('should not render FeeFineDetails button', () => {
+      const alteredLoanFixture = {
+        ...loanFixture,
+        loan: {
+          ...loanFixture.loan,
+          borrower: {
+            ...loanFixture.loan.borrower,
+            lastName: DCB_USER.lastName,
+          }
+        }
+      };
+      renderFeeFineDetailsButton(alteredLoanFixture, accountFixture);
+
+      expect(screen.queryByText(labelIds.feeFineDetails)).toBeNull();
+    });
+  });
 });
diff --git a/src/consts.js b/src/consts.js
index adcd0e5d..1938bbdf 100644
--- a/src/consts.js
+++ b/src/consts.js
@@ -56,3 +56,7 @@ export const cancelFeeClaimReturned = {
 };
 
 export const MAX_RECORDS = '1000';
+
+export const DCB_USER = {
+  lastName: 'DcbSystem',
+};
diff --git a/src/util.js b/src/util.js
index c9f2737e..924e5616 100644
--- a/src/util.js
+++ b/src/util.js
@@ -4,7 +4,10 @@ import {
   includes,
 } from 'lodash';
 
-import { statuses } from './consts';
+import {
+  DCB_USER,
+  statuses,
+} from './consts';
 
 export const escapeValue = (val) => {
   if (typeof val === 'string' && val.startsWith('<Barcode>') && val.endsWith('</Barcode>')) {
@@ -164,4 +167,6 @@ export function shouldConfirmStatusModalBeShown(item) {
   ], item?.status?.name);
 }
 
+export const isDcbUser = (user) => user?.lastName === DCB_USER.lastName;
+
 export default {};
diff --git a/src/util.test.js b/src/util.test.js
index 990b8ea9..f97f3a10 100644
--- a/src/util.test.js
+++ b/src/util.test.js
@@ -12,9 +12,11 @@ import {
   escapeValue,
   getCheckinSettings,
   shouldConfirmStatusModalBeShown,
+  isDcbUser,
 } from './util';
 
 import {
+  DCB_USER,
   statuses,
 } from './consts';
 
@@ -318,3 +320,21 @@ describe('shouldConfirmStatusModalBeShown', () => {
     });
   });
 });
+
+describe('isDcbUser', () => {
+  it('should return true when user has lastName as "DcbSystem"', () => {
+    const user = {
+      lastName: DCB_USER.lastName,
+    };
+
+    expect(isDcbUser(user)).toBeTruthy();
+  });
+
+  it('should return false when user does not have lastName as "DcbSystem"', () => {
+    const user = {
+      lastName: 'test',
+    };
+
+    expect(isDcbUser(user)).toBeFalsy();
+  });
+});