diff --git a/src/__tests__/pages/policy/output/budget/MultiYearBudgetaryImpact.test.js b/src/__tests__/pages/policy/output/budget/MultiYearBudgetaryImpact.test.js index f33644670..61bc63b12 100644 --- a/src/__tests__/pages/policy/output/budget/MultiYearBudgetaryImpact.test.js +++ b/src/__tests__/pages/policy/output/budget/MultiYearBudgetaryImpact.test.js @@ -209,8 +209,11 @@ describe("MultiYearBudgetaryImpact", () => { expect( screen.getByText(mockImpact.budget.budgetary_impact / 1e9), ).toBeInTheDocument(); + const expectedBenefitImpact = -Math.abs( + mockImpact.budget.benefit_spending_impact / 1e9, + ); expect( - screen.getByText(mockImpact.budget.benefit_spending_impact / 1e9), + screen.getByText(expectedBenefitImpact.toString()), ).toBeInTheDocument(); expect( screen.getByText( diff --git a/src/pages/policy/output/budget/MultiYearBudgetaryImpact.jsx b/src/pages/policy/output/budget/MultiYearBudgetaryImpact.jsx index 8f9bf6443..21fdf0c09 100644 --- a/src/pages/policy/output/budget/MultiYearBudgetaryImpact.jsx +++ b/src/pages/policy/output/budget/MultiYearBudgetaryImpact.jsx @@ -114,10 +114,13 @@ export default function MultiYearBudgetaryImpact(props) { item.budgetKey ? item.budgetKey : null, item.formula ? item.formula : null, ), + yearRange: roundToBillions( - item.budgetKey - ? impact.budget[item.budgetKey] - : item.formula(impact.budget), + item.budgetKey === "benefit_spending_impact" + ? -Math.abs(impact.budget[item.budgetKey]) + : item.budgetKey + ? impact.budget[item.budgetKey] + : item.formula(impact.budget), roundingPrecisionByCountry[metadata.countryId] || roundingPrecisionByCountry.default, ), @@ -198,6 +201,10 @@ export function getYearlyImpacts( impact = item.result.budget[budgetKey]; } + if (budgetKey === "benefit_spending_impact") { + impact = -Math.abs(impact); + } + yearlyImpacts[year] = roundToBillions( impact, roundingPrecisionByCountry[countryId] ||