Re: [pgAdmin][RM5050] Language node code coverage.

2020-01-19 Thread Akshay Joshi
Thanks, patch applied. On Mon, Jan 20, 2020 at 11:43 AM Pradip Parkale < pradip.park...@enterprisedb.com> wrote: > Hi Akshay, > > Please find the updated patch. > I have accommodated all the changes mentioned in the review. > > On Thu, Jan 16, 2020 at 5:54 PM Akshay Joshi < > akshay.jo...@enterpr

Re: [pgAdmin][RM5050] Language node code coverage.

2020-01-19 Thread Pradip Parkale
Hi Akshay, Please find the updated patch. I have accommodated all the changes mentioned in the review. On Thu, Jan 16, 2020 at 5:54 PM Akshay Joshi wrote: > Hi Pradip > > Below are the review comments: > >- Format the JSON file. Rename the JSON file from >'language_testcases.json' to 'l

Re: [pgAdmin][RM5050] Language node code coverage.

2020-01-16 Thread Akshay Joshi
Hi Pradip Below are the review comments: - Format the JSON file. Rename the JSON file from 'language_testcases.json' to 'language_test_data.json'. - For consistency please follow the naming convention used in ' *cast_test_data.json*' file. For example, in Cast we have used 'name' a

[pgAdmin][RM5050] Language node code coverage.

2020-01-15 Thread Pradip Parkale
Hi All, Attached is a patch for language node code coverage. This increases the code coverage of the language node from 45% to 69%. Command to run the tests - - python runtests.py --pkg browser.server_groups.servers.databases.languages.tests -- Thanks & Regards, Pradip Parkale QMG, Ent