Test locations with ? or #

Historically these were forbidden. The cases are interesting to test.
hovaesco · Feb 21, 2024 · b4e7678 · b4e7678
1 parent 9846884
commit b4e7678
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/lib/trino-filesystem/src/test/java/io/trino/filesystem/TestLocations.java b/lib/trino-filesystem/src/test/java/io/trino/filesystem/TestLocations.java
@@ -40,6 +40,8 @@ private static Stream<Arguments> locations()
                 Arguments.of("s3:/test_dir", "test_file.txt", "s3:/test_dir/test_file.txt"),
                 Arguments.of("s3://test_dir", "test_file.txt", "s3://test_dir/test_file.txt"),
                 Arguments.of("s3://test_dir/", "test_file.txt", "s3://test_dir/test_file.txt"),
+                Arguments.of("s3://test_dir/", "location?", "s3://test_dir/location?"),
+                Arguments.of("s3://test_dir/", "location#", "s3://test_dir/location#"),
                 Arguments.of("s3://dir_with_space ", "test_file.txt", "s3://dir_with_space /test_file.txt"),
                 Arguments.of("s3://dir_with_double_space  ", "test_file.txt", "s3://dir_with_double_space  /test_file.txt"));
     }