Skip to content

Add tests for applyRelativePath method in StringUtils #35397

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

Add tests for applyRelativePath method in StringUtils #35397

wants to merge 3 commits into from
+31 −12

Conversation

J-unStiN
Copy link
Contributor

Documenting Test Scenarios:

  • Basic Joins
  • Relative paths start with a slash
  • Include the root path
  • No slashes in the base path
  • Preserve parent directory tokens (..)
  • Base paths end with a slash
  • Empty relative path
    Scope of Impact: Changes only to test code. No impact to production code.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Aug 31, 2025
@J-unStiN J-unStiN force-pushed the test/apply-relative-path branch from d8d414f to 87e161a Compare August 31, 2025 16:29
@bclozel bclozel closed this in 1653ec3 Sep 1, 2025
@bclozel bclozel self-assigned this Sep 1, 2025
@bclozel bclozel added type: task A general task in: core Issues in core modules (aop, beans, core, context, expression) and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Sep 1, 2025
@bclozel bclozel added this to the 7.0.0-M9 milestone Sep 1, 2025
@bclozel
Copy link
Member

bclozel commented Sep 1, 2025

Thanks for your contribution @J-unStiN !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@bclozel bclozel

Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: task A general task
Projects
None yet
Milestone
7.0.0-M9
Development

Successfully merging this pull request may close these issues.
5 participants
@J-unStiN @bclozel @spring-projects-issues @ngocnhan-tran1996 @sbrannen