From: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: RM 4463 - MSQL test cases for domain node |
Date: | 2019-09-02 06:05:21 |
Message-ID: | CACCA4P1kPTG+YK4PAJT5XbDNAt785KgJUHkGuKbtb9vd-+rTCg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi,
Please find attached updated patch. I have tested with PG 9.4, PG 10 & EPAS
10.
Do review it and let me know in case of any failures.
Thanks,
Neel Patel
On Fri, Aug 23, 2019 at 1:52 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Hi
>
> On Wed, Aug 21, 2019 at 8:58 AM Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
> wrote:
>
>> Hi,
>>
>> Please find attached patch that will add more resql test cases and msql
>> test cases for domain node.
>>
>> - Covered test cases for Length, Precision and Collation.
>> - Also added msql test cases.
>>
>> While writing resql test cases for domain node, I found one more issue
>> RM-4644 <https://redmine.postgresql.org/issues/4644>. I have sent
>> separate patch for the same.
>>
>
> I'm getting the following error with this patch:
>
> ======================================================================
> ERROR: runTest
> (regression.re_sql.tests.test_resql.ReverseEngineeredSQLTestCases)
> Reverse Engineered SQL Test Cases
> ----------------------------------------------------------------------
> Traceback (most recent call last):
> File
> "/Users/dpage/git/pgadmin4/web/regression/re_sql/tests/test_resql.py", line
> 126, in runTest
> self.execute_test_case(scenarios)
> File
> "/Users/dpage/git/pgadmin4/web/regression/re_sql/tests/test_resql.py", line
> 204, in execute_test_case
> if not self.check_msql(scenario, object_id):
> File
> "/Users/dpage/git/pgadmin4/web/regression/re_sql/tests/test_resql.py", line
> 329, in check_msql
> object_id)
> File
> "/Users/dpage/git/pgadmin4/web/regression/re_sql/tests/test_resql.py", line
> 178, in get_url
> object_url = url_for(rule.endpoint, **options)
> File
> "/Users/dpage/.virtualenvs/pgadmin4/lib/python3.7/site-packages/flask/helpers.py",
> line 356, in url_for
> return appctx.app.handle_url_build_error(error, endpoint, values)
> File
> "/Users/dpage/.virtualenvs/pgadmin4/lib/python3.7/site-packages/flask/app.py",
> line 2061, in handle_url_build_error
> reraise(exc_type, exc_value, tb)
> File
> "/Users/dpage/.virtualenvs/pgadmin4/lib/python3.7/site-packages/flask/_compat.py",
> line 35, in reraise
> raise value
> File
> "/Users/dpage/.virtualenvs/pgadmin4/lib/python3.7/site-packages/flask/helpers.py",
> line 345, in url_for
> force_external=external)
> File
> "/Users/dpage/.virtualenvs/pgadmin4/lib/python3.7/site-packages/werkzeug/routing.py",
> line 1776, in build
> raise BuildError(endpoint, values, method, self)
> werkzeug.routing.BuildError: Could not build url for endpoint
> 'NODE-domain.msql_id' with values ['did', 'gid', 'scid', 'sid']. Did you
> forget to specify values ['doid']?
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
Attachment | Content-Type | Size |
---|---|---|
RM_4463_V3.patch | application/octet-stream | 31.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2019-09-02 06:22:32 | pgAdmin 4 commit: Added MSQL test cases for Domain. |
Previous Message | Domenico Sgarbossa | 2019-09-01 10:49:42 | Re: Italian translation |