@agoric/swingset-liveslots 0.10.3-dev-eb7e9eb.0 → 0.10.3-u11.0

package/src/virtualObjectManager.js

@@ -16,19 +16,10 @@ import {
 
 /** @template T @typedef {import('@agoric/vat-data').DefineKindOptions<T>} DefineKindOptions */
 
-const { hasOwn, defineProperty, getOwnPropertyNames, entries } = Object;
+const { hasOwn, defineProperty, getOwnPropertyNames } = Object;
 const { ownKeys } = Reflect;
 const { quote: q } = assert;
 
-// See https://github.com/Agoric/agoric-sdk/issues/8005
-// Once agoric-sdk is upgraded to depend on endo post
-// https://github.com/endojs/endo/pull/1606 then remove this
-// definition of `b` and say instead
-// ```js
-//   const { quote: q, base: b } = assert;
-// ```
-const b = index => q(Number(index));
-
 // import { kdebug } from './kdebug.js';
 
 // TODO Use environment-options.js currently in ses/src after factoring it out
@@ -150,8 +141,9 @@ const makeContextCache = (makeState, makeContext) => {
  * @param {*} getSlotForVal
  * @returns {ContextProvider}
  */
-const makeContextProvider = (contextCache, getSlotForVal) =>
-  harden(rep => contextCache.get(getSlotForVal(rep)));
+const makeContextProvider = (contextCache, getSlotForVal) => {
+  return harden(rep => contextCache.get(getSlotForVal(rep)));
+};
 
 const makeContextProviderKit = (contextCache, getSlotForVal, facetNames) => {
   /** @type { Record<string, any> } */
@@ -268,15 +260,15 @@ const makeFacets = (
 };
 
 const insistDurableCapdata = (vrm, what, capdata, valueFor) => {
-  for (const [idx, vref] of entries(capdata.slots)) {
+  capdata.slots.forEach((vref, idx) => {
     if (!vrm.isDurable(vref)) {
       if (valueFor) {
-        Fail`value for ${what} is not durable: slot ${b(idx)} of ${capdata}`;
+        Fail`value for ${what} is not durable: slot ${q(idx)} of ${capdata}`;
       } else {
-        Fail`${what} is not durable: slot ${b(idx)} of ${capdata}`;
+        Fail`${what} is not durable: slot ${q(idx)} of ${capdata}`;
       }
     }
-  }
+  });
 };
 
 const insistSameCapData = (oldCD, newCD) => {
@@ -289,11 +281,11 @@ const insistSameCapData = (oldCD, newCD) => {
   if (oldCD.slots.length !== newCD.slots.length) {
     Fail`durable Kind stateShape mismatch (slots.length)`;
   }
-  for (const [idx, oldVref] of entries(oldCD.slots)) {
+  oldCD.slots.forEach((oldVref, idx) => {
     if (newCD.slots[idx] !== oldVref) {
       Fail`durable Kind stateShape mismatch (slot[${idx}])`;
     }
-  }
+  });
 };
 
 /**
@@ -715,7 +707,7 @@ export const makeVirtualObjectManager = (
     durableKindDescriptor = undefined, // only for durables
   ) => {
     const {
-      finish = undefined,
+      finish,
       stateShape = undefined,
       thisfulMethods = false,
       interfaceGuard = undefined,
@@ -982,9 +974,9 @@ export const makeVirtualObjectManager = (
       let doMoreGC = false;
       const record = dataCache.get(baseRef);
       for (const valueCD of Object.values(record.capdatas)) {
-        for (const vref of valueCD.slots) {
+        valueCD.slots.forEach(vref => {
           doMoreGC = vrm.removeReachableVref(vref) || doMoreGC;
-        }
+        });
       }
       dataCache.delete(baseRef);
       return doMoreGC;
@@ -1023,7 +1015,6 @@ export const makeVirtualObjectManager = (
         if (isDurable) {
           insistDurableCapdata(vrm, prop, valueCD, true);
         }
-        // eslint-disable-next-line github/array-foreach
         valueCD.slots.forEach(vrm.addReachableVref);
         capdatas[prop] = valueCD;
         valueMap.set(prop, value);

package/src/virtualReferences.js

@@ -682,54 +682,6 @@ export function makeVirtualReferenceManager(
     return size;
   }
 
-  /**
-   * Counters to track the next number for various categories of allocation.
-   * `exportID` starts at 1 because 'o+0' is always automatically
-   * pre-assigned to the root object.
-   * `promiseID` starts at 5 as a very minor aid to debugging: when puzzling
-   * over trace logs and the like, it helps for the numbers in various species
-   * of IDs that are jumbled together to be a little out of sync and thus a
-   * little less similar to each other.
-   */
-  const initialIDCounters = { exportID: 1, collectionID: 1, promiseID: 5 };
-  /** @type {Record<string, number>} */
-  let idCounters;
-  let idCountersAreDirty = false;
-
-  function initializeIDCounters() {
-    if (!idCounters) {
-      // the saved value might be missing, or from an older liveslots
-      // (with fewer counters), so merge it with our initial values
-      const saved = JSON.parse(syscall.vatstoreGet('idCounters') || '{}');
-      idCounters = { ...initialIDCounters, ...saved };
-      idCountersAreDirty = true;
-    }
-  }
-
-  function allocateNextID(name) {
-    if (!idCounters) {
-      // Normally `initializeIDCounters` would be called from startVat, but some
-      // tests bypass that so this is a backstop.  Note that the invocation from
-      // startVat is there to make vatStore access patterns a bit more
-      // consistent from one vat to another, principally as a confusion
-      // reduction measure in service of debugging; it is not a correctness
-      // issue.
-      initializeIDCounters();
-    }
-    const result = idCounters[name];
-    result !== undefined || Fail`unknown idCounters[${name}]`;
-    idCounters[name] += 1;
-    idCountersAreDirty = true;
-    return result;
-  }
-
-  function flushIDCounters() {
-    if (idCountersAreDirty) {
-      syscall.vatstoreSet('idCounters', JSON.stringify(idCounters));
-      idCountersAreDirty = false;
-    }
-  }
-
   const testHooks = {
     getReachableRefCount,
     countCollectionsForWeakKey,
@@ -774,9 +726,6 @@ export function makeVirtualReferenceManager(
     ceaseRecognition,
     setDeleteCollectionEntry,
     getRetentionStats,
-    initializeIDCounters,
-    allocateNextID,
-    flushIDCounters,
     testHooks,
   });
 }

package/test/liveslots-helpers.js

@@ -16,8 +16,8 @@ import { kser } from './kmarshal.js';
 
 /**
  * @param {object} [options]
- * @param {boolean} [options.skipLogging]
- * @param {Map<string, string>} [options.kvStore]
+ * @param {boolean} [options.skipLogging = false]
+ * @param {Map<string, string>} [options.kvStore = new Map()]
  */
 export function buildSyscall(options = {}) {
   const { skipLogging = false, kvStore: fakestore = new Map() } = options;
@@ -168,9 +168,9 @@ function makeRPMaker(nextNumber = 1) {
  * @param {string} vatName
  * @param {object} [options]
  * @param {boolean} [options.forceGC]
- * @param {Map<string, string>} [options.kvStore]
+ * @param {Map<string, string>} [options.kvStore = new Map()]
  * @param {number} [options.nextPromiseImportNumber]
- * @param {boolean} [options.skipLogging]
+ * @param {boolean} [options.skipLogging = false]
  */
 export async function setupTestLiveslots(
   t,

package/test/mock-gc.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import { waitUntilQuiescent } from './waitUntilQuiescent.js';
 import { makeDummyMeterControl } from './dummyMeterControl.js';
 

package/test/storeGC/test-lifecycle.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';
 

package/test/test-collection-upgrade.js

@@ -1,3 +1,4 @@
+/* eslint-disable no-await-in-loop, @jessie.js/no-nested-await, no-shadow */
 import test from 'ava';
 import '@endo/init/debug.js';
 

package/test/test-collections.js

@@ -1,10 +1,8 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';
 
 import { Far } from '@endo/marshal';
 import { M } from '@agoric/store';
-import { makeCopyMap, makeCopySet } from '@endo/patterns';
 import { makeFakeCollectionManager } from '../tools/fakeVirtualSupport.js';
 
 const {
@@ -193,89 +191,6 @@ test('basic weak set operations', t => {
   );
 });
 
-function exerciseSetAddAll(t, weak, testStore) {
-  const allThatStuff = stuff.map(entry => entry[0]);
-
-  testStore.addAll(allThatStuff);
-  for (const elem of allThatStuff) {
-    t.truthy(testStore.has(elem));
-    testStore.delete(elem);
-  }
-  if (!weak) {
-    t.is(testStore.getSize(), 0);
-  }
-
-  testStore.addAll(makeCopySet(allThatStuff));
-  for (const elem of allThatStuff) {
-    t.truthy(testStore.has(elem));
-    testStore.delete(elem);
-  }
-  if (!weak) {
-    t.is(testStore.getSize(), 0);
-  }
-
-  t.throws(
-    () => testStore.addAll({ bogus: 47 }),
-    m(/provided data source is not iterable/),
-  );
-}
-
-test('set addAll', t => {
-  exerciseSetAddAll(t, false, makeScalarBigSetStore('test set'));
-});
-
-test('weak set addAll', t => {
-  exerciseSetAddAll(t, true, makeScalarBigWeakSetStore('test weak set'));
-});
-
-test('set snapshot', t => {
-  const testStore = makeScalarBigSetStore('test set');
-  const allThatStuff = stuff.map(entry => entry[0]);
-  testStore.addAll(allThatStuff);
-  t.deepEqual(testStore.snapshot(), makeCopySet(allThatStuff));
-});
-
-function exerciseMapAddAll(t, weak, testStore) {
-  testStore.addAll(stuff);
-  for (const [k, v] of stuff) {
-    t.truthy(testStore.has(k));
-    t.is(testStore.get(k), v);
-    testStore.delete(k);
-  }
-  if (!weak) {
-    t.is(testStore.getSize(), 0);
-  }
-
-  testStore.addAll(makeCopyMap(stuff));
-  for (const [k, v] of stuff) {
-    t.truthy(testStore.has(k));
-    t.is(testStore.get(k), v);
-    testStore.delete(k);
-  }
-  if (!weak) {
-    t.is(testStore.getSize(), 0);
-  }
-
-  t.throws(
-    () => testStore.addAll({ bogus: 47 }),
-    m(/provided data source is not iterable/),
-  );
-}
-
-test('map addAll', t => {
-  exerciseMapAddAll(t, false, makeScalarBigMapStore('test map'));
-});
-
-test('weak map addAll', t => {
-  exerciseMapAddAll(t, true, makeScalarBigWeakMapStore('test weak map'));
-});
-
-test('map snapshot', t => {
-  const testStore = makeScalarBigMapStore('test map');
-  testStore.addAll(stuff);
-  t.deepEqual(testStore.snapshot(), makeCopyMap(stuff));
-});
-
 test('constrain map key shape', t => {
   const stringsOnly = makeScalarBigMapStore('map key strings only', {
     keyShape: M.string(),
@@ -490,9 +405,7 @@ test('map fail on concurrent modification', t => {
   const primeMap = makeScalarBigMapStore('fmap', {
     keyShape: M.number(),
   });
-  for (const [i, v] of primes.entries()) {
-    primeMap.init(v, `${v} is prime #${i + 1}`);
-  }
+  primes.forEach((v, i) => primeMap.init(v, `${v} is prime #${i + 1}`));
 
   let iter = primeMap.keys()[Symbol.iterator]();
   t.deepEqual(iter.next(), { done: false, value: 2 });
@@ -520,9 +433,7 @@ test('set fail on concurrent modification', t => {
   const primeSet = makeScalarBigSetStore('fset', {
     keyShape: M.number(),
   });
-  for (const v of primes) {
-    primeSet.add(v);
-  }
+  primes.forEach(v => primeSet.add(v));
 
   let iter = primeSet.keys()[Symbol.iterator]();
   t.deepEqual(iter.next(), { done: false, value: 2 });
@@ -550,9 +461,7 @@ test('map ok with concurrent deletion', t => {
   const primeMap = makeScalarBigMapStore('fmap', {
     keyShape: M.number(),
   });
-  for (const [i, v] of primes.entries()) {
-    primeMap.init(v, `${v} is prime #${i + 1}`);
-  }
+  primes.forEach((v, i) => primeMap.init(v, `${v} is prime #${i + 1}`));
   const iter = primeMap.keys()[Symbol.iterator]();
   t.deepEqual(iter.next(), { done: false, value: 2 });
   primeMap.delete(3);
@@ -567,9 +476,7 @@ test('set ok with concurrent deletion', t => {
   const primeSet = makeScalarBigSetStore('fset', {
     keyShape: M.number(),
   });
-  for (const v of primes) {
-    primeSet.add(v);
-  }
+  primes.forEach(v => primeSet.add(v));
 
   const iter = primeSet.keys()[Symbol.iterator]();
   t.deepEqual(iter.next(), { done: false, value: 2 });
@@ -859,6 +766,13 @@ test('set queries', t => {
     symbolKrusty,
     undefined,
   ]);
+
+  // @ts-expect-error our BigSetStore has .entries, but not the SetStore type
+  t.deepEqual(Array.from(testStore.entries(M.number())), [
+    [-29, -29],
+    [3, 3],
+    [47, 47],
+  ]);
 });
 
 test('remotable sort order', t => {
@@ -884,9 +798,7 @@ test('complex map queries', t => {
   const primeStore = makeScalarBigMapStore('prime map', {
     keyShape: M.number(),
   });
-  for (const [i, v] of primes.entries()) {
-    primeStore.init(v, `${v} is prime #${i + 1}`);
-  }
+  primes.forEach((v, i) => primeStore.init(v, `${v} is prime #${i + 1}`));
 
   t.deepEqual(Array.from(primeStore.values()), [
     '2 is prime #1',
@@ -1061,9 +973,7 @@ test('complex set queries', t => {
   const primeStore = makeScalarBigSetStore('prime set', {
     keyShape: M.number(),
   });
-  for (const v of primes) {
-    primeStore.add(v);
-  }
+  primes.forEach(v => primeStore.add(v));
 
   t.deepEqual(
     Array.from(primeStore.values()),

package/test/test-durabilityChecks.js

@@ -269,5 +269,5 @@ async function runDurabilityCheckTest(t, relaxDurabilityRules) {
   }
 }
 
-test('durability checks (strict)', runDurabilityCheckTest, false);
-test('durability checks (relaxed)', runDurabilityCheckTest, true);
+test('durability checks (strict)', t => runDurabilityCheckTest(t, false));
+test('durability checks (relaxed)', t => runDurabilityCheckTest(t, true));

package/test/test-facetiousness.js

@@ -126,7 +126,7 @@ test('checkAndUpdateFacetiousness', t => {
   t.deepEqual(cauf({}, barfoo), barfoo);
 
   // a single Kind can only be redefined as another single
-  t.is(cauf(desc(), undefined), undefined);
+  t.deepEqual(cauf(desc(), undefined), undefined);
   t.throws(() => cauf(desc(), foo), {
     message: 'defineDurableKindMulti called for unfaceted KindHandle "tag"',
   });

package/test/test-gc-sensitivity.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';
 import { Far } from '@endo/marshal';

package/test/test-handled-promises.js

@@ -1,3 +1,4 @@
+/* eslint-disable no-await-in-loop, @jessie.js/no-nested-await, no-shadow */
 import test from 'ava';
 import '@endo/init/debug.js';
 
@@ -232,12 +233,12 @@ test('past-incarnation watched promises', async t => {
   t.deepEqual(getDispatchLogs(), [
     fulfillmentMessage(`p-${nextPImport()}`, 'created local promise: rejected'),
   ]);
-  t.is(
+  t.deepEqual(
     lastPImport - firstPImport + 1,
     4,
     'imported 4 promises (1 per dispatch)',
   );
-  t.is(lastPExport - firstPExport + 1, 1, 'exported 1 promise: first');
+  t.deepEqual(lastPExport - firstPExport + 1, 1, 'exported 1 promise: first');
 
   await dispatchMessage('watchLocalPromise', 'orphaned');
   t.deepEqual(getDispatchLogs(), [
@@ -259,12 +260,12 @@ test('past-incarnation watched promises', async t => {
     fulfillmentMessage(`p-${nextPImport()}`, 'watched local promise: rejected'),
     rejectionMessage(`p+${lastPExport}`, S),
   ]);
-  t.is(
+  t.deepEqual(
     lastPImport - firstPImport + 1,
     7,
     'imported 7 promises (1 per dispatch)',
   );
-  t.is(
+  t.deepEqual(
     lastPExport - firstPExport + 1,
     4,
     'exported 4 promises: first, orphaned, fulfilled, rejected',

package/test/test-initial-vrefs.js

@@ -148,5 +148,5 @@ test('vrefs', async t => {
   const expectedStore1Vref = `o+v${initialKindIDs.scalarMapStore}/5`;
   const store1Vref = (await run('getStore1')).slots[0];
   t.is(store1Vref, expectedStore1Vref);
-  t.is(kunser(JSON.parse(fakestore.get(`vc.5.s${'key'}`))), 'value');
+  t.deepEqual(kunser(JSON.parse(fakestore.get(`vc.5.s${'key'}`))), 'value');
 });

package/test/test-liveslots-mock-gc.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';
 

package/test/test-liveslots-real-gc.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 /* global WeakRef */
 import test from 'ava';
 import '@endo/init/debug.js';

package/test/test-liveslots.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';
 
@@ -46,14 +45,14 @@ test('calls', async t => {
 
   // root!one() // sendOnly
   await dispatch(makeMessage(rootA, 'one', ['args']));
-  t.is(log.shift(), 'one');
+  t.deepEqual(log.shift(), 'one');
 
   // pr = makePromise()
   // root!two(pr.promise)
   // pr.resolve('result')
   await dispatch(makeMessage(rootA, 'two', [kslot('p-1')]));
   t.deepEqual(log.shift(), { type: 'subscribe', target: 'p-1' });
-  t.is(log.shift(), 'two true');
+  t.deepEqual(log.shift(), 'two true');
 
   await dispatch(makeResolve('p-1', kser('result')));
   t.deepEqual(log.shift(), ['res', 'result']);
@@ -64,7 +63,7 @@ test('calls', async t => {
 
   await dispatch(makeMessage(rootA, 'two', [kslot('p-2')]));
   t.deepEqual(log.shift(), { type: 'subscribe', target: 'p-2' });
-  t.is(log.shift(), 'two true');
+  t.deepEqual(log.shift(), 'two true');
 
   await dispatch(makeReject('p-2', kser('rejection')));
   t.deepEqual(log.shift(), ['rej', 'rejection']);
@@ -100,7 +99,7 @@ test('liveslots pipelines to syscall.send', async t => {
   // for x!pipe1(), a second pipelined to the result promise of it, and a
   // third pipelined to the result of the second.
 
-  t.is(log.shift(), 'sent p1p2p3');
+  t.deepEqual(log.shift(), 'sent p1p2p3');
   t.deepEqual(log.shift(), {
     type: 'send',
     targetSlot: x,
@@ -657,7 +656,8 @@ test('capdata size limit on syscalls', async t => {
   };
 
   const send = op => dispatch(makeMessage(rootA, op, [kslot(target)], rp));
-  const expectFail = () => t.is(log.shift(), 'fail: syscall capdata too large');
+  const expectFail = () =>
+    t.deepEqual(log.shift(), 'fail: syscall capdata too large');
   const expectVoidReturn = () =>
     t.deepEqual(log.shift(), {
       type: 'resolve',
@@ -906,7 +906,7 @@ test('disable disavow', async t => {
 
   // root~.one() // sendOnly
   await dispatch(makeMessage(rootA, 'one', []));
-  t.is(log.shift(), false);
+  t.deepEqual(log.shift(), false);
   t.deepEqual(log, []);
 });
 
@@ -967,7 +967,7 @@ test('disavow', async t => {
   // root~.one(import1) // sendOnly
   await dispatch(makeMessage(rootA, 'one', [kslot(import1)]));
   t.deepEqual(log.shift(), { type: 'dropImports', slots: [import1] });
-  t.is(log.shift(), 'disavowed pres1');
+  t.deepEqual(log.shift(), 'disavowed pres1');
 
   function loggedError(re) {
     const l = log.shift();
@@ -975,11 +975,11 @@ test('disavow', async t => {
     t.truthy(re.test(l.message));
   }
   loggedError(/attempt to disavow unknown/);
-  t.is(log.shift(), 'tried duplicate disavow');
+  t.deepEqual(log.shift(), 'tried duplicate disavow');
   loggedError(/attempt to disavow unknown/);
-  t.is(log.shift(), 'tried to disavow Promise');
+  t.deepEqual(log.shift(), 'tried to disavow Promise');
   loggedError(/attempt to disavow an export/);
-  t.is(log.shift(), 'tried to disavow export');
+  t.deepEqual(log.shift(), 'tried to disavow export');
   const msg = log.shift();
   t.like(msg, {
     type: 'exit',
@@ -987,7 +987,7 @@ test('disavow', async t => {
   });
   expectError(t, msg.info, /this Presence has been disavowed/);
   t.deepEqual(log.shift(), Error('this Presence has been disavowed'));
-  t.is(log.shift(), 'tried to send to disavowed');
+  t.deepEqual(log.shift(), 'tried to send to disavowed');
   t.deepEqual(log, []);
 });
 

package/test/test-vpid-liveslots.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';
 
@@ -294,6 +293,7 @@ async function doVatResolveCase23(t, which, mode, stalls) {
         // another few turns. We wait some number of turns before using p1
         // again, to exercise as many race conditions as possible.
         for (let i = 0; i < stalls; i += 1) {
+          // eslint-disable-next-line no-await-in-loop
           await Promise.resolve();
         }
 

package/test/util.js

@@ -48,7 +48,7 @@ export function buildDispatch(onDispatchCallback) {
 
 /**
  * @param {unknown} target
- * @param {string | symbol} method
+ * @param {string} method
  * @param {any[]} args
  * @param {unknown} result
  */

package/test/virtual-objects/test-cease-recognition.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 /* global FinalizationRegistry WeakRef */
 import test from 'ava';
 import '@endo/init/debug.js';
@@ -39,7 +38,7 @@ function weakKeyCheck(t, log, vref) {
   t.true(result === undefined || !result.startsWith(prefix), `ew:${result}`);
 }
 
-test('only enumerate virtual objects', t => {
+test('only enumerate virtual objects', async t => {
   const { log, vrm } = makeVRM();
 
   // retiring a plain Remotable does a is-it-a-weak-key chck

package/test/virtual-objects/test-cross-facet.js

@@ -15,7 +15,7 @@ function attack2(mut1, immut2) {
   Reflect.apply(mutableProto.set, immut2, [6]);
 }
 
-test('forbid cross-facet prototype attack', t => {
+test('forbid cross-facet prototype attack', async t => {
   const vom = makeFakeVirtualObjectManager();
   const init = () => ({ value: 0 });
   const behavior = {

package/test/virtual-objects/test-empty-data.js

@@ -3,7 +3,7 @@ import '@endo/init/debug.js';
 
 import { makeFakeVirtualObjectManager } from '../../tools/fakeVirtualSupport.js';
 
-test('non-object initial data message', t => {
+test('non-object initial data message', async t => {
   const vom = makeFakeVirtualObjectManager();
   const goodInit = () => ({ value: 0 });
   // 'badInit' is  () =>  { value: 0 }

package/test/virtual-objects/test-facets.js

@@ -3,7 +3,7 @@ import '@endo/init/debug.js';
 
 import { makeFakeVirtualObjectManager } from '../../tools/fakeVirtualSupport.js';
 
-test('facets', t => {
+test('facets', async t => {
   const vom = makeFakeVirtualObjectManager();
   const init = () => ({ value: 0 });
   const behavior = {

package/test/virtual-objects/test-kind-changes.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';
 import { Far } from '@endo/marshal';

package/test/virtual-objects/test-reachable-vrefs.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';
 
@@ -8,7 +7,7 @@ import { initEmpty } from '@agoric/store';
 import { makeVatSlot } from '../../src/parseVatSlots.js';
 import { makeFakeVirtualStuff } from '../../tools/fakeVirtualSupport.js';
 
-test('VOM tracks reachable vrefs', t => {
+test('VOM tracks reachable vrefs', async t => {
   const { vom, vrm, cm } = makeFakeVirtualStuff();
   const { defineKind } = vom;
   const { makeScalarBigWeakMapStore } = cm;

package/test/virtual-objects/test-rep-tostring.js

@@ -16,7 +16,7 @@ const init = () => ({});
 const behavior = {};
 const facets = { foo: {}, bar: {} };
 
-test('representatives with label-instances', t => {
+test('representatives with label-instances', async t => {
   const { fakeStuff, vom } = makeFakeVirtualStuff();
   const { getSlotForVal } = fakeStuff;
   const makeThing = vom.defineKind('thing', init, behavior);
@@ -33,7 +33,7 @@ test('representatives with label-instances', t => {
   t.is(`${q(thing2)}`, `"[Alleged: thing#${thing2vref}]"`);
 });
 
-test('facets with label-instances', t => {
+test('facets with label-instances', async t => {
   const { fakeStuff, vom } = makeFakeVirtualStuff();
   const { getSlotForVal } = fakeStuff;
   const makeThings = vom.defineKindMulti('thing', init, facets);

package/test/virtual-objects/test-retain-remotable.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 /* global WeakRef */
 import test from 'ava';
 import '@endo/init/debug.js';

package/test/virtual-objects/test-state-shape.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import test from 'ava';
 import '@endo/init/debug.js';