chore: fix super strange format issue

Author: ovr

packages/cubejs-schema-compiler/src/adapter/ClickHouseQuery.ts

@@ -74,16 +74,13 @@ export class ClickHouseQuery extends BaseQuery {
     const timeUnit = this.diffTimeUnitForInterval(interval);
     const beginOfTime = 'fromUnixTimestamp(0)';
 
-    const dateBinResult = `date_add(${timeUnit},
+    return `date_add(${timeUnit},
         FLOOR(
           date_diff(${timeUnit}, ${alignedOrigin}, ${source}) /
           date_diff(${timeUnit}, ${beginOfTime}, ${beginOfTime} + ${intervalFormatted})
         ) * date_diff(${timeUnit}, ${beginOfTime}, ${beginOfTime} + ${intervalFormatted}),
         ${alignedOrigin}
     )`;
-
-    // Normalize the result to DateTime64(0) for consistent formatting
-    return `toDateTime64(${dateBinResult}, 0, '${this.timezone}')`;
   }
 
   public subtractInterval(date: string, interval: string): string {

packages/cubejs-schema-compiler/test/integration/clickhouse/ClickHouseDbRunner.ts

@@ -4,6 +4,7 @@ import { GenericContainer } from 'testcontainers';
 import type { StartedTestContainer } from 'testcontainers';
 import { format as formatSql } from 'sqlstring';
 import { v4 as uuidv4 } from 'uuid';
+import * as moment from 'moment/moment';
 import { ClickHouseQuery } from '../../../src/adapter/ClickHouseQuery';
 import { BaseDbRunner } from '../utils/BaseDbRunner';
 
@@ -210,10 +211,14 @@ export class ClickHouseDbRunner extends BaseDbRunner {
           if (fieldMeta === undefined) {
             throw new Error(`Missing meta for field ${field}`);
           }
-          if (fieldMeta.type.includes('DateTime')) {
+
+          if (fieldMeta.type.includes('DateTime64')) {
+            row[field] = moment.utc(value).format(moment.HTML5_FMT.DATETIME_LOCAL_MS);
+          } else if (fieldMeta.type.includes('DateTime') /** Can be DateTime or DateTime('timezone') */) {
             if (typeof value !== 'string') {
               throw new Error(`Unexpected value for ${field}`);
             }
+
             row[field] = `${value.substring(0, 10)}T${value.substring(11, 22)}.000`;
           } else if (fieldMeta.type.includes('Date')) {
             row[field] = `${value}T00:00:00.000`;

packages/cubejs-schema-compiler/test/unit/base-query.test.ts

@@ -1012,7 +1012,7 @@ describe('SQL Generation', () => {
               relationship: 'one_to_one'
             },
           }`
-        }).replace(`sql: \`\${CUBE}.location = 'San Francisco'\``, `sql: \`\${FILTER_PARAMS.cardsA.location.filter('location')}\``),
+        }).replace('sql: `${CUBE}.location = \'San Francisco\'`', 'sql: `${FILTER_PARAMS.cardsA.location.filter(\'location\')}`'),
         createCubeSchema({
           name: 'cardsB',
           sqlTable: 'card2_tbl',