tigerbeetle-node 0.17.3 → 0.17.5

package/src/test.ts

@@ -17,9 +17,22 @@ import {
   RequestError,
 } from '.'
 
+async function sleep_ms(ms: number): Promise<void> {
+  await new Promise(resolve => setTimeout(resolve, ms))
+}
+
+function range(n: number): number[] {
+  return Array.from({ length: n }, (_, i) => i);
+}
+
+function random_index(array: Array<any>): number {
+  return Math.floor(Math.random() * array.length);
+}
+
+const REPLICA_ADDRESSES = [process.env.TB_ADDRESS || '3000'];
 const client = createClient({
   cluster_id: 0n,
-  replica_addresses: [process.env.TB_ADDRESS || '3000']
+  replica_addresses: REPLICA_ADDRESSES
 })
 
 // Test data
@@ -69,7 +82,7 @@ test('id() monotonically increasing', async (): Promise<void> => {
   for (let i = 0; i < 10_000_000; i++) {
     // Ensure ID is monotonic between milliseconds if the loop executes too fast.
     if (i % 10_000 == 0) {
-      await new Promise(resolve => setTimeout(resolve, 1))
+      await sleep_ms(1)
     }
 
     const idB = id();
@@ -115,8 +128,8 @@ test('error if timestamp is not set to 0n on account', async (): Promise<void> =
 
 test('batch max size', async (): Promise<void> => {
   const BATCH_SIZE = 10_000;
-  var transfers: Transfer[] = [];
-  for (var i=0; i<BATCH_SIZE;i++) {
+  const transfers: Transfer[] = [];
+  for (let i=0; i<BATCH_SIZE;i++) {
     transfers.push({
       id: 0n,
       debit_account_id: 0n,
@@ -140,6 +153,17 @@ test('batch max size', async (): Promise<void> => {
   })
 })
 
+test('batch invalid size', async (): Promise<void> => {
+  const transfers: Transfer[] = [];
+  transfers.length = 0xffffffff;
+
+  assert.rejects(async() => await client.createTransfers(transfers), (err) => {
+    assert.ok(err instanceof RequestError)
+    assert.strictEqual(err.code,  ErrorCodes.ERR_TOO_MUCH_DATA)
+    return true
+  })
+})
+
 test('can lookup accounts', async (): Promise<void> => {
   const accounts = await client.lookupAccounts([accountA.id, accountB.id])
 
@@ -431,7 +455,7 @@ test('cannot void an expired transfer', async (): Promise<void> => {
   assert.ok(transfers_results[0].timestamp > 0)
   assert.deepStrictEqual(transfers_results[0].status, CreateTransferStatus.created)
 
-  var accounts = await client.lookupAccounts([accountA.id, accountB.id])
+  let accounts = await client.lookupAccounts([accountA.id, accountB.id])
   assert.strictEqual(accounts.length, 2)
   assert.strictEqual(accounts[0].credits_posted, 150n)
   assert.strictEqual(accounts[0].credits_pending, 50n)
@@ -446,9 +470,8 @@ test('cannot void an expired transfer', async (): Promise<void> => {
   // We need to wait 1s for the server to expire the transfer, however the
   // server can pulse the expiry operation anytime after the timeout,
   // so adding an extra delay to avoid flaky tests.
-  // TODO: Use `await setTimeout(1000)` when upgrade to Node.js > 15.
   const extra_wait_time = 500;
-  await new Promise(_ => setTimeout(_, (transfer.timeout * 1000) + extra_wait_time));
+  await sleep_ms((transfer.timeout * 1000) + extra_wait_time);
 
   // Looking up the accounts again for the updated balance.
   accounts = await client.lookupAccounts([accountA.id, accountB.id])
@@ -566,9 +589,9 @@ test('can get account transfers', async (): Promise<void> => {
   assert.ok(account_results[0].timestamp > 0)
   assert.deepStrictEqual(account_results[0].status, CreateAccountStatus.created)
 
-  var transfers_created : Transfer[] = [];
+  const transfers_created : Transfer[] = [];
   // Create transfers where the new account is either the debit or credit account:
-  for (var i=0; i<10;i++) {
+  for (let i=0; i<10;i++) {
     transfers_created.push({
       id: BigInt(i + 10000),
       debit_account_id: i % 2 == 0 ? accountC.id : accountA.id,
@@ -594,7 +617,7 @@ test('can get account transfers', async (): Promise<void> => {
   }
 
   // Query all transfers for accountC:
-  var filter: AccountFilter = {
+  let filter: AccountFilter = {
     account_id: accountC.id,
     user_data_128: 0n,
     user_data_64: 0n,
@@ -605,14 +628,14 @@ test('can get account transfers', async (): Promise<void> => {
     limit: BATCH_MAX,
     flags: AccountFilterFlags.credits | AccountFilterFlags.debits,
   }
-  var transfers = await client.getAccountTransfers(filter)
-  var account_balances = await client.getAccountBalances(filter)
+  let transfers = await client.getAccountTransfers(filter)
+  let account_balances = await client.getAccountBalances(filter)
   assert.strictEqual(transfers.length, transfers_created.length)
   assert.strictEqual(account_balances.length, transfers.length)
 
-  var timestamp = 0n;
-  var i = 0;
-  for (var transfer of transfers) {
+  let timestamp = 0n;
+  let i = 0;
+  for (const transfer of transfers) {
     assert.ok(timestamp < transfer.timestamp);
     timestamp = transfer.timestamp;
 
@@ -640,7 +663,7 @@ test('can get account transfers', async (): Promise<void> => {
 
   timestamp = 1n << 64n;
   i = 0;
-  for (var transfer of transfers) {
+  for (const transfer of transfers) {
     assert.ok(transfer.timestamp < timestamp);
     timestamp = transfer.timestamp;
 
@@ -668,7 +691,7 @@ test('can get account transfers', async (): Promise<void> => {
 
   timestamp = 1n << 64n;
   i = 0;
-  for (var transfer of transfers) {
+  for (const transfer of transfers) {
     assert.ok(transfer.timestamp < timestamp);
     timestamp = transfer.timestamp;
 
@@ -696,7 +719,7 @@ test('can get account transfers', async (): Promise<void> => {
 
   timestamp = 0n;
   i = 0;
-  for (var transfer of transfers) {
+  for (const transfer of transfers) {
     assert.ok(timestamp < transfer.timestamp);
     timestamp = transfer.timestamp;
 
@@ -723,7 +746,7 @@ test('can get account transfers', async (): Promise<void> => {
   assert.strictEqual(account_balances.length, transfers.length)
 
   i = 0;
-  for (var transfer of transfers) {
+  for (const transfer of transfers) {
     assert.ok(timestamp < transfer.timestamp);
     timestamp = transfer.timestamp;
 
@@ -769,7 +792,7 @@ test('can get account transfers', async (): Promise<void> => {
 
   timestamp = 1n << 64n;
   i = 0;
-  for (var transfer of transfers) {
+  for (const transfer of transfers) {
     assert.ok(timestamp > transfer.timestamp);
     timestamp = transfer.timestamp;
 
@@ -796,7 +819,7 @@ test('can get account transfers', async (): Promise<void> => {
   assert.strictEqual(account_balances.length, transfers.length)
 
   i = 0;
-  for (var transfer of transfers) {
+  for (const transfer of transfers) {
     assert.ok(timestamp > transfer.timestamp);
     timestamp = transfer.timestamp;
 
@@ -954,9 +977,9 @@ test('can get account transfers', async (): Promise<void> => {
 
 test('can query accounts', async (): Promise<void> => {
   {
-    var accounts : Account[] = [];
+    const accounts : Account[] = [];
     // Create transfers:
-    for (var i=0; i<10;i++) {
+    for (let i=0; i<10;i++) {
       accounts.push({
         id: id(),
         debits_pending: 0n,
@@ -986,7 +1009,7 @@ test('can query accounts', async (): Promise<void> => {
     // Querying accounts where:
     // `user_data_128=1000 AND user_data_64=100 AND user_data_32=10
     // AND code=999 AND ledger=1 ORDER BY timestamp ASC`.
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 1000n,
       user_data_64: 100n,
       user_data_32: 10,
@@ -997,11 +1020,11 @@ test('can query accounts', async (): Promise<void> => {
       limit: BATCH_MAX,
       flags: QueryFilterFlags.none,
     }
-    var query: Account[] = await client.queryAccounts(filter)
+    const query: Account[] = await client.queryAccounts(filter)
     assert.strictEqual(query.length, 5)
 
-    var timestamp = 0n;
-    for (var account of query) {
+    let timestamp = 0n;
+    for (const account of query) {
       assert.ok(timestamp < account.timestamp);
       timestamp = account.timestamp;
 
@@ -1017,7 +1040,7 @@ test('can query accounts', async (): Promise<void> => {
     // Querying accounts where:
     // `user_data_128=2000 AND user_data_64=200 AND user_data_32=20
     // AND code=999 AND ledger=1 ORDER BY timestamp DESC`.
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 2000n,
       user_data_64: 200n,
       user_data_32: 20,
@@ -1028,11 +1051,11 @@ test('can query accounts', async (): Promise<void> => {
       limit: BATCH_MAX,
       flags: QueryFilterFlags.reversed,
     }
-    var query: Account[] = await client.queryAccounts(filter)
+    const query: Account[] = await client.queryAccounts(filter)
     assert.strictEqual(query.length, 5)
 
-    var timestamp = 1n << 64n;
-    for (var account of query) {
+    let timestamp = 1n << 64n;
+    for (const account of query) {
       assert.ok(timestamp > account.timestamp);
       timestamp = account.timestamp;
 
@@ -1047,7 +1070,7 @@ test('can query accounts', async (): Promise<void> => {
   {
     // Querying accounts where:
     // `code=999 ORDER BY timestamp ASC`
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 0n,
       user_data_64: 0n,
       user_data_32: 0,
@@ -1058,11 +1081,11 @@ test('can query accounts', async (): Promise<void> => {
       limit: BATCH_MAX,
       flags: QueryFilterFlags.none,
     }
-    var query: Account[] = await client.queryAccounts(filter)
+    const query: Account[] = await client.queryAccounts(filter)
     assert.strictEqual(query.length, 10)
 
-    var timestamp = 0n;
-    for (var account of query) {
+    let timestamp = 0n;
+    for (const account of query) {
       assert.ok(timestamp < account.timestamp);
       timestamp = account.timestamp;
 
@@ -1073,7 +1096,7 @@ test('can query accounts', async (): Promise<void> => {
   {
     // Querying accounts where:
     // `code=999 ORDER BY timestamp DESC LIMIT 5`.
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 0n,
       user_data_64: 0n,
       user_data_32: 0,
@@ -1086,11 +1109,11 @@ test('can query accounts', async (): Promise<void> => {
     }
 
     // First 5 items:
-    var query: Account[] = await client.queryAccounts(filter)
+    let query: Account[] = await client.queryAccounts(filter)
     assert.strictEqual(query.length, 5)
 
-    var timestamp = 1n << 64n;
-    for (var account of query) {
+    let timestamp = 1n << 64n;
+    for (const account of query) {
       assert.ok(timestamp > account.timestamp);
       timestamp = account.timestamp;
 
@@ -1102,7 +1125,7 @@ test('can query accounts', async (): Promise<void> => {
     query = await client.queryAccounts(filter)
     assert.strictEqual(query.length, 5)
 
-    for (var account of query) {
+    for (const account of query) {
       assert.ok(timestamp > account.timestamp);
       timestamp = account.timestamp;
 
@@ -1117,7 +1140,7 @@ test('can query accounts', async (): Promise<void> => {
 
   {
     // Not found:
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 0n,
       user_data_64: 200n,
       user_data_32: 10,
@@ -1128,7 +1151,7 @@ test('can query accounts', async (): Promise<void> => {
       limit: BATCH_MAX,
       flags: QueryFilterFlags.none,
     }
-    var query: Account[] = await client.queryAccounts(filter)
+    const query: Account[] = await client.queryAccounts(filter)
     assert.strictEqual(query.length, 0)
   }
 })
@@ -1155,9 +1178,9 @@ test('can query transfers', async (): Promise<void> => {
     assert.ok(account_results[0].timestamp > 0)
     assert.deepStrictEqual(account_results[0].status, CreateAccountStatus.created)
 
-    var transfers_created : Transfer[] = [];
+    const transfers_created : Transfer[] = [];
     // Create transfers:
-    for (var i=0; i<10;i++) {
+    for (let i=0; i<10;i++) {
       transfers_created.push({
         id: id(),
         debit_account_id: i % 2 == 0 ? account.id : accountA.id,
@@ -1187,7 +1210,7 @@ test('can query transfers', async (): Promise<void> => {
     // Querying transfers where:
     // `user_data_128=1000 AND user_data_64=100 AND user_data_32=10
     // AND code=999 AND ledger=1 ORDER BY timestamp ASC`.
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 1000n,
       user_data_64: 100n,
       user_data_32: 10,
@@ -1198,11 +1221,11 @@ test('can query transfers', async (): Promise<void> => {
       limit: BATCH_MAX,
       flags: QueryFilterFlags.none,
     }
-    var query: Transfer[] = await client.queryTransfers(filter)
+    const query: Transfer[] = await client.queryTransfers(filter)
     assert.strictEqual(query.length, 5)
 
-    var timestamp = 0n;
-    for (var transfer of query) {
+    let timestamp = 0n;
+    for (const transfer of query) {
       assert.ok(timestamp < transfer.timestamp);
       timestamp = transfer.timestamp;
 
@@ -1218,7 +1241,7 @@ test('can query transfers', async (): Promise<void> => {
     // Querying transfers where:
     // `user_data_128=2000 AND user_data_64=200 AND user_data_32=20
     // AND code=999 AND ledger=1 ORDER BY timestamp DESC`.
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 2000n,
       user_data_64: 200n,
       user_data_32: 20,
@@ -1229,11 +1252,11 @@ test('can query transfers', async (): Promise<void> => {
       limit: BATCH_MAX,
       flags: QueryFilterFlags.reversed,
     }
-    var query: Transfer[] = await client.queryTransfers(filter)
+    const query: Transfer[] = await client.queryTransfers(filter)
     assert.strictEqual(query.length, 5)
 
-    var timestamp = 1n << 64n;
-    for (var transfer of query) {
+    let timestamp = 1n << 64n;
+    for (const transfer of query) {
       assert.ok(timestamp > transfer.timestamp);
       timestamp = transfer.timestamp;
 
@@ -1248,7 +1271,7 @@ test('can query transfers', async (): Promise<void> => {
   {
     // Querying transfers where:
     // `code=999 ORDER BY timestamp ASC`
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 0n,
       user_data_64: 0n,
       user_data_32: 0,
@@ -1259,11 +1282,11 @@ test('can query transfers', async (): Promise<void> => {
       limit: BATCH_MAX,
       flags: QueryFilterFlags.none,
     }
-    var query: Transfer[] = await client.queryTransfers(filter)
+    const query: Transfer[] = await client.queryTransfers(filter)
     assert.strictEqual(query.length, 10)
 
-    var timestamp = 0n;
-    for (var transfer of query) {
+    let timestamp = 0n;
+    for (const transfer of query) {
       assert.ok(timestamp < transfer.timestamp);
       timestamp = transfer.timestamp;
 
@@ -1274,7 +1297,7 @@ test('can query transfers', async (): Promise<void> => {
   {
     // Querying transfers where:
     // `code=999 ORDER BY timestamp DESC LIMIT 5`.
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 0n,
       user_data_64: 0n,
       user_data_32: 0,
@@ -1287,11 +1310,11 @@ test('can query transfers', async (): Promise<void> => {
     }
 
     // First 5 items:
-    var query: Transfer[] = await client.queryTransfers(filter)
+    let query: Transfer[] = await client.queryTransfers(filter)
     assert.strictEqual(query.length, 5)
 
-    var timestamp = 1n << 64n;
-    for (var transfer of query) {
+    let timestamp = 1n << 64n;
+    for (const transfer of query) {
       assert.ok(timestamp > transfer.timestamp);
       timestamp = transfer.timestamp;
 
@@ -1303,7 +1326,7 @@ test('can query transfers', async (): Promise<void> => {
     query = await client.queryTransfers(filter)
     assert.strictEqual(query.length, 5)
 
-    for (var transfer of query) {
+    for (const transfer of query) {
       assert.ok(timestamp > transfer.timestamp);
       timestamp = transfer.timestamp;
 
@@ -1318,7 +1341,7 @@ test('can query transfers', async (): Promise<void> => {
 
   {
     // Not found:
-    var filter: QueryFilter = {
+    const filter: QueryFilter = {
       user_data_128: 0n,
       user_data_64: 200n,
       user_data_32: 10,
@@ -1329,7 +1352,7 @@ test('can query transfers', async (): Promise<void> => {
       limit: BATCH_MAX,
       flags: QueryFilterFlags.none,
     }
-    var query: Transfer[] = await client.queryTransfers(filter)
+    const query: Transfer[] = await client.queryTransfers(filter)
     assert.strictEqual(query.length, 0)
   }
 })
@@ -1461,7 +1484,7 @@ test('can import accounts and transfers', async (): Promise<void> => {
 
   // Wait 10 ms so we can use the account's timestamp as the reference for past time
   // after the last object inserted.
-  await new Promise(_ => setTimeout(_, 10));
+  await sleep_ms(10);
 
   const accountA: Account = {
     id: id(),
@@ -1552,14 +1575,48 @@ test('accept zero-length lookup_transfers', async (): Promise<void> => {
 })
 
 test("destroy client in-flight", async (): Promise<void> => {
-  // Non-existing cluster.
-  const client = createClient({ cluster_id: 92n, replica_addresses: ["99"] });
-  setTimeout(() => client.destroy(), 30);
-  assert.rejects(async () => await client.lookupAccounts([0n]), (err) => {
-    assert.ok(err instanceof RequestError)
-    assert.strictEqual(err.code,  ErrorCodes.ERR_CLIENT_CLOSED)
-    return true
-  })
+  const client_count = 5;
+  const action_count = 50;
+
+  const clients = range(client_count).map(() =>
+    createClient({
+      cluster_id: 0n,
+      replica_addresses: REPLICA_ADDRESSES,
+    })
+  );
+
+  const ids: Array<bigint> = [];
+  const actions = range(action_count).map(() => async () => {
+    await sleep_ms(Math.random() < 0.2 ? 0 : Math.random());
+    const client = clients[random_index(clients)];
+    if (Math.random() < 0.1) {
+      client.destroy();
+      return;
+    }
+    if (Math.random() < 0.7) {
+      const id_new = id();
+      ids.push(id_new);
+      try {
+        await client.createAccounts([{ ...accountA, id: id_new }]);
+      } catch (err) {
+        assert.ok(err instanceof RequestError);
+        assert.strictEqual(err.code, ErrorCodes.ERR_CLIENT_CLOSED);
+      }
+      return;
+    }
+    try {
+      const id_lookup = (Math.random() < 0.2 || ids.length == 0)
+        ? BigInt(Math.floor(Math.random() * 10000))
+        : ids[random_index(ids)];
+      await client.lookupAccounts([id_lookup]);
+    } catch (err) {
+        assert.ok(err instanceof RequestError);
+        assert.strictEqual(err.code, ErrorCodes.ERR_CLIENT_CLOSED);
+    }
+  });
+
+  await Promise.all(actions.map((f) => f()));
+  for (const client of clients) client.destroy();
 });
 
 async function main () {