| Index: components/gcm_driver/gcm_driver_desktop_unittest.cc
|
| diff --git a/components/gcm_driver/gcm_driver_desktop_unittest.cc b/components/gcm_driver/gcm_driver_desktop_unittest.cc
|
| index 26e1573d584128157c912f1e597d9eded093952f..77ca103279eed67ea40fdc101ba926ea3ea5bd6c 100644
|
| --- a/components/gcm_driver/gcm_driver_desktop_unittest.cc
|
| +++ b/components/gcm_driver/gcm_driver_desktop_unittest.cc
|
| @@ -326,12 +326,10 @@ TEST_F(GCMDriverTest, SignInAndSignOutOnGCMEnabled) {
|
|
|
| // GCMClient should be started after sign-in.
|
| SignIn(kTestAccountID1);
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
|
|
| // GCMClient should be checked out after sign-out.
|
| SignOut();
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::CHECKED_OUT, GetGCMClient()->status());
|
| }
|
|
|
| @@ -345,12 +343,10 @@ TEST_F(GCMDriverTest, SignInAndSignOutOnGCMDisabled) {
|
|
|
| // GCMClient should not be started after sign-in.
|
| SignIn(kTestAccountID1);
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::UNINITIALIZED, GetGCMClient()->status());
|
|
|
| // Check-out should still be performed after sign-out.
|
| SignOut();
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::CHECKED_OUT, GetGCMClient()->status());
|
| }
|
|
|
| @@ -360,19 +356,16 @@ TEST_F(GCMDriverTest, SignOutAndThenSignIn) {
|
|
|
| // GCMClient should be started after sign-in.
|
| SignIn(kTestAccountID1);
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
|
|
| // GCMClient should be checked out after sign-out.
|
| SignOut();
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::CHECKED_OUT, GetGCMClient()->status());
|
|
|
| // Sign-in with a different account.
|
| SignIn(kTestAccountID2);
|
|
|
| // GCMClient should be started again.
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
| }
|
|
|
| @@ -382,7 +375,6 @@ TEST_F(GCMDriverTest, DisableAndReenableGCM) {
|
| SignIn(kTestAccountID1);
|
|
|
| // GCMClient should be started.
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
|
|
| // Disables the GCM.
|
| @@ -391,7 +383,6 @@ TEST_F(GCMDriverTest, DisableAndReenableGCM) {
|
| PumpUILoop();
|
|
|
| // GCMClient should be stopped.
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STOPPED, GetGCMClient()->status());
|
|
|
| // Enables the GCM.
|
| @@ -400,7 +391,6 @@ TEST_F(GCMDriverTest, DisableAndReenableGCM) {
|
| PumpUILoop();
|
|
|
| // GCMClient should be started.
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
|
|
| // Disables the GCM.
|
| @@ -409,14 +399,12 @@ TEST_F(GCMDriverTest, DisableAndReenableGCM) {
|
| PumpUILoop();
|
|
|
| // GCMClient should be stopped.
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STOPPED, GetGCMClient()->status());
|
|
|
| // Sign out.
|
| SignOut();
|
|
|
| // GCMClient should be checked out.
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::CHECKED_OUT, GetGCMClient()->status());
|
| }
|
|
|
| @@ -425,42 +413,36 @@ TEST_F(GCMDriverTest, StartOrStopGCMOnDemand) {
|
| SignIn(kTestAccountID1);
|
|
|
| // GCMClient is not started.
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::UNINITIALIZED, GetGCMClient()->status());
|
|
|
| // GCMClient is started after an app handler has been added.
|
| driver()->AddAppHandler(kTestAppID1, gcm_app_handler());
|
| PumpIOLoop();
|
| PumpUILoop();
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
|
|
| // Add another app handler.
|
| driver()->AddAppHandler(kTestAppID2, gcm_app_handler());
|
| PumpIOLoop();
|
| PumpUILoop();
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
|
|
| // GCMClient remains active after one app handler is gone.
|
| driver()->RemoveAppHandler(kTestAppID1);
|
| PumpIOLoop();
|
| PumpUILoop();
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
|
|
| // GCMClient should be stopped after the last app handler is gone.
|
| driver()->RemoveAppHandler(kTestAppID2);
|
| PumpIOLoop();
|
| PumpUILoop();
|
| - EXPECT_FALSE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STOPPED, GetGCMClient()->status());
|
|
|
| // GCMClient is restarted after an app handler has been added.
|
| driver()->AddAppHandler(kTestAppID2, gcm_app_handler());
|
| PumpIOLoop();
|
| PumpUILoop();
|
| - EXPECT_TRUE(driver()->IsGCMClientReady());
|
| EXPECT_EQ(FakeGCMClient::STARTED, GetGCMClient()->status());
|
| }
|
|
|
|
|