Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions src/libs/SearchUIUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1040,6 +1040,7 @@ function getSuggestedSearchesVisibility(
const isPaidPolicy = isPaidGroupPolicy(policy);
const isPayer = isPolicyPayer(policy, currentUserEmail);
const isAdmin = policy.role === CONST.POLICY.ROLE.ADMIN;
const isAuditor = policy.role === CONST.POLICY.ROLE.AUDITOR;
const isExporter = policy.exporter === currentUserEmail;

const isSubmittedTo =
Expand All @@ -1060,11 +1061,10 @@ function getSuggestedSearchesVisibility(
const isPolicyEligibleForApproveSuggestion = isPaidPolicy && isEligibleForApproveSuggestion(policy.approvalMode, isUserApprover, isSubmittedTo);
const isEligibleForExportSuggestion = isExporter && !hasExportError;
const isEligibleForStatementsSuggestion = isPaidPolicy && !!policy.areCompanyCardsEnabled && hasCardFeed;
const isEligibleForUnapprovedCashSuggestion = isPaidPolicy && isAdmin && isApprovalEnabled && isPaymentEnabled;
const isEligibleForUnapprovedCardSuggestion = isPaidPolicy && isAdmin && isApprovalEnabled && (hasCardFeed || !!defaultExpensifyCard);
const isEligibleForExpensifyCardSuggestion = isPaidPolicy && isAdmin && isECardEnabled;
const isEligibleForReimbursementsSuggestion = isPaidPolicy && isAdmin && isPaymentEnabled && hasVBBA && hasReimburser;
const isAuditor = policy.role === CONST.POLICY.ROLE.AUDITOR;
const isEligibleForUnapprovedCashSuggestion = isPaidPolicy && (isAdmin || isAuditor) && isApprovalEnabled && isPaymentEnabled;
const isEligibleForUnapprovedCardSuggestion = isPaidPolicy && (isAdmin || isAuditor) && isApprovalEnabled && (hasCardFeed || !!defaultExpensifyCard);
const isEligibleForExpensifyCardSuggestion = isPaidPolicy && (isAdmin || isAuditor) && isECardEnabled;
const isEligibleForReimbursementsSuggestion = isPaidPolicy && (isAdmin || isAuditor) && isPaymentEnabled && hasVBBA && hasReimburser;
const memberCount = Object.keys(policy.employeeList ?? {}).length;
const isEligibleForTopSpendersSuggestion = isPaidPolicy && (isAdmin || isAuditor || isUserApprover) && memberCount >= 2;
const isEligibleForTopCategoriesSuggestion = isPaidPolicy && policy.areCategoriesEnabled === true;
Expand Down
90 changes: 90 additions & 0 deletions tests/unit/Search/SearchUIUtilsTest.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7769,6 +7769,96 @@ describe('SearchUIUtils', () => {
const response = SearchUIUtils.getSuggestedSearchesVisibility(regularEmail, {}, policies, undefined);
expect(response.visibility.topSpenders).toBe(false);
});

test('Should show Unapproved Cash for Auditor role in paid policy with approvals and payments enabled', () => {
const auditorEmail = 'auditor@policy.com';
const policyKey = `policy_${policyID}`;

const policies: OnyxCollection<OnyxTypes.Policy> = {
[policyKey]: {
id: policyID,
type: CONST.POLICY.TYPE.TEAM,
role: CONST.POLICY.ROLE.AUDITOR,
approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED,
} as OnyxTypes.Policy,
};

const response = SearchUIUtils.getSuggestedSearchesVisibility(auditorEmail, {}, policies, undefined);
expect(response.visibility.unapprovedCash).toBe(true);
});

test('Should show Unapproved Card for Auditor role in paid policy with approvals enabled and a default Expensify card', () => {
const auditorEmail = 'auditor@policy.com';
const policyKey = `policy_${policyID}`;

const policies: OnyxCollection<OnyxTypes.Policy> = {
[policyKey]: {
id: policyID,
type: CONST.POLICY.TYPE.TEAM,
role: CONST.POLICY.ROLE.AUDITOR,
approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED,
} as OnyxTypes.Policy,
};

const response = SearchUIUtils.getSuggestedSearchesVisibility(auditorEmail, {}, policies, {} as CardFeedForDisplay);
expect(response.visibility.unapprovedCard).toBe(true);
});

test('Should show Reconciliation for Auditor role in paid policy with Expensify Cards enabled', () => {
const auditorEmail = 'auditor@policy.com';
const policyKey = `policy_${policyID}`;

const policies: OnyxCollection<OnyxTypes.Policy> = {
[policyKey]: {
id: policyID,
type: CONST.POLICY.TYPE.TEAM,
role: CONST.POLICY.ROLE.AUDITOR,
areExpensifyCardsEnabled: true,
} as OnyxTypes.Policy,
};

const response = SearchUIUtils.getSuggestedSearchesVisibility(auditorEmail, {}, policies, undefined);
expect(response.visibility.reconciliation).toBe(true);
});

test('Should hide Unapproved Cash, Unapproved Card, and Reconciliation for User role even when prerequisites are met', () => {
const userEmail = 'user@policy.com';
const policyKey = `policy_${policyID}`;

const policies: OnyxCollection<OnyxTypes.Policy> = {
[policyKey]: {
id: policyID,
type: CONST.POLICY.TYPE.TEAM,
role: CONST.POLICY.ROLE.USER,
approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED,
areExpensifyCardsEnabled: true,
} as OnyxTypes.Policy,
};

const response = SearchUIUtils.getSuggestedSearchesVisibility(userEmail, {}, policies, {} as CardFeedForDisplay);
expect(response.visibility.unapprovedCash).toBe(false);
expect(response.visibility.unapprovedCard).toBe(false);
expect(response.visibility.reconciliation).toBe(false);
});

test('Should still show Unapproved Cash, Unapproved Card, and Reconciliation for Admin role when prerequisites are met (regression)', () => {
const policyKey = `policy_${policyID}`;

const policies: OnyxCollection<OnyxTypes.Policy> = {
[policyKey]: {
id: policyID,
type: CONST.POLICY.TYPE.TEAM,
role: CONST.POLICY.ROLE.ADMIN,
approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED,
areExpensifyCardsEnabled: true,
} as OnyxTypes.Policy,
};

const response = SearchUIUtils.getSuggestedSearchesVisibility(adminEmail, {}, policies, {} as CardFeedForDisplay);
expect(response.visibility.unapprovedCash).toBe(true);
expect(response.visibility.unapprovedCard).toBe(true);
expect(response.visibility.reconciliation).toBe(true);
});
});

describe('Test getSuggestedSearches sort defaults', () => {
Expand Down
Loading