Merge pull request #7816 from bitjammer/reenable-threadsafe-caching-test

bitjammer · web-flow · commit 8cf99954e3d2 · 2017-02-28T14:03:33.000-08:00
[Syntax] Reenable thread-safe caching test
diff --git a/unittests/Syntax/ThreadSafeCachingTests.cpp b/unittests/Syntax/ThreadSafeCachingTests.cpp
@@ -81,7 +81,6 @@ class Pool {
 // - Only one thread inserts the realized child into the parent
 // - Both threads get the exact same child (by identity)
 TEST(ThreadSafeCachingTests, ReturnGetExpression) {
-#if 0
   auto ReturnKW = SyntaxFactory::makeReturnKeyword({}, Trivia::spaces(1));
   auto Minus = SyntaxFactory::makePrefixOpereator("-", {});
   auto One = SyntaxFactory::makeIntegerLiteralToken("1", {}, {});
@@ -104,6 +103,5 @@ TEST(ThreadSafeCachingTests, ReturnGetExpression) {
     ASSERT_EQ(FirstDataPointer, SecondDataPointer);
     ASSERT_EQ(FirstDataPointer, DataPointer);
   }
-#endif
 }
 

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,6 @@ class Pool {`
`81`	`81`	`// - Only one thread inserts the realized child into the parent`
`82`	`82`	`// - Both threads get the exact same child (by identity)`
`83`	`83`	`TEST(ThreadSafeCachingTests, ReturnGetExpression) {`
`84`		`-#if 0`
`85`	`84`	`auto ReturnKW = SyntaxFactory::makeReturnKeyword({}, Trivia::spaces(1));`
`86`	`85`	`auto Minus = SyntaxFactory::makePrefixOpereator("-", {});`
`87`	`86`	`auto One = SyntaxFactory::makeIntegerLiteralToken("1", {}, {});`
`@@ -104,6 +103,5 @@ TEST(ThreadSafeCachingTests, ReturnGetExpression) {`
`104`	`103`	`ASSERT_EQ(FirstDataPointer, SecondDataPointer);`
`105`	`104`	`ASSERT_EQ(FirstDataPointer, DataPointer);`
`106`	`105`	`}`
`107`		`-#endif`
`108`	`106`	`}`
`109`	`107`