Re: RE-SQL tests for Roles & Resource Group nodes

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: RE-SQL tests for Roles & Resource Group nodes
Date: 2019-07-03 03:53:36
Message-ID: CAKKotZRTCiUD0Y1+Zomiun2-fvEpQKJkkDvcwvhbFNpwV6CSHg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Dave,

PFA updated patch.

Please note that our framework considers test cases from all the modules as
one single test case, If we need module wise detailed summary report for
the reverse engineered SQL tests then we may have to change the framework.

Regards,
Murtuza

On Tue, Jul 2, 2019 at 5:52 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> On Tue, Jul 2, 2019 at 8:17 AM Murtuza Zabuawala <
> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>
>> ++ Attaching patch
>>
>> On Tue, Jul 2, 2019 at 12:46 PM Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Dave,
>>>
>>> Please find the updated patch,
>>>
>>> - Added a mechanism to check for supported minimum/maximum version and a
>>> server type in the framework.
>>>
>>
> There's still a problem with this I'm afraid - it says in the summary that
> the re_sql test cases were all skipped, which is not the case. I think we
> should just skip the irrelevant tests in the re_sql module, but not mark
> the entire module as being skipped:
>

> =============Running the test cases for 'Regression - PG 11'=============
>
> runTest (regression.re_sql.tests.test_resql.ReverseEngineeredSQLTestCases)
>
> Reverse Engineered SQL Test Cases
>
>
> Create IMPLICIT Cast
>
> Alter IMPLICIT Cast
>
> Drop IMPLICIT Cast
>
> Create EXPLICIT Cast
>
> Alter EXPLICIT Cast
>
> Drop EXPLICIT Cast
>
> ... skipped 'Server(Type: #ppas) is not supported for the Resource
> Groups module'
>
>
> ----------------------------------------------------------------------
>
> Ran 1 test in 0.480s
>
>
> OK (skipped=1)
>
>
> ======================================================================
>
> Test Result Summary
>
> ======================================================================
>
>
> Regression - PG 11:
>
>
> 0 tests passed
>
> 0 tests failed
>
> 1 test skipped:
>
> ReverseEngineeredSQLTestCases (Reverse Engineered SQL Test Cases)
>
>
> ======================================================================
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Attachment Content-Type Size
RM_4415_v2.diff application/octet-stream 38.4 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Aditya Toshniwal 2019-07-03 07:25:19 [pgAdmin][RM4404] Feature tests
Previous Message Akshay Joshi 2019-07-02 16:27:01 Re: pgAdmin4 v4.10 candidate builds