On 27 July 2016 at 20:12, Dave Page <dave.p...@enterprisedb.com> wrote:
> Ahh, that explains it. Nice catch! It's working in both server and > desktop modes now :-). > Great. Thanks to Khushboo for helping Navnath in this task. Thank you. > On Wed, Jul 27, 2016 at 1:19 PM, Navnath Gadakh > <navnath.gad...@enterprisedb.com> wrote: > > 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.p...@enterprisedb.com> > wrote: > >> Hi > >> > >> On Fri, Jul 22, 2016 at 2:22 PM, Navnath Gadakh > >> <navnath.gad...@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 > > > > -- > Dave Page > VP, Chief Architect, Tools & Installers > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Best, Priyanka EnterpriseDB Corporation The Enterprise PostgreSQL Company
