Skip to content

Improve Test Suite - #293

Open
rcosta358 wants to merge 8 commits into
mainfrom
improve-tests
Open

rcosta358 wants to merge 8 commits into
mainfrom
improve-tests

Conversation

@rcosta358

@rcosta358 rcosta358 commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Description

Closes #196, #197, #198, and #174.

  • Added inline expectations for both errors and warnings using // Expect: ...
  • Made so every reported diagnostic must have a matching expectation, and every expectation must be reported
  • Removed pass/fail decisions based on file or directory names
  • Simplified test discovery by running Java files individually and leaf directories as grouped tests

Related Issues

#196, #197, #198, and #174.

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • Code refactoring

Checklist

  • Added/updated tests under liquidjava-example/src/main/java/testSuite/
  • ./mvnw test passes locally
  • Updated docs/README if behavior or API changed

@rcosta358 rcosta358 self-assigned this Sep 12, 2026
@rcosta358 rcosta358 added the tests Related to tests label Sep 12, 2026
@rcosta358 rcosta358 linked an issue Sep 12, 2026 that may be closed by this pull request
@rcosta358 rcosta358 changed the title Improve Tests Improve Test Suite Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve Expected Pattern in Tests Diagnostic Based Test Results Inline Expected Warnings in Tests Improve Test Suite

1 participant