fix: allow empty order by as an input (#149)

* allow empty order by as an input

* added test case

Author: zaidjan-devrev

meerkat-core/src/ast-builder/ast-builder.ts

@@ -124,7 +124,7 @@ export const cubeToDuckdbAST = (
     node.group_sets = [groupSets];
   }
   node.modifiers = [];
-  if (query.order) {
+  if (query.order && Object.keys(query.order).length > 0) {
     node.modifiers.push(cubeOrderByToAST(query.order));
   }
   if (query.limit || query.offset) {

meerkat-node/src/__tests__/cube-to-sql.spec.ts

@@ -119,4 +119,21 @@ describe('cube-to-sql', () => {
       'SELECT orders.* FROM (SELECT * FROM (select * from orders) AS orders) AS orders'
     );
   });
+  it('Should handle empty order', async () => {
+    const query = {
+      order: {},
+      measures: ['*'],
+      filters: [
+        {
+          or: [],
+        },
+      ],
+      dimensions: [],
+    };
+    const sql = await cubeQueryToSQL({ query, tableSchemas: [TABLE_SCHEMA] });
+    console.info(`SQL for Simple Cube Query: `, sql);
+    expect(sql).toEqual(
+      'SELECT orders.* FROM (SELECT * FROM (select * from orders) AS orders) AS orders'
+    );
+  });
 });