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

Unified Diff: minidump/minidump_crashpad_info_writer_test.cc

Issue 654573003: Add MinidumpCrashpadInfoWriter and its test (Closed) Base URL: https://chromium.googlesource.com/crashpad/crashpad@master
Patch Set: Created 6 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
Index: minidump/minidump_crashpad_info_writer_test.cc
diff --git a/minidump/minidump_crashpad_info_writer_test.cc b/minidump/minidump_crashpad_info_writer_test.cc
new file mode 100644
index 0000000000000000000000000000000000000000..f3927a21601a7c0cb065ba3372827dba3e0c6759
--- /dev/null
+++ b/minidump/minidump_crashpad_info_writer_test.cc
@@ -0,0 +1,155 @@
+// Copyright 2014 The Crashpad Authors. All rights reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+#include "minidump/minidump_crashpad_info_writer.h"
+
+#include <dbghelp.h>
+
+#include "gtest/gtest.h"
+#include "minidump/minidump_extensions.h"
+#include "minidump/minidump_file_writer.h"
+#include "minidump/minidump_file_writer_test_util.h"
+#include "minidump/minidump_simple_string_dictionary_writer.h"
+#include "minidump/minidump_string_writer_test_util.h"
+#include "util/file/string_file_writer.h"
+
+namespace crashpad {
+namespace test {
+namespace {
+
+void GetCrashpadInfoStream(
+ const std::string& file_contents,
+ const MinidumpCrashpadInfo** crashpad_info,
+ const MinidumpSimpleStringDictionary** simple_annotations) {
+ const size_t kDirectoryOffset = sizeof(MINIDUMP_HEADER);
+ const size_t kCrashpadInfoStreamOffset =
+ kDirectoryOffset + sizeof(MINIDUMP_DIRECTORY);
+ size_t end = kCrashpadInfoStreamOffset + sizeof(MinidumpCrashpadInfo);
+ const size_t kSimpleAnnotationsOffset = simple_annotations ? end : 0;
+ if (simple_annotations) {
+ end += sizeof(MinidumpSimpleStringDictionary);
+ }
+ const size_t kFileSize = end;
+
+ if (!simple_annotations) {
+ ASSERT_EQ(kFileSize, file_contents.size());
+ } else {
+ EXPECT_GE(file_contents.size(), kFileSize);
+ }
+
+ const MINIDUMP_HEADER* header =
+ reinterpret_cast<const MINIDUMP_HEADER*>(&file_contents[0]);
+
+ ASSERT_NO_FATAL_FAILURE(VerifyMinidumpHeader(header, 1, 0));
+
+ const MINIDUMP_DIRECTORY* directory =
+ reinterpret_cast<const MINIDUMP_DIRECTORY*>(
+ &file_contents[kDirectoryOffset]);
+
+ ASSERT_EQ(kMinidumpStreamTypeCrashpadInfo, directory->StreamType);
+ ASSERT_EQ(sizeof(MinidumpCrashpadInfo), directory->Location.DataSize);
+ ASSERT_EQ(kCrashpadInfoStreamOffset, directory->Location.Rva);
+
+ *crashpad_info = reinterpret_cast<const MinidumpCrashpadInfo*>(
+ &file_contents[kCrashpadInfoStreamOffset]);
+
+ if (simple_annotations) {
+ ASSERT_GE((*crashpad_info)->simple_annotations.DataSize,
+ sizeof(MinidumpSimpleStringDictionary));
+ ASSERT_EQ(kSimpleAnnotationsOffset,
+ (*crashpad_info)->simple_annotations.Rva);
+ *simple_annotations =
+ reinterpret_cast<const MinidumpSimpleStringDictionary*>(
+ &file_contents[kSimpleAnnotationsOffset]);
+ } else {
+ ASSERT_EQ(0u, (*crashpad_info)->simple_annotations.DataSize);
+ ASSERT_EQ(0u, (*crashpad_info)->simple_annotations.Rva);
+ }
+}
+
+TEST(MinidumpCrashpadInfoWriter, Empty) {
+ MinidumpFileWriter minidump_file_writer;
+ MinidumpCrashpadInfoWriter crashpad_info_writer;
+
+ minidump_file_writer.AddStream(&crashpad_info_writer);
+
+ StringFileWriter file_writer;
+ ASSERT_TRUE(minidump_file_writer.WriteEverything(&file_writer));
+
+ const MinidumpCrashpadInfo* crashpad_info;
+
+ ASSERT_NO_FATAL_FAILURE(
+ GetCrashpadInfoStream(file_writer.string(), &crashpad_info, NULL));
Robert Sesek 2014/10/17 21:54:13 nullptr
Mark Mentovai 2014/10/17 22:00:25 Robert Sesek wrote:
+
+ EXPECT_EQ(sizeof(*crashpad_info), crashpad_info->size);
+ EXPECT_EQ(1u, crashpad_info->version);
+ EXPECT_EQ(0u, crashpad_info->simple_annotations.DataSize);
+ EXPECT_EQ(0u, crashpad_info->simple_annotations.Rva);
+}
+
+TEST(MinidumpCrashpadInfoWriter, SimpleAnnotations) {
+ MinidumpFileWriter minidump_file_writer;
+ MinidumpCrashpadInfoWriter crashpad_info_writer;
+
+ minidump_file_writer.AddStream(&crashpad_info_writer);
+
+ MinidumpSimpleStringDictionaryWriter simple_annotations_writer;
+
+ // Set a key and value before adding the simple_annotations_writer to
+ // crashpad_info_writer, and another one after.
+ const char kKey0[] = "k0";
+ const char kValue0[] = "v";
+ const char kKey1[] = "KEY1";
+ const char kValue1[] = "";
+ MinidumpSimpleStringDictionaryEntryWriter entry_0;
+ entry_0.SetKeyValue(kKey0, kValue0);
+ simple_annotations_writer.AddEntry(&entry_0);
+
+ crashpad_info_writer.SetSimpleAnnotations(&simple_annotations_writer);
+
+ MinidumpSimpleStringDictionaryEntryWriter entry_1;
+ entry_1.SetKeyValue(kKey1, kValue1);
+ simple_annotations_writer.AddEntry(&entry_1);
+
+ StringFileWriter file_writer;
+ ASSERT_TRUE(minidump_file_writer.WriteEverything(&file_writer));
+
+ const MinidumpCrashpadInfo* crashpad_info;
+ const MinidumpSimpleStringDictionary* simple_annotations;
+
+ ASSERT_NO_FATAL_FAILURE(GetCrashpadInfoStream(
+ file_writer.string(), &crashpad_info, &simple_annotations));
+
+ EXPECT_EQ(sizeof(*crashpad_info), crashpad_info->size);
+ EXPECT_EQ(1u, crashpad_info->version);
+
+ ASSERT_EQ(2u, simple_annotations->count);
+
+ EXPECT_EQ(
+ kKey1,
+ MinidumpUTF8StringAtRVA(file_writer, simple_annotations->entries[0].key));
+ EXPECT_EQ(kValue1,
+ MinidumpUTF8StringAtRVA(file_writer,
+ simple_annotations->entries[0].value));
+ EXPECT_EQ(
+ kKey0,
+ MinidumpUTF8StringAtRVA(file_writer, simple_annotations->entries[1].key));
+ EXPECT_EQ(kValue0,
+ MinidumpUTF8StringAtRVA(file_writer,
+ simple_annotations->entries[1].value));
+}
+
+} // namespace
+} // namespace test
+} // namespace crashpad

Powered by Google App Engine
This is Rietveld 408576698