[Autofill] Use TestdataSource that uses complete countryinfo.txt data |
|||
Issue descriptiongood way: https://cs.chromium.org/chromium/src/components/autofill/core/browser/field_filler_unittest.cc?rcl=514d9f3232687f8eb35ffa74993d1b5ad2a0b219&l=625 bad way: https://cs.chromium.org/chromium/src/components/autofill/core/browser/autofill_profile_validator_unittest.cc?type=cs&q=TestdataSource+file:autofill&sq=package:chromium&l=40
,
Dec 20 2017
,
Dec 20 2017
mathp: For autofill profile validator unittests (your example of a bad way), it wasn't changed because of this test: ValidateAddress_RuleNotExists. We need to have a source for a valid country, where the rules are not available in the source. This is not the case with the countryinfo.txt. What do you think?
,
Dec 20 2017
The same is true for this test: https://cs.chromium.org/chromium/src/components/autofill/core/browser/address_normalizer_impl_unittest.cc?l=201
,
Dec 20 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by parastoog@chromium.org
, Dec 20 2017Status: Started (was: Available)