Skip to content

fix: include configured test class name patterns#294

Merged
rcasia merged 1 commit into
rcasia:mainfrom
lucas-garcia-rubio:main
May 20, 2026
Merged

fix: include configured test class name patterns#294
rcasia merged 1 commit into
rcasia:mainfrom
lucas-garcia-rubio:main

Conversation

@lucas-garcia-rubio

@lucas-garcia-rubio lucas-garcia-rubio commented May 16, 2026

Copy link
Copy Markdown
Contributor

This fixes two problems:

  1. Fix a problem where integration tests classes sufixed as *IT.java had its tests parsed correctly by treesitter, but not found by junit console since Junit Classname default is ^(Test.*|.+[.$]Test.*|.*Tests?)$ as you can see here: https://docs.junit.org/6.0.3/running-tests/console-launcher.html#options-discovering-tests
  2. Fix a bug during result parse containing '>' sign.

All suggestions are welcome.

@lucasliet

Copy link
Copy Markdown

LGTM

@lucas-garcia-rubio lucas-garcia-rubio changed the title Run integration tests sufixed as *IT.java and handle its results fix: include configured test class name patterns May 20, 2026
@rcasia

rcasia commented May 20, 2026

Copy link
Copy Markdown
Owner

Fix a bug during result parse containing '>' sign.

Moved the related work to #296

@rcasia rcasia merged commit 64afb8b into rcasia:main May 20, 2026
15 checks passed
@rcasia

rcasia commented May 20, 2026

Copy link
Copy Markdown
Owner

Thanks @lucas-garcia-rubio !

rcasia added a commit that referenced this pull request May 23, 2026
part of the work originally on #294

Co-authored-by: Lucas Garcia Rubio <lucasgarciarubio@gmail.com>
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.

3 participants