Remove the configuration of Tether-associated Wi-Fi networks once connectivity is lost.

chromeos/components/tether/tether_network_disconnection_handler_unittest.cc

 // Copyright 2017 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.
 
 #include "chromeos/components/tether/tether_network_disconnection_handler.h"
 
 #include <memory>
 
 #include "base/memory/ptr_util.h"
 #include "base/test/scoped_task_environment.h"
 #include "chromeos/components/tether/fake_active_host.h"
+#include "chromeos/components/tether/fake_managed_network_configuration_handler.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/network/network_state.h"
 #include "chromeos/network/network_state_test.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/cros_system_api/dbus/shill/dbus-constants.h"
 
 namespace chromeos {
 
 namespace tether {
@@ skipping 22 matching lines @@
   ~TetherNetworkDisconnectionHandlerTest() override {}
 
   void SetUp() override {
     DBusThreadManager::Initialize();
     NetworkStateTest::SetUp();
 
     wifi_service_path_ =
         ConfigureService(CreateConnectedWifiConfigurationJsonString());
 
     fake_active_host_ = base::MakeUnique<FakeActiveHost>();
+    fake_managed_network_configuration_handler_ =
+        base::WrapUnique(new FakeManagedNetworkConfigurationHandler());
 
     handler_ = base::WrapUnique(new TetherNetworkDisconnectionHandler(
-        fake_active_host_.get(), network_state_handler()));
+        fake_active_host_.get(), network_state_handler(),
+        fake_managed_network_configuration_handler_.get()));
   }
 
   void TearDown() override {
     // Delete handler before the NetworkStateHandler and |fake_active_host_|.
     handler_.reset();
     ShutdownNetworkState();
     NetworkStateTest::TearDown();
     DBusThreadManager::Shutdown();
   }
 
   void NotifyDisconnected() {
     SetServiceProperty(wifi_service_path_, std::string(shill::kStateProperty),
                        base::Value(shill::kStateIdle));
   }
 
   base::test::ScopedTaskEnvironment scoped_task_environment_;
   std::string wifi_service_path_;
 
   std::unique_ptr<FakeActiveHost> fake_active_host_;
+  std::unique_ptr<FakeManagedNetworkConfigurationHandler>
+      fake_managed_network_configuration_handler_;
 
   std::unique_ptr<TetherNetworkDisconnectionHandler> handler_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(TetherNetworkDisconnectionHandlerTest);
 };
 
 TEST_F(TetherNetworkDisconnectionHandlerTest, TestConnectAndDisconnect) {
   // Connect to the network. |handler_| should start tracking the connection.
   fake_active_host_->SetActiveHostConnecting(kDeviceId, kTetherNetworkGuid);
   fake_active_host_->SetActiveHostConnected(kDeviceId, kTetherNetworkGuid,
                                             kWifiNetworkGuid);
 
   // Now, disconnect the Wi-Fi network. This should result in
   // |fake_active_host_| becoming disconnected.
   NotifyDisconnected();
   EXPECT_EQ(ActiveHost::ActiveHostStatus::DISCONNECTED,
             fake_active_host_->GetActiveHostStatus());
+
+  EXPECT_EQ(wifi_service_path_, fake_managed_network_configuration_handler_

[Kyle Horimoto, 2017/04/17 20:34:53]

nit: Can you fit the second parameter on its own line? This looks weird.

[stevenjb, 2017/04/18 21:03:36]

'git cl format'

[Ryan Hansberry, 2017/04/20 20:57:58]

This was the style given by git cl format :(

+                                    ->last_removed_configuration_path());
 }
 
 }  // namespace tether
 
-}  // namespace cryptauth
+}  // namespace chromeos