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

Derek Chen-Becker commented on CASSANDRA-18750:
-----------------------------------------------

Before (tested against current trunk):

{code}
...
[checkstyle] [ERROR] 
/ssdhome/software/projects/cassandra/test/unit/org/apache/cassandra/streaming/StreamSessionTest.java:79:
 Please do not hardcode /tmp for test files and directories. Use 
Files.createTempDirectory to create a base test directory instead. 
[hardcodeTmpDirectoryUsage]

BUILD FAILED
/ssdhome/software/projects/cassandra/build.xml:521: The following error 
occurred while executing this line:
/ssdhome/software/projects/cassandra/.build/build-checkstyle.xml:57: Got 30 
errors and 0 warnings.
{code}

After the change it still failed, but on a CVE vulnerability in {{bcprov}}. 
Total runtime with the new rule was 2:42.



> Make unit tests compatible with the tmp.dir property
> ----------------------------------------------------
>
>                 Key: CASSANDRA-18750
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18750
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Build
>            Reporter: Derek Chen-Becker
>            Assignee: Derek Chen-Becker
>            Priority: Normal
>
> Several unit tests hard-code file paths under the "/tmp" directory, which 
> means they do not honor the {{tmp.dir}} ant build property. These should be 
> updated so that when the user specifies {{tmp.dir}}, they can be certain that 
> any files or directories created by the unit tests will be in their specified 
> location.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to