chromeos-base/quipper/files/quipper-disable-flaky-tests.patch

disable tests that are flaky until someone can look into them

https://crbug.com/812425

--- a/perf_parser_test.cc
+++ b/perf_parser_test.cc
@@ -1968,7 +1968,7 @@ class RunInMountNamespaceThread : public quipper::Thread {
 // (Not tried):            /proc/<pid>/root/<path>
 // (Not tried): /<path>
 // Expected buildid for <path>: "deadbeef"
-TEST(PerfParserTest, ReadsBuildidsInMountNamespace) {
+TEST(PerfParserTest, DISABLED_ReadsBuildidsInMountNamespace) {
   if (!HaveCapability(CAP_SYS_ADMIN)) return;  // Skip test.
   ScopedTempDir tmpdir("/tmp/quipper_tmp.");
   ScopedTempDir mntdir("/tmp/quipper_mnt.");
@@ -2103,7 +2103,7 @@ class RunInMountNamespaceProcess {
 // Accept:                 /proc/<pid2>/root/<path>
 // (Not tried): /<path>
 // Expected buildid for <path>: "deadbeef"
-TEST(PerfParserTest, ReadsBuildidsInMountNamespace_TriesOwningProcess) {
+TEST(PerfParserTest, DISABLED_ReadsBuildidsInMountNamespace_TriesOwningProcess) {
   if (!HaveCapability(CAP_SYS_ADMIN)) return;  // Skip test.
   ScopedTempDir tmpdir("/tmp/quipper_tmp.");
   ScopedTempDir mntdir("/tmp/quipper_mnt.");
@@ -2189,7 +2189,7 @@ TEST(PerfParserTest, ReadsBuildidsInMountNamespace_TriesOwningProcess) {
 // Reject (doesn't exist): /proc/<pid+10>/root/<path>
 // Accept (same inode): /<path>
 // Expected buildid for <path>: "deadbeef"
-TEST(PerfParserTest, ReadsBuildidsInMountNamespace_TriesRootFs) {
+TEST(PerfParserTest, DISABLED_ReadsBuildidsInMountNamespace_TriesRootFs) {
   if (!HaveCapability(CAP_SYS_ADMIN)) return;  // Skip test.
   ScopedTempDir tmpdir("/tmp/quipper_tmp.");
   ScopedTempDir mntdir("/tmp/quipper_mnt.");
@@ -2274,7 +2274,7 @@ TEST(PerfParserTest, ReadsBuildidsInMountNamespace_TriesRootFs) {
 // Reject (wrong inode): /proc/<pid>/root/<path>
 // Reject (wrong inode): /<path>
 // Expected buildid for <path>: ""
-TEST(PerfParserTest, ReadsBuildidsInMountNamespace_TriesRootFsRejectsInode) {
+TEST(PerfParserTest, DISABLED_ReadsBuildidsInMountNamespace_TriesRootFsRejectsInode) {
   if (!HaveCapability(CAP_SYS_ADMIN)) return;  // Skip test.
   ScopedTempDir tmpdir("/tmp/quipper_tmp.");
   ScopedTempDir mntdir("/tmp/quipper_mnt.");
@@ -2363,7 +2363,7 @@ TEST(PerfParserTest, ReadsBuildidsInMountNamespace_TriesRootFsRejectsInode) {
 // Reject (not found): /proc/<pid+10>/root/<path>
 // Accept (falsely): /<path>
 // Expected buildid for <path>: "baadf00d" (even though incorrect)
-TEST(PerfParserTest, ReadsBuildidsInMountNamespace_TriesRootFsNoInodeToReject) {
+TEST(PerfParserTest, DISABLED_ReadsBuildidsInMountNamespace_TriesRootFsNoInodeToReject) {
   if (!HaveCapability(CAP_SYS_ADMIN)) return;  // Skip test.
   ScopedTempDir tmpdir("/tmp/quipper_tmp.");
   ScopedTempDir mntdir("/tmp/quipper_mnt.");
@@ -2460,7 +2460,7 @@ TEST(PerfParserTest, ReadsBuildidsInMountNamespace_TriesRootFsNoInodeToReject) {
 // A better thing to do might be to track a
 // map<tuple<maj,min,ino,path>, DSOInfo>, but even so, it will be impossible
 // to store unambiguously in perf.data.
-TEST(PerfParserTest, ReadsBuildidsInMountNamespace_DifferentDevOrIno) {
+TEST(PerfParserTest, DISABLED_ReadsBuildidsInMountNamespace_DifferentDevOrIno) {
   if (!HaveCapability(CAP_SYS_ADMIN)) return;  // Skip test.
   ScopedTempDir tmpdir("/tmp/quipper_tmp.");
   ScopedTempDir mntdir("/tmp/quipper_mnt.");

https://crbug.com/1000545

--- a/perf_serializer_test.cc
+++ b/perf_serializer_test.cc
@@ -457,7 +457,7 @@ TEST_P(SerializePerfDataFiles, TestBuildIDs) {
   EXPECT_TRUE(deserializer.Deserialize(perf_data_proto));
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesTraceMetadata) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesTraceMetadata) {
   std::stringstream input;
 
   const size_t data_size =
@@ -501,7 +501,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesTraceMetadata) {
   EXPECT_EQ(tracing_metadata_str, deserializer.tracing_data());
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesPerfEventAttrEvent) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesPerfEventAttrEvent) {
   std::stringstream input;
 
   // header
@@ -541,7 +541,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesPerfEventAttrEvent) {
   EXPECT_EQ(0, perf_data_proto.events().size());
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesPerfFileAttr) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesPerfFileAttr) {
   std::stringstream input;
 
   // header
@@ -584,7 +584,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesPerfFileAttr) {
   EXPECT_EQ(0, perf_data_proto.events().size());
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesMmapEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesMmapEvents) {
   std::stringstream input;
 
   // header
@@ -651,7 +651,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesMmapEvents) {
   }
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesAuxtraceInfoEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesAuxtraceInfoEvents) {
   std::stringstream input;
 
   // header
@@ -693,7 +693,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesAuxtraceInfoEvents) {
   }
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesAuxtraceEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesAuxtraceEvents) {
   std::stringstream input;
 
   // header
@@ -734,7 +734,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesAuxtraceEvents) {
   }
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesAuxtraceErrorEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesAuxtraceErrorEvents) {
   std::stringstream input;
 
   // header
@@ -794,7 +794,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesAuxtraceErrorEvents) {
   EXPECT_TRUE(matches) << difference;
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesThreadMapEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesThreadMapEvents) {
   std::stringstream input;
 
   // header
@@ -840,7 +840,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesThreadMapEvents) {
   }
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesStatConfigEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesStatConfigEvents) {
   std::stringstream input;
 
   // header
@@ -910,7 +910,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesStatConfigEvents) {
   EXPECT_TRUE(matches) << difference;
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesStatEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesStatEvents) {
   std::stringstream input;
 
   // header
@@ -961,7 +961,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesStatEvents) {
   EXPECT_TRUE(matches) << difference;
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesStatRoundEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesStatRoundEvents) {
   std::stringstream input;
 
   // header
@@ -1009,7 +1009,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesStatRoundEvents) {
   EXPECT_TRUE(matches) << difference;
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesTimeConvEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesTimeConvEvents) {
   std::stringstream input;
 
   // header
@@ -1047,7 +1047,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesTimeConvEvents) {
 }
 
 // Regression test for http://crbug.com/501004.
-TEST(PerfSerializerTest, SerializesAndDeserializesBuildIDs) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesBuildIDs) {
   std::stringstream input;
 
   // header
@@ -1178,7 +1178,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesBuildIDs) {
 }
 
 // Regression test for http://crbug.com/500746.
-TEST(PerfSerializerTest, SerializesAndDeserializesForkAndExitEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesForkAndExitEvents) {
   std::stringstream input;
 
   // header
@@ -1287,7 +1287,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesForkAndExitEvents) {
   }
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesContextSwitchEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesContextSwitchEvents) {
   std::stringstream input;
 
   // header
@@ -1373,7 +1373,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesContextSwitchEvents) {
   }
 }
 
-TEST(PerfSerializerTest, SerializesAndDeserializesNamespacesEvents) {
+TEST(PerfSerializerTest, DISABLED_SerializesAndDeserializesNamespacesEvents) {
   std::stringstream input;
 
   // header
@@ -1430,7 +1430,7 @@ TEST(PerfSerializerTest, SerializesAndDeserializesNamespacesEvents) {
 }
 
 // Regression test for http://crbug.com/500746.
-TEST(PerfSerializerTest, DeserializeLegacyExitEvents) {
+TEST(PerfSerializerTest, DISABLED_DeserializeLegacyExitEvents) {
   std::stringstream input;
 
   // header

https://crbug.com/1021952

--- a/perf_recorder_test.cc
+++ b/perf_recorder_test.cc
@@ -41,7 +41,7 @@ class PerfRecorderTest : public ::testing::Test {
   PerfRecorder perf_recorder_;
 };
 
-TEST_F(PerfRecorderTest, RecordToProtobuf) {
+TEST_F(PerfRecorderTest, DISABLED_RecordToProtobuf) {
   // Read perf data using the PerfReader class.
   // Dump it to a string and convert to a protobuf.
   // Read the protobuf, and reconstruct the perf data.
@@ -69,7 +69,7 @@ TEST_F(PerfRecorderTest, RecordToProtobuf) {
   EXPECT_EQ("0.2", command.Get(6).value());
 }
 
-TEST_F(PerfRecorderTest, StatToProtobuf) {
+TEST_F(PerfRecorderTest, DISABLED_StatToProtobuf) {
   // Run perf stat and verify output.
   std::string output_string;
   EXPECT_TRUE(perf_recorder_.RunCommandAndGetSerializedOutput(
@@ -81,7 +81,7 @@ TEST_F(PerfRecorderTest, StatToProtobuf) {
   EXPECT_GT(stat.line_size(), 0);
 }
 
-TEST_F(PerfRecorderTest, MemRecordToProtobuf) {
+TEST_F(PerfRecorderTest, DISABLED_MemRecordToProtobuf) {
   if (!IsPerfMemRecordAvailable()) return;
 
   // Run perf mem record and verify output.
@@ -94,7 +94,7 @@ TEST_F(PerfRecorderTest, MemRecordToProtobuf) {
   ASSERT_TRUE(perf_data_proto.ParseFromString(output_string));
 }
 
-TEST_F(PerfRecorderTest, StatSingleEvent) {
+TEST_F(PerfRecorderTest, DISABLED_StatSingleEvent) {
   std::string output_string;
   ASSERT_TRUE(perf_recorder_.RunCommandAndGetSerializedOutput(
       {"perf", "stat", "-a", "-e", "cpu-clock"}, 0.2, &output_string));
@@ -118,7 +118,7 @@ TEST_F(PerfRecorderTest, StatSingleEvent) {
   EXPECT_EQ("cpu-clock", stat.line(0).event_name());
 }
 
-TEST_F(PerfRecorderTest, StatMultipleEvents) {
+TEST_F(PerfRecorderTest, DISABLED_StatMultipleEvents) {
   std::string output_string;
   ASSERT_TRUE(perf_recorder_.RunCommandAndGetSerializedOutput(
       {"perf", "stat", "-a", "-e", "cpu-clock", "-e", "context-switches", "-e",
@@ -173,7 +173,7 @@ TEST_F(PerfRecorderTest, StatMultipleEvents) {
   EXPECT_EQ("page-faults", stat.line(3).event_name());
 }
 
-TEST_F(PerfRecorderTest, DontAllowCommands) {
+TEST_F(PerfRecorderTest, DISABLED_DontAllowCommands) {
   std::string output_string;
   EXPECT_FALSE(perf_recorder_.RunCommandAndGetSerializedOutput(
       {"perf", "record", "--", "sh", "-c", "echo 'malicious'"}, 0.2,
@@ -183,7 +183,7 @@ TEST_F(PerfRecorderTest, DontAllowCommands) {
       &output_string));
 }
 
-TEST(PerfRecorderNoPerfTest, FailsIfPerfDoesntExist) {
+TEST(PerfRecorderNoPerfTest, DISABLED_FailsIfPerfDoesntExist) {
   std::string output_string;
   PerfRecorder perf_recorder({"sudo", "/doesnt-exist/usr/not-bin/not-perf"});
   EXPECT_FALSE(perf_recorder.RunCommandAndGetSerializedOutput(