Skip to content

Commit

Permalink
fix: batch insert length assertion (#2042)
Browse files Browse the repository at this point in the history
* fix: batch insert length assertion

* build: upgrade docker-compose

* build: use docker compose
  • Loading branch information
rafaelcr authored Aug 5, 2024
1 parent b1b32c2 commit fe720d0
Show file tree
Hide file tree
Showing 5 changed files with 22 additions and 10 deletions.
22 changes: 17 additions & 5 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@
"@typescript-eslint/eslint-plugin": "5.46.1",
"@typescript-eslint/parser": "5.51.0",
"concurrently": "7.3.0",
"docker-compose": "0.23.14",
"docker-compose": "0.24.8",
"eslint": "8.29.0",
"eslint-plugin-prettier": "4.2.1",
"eslint-plugin-tsdoc": "0.2.17",
Expand Down
4 changes: 2 additions & 2 deletions src/datastore/pg-write-store.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1074,7 +1074,7 @@ export class PgWriteStore extends PgStore {
DO UPDATE
SET balance = ft_balances.balance + EXCLUDED.balance
`;
assert(res.count === values.length, `Expecting ${values.length} inserts, got ${res.count}`);
assert(res.count === batch.length, `Expecting ${batch.length} inserts, got ${res.count}`);
}
}

Expand Down Expand Up @@ -1119,7 +1119,7 @@ export class PgWriteStore extends PgStore {
DO UPDATE
SET balance = ft_balances.balance + EXCLUDED.balance
`;
assert(res.count === values.length, `Expecting ${values.length} inserts, got ${res.count}`);
assert(res.count === batch.length, `Expecting ${batch.length} inserts, got ${res.count}`);
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import {
import { StacksTestnet } from '@stacks/network';
import * as fs from 'fs';
import { StacksCoreRpcClient, getCoreNodeEndpoint } from '../core-rpc/client';
import * as compose from 'docker-compose';
import { v2 as compose } from 'docker-compose';
import * as path from 'path';
import { PgWriteStore } from '../datastore/pg-write-store';
import { NonceJar, migrate, standByForTxSuccess } from '../test-utils/test-helpers';
Expand Down
2 changes: 1 addition & 1 deletion src/tests-rosetta-cli-data/validate-rosetta.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import { StacksTestnet } from '@stacks/network';
import * as fs from 'fs';
import { StacksCoreRpcClient, getCoreNodeEndpoint } from '../core-rpc/client';
import { unwrapOptional } from '../helpers';
import * as compose from 'docker-compose';
import { v2 as compose } from 'docker-compose';
import * as path from 'path';
import { PgWriteStore } from '../datastore/pg-write-store';
import { EventStreamServer, startEventServer } from '../event-stream/event-server';
Expand Down

0 comments on commit fe720d0

Please sign in to comment.