Merge pull request #2057 from malloydata/jlsmart-modbigquery

Added test verifying proper behavior for modulus operator.
malloydata · Dec 17, 2024 · bbf6027 · bbf6027
2 parents 845d52d + e01e32a
commit bbf6027
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/test/src/databases/all/expr.spec.ts b/test/src/databases/all/expr.spec.ts
@@ -357,6 +357,16 @@ describe.each(runtimes.runtimeList)('%s', (databaseName, runtime) => {
     }
   );
 
+  it('uses the correct symbol for modulus, either % or MOD', async () => {
+    await expect(`
+      run: aircraft_models->{
+        group_by: mod_example is 10 % 4
+      }
+    `).malloyResultMatches(expressionModel, {
+      mod_example: 2,
+    });
+  });
+
   it('named query metadata undefined', async () => {
     const result = await expressionModel
       .loadQuery(