jennie/ghostguild-org
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor(contributions): tighten requiresPayment contract; use findLast

Browse source
This commit is contained in:
Jennie Robinson Faber 2026-04-19 18:10:57 +01:00
parent 62c606b30a
commit 03eee45cbd
2 changed files with 6 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

4
app/config/contributions.js
View file

@ -8,7 +8,7 @@ export const CONTRIBUTION_PRESETS = [
{ amount: 50, label: "I want to sponsor multiple members" },
]
export const requiresPayment = (amount) => Number(amount) > 0
export const requiresPayment = (amount) => amount > 0
export const isValidContributionAmount = (amount) =>
Number.isInteger(amount) && amount >= 0
@ -17,6 +17,6 @@ export const getGuidanceLabel = (amount) => {
if (amount === null || amount === undefined) return null
const n = Number(amount)
if (!Number.isFinite(n) || n < 0) return null
const match = [...CONTRIBUTION_PRESETS].reverse().find(p => p.amount <= n)
const match = CONTRIBUTION_PRESETS.findLast(p => p.amount <= n)
return match?.label ?? null
}

7
tests/client/contributions.test.js
View file

@ -26,9 +26,10 @@ describe('requiresPayment', () => {
expect(requiresPayment(1)).toBe(true)
expect(requiresPayment(15)).toBe(true)
})
it('coerces string numbers', () => {
expect(requiresPayment('15')).toBe(true)
expect(requiresPayment('0')).toBe(false)
it('returns false for null, undefined, NaN', () => {
expect(requiresPayment(null)).toBe(false)
expect(requiresPayment(undefined)).toBe(false)
expect(requiresPayment(NaN)).toBe(false)
})
})