[ 
https://issues.apache.org/jira/browse/GEODE-5035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16432795#comment-16432795
 ] 

Michael Dodge edited comment on GEODE-5035 at 4/10/18 7:16 PM:
---------------------------------------------------------------

Modifying {{build.gradle}} to explicitly pass the value of {{java.io.tmpdir}} 
down to the JVMs it invokes resulted in hundreds of test failures with stack 
traces like:

{quote}
java.io.IOException: No such file or directory
            at java.io.UnixFileSystem.createFileExclusively(Native Method)
            at java.io.File.createTempFile(File.java:2024)
            at 
org.junit.rules.TemporaryFolder.createTemporaryFolderIn(TemporaryFolder.java:134)
            at org.junit.rules.TemporaryFolder.create(TemporaryFolder.java:59)
            at 
org.apache.geode.test.junit.rules.GfshCommandRule.<init>(GfshCommandRule.java:87)
{quote}


was (Author: pivotalsarge):
Modifying {{build.gradle}} to explicitly pass the value of {{java.io.tmpdir}} 
down to the JVMs it invokes resulted in hundreds of test failures with stack 
traces like:
{{java.io.IOException: No such file or directory
            at java.io.UnixFileSystem.createFileExclusively(Native Method)
            at java.io.File.createTempFile(File.java:2024)
            at 
org.junit.rules.TemporaryFolder.createTemporaryFolderIn(TemporaryFolder.java:134)
            at org.junit.rules.TemporaryFolder.create(TemporaryFolder.java:59)
            at 
org.apache.geode.test.junit.rules.GfshCommandRule.<init>(GfshCommandRule.java:87)}}

> Some tests use incorrect temp directory
> ---------------------------------------
>
>                 Key: GEODE-5035
>                 URL: https://issues.apache.org/jira/browse/GEODE-5035
>             Project: Geode
>          Issue Type: Improvement
>          Components: tests
>            Reporter: Michael Dodge
>            Assignee: Michael Dodge
>            Priority: Major
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> Some tests that use {{JUnit}}'s {{TemporaryFolder}} rule (e.g., 
> {{DiskSpaceLimitIntegrationTest}} {{FileProcessControllerIntegrationTest}}) 
> do not specify the temp directory to use as the parent folder for the 
> temporary folder. This can result in the default location being used, which 
> is different from what is specified by the pipeline. This can result in 
> strange behavior, such as a test writing contents to a temporary file without 
> exception but then failing an existence test on the same file.
> The {{TemporaryFolder}} rule should be synchronized with the 
> {{java.io.tmpdir}} system property.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to