Hi,

I'll look into it.

On Thu, Aug 27, 2020 at 2:41 PM Murtuza Zabuawala <
murtuza.zabuaw...@enterprisedb.com> wrote:

> Hi,
>
> RE-SQL test for the role node is failing if the timezone of DB is
> different.
>
> Traceback (most recent call last):
>   File
> "/home/murtuza/projects/PEM/web/regression/re_sql/tests/test_resql.py",
> line 500, in check_re_sql
>     self.assertEquals(sql, resp_sql)
>   File "/usr/lib/python3.6/unittest/case.py", line 1321, in deprecated_func
>     return original_func(*args, **kwargs)
>   File "/usr/lib/python3.6/unittest/case.py", line 829, in assertEqual
>     assertion_func(first, second, msg=msg)
>   File "/usr/lib/python3.6/unittest/case.py", line 1203, in
> assertMultiLineEqual
>     self.fail(self._formatMessage(msg, standardMsg))
>   File "/usr/lib/python3.6/unittest/case.py", line 670, in fail
>     raise self.failureException(msg)
> AssertionError: '-- R[313 chars] \'2050-01-01 00:00:00+05:30\';\n\nGRANT
> pg_si[300 chars]n\';' != '-- R[313 chars] \'2049-12-31
> 10:30:00-08\';\n\nGRANT pg_signa[297 chars]n\';'
>   -- Role: "Role2_$%{}[]()&*^!@""'`\/#"
>   -- DROP ROLE "Role2_$%{}[]()&*^!@""'`\/#";
>
>   CREATE ROLE "Role2_$%{}[]()&*^!@""'`\/#" WITH
>     NOLOGIN
>     SUPERUSER
>     INHERIT
>     CREATEDB
>     NOCREATEROLE
>     NOREPLICATION
>     CONNECTION LIMIT 100
>     ENCRYPTED PASSWORD 'md5cc9a2576a43b841dcacea0f65d961e41'
> -   VALID UNTIL '2050-01-01 00:00:00+05:30';
> +   VALID UNTIL '2049-12-31 10:30:00-08';
>
>   GRANT pg_signal_backend TO "Role2_$%{}[]()&*^!@""'`\/#";
>   GRANT pg_monitor TO "Role2_$%{}[]()&*^!@""'`\/#" WITH ADMIN OPTION;
>
>   ALTER ROLE "Role2_$%{}[]()&*^!@""'`\/#" IN DATABASE postgres SET
> application_name TO 'pg4';
>
>   COMMENT ON ROLE "Role2_$%{}[]()&*^!@""'`\/#" IS 'This is detailed
> description';
> Alter Role options 2  Expected SQL File:alter_role_options2.sql... FAIL
> Alter Role options 3 (MSQL) ... ok
> Traceback (most recent call last):
>   File
> "/home/murtuza/projects/PEM/web/regression/re_sql/tests/test_resql.py",
> line 500, in check_re_sql
>     self.assertEquals(sql, resp_sql)
>   File "/usr/lib/python3.6/unittest/case.py", line 1321, in deprecated_func
>     return original_func(*args, **kwargs)
>   File "/usr/lib/python3.6/unittest/case.py", line 829, in assertEqual
>     assertion_func(first, second, msg=msg)
>   File "/usr/lib/python3.6/unittest/case.py", line 1203, in
> assertMultiLineEqual
>     self.fail(self._formatMessage(msg, standardMsg))
>   File "/usr/lib/python3.6/unittest/case.py", line 670, in fail
>     raise self.failureException(msg)
> AssertionError: '-- R[313 chars] \'2050-01-01 00:00:00+05:30\';\n\nGRANT
> pg_mo[240 chars]n\';' != '-- R[313 chars] \'2049-12-31
> 10:30:00-08\';\n\nGRANT pg_monit[237 chars]n\';'
>   -- Role: "Role2_$%{}[]()&*^!@""'`\/#"
>   -- DROP ROLE "Role2_$%{}[]()&*^!@""'`\/#";
>
>   CREATE ROLE "Role2_$%{}[]()&*^!@""'`\/#" WITH
>     NOLOGIN
>     SUPERUSER
>     INHERIT
>     CREATEDB
>     NOCREATEROLE
>     NOREPLICATION
>     CONNECTION LIMIT 100
>     ENCRYPTED PASSWORD 'md5cc9a2576a43b841dcacea0f65d961e41'
> -   VALID UNTIL '2050-01-01 00:00:00+05:30';
> +   VALID UNTIL '2049-12-31 10:30:00-08';
>
>   GRANT pg_monitor TO "Role2_$%{}[]()&*^!@""'`\/#" WITH ADMIN OPTION;
>
>   ALTER ROLE "Role2_$%{}[]()&*^!@""'`\/#" IN DATABASE postgres SET
> application_name TO 'pg4';
>
>   COMMENT ON ROLE "Role2_$%{}[]()&*^!@""'`\/#" IS 'This is detailed
> description';
> Alter Role options 3  Expected SQL File:alter_role_options3.sql... FAIL
> Alter Role options 4 (MSQL) ... ok
> Traceback (most recent call last):
>   File
> "/home/murtuza/projects/PEM/web/regression/re_sql/tests/test_resql.py",
> line 500, in check_re_sql
>     self.assertEquals(sql, resp_sql)
>   File "/usr/lib/python3.6/unittest/case.py", line 1321, in deprecated_func
>     return original_func(*args, **kwargs)
>   File "/usr/lib/python3.6/unittest/case.py", line 829, in assertEqual
>     assertion_func(first, second, msg=msg)
>   File "/usr/lib/python3.6/unittest/case.py", line 1203, in
> assertMultiLineEqual
>     self.fail(self._formatMessage(msg, standardMsg))
>   File "/usr/lib/python3.6/unittest/case.py", line 670, in fail
>     raise self.failureException(msg)
> AssertionError: '-- R[313 chars] \'2050-01-01 00:00:00+05:30\';\n\nGRANT
> pg_mo[259 chars]n\';' != '-- R[313 chars] \'2049-12-31
> 10:30:00-08\';\n\nGRANT pg_monit[256 chars]n\';'
>   -- Role: "Role2_$%{}[]()&*^!@""'`\/#"
>   -- DROP ROLE "Role2_$%{}[]()&*^!@""'`\/#";
>
>   CREATE ROLE "Role2_$%{}[]()&*^!@""'`\/#" WITH
>     NOLOGIN
>     SUPERUSER
>     INHERIT
>     CREATEDB
>     NOCREATEROLE
>     NOREPLICATION
>     CONNECTION LIMIT 100
>     ENCRYPTED PASSWORD 'md5cc9a2576a43b841dcacea0f65d961e41'
> -   VALID UNTIL '2050-01-01 00:00:00+05:30';
> +   VALID UNTIL '2049-12-31 10:30:00-08';
>
>   GRANT pg_monitor, pg_signal_backend TO "Role2_$%{}[]()&*^!@""'`\/#" WITH
> ADMIN OPTION;
>
>   ALTER ROLE "Role2_$%{}[]()&*^!@""'`\/#" IN DATABASE postgres SET
> application_name TO 'pg4';
>
>   COMMENT ON ROLE "Role2_$%{}[]()&*^!@""'`\/#" IS 'This is detailed
> description';
> Alter Role options 4  Expected SQL File:alter_role_options4.sql... FAIL
>
>
> --
> Regards,
> Murtuza Zabuawala
> *EDB*
> *POWER TO POSTGRES*
> https://www.edbpostgres.com
>
>
> On Thu, Aug 27, 2020 at 1:10 PM Akshay Joshi <
> akshay.jo...@enterprisedb.com> wrote:
>
>> Fixed cognitive complexity issues and added some more RESQL test cases
>> for roles module.
>>
>> Branch
>> ------
>> master
>>
>> Details
>> -------
>>
>> https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=53a54103379840795fc03f5940cb53dd99ff0ee9
>> Author: Aditya Toshniwal <aditya.toshni...@enterprisedb.com>
>>
>> Modified Files
>> --------------
>> .../server_groups/servers/roles/__init__.py        | 609
>> +++++++++------------
>> .../templates/roles/sql/9.4_plus/properties.sql    |   1 +
>> ..._role_options.msql => alter_role_options1.msql} |   4 +-
>> .../roles/tests/9.4_plus/alter_role_options1.sql   |  20 +
>> .../roles/tests/9.4_plus/alter_role_options2.msql  |   2 +
>> .../roles/tests/9.4_plus/alter_role_options2.sql   |  20 +
>> .../roles/tests/9.4_plus/alter_role_options3.msql  |   1 +
>> ...er_role_options.sql => alter_role_options3.sql} |   4 +-
>> .../roles/tests/9.4_plus/alter_role_options4.msql  |   1 +
>> .../roles/tests/9.4_plus/alter_role_options4.sql   |  19 +
>> .../servers/roles/tests/9.4_plus/test.json         |  54 +-
>> 11 files changed, 370 insertions(+), 365 deletions(-)
>>
>>

-- 
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | *edbpostgres.com*
<http://edbpostgres.com>
"Don't Complain about Heat, Plant a TREE"

Reply via email to