public inbox for [email protected]
help / color / mirror / Atom feedFrom: Nagesh Dhope <[email protected]>
To: Dave Page <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: RE-SQL test patch for extension node
Date: Tue, 9 Jul 2019 10:46:35 +0530
Message-ID: <CAA3zdZpY=KkTm+EibfDUUeCJY1pRNCz_gEVYXmHThRb_04b6TQ@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxoy2tf9WXu7_E35=Pa3vMCaBegVMQGWNcpgQxS93_TJeeQ@mail.gmail.com>
References: <CAA3zdZo_bNe=140wpy4tLkR3qHWY9xL--ymf=yKaab6-Z98OMw@mail.gmail.com>
<CA+OCxoy2tf9WXu7_E35=Pa3vMCaBegVMQGWNcpgQxS93_TJeeQ@mail.gmail.com>
Hi Dave,
Please find updated patch with *sslinfo.*
On Mon, Jul 8, 2019 at 2:03 PM Dave Page <[email protected]> wrote:
> Hi
>
> On Fri, Jul 5, 2019 at 5:16 PM Nagesh Dhope <[email protected]>
> wrote:
>
>> Hi Hackers,
>> Please review attached patch for RE-SQL test cases for *extension* node.
>>
>
> Unfortunately this fails on PostgreSQL 12 as the deprecated timetravel
> contrib module has finally been removed.
>
> Perhaps use sslinfo? That seems less likely to go away anytime soon.
>
> Thanks.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
Attachments:
[application/octet-stream] extension_resql_tests_v2.patch (4.1K, 3-extension_resql_tests_v2.patch)
download | inline diff:
diff --git a/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/alter_extension_schema.sql b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/alter_extension_schema.sql
new file mode 100644
index 0000000..fc8fc10
--- /dev/null
+++ b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/alter_extension_schema.sql
@@ -0,0 +1,7 @@
+-- Extension: sslinfo
+
+-- DROP EXTENSION sslinfo;
+
+CREATE EXTENSION sslinfo
+ SCHEMA test_extension_schema
+ VERSION "1.2";
diff --git a/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/alter_extension_version.sql b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/alter_extension_version.sql
new file mode 100644
index 0000000..6d193fb
--- /dev/null
+++ b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/alter_extension_version.sql
@@ -0,0 +1,7 @@
+-- Extension: adminpack
+
+-- DROP EXTENSION adminpack;
+
+CREATE EXTENSION adminpack
+ SCHEMA pg_catalog
+ VERSION "1.1";
diff --git a/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/create_extension_with_all_options.sql b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/create_extension_with_all_options.sql
new file mode 100644
index 0000000..37c620c
--- /dev/null
+++ b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/create_extension_with_all_options.sql
@@ -0,0 +1,7 @@
+-- Extension: sslinfo
+
+-- DROP EXTENSION sslinfo;
+
+CREATE EXTENSION sslinfo
+ SCHEMA public
+ VERSION "1.2";
diff --git a/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/create_extension_with_default_schema.sql b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/create_extension_with_default_schema.sql
new file mode 100644
index 0000000..a9a61d4
--- /dev/null
+++ b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/create_extension_with_default_schema.sql
@@ -0,0 +1,7 @@
+-- Extension: adminpack
+
+-- DROP EXTENSION adminpack;
+
+CREATE EXTENSION adminpack
+ SCHEMA pg_catalog
+ VERSION "1.0";
diff --git a/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/tests.json b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/tests.json
new file mode 100644
index 0000000..a450120
--- /dev/null
+++ b/web/pgadmin/browser/server_groups/servers/databases/extensions/tests/default/tests.json
@@ -0,0 +1,59 @@
+{
+ "scenarios": [
+ {
+ "type": "create",
+ "name": "Create Extension with all options",
+ "endpoint": "NODE-extension.obj",
+ "sql_endpoint": "NODE-extension.sql_id",
+ "data": {
+ "name": "sslinfo",
+ "schema": "public",
+ "version": "1.2"
+ },
+ "expected_sql_file": "create_extension_with_all_options.sql"
+ },
+ {
+ "type": "alter",
+ "name": "Alter Extension version",
+ "endpoint": "NODE-extension.obj_id",
+ "sql_endpoint": "NODE-extension.sql_id",
+ "data": {
+ "schema": "test_extension_schema"
+ },
+ "expected_sql_file": "alter_extension_schema.sql"
+ },
+ {
+ "type": "delete",
+ "name": "Drop Extension with all options",
+ "endpoint": "NODE-extension.obj_id",
+ "data": {}
+ },
+ {
+ "type": "create",
+ "name": "Create Extension with default(pg_catalog) schema",
+ "endpoint": "NODE-extension.obj",
+ "sql_endpoint": "NODE-extension.sql_id",
+ "data": {
+ "name": "adminpack",
+ "version": "1.0"
+ },
+ "expected_sql_file": "create_extension_with_default_schema.sql"
+ },
+ {
+ "type": "alter",
+ "name": "Alter Extension version",
+ "endpoint": "NODE-extension.obj_id",
+ "sql_endpoint": "NODE-extension.sql_id",
+ "data": {
+ "version": "1.1"
+ },
+ "expected_sql_file": "alter_extension_version.sql"
+ },
+ {
+ "type": "delete",
+ "name": "Drop Extension default(pg_catalog) schema",
+ "endpoint": "NODE-extension.obj_id",
+ "data": {}
+ }
+ ]
+}
view thread (7+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected]
Subject: Re: RE-SQL test patch for extension node
In-Reply-To: <CAA3zdZpY=KkTm+EibfDUUeCJY1pRNCz_gEVYXmHThRb_04b6TQ@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox