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

Unified Diff: trunk/src/chrome/browser/component_updater/test/component_patcher_unittest.cc

Issue 17551004: Revert 207805 "Differential updates for components. We are addin..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 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
Index: trunk/src/chrome/browser/component_updater/test/component_patcher_unittest.cc
===================================================================
--- trunk/src/chrome/browser/component_updater/test/component_patcher_unittest.cc (revision 207822)
+++ trunk/src/chrome/browser/component_updater/test/component_patcher_unittest.cc (working copy)
@@ -1,200 +0,0 @@
-// Copyright 2013 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 "base/compiler_specific.h"
-#include "base/file_util.h"
-#include "base/files/file_path.h"
-#include "base/files/scoped_temp_dir.h"
-#include "base/path_service.h"
-#include "base/values.h"
-#include "chrome/browser/component_updater/component_patcher.h"
-#include "chrome/browser/component_updater/component_patcher_operation.h"
-#include "chrome/browser/component_updater/component_updater_service.h"
-#include "chrome/browser/component_updater/test/component_patcher_mock.h"
-#include "chrome/browser/component_updater/test/test_installer.h"
-#include "chrome/common/chrome_paths.h"
-#include "courgette/courgette.h"
-#include "courgette/third_party/bsdiff.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-const char binary_output_hash[] =
- "599aba6d15a7da390621ef1bacb66601ed6aed04dadc1f9b445dcfe31296142a";
-
-// These constants are duplicated from chrome/installer/util/util_constants.h,
-// to avoid introducing a dependency from the unit tests to the installer.
-const int kCourgetteErrorOffset = 300;
-const int kBsdiffErrorOffset = 600;
-
-base::FilePath test_file(const char* file);
-
-class ComponentPatcherOperationTest : public testing::Test {
- public:
- ComponentPatcherOperationTest();
-
- protected:
- base::ScopedTempDir input_dir_;
- base::ScopedTempDir installed_dir_;
- base::ScopedTempDir unpack_dir_;
- scoped_ptr<MockComponentPatcher> patcher_;
- scoped_ptr<ReadOnlyTestInstaller> installer_;
-};
-
-base::FilePath test_file(const char* file) {
- base::FilePath path;
- PathService::Get(chrome::DIR_TEST_DATA, &path);
- return path.AppendASCII("components").AppendASCII(file);
-}
-
-ComponentPatcherOperationTest::ComponentPatcherOperationTest() {
- EXPECT_TRUE(unpack_dir_.CreateUniqueTempDir());
- EXPECT_TRUE(input_dir_.CreateUniqueTempDir());
- EXPECT_TRUE(installed_dir_.CreateUniqueTempDir());
- patcher_.reset(new MockComponentPatcher());
- installer_.reset(new ReadOnlyTestInstaller(installed_dir_.path()));
-}
-
-ComponentUnpacker::Error MockComponentPatcher::Patch(
- PatchType patch_type,
- const base::FilePath& input_file,
- const base::FilePath& patch_file,
- const base::FilePath& output_file,
- int* error) {
- *error = 0;
- int exit_code;
- if (patch_type == kPatchTypeCourgette) {
- exit_code = courgette::ApplyEnsemblePatch(input_file.value().c_str(),
- patch_file.value().c_str(),
- output_file.value().c_str());
- if (exit_code == courgette::C_OK)
- return ComponentUnpacker::kNone;
- *error = exit_code + kCourgetteErrorOffset;
- } else if (patch_type == kPatchTypeBsdiff) {
- exit_code = courgette::ApplyBinaryPatch(input_file,
- patch_file,
- output_file);
- if (exit_code == courgette::OK)
- return ComponentUnpacker::kNone;
- *error = exit_code + kBsdiffErrorOffset;
- }
- return ComponentUnpacker::kDeltaOperationFailure;
-}
-
-// Verify that a 'create' delta update operation works correctly.
-TEST_F(ComponentPatcherOperationTest, CheckCreateOperation) {
- EXPECT_TRUE(file_util::CopyFile(
- test_file("binary_output.bin"),
- input_dir_.path().Append(FILE_PATH_LITERAL("binary_output.bin"))));
-
- scoped_ptr<base::DictionaryValue> command_args(new base::DictionaryValue());
- command_args->SetString("output", "output.bin");
- command_args->SetString("sha256", binary_output_hash);
- command_args->SetString("op", "create");
- command_args->SetString("patch", "binary_output.bin");
-
- int error = 0;
- scoped_ptr<DeltaUpdateOp> op(new DeltaUpdateOpCreate());
- ComponentUnpacker::Error result = op->Run(command_args.get(),
- input_dir_.path(),
- unpack_dir_.path(),
- patcher_.get(),
- NULL,
- &error);
-
- EXPECT_EQ(ComponentUnpacker::kNone, result);
- EXPECT_EQ(0, error);
- EXPECT_TRUE(file_util::ContentsEqual(
- unpack_dir_.path().Append(FILE_PATH_LITERAL("output.bin")),
- test_file("binary_output.bin")));
-}
-
-// Verify that a 'copy' delta update operation works correctly.
-TEST_F(ComponentPatcherOperationTest, CheckCopyOperation) {
- EXPECT_TRUE(file_util::CopyFile(
- test_file("binary_output.bin"),
- installed_dir_.path().Append(FILE_PATH_LITERAL("binary_output.bin"))));
-
- scoped_ptr<base::DictionaryValue> command_args(new base::DictionaryValue());
- command_args->SetString("output", "output.bin");
- command_args->SetString("sha256", binary_output_hash);
- command_args->SetString("op", "copy");
- command_args->SetString("input", "binary_output.bin");
-
- int error = 0;
- scoped_ptr<DeltaUpdateOp> op(new DeltaUpdateOpCopy());
- ComponentUnpacker::Error result = op->Run(command_args.get(),
- input_dir_.path(),
- unpack_dir_.path(),
- patcher_.get(),
- installer_.get(),
- &error);
- EXPECT_EQ(ComponentUnpacker::kNone, result);
- EXPECT_EQ(0, error);
- EXPECT_TRUE(file_util::ContentsEqual(
- unpack_dir_.path().Append(FILE_PATH_LITERAL("output.bin")),
- test_file("binary_output.bin")));
-}
-
-// Verify that a 'courgette' delta update operation works correctly.
-TEST_F(ComponentPatcherOperationTest, CheckCourgetteOperation) {
- EXPECT_TRUE(file_util::CopyFile(
- test_file("binary_input.bin"),
- installed_dir_.path().Append(FILE_PATH_LITERAL("binary_input.bin"))));
- EXPECT_TRUE(file_util::CopyFile(
- test_file("binary_courgette_patch.bin"),
- input_dir_.path().Append(
- FILE_PATH_LITERAL("binary_courgette_patch.bin"))));
-
- scoped_ptr<base::DictionaryValue> command_args(new base::DictionaryValue());
- command_args->SetString("output", "output.bin");
- command_args->SetString("sha256", binary_output_hash);
- command_args->SetString("op", "courgette");
- command_args->SetString("input", "binary_input.bin");
- command_args->SetString("patch", "binary_courgette_patch.bin");
-
- int error = 0;
- scoped_ptr<DeltaUpdateOp> op(new DeltaUpdateOpPatchCourgette());
- ComponentUnpacker::Error result = op->Run(command_args.get(),
- input_dir_.path(),
- unpack_dir_.path(),
- patcher_.get(),
- installer_.get(),
- &error);
- EXPECT_EQ(ComponentUnpacker::kNone, result);
- EXPECT_EQ(0, error);
- EXPECT_TRUE(file_util::ContentsEqual(
- unpack_dir_.path().Append(FILE_PATH_LITERAL("output.bin")),
- test_file("binary_output.bin")));
-}
-
-// Verify that a 'bsdiff' delta update operation works correctly.
-TEST_F(ComponentPatcherOperationTest, CheckBsdiffOperation) {
- EXPECT_TRUE(file_util::CopyFile(
- test_file("binary_input.bin"),
- installed_dir_.path().Append(FILE_PATH_LITERAL("binary_input.bin"))));
- EXPECT_TRUE(file_util::CopyFile(
- test_file("binary_bsdiff_patch.bin"),
- input_dir_.path().Append(FILE_PATH_LITERAL("binary_bsdiff_patch.bin"))));
-
- scoped_ptr<base::DictionaryValue> command_args(new base::DictionaryValue());
- command_args->SetString("output", "output.bin");
- command_args->SetString("sha256", binary_output_hash);
- command_args->SetString("op", "courgette");
- command_args->SetString("input", "binary_input.bin");
- command_args->SetString("patch", "binary_bsdiff_patch.bin");
-
- int error = 0;
- scoped_ptr<DeltaUpdateOp> op(new DeltaUpdateOpPatchBsdiff());
- ComponentUnpacker::Error result = op->Run(command_args.get(),
- input_dir_.path(),
- unpack_dir_.path(),
- patcher_.get(),
- installer_.get(),
- &error);
- EXPECT_EQ(ComponentUnpacker::kNone, result);
- EXPECT_EQ(0, error);
- EXPECT_TRUE(file_util::ContentsEqual(
- unpack_dir_.path().Append(FILE_PATH_LITERAL("output.bin")),
- test_file("binary_output.bin")));
-}
-

Powered by Google App Engine
This is Rietveld 408576698