Commit 1875da35 authored by sandromaggi's avatar sandromaggi Committed by Commit Bot

[Autofill Assistant] Cleanup of unit tests.

The privacy notice is no longer required, removing it from tests where
it was only added due to this requirement.

Add new test for empty card network to make sure we don't exclude all
cards in this case.

Bug: b/146126954
Bug: b/146424039
Change-Id: I35ae536c00d8774efac23eb4a586fa53f472642a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975911Reviewed-by: default avatarClemens Arbesser <arbesser@google.com>
Commit-Queue: Sandro Maggi <sandromaggi@google.com>
Cr-Commit-Position: refs/heads/master@{#726349}
parent c1c0ab3e
......@@ -101,7 +101,6 @@ public class AutofillAssistantPersonalDataManagerTest {
.setRequestPayerName(true)
.setRequestPayerEmail(true)
.setRequestPayerPhone(false))
.setPrivacyNoticeText("3rd party privacy text")
.setRequestTermsAndConditions(false))
.build());
list.add((ActionProto) ActionProto.newBuilder()
......@@ -178,7 +177,6 @@ public class AutofillAssistantPersonalDataManagerTest {
.setRequestPayerName(true)
.setRequestPayerEmail(true)
.setRequestPayerPhone(false))
.setPrivacyNoticeText("3rd party privacy text")
.setRequestTermsAndConditions(false))
.build());
list.add(
......@@ -235,7 +233,6 @@ public class AutofillAssistantPersonalDataManagerTest {
.setRequestPayerName(true)
.setRequestPayerEmail(true)
.setRequestPayerPhone(false))
.setPrivacyNoticeText("3rd party privacy text")
.setRequestTermsAndConditions(false))
.build());
list.add(
......@@ -298,7 +295,6 @@ public class AutofillAssistantPersonalDataManagerTest {
.setRequestPayerName(true)
.setRequestPayerEmail(true)
.setRequestPayerPhone(false))
.setPrivacyNoticeText("3rd party privacy text")
.setRequestTermsAndConditions(false))
.build());
list.add(
......@@ -371,7 +367,6 @@ public class AutofillAssistantPersonalDataManagerTest {
.setCollectUserData(CollectUserDataProto.newBuilder()
.setRequestPaymentMethod(true)
.addSupportedBasicCardNetworks("visa")
.setPrivacyNoticeText("3rd party privacy text")
.setRequestTermsAndConditions(false))
.build());
// No UseCreditCardAction, that is tested in PaymentTest.
......@@ -412,7 +407,6 @@ public class AutofillAssistantPersonalDataManagerTest {
.addSupportedBasicCardNetworks("visa")
.setRequireBillingPostalCode(true)
.setBillingPostalCodeMissingText("Missing Billing Code")
.setPrivacyNoticeText("3rd party privacy text")
.setRequestTermsAndConditions(false))
.build());
AutofillAssistantTestScript script = new AutofillAssistantTestScript(
......
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