Skip to content

Comments

Improved: FlexibleStringExpander tests rewrite (OFBIZ-13336)#939

Merged
gtchaboussie merged 1 commit intoapache:trunkfrom
gtchaboussie:OFBIZ-13336
Jan 30, 2026
Merged

Improved: FlexibleStringExpander tests rewrite (OFBIZ-13336)#939
gtchaboussie merged 1 commit intoapache:trunkfrom
gtchaboussie:OFBIZ-13336

Conversation

@gtchaboussie
Copy link
Contributor

  • Rewrote the FlexibleStringExpander in groovy.
  • Simplified the reading by splitting the tests into two distinct classes : Parser, and actual FSE test.
  • Actual tests are now grouped by theme.
  • Small tool classes that were used are now dedicated classes.
  • Also removed the explicit groovy test sources, no longer required, with the added useJunit()
    config to allow this behavior

- Rewrote the FlexibleStringExpander in groovy.
- Simplified the reading by splitting the tests into two distinct classes : Parser, and actual FSE test.
- Actual tests are now grouped by theme.
- Small tool classes that were used are now dedicated classes.
- Also removed the explicit groovy test sources, no longer required, and added the useJunit()
setting to allow this behavior
@sonarqubecloud
Copy link

@gtchaboussie gtchaboussie merged commit e3d73ac into apache:trunk Jan 30, 2026
7 checks passed
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