Mark bunch of pub tests as flaky.

Mark bunch of pub tests as flaky.

R=ricow@google.com

Committed: https://code.google.com/p/dart/source/detail?r=26775

[ricow1, 2013-08-28 08:19:12]

LGTM with 3 : added

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.status
File sdk/lib/_internal/pub/pub.status (right):

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.sta...
sdk/lib/_internal/pub/pub.status:16:
test/install/hosted/unlock_if_incompatible_test Pass, Fail # Issue 12837
you are missing a : after the test here, + the two lines below

[Søren Gjesse, 2013-08-28 08:28:13]

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.status
File sdk/lib/_internal/pub/pub.status (right):

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.sta...
sdk/lib/_internal/pub/pub.status:6:
test/oauth2/with_an_expired_credentials_refreshes_and_saves_test: Pass, Fail #
Issue 12581
Are there any evidence that
test/oauth2/with_an_expired_credentials_refreshes_and_saves_test fails on other
platforms than Mac OS?

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.sta...
sdk/lib/_internal/pub/pub.status:7: test/hosted/remove_removed_dependency_test:
Pass, Fail # Issue 12582
Are there any evidence that
test/hosted/remove_removed_dependency_test fails on other
platforms than Mac OS?

[ricow1, 2013-08-28 08:59:03]

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.status
File sdk/lib/_internal/pub/pub.status (right):

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.sta...
sdk/lib/_internal/pub/pub.status:6:
test/oauth2/with_an_expired_credentials_refreshes_and_saves_test: Pass, Fail #
Issue 12581
On 2013/08/28 08:28:13, Søren Gjesse wrote:
> Are there any evidence that
> test/oauth2/with_an_expired_credentials_refreshes_and_saves_test fails on
other
> platforms than Mac OS?

I don't know, Sarah was just nice and actually change these lines to follow
standard way of writing issue numbers

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.sta...
sdk/lib/_internal/pub/pub.status:7: test/hosted/remove_removed_dependency_test:
Pass, Fail # Issue 12582
On 2013/08/28 08:28:13, Søren Gjesse wrote:
> Are there any evidence that
> test/hosted/remove_removed_dependency_test fails on other
> platforms than Mac OS?

Same as above

[zarah, 2013-08-28 09:32:56]

Committed patchset #2 manually as r26775 (presubmit successful).

[zarah, 2013-08-28 10:17:19]

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.status
File sdk/lib/_internal/pub/pub.status (right):

https://codereview.chromium.org/23478007/diff/1/sdk/lib/_internal/pub/pub.sta...
sdk/lib/_internal/pub/pub.status:16:
test/install/hosted/unlock_if_incompatible_test Pass, Fail # Issue 12837
On 2013/08/28 08:19:12, ricow1 wrote:
> you are missing a : after the test here, + the two lines below

Done.