folly: fixup folly::Random FixedSeed test expectations on macOS
authorWez Furlong <wez@fb.com>
Fri, 22 Jul 2016 00:51:38 +0000 (17:51 -0700)
committerFacebook Github Bot 9 <facebook-github-bot-9-bot@fb.com>
Fri, 22 Jul 2016 00:53:25 +0000 (17:53 -0700)
Summary:
uniform_int_distribution is implemented differently on this system and
always returns the generated number from our constant random number generator.

Reviewed By: yfeldblum

Differential Revision: D3585418

fbshipit-source-id: 754076599ba77b49c17f03bb0c92264a3bed4ab0

folly/test/RandomTest.cpp

index cc576ae4995a2b09fc81394e76ee0d2b87a32543..d7c4f4a58ab988ac4a64b00bbbad1b069e0a3e17 100644 (file)
@@ -65,12 +65,14 @@ TEST(Random, FixedSeed) {
   // clang-format on
 
   ConstantRNG gen;
+
+  // Pick a constant random number...
+  auto value = Random::rand32(10, gen);
+
   // Loop to make sure it really is constant.
   for (int i = 0; i < 1024; ++i) {
     auto result = Random::rand32(10, gen);
-    // TODO: This is a little bit brittle; standard library changes could break
-    // it, if it starts implementing distribution types differently.
-    EXPECT_EQ(0, result);
+    EXPECT_EQ(value, result);
   }
 }