Hi Khushboo The tests are still failing and seems flaky: https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-tests/builds/113
Sincerely, Victoria On Tue, Jun 5, 2018 at 4:50 AM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > > > On Tue, Jun 5, 2018 at 2:09 PM, Dave Page <dp...@pgadmin.org> wrote: > >> >> >> On Tue, Jun 5, 2018 at 9:37 AM, Khushboo Vashi < >> khushboo.va...@enterprisedb.com> wrote: >> >>> >>> >>> On Tue, Jun 5, 2018 at 1:36 PM, Dave Page <dp...@pgadmin.org> wrote: >>> >>>> Hi >>>> >>>> On Tue, Jun 5, 2018 at 4:39 AM, Khushboo Vashi < >>>> khushboo.va...@enterprisedb.com> wrote: >>>> >>>>> >>>>> >>>>> On Mon, Jun 4, 2018 at 8:41 PM, Joao De Almeida Pereira < >>>>> jdealmeidapere...@pivotal.io> wrote: >>>>> >>>>>> Hi Khushboo, >>>>>> >>>>>> Some tests are failing in greenplum: >>>>>> >>>>>> https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-tests/builds/108 >>>>>> The piece of code responsible for the error is: >>>>>> >>>>>> if server['default_binary_paths'] is not None: >>>>>> test_utils.set_preference(server['default_binary_paths']) >>>>>> >>>>>> config.DEFAULT_BINARY_PATHS = { >>>>>> "pg": str(server['default_binary_paths']['pg']), >>>>>> "ppas": str(server['default_binary_paths']['ppas']), >>>>>> "gpdb": "" >>>>>> } >>>>>> >>>>>> >>>>>> Can you send me the test_config.json file? The above code sets the >>>>> paths to the SQLite database and through the logs couldn't figure out the >>>>> exact failure. >>>>> >>>> >>>> It seems clear from the code shown that it's not setting the binary >>>> paths for gpdb database servers. Shouldn't it be something like: >>>> >>>> config.DEFAULT_BINARY_PATHS = { >>>> "pg": str(server['default_binary_paths']['pg']), >>>> "ppas": str(server['default_binary_paths']['ppas']), >>>> "gpdb": str(server['default_binary_paths']['gpdb']) >>>> } >>>> >>>> Without this code, the test cases should work as I already set paths >>> through below code. >>> >>> test_utils.set_preference(server['default_binary_paths']) >>> >>> >> In that case, why is the code above required at all? >> >> My bad. Removed this code and also updated set_preference function for > greenplum database. > Please find the attached updated patch. > >> >> >>> >>>> >>>>> test_backup_utils.py file name is misleading, these are not tests, >>>>>> are helpers. >>>>>> >>>>>> >>>>>> >>>>>> Thanks >>>>>> Victoria & Joao >>>>>> >>>>>> On Mon, Jun 4, 2018 at 1:36 AM Khushboo Vashi < >>>>>> khushboo.va...@enterprisedb.com> wrote: >>>>>> >>>>>>> >>>>>>> >>>>>>> On Sat, Jun 2, 2018 at 3:01 AM, Dave Page <dp...@pgadmin.org> wrote: >>>>>>> >>>>>>>> Hi >>>>>>>> >>>>>>>> This looks good, except that it's leaving the test_restore_database >>>>>>>> behind. Can we clean that up please? >>>>>>>> >>>>>>>> PFA updated patch. >>>>>>> >>>>>>>> Thanks. >>>>>>>> >>>>>>>> On Fri, Jun 1, 2018 at 7:06 AM, Khushboo Vashi < >>>>>>>> khushboo.va...@enterprisedb.com> wrote: >>>>>>>> >>>>>>>>> Hi Victoria, >>>>>>>>> >>>>>>>>> Thanks for reviewing the patch. >>>>>>>>> The tests were failing due to the latest commit >>>>>>>>> #2b4605a9d390cb44e5dfe9967c3adf2b28d04f1f - Ensure >>>>>>>>> backup/restore/maintenance work via SSH tunnels. Fixes #3355 >>>>>>>>> >>>>>>>>> I have fixed the issues and attached the updated patch. >>>>>>>>> >>>>>>>>> Thanks, >>>>>>>>> Khushboo >>>>>>>>> >>>>>>>>> On Thu, May 31, 2018 at 10:00 PM, Victoria Henry < >>>>>>>>> vhe...@pivotal.io> wrote: >>>>>>>>> >>>>>>>>>> Hi there, >>>>>>>>>> >>>>>>>>>> We've been noticing some issues with the tests on both our CI and >>>>>>>>>> local Mac workstations. >>>>>>>>>> >>>>>>>>>> 1. When the following code blocks are invoked - we get plenty >>>>>>>>>> of app.context() issues. It must not be valid when running >>>>>>>>>> tests. >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> from pgadmin.utils.driver import get_driver >>>>>>>>>> driver = get_driver(PG_DEFAULT_DRIVER) >>>>>>>>>> manager = driver.connection_manager(self.sid) >>>>>>>>>> >>>>>>>>>> host = manager.local_bind_host if manager.use_ssh_tunnel else s.host >>>>>>>>>> port = manager.local_bind_port if manager.use_ssh_tunnel else s.port >>>>>>>>>> >>>>>>>>>> 2. When we finally enable >>>>>>>>>> >>>>>>>>>> "default_binary_paths": { >>>>>>>>>> >>>>>>>>>> in our test_config, we get more failing tests that look like: >>>>>>>>>> >>>>>>>>>> ====================================================================== >>>>>>>>>> FAIL: runTest >>>>>>>>>> (pgadmin.tools.restore.tests.test_restore_create_job_unit_test.RestoreCreateJobTest) >>>>>>>>>> When restore object with option - Miscellaneous >>>>>>>>>> ---------------------------------------------------------------------- >>>>>>>>>> Traceback (most recent call last): >>>>>>>>>> File >>>>>>>>>> "/Users/pivotal/.pyenv/versions/3.6.5/lib/python3.6/unittest/mock.py", >>>>>>>>>> line 1179, in patched >>>>>>>>>> return func(*args, **keywargs) >>>>>>>>>> File >>>>>>>>>> "/Users/pivotal/workspace/pgadmin4/web/pgadmin/tools/restore/tests/test_restore_create_job_unit_test.py", >>>>>>>>>> line 295, in runTest >>>>>>>>>> self.assertEquals(response.status_code, 200) >>>>>>>>>> AssertionError: 410 != 200 >>>>>>>>>> >>>>>>>>>> And >>>>>>>>>> >>>>>>>>>> When restore object with the sections options ... 2018-05-31 >>>>>>>>>> 12:24:42,988: ERROR pgadmin: illegal environment variable name >>>>>>>>>> Traceback (most recent call last): >>>>>>>>>> File >>>>>>>>>> "/Users/pivotal/workspace/pgadmin4/web/pgadmin/tools/restore/__init__.py", >>>>>>>>>> line 352, in create_restore_job >>>>>>>>>> manager.export_password_env(p.id) >>>>>>>>>> File >>>>>>>>>> "/Users/pivotal/workspace/pgadmin4/web/pgadmin/utils/driver/psycopg2/server_manager.py", >>>>>>>>>> line 365, in export_password_env >>>>>>>>>> os.environ[str(env)] = password >>>>>>>>>> File "/Users/pivotal/.pyenv/versions/3.6.5/lib/python3.6/os.py", >>>>>>>>>> line 675, in __setitem__ >>>>>>>>>> self.putenv(key, value) >>>>>>>>>> ValueError: illegal environment variable name >>>>>>>>>> FAIL >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> Sincerely, >>>>>>>>>> >>>>>>>>>> Victoria && Anthony >>>>>>>>>> >>>>>>>>>> On Thu, May 31, 2018 at 1:16 AM Khushboo Vashi < >>>>>>>>>> khushboo.va...@enterprisedb.com> wrote: >>>>>>>>>> >>>>>>>>>>> Hi, >>>>>>>>>>> >>>>>>>>>>> Please find the attached updated patch with the fixes. >>>>>>>>>>> The test cases were only failing on MAC not on Linux. >>>>>>>>>>> >>>>>>>>>>> Thanks, >>>>>>>>>>> Khushboo >>>>>>>>>>> >>>>>>>>>>> On Wed, May 30, 2018 at 10:13 AM, Khushboo Vashi < >>>>>>>>>>> khushboo.va...@enterprisedb.com> wrote: >>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> On Wed, May 30, 2018 at 1:05 AM, Dave Page <dp...@pgadmin.org> >>>>>>>>>>>> wrote: >>>>>>>>>>>> >>>>>>>>>>>>> Hi >>>>>>>>>>>>> >>>>>>>>>>>>> On Mon, May 28, 2018 at 8:09 AM, Khushboo Vashi < >>>>>>>>>>>>> khushboo.va...@enterprisedb.com> wrote: >>>>>>>>>>>>> >>>>>>>>>>>>>> Hi, >>>>>>>>>>>>>> >>>>>>>>>>>>>> please find the attached updated patch for the test cases of >>>>>>>>>>>>>> Backup, Restore and Maintenance modules which includes: >>>>>>>>>>>>>> >>>>>>>>>>>>>> 1. Unit test cases >>>>>>>>>>>>>> 2. End to end regression test cases >>>>>>>>>>>>>> 3. Feature test cases >>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> Thanks. I've yet to be able to run the feature tests >>>>>>>>>>>>> successfully. Here's what I've found so far: >>>>>>>>>>>>> >>>>>>>>>>>>> 1) DEFAULT_BINARY_PATHS should be default_binary_paths in the >>>>>>>>>>>>> JSON config file. >>>>>>>>>>>>> >>>>>>>>>>>>> Will do. >>>>>>>>>>>> >>>>>>>>>>>>> 2) I've hit screensize related issues: >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> ====================================================================== >>>>>>>>>>>>> >>>>>>>>>>>>> ERROR: runTest >>>>>>>>>>>>> (pgadmin.feature_tests.pg_utilities_maintenance_test.PGUtilitiesMaintenanceFeatureTest) >>>>>>>>>>>>> >>>>>>>>>>>>> Test for PG maintenance: database >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> ---------------------------------------------------------------------- >>>>>>>>>>>>> >>>>>>>>>>>>> Traceback (most recent call last): >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/pg_utilities_maintenance_test.py", >>>>>>>>>>>>> line 56, in runTest >>>>>>>>>>>>> >>>>>>>>>>>>> self._open_maintenance_dialogue() >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/pg_utilities_maintenance_test.py", >>>>>>>>>>>>> line 75, in _open_maintenance_dialogue >>>>>>>>>>>>> >>>>>>>>>>>>> "*[.='" + self.table_name + "']/../*[@class='aciTreeItem'" >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/selenium/webdriver/remote/webelement.py", >>>>>>>>>>>>> line 80, in click >>>>>>>>>>>>> >>>>>>>>>>>>> self._execute(Command.CLICK_ELEMENT) >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/selenium/webdriver/remote/webelement.py", >>>>>>>>>>>>> line 628, in _execute >>>>>>>>>>>>> >>>>>>>>>>>>> return self._parent.execute(command, params) >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/selenium/webdriver/remote/webdriver.py", >>>>>>>>>>>>> line 312, in execute >>>>>>>>>>>>> >>>>>>>>>>>>> self.error_handler.check_response(response) >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/selenium/webdriver/remote/errorhandler.py", >>>>>>>>>>>>> line 242, in check_response >>>>>>>>>>>>> >>>>>>>>>>>>> raise exception_class(message, screen, stacktrace) >>>>>>>>>>>>> >>>>>>>>>>>>> WebDriverException: Message: unknown error: Element <span >>>>>>>>>>>>> class="aciTreeItem">...</span> is not clickable at point (223, >>>>>>>>>>>>> 604). Other >>>>>>>>>>>>> element would receive the click: <div class="wcFrameCenter >>>>>>>>>>>>> wcPanelBackground wcScrollableX wcScrollableY" style="left: 0px; >>>>>>>>>>>>> right: >>>>>>>>>>>>> 0px; bottom: 0px;">...</div> >>>>>>>>>>>>> >>>>>>>>>>>>> (Session info: chrome=66.0.3359.181) >>>>>>>>>>>>> >>>>>>>>>>>>> (Driver info: chromedriver=2.38.552518 >>>>>>>>>>>>> (183d19265345f54ce39cbb94cf81ba5f15905011),platform=Mac OS X >>>>>>>>>>>>> 10.12.6 x86_64) >>>>>>>>>>>>> >>>>>>>>>>>>> 3) One time the test did start, but then I saw this failure: >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> ====================================================================== >>>>>>>>>>>>> >>>>>>>>>>>>> ERROR: runTest >>>>>>>>>>>>> (pgadmin.feature_tests.pg_utilities_backup_restore_test.PGUtilitiesBackupFeatureTest) >>>>>>>>>>>>> >>>>>>>>>>>>> Test for PG utilities - Backup and Restore >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> ---------------------------------------------------------------------- >>>>>>>>>>>>> >>>>>>>>>>>>> Traceback (most recent call last): >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/pg_utilities_backup_restore_test.py", >>>>>>>>>>>>> line 93, in runTest >>>>>>>>>>>>> >>>>>>>>>>>>> self.page.fill_input_by_field_name("file", >>>>>>>>>>>>> "test_backup_file") >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/git/pgadmin4/web/regression/feature_utils/pgadmin_page.py", >>>>>>>>>>>>> line 211, in fill_input_by_field_name >>>>>>>>>>>>> >>>>>>>>>>>>> self.wait_for_input_field_content(field_name, >>>>>>>>>>>>> field_content) >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/git/pgadmin4/web/regression/feature_utils/pgadmin_page.py", >>>>>>>>>>>>> line 251, in wait_for_input_field_content >>>>>>>>>>>>> >>>>>>>>>>>>> "field to contain '" + str(content) + "'", >>>>>>>>>>>>> input_field_has_content >>>>>>>>>>>>> >>>>>>>>>>>>> File >>>>>>>>>>>>> "/Users/dpage/git/pgadmin4/web/regression/feature_utils/pgadmin_page.py", >>>>>>>>>>>>> line 337, in _wait_for >>>>>>>>>>>>> >>>>>>>>>>>>> "Timed out waiting for " + waiting_for_message >>>>>>>>>>>>> >>>>>>>>>>>>> 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: Timed out waiting for field to >>>>>>>>>>>>> contain 'test_backup_file' >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> (with screenshot attached) >>>>>>>>>>>>> >>>>>>>>>>>>> Thanks. >>>>>>>>>>>>> >>>>>>>>>>>>> I have ran the feature tests with multiple servers many times >>>>>>>>>>>> but didn't get a single failure. >>>>>>>>>>>> I have asked Akshay to run on his machine, let see what he gets. >>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> Thanks, >>>>>>>>>>>>>> Khushboo >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> On Wed, Apr 25, 2018 at 9:40 PM, Joao De Almeida Pereira < >>>>>>>>>>>>>> jdealmeidapere...@pivotal.io> wrote: >>>>>>>>>>>>>> >>>>>>>>>>>>>>> Hi Khushboo, >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> We reviewed the patch and it is very nice to see some more >>>>>>>>>>>>>>> coverage in this area. Good job :D >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> We passed the tests through our CI the feature tests are not >>>>>>>>>>>>>>> passing, but the linter fails: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> ./pgadmin/feature_tests/pg_utilities_backup_test.py:37: [E501] >>>>>>>>>>>>>>> line too long (91 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:265> >>>>>>>>>>>>>>> ./pgadmin/feature_tests/pg_utilities_backup_test.py:53: [E501] >>>>>>>>>>>>>>> line too long (104 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:266> >>>>>>>>>>>>>>> ./pgadmin/feature_tests/pg_utilities_backup_test.py:59: [E501] >>>>>>>>>>>>>>> line too long (85 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:267> >>>>>>>>>>>>>>> ./pgadmin/feature_tests/pg_utilities_backup_test.py:62: [E501] >>>>>>>>>>>>>>> line too long (96 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:268> >>>>>>>>>>>>>>> ./pgadmin/feature_tests/pg_utilities_backup_test.py:63: [E501] >>>>>>>>>>>>>>> line too long (91 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:269> >>>>>>>>>>>>>>> ./pgadmin/feature_tests/pg_utilities_backup_test.py:70: [E501] >>>>>>>>>>>>>>> line too long (118 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:270> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:37: [E121] >>>>>>>>>>>>>>> continuation line under-indented for hanging indent >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:271> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:48: [E122] >>>>>>>>>>>>>>> continuation line missing indentation or outdented >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:272> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:49: [E251] >>>>>>>>>>>>>>> unexpected spaces around keyword / parameter equals >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:273> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:49: [E251] >>>>>>>>>>>>>>> unexpected spaces around keyword / parameter equals >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:274> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:51: [E501] >>>>>>>>>>>>>>> line too long (91 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:275> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:52: [E501] >>>>>>>>>>>>>>> line too long (94 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:276> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:53: [E501] >>>>>>>>>>>>>>> line too long (108 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:277> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:81: [E501] >>>>>>>>>>>>>>> line too long (113 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:278> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:82: [E501] >>>>>>>>>>>>>>> line too long (94 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:279> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:83: [E501] >>>>>>>>>>>>>>> line too long (108 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:280> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:111: [E501] >>>>>>>>>>>>>>> line too long (100 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:281> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:113: [E501] >>>>>>>>>>>>>>> line too long (94 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:282> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:114: [E501] >>>>>>>>>>>>>>> line too long (108 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:283> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_backup_message.py:147: [E501] >>>>>>>>>>>>>>> line too long (93 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:284> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:40: [E121] >>>>>>>>>>>>>>> continuation line under-indented for hanging indent >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:285> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:51: [E122] >>>>>>>>>>>>>>> continuation line missing indentation or outdented >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:286> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:135: [E501] >>>>>>>>>>>>>>> line too long (80 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:287> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:137: [E501] >>>>>>>>>>>>>>> line too long (83 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:288> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:138: [E122] >>>>>>>>>>>>>>> continuation line missing indentation or outdented >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:289> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:139: [E122] >>>>>>>>>>>>>>> continuation line missing indentation or outdented >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:290> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:140: [E122] >>>>>>>>>>>>>>> continuation line missing indentation or outdented >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:291> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:191: [E501] >>>>>>>>>>>>>>> line too long (81 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:292> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:203: [E501] >>>>>>>>>>>>>>> line too long (80 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:293> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:204: [E128] >>>>>>>>>>>>>>> continuation line under-indented for visual indent >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:294> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:204: [E501] >>>>>>>>>>>>>>> line too long (94 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:295> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:205: [E128] >>>>>>>>>>>>>>> continuation line under-indented for visual indent >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:296> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:205: [E501] >>>>>>>>>>>>>>> line too long (94 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:297> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_batch_process.py:216: [W391] >>>>>>>>>>>>>>> blank line at end of file >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:298> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_create_backup_job.py:296: >>>>>>>>>>>>>>> [E501] line too long (97 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:299> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_create_backup_job.py:317: >>>>>>>>>>>>>>> [E303] too many blank lines (2) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:300> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_create_backup_job.py:336: >>>>>>>>>>>>>>> [E501] line too long (84 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:301> >>>>>>>>>>>>>>> ./pgadmin/tools/backup/tests/test_create_backup_job.py:371: >>>>>>>>>>>>>>> [W391] blank line at end of file >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:302> >>>>>>>>>>>>>>> 2 E121 continuation line under-indented for hanging indent >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:303> >>>>>>>>>>>>>>> 5 E122 continuation line missing indentation or outdented >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:304> >>>>>>>>>>>>>>> 2 E128 continuation line under-indented for visual indent >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:305> >>>>>>>>>>>>>>> 2 E251 unexpected spaces around keyword / parameter equals >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:306> >>>>>>>>>>>>>>> 1 E303 too many blank lines (2) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:307> >>>>>>>>>>>>>>> 24 E501 line too long (91 > 79 characters) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:308> >>>>>>>>>>>>>>> 2 W391 blank line at end of file >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> <https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/pipelines/pgadmin-patch/jobs/run-linter/builds/17#L5ad0f3d8:309> >>>>>>>>>>>>>>> 38 >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> For the feature tests, we realized we had to update the >>>>>>>>>>>>>>> configuration, and we did that, but we get the following error >>>>>>>>>>>>>>> attached. We >>>>>>>>>>>>>>> spent some time trying to understand the problem but we were >>>>>>>>>>>>>>> not successful. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Codewise: >>>>>>>>>>>>>>> - We just found some One Letter Variables in the code... >>>>>>>>>>>>>>> - Looks like there is a bug report in this area of the code >>>>>>>>>>>>>>> and we do not have coverage for it: >>>>>>>>>>>>>>> https://redmine.postgresql.org/issues/3232 >>>>>>>>>>>>>>> Looks like in some of the unit tests we only have happy >>>>>>>>>>>>>>> path tests, maybe we should see if there are any sad paths that >>>>>>>>>>>>>>> also need >>>>>>>>>>>>>>> coverage. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> The configuration change, maybe need to be updated. When we >>>>>>>>>>>>>>> install multiple versions of postgres the binaries live in >>>>>>>>>>>>>>> `/usr/lib/postgresql/{{db_version}}/bin`, which makes us think >>>>>>>>>>>>>>> that this >>>>>>>>>>>>>>> configuration should live near the server configuration, maybe? >>>>>>>>>>>>>>> Also to >>>>>>>>>>>>>>> maintain coherency on the naming maybe we should make it all >>>>>>>>>>>>>>> lower case. >>>>>>>>>>>>>>> Just as an aside, you can add the gpdb configuration as well >>>>>>>>>>>>>>> in you patch. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Thanks >>>>>>>>>>>>>>> Victoria & Joao >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> On Wed, Apr 25, 2018 at 5:20 AM Khushboo Vashi < >>>>>>>>>>>>>>> khushboo.va...@enterprisedb.com> wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Hi, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Please find the attached patch which covers test cases for >>>>>>>>>>>>>>>> the backup module (RM #3206). >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> 1. Unit test cases >>>>>>>>>>>>>>>> 2. End to end regression test cases >>>>>>>>>>>>>>>> 3. Feature test cases >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Thanks, >>>>>>>>>>>>>>>> Khushboo >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>