sql-formatter-org / sql-formatter

96%

push

github

fix(postgresql): keep spaces around OPERATOR() with denseOperators (#958)

I was trying out `denseOperators` on some Postgres queries and noticed
it mangles the `OPERATOR(schema.op)` syntax:

```
format('SELECT foo OPERATOR(public.===) bar;', { language: 'postgresql', denseOperators: true })
// =>
// SELECT
//   fooOPERATOR(public.===)bar;
```

The spaces around the operator get stripped, so `foo` runs into the
`OPERATOR` keyword and the result no longer parses. Unlike `+`/`=`/etc.,
this one is keyword-like and can't be glued to its operands, so I made
it keep its surrounding spaces even in dense mode — same idea as the
existing `-` exception for dialects with dashed identifiers.

Added a test under the existing OPERATOR() case in postgresql.test.ts.

593 of 663 branches covered (89.44%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

1324 of 1342 relevant lines covered (98.66%)

16463.25 hits per line