[GitHub] [pulsar] skysiders commented on a diff in pull request #15412: [fix][Broker] NarUnpacker create directory and file without permission check

[GitBox <gi...@apache.org>]

skysiders commented on code in PR #15412:
URL: https://github.com/apache/pulsar/pull/15412#discussion_r886905189


##########
pulsar-common/src/test/java/org/apache/pulsar/common/nar/NarUnpackerTest.java:
##########
@@ -157,4 +160,17 @@ File findJavaExe() {
         File javaExe = new File(javaHome, "bin/java" + (SystemUtils.IS_OS_WINDOWS ? ".exe" : ""));
         return javaExe;
     }
-}
\ No newline at end of file
+    @Test
+    public void testForPermissionSet() throws IOException{
+        File file = File.createTempFile("TestFilePerm", "test");
+	file.delete();

Review Comment:
   Thanks @Jason918 ,I fix this problem in this commit. Could you please have a look at this?



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org