fix: fast_transfer_file API 分发文件，如果源文件中的文件名包含空格，会报错 #812

TencentBlueKing · May 15, 2023 · f8e422b · f8e422b
1 parent 9353d9c
commit f8e422b
Showing 1 changed file with 4 additions and 9 deletions.
diff --git a/...s/common-utils/src/test/java/com/tencent/bk/job/common/util/FilePathValidateUtilTest.java b/...s/common-utils/src/test/java/com/tencent/bk/job/common/util/FilePathValidateUtilTest.java
@@ -6,8 +6,7 @@
 
 public class FilePathValidateUtilTest {
     @Test
-    void testFileSystemAbsolutePath(){
-        // 传统DOS路径
+    void testWindowsFilePath(){
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("C:\\Documents\\abc.txt")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("c:\\Documents\\abc.txt")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("C:\\Documents\\嘉 abc.txt")).isTrue();
@@ -37,9 +36,10 @@ void testFileSystemAbsolutePath(){
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("${path}\\test.txt")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("C:\\tmp\\${path}")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("C:\\${date}\\${path}")).isTrue();
+    }
 
-        // linux路径
-        assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/data/test_2022-04-12.apk")).isTrue();
+    @Test
+    void testLinuxFilePath(){
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/data/test_2022 04 12.apk")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp/")).isTrue();
@@ -49,11 +49,6 @@ void testFileSystemAbsolutePath(){
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("///")).isTrue(); // 根目录
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp////")).isTrue(); // /tmp/
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp//test/")).isTrue();// /tmp/test/
-        assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp/abc>a")).isTrue();
-        assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp/abc:a")).isTrue();
-        assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp/abc|a")).isTrue();
-        assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp/abc?a")).isTrue();
-        assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp/abc<a")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/logs/log*.log")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp/REGEX:myfile-[A-Za-z]{0,10}.tar.gz")).isTrue();
         assertThat(FilePathValidateUtil.validateFileSystemAbsolutePath("/tmp/REGEX:aa|bb")).isTrue();