trac-msb 0.2.8 → 0.2.10

package/tests/integration/apply/addWriter.test.js

@@ -1,245 +0,0 @@
-import b4a from 'b4a';
-import { test, hook } from '../../helpers/wrapper.js';
-
-import {
-    setupMsbAdmin,
-    setupMsbPeer,
-    setupWhitelist,
-    fundPeer,
-    initTemporaryDirectory,
-    removeTemporaryDirectory,
-    randomBytes,
-    setupMsbIndexer,
-    setupMsbWriter,
-    tryToSyncWriters,
-    waitForNodeState,
-    waitWritable
-} from '../../helpers/setupApplyTests.js';
-import {
-    testKeyPair1,
-    testKeyPair2,
-    testKeyPair3,
-    testKeyPair4,
-    testKeyPair5,
-    testKeyPair6
-} from '../../fixtures/apply.fixtures.js';
-import PartialStateMessageOperations from "../../../src/messages/partialStateMessages/PartialStateMessageOperations.js";
-import CompleteStateMessageOperations from '../../../src/messages/completeStateMessages/CompleteStateMessageOperations.js';
-import {ZERO_WK} from '../../../src/utils/buffer.js';
-import { $TNK } from '../../../src/core/state/utils/balance.js';
-import { config } from '../../helpers/config.js';
-
-const sendAddWriter = async (invoker, broadcaster) => {
-    const validity = await invoker.msb.state.getIndexerSequenceState()
-    const req = await new PartialStateMessageOperations(invoker.wallet, config)
-        .assembleAddWriterMessage(
-            b4a.toString(invoker.msb.state.writingKey, 'hex'),
-            b4a.toString(validity, 'hex'));
-
-    const raw = await new CompleteStateMessageOperations(broadcaster.wallet, config)
-        .assembleAddWriterMessage(
-            req.address,
-            b4a.from(req.rao.tx, 'hex'),
-            b4a.from(req.rao.txv, 'hex'),
-            b4a.from(req.rao.iw, 'hex'),
-            b4a.from(req.rao.in, 'hex'),
-            b4a.from(req.rao.is, 'hex')
-        )
-    return await broadcaster.msb.state.append(raw)
-}
-
-let admin, writer1, writer2, writer3, writer4, indexer1, tmpDirectory;
-
-hook('Initialize nodes for addWriter tests', async t => {
-    const randomChannel = randomBytes(32).toString('hex');
-    const baseOptions = {
-        enableTxApplyLogs: false,
-        enableInteractiveMode: false,
-        enableRoleRequester: false,
-        channel: randomChannel,
-        enableValidatorObserver: false
-    }
-    tmpDirectory = await initTemporaryDirectory();
-    admin = await setupMsbAdmin(testKeyPair1, tmpDirectory, baseOptions);
-    writer1 = await setupMsbPeer('writer1', testKeyPair2, tmpDirectory, admin.options); // just a peer, not a writer yet
-    await fundPeer(admin, writer1, $TNK(10n))
-    writer2 = await setupMsbPeer('writer2', testKeyPair3, tmpDirectory, admin.options); // just a peer, not a writer yet
-    await fundPeer(admin, writer2, $TNK(10n))
-    writer3 = await setupMsbPeer('writer3', testKeyPair4, tmpDirectory, admin.options); // just a peer, not a writer yet
-    await fundPeer(admin, writer3, $TNK(10n))
-    writer4 = await setupMsbPeer('writer4', testKeyPair5, tmpDirectory, admin.options); // just a peer, not a writer yet
-    indexer1 = await setupMsbWriter(admin, 'indexer1', testKeyPair6, tmpDirectory, admin.options); // writer and cannot be funded
-
-    const whitelistKeys = [writer1.wallet.address, writer2.wallet.address, writer3.wallet.address, indexer1.wallet.address];
-    await setupWhitelist(admin, whitelistKeys);
-    indexer1 = await setupMsbIndexer(indexer1, admin);
-});
-
-test('handleApplyAddWriterOperation (apply) - Append addWriter payload into the base - happy path for peer', async (t) => {
-    // writer1 is not a writer yet, but it is whitelisted -> It will become a writer after the operation.
-    // writer2 is not a writer yet, but it is whitelisted.
-    // writer3 is not a writer yet, but it is whitelisted.
-    // writer4 is reader
-    // indexer1 is already an indexer.
-    await admin.msb.state.append(null);
-    await waitWritable(admin, writer1, async () => await sendAddWriter(writer1, admin))
-    await waitForNodeState(admin, writer1.wallet.address, {
-        wk: writer1.msb.state.writingKey,
-        isWhitelisted: true,
-        isWriter: true,
-        isIndexer: false
-    });
-    await tryToSyncWriters(writer1, indexer1);
-
-    const result = await admin.msb.state.getNodeEntry(writer1.wallet.address); // check if the writer entry was added successfully in the base
-
-    t.ok(result, 'Result should not be null');
-    t.ok(b4a.equals(result.wk, writer1.msb.state.writingKey), 'Result writing key should match writer writing key');
-    t.ok(result.isWriter, 'Result should indicate that the peer is a valid writer');
-    t.is(result.isIndexer, false, 'Result should not indicate that the peer is an indexer');
-});
-
-test('handleApplyAddWriterOperation (apply) - Append addWriter payload into the base - doesnt change the signed length', async (t) => {
-    // writer1 is already a writer.
-    // writer2 is not a writer yet, but it is whitelisted -> It will become a writer after the operation.
-    // writer3 is not a writer yet, but it is whitelisted.
-    // writer4 is reader
-    // indexer1 is already an indexer.
-    await admin.msb.state.append(null);
-    await waitWritable(admin, writer2, async () => await sendAddWriter(writer2, admin))
-    await tryToSyncWriters(writer2, admin, indexer1);
-
-    await waitForNodeState(writer2, writer2.wallet.address, {
-        wk: writer2.msb.state.writingKey,
-        isWhitelisted: true,
-        isWriter: true,
-        isIndexer: false
-    });
-
-    const signedLengthAdminBefore = admin.msb.state.getSignedLength();
-    const signedLengthWriter2Before = writer2.msb.state.getSignedLength();
-
-    await sendAddWriter(writer2, admin)
-
-    await tryToSyncWriters(admin, writer2, indexer1);
-    await waitForNodeState(writer2, writer2.wallet.address, {
-        wk: writer2.msb.state.writingKey,
-        isWhitelisted: true,
-        isWriter: true,
-        isIndexer: false
-    });
-
-    const signedLengthAdminAfter = admin.msb.state.getSignedLength();
-    const signedLengthWriter2After = writer2.msb.state.getSignedLength();
-
-    t.is(signedLengthAdminBefore, signedLengthAdminAfter, 'Admin signed length should not change');
-    t.is(signedLengthWriter2Before, signedLengthWriter2After, 'Writer2 signed length should not change');
-});
-
-test('handleApplyAddWriterOperation (apply) - Append addWriter payload into the base - ZERO_WK', async (t) => {
-    // writer1 is already a writer.
-    // writer2 is already a writer.
-    // writer3 is not a writer yet, but it is whitelisted and won't become a writer.
-    // writer4 is reader
-    // indexer1 is already an indexer.
-    const signedLengthAdminBefore = admin.msb.state.getSignedLength();
-    const signedLengthWriter1Before = writer1.msb.state.getSignedLength();
-
-    const validity = await writer3.msb.state.getIndexerSequenceState()
-    const req = await new PartialStateMessageOperations(writer3.wallet, config)
-        .assembleAddWriterMessage(
-            b4a.toString(ZERO_WK, 'hex'),
-            b4a.toString(validity, 'hex'));
-
-    const raw = await new CompleteStateMessageOperations(admin.wallet, config)
-        .assembleAddWriterMessage(
-            admin.wallet.address,
-            b4a.from(req.rao.tx, 'hex'),
-            b4a.from(req.rao.txv, 'hex'),
-            b4a.from(req.rao.iw, 'hex'),
-            b4a.from(req.rao.in, 'hex'),
-            b4a.from(req.rao.is, 'hex')
-        )
-    await admin.msb.state.append(raw)
-    await tryToSyncWriters(writer3, admin, writer1, writer2, indexer1);
-
-    const result = await writer1.msb.state.getNodeEntry(writer3.wallet.address);
-    const signedLengthAdminAfter = admin.msb.state.getSignedLength();
-    const signedLengthWriter1After = writer1.msb.state.getSignedLength();
-
-    t.is(signedLengthAdminBefore, signedLengthAdminAfter, 'admin signed length should not change');
-    t.is(signedLengthWriter1Before, signedLengthWriter1After, 'Writer1 signed length should not change');
-    t.ok(result, 'Result should not be null');
-    t.ok(!b4a.equals(result.wk, writer3.msb.state.writingKey), 'Result writing key should not match writer writing key');
-    t.ok(b4a.equals(result.wk, ZERO_WK), 'Result writing key should be ZERO_WK');
-    t.is(result.isWriter, false, 'Result should indicate that the peer is not a writer');
-    //t.is(writer3.msb.state.isWritable(), false, 'peer should not be writable');
-});
-
-test('handleApplyAddWriterOperation (apply) - Append addWriter payload into the base - node is already an indexer', async (t) => {
-    // writer1 is already a writer.
-    // writer2 is already a writer.
-    // writer3 is not a writer, but it is whitelisted.
-    // writer4 is reader
-    // indexer1 is already an indexer.
-    const resultBefore = await admin.msb.state.getNodeEntry(indexer1.wallet.address);
-    await indexer1.msb.state.append(null);
-    const signedLengthAdminBefore = admin.msb.state.getSignedLength();
-    const signedLengthIndexer1Before = indexer1.msb.state.getSignedLength();
-
-    await sendAddWriter(indexer1, admin)
-    await tryToSyncWriters(indexer1, admin, writer1, writer2);
-
-    const result = await admin.msb.state.getNodeEntry(indexer1.wallet.address); // check if the writer entry was added successfully in the base
-    const signedLengthAdminAfter = admin.msb.state.getSignedLength();
-    const signedLengthIndexer1After = indexer1.msb.state.getSignedLength();
-
-    t.is(signedLengthAdminBefore, signedLengthAdminAfter, 'Admin signed length should not change');
-    t.is(signedLengthIndexer1Before, signedLengthIndexer1After, 'Indexer1 signed length should not change');
-    t.ok(result, 'Result should not be null');
-    t.ok(b4a.equals(result.wk, indexer1.msb.state.writingKey), 'Result writing key should match writer writing key');
-    t.is(result.isWriter, true, 'Result should indicate that the peer is a writer');
-    t.is(result.isIndexer, true, 'Result should indicate that the peer is an indexer');
-    t.is(indexer1.msb.state.isWritable(), true, 'peer should still be writable');
-    t.is(indexer1.msb.state.isIndexer(), true, 'peer should still be an indexer');
-});
-
-test('handleApplyAddWriterOperation (apply) - Append addWriter payload into the base - non-whitelisted peer', async (t) => {
-    // writer1 is already a writer
-    // writer2 is already a writer
-    // writer3 is already a writer
-    // writer4 is reader -> this node will not become a writer.
-    // indexer1 is already an indexer
-    await writer4.msb.state.append(null);
-    const signedLengthAdminBefore = admin.msb.state.getSignedLength();
-
-    await sendAddWriter(writer4, admin)
-    await tryToSyncWriters(writer4, writer1, writer2, writer3, indexer1);
-
-    const result = await writer2.msb.state.getNodeEntry(writer4.wallet.address); // check if the writer entry was added successfully in the base
-    const signedLengthAdminAfter = admin.msb.state.getSignedLength();
-
-    t.is(signedLengthAdminBefore, signedLengthAdminAfter, 'Admin signed length should not change');
-    t.is(writer4.msb.state.isWritable(), false, 'peer should not be writable');
-    t.is(writer4.msb.state.isIndexer(), false, 'peer should not be an indexer');
-    t.is(result, null, 'Result should be null');
-});
-
-test('handleApplyAddWriterOperation (apply) - validator and invoker are the same', async (t) => {
-    // TODO: Implement when apply tests are fixed
-    t.pass('Skipping test: Placeholder. To be implemented later.');
-});
-
-hook('Clean up addWriter setup', async t => {
-    const toClose = []
-    // close msb instances and remove temp directory
-    if (admin?.msb) toClose.push(admin.msb.close());
-    if (writer1?.msb) toClose.push(writer1.msb.close());
-    if (writer2?.msb) toClose.push(writer2.msb.close());
-    if (writer3?.msb) toClose.push(writer3.msb.close());
-    if (writer4?.msb) toClose.push(writer4.msb.close());
-    if (indexer1?.msb) toClose.push(indexer1.msb.close());
-
-    await Promise.all(toClose)
-    if (tmpDirectory) await removeTemporaryDirectory(tmpDirectory);
-})

