@malloydata/malloy-tests 0.0.138-dev240405133532 → 0.0.138

package/package.json

@@ -21,13 +21,13 @@
   },
   "dependencies": {
     "@jest/globals": "^29.4.3",
-    "@malloydata/db-bigquery": "^0.0.138-dev240405133532",
-    "@malloydata/db-duckdb": "^0.0.138-dev240405133532",
-    "@malloydata/db-postgres": "^0.0.138-dev240405133532",
-    "@malloydata/db-snowflake": "^0.0.138-dev240405133532",
-    "@malloydata/db-trino": "^0.0.138-dev240405133532",
-    "@malloydata/malloy": "^0.0.138-dev240405133532",
-    "@malloydata/render": "^0.0.138-dev240405133532",
+    "@malloydata/db-bigquery": "^0.0.138",
+    "@malloydata/db-duckdb": "^0.0.138",
+    "@malloydata/db-postgres": "^0.0.138",
+    "@malloydata/db-snowflake": "^0.0.138",
+    "@malloydata/db-trino": "^0.0.138",
+    "@malloydata/malloy": "^0.0.138",
+    "@malloydata/render": "^0.0.138",
     "jsdom": "^22.1.0",
     "luxon": "^2.4.0",
     "madge": "^6.0.0"
@@ -36,5 +36,5 @@
     "@types/jsdom": "^21.1.1",
     "@types/luxon": "^2.4.0"
   },
-  "version": "0.0.138-dev240405133532"
+  "version": "0.0.138"
 }

package/src/databases/all/functions.spec.ts

@@ -105,6 +105,8 @@ expressionModels.forEach((expressionModel, databaseName) => {
 
     const result = await run();
     testCases.forEach((testCase, i) => {
+      // console.log(databaseName, result.sql);
+      // console.log(result.data);
       expect(result.data.path(0, `f${i}`).value).toBe(testCase[1]);
     });
   };
@@ -170,11 +172,7 @@ expressionModels.forEach((expressionModel, databaseName) => {
     it(`works - ${databaseName}`, async () => {
       await funcTestMultiple(
         ['ceil(1.9)', 2],
-        // TODO Remove when we upgrade to DuckDB 0.8.X -- DuckDB has some bugs with rounding
-        // that are fixed in 0.8.
-        ...(databaseName === 'duckdb_wasm'
-          ? []
-          : ([['ceil(-1.9)', -1]] as [string, number][])),
+        ['ceil(-1.9)', -1],
         ['ceil(null)', null]
       );
     });
@@ -216,9 +214,16 @@ expressionModels.forEach((expressionModel, databaseName) => {
         ["replace('aaaa', r'.', 'c')", 'cccc'],
         [
           "replace('axbxc', r'(a).(b).(c)', '\\\\0 - \\\\1 - \\\\2 - \\\\3')",
-          databaseName === 'postgres' ? '\\0 - a - b - c' : 'axbxc - a - b - c',
+          databaseName === 'postgres'
+            ? '\\0 - a - b - c'
+            : databaseName === 'trino'
+            ? '0 - 1 - 2 - 3'
+            : 'axbxc - a - b - c',
+        ],
+        [
+          "replace('aaaa', '', 'c')",
+          databaseName === 'trino' ? 'cacacacac' : 'aaaa',
         ],
-        ["replace('aaaa', '', 'c')", 'aaaa'],
         ["replace(null, 'a', 'c')", null],
         ["replace('aaaa', null, 'c')", null],
         ["replace('aaaa', 'a', null)", null]
@@ -232,7 +237,7 @@ expressionModels.forEach((expressionModel, databaseName) => {
         ["substr('foo', 2)", 'oo'],
         ["substr('foo', 2, 1)", 'o'],
         ["substr('foo bar baz', -3)", 'baz'],
-        ['substr(null, 1, 2)', null],
+        ["substr(nullif('x','x'), 1, 2)", null], //  nullMatchesFunctionSignature.
         ["substr('aaaa', null, 1)", null],
         ["substr('aaaa', 1, null)", null]
       );
@@ -245,7 +250,7 @@ expressionModels.forEach((expressionModel, databaseName) => {
         ['floor(cbrt!(27)::number)', 3],
         ['floor(cbrt!number(27))', 3],
         ["substr('foo bar baz', -3)", 'baz'],
-        ['substr(null, 1, 2)', null],
+        ["substr(nullif('x','x'), 1, 2)", null], // nullMatchesFunctionSignature
         ["substr('aaaa', null, 1)", null],
         ["substr('aaaa', 1, null)", null]
       );
@@ -254,7 +259,7 @@ expressionModels.forEach((expressionModel, databaseName) => {
 
   describe('stddev', () => {
     // TODO symmetric aggregates don't work with custom aggregate functions in BQ currently
-    if (['bigquery', 'snowflake'].includes(databaseName)) return;
+    if (['bigquery', 'snowflake', 'trino'].includes(databaseName)) return;
     it(`works - ${databaseName}`, async () => {
       await funcTestAgg('round(stddev(aircraft_models.seats))', 29);
     });