[clang-tools-extra] r230982 - clang-tidy: Update test as underlying problem apparently was fixed.

Daniel Jasper djasper at google.com
Mon Mar 2 10:07:00 PST 2015


Author: djasper
Date: Mon Mar  2 12:07:00 2015
New Revision: 230982

URL: http://llvm.org/viewvc/llvm-project?rev=230982&view=rev
Log:
clang-tidy: Update test as underlying problem apparently was fixed.

Modified:
    clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp

Modified: clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp?rev=230982&r1=230981&r2=230982&view=diff
==============================================================================
--- clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp (original)
+++ clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp Mon Mar  2 12:07:00 2015
@@ -15,8 +15,7 @@ TEST(ParseLineFilter, EmptyFilter) {
 
 TEST(ParseLineFilter, InvalidFilter) {
   ClangTidyGlobalOptions Options;
-  // TODO: Figure out why parsing succeeds here.
-  EXPECT_FALSE(parseLineFilter("asdf", Options));
+  EXPECT_TRUE(!!parseLineFilter("asdf", Options));
   EXPECT_TRUE(Options.LineFilter.empty());
 
   EXPECT_TRUE(!!parseLineFilter("[{}]", Options));





More information about the cfe-commits mailing list