Re: pgAdmin4: Test-suite OS compatability issue

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Navnath Gadakh <navnath(dot)gadakh(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Kanchan Mohitey <kanchan(dot)mohitey(at)enterprisedb(dot)com>, Ikram Khan <ikram(dot)khan(at)enterprisedb(dot)com>
Subject: Re: pgAdmin4: Test-suite OS compatability issue
Date: 2017-05-02 11:09:55
Message-ID: CA+OCxozWSsEe=NWMd4GD7YvkcnHf60=1n8+RmCHUPOb--7YV8g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

When testing on Mac, the testsuite never exits with this patch applied. It
just hangs after printing the result summary.

On Fri, Apr 28, 2017 at 2:51 PM, Navnath Gadakh <
navnath(dot)gadakh(at)enterprisedb(dot)com> wrote:

> Hi Dave,
>
> Please find the patch for OS compatability issue of test-suite.
>
> *Code Changes:*
>
> 1. Minor code changes done related to os module in
> *pgadmin4/web/regression/feature_utils/app_starter.py*
> 2. Code added in pgadmin4/*web/pgadmin/utils/tests/test_versioned_template_loader.py
> *to handle path for SQL file.
> 3. Code added to convert unicode to string in some .py files.
>
> *Note:*
> With python2.6.6 test-suite is failed to execute. I have
> created the RM 2367 <https://redmine.postgresql.org/issues/2367>
>
> Thank you!
>
>
>
> --
> Regards,
> Navnath Gadakh
>
> EnterpriseDB Corporation
> The Enterprise PostgreSQL Company
>
>
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>

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

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-05-02 11:20:34 pgAdmin 4 commit: Ensure errors thrown when deleting rows in the query
Previous Message Dave Page 2017-05-02 11:00:38 Re: [Patch] Rename feature test server