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

Unified Diff: content/browser/payments/payment_app_context_impl_unittest.cc

Issue 2609103002: PaymentApp: Add PaymentAppProvider class. (Closed)
Patch Set: Remove override keyword Created 3 years, 11 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: content/browser/payments/payment_app_context_impl_unittest.cc
diff --git a/content/browser/payments/payment_app_context_impl_unittest.cc b/content/browser/payments/payment_app_context_impl_unittest.cc
deleted file mode 100644
index 0a765089f0772e5de8f38120ea1253e11b32476a..0000000000000000000000000000000000000000
--- a/content/browser/payments/payment_app_context_impl_unittest.cc
+++ /dev/null
@@ -1,104 +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 <cstddef>
-#include <string>
-#include <utility>
-
-#include "base/macros.h"
-#include "base/run_loop.h"
-#include "components/payments/payment_app.mojom.h"
-#include "content/browser/payments/payment_app_content_unittest_base.h"
-#include "content/browser/payments/payment_app_context_impl.h"
-#include "content/public/browser/payment_app_context.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "url/gurl.h"
-
-using payments::mojom::PaymentAppManifestError;
-using payments::mojom::PaymentAppManifestPtr;
-
-namespace content {
-
-class PaymentAppManager;
-
-namespace {
-
-void SetManifestCallback(bool* called,
- PaymentAppManifestError* out_error,
- PaymentAppManifestError error) {
- *called = true;
- *out_error = error;
-}
-
-void GetAllManifestsCallback(bool* called,
- PaymentAppContext::Manifests* out_manifests,
- PaymentAppContext::Manifests manifests) {
- *called = true;
- *out_manifests = std::move(manifests);
-}
-
-} // namespace
-
-class PaymentAppContextTest : public PaymentAppContentUnitTestBase {
- public:
- PaymentAppContextTest() {}
- ~PaymentAppContextTest() override {}
-
- void GetAllManifests(PaymentAppContext::GetAllManifestsCallback callback) {
- payment_app_context()->GetAllManifests(callback);
- base::RunLoop().RunUntilIdle();
- }
-
- void CreatePaymentApp(const GURL& scope_url, const GURL& sw_script_url) {
- PaymentAppManager* manager =
- CreatePaymentAppManager(scope_url, sw_script_url);
-
- PaymentAppManifestError error =
- PaymentAppManifestError::MANIFEST_STORAGE_OPERATION_FAILED;
- bool called = false;
- SetManifest(manager, CreatePaymentAppManifestForTest(scope_url.spec()),
- base::Bind(&SetManifestCallback, &called, &error));
- ASSERT_TRUE(called);
-
- ASSERT_EQ(PaymentAppManifestError::NONE, error);
- }
-
- private:
- DISALLOW_COPY_AND_ASSIGN(PaymentAppContextTest);
-};
-
-TEST_F(PaymentAppContextTest, Test) {
- static const struct {
- const char* scopeUrl;
- const char* scriptUrl;
- } kPaymentAppInfo[] = {
- {"https://example.com/a", "https://example.com/a/script.js"},
- {"https://example.com/b", "https://example.com/b/script.js"},
- {"https://example.com/c", "https://example.com/c/script.js"}};
-
- for (size_t i = 0; i < arraysize(kPaymentAppInfo); i++) {
- CreatePaymentApp(GURL(kPaymentAppInfo[i].scopeUrl),
- GURL(kPaymentAppInfo[i].scriptUrl));
- }
-
- PaymentAppContext::Manifests manifests;
- bool called = false;
- GetAllManifests(base::Bind(&GetAllManifestsCallback, &called, &manifests));
- ASSERT_TRUE(called);
-
- ASSERT_EQ(3U, manifests.size());
- size_t i = 0;
- for (const auto& manifest : manifests) {
- EXPECT_EQ("payment-app-icon", manifest.second->icon.value());
- EXPECT_EQ(kPaymentAppInfo[i++].scopeUrl, manifest.second->name);
- ASSERT_EQ(1U, manifest.second->options.size());
- EXPECT_EQ("payment-app-icon", manifest.second->options[0]->icon.value());
- EXPECT_EQ("Visa ****", manifest.second->options[0]->name);
- EXPECT_EQ("payment-app-id", manifest.second->options[0]->id);
- ASSERT_EQ(1U, manifest.second->options[0]->enabled_methods.size());
- EXPECT_EQ("visa", manifest.second->options[0]->enabled_methods[0]);
- }
-}
-
-} // namespace content
« no previous file with comments | « content/browser/payments/payment_app_context_impl.cc ('k') | content/browser/payments/payment_app_provider_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698