diff options
author | Hossein Ghahramanzadeh <hgsilverman@gmail.com> | 2021-12-17 01:45:56 (GMT) |
---|---|---|
committer | Hossein Ghahramanzadeh <hgsilverman@gmail.com> | 2022-01-06 14:46:52 (GMT) |
commit | 4adbc9c9b24a9e9a9ac6ad18b3e43b4363b87eb8 (patch) | |
tree | f0de12423e872c618cbc0cfe53d0b5ca353a6cc1 /googletest/src | |
parent | d03d23a6e5ec58699955dd951957532e5aea3a84 (diff) | |
download | googletest-4adbc9c9b24a9e9a9ac6ad18b3e43b4363b87eb8.zip googletest-4adbc9c9b24a9e9a9ac6ad18b3e43b4363b87eb8.tar.gz googletest-4adbc9c9b24a9e9a9ac6ad18b3e43b4363b87eb8.tar.bz2 |
Apply requested changes to preserve old behavior.
Diffstat (limited to 'googletest/src')
-rw-r--r-- | googletest/src/gtest.cc | 33 |
1 files changed, 20 insertions, 13 deletions
diff --git a/googletest/src/gtest.cc b/googletest/src/gtest.cc index 1537fd2..49fa896 100644 --- a/googletest/src/gtest.cc +++ b/googletest/src/gtest.cc @@ -50,6 +50,7 @@ #include <limits> #include <list> #include <map> +#include <numeric> #include <ostream> // NOLINT #include <sstream> #include <vector> @@ -724,26 +725,26 @@ static bool PatternMatchesString(const std::string& name_str, } namespace { + class UnitTestFilter { public: UnitTestFilter() = default; // Constructs a filter form a string of patterns separated by `:`. explicit UnitTestFilter(const std::string& filter) { - if (filter.empty()) return; - + // By design "" filter matches "" string. SplitString(filter, ':', &patterns_); } // Returns true if and only if name matches at least one of the patterns in // the filter. bool MatchesName(const std::string& name) const { - const auto pattern_matches_name = [&name](const std::string& pattern) { - return PatternMatchesString(name, pattern.c_str(), - pattern.c_str() + pattern.size()); - }; return std::any_of(patterns_.begin(), patterns_.end(), - pattern_matches_name); + [&name](const std::string& pattern) { + return PatternMatchesString( + name, pattern.c_str(), + pattern.c_str() + pattern.size()); + }); } private: @@ -766,10 +767,18 @@ class PositiveAndNegativeUnitTestFilter { if (positive_and_negative_filters.size() > 1) { positive_filter_ = UnitTestFilter{ positive_filter.size() ? positive_filter : kUniversalFilter}; - negative_filter_ = UnitTestFilter{positive_and_negative_filters.back()}; + + // TODO: Fail on multiple '-' characters + // For the moment to preserve old behavior we concatenate the rest of the + // string parts with `-` as separator to generate the negative filter. + negative_filter_ = UnitTestFilter{std::accumulate( + positive_and_negative_filters.begin() + 2, + positive_and_negative_filters.end(), positive_and_negative_filters[1], + [](const auto& lhs, const auto& rhs) { return lhs + '-' + rhs; })}; } else { - // In case positive filter is empty - // we do not use kUniversalFilter by design + // In case we don't have a negative filter and positive filter is "" + // we do not use kUniversalFilter by design as opposed to when we have a + // negative filter. positive_filter_ = UnitTestFilter{positive_filter}; } } @@ -779,9 +788,7 @@ class PositiveAndNegativeUnitTestFilter { // and does not match the negative filter. bool MatchesTest(const std::string& test_suite_name, const std::string& test_name) const { - const std::string& full_name = test_suite_name + "." + test_name.c_str(); - - return MatchesName(full_name); + return MatchesName(test_suite_name + "." + test_name); } // Returns true if and only if name matches the positive filter and does not |