Re: pgAdmin IV : Unittest modular patch(database child nodes) and trigger function - Mailing list pgadmin-hackers

From Priyanka Shendge
Subject Re: pgAdmin IV : Unittest modular patch(database child nodes) and trigger function
Date
Msg-id CAKmZXFRiapbdOJKGW9KJ1Uz4OcShOOx4iRHgtTsnE_RsRzgG8Q@mail.gmail.com
Whole thread Raw
In response to Re: pgAdmin IV : Unittest modular patch(database child nodes) and trigger function  (Dave Page <dave.page@enterprisedb.com>)
Responses Re: pgAdmin IV : Unittest modular patch(database child nodes) and trigger function  (Priyanka Shendge <priyanka.shendge@enterprisedb.com>)
List pgadmin-hackers


On 23 August 2016 at 18:38, Dave Page <dave.page@enterprisedb.com> wrote:


On Tue, Aug 23, 2016 at 1:56 PM, Priyanka Shendge <priyanka.shendge@enterprisedb.com> wrote:
I noticed whilst testing that I've started seeing the error below
(with or without the patches I just applied). Any idea what's causing
it?

The user may already present in database (i.e. abc@EXAMPLE.COM) which we creating as dummy user.
When we run test case it tries to create dummy user. As user is already present it throws an error.
It does not get an id for same user.

Ahh, yes - can you add a check for that so it throws a more helpful error please?
 
Sure, i'll update the code accordingly. 

Also; related to the issue I've been seeing with databases not being removed, I recently ran into problems where tablespaces weren't removed, following a ctrl+c, which then required manual cleanup once I'd figured out what was wrong. We need to fix this; I'd suggest:

- Keep track of objects that are created during tests. This probably just needs to be objects to the database level, e.g. pgAdmin users, tablespaces, databases and roles (because, if we drop the test database then everything in it will go anyway).

- Remove objects from the list again if a test drops them.

- Call a function on exit (or abort/ctrl+c/whatever) that iterates through the list of objects that have been created but not dropped, and clean them up manually before exiting. 

Sure, this can be done and will update the code. 
Thank you.

Thanks!

--
Dave Page
VP, Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake



--
Best,
Priyanka

EnterpriseDB Corporation
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: pgAdmin IV : Unittest modular patch(database child nodes) and trigger function
Next
From: Ashesh Vashi
Date:
Subject: Re: Fixed RM #1209 Trigger issues through right click