comments cleanup

makeDockerfile.ts

@@ -46,7 +46,7 @@ RUN /usr/src/upgrade-test-scripts/start_ag0.sh
   ) {
     return `
 # PREPARE ${proposalName}
-// 
+
 # upgrading to ${planName}
 FROM use-${lastProposal.proposalName} as prepare-${proposalName}
 ENV UPGRADE_TO=${planName}

proposals/55:statom-vaults/add-collateral.test.js

@@ -41,6 +41,7 @@ const test = anyTest;
 const assetInfo = {
   repo: {
     release:
+      // TODO update with actual release https://github.com/0xpatrickdev/agoric-vault-collateral-proposal/releases/tag/2023-16-10-agoric-3
       'https://github.com/dckc/agoric-vault-collateral-proposal/releases/tag/v0.13.0-beta1',
     url: 'https://github.com/0xpatrickdev/agoric-vault-collateral-proposal',
     name: 'agoric-vault-collateral-proposal',
@@ -103,18 +104,16 @@ const makeTestContext = async (t, io = {}) => {
     tmpName = tmpNameAmbient,
   } = io;
 
-  // @@ const src = makeWebRd(staticConfig.assetBase, { fetch });
   const src = makeWebRd(staticConfig.releaseAssets, { fetch });
 
   const td = await new Promise((resolve, reject) =>
     tmpName({ prefix: 'assets' }, (err, x) => (err ? reject(err) : resolve(x))),
   );
   const dest = makeFileRW(td, { fsp, path });
-  // @@ Error: `t.teardown()` is not allowed in hooks
+  // FIXME Error: `t.teardown()` is not allowed in hooks
   //   t.teardown(() => assets.remove());
   const assets = makeWebCache(src, dest);
   // assume filenames don't overlap
-  //@@alt const bundleAssets = makeWebCache(src.join('bundles/'), dest);
   const bundleAssets = makeWebCache(src, dest);
   console.log(`bundleAssets: ${bundleAssets}`);
 

upgrade-test-scripts/lib/econHelpers.js

@@ -2,6 +2,7 @@ import { agd, agops } from './cliHelper.js';
 import { ATOM_DENOM, CHAINID, VALIDATORADDR } from './constants.js';
 import { executeOffer } from './commonUpgradeHelpers.js';
 
+// TODO return the id of the new vault so subsquent commands can use it
 export const openVault = (address, mint, collateral) => {
   return executeOffer(
     address,