Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4)

Unified Diff: crash_collector_test.cc

Issue 4018008: crash-reporter: Generate kernel crash signatures for server-side grouping of similar crashes (Closed) Base URL: http://git.chromium.org/git/crash-reporter.git
Patch Set: Respond to petkov review Created 10 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « crash_collector.cc ('k') | crash_reporter.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: crash_collector_test.cc
diff --git a/crash_collector_test.cc b/crash_collector_test.cc
index 2dbf2bb2d433578cbec9affd5bd4c97478258747..ffab06af85cdbfea44a7955b3be01d43188e72f5 100644
--- a/crash_collector_test.cc
+++ b/crash_collector_test.cc
@@ -218,6 +218,30 @@ TEST_F(CrashCollectorTest, ReadKeyValueFile) {
EXPECT_TRUE(i != dictionary.end() && i->second == "");
}
+TEST_F(CrashCollectorTest, MetaData) {
+ FilePath meta_file = test_dir_.Append("generated.meta");
+ FilePath lsb_release = test_dir_.Append("lsb-release");
+ FilePath payload_file = test_dir_.Append("payload-file");
+ std::string contents;
+ collector_.lsb_release_ = lsb_release.value().c_str();
+ const char kLsbContents[] = "CHROMEOS_RELEASE_VERSION=version\n";
+ ASSERT_TRUE(
+ file_util::WriteFile(lsb_release,
+ kLsbContents, strlen(kLsbContents)));
+ const char kPayload[] = "foo";
+ ASSERT_TRUE(
+ file_util::WriteFile(payload_file,
+ kPayload, strlen(kPayload)));
+ collector_.AddCrashMetaData("foo", "bar");
+ collector_.WriteCrashMetaData(meta_file, "kernel", payload_file.value());
+ EXPECT_TRUE(file_util::ReadFileToString(meta_file, &contents));
+ EXPECT_EQ("foo=bar\n"
+ "exec_name=kernel\n"
+ "ver=version\n"
+ "payload_size=3\n"
+ "done=1\n", contents);
+}
+
int main(int argc, char **argv) {
::testing::InitGoogleTest(&argc, argv);
return RUN_ALL_TESTS();
« no previous file with comments | « crash_collector.cc ('k') | crash_reporter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698