Empty target list support (pganalyze#156)

olleolleolle · Nov 11, 2019 · 467c762 · 467c762
1 parent 1929594
commit 467c762
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/pg_query/deparse.rb b/lib/pg_query/deparse.rb
@@ -1058,8 +1058,9 @@ def deparse_select(node) # rubocop:disable Metrics/CyclomaticComplexity
         output.join(' ')
       end
 
+      output << 'SELECT' if node[FROM_CLAUSE_FIELD] || node[TARGET_LIST_FIELD]
+
       if node[TARGET_LIST_FIELD]
-        output << 'SELECT'
         if node['distinctClause']
           output << 'DISTINCT'
           unless node['distinctClause'].compact.empty?

diff --git a/spec/lib/pg_query/deparse_spec.rb b/spec/lib/pg_query/deparse_spec.rb
@@ -14,6 +14,12 @@
         it { is_expected.to eq query }
       end
 
+      context 'with empty target list' do
+        let(:query) { 'SELECT FROM "x" WHERE "y" = 5 AND "z" = "y"' }
+
+        it { is_expected.to eq query }
+      end
+
       context 'basic statement with schema' do
         let(:query) { 'SELECT "a" AS b FROM "public"."x" WHERE "y" = 5 AND "z" = "y"' }