Commit 23643797 authored by Maksym Onufriienko's avatar Maksym Onufriienko Committed by Commit Bot

[iOS, EG1] Disable PasswordsSettingsTestCase tests for iPad 13.2

List of disabled tests:
[PasswordsSettingsTestCase testBlacklistedFormDeletionInDetailView]
[PasswordsSettingsTestCase testCancelDeletionInDetailView]
[PasswordsSettingsTestCase testDuplicatedSavedFormDeletionInDetailView]
[PasswordsSettingsTestCase testExportFlow]
[PasswordsSettingsTestCase testSavedFormDeletionInDetailView]

Test:https://cr-buildbucket.appspot.com/build/8897021784996062192

Bug:1023619

Change-Id: Ie96bc780e5a8b4a7b828095b032843e3ebaa0576
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1909280Reviewed-by: default avatarSergio Collazos <sczs@chromium.org>
Commit-Queue: Maksym Onufriienko <monufriienko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714577}
parent b9037986
......@@ -647,6 +647,11 @@ PasswordForm CreateSampleFormWithIndex(int index) {
// Checks that deleting a saved password from password details view goes back
// to the list-of-passwords view which doesn't display that form anymore.
- (void)testSavedFormDeletionInDetailView {
// TODO(crbug.com/1023619): Enable the test on 13.2+ iPad once the bug is
// fixed.
if (base::ios::IsRunningOnOrLater(13, 2, 0) && [ChromeEarlGrey isIPadIdiom]) {
EARL_GREY_TEST_SKIPPED(@"Test disabled on iPad and iOS 13.2+");
}
// Save form to be deleted later.
SaveExamplePasswordForm();
......@@ -698,6 +703,11 @@ PasswordForm CreateSampleFormWithIndex(int index) {
// goes back to the list-of-passwords view which doesn't display that form
// anymore.
- (void)testDuplicatedSavedFormDeletionInDetailView {
// TODO(crbug.com/1023619): Enable the test on 13.2+ iPad once the bug is
// fixed.
if (base::ios::IsRunningOnOrLater(13, 2, 0) && [ChromeEarlGrey isIPadIdiom]) {
EARL_GREY_TEST_SKIPPED(@"Test disabled on iPad and iOS 13.2+");
}
// Save form to be deleted later.
SaveExamplePasswordForm();
// Save duplicate of the previously saved form to be deleted at the same time.
......@@ -757,6 +767,11 @@ PasswordForm CreateSampleFormWithIndex(int index) {
// Checks that deleting a blacklisted form from password details view goes
// back to the list-of-passwords view which doesn't display that form anymore.
- (void)testBlacklistedFormDeletionInDetailView {
// TODO(crbug.com/1023619): Enable the test on 13.2+ iPad once the bug is
// fixed.
if (base::ios::IsRunningOnOrLater(13, 2, 0) && [ChromeEarlGrey isIPadIdiom]) {
EARL_GREY_TEST_SKIPPED(@"Test disabled on iPad and iOS 13.2+");
}
// Save blacklisted form to be deleted later.
PasswordForm blacklisted;
blacklisted.origin = GURL("https://blacklisted.com");
......@@ -809,6 +824,11 @@ PasswordForm CreateSampleFormWithIndex(int index) {
// Checks that deleting a password from password details can be cancelled.
- (void)testCancelDeletionInDetailView {
// TODO(crbug.com/1023619): Enable the test on 13.2+ iPad once the bug is
// fixed.
if (base::ios::IsRunningOnOrLater(13, 2, 0) && [ChromeEarlGrey isIPadIdiom]) {
EARL_GREY_TEST_SKIPPED(@"Test disabled on iPad and iOS 13.2+");
}
// Save form to be deleted later.
SaveExamplePasswordForm();
......@@ -1503,6 +1523,11 @@ PasswordForm CreateSampleFormWithIndex(int index) {
// Test export flow
- (void)testExportFlow {
// TODO(crbug.com/1023619): Enable the test on 13.2+ iPad once the bug is
// fixed.
if (base::ios::IsRunningOnOrLater(13, 2, 0) && [ChromeEarlGrey isIPadIdiom]) {
EARL_GREY_TEST_SKIPPED(@"Test disabled on iPad and iOS 13.2+");
}
// Saving a form is needed for exporting passwords.
SaveExamplePasswordForm();
......
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