Skip to content

Conversation

@lukaszlenart
Copy link
Member

@lukaszlenart lukaszlenart commented Jan 11, 2026

Summary

  • Move xwork-default.xml from core/src/main/resources to core/src/test/resources and rename to struts-tests-default.xml
  • Copy the file to plugins/spring/src/test/resources for Spring plugin tests
  • Update all test file references to use the new filename
  • Update Javadoc examples to use modern Struts terminology (<xwork><struts>, xwork-defaultstruts-default)

Closes WW-5603

Test plan

  • Run affected core module tests (AnnotationWorkflowInterceptorTest, ChainingInterceptorWithConfigTest, ResolverUtilTest, DoubleRangeFieldValidatorTest, ValidatorAnnotationTest) - all pass
  • Run affected Spring plugin test (ActionAutowiringInterceptorTest) - passes
  • Verify the Struts IDEA plugin no longer shows xwork-default.xml as a framework config file

🤖 Generated with Claude Code

The Struts IDEA plugin incorrectly displayed xwork-default.xml as a
framework configuration file. This was misleading since the file is
only used in testing and is not loaded by the framework by default.

Changes:
- Move xwork-default.xml from core/src/main/resources to
  core/src/test/resources and rename to struts-tests-default.xml
- Copy struts-tests-default.xml to plugins/spring/src/test/resources
- Update all test file references to use the new filename
- Update Javadoc examples to use modern Struts terminology
  (xwork -> struts, xwork-default -> struts-default)

Closes [WW-5603](https://issues.apache.org/jira/browse/WW-5603)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@lukaszlenart lukaszlenart force-pushed the fix/WW-5603-move-xwork-default-to-test-resources branch from ea9d221 to 65b7efb Compare January 11, 2026 09:49
@sonarqubecloud
Copy link

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