New issue
Advanced search Search tips

Issue 662968 link

Starred by 1 user
Status: WontFix
Owner: ----
Closed: Nov 2017
Cc:
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

Make EarlGrey tests clean up after themselves, or put into ChromeTestCase setUp.

Project Member Reported by baxley@chromium.org, Nov 7 2016 
Some cleanup is done before tests to guarantee a particular state. Tests shouldn't have to worry about this. Either tests should properly clean up so we never enter this state, or if that is impossible, then it should be handled in ChromeTestCase setUp for each test.

For example,
testYoutubeInfobarDoesNotAppearMoreThanThreeTimes clears the browser state
chrome_test_util::ClearBrowsingHistory() is called sporadically by some tests

These are things that tests shouldn't have to clean up in advance.
Project Member

Comment 1 by sheriffbot@chromium.org, Nov 8 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 2 by baxley@chromium.org, Nov 8 2017

Status: WontFix (was: Untriaged)
Closing, we've done a good job fixing up many tests. For cases that aren't fixed, we should file specific bugs for those.
►

Sign in to add a comment