diff options
author | Duarte Meneses <duarte.meneses@sonarsource.com> | 2019-01-10 15:54:51 +0100 |
---|---|---|
committer | SonarTech <sonartech@sonarsource.com> | 2019-01-10 20:21:03 +0100 |
commit | 355a3b65066e968e3669a97892df81ef15770a55 (patch) | |
tree | dd54d1d2adc31321e31cd44da83d7e3ec2d3626b /sonar-plugin-api/src/test/java/org/sonar/api/utils/PathUtilsTest.java | |
parent | a71d799bc56dfa834aaadf8a0b13b88496ac0585 (diff) | |
download | sonarqube-355a3b65066e968e3669a97892df81ef15770a55.tar.gz sonarqube-355a3b65066e968e3669a97892df81ef15770a55.zip |
SONAR-11225 NPE when using RelativePathPredicate with a path starting by ..
Diffstat (limited to 'sonar-plugin-api/src/test/java/org/sonar/api/utils/PathUtilsTest.java')
-rw-r--r-- | sonar-plugin-api/src/test/java/org/sonar/api/utils/PathUtilsTest.java | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/utils/PathUtilsTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/utils/PathUtilsTest.java index fc0f7a0cb1c..80371dc5aa2 100644 --- a/sonar-plugin-api/src/test/java/org/sonar/api/utils/PathUtilsTest.java +++ b/sonar-plugin-api/src/test/java/org/sonar/api/utils/PathUtilsTest.java @@ -42,14 +42,19 @@ public class PathUtilsTest { public TemporaryFolder temp = new TemporaryFolder(); @Test - public void testSanitize() throws Exception { + public void sanitize_succeeds() { assertThat(PathUtils.sanitize("foo/bar/..")).isEqualTo("foo/"); assertThat(PathUtils.sanitize("C:\\foo\\..\\bar")).isEqualTo("C:/bar"); assertThat(PathUtils.sanitize(null)).isNull(); } @Test - public void test_canonicalPath() throws Exception { + public void sanitize_invalid_paths_returns_null() { + assertThat(PathUtils.sanitize("../foo")).isNull(); + } + + @Test + public void canonicalPath_succeeds() throws Exception { File file = temp.newFile(); String path = PathUtils.canonicalPath(file); assertThat(path).isEqualTo(FilenameUtils.separatorsToUnix(file.getCanonicalPath())); |