Re: fixes for browser.tests api test cases - Mailing 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