Commit 8a06d7e1 authored by Sujie Zhu's avatar Sujie Zhu Committed by Commit Bot

Add corresponding nickname autofill_private api test

addNewCreditCard creates a credit card without nickname.
updateExistingCreditCard will add a nickname to this card.

Bug: 1082013
Change-Id: I0e333f55552481070437777d40e1376a6f10404e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2226088Reviewed-by: default avatarDevlin <rdevlin.cronin@chromium.org>
Commit-Queue: Sujie Zhu <sujiezhu@google.com>
Cr-Commit-Position: refs/heads/master@{#774681}
parent 28470cbf
...@@ -348,7 +348,6 @@ ExtensionFunction::ResponseAction AutofillPrivateSaveCreditCardFunction::Run() { ...@@ -348,7 +348,6 @@ ExtensionFunction::ResponseAction AutofillPrivateSaveCreditCardFunction::Run() {
} }
// TODO(crbug.com/1063426): Log nickname management via settings page. // TODO(crbug.com/1063426): Log nickname management via settings page.
// TODO(crbug.com/1063426): add corresponding nickname autofill_private test.
if (card->nickname) { if (card->nickname) {
credit_card.SetNickname(base::UTF8ToUTF16(*card->nickname)); credit_card.SetNickname(base::UTF8ToUTF16(*card->nickname));
} }
......
...@@ -185,8 +185,8 @@ var availableTests = [ ...@@ -185,8 +185,8 @@ var availableTests = [
function filterCardProperties(cards) { function filterCardProperties(cards) {
return cards.map(cards => { return cards.map(cards => {
var filteredCards = {}; var filteredCards = {};
['name', 'cardNumber', 'expirationMonth', 'expirationYear'].forEach( ['name', 'cardNumber', 'expirationMonth', 'expirationYear', 'nickname']
property => { .forEach(property => {
filteredCards[property] = cards[property]; filteredCards[property] = cards[property];
}); });
return filteredCards; return filteredCards;
...@@ -206,7 +206,8 @@ var availableTests = [ ...@@ -206,7 +206,8 @@ var availableTests = [
name: CARD_NAME, name: CARD_NAME,
cardNumber: NUMBER, cardNumber: NUMBER,
expirationMonth: EXP_MONTH, expirationMonth: EXP_MONTH,
expirationYear: EXP_YEAR expirationYear: EXP_YEAR,
nickname: undefined
}], }],
filterCardProperties(cardList)); filterCardProperties(cardList));
})); }));
...@@ -223,11 +224,13 @@ var availableTests = [ ...@@ -223,11 +224,13 @@ var availableTests = [
function updateExistingCreditCard() { function updateExistingCreditCard() {
var UPDATED_CARD_NAME = 'UpdatedCardName'; var UPDATED_CARD_NAME = 'UpdatedCardName';
var UPDATED_EXP_YEAR = '2888'; var UPDATED_EXP_YEAR = '2888';
var UPDATED_NICKNAME = 'New nickname';
function filterCardProperties(cards) { function filterCardProperties(cards) {
return cards.map(cards => { return cards.map(cards => {
var filteredCards = {}; var filteredCards = {};
['guid', 'name', 'cardNumber', 'expirationMonth', 'expirationYear'] ['guid', 'name', 'cardNumber', 'expirationMonth', 'expirationYear',
'nickname']
.forEach(property => { .forEach(property => {
filteredCards[property] = cards[property]; filteredCards[property] = cards[property];
}); });
...@@ -252,7 +255,8 @@ var availableTests = [ ...@@ -252,7 +255,8 @@ var availableTests = [
name: UPDATED_CARD_NAME, name: UPDATED_CARD_NAME,
cardNumber: NUMBER, cardNumber: NUMBER,
expirationMonth: EXP_MONTH, expirationMonth: EXP_MONTH,
expirationYear: UPDATED_EXP_YEAR expirationYear: UPDATED_EXP_YEAR,
nickname: UPDATED_NICKNAME
}], }],
filterCardProperties(cardList)); filterCardProperties(cardList));
})); }));
...@@ -262,7 +266,8 @@ var availableTests = [ ...@@ -262,7 +266,8 @@ var availableTests = [
chrome.autofillPrivate.saveCreditCard({ chrome.autofillPrivate.saveCreditCard({
guid: cardGuid, guid: cardGuid,
name: UPDATED_CARD_NAME, name: UPDATED_CARD_NAME,
expirationYear: UPDATED_EXP_YEAR expirationYear: UPDATED_EXP_YEAR,
nickname: UPDATED_NICKNAME
}); });
})); }));
}, },
......
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