iSharkFly-Docs/java-tutorials
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #12373 from anuragkumawat/JAVA-12359

Browse Source 
JAVA-12359 Potential issue in Validate String as Filename in Java article
This commit is contained in:
kwoyke 2022-06-23 21:26:38 +02:00 committed by GitHub
commit 69d21288b9
2 changed files with 35 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
core-java-modules/core-java-string-operations-3/src/main/java/com/baeldung/stringfilenamevalidaiton/StringFilenameValidationUtils.java
View File

@ -7,7 +7,7 @@ import java.util.Arrays;
public class StringFilenameValidationUtils {
public static final Character[] INVALID_WINDOWS_SPECIFIC_CHARS = {'"', '*', ':', '<', '>', '?', '\\', '|', 0x7F};
public static final Character[] INVALID_WINDOWS_SPECIFIC_CHARS = {'"', '*', '<', '>', '?', '|'};
public static final Character[] INVALID_UNIX_SPECIFIC_CHARS = {'\000'};
public static final String REGEX_PATTERN = "^[A-za-z0-9.]{1,255}$";

42
core-java-modules/core-java-string-operations-3/src/test/java/com/baeldung/stringfilenamevalidaiton/StringFilenameValidationUnitTest.java
View File

@ -2,6 +2,7 @@ package com.baeldung.stringfilenamevalidaiton;
import org.apache.commons.lang3.RandomStringUtils;
import org.apache.commons.lang3.RandomUtils;
import org.assertj.core.api.InstanceOfAssertFactories;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.condition.EnabledOnOs;
import org.junit.jupiter.api.condition.OS;
@ -25,6 +26,9 @@ import static org.assertj.core.api.Assertions.assertThatThrownBy;
public class StringFilenameValidationUnitTest {
private static final String CORRECT_FILENAME_PATTERN = "baeldung.txt";
private static final String FILENAME_WITH_COLON = "bael:dung.txt";
private static final String FILENAME_WITH_FORWARD_SLASH = "bael/dung.txt";
private static final String FILENAME_WITH_BACKWARD_SLASH = "bael\\dung.txt";
@ParameterizedTest
@MethodSource("correctAlphanumericFilenamesProvider")
@ -79,17 +83,37 @@ public class StringFilenameValidationUnitTest {
@ParameterizedTest
@EnabledOnOs(OS.WINDOWS)
@MethodSource("filenamesWithInvalidWindowsChars")
public void givenFilenameStringWithInvalidWindowsCharAndIsWindows_whenValidateUsingIO_thenRaiseException(String filename) {
assertThatThrownBy(() -> validateStringFilenameUsingIO(filename))
.isInstanceOf(IOException.class)
.hasMessageContaining("Invalid file path");
public void givenFilenameStringWithInvalidWindowsCharAndIsWindows_whenValidateUsingIO_thenRaiseException(
String filename) {
assertThatThrownBy(() -> validateStringFilenameUsingIO(filename)).isInstanceOf(IOException.class)
.extracting(Throwable::getMessage, InstanceOfAssertFactories.STRING)
.containsAnyOf("The system cannot find the path specified",
"The filename, directory name, or volume label syntax is incorrect");
assertThatThrownBy(() -> validateStringFilenameUsingNIO2(filename)).isInstanceOf(InvalidPathException.class)
.hasMessageContaining("Illegal char");
assertThatThrownBy(() -> validateStringFilenameUsingNIO2(filename))
.isInstanceOf(InvalidPathException.class)
.hasMessage("character not allowed");
assertThat(validateStringFilenameUsingContains(filename)).isFalse();
}
assertThat(validateStringFilenameUsingContains(filename)).isFalse();
}
@Test
@EnabledOnOs(OS.WINDOWS)
public void givenFilenameStringWithInvalidColonWindowsCharAndIsWindows_thenNIO2FailsIOSucceed()
throws IOException {
assertThat(validateStringFilenameUsingIO(FILENAME_WITH_COLON)).isTrue();
assertThatThrownBy(() -> validateStringFilenameUsingNIO2(FILENAME_WITH_COLON))
.isInstanceOf(InvalidPathException.class).hasMessageContaining("Illegal char");
}
@Test
@EnabledOnOs(OS.WINDOWS)
public void givenFilenameStringWithInvalidSlashWindowsCharAndIsWindows_thenIOFailsNIO2Succeed() {
assertThatThrownBy(() -> validateStringFilenameUsingIO(FILENAME_WITH_FORWARD_SLASH))
.isInstanceOf(IOException.class).hasMessageContaining("The system cannot find the path specified");
assertThatThrownBy(() -> validateStringFilenameUsingIO(FILENAME_WITH_BACKWARD_SLASH))
.isInstanceOf(IOException.class).hasMessageContaining("The system cannot find the path specified");
assertThat(validateStringFilenameUsingNIO2(FILENAME_WITH_FORWARD_SLASH)).isTrue();
assertThat(validateStringFilenameUsingNIO2(FILENAME_WITH_BACKWARD_SLASH)).isTrue();
}
@ParameterizedTest
@EnabledOnOs({OS.LINUX, OS.MAC})