Fix ROWS BETWEEN causing crash in BigQuery

Fixes #415

Author: nene

src/languages/bigquery/bigquery.formatter.ts

@@ -127,14 +127,16 @@ const reservedJoins = expandPhrases([
   '{INNER | CROSS} JOIN',
 ]);
 
-const reservedPhrases = [
+const reservedPhrases = expandPhrases([
   // https://cloud.google.com/bigquery/docs/reference/standard-sql/query-syntax#tablesample_operator
   'TABLESAMPLE SYSTEM',
   // From DDL: https://cloud.google.com/bigquery/docs/reference/standard-sql/data-definition-language
   'ANY TYPE',
   'ALL COLUMNS',
   'NOT DETERMINISTIC',
-];
+  // https://cloud.google.com/bigquery/docs/reference/standard-sql/window-function-calls#def_over_clause
+  '{ROWS | RANGE} BETWEEN',
+]);
 
 // https://cloud.google.com/bigquery/docs/reference/#standard-sql-reference
 export default class BigQueryFormatter extends Formatter {

test/bigquery.test.ts

@@ -17,6 +17,7 @@ import supportsComments from './features/comments';
 import supportsIdentifiers from './features/identifiers';
 import supportsParams from './options/param';
 import supportsWindow from './features/window';
+import supportsWindowFunctions from './features/windowFunctions';
 import supportsSetOperations from './features/setOperations';
 import supportsLimiting from './features/limiting';
 import supportsInsertInto from './features/insertInto';
@@ -59,6 +60,7 @@ describe('BigQueryFormatter', () => {
   supportsOperators(format, BigQueryFormatter.operators);
   supportsParams(format, { positional: true, named: ['@'], quoted: ['@``'] });
   supportsWindow(format);
+  supportsWindowFunctions(format);
   supportsLimiting(format, { limit: true, offset: true });
 
   // Note: BigQuery supports single dashes inside identifiers, so my-ident would be

test/features/windowFunctions.ts

@@ -0,0 +1,30 @@
+import dedent from 'dedent-js';
+
+import { FormatFn } from 'src/sqlFormatter';
+
+export default function supportsWindowFunctions(format: FormatFn) {
+  it('supports ROWS BETWEEN in window functions', () => {
+    expect(
+      format(`
+        SELECT
+          AVG(amount) OVER (
+            PARTITION BY explosion
+            ORDER BY day ROWS BETWEEN 6 PRECEDING AND CURRENT ROW
+          ) AS amount
+        FROM
+          tbl
+      `)
+    ).toBe(dedent`
+      SELECT
+        AVG(amount) OVER (
+          PARTITION BY
+            explosion
+          ORDER BY
+            day ROWS BETWEEN 6 PRECEDING
+            AND CURRENT ROW
+        ) AS amount
+      FROM
+        tbl
+    `);
+  });
+}