Skip to content

Code coverage increase, and manual code trigger - src/flags.js#87

Open
mrahmed2026 wants to merge 10 commits into
CMU-17313Q:f24from
mrahmed2026:flag_js_coverage_increase
Open

Code coverage increase, and manual code trigger - src/flags.js#87
mrahmed2026 wants to merge 10 commits into
CMU-17313Q:f24from
mrahmed2026:flag_js_coverage_increase

Conversation

@mrahmed2026
Copy link
Copy Markdown

@mrahmed2026 mrahmed2026 commented Sep 4, 2024

I was able to increase the overall code coverage of this file.

83.26% Statements 408/490 67.34% Branches 165/245 82.69% Functions 86/104 83.87% Lines 390/465

85.3% Statements 418/490 71.02% Branches 174/245 84.61% Functions 88/104 86.02% Lines 400/465

This was indicated in the npm run test locally, and on the code coverage tool, and shown to Professor Eduardo as well.

Further, I addded a console log to the flag.js file and it was shown after i triggered the code, by flagging a blog post or annoucement.

@coveralls
Copy link
Copy Markdown

coveralls commented Sep 4, 2024

Pull Request Test Coverage Report for Build 10719743949

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.04%) to 82.715%

Files with Coverage Reduction New Missed Lines %
src/meta/errors.js 1 76.74%
Totals Coverage Status
Change from base Build 10550029021: 0.04%
Covered Lines: 22333
Relevant Lines: 25583

💛 - Coveralls

@mrahmed2026 mrahmed2026 changed the title Flag js coverage increase Code coverage increase, and manual code trigger - src/flags.js Sep 5, 2024
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 5, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants