[ 
https://issues.apache.org/jira/browse/MESOS-407?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Vinod Kone resolved MESOS-407.
------------------------------

       Resolution: Fixed
    Fix Version/s: 0.13.0
    
> Google test filter processing is incorrect for the empty string.
> ----------------------------------------------------------------
>
>                 Key: MESOS-407
>                 URL: https://issues.apache.org/jira/browse/MESOS-407
>             Project: Mesos
>          Issue Type: Bug
>            Reporter: Benjamin Mahler
>            Assignee: Benjamin Hindman
>             Fix For: 0.13.0
>
>
> We regressed in that 'make check GTEST_FILTER=""' works for libprocess / 
> stout, but *not* for the mesos tests. For mesos, it runs all the tests:
> Note: Google Test filter = *-
> [==========] Running 224 tests from 48 test cases.
> [----------] Global test environment set-up.
> [----------] 2 tests from AllocatorZooKeeperTest/0
> ...
> This should just be - instead of *-.
> The bug is in the Environment() constructor:
>   // Use universal filter if not specified.
>   if (enabled.empty()) {
>     enabled = "*";
>   }
> The default filter seems to be "*", so this comment is incorrect.
> From the gtest code:
> // A test filter that matches everything.
> static const char kUniversalFilter[] = "*";
> GTEST_DEFINE_string_(
>     filter,
>     internal::StringFromGTestEnv("filter", kUniversalFilter),
>     "A colon-separated list of glob (not regex) patterns "
>     "for filtering the tests to run, optionally followed by a "
>     "'-' and a : separated list of negative patterns (tests to "
>     "exclude).  A test is run if it matches one of the positive "
>     "patterns and does not match any of the negative patterns.");

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to