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

Unified Diff: courgette/disassembler_win32_x86_unittest.cc

Issue 8166013: Further refactoring, move ImageInfo into Disassembler/DisassemblerWin32X86. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 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: courgette/disassembler_win32_x86_unittest.cc
diff --git a/courgette/disassembler_win32_x86_unittest.cc b/courgette/disassembler_win32_x86_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8c95eba1e730acceceaba6455883e92dbde519bf
--- /dev/null
+++ b/courgette/disassembler_win32_x86_unittest.cc
@@ -0,0 +1,117 @@
+// Copyright (c) 2011 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 "courgette/disassembler_win32_x86.h"
+
+#include <string>
+
+#include "base/path_service.h"
+#include "base/file_util.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/string_util.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+class DisassemblerWin32X86Test : public testing::Test {
+ public:
+
+ void TestExe() const;
+ void TestResourceDll() const;
+
+ private:
+ void SetUp() {
+ PathService::Get(base::DIR_SOURCE_ROOT, &test_dir_);
+ test_dir_ = test_dir_.AppendASCII("courgette");
+ test_dir_ = test_dir_.AppendASCII("testdata");
+ }
+
+ void TearDown() {
+ }
+
+ void ExpectExecutable(courgette::DisassemblerWin32X86* disassembler) const;
+
+ std::string FileContents(const char* file_name) const;
+
+ FilePath test_dir_;
+};
+
+// Reads a test file into a string.
+std::string DisassemblerWin32X86Test::FileContents(
+ const char* file_name) const {
+
+ FilePath file_path = test_dir_;
+ file_path = file_path.AppendASCII(file_name);
+ std::string file_bytes;
+ if (!file_util::ReadFileToString(file_path, &file_bytes)) {
+ EXPECT_TRUE(!"Could not read test data");
+ }
+ return file_bytes;
+}
+
+void DisassemblerWin32X86Test::ExpectExecutable(
+ courgette::DisassemblerWin32X86* disassembler) const {
+
+ EXPECT_TRUE(disassembler->ok());
+ EXPECT_TRUE(disassembler->has_text_section());
+}
+
+void DisassemblerWin32X86Test::TestExe() const {
+ std::string file1 = FileContents("setup1.exe");
+
+ scoped_ptr<courgette::DisassemblerWin32X86> disassembler(
+ new courgette::DisassemblerWin32X86(file1.c_str(), file1.length()));
+
+ bool can_parse_header = disassembler->ParseHeader();
+ EXPECT_TRUE(can_parse_header);
+
+ // The executable is the whole file, not 'embedded' with the file
+ EXPECT_EQ(file1.length(), disassembler->length());
+
+ ExpectExecutable(disassembler.get());
+ EXPECT_EQ(449536U, disassembler->size_of_code());
+ EXPECT_EQ(courgette::DisassemblerWin32X86::SectionName(
+ disassembler->RVAToSection(0x00401234 - 0x00400000)),
+ std::string(".text"));
+
+ EXPECT_EQ(0, disassembler->RVAToFileOffset(0));
+ EXPECT_EQ(1024, disassembler->RVAToFileOffset(4096));
+ EXPECT_EQ(46928, disassembler->RVAToFileOffset(50000));
+
+ std::vector<courgette::RVA> relocs;
+ bool can_parse_relocs = disassembler->ParseRelocs(&relocs);
+ EXPECT_TRUE(can_parse_relocs);
+
+ const uint8* offset_p = disassembler->OffsetToPointer(0);
+ EXPECT_EQ(reinterpret_cast<const void*>(file1.c_str()),
+ reinterpret_cast<const void*>(offset_p));
+ EXPECT_EQ('M', offset_p[0]);
+ EXPECT_EQ('Z', offset_p[1]);
+
+ const uint8* rva_p = disassembler->RVAToPointer(0);
+ EXPECT_EQ(reinterpret_cast<const void*>(file1.c_str()),
+ reinterpret_cast<const void*>(rva_p));
+ EXPECT_EQ('M', rva_p[0]);
+ EXPECT_EQ('Z', rva_p[1]);
+}
+
+void DisassemblerWin32X86Test::TestResourceDll() const {
+ std::string file1 = FileContents("en-US.dll");
+
+ scoped_ptr<courgette::DisassemblerWin32X86> disassembler(
+ new courgette::DisassemblerWin32X86(file1.c_str(), file1.length()));
+
+ bool can_parse_header = disassembler->ParseHeader();
+ EXPECT_TRUE(can_parse_header);
+
+ // The executable is the whole file, not 'embedded' with the file
+ EXPECT_EQ(file1.length(), disassembler->length());
+
+ EXPECT_TRUE(disassembler->ok());
+ EXPECT_FALSE(disassembler->has_text_section());
+ EXPECT_EQ(0U, disassembler->size_of_code());
+}
+
+TEST_F(DisassemblerWin32X86Test, All) {
+ TestExe();
+ TestResourceDll();
+}

Powered by Google App Engine
This is Rietveld 408576698