Re: fixes for browser.tests api test cases - Mailing list pgadmin-hackers

From Dave Page
Subject Re: fixes for browser.tests api test cases
Date
Msg-id CA+OCxozPUp-1ur8hA7EBsGu74Me-Ayww=uQdNnfTgbQ+0BTz3w@mail.gmail.com
Whole thread Raw
In response to fixes for browser.tests api test cases  (Usman Muzaffar <usman.muzaffar@enterprisedb.com>)
Responses Re: fixes for browser.tests api test cases  (Usman Muzaffar <usman.muzaffar@enterprisedb.com>)
List pgadmin-hackers
Hi

On Tue, Oct 16, 2018 at 1:35 PM Usman Muzaffar <usman.muzaffar@enterprisedb.com> wrote:
Hi Hackers,

Please find here attached patch for the following test cases,(in package pgadmin.browser.tests) which were failing.
  • test_change_password
  • test_gravatar_image_display
  • test_login
  • test_logout
  • test_reset_password
  • test_version_in_range

following fixes are provided
  • Overridden base class setUp function, as if there is any issue in the base class setup code, it fails the test case uselessly.
  • Changed 'active' parameter value to boolean from integer in following api '/user_management/user/'
         in the test case ,as the relevant post function accepts value in boolean. Currently due to this the test case was failing.


Please review the patch and commit.

Note: I have verified it on Python2.7 & Python3.6

That does fix most issues, however I still see this:

======================================================================

FAIL: runTest (pgadmin.browser.tests.test_internal_version.InternalVersionTestCase)

TestCase with INTERNAL_VERSION_PARAM present

----------------------------------------------------------------------

Traceback (most recent call last):

  File "/Users/dpage/git/pgadmin4/web/pgadmin/browser/tests/test_internal_version.py", line 60, in runTest

    self.assertTrue(url.endswith(version_string))

AssertionError: False is not true


----------------------------------------------------------------------

Another issue of note is this seen when I try to run the feature tests. Notice how the counts of tests don't add up?

----------------------------------------------------------------------

Ran 16 tests in 0.006s


OK (skipped=16)


======================================================================

Test Result Summary

======================================================================


Regression - PG 9.6:


1 test passed

0 tests failed

15 tests skipped:

BrowserToolBarFeatureTest (Browser tool bar feature test)

CopySelectedQueryResultsFeatureTest (Copy rows, column using button and keyboard shortcut)

CheckFileManagerFeatureTest (File manager feature test)

KeyboardShortcutFeatureTest (Test for keyboard shortcut)

PGDataypeFeatureTest (Test checks for PG data-types output)

PGUtilitiesBackupFeatureTest (Test for PG utilities - Backup and Restore)

PGUtilitiesMaintenanceFeatureTest (Test for PG maintenance: database,

Test for PG maintenance: table)

QueryToolAutoCompleteFeatureTest (Query tool auto complete feature test)

QueryToolJourneyTest (Tests the path through the query tool)

QueryToolFeatureTest (Query tool feature test)

TableDdlFeatureTest (Test table DDL generation)

CheckForViewDataTest (Validate Insert, Update operations in View/Edit data with given test data)

CheckForXssFeatureTest (Test XSS check for panels and query tool)

CheckDebuggerForXssFeatureTest (Tests to check if Debugger is vulnerable to XSS)

CheckRoleMembershipControlFeatureTest (Tests to check if Role membership control is vulnerable to XSS)


======================================================================

Thanks! 

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

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

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin4][patch] Changes to stabilise feature tests
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Fix syntax error when creating new pgAgentschedules