Commit 023b0f79 authored by Viktor Semeniuk's avatar Viktor Semeniuk Committed by Commit Bot

[iOS][Password Check] Rewriting Passwords test with new details screen

Reland of https://chromium-review.googlesource.com/c/chromium/src/+/2450409
In this CL removed changes to PasswordViewControllerTestCase which
caused flakiness.

This change rewrites Passwords tests with new Password Details screen.
Obsolete tests are deleted. Fixed flakiness for
testSearchAndDeleteAllPasswords.

Bug: 1075494
Bug: 1129441
Change-Id: I36fe982d496f641ce1855ac34c7d816a30e3fcad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2456609Reviewed-by: default avatarGauthier Ambard <gambard@chromium.org>
Reviewed-by: default avatarEugene But <eugenebut@chromium.org>
Commit-Queue: Viktor Semeniuk <vsemeniuk@google.com>
Cr-Commit-Position: refs/heads/master@{#815935}
parent db43c89e
......@@ -233,6 +233,7 @@ source_set("eg2_tests") {
":password_constants",
"//base:base",
"//base/test:test_support",
"//components/password_manager/core/common",
"//components/strings:components_strings_grit",
"//ios/chrome/app/strings:ios_strings_grit",
"//ios/chrome/browser/ui/settings:settings_root_constants",
......
......@@ -9,4 +9,4 @@
#endif
NSString* const kPasswordDetailsViewControllerId =
@"kPasswordDetailsViewControllerId";
@"PasswordDetailsTableViewId";
......@@ -210,6 +210,8 @@ typedef NS_ENUM(NSInteger, ReauthenticationReason) {
item.identifyingIcon = [[UIImage imageNamed:image]
imageWithRenderingMode:UIImageRenderingModeAlwaysTemplate];
item.identifyingIconEnabled = YES;
item.identifyingIconAccessibilityLabel =
l10n_util::GetNSString(IDS_IOS_SETTINGS_PASSWORD_SHOW_BUTTON);
}
return item;
}
......
......@@ -81,6 +81,7 @@ source_set("test_support") {
"//ios/chrome/browser/ui/settings/password",
"//ios/chrome/browser/ui/settings/password:password_ui",
"//ios/chrome/browser/ui/settings/password:test_support",
"//ios/chrome/browser/ui/settings/password/password_details:password_details_ui",
"//ios/chrome/browser/ui/tab_grid",
"//ios/chrome/browser/ui/tabs",
"//ios/chrome/browser/ui/util",
......
......@@ -6,6 +6,7 @@
#include "base/mac/foundation_util.h"
#import "ios/chrome/browser/ui/settings/password/legacy_password_details_table_view_controller+testing.h"
#import "ios/chrome/browser/ui/settings/password/password_details/password_details_table_view_controller.h"
#import "ios/chrome/browser/ui/settings/password/passwords_table_view_controller.h"
#import "ios/chrome/browser/ui/settings/settings_navigation_controller.h"
#import "ios/chrome/browser/ui/util/top_view_controller.h"
......@@ -53,12 +54,11 @@ MockReauthenticationModule* SetUpAndReturnMockReauthenticationModule() {
SettingsNavigationController* settings_navigation_controller =
base::mac::ObjCCastStrict<SettingsNavigationController>(
top_view_controller::TopPresentedViewController());
LegacyPasswordDetailsTableViewController*
password_details_table_view_controller =
base::mac::ObjCCastStrict<LegacyPasswordDetailsTableViewController>(
settings_navigation_controller.topViewController);
[password_details_table_view_controller
setReauthenticationModule:mock_reauthentication_module];
PasswordDetailsTableViewController* password_details_table_view_controller =
base::mac::ObjCCastStrict<PasswordDetailsTableViewController>(
settings_navigation_controller.topViewController);
password_details_table_view_controller.reauthModule =
mock_reauthentication_module;
return mock_reauthentication_module;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment