Skip to content

Commit f6a3981

Browse files
committed
fix(MUSD-716): drop redundant earnings divider when bonus card is rendered
1 parent 414af85 commit f6a3981

2 files changed

Lines changed: 17 additions & 6 deletions

File tree

app/components/UI/Money/Views/MoneyHomeView/MoneyHomeView.test.tsx

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -537,20 +537,29 @@ describe('MoneyHomeView', () => {
537537
).not.toBeOnTheScreen();
538538
});
539539

540-
it('renders the structural divider after MoneyEarnings when the bonus card is visible', () => {
540+
it('omits the parent divider when the bonus card is visible to avoid stacking two dividers', () => {
541+
// Regression guard for the double-divider bug: AssetOverviewClaimBonus
542+
// already renders its own internal top divider for the asset-overview
543+
// page consumer, so MoneyHomeView must NOT render the parent divider
544+
// when the bonus card is visible — otherwise the user sees two
545+
// adjacent dividers (~80px combined spacing).
541546
mockUseMerklBonusClaim.mockReturnValue(
542547
buildMerklClaimDataMock({
543548
claimableReward: '5.00',
544549
hasPendingClaim: false,
545550
}),
546551
);
547-
const { getByTestId } = renderWithProvider(<MoneyHomeView />);
548-
expect(
549-
getByTestId(MoneyHomeViewTestIds.EARNINGS_DIVIDER),
550-
).toBeOnTheScreen();
552+
const { getByTestId, queryByTestId, queryAllByTestId } =
553+
renderWithProvider(<MoneyHomeView />);
551554
expect(
552555
getByTestId(ASSET_OVERVIEW_CLAIM_BONUS_TEST_IDS.CONTAINER),
553556
).toBeOnTheScreen();
557+
expect(
558+
queryByTestId(MoneyHomeViewTestIds.EARNINGS_DIVIDER),
559+
).not.toBeOnTheScreen();
560+
expect(
561+
queryAllByTestId(MoneyHomeViewTestIds.EARNINGS_DIVIDER),
562+
).toHaveLength(0);
554563
});
555564
});
556565

app/components/UI/Money/Views/MoneyHomeView/MoneyHomeView.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,9 @@ const MoneyHomeView = () => {
269269
isLoading={vaultApyQuery.isLoading || isAggregatedBalanceLoading}
270270
onInfoPress={handleEarningsInfoPress}
271271
/>
272-
<Divider testID={MoneyHomeViewTestIds.EARNINGS_DIVIDER} />
272+
{!isBonusEligible && (
273+
<Divider testID={MoneyHomeViewTestIds.EARNINGS_DIVIDER} />
274+
)}
273275
{isBonusEligible && (
274276
<AssetOverviewClaimBonus
275277
asset={LINEA_MUSD_ASSET_FOR_MERKL}

0 commit comments

Comments
 (0)