Commit 468cff75 authored by Jérôme Lebel's avatar Jérôme Lebel Committed by Commit Bot

[iOS] Adding kGoogleServicesSettingsViewIdentifier

Adding kGoogleServicesSettingsViewIdentifier const variable for EarlGrey
tests.

Change-Id: Iea324479412bd7d5b3fbaf2df4dd5f7a719e616f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1836504
Commit-Queue: Jérôme Lebel <jlebel@chromium.org>
Reviewed-by: default avatarSylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#703651}
parent 460f2121
...@@ -240,6 +240,7 @@ source_set("eg_tests") { ...@@ -240,6 +240,7 @@ source_set("eg_tests") {
"//ios/chrome/browser/ui/authentication/cells", "//ios/chrome/browser/ui/authentication/cells",
"//ios/chrome/browser/ui/popup_menu:constants", "//ios/chrome/browser/ui/popup_menu:constants",
"//ios/chrome/browser/ui/settings", "//ios/chrome/browser/ui/settings",
"//ios/chrome/browser/ui/settings/google_services",
"//ios/chrome/browser/ui/toolbar/buttons", "//ios/chrome/browser/ui/toolbar/buttons",
"//ios/chrome/browser/ui/toolbar/public", "//ios/chrome/browser/ui/toolbar/public",
"//ios/chrome/browser/ui/util", "//ios/chrome/browser/ui/util",
......
...@@ -18,6 +18,7 @@ ...@@ -18,6 +18,7 @@
#import "ios/chrome/browser/ui/authentication/cells/signin_promo_view.h" #import "ios/chrome/browser/ui/authentication/cells/signin_promo_view.h"
#import "ios/chrome/browser/ui/authentication/signin_earl_grey_ui.h" #import "ios/chrome/browser/ui/authentication/signin_earl_grey_ui.h"
#import "ios/chrome/browser/ui/authentication/signin_earlgrey_utils.h" #import "ios/chrome/browser/ui/authentication/signin_earlgrey_utils.h"
#import "ios/chrome/browser/ui/settings/google_services/google_services_settings_view_controller.h"
#include "ios/chrome/browser/ui/util/ui_util.h" #include "ios/chrome/browser/ui/util/ui_util.h"
#include "ios/chrome/grit/ios_strings.h" #include "ios/chrome/grit/ios_strings.h"
#import "ios/chrome/test/app/chrome_test_util.h" #import "ios/chrome/test/app/chrome_test_util.h"
...@@ -343,7 +344,7 @@ void SignOut() { ...@@ -343,7 +344,7 @@ void SignOut() {
YES)] YES)]
usingSearchAction:grey_scrollInDirection(kGREYDirectionDown, 200) usingSearchAction:grey_scrollInDirection(kGREYDirectionDown, 200)
onElementWithMatcher:grey_accessibilityID( onElementWithMatcher:grey_accessibilityID(
@"google_services_settings_view_controller")] kGoogleServicesSettingsViewIdentifier)]
performAction:chrome_test_util::TurnSettingsSwitchOn(NO)]; performAction:chrome_test_util::TurnSettingsSwitchOn(NO)];
} else { } else {
......
...@@ -11,6 +11,7 @@ ...@@ -11,6 +11,7 @@
#import "ios/chrome/browser/tabs/tab_model.h" #import "ios/chrome/browser/tabs/tab_model.h"
#import "ios/chrome/browser/ui/authentication/signin_earl_grey_ui.h" #import "ios/chrome/browser/ui/authentication/signin_earl_grey_ui.h"
#import "ios/chrome/browser/ui/authentication/signin_earlgrey_utils.h" #import "ios/chrome/browser/ui/authentication/signin_earlgrey_utils.h"
#import "ios/chrome/browser/ui/settings/google_services/google_services_settings_view_controller.h"
#import "ios/chrome/browser/ui/settings/google_services/manage_sync_settings_table_view_controller.h" #import "ios/chrome/browser/ui/settings/google_services/manage_sync_settings_table_view_controller.h"
#import "ios/chrome/browser/web_state_list/web_state_list.h" #import "ios/chrome/browser/web_state_list/web_state_list.h"
#include "ios/chrome/grit/ios_chromium_strings.h" #include "ios/chrome/grit/ios_chromium_strings.h"
...@@ -137,7 +138,7 @@ TabModel* GetNormalTabModel() { ...@@ -137,7 +138,7 @@ TabModel* GetNormalTabModel() {
[ChromeEarlGreyUI openSettingsMenu]; [ChromeEarlGreyUI openSettingsMenu];
[ChromeEarlGreyUI tapSettingsMenuButton:GoogleServicesSettingsButton()]; [ChromeEarlGreyUI tapSettingsMenuButton:GoogleServicesSettingsButton()];
self.scrollViewMatcher = self.scrollViewMatcher =
grey_accessibilityID(@"google_services_settings_view_controller"); grey_accessibilityID(kGoogleServicesSettingsViewIdentifier);
[[EarlGrey selectElementWithMatcher:self.scrollViewMatcher] [[EarlGrey selectElementWithMatcher:self.scrollViewMatcher]
assertWithMatcher:grey_notNil()]; assertWithMatcher:grey_notNil()];
} }
......
...@@ -13,6 +13,8 @@ ...@@ -13,6 +13,8 @@
@protocol GoogleServicesSettingsServiceDelegate; @protocol GoogleServicesSettingsServiceDelegate;
@protocol GoogleServicesSettingsViewControllerModelDelegate; @protocol GoogleServicesSettingsViewControllerModelDelegate;
extern NSString* const kGoogleServicesSettingsViewIdentifier;
// Delegate for presentation events related to // Delegate for presentation events related to
// GoogleServicesSettingsViewController. // GoogleServicesSettingsViewController.
@protocol GoogleServicesSettingsViewControllerPresentationDelegate <NSObject> @protocol GoogleServicesSettingsViewControllerPresentationDelegate <NSObject>
......
...@@ -16,12 +16,16 @@ ...@@ -16,12 +16,16 @@
#error "This file requires ARC support." #error "This file requires ARC support."
#endif #endif
// Accessibility identifier for the Google services settings table view.
NSString* const kGoogleServicesSettingsViewIdentifier =
@"google_services_settings_view_controller";
@implementation GoogleServicesSettingsViewController @implementation GoogleServicesSettingsViewController
- (void)viewDidLoad { - (void)viewDidLoad {
[super viewDidLoad]; [super viewDidLoad];
self.tableView.accessibilityIdentifier = self.tableView.accessibilityIdentifier =
@"google_services_settings_view_controller"; kGoogleServicesSettingsViewIdentifier;
self.title = l10n_util::GetNSString(IDS_IOS_GOOGLE_SERVICES_SETTINGS_TITLE); self.title = l10n_util::GetNSString(IDS_IOS_GOOGLE_SERVICES_SETTINGS_TITLE);
} }
......
...@@ -69,6 +69,7 @@ source_set("eg_tests") { ...@@ -69,6 +69,7 @@ source_set("eg_tests") {
"//ios/chrome/browser/ui/content_suggestions:content_suggestions_constant", "//ios/chrome/browser/ui/content_suggestions:content_suggestions_constant",
"//ios/chrome/browser/ui/recent_tabs:recent_tabs_ui_constants", "//ios/chrome/browser/ui/recent_tabs:recent_tabs_ui_constants",
"//ios/chrome/browser/ui/settings", "//ios/chrome/browser/ui/settings",
"//ios/chrome/browser/ui/settings/google_services",
"//ios/chrome/browser/ui/table_view/cells", "//ios/chrome/browser/ui/table_view/cells",
"//ios/chrome/browser/ui/util", "//ios/chrome/browser/ui/util",
"//ios/chrome/browser/web_state_list", "//ios/chrome/browser/web_state_list",
......
...@@ -21,6 +21,7 @@ ...@@ -21,6 +21,7 @@
#import "ios/chrome/browser/ui/commands/open_new_tab_command.h" #import "ios/chrome/browser/ui/commands/open_new_tab_command.h"
#import "ios/chrome/browser/ui/content_suggestions/ntp_home_constant.h" #import "ios/chrome/browser/ui/content_suggestions/ntp_home_constant.h"
#import "ios/chrome/browser/ui/recent_tabs/recent_tabs_constants.h" #import "ios/chrome/browser/ui/recent_tabs/recent_tabs_constants.h"
#import "ios/chrome/browser/ui/settings/google_services/google_services_settings_view_controller.h"
#import "ios/chrome/browser/ui/signin_interaction/signin_interaction_controller_egtest_util.h" #import "ios/chrome/browser/ui/signin_interaction/signin_interaction_controller_egtest_util.h"
#include "ios/chrome/browser/ui/util/ui_util.h" #include "ios/chrome/browser/ui/util/ui_util.h"
#import "ios/chrome/browser/web_state_list/web_state_list.h" #import "ios/chrome/browser/web_state_list/web_state_list.h"
...@@ -385,7 +386,7 @@ void RemoveBrowsingData() { ...@@ -385,7 +386,7 @@ void RemoveBrowsingData() {
[[EarlGrey [[EarlGrey
selectElementWithMatcher:grey_accessibilityID( selectElementWithMatcher:grey_accessibilityID(
@"google_services_settings_view_controller")] kGoogleServicesSettingsViewIdentifier)]
performAction:grey_swipeFastInDirection(kGREYDirectionDown)]; performAction:grey_swipeFastInDirection(kGREYDirectionDown)];
[[EarlGrey [[EarlGrey
selectElementWithMatcher: selectElementWithMatcher:
......
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