Skip to content

fix: resolve Java test paths with .java extension in PR results#1674

Merged
HeshamHM28 merged 1 commit intoomni-javafrom
fix/java/pr-test-path-resolution
Feb 26, 2026
Merged

fix: resolve Java test paths with .java extension in PR results#1674
HeshamHM28 merged 1 commit intoomni-javafrom
fix/java/pr-test-path-resolution

Conversation

@HeshamHM28
Copy link
Copy Markdown
Contributor

@HeshamHM28 HeshamHM28 commented Feb 26, 2026

Java test module paths were falling through to the Python fallback in existing_tests_source_for, resolving as .py instead of .java. This caused all existing test runtime comparisons to be skipped (matched=0, skipped=290).
Added a elif is_java() branch to correctly match Java class names against registered test files.

@HeshamHM28 HeshamHM28 marked this pull request as ready for review February 26, 2026 04:40
@HeshamHM28 HeshamHM28 merged commit 04b0dbd into omni-java Feb 26, 2026
24 of 32 checks passed
@HeshamHM28 HeshamHM28 deleted the fix/java/pr-test-path-resolution branch February 26, 2026 04:56
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