Re-write two reading list tests |
|||||
Issue descriptiontestSavingToReadingListAndLoadNormal and testSavingToReadingListAndLoadNoNetwork do not work under the embedded test server. The tests need to stop() and restart() the server that causes the problem.
,
May 23 2017
This issue is marked as a release blocker with no milestone associated. Please add an appropriate milestone. All release blocking issues should have milestones associated to it, so that the issue can tracked and the fixes can be pushed promptly. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 23 2017
,
Jun 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c72b03e973cdca53fabc5b5f2e3776e4ae478e7a commit c72b03e973cdca53fabc5b5f2e3776e4ae478e7a Author: huangml <huangml@chromium.org> Date: Thu Jun 08 17:52:07 2017 Re-enable reading list tests The tests are disabled for landing the new server. Here is the fix for them. BUG= 724555 Review-Url: https://codereview.chromium.org/2931453002 Cr-Commit-Position: refs/heads/master@{#478019} [modify] https://crrev.com/c72b03e973cdca53fabc5b5f2e3776e4ae478e7a/ios/chrome/browser/ui/reading_list/reading_list_egtest.mm [modify] https://crrev.com/c72b03e973cdca53fabc5b5f2e3776e4ae478e7a/ios/web/public/test/http_server/http_server.h [modify] https://crrev.com/c72b03e973cdca53fabc5b5f2e3776e4ae478e7a/ios/web/public/test/http_server/http_server.mm
,
Sep 14 2017
Is this now fixed per comment #4?
,
Oct 26 2017
,
Jan 24 2018
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by huangml@chromium.org
, May 19 2017