[jira] [Comment Edited] (GEODE-5035) Some tests use incorrect temp directory

["Michael Dodge (JIRA)" <ji...@apache.org>]

    [ 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)