Disable SpdyCredentialBuilder tests with OpenSSL.

net/spdy/spdy_credential_builder_unittest.cc

Index: net/spdy/spdy_credential_builder_unittest.cc
diff --git a/net/spdy/spdy_credential_builder_unittest.cc b/net/spdy/spdy_credential_builder_unittest.cc
index 575ce88681d66776608128517e42e732de37f6da..85c61c60cc06d01b226c670de2a7448c4f69b3fa 100644
--- a/net/spdy/spdy_credential_builder_unittest.cc
+++ b/net/spdy/spdy_credential_builder_unittest.cc
@@ -75,28 +75,38 @@ class SpdyCredentialBuilderTest : public testing::Test {
   SpdyCredential credential_;
 };
 
-TEST_F(SpdyCredentialBuilderTest, GetCredentialSecret) {
+// http://crbug.com/142833. The following tests fail on Linux redux which uses
+// OpenSSL due to the unimplemented ec_private_key_openssl.cc.

[Ryan Sleevi, 2012/08/16 17:34:05]

Is there a reason you're using OS_LINUX here? Android also uses OpenSSL, and
also is unimplemented there.

Also, the Chromium style guide recommends MAYBE_ as the prefix here -
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium/han...

Yes, this means duplicating the #ifdef between the code, but this is the
existing pattern in Chromium code.

[Philippe, 2012/08/17 09:10:34]

Android had its own way of disabling this test through an entry in
build/android/gtest_filter/net_unittests_disabled. I removed it from there and
made MAYBE_ depend only on USE_OPENSSL here.

I also updated the code by following the link you pointed out.

Also, would it be reasonable to keep the old approach by renaming
DISABLED_ON_LINUX_REDUX() to MAYBE() (although this could lead to name clashes)
to avoid duplicating all this code?

[Ryan Sleevi, 2012/08/17 17:17:25]

The use of macro functions tends to harm readability - because it adds an extra
layer of indirection that has to be followed by the caller. This is especially
true with stringizing macros, since it can be hard to find the final string.

I think how you've written it is fine, and makes it clear at each test site
which tests are disabled on which platforms. This is also important because it
still leaves the possibility that individual tests can be disabled on other
platforms as necessary.

+#if defined(OS_LINUX) && defined(USE_OPENSSL)
+#define DISABLED_ON_LINUX_REDUX(Test) DISABLED_##Test
+#else
+#define DISABLED_ON_LINUX_REDUX(Test) Test
+#endif
+
+TEST_F(SpdyCredentialBuilderTest,
+       DISABLED_ON_LINUX_REDUX(GetCredentialSecret)) {
   std::string secret_str(kSecretPrefix, arraysize(kSecretPrefix));
   secret_str.append(MockClientSocket::kTlsUnique);
 
   EXPECT_EQ(secret_str, GetCredentialSecret());
 }
 
-TEST_F(SpdyCredentialBuilderTest, SucceedsWithECDSACert) {
+TEST_F(SpdyCredentialBuilderTest,
+       DISABLED_ON_LINUX_REDUX(SucceedsWithECDSACert)) {
   EXPECT_EQ(OK, BuildWithType(CLIENT_CERT_ECDSA_SIGN));
 }
 
-TEST_F(SpdyCredentialBuilderTest, FailsWithRSACert) {
+TEST_F(SpdyCredentialBuilderTest, DISABLED_ON_LINUX_REDUX(FailsWithRSACert)) {
   EXPECT_EQ(ERR_BAD_SSL_CLIENT_AUTH_CERT,
             BuildWithType(CLIENT_CERT_RSA_SIGN));
 }
 
-TEST_F(SpdyCredentialBuilderTest, SetsSlotCorrectly) {
+TEST_F(SpdyCredentialBuilderTest, DISABLED_ON_LINUX_REDUX(SetsSlotCorrectly)) {
   ASSERT_EQ(OK, Build());
   EXPECT_EQ(kSlot, credential_.slot);
 }
 
-TEST_F(SpdyCredentialBuilderTest, SetsCertCorrectly) {
+TEST_F(SpdyCredentialBuilderTest, DISABLED_ON_LINUX_REDUX(SetsCertCorrectly)) {
   ASSERT_EQ(OK, Build());
   base::StringPiece spki;
   ASSERT_TRUE(asn1::ExtractSPKIFromDERCert(cert_, &spki));
@@ -108,7 +118,7 @@ TEST_F(SpdyCredentialBuilderTest, SetsCertCorrectly) {
   EXPECT_EQ(spk.substr(2, spk.length()).as_string(), credential_.certs[0]);
 }
 
-TEST_F(SpdyCredentialBuilderTest, SetsProofCorrectly) {
+TEST_F(SpdyCredentialBuilderTest, DISABLED_ON_LINUX_REDUX(SetsProofCorrectly)) {
   ASSERT_EQ(OK, Build());
   base::StringPiece spki;
   ASSERT_TRUE(asn1::ExtractSPKIFromDERCert(cert_, &spki));
@@ -130,4 +140,6 @@ TEST_F(SpdyCredentialBuilderTest, SetsProofCorrectly) {
   EXPECT_EQ(proof, credential_.proof);
 }
 
+#undef DISABLED_ON_LINUX_REDUX
+
 }  // namespace net