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

Unified Diff: courgette/program_detector_unittest.cc

Issue 2055343002: Courgette: Add static method QuickDetect() to optimize program detection. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Propagate uint8_t* to unittests and Nits Created 4 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
« courgette/program_detector.cc ('K') | « courgette/program_detector.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: courgette/program_detector_unittest.cc
diff --git a/courgette/program_detector_unittest.cc b/courgette/program_detector_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d78f72ecf4a1c6d42a36b845ef0deef41f080a0f
--- /dev/null
+++ b/courgette/program_detector_unittest.cc
@@ -0,0 +1,80 @@
+// Copyright 2016 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/program_detector.h"
+
+#include <string>
+
+#include "courgette/base_test_unittest.h"
+#include "courgette/courgette.h"
+#include "courgette/disassembler.h"
+#include "courgette/disassembler_elf_32_arm.h"
+#include "courgette/disassembler_elf_32_x86.h"
+#include "courgette/disassembler_win32_x64.h"
+#include "courgette/disassembler_win32_x86.h"
+
+namespace courgette {
+
+namespace {
+
+class ProgramDetectorTest : public BaseTest {
+ public:
+ void TestQuickDetect(const std::string& test_data,
+ ExecutableType expected_type) const;
+ void TestDetectDisassembler(const std::string& test_data,
+ ExecutableType expected_type) const;
+};
+
+void ProgramDetectorTest::TestQuickDetect(const std::string& test_data,
+ ExecutableType expected_type) const {
+ // QuickDetect() should return true only for the |expected_type|.
+ EXPECT_EQ(expected_type == EXE_WIN_32_X86,
+ DisassemblerWin32X86::QuickDetect(
+ reinterpret_cast<const uint8_t*>(test_data.data()),
+ test_data.size()));
+ EXPECT_EQ(expected_type == EXE_WIN_32_X64,
+ DisassemblerWin32X64::QuickDetect(
+ reinterpret_cast<const uint8_t*>(test_data.data()),
+ test_data.size()));
+ EXPECT_EQ(expected_type == EXE_ELF_32_X86,
+ DisassemblerElf32X86::QuickDetect(
+ reinterpret_cast<const uint8_t*>(test_data.data()),
+ test_data.size()));
+ EXPECT_EQ(expected_type == EXE_ELF_32_ARM,
+ DisassemblerElf32ARM::QuickDetect(
+ reinterpret_cast<const uint8_t*>(test_data.data()),
+ test_data.size()));
+}
+
+void ProgramDetectorTest::TestDetectDisassembler(
+ const std::string& test_data,
+ ExecutableType expected_type) const {
+ ExecutableType detected_type = EXE_UNKNOWN;
+ size_t detected_length = 0;
+ DetectExecutableType(test_data.data(), test_data.size(), &detected_type,
+ &detected_length);
+ EXPECT_EQ(expected_type, detected_type);
+ EXPECT_EQ(test_data.size(), detected_length);
+}
+
+TEST_F(ProgramDetectorTest, All) {
+ std::string win32_x86 = FileContents("setup1.exe");
+ std::string win32_x64 = FileContents("chrome64_1.exe");
+ std::string elf_32 = FileContents("elf-32-1");
+ std::string elf_arm = FileContents("elf-armv7");
+
+ TestQuickDetect(win32_x86, EXE_WIN_32_X86);
+ TestQuickDetect(win32_x64, EXE_WIN_32_X64);
+ TestQuickDetect(elf_32, EXE_ELF_32_X86);
+ TestQuickDetect(elf_arm, EXE_ELF_32_ARM);
+
+ TestDetectDisassembler(win32_x86, EXE_WIN_32_X86);
+ TestDetectDisassembler(win32_x64, EXE_WIN_32_X64);
+ TestDetectDisassembler(elf_32, EXE_ELF_32_X86);
+ TestDetectDisassembler(elf_arm, EXE_ELF_32_ARM);
+}
+
+} // namespace
+
+} // namespace courgette
« courgette/program_detector.cc ('K') | « courgette/program_detector.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698