From d32d4344ec8886c53e32b51bc0cfb99baa1df61b Mon Sep 17 00:00:00 2001 From: Alexey Spiridonov Date: Tue, 22 Oct 2019 22:49:11 -0700 Subject: [PATCH] Don't use private googletest API in TestUtils.h Summary: Until [very recently](https://github.com/google/googletest/pull/2517), `googletest` did not print the skip message in any shape or form, so nothing can possibly depend on us customizing the skip message here. The default is perfectly fine, let's keep it simple. Reviewed By: yfeldblum Differential Revision: D18051635 fbshipit-source-id: f7b889b0adf5e34a70c2579a34748bb159809b3c --- folly/test/TestUtils.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/folly/test/TestUtils.h b/folly/test/TestUtils.h index ae7e0c19f7e..490934edcb1 100644 --- a/folly/test/TestUtils.h +++ b/folly/test/TestUtils.h @@ -76,7 +76,7 @@ // that can identify the "Test skipped by client" in the failure message // and convert this into a skipped test result. #ifdef GTEST_SKIP_ -#define SKIP(msg) GTEST_SKIP_("Test skipped by client") +#define SKIP(msg) GTEST_SKIP() #else #define SKIP() \ GTEST_AMBIGUOUS_ELSE_BLOCKER_ \