New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 869526 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Add sync integration tests for Autofill/Wallet

Project Member Reported by se...@chromium.org, Jul 31

Issue description

Add integration tests for 

AutofillProfile
WalletData
WalletMetadata

The tests should run with the old sync system as well as USS
 
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 10

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0830f62fe426ce2c88db753ea1254c9d66e1693e

commit 0830f62fe426ce2c88db753ea1254c9d66e1693e
Author: Sébastien Séguin-Gagnon <sebsg@chromium.org>
Date: Fri Aug 10 14:37:44 2018

[AF USS] Add single client wallet tests for server profiles.

Bug:  869526 
Change-Id: Iacf585f600d12461dbd62171f0936e50f78d6057
Reviewed-on: https://chromium-review.googlesource.com/1170097
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#582155}
[modify] https://crrev.com/0830f62fe426ce2c88db753ea1254c9d66e1693e/chrome/browser/sync/test/integration/autofill_helper.cc
[modify] https://crrev.com/0830f62fe426ce2c88db753ea1254c9d66e1693e/chrome/browser/sync/test/integration/autofill_helper.h
[modify] https://crrev.com/0830f62fe426ce2c88db753ea1254c9d66e1693e/chrome/browser/sync/test/integration/single_client_wallet_sync_test.cc

Project Member

Comment 4 by bugdroid1@chromium.org, Aug 10

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7f1f504c6ea7f86670600bab25a0d17c38d7d6d4

commit 7f1f504c6ea7f86670600bab25a0d17c38d7d6d4
Author: Jan Krcal <jkrcal@chromium.org>
Date: Fri Aug 10 16:52:01 2018

[AF] Enable wallet integration tests for both Directory and USS

This CL creates the framework for running autofill wallet integration
tests on both implementations.

Bug:  869526 
Change-Id: Ifb44357fa8be5e2e49f099abb53b69c7be1b53d7
Reviewed-on: https://chromium-review.googlesource.com/1170836
Commit-Queue: Jan Krcal <jkrcal@chromium.org>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#582207}
[modify] https://crrev.com/7f1f504c6ea7f86670600bab25a0d17c38d7d6d4/chrome/browser/sync/test/integration/single_client_wallet_sync_test.cc

sync-triage ping.

Can this bug be closed?
sync-triage ping: sebsg@, any updates here, can this be closed?
sync-triage ping: sebsg@, any updates here, can this be closed?
Status: Fixed (was: Started)

Sign in to add a comment