malloy icon indicating copy to clipboard operation
malloy copied to clipboard

Persmission problems with postgres table/schema case test

Open mtoy-googly-moogly opened this issue 1 year ago • 0 comments

This test is implemented by creating a schema and a table with mixed case, and then running some queries against that table in Malloy to make sure that the generated SQL properly quotes the table and schema name.

At the moment this bug was written, this table. in the test database used by clouldbuild, the schema and tables for this test exist, but are only readable by the cloudbuild user, and the "npm publish" user cannot read these tables.

As a workaround, these tests just skip if the table isn't found, but that is not a safe or stable fix.

Better fix is to just directly test the postgres connection to see that it properly parses and quotes table names and schema names rather than having to create a table on the server and generate code and run the code to verify that.

mtoy-googly-moogly avatar Oct 24 '23 21:10 mtoy-googly-moogly