BUGFIX: deliverOneMessage() accidentally skipped insist() msgs.

I don't see how to write a test for this. Suggestions?
Agoric · Jan 13, 2020 · 68f3eda · 68f3eda
1 parent 1d786a0
commit 68f3eda
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/SwingSet/src/kernel/vatManager.js b/packages/SwingSet/src/kernel/vatManager.js
@@ -293,9 +293,10 @@ export default function makeVatManager(
   async function deliverOneMessage(target, msg) {
     insistMessage(msg);
     const targetSlot = mapKernelSlotToVatSlot(target);
-    if (targetSlot.type === 'object') {
+    const { type } = parseVatSlot(targetSlot);
+    if (type === 'object') {
       insist(parseVatSlot(targetSlot).allocatedByVat, `deliver() to wrong vat`);
-    } else if (targetSlot.type === 'promise') {
+    } else if (type === 'promise') {
       const p = kernelKeeper.getKernelPromise(target);
       insist(p.decider === vatID, `wrong decider`);
     }