Re-enable reading list tests

ios/chrome/browser/ui/reading_list/reading_list_egtest.mm

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import <EarlGrey/EarlGrey.h>
 #import <UIKit/UIKit.h>
 #import <XCTest/XCTest.h>
 
 #include "base/memory/ptr_util.h"
 #include "base/strings/sys_string_conversions.h"
@@ skipping 321 matching lines @@
                                                     wait_for_loading),
                @"Page did not load.");
   } else {
     [ChromeEarlGrey waitForStaticHTMLViewContainingText:contentToKeep];
   }
 
   // Test Omnibox URL
   [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
                                           web::test::HttpServer::MakeUrl(
                                               kDistillableURL)
                                               .GetContent())]

[baxley, 2017/06/07 18:21:06]

nit: what do you think about creating a variable for the URL, and possibly add
"using chrome_test_util::OmniboxText" to make this fit on one line?

Or anything else to make this line shorter?

[huangml1, 2017/06/08 01:02:51]

Done.

       assertWithMatcher:grey_notNil()];
 
   // Test presence of online page
   [[EarlGrey selectElementWithMatcher:chrome_test_util::WebViewContainingText(
                                           kContentToKeep)]
       assertWithMatcher:online ? grey_notNil() : grey_nil()];
 
   // Test presence of offline page.
   if (online) {
     [ChromeEarlGrey waitForStaticHTMLViewNotContainingText:contentToKeep];
@@ skipping 22 matching lines @@
 
 - (void)setUp {
   [super setUp];
   ReadingListModel* model = GetReadingListModel();
   for (const GURL& url : model->Keys())
     model->RemoveEntryByURL(url);
 }
 
 - (void)tearDown {
   web::test::HttpServer& server = web::test::HttpServer::GetSharedInstance();
+  server.SetSuspend(NO);

[baxley, 2017/06/07 18:21:06]

are there any side-effects to calling this after tests that don't suspend the
server?

[huangml1, 2017/06/08 01:02:51]

No.  The method just sets the variable to "NO" and cause no effect on the
responseProvider.

   if (!server.IsRunning()) {
     server.StartOrDie();
     base::test::ios::SpinRunLoopWithMinDelay(
         base::TimeDelta::FromSecondsD(kServerOperationDelay));
   }
   [super tearDown];
 }
 
 // Tests that the Reading List view is accessible.
 - (void)testAccessibility {
@@ skipping 48 matching lines @@
       assertWithMatcher:grey_notNil()];
 
   // Verify that the webState's title is correct.
   XCTAssertTrue(chrome_test_util::GetCurrentWebState()->GetTitle() ==
                 base::ASCIIToUTF16(pageTitle.c_str()));
 }
 
 // Tests that sharing a web page to the Reading List results in a snackbar
 // appearing, and that the Reading List entry is present in the Reading List.
 // Loads online version by tapping on entry.
-// TODO(crbug.com/724555): Re-enable the test.
-- (void)DISABLED_testSavingToReadingListAndLoadNormal {
+- (void)testSavingToReadingListAndLoadNormal {
   auto network_change_disabler =
       base::MakeUnique<net::NetworkChangeNotifier::DisableForTest>();
   auto wifi_network = base::MakeUnique<WifiNetworkChangeNotifier>();
   web::test::SetUpSimpleHttpServer(ResponsesForDistillationServer());
   web::test::HttpServer& server = web::test::HttpServer::GetSharedInstance();
   std::string pageTitle(kDistillableTitle);
 
   // Open http://potato
   [ChromeEarlGrey loadURL:web::test::HttpServer::MakeUrl(kDistillableURL)];
 
   AddCurrentPageToReadingList();
 
   // Navigate to http://beans
   [ChromeEarlGrey loadURL:web::test::HttpServer::MakeUrl(kNonDistillableURL)];
   [ChromeEarlGrey waitForPageToFinishLoading];
 
   // Verify that an entry with the correct title is present in the reading list.
   OpenReadingList();
   AssertEntryVisible(pageTitle);
   WaitForDistillation();
 
   // Long press the entry, and open it offline.
   TapEntry(pageTitle);
 
   AssertIsShowingDistillablePage(true);
   // Stop server to reload offline.
-  server.Stop();
+  server.SetSuspend(YES);
   base::test::ios::SpinRunLoopWithMinDelay(
       base::TimeDelta::FromSecondsD(kServerOperationDelay));
 
   chrome_test_util::GetCurrentWebState()->GetNavigationManager()->Reload(
       web::ReloadType::NORMAL, false);
   AssertIsShowingDistillablePage(false);
 }
 
 // Tests that sharing a web page to the Reading List results in a snackbar
 // appearing, and that the Reading List entry is present in the Reading List.
 // Loads offline version by tapping on entry without web server.
-// TODO(crbug.com/724555): Re-enable the test.
-- (void)DISABLED_testSavingToReadingListAndLoadNoNetwork {
+- (void)testSavingToReadingListAndLoadNoNetwork {
   auto network_change_disabler =
       base::MakeUnique<net::NetworkChangeNotifier::DisableForTest>();
   auto wifi_network = base::MakeUnique<WifiNetworkChangeNotifier>();
   web::test::SetUpSimpleHttpServer(ResponsesForDistillationServer());
   std::string pageTitle(kDistillableTitle);
   web::test::HttpServer& server = web::test::HttpServer::GetSharedInstance();
   // Open http://potato
   [ChromeEarlGrey loadURL:web::test::HttpServer::MakeUrl(kDistillableURL)];
 
   AddCurrentPageToReadingList();
 
   // Navigate to http://beans
 
   [ChromeEarlGrey loadURL:web::test::HttpServer::MakeUrl(kNonDistillableURL)];
   [ChromeEarlGrey waitForPageToFinishLoading];
 
   // Verify that an entry with the correct title is present in the reading list.
   OpenReadingList();
   AssertEntryVisible(pageTitle);
   WaitForDistillation();
 
   // Stop server to generate error.
-  server.Stop();
+  server.SetSuspend(YES);
   base::test::ios::SpinRunLoopWithMinDelay(
       base::TimeDelta::FromSecondsD(kServerOperationDelay));
   // Long press the entry, and open it offline.
   TapEntry(pageTitle);
 
   AssertIsShowingDistillablePage(false);
   // Start server to reload online error.
-  server.StartOrDie();
+  server.SetSuspend(NO);
   base::test::ios::SpinRunLoopWithMinDelay(
       base::TimeDelta::FromSecondsD(kServerOperationDelay));
   web::test::SetUpSimpleHttpServer(ResponsesForDistillationServer());
 
   chrome_test_util::GetCurrentWebState()->GetNavigationManager()->Reload(
       web::ReloadType::NORMAL, false);
   AssertIsShowingDistillablePage(true);
 }
 
 // Tests that sharing a web page to the Reading List results in a snackbar
@@ skipping 262 matching lines @@
       assertWithMatcher:grey_nil()];
   TapButtonWithID(IDS_IOS_READING_LIST_EDIT_BUTTON);
   TapButtonWithID(IDS_IOS_READING_LIST_DELETE_ALL_READ_BUTTON);
 
   // Verify the background string is displayed.
   [[EarlGrey selectElementWithMatcher:EmptyBackground()]
       assertWithMatcher:grey_notNil()];
 }
 
 @end