Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1bRjCU-0000XI-AZ for pgadmin-hackers@arkaria.postgresql.org; Mon, 25 Jul 2016 16:57:54 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1bRjCT-0003Au-Jn for pgadmin-hackers@arkaria.postgresql.org; Mon, 25 Jul 2016 16:57:53 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1bRjCG-0002w0-98 for pgadmin-hackers@postgresql.org; Mon, 25 Jul 2016 16:57:40 +0000 Received: from mail-qt0-x235.google.com ([2607:f8b0:400d:c0d::235]) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.84_2) (envelope-from ) id 1bRjCA-0002e9-KX for pgadmin-hackers@postgresql.org; Mon, 25 Jul 2016 16:57:39 +0000 Received: by mail-qt0-x235.google.com with SMTP id w38so100712945qtb.0 for ; Mon, 25 Jul 2016 09:57:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=enterprisedb-com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=3VlCh/JhIGlReIX3d3ARcUMKuOgZpt/wnI9G6+dwT4U=; b=OobTjsXTbAy3UqmFLX1OFttFdfOE7bbb8HiV2lRSPz9tYaNIK/SRsnwbLWSlNJJTbP 4A+kWg4GAjT7aD4ekqeUSxU5JUWrd3s33+7Beaw6lMSOAb3cwuvdvlBPhhGRJ/1C5VSE W5t1v2ydsgQcQRbb7IuG39hM4CsoHXMIPl5Du0xmsC1jWeLDyn51KzM9HPhzmLFg5eE3 w9HlTYD5kjPW3TxM6x+U8eFg80jXnonp5LDYC+gyBGDrRxJMVezRj3Z0olDmPsgCQruX gnCu+b5peVmRdSVR/20qUBflf5cjshfj/F3G5cBOXtiiGu9wMZISqbOLVInlIk0XN3DN Nz5A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=3VlCh/JhIGlReIX3d3ARcUMKuOgZpt/wnI9G6+dwT4U=; b=U5H2eyR+YMZ73UosKeRjieY53GmqeSTLQRX65DxYrcSdXjoejEwqxlA8aV9BjFU8PG Hhkyaz6XTZXn6h5BaNcCtZ+CB94eqyCNA4hZkeDkCFkc7/CobiNNcz/nISLj0e0BDS1F zvUEHyiaHs1pUKHChObh5GxDg+s6J5ugbUs5hK16Ncq4hFPY8r1QPNtLeeBy4fse8RoA PY7SBZVrdNCJDAJY0j29obUyVgiW0IQkoyW/zrH5lJ7gbcpzOPfiTAQoNkQP89C79PSr HGWzOhO4R5gs5EBk8pR6bYEqQYU9AnjSJxTWxllCqXBfvCelgfTsBLnhYWguk+NLGcV2 MSBg== X-Gm-Message-State: AEkooutrOg2gtNIicdqciaBn0Gq97kCSs3dPLSuI25gdVmTwc0xBC7VONXdfj03XsewGlHI7ldKMSC4CUcmTw6FD X-Received: by 10.200.49.129 with SMTP id h1mr30954788qte.103.1469465852959; Mon, 25 Jul 2016 09:57:32 -0700 (PDT) MIME-Version: 1.0 Received: by 10.200.41.90 with HTTP; Mon, 25 Jul 2016 09:57:32 -0700 (PDT) In-Reply-To: References: From: Navnath Gadakh Date: Mon, 25 Jul 2016 22:27:32 +0530 Message-ID: Subject: Re: pgAdmin IV : Unittest modular patch To: Dave Page Cc: pgadmin-hackers , Kanchan Mohitey Content-Type: multipart/alternative; boundary=001a11c003862cf389053878ae46 X-Pg-Spam-Score: -2.6 (--) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgadmin-hackers Precedence: bulk Sender: pgadmin-hackers-owner@postgresql.org --001a11c003862cf389053878ae46 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Hi Dave, I tried to reproduce this issue at my end, but didn't work. Also tried with clean GIT tree. Will look more into this tomorrow. Thanks! On Fri, Jul 22, 2016 at 9:48 PM, Dave Page wrote: > Hi > > On Fri, Jul 22, 2016 at 2:22 PM, Navnath Gadakh > wrote: > > Hi Dave, > > > > Please find the attached patch for pgAdmin4 unittest(Support of API > > testing with different servers i.e. PG and PPAS). > > > > This patch includes- > > > > Test framework support API testing with multiple server for this we nee= d > to > > modify test_config.json(for user it=E2=80=99s test_config.json.in) and > > test_advanced_config.json(for user it=E2=80=99s test_advanced_config.js= on.in). > > Server details of PG and PPAS are included in both .in files. > > > > Removed the logic of logging in the test client on each test scenario(= As > > per Khushboo's comment in previous email). We need this logic in test > cases > > under =E2=80=98browser/tests/=E2=80=99 as for test scenarios like chang= e password and > > invalid login test cases as test client should be logged out first. So, > as > > per this the code is slightly modified in =E2=80=98browser/tests/=E2=80= =99. > > OK in principle, but it doesn't seem to work: > > This function will add the server under default server group. (Default > Server Node url) ... ok > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > FAIL: runTest > (pgadmin.browser.tests.test_reset_password.ResetPasswordTestCase) > This function checks reset password functionality. (TestCase for > Validating Empty Email) > ---------------------------------------------------------------------- > Traceback (most recent call last): > File > "/Users/dpage/git/pgadmin4/web/pgadmin/browser/tests/test_reset_password.= py", > line 44, in runTest > 'utf-8')) > AssertionError: 'Recover pgAdmin 4 Password' not found in u' HTML PUBLIC "-//W3C//DTD HTML 3.2 > Final//EN">\nRedirecting...\n

Redirecting...

\n

Y= ou > should be redirected automatically to target URL: /. > If not click the link.' > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > FAIL: runTest > (pgadmin.browser.tests.test_reset_password.ResetPasswordTestCase) > This function checks reset password functionality. (TestCase for > Validating Invalid_Email) > ---------------------------------------------------------------------- > Traceback (most recent call last): > File > "/Users/dpage/git/pgadmin4/web/pgadmin/browser/tests/test_reset_password.= py", > line 44, in runTest > 'utf-8')) > AssertionError: 'Recover pgAdmin 4 Password' not found in u' HTML PUBLIC "-//W3C//DTD HTML 3.2 > Final//EN">\nRedirecting...\n

Redirecting...

\n

Y= ou > should be redirected automatically to target URL: /. > If not click the link.' > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > FAIL: runTest > (pgadmin.browser.tests.test_reset_password.ResetPasswordTestCase) > This function checks reset password functionality. (TestCase for > Validating Valid_Email) > ---------------------------------------------------------------------- > Traceback (most recent call last): > File > "/Users/dpage/git/pgadmin4/web/pgadmin/browser/tests/test_reset_password.= py", > line 44, in runTest > 'utf-8')) > AssertionError: 'Recover pgAdmin 4 Password' not found in u' HTML PUBLIC "-//W3C//DTD HTML 3.2 > Final//EN">\nRedirecting...\n

Redirecting...

\n

Y= ou > should be redirected automatically to target URL: /. > If not click the link.' > > ---------------------------------------------------------------------- > Ran 26 tests in 30.109s > > FAILED (failures=3D3) > > It's very far from obvious what's wrong here (/reset looks just fine > to me). How can we get more useful info out of the framework? > > -- > Dave Page > VP, Chief Architect, Tools & Installers > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > --=20 Thanks, Navnath Gadakh Software Engineer EnterpriseDB Corporation Mobile: +91 9975389878 --001a11c003862cf389053878ae46 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

Hi Dave,
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0I tried to reproduce this issue at my end, but didn't work. Also tri= ed with clean GIT tree.
Will look more into this tomorrow.
<= div>Thanks!

On Fri, Jul 22, 2016 at 9:48 PM, Dave Page <dave.page@enter= prisedb.com> wrote:
Hi

On Fri, Jul 22, 2016 at 2:22 PM, Navnath Gadakh
<navnath.gadakh@enter= prisedb.com> wrote:
> Hi Dave,
>
>=C2=A0 =C2=A0 Please find the attached patch for pgAdmin4 unittest(Supp= ort of API
> testing with different servers i.e. PG and PPAS).
>
> This patch includes-
>
> Test framework support API testing with multiple server for this we ne= ed to
> modify test_config.json(for user it=E2=80=99s test_config.json.in= ) and
> test_advanced_config.json(for user it=E2=80=99s test_advance= d_config.json.in).
> Server details of PG and=C2=A0 PPAS are included in both .in files. >
> Removed the logic of logging in=C2=A0 the test client on each test sce= nario(As
> per Khushboo's comment in previous email).=C2=A0 We need this logi= c in test cases
> under =E2=80=98browser/tests/=E2=80=99 as for test scenarios like chan= ge password and
> invalid login test cases as test client should be logged out first. So= , as
> per this the code is slightly modified in =E2=80=98browser/tests/=E2= =80=99.

OK in principle, but it doesn't seem to work:

This function will add the server under default server group. (Default
Server Node url) ... ok

=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
FAIL: runTest (pgadmin.browser.tests.test_reset_password.ResetPasswordTestC= ase)
This function checks reset password functionality. (TestCase for
Validating Empty Email)
----------------------------------------------------------------------
Traceback (most recent call last):
=C2=A0 File "/Users/dpage/git/pgadmin4/web/pgadmin/browser/test= s/test_reset_password.py",
line 44, in runTest
=C2=A0 =C2=A0 'utf-8'))
AssertionError: 'Recover pgAdmin 4 Password' not found in u'<= ;!DOCTYPE
HTML PUBLIC "-//W3C//DTD HTML 3.2
Final//EN">\n<title>Redirecting...</title>\n<h1>R= edirecting...</h1>\n<p>You
should be redirected automatically to target URL: <a href=3D"/"= ;>/</a>.
If not click the link.'

=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
FAIL: runTest (pgadmin.browser.tests.test_reset_password.ResetPasswordTestC= ase)
This function checks reset password functionality. (TestCase for
Validating Invalid_Email)
----------------------------------------------------------------------
Traceback (most recent call last):
=C2=A0 File "/Users/dpage/git/pgadmin4/web/pgadmin/browser/test= s/test_reset_password.py",
line 44, in runTest
=C2=A0 =C2=A0 'utf-8'))
AssertionError: 'Recover pgAdmin 4 Password' not found in u'<= ;!DOCTYPE
HTML PUBLIC "-//W3C//DTD HTML 3.2
Final//EN">\n<title>Redirecting...</title>\n<h1>R= edirecting...</h1>\n<p>You
should be redirected automatically to target URL: <a href=3D"/"= ;>/</a>.
If not click the link.'

=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
FAIL: runTest (pgadmin.browser.tests.test_reset_password.ResetPasswordTestC= ase)
This function checks reset password functionality. (TestCase for
Validating Valid_Email)
----------------------------------------------------------------------
Traceback (most recent call last):
=C2=A0 File "/Users/dpage/git/pgadmin4/web/pgadmin/browser/test= s/test_reset_password.py",
line 44, in runTest
=C2=A0 =C2=A0 'utf-8'))
AssertionError: 'Recover pgAdmin 4 Password' not found in u'<= ;!DOCTYPE
HTML PUBLIC "-//W3C//DTD HTML 3.2
Final//EN">\n<title>Redirecting...</title>\n<h1>R= edirecting...</h1>\n<p>You
should be redirected automatically to target URL: <a href=3D"/"= ;>/</a>.
If not click the link.'

----------------------------------------------------------------------
Ran 26 tests in 30.109s

FAILED (failures=3D3)

It's very far from obvious what's wrong here (/reset looks just fin= e
to me). How can we get more useful info out of the framework?

--
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= =C2=A0Gadakh
Software Engineer
EnterpriseDB Corporation
<= /font>
Mobile: +91 9975389878=C2=A0

<= /font>

=
--001a11c003862cf389053878ae46--