Re: [pgAdmin][RM5330]: [Code Coverage] Improve API test cases for Functions

From: Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM5330]: [Code Coverage] Improve API test cases for Functions
Date: 2020-09-22 07:14:31
Message-ID: CAOBg0AN==k1y7fj=LDOzBj2pbbBg1vwM3qWtf9kNxgKBKjhHmg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Akshay/Team,

I checked the code and found an issue in the "msql" function, due to which
test cases were failing.
I have fixed the issue, PFA updated patch for the same.
Sorry for the inconvenience.

Regards,
Nikhil Mohite.

On Tue, Sep 22, 2020 at 11:41 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Hi Nikhil
>
> Test cases are failing for all the database server with the following
> error:
> return True, sql.strip('\n')
> AttributeError: 'tuple' object has no attribute 'strip'
>
> Please check and resend the patch.
>
> On Tue, Sep 22, 2020 at 10:56 AM Nikhil Mohite <
> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>
>> Hi Team,
>>
>> I have added few more test cases for the "Functions" module as per
>> RM-5330 <https://redmine.postgresql.org/issues/5330>
>> PFA patch.
>>
>> --
>> *Thanks & Regards,*
>> *Nikhil Mohite*
>> *Software Engineer.*
>> *EDB Postgres* <https://www.enterprisedb.com/>
>> *Mob.No: +91-7798364578.*
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Sr. Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>

Attachment Content-Type Size
RM_5330_v2.patch application/octet-stream 74.0 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2020-09-22 09:26:29 Re: [pgAdmin][SonarQube] Reduce cognitive complexity
Previous Message Pradip Parkale 2020-09-22 06:40:22 Re: RM5841: server.username can be empty when using .pg_service.conf