Merge pull request #15 from andogq/feat/update-clickhouse

Update ClickHouse to 1.4.0
VVVi · Sep 7, 2024 · de87f00 · de87f00
2 parents a601c5e + 693fe18
commit de87f00
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 23 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -22,17 +22,17 @@
   "license": "MIT",
   "repository": "VVVi/clickhouse-migrations",
   "dependencies": {
-    "@clickhouse/client": "^0.2.9",
+    "@clickhouse/client": "^1.4.0",
     "commander": "^11.1.0"
   },
   "devDependencies": {
     "@types/jest": "^29.5.12",
     "@types/node": "^18.15.11",
-    "@typescript-eslint/parser": "^6.20.0",
     "@typescript-eslint/eslint-plugin": "^6.20.0",
-    "eslint-plugin-prettier": "^5.1.3",
+    "@typescript-eslint/parser": "^6.20.0",
     "eslint": "^8.56.0",
     "eslint-config-prettier": "^9.1.0",
+    "eslint-plugin-prettier": "^5.1.3",
     "jest": "^29.7.0",
     "prettier": "^3.2.4",
     "ts-jest": "^29.1.2",

diff --git a/src/migrate.ts b/src/migrate.ts
@@ -15,14 +15,14 @@ const log = (type: 'info' | 'error' = 'info', message: string, error?: string) =
 };
 
 const connect = (
-  host: string,
+  url: string,
   username: string,
   password: string,
   db_name?: string,
   timeout?: string,
 ): ClickHouseClient => {
   const db_params: NodeClickHouseClientConfigOptions = {
-    host,
+    url,
     username,
     password,
     application: 'clickhouse-migrations',
@@ -62,13 +62,13 @@ const create_db = async (host: string, username: string, password: string, db_na
 
 const init_migration_table = async (client: ClickHouseClient): Promise<void> => {
   const q = `CREATE TABLE IF NOT EXISTS _migrations (
-      uid UUID DEFAULT generateUUIDv4(), 
+      uid UUID DEFAULT generateUUIDv4(),
       version UInt32,
-      checksum String, 
-      migration_name String, 
+      checksum String,
+      migration_name String,
       applied_at DateTime DEFAULT now()
-    ) 
-    ENGINE = MergeTree 
+    )
+    ENGINE = MergeTree
     ORDER BY tuple(applied_at)`;
 
   try {

diff --git a/tests/db.test.ts b/tests/db.test.ts
@@ -38,13 +38,13 @@ describe('Migration tests', () => {
     });
     expect(execSpy).toHaveBeenNthCalledWith(2, {
       query: `CREATE TABLE IF NOT EXISTS _migrations (
-      uid UUID DEFAULT generateUUIDv4(), 
+      uid UUID DEFAULT generateUUIDv4(),
       version UInt32,
-      checksum String, 
-      migration_name String, 
+      checksum String,
+      migration_name String,
       applied_at DateTime DEFAULT now()
-    ) 
-    ENGINE = MergeTree 
+    )
+    ENGINE = MergeTree
     ORDER BY tuple(applied_at)`,
       clickhouse_settings: {
         wait_end_of_query: 1,