package/tests/integration/apply/apply.test.js

@@ -1,19 +0,0 @@
-import {default as test} from 'brittle';
-
-async function runCheckTests() {
-    test.pause();
-    await import('./addAdmin/addAdminBasic.test.js');
-    await import('./addAdmin/addAdminRecovery.test.js');
-    await import('./addWhitelist.test.js');
-    await import('./addWriter.test.js');
-    await import('./removeWriter.test.js');
-    await import('./addIndexer.test.js');
-    await import('./removeIndexer.test.js');
-    await import('./postTx/postTx.test.js');
-    await import('./transfer.test.js');
-    await import('./postTx/invalidSubValues.test.js');
-    await import('./banValidator.test.js')
-    test.resume();
-}
-
-await runCheckTests();

package/tests/integration/apply/banValidator.test.js

@@ -1,116 +0,0 @@
-import { test, hook } from '../../helpers/wrapper.js';
-
-import {
-    initTemporaryDirectory,
-    removeTemporaryDirectory,
-    setupMsbWriter,
-    setupMsbIndexer,
-    setupMsbAdmin,
-    tryToSyncWriters
-} from '../../helpers/setupApplyTests.js';
-import { randomBytes } from '../../helpers/setupApplyTests.js';
-import CompleteStateMessageOperations from '../../../src/messages/completeStateMessages/CompleteStateMessageOperations.js';
-
-import { testKeyPair1, testKeyPair2, testKeyPair3, testKeyPair4 } from '../../fixtures/apply.fixtures.js';
-import { sleep } from '../../../src/utils/helpers.js';
-import b4a from 'b4a'
-import { config } from '../../helpers/config.js';
-
-let admin;
-let indexer, writer1, writer2;
-let tmpDirectory;
-
-hook('Initialize nodes for banValidator tests', async () => {
-    const randomChannel = randomBytes(32).toString('hex');
-    const baseOptions = {
-        enableTxApplyLogs: false,
-        enableInteractiveMode: false,
-        enableRoleRequester: false,
-        channel: randomChannel,
-        enableValidatorObserver: false,
-    }
-    tmpDirectory = await initTemporaryDirectory();
-    admin = await setupMsbAdmin(testKeyPair1, tmpDirectory, baseOptions);
-
-    indexer = await setupMsbWriter(admin, 'indexer', testKeyPair2, tmpDirectory, admin.options);
-    indexer = await setupMsbIndexer(indexer, admin);
-
-    writer1 = await setupMsbWriter(admin, 'writer1', testKeyPair3, tmpDirectory, admin.options);
-    writer2 = await setupMsbWriter(admin, 'writer2', testKeyPair4, tmpDirectory, admin.options);
-});
-
-test('handleApplyBanValidatorOperation (apply) - Append banValidator payload - ban indexer', async t => {
-    const validity = await admin.msb.state.getIndexerSequenceState()
-    const assembledBanWriter = await new CompleteStateMessageOperations(admin.wallet, config)
-        .assembleBanWriterMessage(indexer.wallet.address, validity);
-    await admin.msb.state.append(assembledBanWriter);
-    await tryToSyncWriters(admin, indexer, writer1, writer2);
-
-    const indexersEntry = await indexer.msb.state.getIndexersEntry();
-    const nodeInfo = await indexer.msb.state.getNodeEntry(indexer.wallet.address);
-
-    t.is(indexersEntry.length, 2, 'Indexers entry count should be 2');
-    t.is(!!indexersEntry.find(({ key }) => b4a.equals(key, indexer.msb.state.writingKey)), true, 'Indexer address should be still included in the indexers entry');
-    t.is(nodeInfo.isIndexer, true, 'Node info should indicate that the node is still an indexer');
-});
-
-test('handleApplyBanValidatorOperation (apply) - Append banValidator payload into the base by non-admin node', async t => {
-    const validity = await admin.msb.state.getIndexerSequenceState()
-    const assembledBanWriter = await new CompleteStateMessageOperations(writer1.wallet, config)
-        .assembleBanWriterMessage(writer2.wallet.address, validity);
-    await writer1.msb.state.append(assembledBanWriter);
-    await sleep(5000); // wait for both peers to sync state
-    await tryToSyncWriters(admin);
-
-    const nodeInfo = await writer2.msb.state.getNodeEntry(writer2.wallet.address);
-
-    t.is(nodeInfo.isWriter, true, 'Node info should indicate that the node is still a writer');
-    t.is(nodeInfo.isWhitelisted, true, 'Node info should indicate that the node is still whitelisted');
-});
-
-
-test('handleApplyBanValidatorOperation (apply) - Append banValidator payload into the base - happy path', async t => {
-    const validity = await admin.msb.state.getIndexerSequenceState()
-    const assembledBanWriter = await new CompleteStateMessageOperations(admin.wallet, config)
-        .assembleBanWriterMessage(writer1.wallet.address, validity);
-    await admin.msb.state.append(assembledBanWriter);
-    await sleep(5000); // wait for both peers to sync state
-
-    const nodeInfo = await writer1.msb.state.getNodeEntry(writer1.wallet.address);
-
-    t.is(nodeInfo.isWriter, false, 'Node info should indicate that the node is not a writer anymore');
-    t.is(nodeInfo.isWhitelisted, false, 'Node info should indicate that the node is not whitelisted anymore');
-    t.is(writer1.msb.state.isWritable(), false, 'Writer1 should not be a writer anymore');
-});
-
-test('handleApplyBanValidatorOperation (apply) - Append banValidator payload into the base - idempotence', async t => {
-    const validity = await admin.msb.state.getIndexerSequenceState()
-    const assembledBanWriter = await new CompleteStateMessageOperations(admin.wallet, config)
-        .assembleBanWriterMessage(writer2.wallet.address, validity);
-    await admin.msb.state.append(assembledBanWriter);
-    await sleep(5000); // wait for both peers to sync state
-
-    const nodeInfo = await writer2.msb.state.getNodeEntry(writer2.wallet.address);
-
-    const validity2 = await admin.msb.state.getIndexerSequenceState()
-    const assembledBanWriter2 = await new CompleteStateMessageOperations(admin.wallet, config)
-        .assembleBanWriterMessage(writer2.wallet.address, validity2);
-    await admin.msb.state.append(assembledBanWriter2);
-    await sleep(5000); // wait for both peers to sync state
-
-    t.is(nodeInfo.isIndexer, false, 'Node info should indicate that the node is not a writer anymore');
-    t.is(nodeInfo.isWriter, false, 'Node info should indicate that the node is not a writer anymore');
-    t.is(nodeInfo.isWhitelisted, false, 'Node info should indicate that the node is not whitelisted anymore');
-    t.is(writer1.msb.state.isWritable(), false, 'Writer2 should not be a writer anymore');
-    t.is(writer1.msb.state.isIndexer(), false, 'Writer2 should not be a writer anymore');
-});
-
-hook('Clean up banValidator setup', async t => {
-    const toClose = [];
-    if (admin?.msb) toClose.push(admin.msb.close());
-    if (indexer?.msb) toClose.push(indexer.msb.close());
-    if (writer1?.msb) toClose.push(writer1.msb.close());
-    if (writer2?.msb) toClose.push(writer2.msb.close());
-    await Promise.all(toClose);
-    if (tmpDirectory) await removeTemporaryDirectory(tmpDirectory);
-});

