public inbox for [email protected]
help / color / mirror / Atom feedFrom: Dave Page <[email protected]>
To: Navnath Gadakh <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Cc: Kanchan Mohitey <[email protected]>
Subject: Re: pgAdmin IV: Unittests-drop objects functionality
Date: Wed, 14 Sep 2016 16:26:25 +0100
Message-ID: <CA+OCxozbgojCmSZn=acrQXM7ZR8Oi-vx9rVAS4qGa+4-JFOanA@mail.gmail.com> (raw)
In-Reply-To: <CAOAJCYpedt_jU=s+qzRkW=0UDR+FE+R2hNxjLzKQCVDJMxhrSw@mail.gmail.com>
References: <CAOAJCYoUGdpgm19D8tq2V2tbPqzyHesnRObMfoBpv9WVZWSKUQ@mail.gmail.com>
<CA+OCxoxVx2L+iO_37wsmFCzeYX17-P5Rz8SC5G272-wDRXTA0g@mail.gmail.com>
<CAOAJCYpedt_jU=s+qzRkW=0UDR+FE+R2hNxjLzKQCVDJMxhrSw@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
Cool - thanks, committed!
On Wed, Sep 14, 2016 at 2:57 PM, Navnath Gadakh
<[email protected]> wrote:
> Hi Dave,
>
> Please find the revised patch with suggested modifications.
>
> On Wed, Sep 14, 2016 at 5:25 PM, Dave Page <[email protected]>
> wrote:
>>
>> Hi
>>
>> On Tue, Sep 13, 2016 at 3:39 PM, Navnath Gadakh
>> <[email protected]> wrote:
>> > Hi Dave,
>> > Please find the patch for drop objects functionality for test
>> > framework.
>> > Now, test framework is able to drop all objects(like server, database,
>> > schema, etc.) which are created during test suite execution. Code
>> > included
>> > in the patch is only for server and database node. If everything goes
>> > fine
>> > will send next patch with the remaining nodes in a single patch.
>> >
>> > The following files are also affected by the new changes:
>> > web/setup.py,
>> > web/config.py,
>> > regression/test_advanced_config.json.in
>> >
>> >
>> > To run the test suite follow the commands
>> > 1. python runtionregress/ests.py --pkg browser.server_groups.servers
>> > 2. python regression/runtests.yp --pkg
>> > browser.server_groups.servers.databases
>> >
>> >
>> > Note: Please do not run test suite for all nodes (python
>> > regression/runtests.py) as code modifications have not been done yet.
>>
>> I'm happy with this in general, except the changes in setup.py
>>
>> - Let's use the names PGADMIN_SETUP_EMAIL and PGADMIN_SETUP_PASSWORD
>> for the envvars.
>
> Sure.
>>
>>
>> - Why are you setting the username and password in the environment
>> after prompting the user for them? I can't see why you're doing that,
>> and it's almost certainly a security issue.
>
> I was trying some different logic, by mistake that code remains, removed
> that code in the attached patch.
>>
>>
>> --
>> Dave Page
>> VP, Chief Architect, Tools & Installers
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>
>
>
>
> --
> Thanks,
> Navnath Gadakh
> Software Engineer
> EnterpriseDB Corporation
> Mobile: +91 9975389878
>
>
--
Dave Page
VP, Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
--
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
view thread (4+ messages)
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], [email protected]
Subject: Re: pgAdmin IV: Unittests-drop objects functionality
In-Reply-To: <CA+OCxozbgojCmSZn=acrQXM7ZR8Oi-vx9rVAS4qGa+4-JFOanA@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