Skip to content

Conversation

@J-manLans
Copy link

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • All new code is formatted with clang-format -i --style=file path/to/your/file.java

This PR is part of a test course project and includes the following improvements:

  • Expanded test coverage for the most under-tested packages, improving overall code reliability.
  • Optional Maven dev profile added to allow developers to easily switch between classes and generate reports for PIT mutation testing.
  • ConsoleInterceptor implementation to simplify testing of classes that heavily rely on console input/output. (not thread-safe)
  • Minor refactors to improve testability without changing core logic

Notes:

  • Focus on ConsoleInterceptor usage and test coverage improvements.
  • For a thread-safe version of the interceptor, it might be beneficial to consider enforcing scanners and console output via constructor injection to allow mocking of streams.

J-manLans and others added 30 commits October 29, 2025 12:29
Line coverage finished but need to
check out mutation coverage a little
more
Might check for RL and LR rotations
as well
Pleasant class to test for once :P
Have solved spotbugs issues and
minimized checkstyle errors
J-manLans and others added 25 commits October 29, 2025 12:29
Made corrections according
to checkstyle and spotbugs
in all our files.

Clang is left though...
Autocorrected import order
according to Clang's wishes
According to Clang's fugly ass
suggestions.
According to Clang's fugly ass
suggestions.
According to Clang's fugly ass
suggestions.
According to Clang's fugly ass
suggestions.
According to Clang's fugly ass
suggestions.
According to PMD...
According to Clang again...
According to Clang again...
According to Clang again...
According to Clang again...
According to Clang again...
Added error messages when needed
and display names to all methods.
@codecov-commenter
Copy link

codecov-commenter commented Oct 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.39%. Comparing base (dfd8d69) to head (c047bee).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #7018      +/-   ##
============================================
+ Coverage     78.32%   80.39%   +2.06%     
- Complexity     6615     6749     +134     
============================================
  Files           746      746              
  Lines         22022    22025       +3     
  Branches       4321     4321              
============================================
+ Hits          17249    17706     +457     
+ Misses         4085     3648     -437     
+ Partials        688      671      -17     

☔ 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.

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.

4 participants