Sure Aditya, appreciates your feedback.
I will be doing a cross check with below requirements & get back with updates.
Thanks !
Pramod Ahire
Software Engineer
C: +91-020-66449600/601
D: +91-9028697679
edbpostgres.com
From: Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>Date: Thursday, 6 August 2020 at 6:04 PMTo: Pramod Ahire <pramod.ahire@enterprisedb.com>Cc: pgadmin-hackers <pgadmin-hackers@postgresql.org>, Akshay Joshi <akshay.joshi@enterprisedb.com>Subject: Re: Code coverage report generation for JS files
Hi Pramod,
The patch looks good to me except:
1) Remove commented code.
2) Make coverage report optional using command line flag --reporters. Add a new cmd in yarn - test:karma-coverage.
3) Add the new coverage directory to .gitignore.
On Thu, Aug 6, 2020 at 4:59 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Aditya Please review it. On Thu, Aug 6, 2020 at 12:12 PM Pramod Ahire <pramod.ahire@enterprisedb.com> wrote:Hi, Please find updated patch for below changes: Updated path for test files to module/test dir (As suggested by @Ashesh & @Nil)Added exclusion expression for test cases Added Lcov report generation along with html for sonarqube report combination Please have a review & do let me know if any changes/suggestions. Pramod AhireSoftware Engineer C: +91-020-66449600/601D: +91-9028697679edbpostgres.com From: Pramod Ahire <pramod.ahire@enterprisedb.com>Date: Wednesday, 5 August 2020 at 4:45 PMSubject: Re: Code coverage report generation for JS files Missed Screenshot with previous email. Pramod AhireSoftware Engineer C: +91-020-66449600/601D: +91-9028697679edbpostgres.com From: Pramod Ahire <pramod.ahire@enterprisedb.com>Date: Wednesday, 5 August 2020 at 4:44 PMSubject: Code coverage report generation for JS files Hi Team, As recent updates have added ability to generate code coverage report for all the JS files.Where had added two new packages as below: istanbul-instrumenter-loaderkarma-coverage Steps to generate report:Apply attached patchDo - ‘npm install’Run – ‘yarn run test:karma’It has below type of output formats supported:HtmlLcovCoberturaLcovonlyTeamcityTextText-summeryAll the output files will be generated & stored in ‘root_dir/coverage’ folder .Please do let me know your thoughts or suggestion if any. Pramod AhireSoftware Engineer C: +91-020-66449600/601D: +91-9028697679edbpostgres.com -- Thanks & RegardsAkshay JoshipgAdmin Hacker | Sr. Software ArchitectEDB PostgresMobile: +91 976-788-8246
Hi Aditya
Please review it.
On Thu, Aug 6, 2020 at 12:12 PM Pramod Ahire <pramod.ahire@enterprisedb.com> wrote:
Hi, Please find updated patch for below changes: Updated path for test files to module/test dir (As suggested by @Ashesh & @Nil)Added exclusion expression for test cases Added Lcov report generation along with html for sonarqube report combination Please have a review & do let me know if any changes/suggestions. Pramod AhireSoftware Engineer C: +91-020-66449600/601D: +91-9028697679edbpostgres.com From: Pramod Ahire <pramod.ahire@enterprisedb.com>Date: Wednesday, 5 August 2020 at 4:45 PMSubject: Re: Code coverage report generation for JS files Missed Screenshot with previous email. Pramod AhireSoftware Engineer C: +91-020-66449600/601D: +91-9028697679edbpostgres.com From: Pramod Ahire <pramod.ahire@enterprisedb.com>Date: Wednesday, 5 August 2020 at 4:44 PMSubject: Code coverage report generation for JS files Hi Team, As recent updates have added ability to generate code coverage report for all the JS files.Where had added two new packages as below: istanbul-instrumenter-loaderkarma-coverage Steps to generate report:Apply attached patchDo - ‘npm install’Run – ‘yarn run test:karma’It has below type of output formats supported:HtmlLcovCoberturaLcovonlyTeamcityTextText-summeryAll the output files will be generated & stored in ‘root_dir/coverage’ folder .Please do let me know your thoughts or suggestion if any. Pramod AhireSoftware Engineer C: +91-020-66449600/601D: +91-9028697679edbpostgres.com
Hi,
Please find updated patch for below changes:
Please have a review & do let me know if any changes/suggestions.
From: Pramod Ahire <pramod.ahire@enterprisedb.com>Date: Wednesday, 5 August 2020 at 4:45 PMSubject: Re: Code coverage report generation for JS files
Missed Screenshot with previous email.
From: Pramod Ahire <pramod.ahire@enterprisedb.com>Date: Wednesday, 5 August 2020 at 4:44 PMSubject: Code coverage report generation for JS files
Hi Team,
As recent updates have added ability to generate code coverage report for all the JS files.
Where had added two new packages as below:
Steps to generate report:
It has below type of output formats supported:
All the output files will be generated & stored in ‘root_dir/coverage’ folder .
Please do let me know your thoughts or suggestion if any.
--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Sr. Software Architect
EDB Postgres
Mobile: +91 976-788-8246
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"
pgadmin-hackers by date:
Соглашаюсь с условиями обработки персональных данных