Allow "type" to be used as identifier in PostgreSQL

Fixes #685
sql-formatter-org · Dec 7, 2023 · cd7ec6c · cd7ec6c
1 parent 7e3db7f
commit cd7ec6c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/languages/postgresql/postgresql.formatter.ts b/src/languages/postgresql/postgresql.formatter.ts
@@ -52,7 +52,7 @@ const onelineClauses = expandPhrases([
   'ADD [COLUMN] [IF NOT EXISTS]',
   'DROP [COLUMN] [IF EXISTS]',
   'ALTER [COLUMN]',
-  '[SET DATA] TYPE', // for alter column
+  'SET DATA TYPE', // for alter column
   '{SET | DROP} DEFAULT', // for alter column
   '{SET | DROP} NOT NULL', // for alter column
   // - truncate:

diff --git a/test/postgresql.test.ts b/test/postgresql.test.ts
@@ -301,4 +301,15 @@ describe('PostgreSqlFormatter', () => {
         tbl.salary;
     `);
   });
+
+  // Issue #685
+  it('allows TYPE to be used as an identifier', () => {
+    expect(format(`SELECT type, modified_at FROM items;`)).toBe(dedent`
+      SELECT
+        type,
+        modified_at
+      FROM
+        items;
+    `);
+  });
 });