refactor: DRY on cleanup when bids are deleted

Agoric · Jul 21, 2023 · 45fe766 · 45fe766
1 parent a29a826
commit 45fe766
Showing 1 changed file with 8 additions and 12 deletions.
diff --git a/packages/inter-protocol/src/auction/offerBook.js b/packages/inter-protocol/src/auction/offerBook.js
@@ -190,7 +190,9 @@ export const prepareScaledBidBook = (baggage, makeRecorderKit) => {
       delete(key) {
         const { records } = this.state;
         void deleteNodeIfPresent(key);
-        bidDataKits.delete(key);
+        if (bidDataKits.has(key)) {
+          bidDataKits.delete(key);
+        }
         records.delete(key);
       },
       updateReceived(key, sold) {
@@ -209,11 +211,7 @@ export const prepareScaledBidBook = (baggage, makeRecorderKit) => {
         for (const [key, { seat }] of records.entries()) {
           if (!seat.hasExited()) {
             seat.exit();
-            if (bidDataKits.has(key)) {
-              bidDataKits.delete(key);
-            }
-            records.delete(key);
-            void deleteNodeIfPresent(key);
+            this.self.delete(key);
           }
         }
       },
@@ -338,7 +336,9 @@ export const preparePriceBook = (baggage, makeRecorderKit) => {
       delete(key) {
         const { records } = this.state;
         void deleteNodeIfPresent(key);
-        bidDataKits.delete(key);
+        if (bidDataKits.has(key)) {
+          bidDataKits.delete(key);
+        }
         records.delete(key);
       },
       updateReceived(key, sold) {
@@ -357,11 +357,7 @@ export const preparePriceBook = (baggage, makeRecorderKit) => {
         for (const [key, { seat }] of records.entries()) {
           if (!seat.hasExited()) {
             seat.exit();
-            if (bidDataKits.has(key)) {
-              bidDataKits.delete(key);
-            }
-            void deleteNodeIfPresent(key);
-            records.delete(key);
+            this.self.delete(key);
           }
         }
       },