Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add test coverage improvements, Maven dev profile, and ConsoleInterceptor#7018

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
J-manLans wants to merge57 commits intoTheAlgorithms:master
base:master
Choose a base branch
Loading
fromJ-manLans:test-contributions-master

Conversation

@J-manLans
Copy link

  • I have readCONTRIBUTING.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 withclang-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-manLansand others added30 commitsOctober 29, 2025 12:29
Line coverage finished but need tocheck out mutation coverage a littlemore
Might check for RL and LR rotationsas well
Pleasant class to test for once :P
Have solved spotbugs issues andminimized checkstyle errors
J-manLansand others added25 commitsOctober 29, 2025 12:29
Made corrections accordingto checkstyle and spotbugsin all our files.Clang is left though...
Autocorrected import orderaccording to Clang's wishes
According to Clang's fugly asssuggestions.
According to Clang's fugly asssuggestions.
According to Clang's fugly asssuggestions.
According to Clang's fugly asssuggestions.
According to Clang's fugly asssuggestions.
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 neededand display names to all methods.
@codecov-commenter
Copy link

codecov-commenter commentedOct 31, 2025
edited
Loading

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 freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@DenizAltunkapanDenizAltunkapanAwaiting requested review from DenizAltunkapanDenizAltunkapan is a code owner

@yanglbmeyanglbmeAwaiting requested review from yanglbmeyanglbme is a code owner

@alxkmalxkmAwaiting requested review from alxkmalxkm is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@J-manLans@codecov-commenter@labe2301@hediinnT

[8]ページ先頭

©2009-2025 Movatter.jp