Re: [pgadmin4][patch] Use pytest test runner for unit tests - Mailing list pgadmin-hackers

From Khushboo Vashi
Subject Re: [pgadmin4][patch] Use pytest test runner for unit tests
Date
Msg-id CAFOhELfp-YYctKk5BW228msAh2RmJcxE=QQR7PmhYhrHsr-GPw@mail.gmail.com
Whole thread Raw
In response to Re: [pgadmin4][patch] Use pytest test runner for unit tests  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi,

The supported versions of 
Pytest: Python 2.7, Python 3.4+, PyPy 2.3, Jython 2.5 (untested);
Grappa: Python 2.7+, 3+

References:

So, how are we going to handle unsupported versions (PY 2.6 and 3.3) ?

Thanks,
Khushboo

On Wed, Jun 13, 2018 at 6:07 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Tue, Jun 12, 2018 at 3:41 PM, Victoria Henry <vhenry@pivotal.io> wrote:
Hi Dave,

Attached are four different patches that should be applied in order.  Included is a fix for the failing test above.

This is no longer failing :-)

However, I'm still seeing problems that need to be resolved:

- Passwords are still being written to the log file. This is a complete show-stopper.

- The regression.log file is empty. I'm now only getting the JSON summary. We really need both - the regression log for diagnostics, and the JSON file for feeding into monitoring systems.

- There is still no summary list of tests that failed or were skipped output at the end of the run. This is important because it highlights which tests failed (obviously!) and need investigation, and knowing which ones are skipped at a glance is very useful as they can be skipped due to mis-configuration; e.g. running against a different server than intended, or failing to specify something like a tablespace path which we can then easily see to fix.

- I cannot run against servers other than the first one in the config file. We need to run against all that are enabled, and have the output show which one was used for each one as we have now. This may seem trivial, but it's really not - to the point that for some of us it negates the whole point of the patch by giving us more control over which tests are run but removing all control over which database servers they will run against. In my case, this will almost certainly cost even more time; right now I just have to run more tests than I may need but I can do other things at the same time. With this patch I'll have to keep re-running the tests, modifying the config between each run.

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: pgAdmin 4 Jenkins
Date:
Subject: Build failed in Jenkins: pgadmin4-master-python33 #662
Next
From: Khushboo Vashi
Date:
Subject: [pgAdmin4][Patch]: Improve the database migration steps in README