Skip to content

Add tests for Java 25 parser fallback and CycleRanker graph generation - #219

Merged
jimbethancourt merged 1 commit into
#190-add-java25-supportfrom
coderabbit/add-pr-218-tests/696753a2
Sep 23, 2026
Merged

jimbethancourt merged 1 commit into
#190-add-java25-supportfrom
coderabbit/add-pr-218-tests/696753a2

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Add PR #218 regression tests covering Java 25 parser unavailability and graceful fallback, null or empty CycleRanker source paths, and custom test-directory inclusion/exclusion. Verify rejected requests leave the graph DTO unset and successful generation retains it for cycle ranking.

Validation: Not run. The parser fallback test requires JDK 25+.

View coding task


Devin Review

@coderabbitai

coderabbitai Bot commented Sep 23, 2026

Copy link
Copy Markdown
Contributor Author

Important

Review skipped

This PR was authored by the user configured for CodeRabbit reviews. CodeRabbit does not review PRs authored by this user. It's recommended to use a dedicated user account to post CodeRabbit review feedback.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 2d357e70-f907-491d-8880-567ca16332c6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@jimbethancourt
jimbethancourt merged commit 9bcdbe5 into #190-add-java25-support Sep 23, 2026
3 checks passed
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.

1 participant