Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(138)

Issue 2016993004: Make some pkg tests less timing-dependent. (Closed)

Created:
4 years, 6 months ago by Florian Schneider
Modified:
4 years, 6 months ago
Reviewers:
scheglov
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Make some pkg tests less timing-dependent. Testing should not depend on how long it takes to get a response from the analysis server. Depending on how long an analysis step takes, these tests will randomly fail e.g. when run in -mdebug mode, or with other VM flags that change (i.e. slow-down) timing (--trace-compiler, --optimization-counter-threshold=5" There are probably more problematic places like this. I just fixed the ones that I saw failing when running with --optimization-counter-threshold=5. BUG=#26556 R=scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/2a6847de417ee88ed6e7ec55cf26d262d289d451

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -5 lines) Patch
M pkg/analysis_server/test/analysis_server_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download
M pkg/analysis_server/test/mocks.dart View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (3 generated)
Florian Schneider
4 years, 6 months ago (2016-05-27 12:31:37 UTC) #2
scheglov
LGTM
4 years, 6 months ago (2016-05-27 15:23:42 UTC) #4
Florian Schneider
4 years, 6 months ago (2016-05-30 10:14:22 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
2a6847de417ee88ed6e7ec55cf26d262d289d451 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698