Hi On Wed, May 9, 2018 at 8:10 AM, Akshay Joshi <akshay.jo...@enterprisedb.com> wrote:
> Thanks Anthony for fixing those two test cases. I have verified it and > patch looks good to me. > Attached is the modified patch where I have remove "import time" which is > not required and update the retry message from 10 to 60. > > To be honest I have seen some intermittent "Timeout" issues on *Chrome* > and *FireFox *and those issues are not consistent for any particular test > case. > > @Hackers, Please review it and if it looks good then commit it. > I seem to be getting the following failures consistently (screenshots attached): ====================================================================== ERROR: runTest (pgadmin.feature_tests.pg_datatype_validation_test.PGDataypeFeatureTest) Test checks for PG data-types output ---------------------------------------------------------------------- Traceback (most recent call last): File "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/pg_datatype_validation_test.py", line 135, in runTest self._check_datatype() File "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/pg_datatype_validation_test.py", line 169, in _check_datatype "contains(.,'{}')]".format(batch['datatype'][0]) File "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/selenium/webdriver/support/wait.py", line 80, in until raise TimeoutException(message, screen, stacktrace) TimeoutException: Message: ====================================================================== FAIL: runTest (pgadmin.feature_tests.xss_checks_file_manager_test.CheckFileManagerFeatureTest) Tests to check if File manager is vulnerable to XSS ---------------------------------------------------------------------- Traceback (most recent call last): File "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/xss_checks_file_manager_test.py", line 60, in runTest self._open_file_manager_and_check_xss_file() File "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/xss_checks_file_manager_test.py", line 110, in _open_file_manager_and_check_xss_file 'File manager' File "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/xss_checks_file_manager_test.py", line 117, in _check_escaped_characters ) != -1, "{0} might be vulnerable to XSS ".format(source) AssertionError: File manager might be vulnerable to XSS ---------------------------------------------------------------------- Ran 11 tests in 475.292s FAILED (failures=1, errors=1) ====================================================================== Test Result Summary ====================================================================== Regression - PG 9.4: 9 tests passed 2 tests failed: CheckFileManagerFeatureTest (Tests to check if File manager is vulnerable to XSS) PGDataypeFeatureTest (Test checks for PG data-types output) 0 tests skipped ====================================================================== Please check output in file: /Users/dpage/git/pgadmin4/web/regression/regression.log make: *** [check-feature] Error 1 -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company