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

Unified Diff: chromecast/app/linux/cast_crash_reporter_client_unittest.cc

Issue 1154383006: Adding crash utilities to chromecast/crash. (Closed) Base URL: https://eureka-internal.googlesource.com/chromium/src@master
Patch Set: cast_shell_unittests not built for android Created 5 years, 6 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 | « chromecast/app/linux/cast_crash_reporter_client.cc ('k') | chromecast/base/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromecast/app/linux/cast_crash_reporter_client_unittest.cc
diff --git a/chromecast/app/linux/cast_crash_reporter_client_unittest.cc b/chromecast/app/linux/cast_crash_reporter_client_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..88cbad9c50c328317bc5d28f6c29ff812afa3b06
--- /dev/null
+++ b/chromecast/app/linux/cast_crash_reporter_client_unittest.cc
@@ -0,0 +1,116 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include <fstream>
+
+#include "base/base_paths.h"
+#include "base/bind.h"
+#include "base/files/file.h"
+#include "base/files/file_util.h"
+#include "base/memory/scoped_vector.h"
+#include "base/test/scoped_path_override.h"
+#include "chromecast/app/linux/cast_crash_reporter_client.h"
+#include "chromecast/crash/app_state_tracker.h"
+#include "chromecast/crash/linux/crash_util.h"
+#include "chromecast/crash/linux/dump_info.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace chromecast {
+namespace {
+
+const char kFakeDumpstateContents[] = "Dumpstate Contents\nDumpdumpdumpdump\n";
+const char kFakeMinidumpContents[] = "Minidump Contents\nLine1\nLine2\n";
+
+int WriteFakeDumpStateFile(const std::string& path) {
+ // Append the correct extension and write the data to file.
+ base::File dumpstate(base::FilePath(path).AddExtension(".txt.gz"),
+ base::File::FLAG_CREATE_ALWAYS | base::File::FLAG_WRITE);
+ dumpstate.Write(
+ 0, kFakeDumpstateContents, sizeof(kFakeDumpstateContents) - 1);
+ return 0;
+}
+
+ScopedVector<DumpInfo> GetCurrentDumps(const std::string& logfile_path) {
+ ScopedVector<DumpInfo> dumps;
+ std::string entry;
+
+ std::ifstream in(logfile_path);
+ DCHECK(in.is_open());
+ while (std::getline(in, entry)) {
+ scoped_ptr<DumpInfo> info(new DumpInfo(entry));
+ dumps.push_back(info.Pass());
+ }
+ return dumps.Pass();
+}
+
+} // namespace
+
+TEST(CastCrashReporterClientTest, EndToEnd) {
+ // Set up a temporary directory which will be used as our fake home dir.
+ base::FilePath fake_home_dir;
+ ASSERT_TRUE(base::CreateNewTempDirectory("", &fake_home_dir));
+ base::ScopedPathOverride home(base::DIR_HOME, fake_home_dir);
+
+ // Set a callback to be used in place of the |dumpstate| executable.
+ CrashUtil::SetDumpStateCbForTest(base::Bind(&WriteFakeDumpStateFile));
+
+ // "Launch" YouTube.
+ AppStateTracker::SetLastLaunchedApp("youtube");
+ AppStateTracker::SetCurrentApp("youtube");
+
+ // "Launch" and switch to Pandora.
+ AppStateTracker::SetLastLaunchedApp("pandora");
+ AppStateTracker::SetCurrentApp("pandora");
+
+ // "Launch" Netflix.
+ AppStateTracker::SetLastLaunchedApp("netflix");
+ // Netflix crashed.
+
+ // A minidump file is created.
+ base::FilePath minidump_path;
+ base::CreateTemporaryFile(&minidump_path);
+ base::File minidump(minidump_path,
+ base::File::FLAG_OPEN | base::File::FLAG_APPEND);
+ minidump.Write(0, kFakeMinidumpContents, sizeof(kFakeMinidumpContents) - 1);
+ minidump.Close();
+
+ // Handle the crash.
+ CastCrashReporterClient client;
+ ASSERT_TRUE(client.HandleCrashDump(minidump_path.value().c_str()));
+
+ // Assert that the original file has been moved.
+ ASSERT_FALSE(base::PathExists(minidump_path));
+
+ // Assert that the file has been moved to "minidumps", with the expected
+ // contents.
+ std::string contents;
+ base::FilePath new_minidump =
+ fake_home_dir.Append("minidumps").Append(minidump_path.BaseName());
+ ASSERT_TRUE(base::PathExists(new_minidump));
+ ASSERT_TRUE(base::ReadFileToString(new_minidump, &contents));
+ ASSERT_EQ(kFakeMinidumpContents, contents);
+
+ // Assert that the dumpstate file has been written with the expected contents.
+ base::FilePath dumpstate = new_minidump.AddExtension(".txt.gz");
+ ASSERT_TRUE(base::PathExists(dumpstate));
+ ASSERT_TRUE(base::ReadFileToString(dumpstate, &contents));
+ ASSERT_EQ(kFakeDumpstateContents, contents);
+
+ // Assert that the lockfile has logged the correct information.
+ base::FilePath lockfile =
+ fake_home_dir.Append("minidumps").Append("lockfile");
+ ASSERT_TRUE(base::PathExists(lockfile));
+ ScopedVector<DumpInfo> dumps = GetCurrentDumps(lockfile.value());
+ ASSERT_EQ(1u, dumps.size());
+
+ const DumpInfo& dump_info = *(dumps[0]);
+ ASSERT_TRUE(dump_info.valid());
+ EXPECT_EQ(new_minidump.value(), dump_info.crashed_process_dump());
+ EXPECT_EQ(dumpstate.value(), dump_info.logfile());
+ EXPECT_EQ("youtube", dump_info.params().previous_app_name);
+ EXPECT_EQ("pandora", dump_info.params().current_app_name);
+ EXPECT_EQ("netflix", dump_info.params().last_app_name);
+}
+
+} // namespace chromecast
« no previous file with comments | « chromecast/app/linux/cast_crash_reporter_client.cc ('k') | chromecast/base/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698