Chromium Code Reviews
DescriptionMove sync performance tests into new target
This patch moves all sync performance tests out of sync_integration_tests and
into a new target: sync_performance_tests. Since the performance tests make
use of the existing integration test framework, I have duplicated the
dependencies and conditions in chrome_tests.gypi.
All performance tests as well as the timing helper class have been moved into
live_sync/perf. The class naming convention has been changed as well, but the
tests themselves are unmodified.
The --test-terminate-timeout flag must be used to raise the timout to around 1
minute for all tests to pass. The passwords tests are the culprit here, as
they run significantly slower than the rest of the datatypes. Eventually I
would like to have a SyncPerfTest base class which overrides the default
timeout, but this is not possible with the current inheritance hierarchy.
BUG=89824
TEST=sync_performance_tests
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=93688
Patch Set 1 #Patch Set 2 : rebase #Patch Set 3 : re-upload #Patch Set 4 : re-upload again #Messages
Total messages: 3 (0 generated)
|