package/tests/integration/apply/postTx/invalidSubValues.test.js

@@ -1,103 +0,0 @@
-import {hook, test} from '../../../helpers/wrapper.js';
-import {
-    generatePostTx,
-    initTemporaryDirectory,
-    randomBytes,
-    removeTemporaryDirectory,
-    setupMsbAdmin,
-    setupMsbPeer,
-    setupMsbWriter,
-    fundPeer,
-    deployExternalBootstrap,
-    tryToSyncWriters,
-    waitDemotion,
-    promoteToWriter
-} from '../../../helpers/setupApplyTests.js';
-import {safeDecodeApplyOperation, safeEncodeApplyOperation} from '../../../../src/utils/protobuf/operationHelpers.js'
-import {testKeyPair1, testKeyPair2, testKeyPair4, testKeyPair5} from '../../../fixtures/apply.fixtures.js';
-import { $TNK } from '../../../../src/core/state/utils/balance.js';
-import { decode as decodeNodeEntry } from '../../../../src/core/state/utils/nodeEntry.js';
-
-let tmpDirectory, admin, writer, externalNode, externalBootstrap, maliciousPeer;
-// is and vs is already covered
-const testCases = [
-  { name: 'modified tx', mod: (txPayload, maliciousValue) => { return { ...txPayload.txo, tx: maliciousValue }; } },
-  { name: 'modified incoming writingKey', mod: (txPayload, maliciousValue) => { return { ...txPayload.txo, iw: maliciousValue }; } },
-  { name: 'modified incoming nonce', mod: (txPayload, maliciousValue) => { return { ...txPayload.txo, in: maliciousValue }; } },
-  { name: 'modified content hash', mod: (txPayload, maliciousValue) => { return { ...txPayload.txo, ch: maliciousValue }; } },
-  { name: 'modified external bootstrap', mod: (txPayload, maliciousValue) => { return { ...txPayload.txo, bs: maliciousValue }; } },
-  { name: 'modified MSB bootstrap', mod: (txPayload, maliciousValue) => { return { ...txPayload.txo, mbs: maliciousValue }; } },
-  { name: 'modified validator nonce', mod: (txPayload, maliciousValue) => { return { ...txPayload.txo, vn: maliciousValue }; } },
-];
-
-const close = async node => {
-    await node.msb.close()
-}
-
-hook('Initialize nodes', async t => {
-    const randomChannel = randomBytes(32).toString('hex');
-
-    const baseOptions = {
-        enableTxApplyLogs: false,
-        enableInteractiveMode: false,
-        enableRoleRequester: false,
-        enableValidatorObserver: false,
-        channel: randomChannel,
-    }
-
-    tmpDirectory = await initTemporaryDirectory();
-    admin = await setupMsbAdmin(testKeyPair1, tmpDirectory, baseOptions);
-    writer = await setupMsbWriter(admin, 'writer1', testKeyPair2, tmpDirectory, admin.options);
-    externalNode = await setupMsbPeer('reader1', testKeyPair4, tmpDirectory, admin.options);
-    await fundPeer(admin, externalNode, $TNK(1n)) // we fund it since it will deploy stuff
-    externalBootstrap = await deployExternalBootstrap(writer, externalNode)
-    await tryToSyncWriters(writer, admin)
-    maliciousPeer = await setupMsbPeer('maliciousWriter', testKeyPair5, tmpDirectory, admin.options);
-    await fundPeer(admin, maliciousPeer, $TNK(10n))
-});
-
-test('handleApplyTxOperation (apply) - invalid txo sub-values', async t => {
-  const {postTx, txHash} = await generatePostTx(writer, externalNode, externalBootstrap)
-  const decodedPostTx = safeDecodeApplyOperation(postTx);
-  
-  for (let i = 0; i < testCases.length; i++) {
-    const testCase = testCases[i]
-
-    const maliciousWriter = await promoteToWriter(admin, maliciousPeer)
-    // all of these values are buffers 32 bytes long
-    const maliciousValue = randomBytes(32);
-
-    const modifiedTxo = testCase.mod(decodedPostTx, maliciousValue);
-    const modifiedPostTx = {
-        ...decodedPostTx,
-        txo: modifiedTxo
-    };
-    const encodedMaliciousPostTx = safeEncodeApplyOperation(modifiedPostTx);
-
-    await waitDemotion(maliciousWriter, async () => {
-        await maliciousWriter.msb.state.append(encodedMaliciousPostTx);
-    })
-
-    if (testCase.name === 'modified tx') {
-        const result = await admin.msb.state.get(maliciousValue.toString('hex'));
-        t.absent(result, `post tx with ${testCase.name} should not be added to the base`);
-    } else {
-        const result = await admin.msb.state.get(txHash);
-        t.absent(result, `post tx with ${testCase.name} should not be added to the base`);
-    }
-
-    // should be penalized
-    const node = decodeNodeEntry(await maliciousWriter.msb.state.get(maliciousWriter.wallet.address))
-    t.ok(node, 'Result should not be null');
-    t.is(node.isWriter, false, 'Result should indicate that the peer is not a writer');
-  }
-});
-
-hook('Clean up postTx setup', async t => {
-  // close msbBoostrap and remove temp directory
-  if (writer?.msb) await close(writer)
-  if (externalNode?.msb) await close(externalNode)
-  if (admin?.msb) await close(admin)
-  if (maliciousPeer?.msb) await close(maliciousPeer)
-  if (tmpDirectory) await removeTemporaryDirectory(tmpDirectory);
-});