// Copyright 2023 The Chromium Authors
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
#include "base/functional/bind.h"
#include "base/test/scoped_feature_list.h"
#include "base/test/test_future.h"
#include "chrome/browser/plus_addresses/plus_address_service_factory.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/profiles/profile_test_util.h"
#include "chrome/browser/signin/identity_manager_factory.h"
#include "chrome/browser/ui/android/plus_addresses/plus_address_creation_controller_android.h"
#include "chrome/test/base/android/android_browser_test.h"
#include "chrome/test/base/chrome_test_utils.h"
#include "components/plus_addresses/fake_plus_address_service.h"
#include "components/plus_addresses/features.h"
#include "components/plus_addresses/plus_address_service.h"
#include "components/plus_addresses/plus_address_types.h"
#include "components/plus_addresses/settings/fake_plus_address_setting_service.h"
#include "content/public/browser/web_contents.h"
#include "content/public/test/browser_test.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace plus_addresses {
// TODO(crbug.com/40276862): Consolidate android/desktop controllers, and
// presumably switch to the `PlatformBrowserTest` pattern.
class PlusAddressCreationViewAndroidBrowserTest : public AndroidBrowserTest {
public:
PlusAddressCreationViewAndroidBrowserTest() = default;
void SetUpOnMainThread() override {
AndroidBrowserTest::SetUpOnMainThread();
PlusAddressServiceFactory::GetInstance()->SetTestingFactoryAndUse(
chrome_test_utils::GetActiveWebContents(this)->GetBrowserContext(),
base::BindRepeating(&PlusAddressCreationViewAndroidBrowserTest::
PlusAddressServiceTestFactory,
base::Unretained(this)));
}
std::unique_ptr<KeyedService> PlusAddressServiceTestFactory(
content::BrowserContext* context) {
return std::make_unique<FakePlusAddressService>(
profile()->GetPrefs(), IdentityManagerFactory::GetForProfile(profile()),
&setting_service_);
}
protected:
Profile* profile() {
auto* web_contents = chrome_test_utils::GetActiveWebContents(this);
return Profile::FromBrowserContext(web_contents->GetBrowserContext());
}
private:
// Ensures that the feature is known to be enabled, such that
// `PlusAddressServiceFactory` doesn't bail early with a null return.
base::test::ScopedFeatureList features_{features::kPlusAddressesEnabled};
FakePlusAddressSettingService setting_service_;
};
IN_PROC_BROWSER_TEST_F(PlusAddressCreationViewAndroidBrowserTest, OfferUi) {
PlusAddressCreationControllerAndroid::CreateForWebContents(
chrome_test_utils::GetActiveWebContents(this));
PlusAddressCreationControllerAndroid* controller =
PlusAddressCreationControllerAndroid::FromWebContents(
chrome_test_utils::GetActiveWebContents(this));
base::test::TestFuture<const std::string&> future;
controller->OfferCreation(
url::Origin::Create(GURL("https://mattwashere.com")),
future.GetCallback());
EXPECT_FALSE(future.IsReady());
controller->OnConfirmed();
EXPECT_TRUE(future.IsReady());
EXPECT_EQ(future.Get(), FakePlusAddressService::kFakePlusAddress);
}
IN_PROC_BROWSER_TEST_F(PlusAddressCreationViewAndroidBrowserTest,
OfferUi_RefreshPlusAddress) {
PlusAddressCreationControllerAndroid::CreateForWebContents(
chrome_test_utils::GetActiveWebContents(this));
PlusAddressCreationControllerAndroid* controller =
PlusAddressCreationControllerAndroid::FromWebContents(
chrome_test_utils::GetActiveWebContents(this));
base::test::TestFuture<const std::string&> future;
controller->OfferCreation(
url::Origin::Create(GURL("https://mattwashere.com")),
future.GetCallback());
EXPECT_FALSE(future.IsReady());
controller->OnRefreshClicked();
EXPECT_FALSE(future.IsReady());
controller->OnConfirmed();
EXPECT_TRUE(future.IsReady());
EXPECT_EQ(future.Get(), FakePlusAddressService::kFakePlusAddress);
}
IN_PROC_BROWSER_TEST_F(PlusAddressCreationViewAndroidBrowserTest,
DoubleOfferUi) {
PlusAddressCreationControllerAndroid::CreateForWebContents(
chrome_test_utils::GetActiveWebContents(this));
PlusAddressCreationControllerAndroid* controller =
PlusAddressCreationControllerAndroid::FromWebContents(
chrome_test_utils::GetActiveWebContents(this));
// First, offer creation like normal.
base::test::TestFuture<const std::string&> future;
controller->OfferCreation(
url::Origin::Create(GURL("https://mattwashere.com")),
future.GetCallback());
// Then, offer creation a second time, without first dismissing the UI.
base::test::TestFuture<const std::string&> second_future;
controller->OfferCreation(
url::Origin::Create(GURL("https://mattwashere.com")),
second_future.GetCallback());
controller->OnConfirmed();
EXPECT_TRUE(future.IsReady());
EXPECT_EQ(future.Get(), FakePlusAddressService::kFakePlusAddress);
EXPECT_FALSE(second_future.IsReady());
}
IN_PROC_BROWSER_TEST_F(PlusAddressCreationViewAndroidBrowserTest, Cancel) {
PlusAddressCreationControllerAndroid::CreateForWebContents(
chrome_test_utils::GetActiveWebContents(this));
PlusAddressCreationControllerAndroid* controller =
PlusAddressCreationControllerAndroid::FromWebContents(
chrome_test_utils::GetActiveWebContents(this));
// First, offer creation.
base::test::TestFuture<const std::string&> future;
controller->OfferCreation(
url::Origin::Create(GURL("https://mattwashere.com")),
future.GetCallback());
// Then, cancel, and ensure that `future.GetCallback()` is not run.
EXPECT_FALSE(future.IsReady());
controller->OnCanceled();
EXPECT_FALSE(future.IsReady());
}
IN_PROC_BROWSER_TEST_F(PlusAddressCreationViewAndroidBrowserTest,
CancelThenShowAgain) {
PlusAddressCreationControllerAndroid::CreateForWebContents(
chrome_test_utils::GetActiveWebContents(this));
PlusAddressCreationControllerAndroid* controller =
PlusAddressCreationControllerAndroid::FromWebContents(
chrome_test_utils::GetActiveWebContents(this));
// First, offer creation.
base::test::TestFuture<const std::string&> future;
controller->OfferCreation(
url::Origin::Create(GURL("https://mattwashere.com")),
future.GetCallback());
// Then, cancel, destroy, and ensure that `future.GetCallback()` is not run.
controller->OnCanceled();
controller->OnDialogDestroyed();
EXPECT_FALSE(future.IsReady());
// After re-showing, confirmation should run `second_future.GetCallback()`.
base::test::TestFuture<const std::string&> second_future;
controller->OfferCreation(
url::Origin::Create(GURL("https://mattwashere.com")),
second_future.GetCallback());
controller->OnConfirmed();
EXPECT_TRUE(second_future.IsReady());
EXPECT_EQ(second_future.Get(), FakePlusAddressService::kFakePlusAddress);
}
// Ensure that closing the web contents with the plus_address creation UI open
// doesn't cause issues, and doesn't incorrectly invoke the autofill callback.
IN_PROC_BROWSER_TEST_F(PlusAddressCreationViewAndroidBrowserTest,
CloseWebContents) {
content::WebContents* active_web_contents =
chrome_test_utils::GetActiveWebContents(this);
PlusAddressCreationControllerAndroid::CreateForWebContents(
active_web_contents);
PlusAddressCreationControllerAndroid* controller =
PlusAddressCreationControllerAndroid::FromWebContents(
active_web_contents);
base::test::TestFuture<const std::string&> future;
// First, offer creation.
controller->OfferCreation(
url::Origin::Create(GURL("https://mattwashere.com")),
future.GetCallback());
EXPECT_FALSE(future.IsReady());
// Next, close the web contents. The view and controller will be destroyed.
active_web_contents->Close();
// Expect no autofill callback.
EXPECT_FALSE(future.IsReady());
}
} // namespace plus_addresses
Codebase Browser
chromium