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

Issue 842803003: Fix order of operations dependencies in element_references_test (Closed)

Created:
5 years, 11 months ago by Paul Berry
Modified:
5 years, 11 months ago
Reviewers:
scheglov
CC:
reviews_dartlang.org
Target Ref:
refs/remotes/git-svn
Visibility:
Public.

Description

Fix order of operations dependencies in element_references_test It's not safe for a single test to have two overlapping calls to findElementReferences() because fields in the test class are used to hold the search results and to keep track of whether more results are pending. Fixes a bogus test failure that arose after the relative ordering of futures and microtasks was changed. BUG=dartbug.com/22014 R=scheglov@google.com Committed: https://code.google.com/p/dart/source/detail?r=42738

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -12 lines) Patch
M pkg/analysis_server/test/search/element_references_test.dart View 3 chunks +7 lines, -8 lines 0 comments Download
M pkg/pkg.status View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Paul Berry
5 years, 11 months ago (2015-01-08 19:48:45 UTC) #2
scheglov
LGTM Thank you!
5 years, 11 months ago (2015-01-08 19:50:44 UTC) #3
Paul Berry
5 years, 11 months ago (2015-01-09 16:36:48 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as 42738 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698