mesos-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joseph Wu (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MESOS-4653) Unify test case temporary folder name format
Date Thu, 11 Feb 2016 16:31:18 GMT

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

Joseph Wu commented on MESOS-4653:
----------------------------------

[~haosdent@gmail.com] If you'd like to work on this, feel free to take [MESOS-3848], which
is already scoped.  I believe [~jieyu] should be willing to shepherd (but confirm that he
has cycles first).

> Unify test case temporary folder name format
> --------------------------------------------
>
>                 Key: MESOS-4653
>                 URL: https://issues.apache.org/jira/browse/MESOS-4653
>             Project: Mesos
>          Issue Type: Improvement
>          Components: test
>            Reporter: haosdent
>            Assignee: haosdent
>            Priority: Minor
>              Labels: test
>
> In [environment.cpp#L759https://github.com/apache/mesos/blob/master/src/tests/environment.cpp#L759]
> {code}
>   const string& path =
>     path::join("/tmp", strings::join("_", testCase, testName, "XXXXXX"));
> {code}
> The temporary file format here is {{testCase_testName_xxxxxx}} here.
> But in [utils.hpp#L37|https://github.com/apache/mesos/blob/master/3rdparty/libprocess/3rdparty/stout/include/stout/tests/utils.hpp#L37]
> {code}
>     // Create a temporary directory for the test.
>     Try<std::string> directory = os::mkdtemp();
> {code}
> The temporary folder we create here is {{xxxxxx}}. I think it would be better we could
unify this.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message