Add support for oracle specific syntax (#326)

* oracle LIMIT as mssql

* tests

Author: Sylvestre

src/table.js

@@ -317,7 +317,7 @@ export function makeQueryTemplate(operations, source) {
     appendSql(i ? `, ` : `\nORDER BY `, args);
     appendOrderBy(sort[i], args, escaper);
   }
-  if (source.dialect === "mssql") {
+  if (source.dialect === "mssql" || source.dialect === "oracle") {
     if (slice.to !== null || slice.from !== null) {
       if (!sort.length) {
         if (!select.columns)

test/table-test.js

@@ -361,6 +361,57 @@ describe("makeQueryTemplate", () => {
     assert.deepStrictEqual(parts.join("?"), "SELECT * FROM table1\nORDER BY col2 DESC\nOFFSET 10 ROWS\nFETCH NEXT 90 ROWS ONLY");
     assert.deepStrictEqual(params, []);
   });
+
+  it("makeQueryTemplate throw if no columns are explicitly specified for oracle dialect", () => {
+    const source = {name: "db", dialect: "oracle"};
+    const operations = {
+      ...baseOperations,
+      select: {
+        columns: null
+      },
+      sort: [],
+      slice: {from: 10, to: 100}
+    };
+
+    assert.throws(() => {
+      makeQueryTemplate(operations, source);
+    }, Error);
+  });
+
+  it("makeQueryTemplate select, sort, slice, filter indexed with oracle syntax", () => {
+    const source = {name: "db", dialect: "oracle"};
+    const operations = {
+      ...baseOperations,
+      select: {
+        columns: ["col1", "col2", "col3"]
+      },
+      sort: [{column: "col2", direction: "desc"}],
+      slice: {from: 10, to: 100},
+      filter: [
+        {
+          type: "gte",
+          operands: [
+            {type: "column", value: "col1"},
+            {type: "resolved", value: "val1"}
+          ]
+        },
+        {
+          type: "eq",
+          operands: [
+            {type: "column", value: "col2"},
+            {type: "resolved", value: "val2"}
+          ]
+        }
+      ]
+    };
+
+    const [parts, ...params] = makeQueryTemplate(operations, source);
+    assert.deepStrictEqual(
+      parts.join("?"),
+      "SELECT col1, col2, col3 FROM table1\nWHERE col1 >= ?\nAND col2 = ?\nORDER BY col2 DESC\nOFFSET 10 ROWS\nFETCH NEXT 90 ROWS ONLY"
+    );
+    assert.deepStrictEqual(params, ["val1", "val2"]);
+  });
 });
 
 describe("__table", () => {