lidavidm commented on code in PR #34312:
URL: https://github.com/apache/arrow/pull/34312#discussion_r1150860565
##########
java/dataset/src/test/java/org/apache/arrow/dataset/file/TestFileSystemDataset.java:
##########
@@ -386,6 +388,27 @@ public void testBaseCsvRead() throws Exception {
}
}
+ @Test
+ public void testResourceToLoad() throws IOException {
+ if (System.getProperty("os.name").equalsIgnoreCase("Windows")) {
+ assertEquals("\\", File.separator);
+ final String pathWithoutFileSeparator = "/" + "avroschema" + "/" +
"user.avsc";
+ try (final InputStream stream =
TestFileSystemDataset.class.getResourceAsStream(pathWithoutFileSeparator)) {
+ if (stream == null) {
+ throw new FileNotFoundException(pathWithoutFileSeparator);
+ }
+ }
+ final String pathWithFileSeparator = File.separator + "avroschema" +
File.separator + "user.avsc";
+ Assertions.assertThrows(FileNotFoundException.class, () -> {
+ try (final InputStream stream =
TestFileSystemDataset.class.getResourceAsStream(pathWithFileSeparator)) {
+ if (stream == null) {
+ throw new FileNotFoundException(pathWithFileSeparator);
+ }
+ }
+ }, "FileNotFound \\avroschema\\user.avsc");
+ }
+ }
+
Review Comment:
I don't see how this test changes that. And I don't see what you mean: it
seems this should have failed and been caught if any of the tests ran on
Windows at all. So the main problem is to make sure we test JNI code on Windows
somewhere.
--
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]
