Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[java] Fix #5746: Move test data classes to src/test/java-resources#5755

Draft
oowekyala wants to merge 3 commits intopmd:mainfrom
oowekyala:java-test-resources
Draft

[java] Fix #5746: Move test data classes to src/test/java-resources#5755
oowekyala wants to merge 3 commits intopmd:mainfrom
oowekyala:java-test-resources

Conversation

@oowekyala
Copy link
Member

@oowekyala oowekyala commented May 19, 2025

Describe the PR

Related issues

Ready?

  • Added unit tests for fixed bug/feature
  • Passing all unit tests
  • Complete build ./mvnw clean verify passes (checked automatically by github actions)
  • Added (in-code) documentation (if needed)

@github-actions
Copy link

Documentation Preview

Compared to main:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.

Regression Tester Report

(comment created at 2025-05-21 10:40:43+00:00 for 01e9883)

@adangel
Copy link
Member

adangel commented Jun 9, 2025

  • Need to update checkstyle suppressions in build tools

The suppressions in build-tools only ignore "/generated-sources/", but we have another checkstyle suppression config in pmd-java:
https://github.com/pmd/pmd/blob/main/pmd-java/pmd-java-checkstyle-suppressions.xml

@adangel adangel changed the title [java] Move test data classes to src/test/java-resources Fix #5746: [java] Move test data classes to src/test/java-resources Jun 9, 2025
@adangel adangel changed the title Fix #5746: [java] Move test data classes to src/test/java-resources [java] Fix #5746: Move test data classes to src/test/java-resources Sep 8, 2025
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.

[java] Separate test sources and resources

2 participants