From cce3138cc118027516171ad856f0fa6ae30e0532 Mon Sep 17 00:00:00 2001 From: steve-chavez Date: Fri, 7 Jul 2017 16:32:16 -0500 Subject: [PATCH] Add test for a proc name = return col name, Fix #901 --- test/Feature/QuerySpec.hs | 4 ++++ test/fixtures/schema.sql | 6 ++++++ 2 files changed, 10 insertions(+) diff --git a/test/Feature/QuerySpec.hs b/test/Feature/QuerySpec.hs index 06c35b950..26c1fe319 100644 --- a/test/Feature/QuerySpec.hs +++ b/test/Feature/QuerySpec.hs @@ -667,6 +667,10 @@ spec = do [json| "Return value of no parameters procedure." |] { matchHeaders = [matchContentTypeJson] } + it "returns proper output when having the same return col name as the proc name" $ + post "/rpc/test" [json|{}|] `shouldRespondWith` + [json|[{"test":"hello","value":1}]|] { matchHeaders = [matchContentTypeJson] } + describe "weird requests" $ do it "can query as normal" $ do get "/Escap3e;" `shouldRespondWith` diff --git a/test/fixtures/schema.sql b/test/fixtures/schema.sql index c396c4475..6c7145604 100755 --- a/test/fixtures/schema.sql +++ b/test/fixtures/schema.sql @@ -1215,6 +1215,12 @@ create table grandchild_entities ( jsonb_col jsonb ); +-- Used for testing that having the same return column name as the proc name +-- doesn't conflict with the required output, details in #901 +create function test.test() returns table(test text, value int) as $$ + values ('hello', 1); +$$ language sql; + -- -- PostgreSQL database dump complete --