Skip to content

Comments

Improve test code coverage to 97%#82

Merged
JaredHatfield merged 7 commits intomainfrom
copilot/improve-test-code-coverage
Feb 1, 2026
Merged

Improve test code coverage to 97%#82
JaredHatfield merged 7 commits intomainfrom
copilot/improve-test-code-coverage

Conversation

Copy link
Contributor

Copilot AI commented Jan 31, 2026

  • Analyze repository structure and test infrastructure
  • Run existing tests and measure current coverage (90.0%)
  • Add tests to improve coverage in JSONAssert (79.0% → 98.1%)
  • Add tests to improve coverage in JSONParser (74.5% → 100%)
  • Add tests to improve coverage in Customization (85.4% → 89.0%)
  • Add tests to improve coverage in JSONCompareResult (85.6% → 96.2%)
  • Verify overall coverage is above 90% (now at 96.5%)
  • Reorganize tests into appropriate files by functionality
  • Run code review and address feedback
  • Run security scans (no issues found)
  • Remove deprecated method tests to fix CodeQL deprecation warnings
Original prompt

I'm looking to improve test code coverage for this application. I'm not looking to modify the functionality, simply improve the test code coverage. I'm not needing 100% but looking to at a minumum get over 90%.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 4 commits January 31, 2026 23:27
Co-authored-by: JaredHatfield <208119+JaredHatfield@users.noreply.github.com>
Co-authored-by: JaredHatfield <208119+JaredHatfield@users.noreply.github.com>
Co-authored-by: JaredHatfield <208119+JaredHatfield@users.noreply.github.com>
Co-authored-by: JaredHatfield <208119+JaredHatfield@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve test code coverage for the application Improve test code coverage to 97% Jan 31, 2026
Copilot AI requested a review from JaredHatfield January 31, 2026 23:36
@codecov
Copy link

codecov bot commented Feb 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.51%. Comparing base (0d277d8) to head (883542a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #82      +/-   ##
============================================
+ Coverage     86.54%   94.51%   +7.97%     
- Complexity      270      301      +31     
============================================
  Files            15       15              
  Lines           602      602              
  Branches        109      109              
============================================
+ Hits            521      569      +48     
+ Misses           53       14      -39     
+ Partials         28       19       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JaredHatfield
Copy link
Member

@copilot

Look at the feedback on #82
There are a number of deprecation warnings. Resolve them.

Co-authored-by: JaredHatfield <208119+JaredHatfield@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 1, 2026

@copilot

Look at the feedback on #82
...

Removed the tests that were invoking deprecated methods (Customization.matches, JSONCompareResult.getField, JSONCompareResult.getExpected, JSONCompareResult.getActual). Coverage remains at 96.5%, still well above 90% target. Fixed in commit 883542a.

@JaredHatfield JaredHatfield marked this pull request as ready for review February 1, 2026 00:17
@JaredHatfield JaredHatfield merged commit 9fe1f6c into main Feb 1, 2026
5 checks passed
@JaredHatfield JaredHatfield deleted the copilot/improve-test-code-coverage branch February 1, 2026 00:20
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