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

Issue 1669473004: Beef up analysis server integration test messages. (Closed)

Created:
4 years, 10 months ago by Paul Berry
Modified:
4 years, 10 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Beef up analysis server integration test messages. When an integration test fails, in can be difficult to figure out why, especially if the test had to forcibly terminate the analysis server process. These messages should help debug the problem. R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/bc551cb3089bffae2b20b0dad23571e98dff5fbe

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -11 lines) Patch
M pkg/analysis_server/benchmark/integration/driver.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/test/integration/integration_tests.dart View 7 chunks +11 lines, -9 lines 0 comments Download
M pkg/analysis_server/test/timing/timing_framework.dart View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (2 generated)
Paul Berry
4 years, 10 months ago (2016-02-04 18:27:14 UTC) #2
Brian Wilkerson
LGTM
4 years, 10 months ago (2016-02-04 18:48:18 UTC) #3
Paul Berry
4 years, 10 months ago (2016-02-04 20:48:27 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
bc551cb3089bffae2b20b0dad23571e98dff5fbe (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698