[clang-tools-extra] r344030 - [clangd] Fix nondeterministic test

Sam McCall via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 9 03:02:03 PDT 2018


Author: sammccall
Date: Tue Oct  9 03:02:02 2018
New Revision: 344030

URL: http://llvm.org/viewvc/llvm-project?rev=344030&view=rev
Log:
[clangd] Fix nondeterministic test

Modified:
    clang-tools-extra/trunk/unittests/clangd/DexTests.cpp

Modified: clang-tools-extra/trunk/unittests/clangd/DexTests.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clangd/DexTests.cpp?rev=344030&r1=344029&r2=344030&view=diff
==============================================================================
--- clang-tools-extra/trunk/unittests/clangd/DexTests.cpp (original)
+++ clang-tools-extra/trunk/unittests/clangd/DexTests.cpp Tue Oct  9 03:02:02 2018
@@ -23,6 +23,7 @@
 #include <string>
 #include <vector>
 
+using ::testing::AnyOf;
 using ::testing::ElementsAre;
 using ::testing::UnorderedElementsAre;
 using namespace llvm;
@@ -257,7 +258,9 @@ TEST(DexIterators, StringRepresentation)
   EXPECT_EQ(llvm::to_string(*I2), "T=L2");
 
   auto Tree = C.limit(C.intersect(move(I1), move(I2)), 10);
-  EXPECT_EQ(llvm::to_string(*Tree), "(LIMIT 10 (& [1 3 5] T=L2))");
+  // AND reorders its children, we don't care which order it prints.
+  EXPECT_THAT(llvm::to_string(*Tree), AnyOf("(LIMIT 10 (& [1 3 5] T=L2))",
+                                            "(LIMIT 10 (& T=L2 [1 3 5]))"));
 }
 
 TEST(DexIterators, Limit) {




More information about the cfe-commits mailing list