joerghoh commented on code in PR #228:
URL:
https://github.com/apache/sling-org-apache-sling-resourceresolver/pull/228#discussion_r3765884397
##########
src/test/java/org/apache/sling/resourceresolver/impl/mapping/VanityPathMapEntriesTest.java:
##########
@@ -267,6 +267,66 @@ public void test_simple_vanity_path() {
assertNotNull(vanityMap.get("/" + containerName + "/" + oneMore));
}
+ // see https://issues.apache.org/jira/browse/SLING-13301
+ @Test
+ public void test_syntax_variants() {
+ // absolute path
+ internal_test_syntax_variants("/x", "/x");
+ // relative path
+ internal_test_syntax_variants("x", "/x");
+ // multi-segment absolute path
+ internal_test_syntax_variants("/x/y", "/x/y");
+ // multi-segment relative path
+ internal_test_syntax_variants("x/y", "/x/y");
+ // multi-segment absolute path starting with "//"
+ internal_test_syntax_variants("//x//y", "//x//y");
+ // multi-segment absolute path starting with "/.."
+ internal_test_syntax_variants("/../x", "/../x");
+ // blank path
+ internal_test_syntax_variants(" ", null);
+ // empty path
+ internal_test_syntax_variants("", null);
+ }
+
+ private void internal_test_syntax_variants(String vanityPath, String
expectation) {
Review Comment:
```suggestion
private void assertPathResolvesTo(String vanityPath, String
expectedPath) {
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
