Re: [RM2544][pgAdmin4] Add SSL database connection support in Regression tests

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [RM2544][pgAdmin4] Add SSL database connection support in Regression tests
Date: 2017-07-18 05:30:46
Message-ID: CAKKotZRbdv11wcS=DjvQx1_KMX1MURDBNat4_GLFdVEN7XcpVQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Dave,

Please find updated patch for the same.

On Mon, Jul 17, 2017 at 8:59 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> On Mon, Jul 17, 2017 at 2:59 PM, Murtuza Zabuawala <murtuza.zabuawala@
> enterprisedb.com> wrote:
>
>> Hi,
>>
>> PFA patch to add SSL support for database connection.
>> RM#2170
>>
>> Included test to check if current connection is using SSL or not
>>
>
> I think the SSL test needs a little work. Instead of:
>
> Test for SSL connection check ... Traceback (most recent call last):
> File "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tests/test_check_ssl_connection.py", line 42, in runTest
> self.server['sslmode']
> File "/System/Library/Frameworks/Python.framework/Versions/2.7/
> lib/python2.7/unittest/case.py", line 408, in skipTest
> raise SkipTest(reason)
> SkipTest: Cannot run SSL connection check test with 'prefer' sslmode
> skipped "Cannot run SSL connection check test with 'prefer' sslmode"
>
> I would expect to see:
>
> runTest (pgadmin.browser.server_groups.servers.tests.test_
> check_ssl_connection.TestSSLConnectionCheck)
> Test SSL Connection ... skipped "Cannot run SSL connection check test with
> 'prefer' sslmode"
> Thanks.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Attachment Content-Type Size
RM_2170_v1.diff text/plain 83.0 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Harshal Dhumal 2017-07-18 07:26:03 Re: [RM2074][[RM2080]][pgAdmin4] handle large bytea and bytea[] data in datagrid
Previous Message Shirley Wang 2017-07-17 19:51:19 Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard