Re: pgAdmin IV : Unittest modular patch - Mailing list pgadmin-hackers
From | Navnath Gadakh |
---|---|
Subject | Re: pgAdmin IV : Unittest modular patch |
Date | |
Msg-id | CAOAJCYpcye6nQAPB8os9vZXOhAf9wEvr6rA9i4vuZLDVrRYF0g@mail.gmail.com Whole thread Raw |
In response to | Re: pgAdmin IV : Unittest modular patch (Dave Page <dave.page@enterprisedb.com>) |
Responses |
Re: pgAdmin IV : Unittest modular patch
(Dave Page <dave.page@enterprisedb.com>)
|
List | pgadmin-hackers |
Hi Dave, We have reproduced the issue you mentioned in the previous email. We need to skip the login related test cases when SERVER_MODE = False in the web/config_local.py. Please find the revised patch for the same. Thanks! On Fri, Jul 22, 2016 at 9:48 PM, Dave Page <dave.page@enterprisedb.com> wrote: > Hi > > On Fri, Jul 22, 2016 at 2:22 PM, Navnath Gadakh > <navnath.gadakh@enterprisedb.com> 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 need to >> modify test_config.json(for user it’s test_config.json.in) and >> test_advanced_config.json(for user it’s test_advanced_config.json.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 ‘browser/tests/’ as for test scenarios like change password and >> invalid login test cases as test client should be logged out first. So, as >> per this the code is slightly modified in ‘browser/tests/’. > > 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 > > ====================================================================== > 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'<!DOCTYPE > HTML PUBLIC "-//W3C//DTD HTML 3.2 > Final//EN">\n<title>Redirecting...</title>\n<h1>Redirecting...</h1>\n<p>You > should be redirected automatically to target URL: <a href="/">/</a>. > If not click the link.' > > ====================================================================== > 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'<!DOCTYPE > HTML PUBLIC "-//W3C//DTD HTML 3.2 > Final//EN">\n<title>Redirecting...</title>\n<h1>Redirecting...</h1>\n<p>You > should be redirected automatically to target URL: <a href="/">/</a>. > If not click the link.' > > ====================================================================== > 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'<!DOCTYPE > HTML PUBLIC "-//W3C//DTD HTML 3.2 > Final//EN">\n<title>Redirecting...</title>\n<h1>Redirecting...</h1>\n<p>You > should be redirected automatically to target URL: <a href="/">/</a>. > If not click the link.' > > ---------------------------------------------------------------------- > Ran 26 tests in 30.109s > > FAILED (failures=3) > > 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 -- Thanks, Navnath Gadakh Software Engineer EnterpriseDB Corporation Mobile: +91 9975389878
Attachment
pgadmin-hackers by date: