add tests for postgres & mysql drivers

cube-js · Jan 21, 2025 · 23b3c18 · 23b3c18
1 parent 544cdc8
commit 23b3c18
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 0 deletions.
diff --git a/packages/cubejs-mysql-driver/test/MySqlDriver.test.ts b/packages/cubejs-mysql-driver/test/MySqlDriver.test.ts
@@ -116,4 +116,18 @@ describe('MySqlDriver', () => {
       await tableData.release();
     }
   });
+
+  test('table name check', async () => {
+    const tblName = 'really-really-really-looooooooooooooooooooooooooooooooooooooooooooooooooooong-table-name';
+    try {
+      await mySqlDriver.createTable(tblName, [{ name: 'id', type: 'bigint' }]);
+
+      throw new Error('createTable must throw an exception');
+    } catch (e: any) {
+      expect(e.message).toEqual(
+        'MySQL can not work with table names longer than 64 symbols. ' +
+        `Consider using the 'sqlAlias' attribute in your cube definition for ${tblName}.`
+      );
+    }
+  });
 });
diff --git a/packages/cubejs-postgres-driver/test/PostgresDriver.test.ts b/packages/cubejs-postgres-driver/test/PostgresDriver.test.ts
@@ -142,6 +142,20 @@ describe('PostgresDriver', () => {
     }
   });
 
+  test('table name check', async () => {
+    const tblName = 'really-really-really-looooooooooooooooooooooooooooooooooooooooooooooooooooong-table-name';
+    try {
+      await driver.createTable(tblName, [{ name: 'id', type: 'bigint' }]);
+
+      throw new Error('createTable must throw an exception');
+    } catch (e: any) {
+      expect(e.message).toEqual(
+        'PostgreSQL can not work with table names longer than 63 symbols. ' +
+        `Consider using the 'sqlAlias' attribute in your cube definition for ${tblName}.`
+      );
+    }
+  });
+
   // Note: This test MUST be the last in the list.
   test('release', async () => {
     expect(async () => {