open-dsi
/
postgres
mirror of https://github.com/postgres/postgres
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix some broken key provider test

Browse Source 
These tests did not assert what they claimed to assert.
pull/220/head
Anders Åstrand 4 months ago committed by AndersAstrand
parent 784e45d414
commit 34438f5229
2 changed files with 18 additions and 18 deletions
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Show Stats Download Patch File Download Diff File
  1. 24
      contrib/pg_tde/expected/key_provider.out
  2. 12
      contrib/pg_tde/sql/key_provider.sql

24
contrib/pg_tde/expected/key_provider.out
Unescape View File

@ -258,16 +258,16 @@ ERROR: key provider options must be an object
-- Modifying key providers fails if an external value object doesn't have all required keys
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {}}');
ERROR: external value must contain "type" in field "path"
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": null}}');
ERROR: key provider "provider" does not exists
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": null}}');
ERROR: external value must contain "type" in field "path"
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "remote"}}');
ERROR: external remote value must contain "url" in field "path"
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": "remote", "url": null}}');
ERROR: key provider "provider" does not exists
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "remote", "url": null}}');
ERROR: external remote value must contain "url" in field "path"
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "file"}}');
ERROR: external file value must contain "path" in field "path"
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": "file", "path": null}}');
ERROR: key provider "provider" does not exists
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "file", "path": null}}');
ERROR: external file value must contain "path" in field "path"
-- Modifying key providers fails if values are array instead of scalar
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": ["array"]}');
ERROR: unexpected array in field "path"
@ -276,12 +276,12 @@ ERROR: unexpected array in field "path"
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "file", "path": ["array"]}}');
ERROR: unexpected array in field "path"
-- Modifying key providers fails if values are boolean
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": true}');
ERROR: key provider "provider" does not exists
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": true}}');
ERROR: key provider "provider" does not exists
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": "file", "path": true}}');
ERROR: key provider "provider" does not exists
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": true}');
ERROR: unexpected boolean in field "path"
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": true}}');
ERROR: unexpected boolean in field "path"
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "file", "path": true}}');
ERROR: unexpected boolean in field "path"
-- Deleting key providers fails if key name is NULL
SELECT pg_tde_delete_database_key_provider(NULL);
ERROR: provider_name cannot be null

12
contrib/pg_tde/sql/key_provider.sql
Unescape View File

@ -119,11 +119,11 @@ SELECT pg_tde_change_database_key_provider('file', 'file-provider', 'null');
-- Modifying key providers fails if an external value object doesn't have all required keys
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {}}');
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": null}}');
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": null}}');
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "remote"}}');
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": "remote", "url": null}}');
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "remote", "url": null}}');
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "file"}}');
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": "file", "path": null}}');
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "file", "path": null}}');
-- Modifying key providers fails if values are array instead of scalar
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": ["array"]}');
@ -131,9 +131,9 @@ SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "file", "path": ["array"]}}');
-- Modifying key providers fails if values are boolean
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": true}');
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": true}}');
SELECT pg_tde_change_database_key_provider('file', 'provider', '{"path": {"type": "file", "path": true}}');
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": true}');
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": true}}');
SELECT pg_tde_change_database_key_provider('file', 'file-provider', '{"path": {"type": "file", "path": true}}');
-- Deleting key providers fails if key name is NULL
SELECT pg_tde_delete_database_key_provider(NULL);

Write Preview
Loading…
Cancel
Save