[PATCH] D119040: Fix LookupTest where it was missing an assertion

[Paul Robinson via Phabricator via cfe-commits]

probinson created this revision.
probinson added a reviewer: ioeric.
probinson requested review of this revision.

The preceding EXPECT_EQ was never executed; modifying the test input
made that happen.

Found by the Rotten Green Tests project.


https://reviews.llvm.org/D119040

Files:
  clang/unittests/Tooling/LookupTest.cpp


Index: clang/unittests/Tooling/LookupTest.cpp
===================================================================
--- clang/unittests/Tooling/LookupTest.cpp
+++ clang/unittests/Tooling/LookupTest.cpp
@@ -210,7 +210,7 @@
     }
   };
   Visitor.runOver("namespace a { namespace b { class Foo {}; } }\n"
-                  "namespace c { using a::b::Foo; Foo f();; }\n");
+                  "namespace c { a::b::Foo f();; }\n");
 
   // Rename TypeLoc `x::y::Old` to new name `x::Foo` at [0] and check that the
   // type is replaced with "Foo" instead of "x::Foo". Although there is a 
symbol



Index: clang/unittests/Tooling/LookupTest.cpp
===================================================================
--- clang/unittests/Tooling/LookupTest.cpp
+++ clang/unittests/Tooling/LookupTest.cpp
@@ -210,7 +210,7 @@
     }
   };
   Visitor.runOver("namespace a { namespace b { class Foo {}; } }\n"
-                  "namespace c { using a::b::Foo; Foo f();; }\n");
+                  "namespace c { a::b::Foo f();; }\n");
 
   // Rename TypeLoc `x::y::Old` to new name `x::Foo` at [0] and check that the
   // type is replaced with "Foo" instead of "x::Foo". Although there is a symbol

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits