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

Unified Diff: blimp/helium/lww_register_unittest.cc

Issue 2602103002: Delete blimp/helium and remove references to it from dependent targets (Closed)
Patch Set: . Created 4 years 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 | « blimp/helium/lww_register.h ('k') | blimp/helium/mock_objects.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/helium/lww_register_unittest.cc
diff --git a/blimp/helium/lww_register_unittest.cc b/blimp/helium/lww_register_unittest.cc
deleted file mode 100644
index 8629a42fd7aee7da82633f873359d0a085c3973c..0000000000000000000000000000000000000000
--- a/blimp/helium/lww_register_unittest.cc
+++ /dev/null
@@ -1,158 +0,0 @@
-// 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 "blimp/helium/lww_register.h"
-
-#include <memory>
-#include <string>
-
-#include "base/bind.h"
-#include "base/macros.h"
-#include "base/memory/ptr_util.h"
-#include "blimp/helium/helium_test.h"
-#include "blimp/helium/revision_generator.h"
-#include "blimp/helium/version_vector.h"
-#include "testing/gmock/include/gmock/gmock.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "third_party/protobuf/src/google/protobuf/io/zero_copy_stream_impl_lite.h"
-
-namespace blimp {
-namespace helium {
-namespace {
-
-class LwwRegisterTest : public HeliumTest {
- public:
- LwwRegisterTest() {}
- ~LwwRegisterTest() override = default;
-
- MOCK_METHOD0(OnEngineCallbackCalled, void());
- MOCK_METHOD0(OnClientCallbackCalled, void());
-
- protected:
- void Initialize(Peer bias) {
- client_ = base::MakeUnique<LwwRegister<int>>(bias, Peer::CLIENT);
- engine_ = base::MakeUnique<LwwRegister<int>>(bias, Peer::ENGINE);
-
- client_->SetLocalUpdateCallback(base::Bind(
- &LwwRegisterTest::OnClientCallbackCalled, base::Unretained(this)));
- engine_->SetLocalUpdateCallback(base::Bind(
- &LwwRegisterTest::OnEngineCallbackCalled, base::Unretained(this)));
- }
-
- // Takes a changeset from |from_lww_register| and applies it to
- // |to_lww_register|.
- void Sync(const LwwRegister<int>& from_lww_register,
- LwwRegister<int>* to_lww_register);
-
- std::unique_ptr<LwwRegister<int>> client_;
- std::unique_ptr<LwwRegister<int>> engine_;
-
- private:
- DISALLOW_COPY_AND_ASSIGN(LwwRegisterTest);
-};
-
-void LwwRegisterTest::Sync(const LwwRegister<int>& from_lww_register,
- LwwRegister<int>* to_lww_register) {
- auto changeset =
- from_lww_register.CreateChangeset(from_lww_register.GetRevision());
- ASSERT_TRUE(to_lww_register->ValidateChangeset(*changeset));
- to_lww_register->ApplyChangeset(*changeset);
-}
-
-TEST_F(LwwRegisterTest, SetIncrementsLocalVersion) {
- Initialize(Peer::CLIENT);
-
- EXPECT_CALL(*this, OnClientCallbackCalled()).Times(1);
- EXPECT_CALL(*this, OnEngineCallbackCalled()).Times(0);
-
- Revision earlier_version = client_->GetRevision();
- client_->Set(42);
- Revision current_version = client_->GetRevision();
-
- EXPECT_EQ(42, client_->Get());
- EXPECT_LT(earlier_version, current_version);
-}
-
-TEST_F(LwwRegisterTest, ApplyLaterChangeset) {
- Initialize(Peer::CLIENT);
-
- EXPECT_CALL(*this, OnClientCallbackCalled()).Times(1);
- EXPECT_CALL(*this, OnEngineCallbackCalled()).Times(0);
-
- client_->Set(123);
- Sync(*client_, engine_.get());
-
- EXPECT_EQ(123, engine_->Get());
-}
-
-TEST_F(LwwRegisterTest, ApplyEarlierChangeset) {
- Initialize(Peer::CLIENT);
-
- EXPECT_CALL(*this, OnClientCallbackCalled()).Times(1);
- EXPECT_CALL(*this, OnEngineCallbackCalled()).Times(1);
-
- client_->Set(123);
- Sync(*client_, engine_.get());
-
- engine_->Set(456);
- Sync(*client_, engine_.get());
-
- EXPECT_EQ(456, engine_->Get());
-}
-
-TEST_F(LwwRegisterTest, ClientApplyChangesetConflictClientWins) {
- Initialize(Peer::CLIENT);
-
- EXPECT_CALL(*this, OnClientCallbackCalled()).Times(1);
- EXPECT_CALL(*this, OnEngineCallbackCalled()).Times(1);
-
- client_->Set(123);
- engine_->Set(456);
- Sync(*engine_, client_.get());
-
- EXPECT_EQ(123, client_->Get());
-}
-
-TEST_F(LwwRegisterTest, EngineApplyChangesetConflictClientWins) {
- Initialize(Peer::CLIENT);
-
- EXPECT_CALL(*this, OnClientCallbackCalled()).Times(1);
- EXPECT_CALL(*this, OnEngineCallbackCalled()).Times(1);
-
- client_->Set(123);
- engine_->Set(456);
- Sync(*client_, engine_.get());
-
- EXPECT_EQ(123, engine_->Get());
-}
-
-TEST_F(LwwRegisterTest, ClientApplyChangesetConflictEngineWins) {
- Initialize(Peer::ENGINE);
-
- EXPECT_CALL(*this, OnClientCallbackCalled()).Times(1);
- EXPECT_CALL(*this, OnEngineCallbackCalled()).Times(1);
-
- client_->Set(123);
- engine_->Set(456);
- Sync(*engine_, client_.get());
-
- EXPECT_EQ(456, client_->Get());
-}
-
-TEST_F(LwwRegisterTest, EngineApplyChangesetConflictEngineWins) {
- Initialize(Peer::ENGINE);
-
- EXPECT_CALL(*this, OnClientCallbackCalled()).Times(1);
- EXPECT_CALL(*this, OnEngineCallbackCalled()).Times(1);
-
- client_->Set(123);
- engine_->Set(456);
- Sync(*client_, engine_.get());
-
- EXPECT_EQ(456, engine_->Get());
-}
-
-} // namespace
-} // namespace helium
-} // namespace blimp
« no previous file with comments | « blimp/helium/lww_register.h ('k') | blimp/helium/mock_objects.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698