Add duration_cast when printing expected ms

Summary: As per Yedidya Feldblum comment on my previous diff, we need a duration cast in here.

Reviewed By: yfeldblum

Differential Revision: D13833032

fbshipit-source-id: 8819694875bd3a7e4f54102d8ac77490f8287fbb

folly/io/async/test/Util.h

@@ -35,13 +35,15 @@
  * @param expectedMS            The timeout duration, in milliseconds
  * @param tolerance             The tolerance, in milliseconds.
  */
-#define T_CHECK_TIMEOUT(start, end, expectedMS, ...)                           \
-  EXPECT_TRUE(::folly::checkTimeout(                                           \
-      (start), (end), (expectedMS), false, ##__VA_ARGS__))                     \
-      << "Timeout violates constraints, expectedMs = " << (expectedMS).count() \
-      << ", elapsed wall time ms = "                                           \
-      << std::chrono::duration_cast<std::chrono::milliseconds>(                \
-             (end).getTime() - (start).getTime())                              \
+#define T_CHECK_TIMEOUT(start, end, expectedMS, ...)                       \
+  EXPECT_TRUE(::folly::checkTimeout(                                       \
+      (start), (end), (expectedMS), false, ##__VA_ARGS__))                 \
+      << "Timeout violates constraints, expectedMs = "                     \
+      << std::chrono::duration_cast<std::chrono::milliseconds>(expectedMS) \
+             .count()                                                      \
+      << ", elapsed wall time ms = "                                       \
+      << std::chrono::duration_cast<std::chrono::milliseconds>(            \
+             (end).getTime() - (start).getTime())                          \
              .count();
 
 /**
@@ -50,11 +52,13 @@
  * This is similar to T_CHECK_TIMEOUT, but does not fail if the event took less
  * than the allowed time.
  */
-#define T_CHECK_TIME_LT(start, end, expectedMS, ...)            \
-  EXPECT_TRUE(::folly::checkTimeout(                            \
-      (start), (end), (expectedMS), true, ##__VA_ARGS__))       \
-      << "Interval violates constraints, expectedMs = "         \
-      << (expectedMS).count() << ", elapsed wall time ms = "    \
-      << std::chrono::duration_cast<std::chrono::milliseconds>( \
-             (end).getTime() - (start).getTime())               \
+#define T_CHECK_TIME_LT(start, end, expectedMS, ...)                       \
+  EXPECT_TRUE(::folly::checkTimeout(                                       \
+      (start), (end), (expectedMS), true, ##__VA_ARGS__))                  \
+      << "Interval violates constraints, expectedMs = "                    \
+      << std::chrono::duration_cast<std::chrono::milliseconds>(expectedMS) \
+             .count()                                                      \
+      << ", elapsed wall time ms = "                                       \
+      << std::chrono::duration_cast<std::chrono::milliseconds>(            \
+             (end).getTime() - (start).getTime())                          \
              .count();