test(internal): fake vstorage advance blockHeight

[mhofman]

refs: #10890
refs: #10931

Description

Add a driver to update the block height in fake vstorage cells.

Add test of sequence: false based deletion.

Security Considerations

None

Scaling Considerations

None

Documentation Considerations

Internal test hooks

Testing Considerations

Updated unit tests of fake storage

Upgrade Considerations

None

[cloudflare-workers-and-pages]

Deploying agoric-sdk with    Cloudflare Pages

Latest commit:	68416ba
Status:	✅  Deploy successful!
Preview URL:	https://30ee7db8.agoric-sdk.pages.dev
Branch Preview URL:	https://mhofman-fake-vstorage-blocks.agoric-sdk.pages.dev

View logs

[gibson042]

Suggestion for clarifying the StreamCell reset behavior:

        case 'append': {
          trace('toStorage append', message);
          /** @type {StorageEntry[]} */
          const newEntries = message.args;
          for (const [key, value] of newEntries) {
            value != null || Fail`attempt to append with no value`;

            let oldVal = data.get(key);
            let streamCell;
            if (oldVal != null) {
              try {
                streamCell = JSON.parse(oldVal);
                assert(isStreamCell(streamCell));
              } catch (_err) {
                streamCell = undefined;
              }
              // StreamCells reset at block boundaries.
              if (streamCell?.blockHeight !== currentBlockHeight) {
                streamCell = undefined;
                oldVal = undefined;
              }
            }

            if (streamCell === undefined) {
              streamCell = {
                blockHeight: String(currentBlockHeight),
                values: oldVal != null ? [oldVal] : [],
              };
            }

            streamCell.values.push(value);
            data.set(key, JSON.stringify(streamCell));
          }
          break;
        }

[mhofman]

Sorry this was set on auto merge. I did consider something along those lines. I agree it does read better