diff options
| author | rsimha@chromium.org <rsimha@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-08-03 17:37:16 +0000 |
|---|---|---|
| committer | rsimha@chromium.org <rsimha@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-08-03 17:37:16 +0000 |
| commit | 476d0f61b6d5f226eb4950dce6c750e4a061cb18 (patch) | |
| tree | e9ed2f447b12828e068630cd3519bd75f3797b64 /chrome/chrome_tests.gypi | |
| parent | 5344e627bc972dc829ab4b720f4c77c023ce7f59 (diff) | |
| download | chromium_src-476d0f61b6d5f226eb4950dce6c750e4a061cb18.zip chromium_src-476d0f61b6d5f226eb4950dce6c750e4a061cb18.tar.gz chromium_src-476d0f61b6d5f226eb4950dce6c750e4a061cb18.tar.bz2 | |
Allow sync integration tests to operate on multiple datatypes: Autofill
The sync integration tests currently use a class hierarchy where the test classes for each datatype are subclasses of LiveSyncTest. While this design worked in the past, it allows tests to work with only one datatype at a time, and therefore doesn't allow us to test the interplay between datatypes.
This patch is another in the series of patches that will move away from an inheritance model to one where test cases can operate on more than one datatype. It updates the Autofill datatype to the new model, and contains the following changes:
- LiveAutofillSyncTest no longer inherits from LiveSyncTest, but is renamed to AutofillHelper, and contains a bunch of static methods that perform various operations related to Autofill.
- TwoClientLiveAutofillSyncTest is renamed to TwoClientAutofillSyncTest, and is a subclass of LiveSyncTest. It uses the methods in AutofillHelper by including its header file and attaching itself to the helper.
BUG=88510
TEST=sync_integration_tests, sync_performance_tests
Review URL: http://codereview.chromium.org/7536001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95263 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/chrome_tests.gypi')
| -rw-r--r-- | chrome/chrome_tests.gypi | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi index 6b1accd..d0356e1 100644 --- a/chrome/chrome_tests.gypi +++ b/chrome/chrome_tests.gypi @@ -3503,12 +3503,12 @@ 'test/base/test_notification_tracker.cc', 'test/base/test_notification_tracker.h', 'test/data/resource.rc', + 'test/live_sync/autofill_helper.cc', + 'test/live_sync/autofill_helper.h', 'test/live_sync/bookmarks_helper.cc', 'test/live_sync/bookmarks_helper.h', 'test/live_sync/live_apps_sync_test.cc', 'test/live_sync/live_apps_sync_test.h', - 'test/live_sync/live_autofill_sync_test.cc', - 'test/live_sync/live_autofill_sync_test.h', 'test/live_sync/live_extensions_sync_test.cc', 'test/live_sync/live_extensions_sync_test.h', 'test/live_sync/live_passwords_sync_test.cc', @@ -3545,9 +3545,9 @@ 'test/live_sync/sync_datatype_helper.cc', 'test/live_sync/sync_datatype_helper.h', 'test/live_sync/sync_errors_test.cc', + 'test/live_sync/two_client_autofill_sync_test.cc', 'test/live_sync/two_client_bookmarks_sync_test.cc', 'test/live_sync/two_client_live_apps_sync_test.cc', - 'test/live_sync/two_client_live_autofill_sync_test.cc', 'test/live_sync/two_client_live_extensions_sync_test.cc', 'test/live_sync/two_client_live_passwords_sync_test.cc', 'test/live_sync/two_client_live_sessions_sync_test.cc', @@ -3640,10 +3640,10 @@ 'sources': [ 'browser/password_manager/password_form_data.cc', 'test/base/out_of_proc_test_runner.cc', + 'test/live_sync/autofill_helper.cc', + 'test/live_sync/autofill_helper.h', 'test/live_sync/bookmarks_helper.cc', 'test/live_sync/bookmarks_helper.h', - 'test/live_sync/live_autofill_sync_test.cc', - 'test/live_sync/live_autofill_sync_test.h', 'test/live_sync/live_extensions_sync_test.cc', 'test/live_sync/live_extensions_sync_test.h', 'test/live_sync/live_passwords_sync_test.cc', |