Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "chrome/browser/extensions/extension_warning_service.h" | 5 #include "extensions/browser/extension_warning_service.h" |
| 6 | 6 |
| 7 #include "chrome/browser/profiles/profile.h" | 7 #include "content/public/test/test_browser_context.h" |
| 8 #include "chrome/browser/ui/global_error/global_error_service.h" | 8 #include "extensions/browser/extensions_test.h" |
| 9 #include "chrome/browser/ui/global_error/global_error_service_factory.h" | |
| 10 #include "chrome/test/base/testing_profile.h" | |
| 11 #include "testing/gmock/include/gmock/gmock.h" | 9 #include "testing/gmock/include/gmock/gmock.h" |
| 12 #include "testing/gtest/include/gtest/gtest.h" | 10 #include "testing/gtest/include/gtest/gtest.h" |
| 13 | 11 |
| 14 namespace extensions { | 12 namespace extensions { |
| 15 | 13 |
| 16 namespace { | 14 namespace { |
| 17 | 15 |
| 18 class TestExtensionWarningService : public ExtensionWarningService { | 16 class TestExtensionWarningService : public ExtensionWarningService { |
| 19 public: | 17 public: |
| 20 explicit TestExtensionWarningService(Profile* profile) | 18 explicit TestExtensionWarningService(content::BrowserContext* browser_context) |
| 21 : ExtensionWarningService(profile) { | 19 : ExtensionWarningService(browser_context) { |
| 22 } | 20 } |
| 23 virtual ~TestExtensionWarningService() {} | 21 virtual ~TestExtensionWarningService() {} |
| 24 | 22 |
| 25 void AddWarning(const ExtensionWarning& warning) { | 23 void AddWarning(const ExtensionWarning& warning) { |
| 26 ExtensionWarningSet warnings; | 24 ExtensionWarningSet warnings; |
| 27 warnings.insert(warning); | 25 warnings.insert(warning); |
| 28 AddWarnings(warnings); | 26 AddWarnings(warnings); |
| 29 } | 27 } |
| 30 }; | 28 }; |
| 31 | 29 |
| 32 class MockObserver : public ExtensionWarningService::Observer { | 30 class MockObserver : public ExtensionWarningService::Observer { |
| 33 public: | 31 public: |
| 34 virtual ~MockObserver() {} | 32 virtual ~MockObserver() {} |
| 35 MOCK_METHOD0(ExtensionWarningsChanged, void()); | 33 MOCK_METHOD0(ExtensionWarningsChanged, void()); |
| 36 }; | 34 }; |
| 37 | 35 |
| 36 class ExtensionWarningServiceTest : public ExtensionsTest { | |
|
James Cook
2014/08/25 20:58:11
You could just say "typedef ExtensionsTest Extensi
Xi Han
2014/08/26 18:21:40
Done.
| |
| 37 public: | |
| 38 ExtensionWarningServiceTest() {} | |
| 39 virtual ~ExtensionWarningServiceTest() {} | |
| 40 private: | |
| 41 DISALLOW_COPY_AND_ASSIGN(ExtensionWarningServiceTest); | |
| 42 }; | |
| 43 | |
| 38 const char* ext1_id = "extension1"; | 44 const char* ext1_id = "extension1"; |
| 39 const char* ext2_id = "extension2"; | 45 const char* ext2_id = "extension2"; |
| 40 const ExtensionWarning::WarningType warning_1 = | 46 const ExtensionWarning::WarningType warning_1 = |
| 41 ExtensionWarning::kNetworkDelay; | 47 ExtensionWarning::kNetworkDelay; |
| 42 const ExtensionWarning::WarningType warning_2 = | 48 const ExtensionWarning::WarningType warning_2 = |
| 43 ExtensionWarning::kNetworkConflict; | 49 ExtensionWarning::kNetworkConflict; |
| 44 | 50 |
| 45 } // namespace | 51 } // namespace |
| 46 | 52 |
| 47 // Check that inserting a warning triggers notifications, whereas inserting | 53 // Check that inserting a warning triggers notifications, whereas inserting |
| 48 // the same warning again is silent. | 54 // the same warning again is silent. |
| 49 TEST(ExtensionWarningServiceTest, SetWarning) { | 55 TEST_F(ExtensionWarningServiceTest, SetWarning) { |
| 50 TestingProfile profile; | 56 content::TestBrowserContext browser_context; |
| 51 TestExtensionWarningService warning_service(&profile); | 57 TestExtensionWarningService warning_service(&browser_context); |
| 52 MockObserver observer; | 58 MockObserver observer; |
| 53 warning_service.AddObserver(&observer); | 59 warning_service.AddObserver(&observer); |
| 54 | 60 |
| 55 // Insert warning for the first time. | 61 // Insert warning for the first time. |
| 56 EXPECT_CALL(observer, ExtensionWarningsChanged()); | 62 EXPECT_CALL(observer, ExtensionWarningsChanged()); |
| 57 warning_service.AddWarning( | 63 warning_service.AddWarning( |
| 58 ExtensionWarning::CreateNetworkDelayWarning(ext1_id)); | 64 ExtensionWarning::CreateNetworkDelayWarning(ext1_id)); |
| 59 testing::Mock::VerifyAndClearExpectations(&warning_service); | 65 testing::Mock::VerifyAndClearExpectations(&warning_service); |
| 60 | 66 |
| 61 // Second insertion of same warning does not trigger anything. | 67 // Second insertion of same warning does not trigger anything. |
| 62 warning_service.AddWarning( | 68 warning_service.AddWarning( |
| 63 ExtensionWarning::CreateNetworkDelayWarning(ext1_id)); | 69 ExtensionWarning::CreateNetworkDelayWarning(ext1_id)); |
| 64 testing::Mock::VerifyAndClearExpectations(&warning_service); | 70 testing::Mock::VerifyAndClearExpectations(&warning_service); |
| 65 | 71 |
| 66 warning_service.RemoveObserver(&observer); | 72 warning_service.RemoveObserver(&observer); |
| 67 } | 73 } |
| 68 | 74 |
| 69 // Check that ClearWarnings deletes exactly the specified warnings and | 75 // Check that ClearWarnings deletes exactly the specified warnings and |
| 70 // triggers notifications where appropriate. | 76 // triggers notifications where appropriate. |
| 71 TEST(ExtensionWarningServiceTest, ClearWarnings) { | 77 TEST_F(ExtensionWarningServiceTest, ClearWarnings) { |
| 72 TestingProfile profile; | 78 content::TestBrowserContext browser_context; |
| 73 TestExtensionWarningService warning_service(&profile); | 79 TestExtensionWarningService warning_service(&browser_context); |
| 74 MockObserver observer; | 80 MockObserver observer; |
| 75 warning_service.AddObserver(&observer); | 81 warning_service.AddObserver(&observer); |
| 76 | 82 |
| 77 // Insert two unique warnings in one batch. | 83 // Insert two unique warnings in one batch. |
| 78 EXPECT_CALL(observer, ExtensionWarningsChanged()); | 84 EXPECT_CALL(observer, ExtensionWarningsChanged()); |
| 79 ExtensionWarningSet warning_set; | 85 ExtensionWarningSet warning_set; |
| 80 warning_set.insert(ExtensionWarning::CreateNetworkDelayWarning(ext1_id)); | 86 warning_set.insert(ExtensionWarning::CreateNetworkDelayWarning(ext1_id)); |
| 81 warning_set.insert(ExtensionWarning::CreateNetworkConflictWarning(ext2_id)); | 87 warning_set.insert(ExtensionWarning::CreateNetworkConflictWarning(ext2_id)); |
| 82 warning_service.AddWarnings(warning_set); | 88 warning_service.AddWarnings(warning_set); |
| 83 testing::Mock::VerifyAndClearExpectations(&warning_service); | 89 testing::Mock::VerifyAndClearExpectations(&warning_service); |
| (...skipping 24 matching lines...) Expand all Loading... | |
| 108 warning_service.GetWarningTypesAffectingExtension(ext1_id); | 114 warning_service.GetWarningTypesAffectingExtension(ext1_id); |
| 109 EXPECT_EQ(0u, existing_warnings.size()); | 115 EXPECT_EQ(0u, existing_warnings.size()); |
| 110 existing_warnings = | 116 existing_warnings = |
| 111 warning_service.GetWarningTypesAffectingExtension(ext2_id); | 117 warning_service.GetWarningTypesAffectingExtension(ext2_id); |
| 112 EXPECT_EQ(0u, existing_warnings.size()); | 118 EXPECT_EQ(0u, existing_warnings.size()); |
| 113 | 119 |
| 114 warning_service.RemoveObserver(&observer); | 120 warning_service.RemoveObserver(&observer); |
| 115 } | 121 } |
| 116 | 122 |
| 117 } // namespace extensions | 123 } // namespace extensions |
| OLD | NEW |