haosdent created MESOS-4653:
-------------------------------

             Summary: Unify test case temporary folder name format
                 Key: MESOS-4653
                 URL: https://issues.apache.org/jira/browse/MESOS-4653
             Project: Mesos
          Issue Type: Improvement
            Reporter: haosdent
            Assignee: haosdent
            Priority: Minor


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)

Reply via email to