fix: include paymentRecoverySets when calling preparePurseKind()

cleanerRecoverySet() asserts paymentRecoverySet !== undefined when recoverSetState === noRecoverySets, so paymentLedger must provide it when calling preparePurseKind().
Agoric · Nov 7, 2023 · 11dfdb5 · 11dfdb5
1 parent aedaf46
commit 11dfdb5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/ERTP/src/paymentLedger.js b/packages/ERTP/src/paymentLedger.js
@@ -334,7 +334,7 @@ export const preparePaymentLedger = (
       withdrawInternal,
     }),
     recoverySetsState,
-    recoverySetsState === 'noRecoverySets' ? undefined : paymentRecoverySets,
+    paymentRecoverySets,
   );
 
   /** @type {Issuer<K>} */

diff --git a/packages/ERTP/src/purse.js b/packages/ERTP/src/purse.js
@@ -25,7 +25,7 @@ const EMPTY_COPY_SET = makeCopySet([]);
  *   withdrawInternal: any;
  * }} purseMethods
  * @param {RecoverySetsOption} recoverySetsState
- * @param {WeakMapStore<Payment, SetStore<Payment>>} [paymentRecoverySets]
+ * @param {WeakMapStore<Payment, SetStore<Payment>>} paymentRecoverySets
  */
 export const preparePurseKind = (
   issuerBaggage,
@@ -35,7 +35,7 @@ export const preparePurseKind = (
   PurseIKit,
   purseMethods,
   recoverySetsState,
-  paymentRecoverySets = undefined,
+  paymentRecoverySets,
 ) => {
   const amountShape = brand.getAmountShape();