[CrOS Tether] Disconnect cleanly from active Tether networks when the user logs out or the Tether c…

chrome/browser/chromeos/tether/tether_service_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 "chrome/browser/chromeos/tether/tether_service.h"
 
 #include <memory>
 
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
@@ skipping 215 matching lines @@
  private:
   DISALLOW_COPY_AND_ASSIGN(TetherServiceTest);
 };
 
 TEST_F(TetherServiceTest, TestShutdown) {
   CreateTetherService();
   EXPECT_TRUE(test_initializer_delegate_->is_tether_running());
 
   ShutdownTetherService();
 
-  EXPECT_EQ(
-      chromeos::NetworkStateHandler::TechnologyState::TECHNOLOGY_UNAVAILABLE,
-      network_state_handler()->GetTechnologyState(
-          chromeos::NetworkTypePattern::Tether()));
+  // The TechnologyState should not have changed due to Shutdown() being called.
+  // If it had changed, any settings UI that was previously open would have
+  // shown visual jank.
+  EXPECT_EQ(chromeos::NetworkStateHandler::TechnologyState::TECHNOLOGY_ENABLED,
+            network_state_handler()->GetTechnologyState(
+                chromeos::NetworkTypePattern::Tether()));
   EXPECT_FALSE(test_initializer_delegate_->is_tether_running());
 }
 
 TEST_F(TetherServiceTest, TestSuspend) {
   CreateTetherService();
   EXPECT_TRUE(test_initializer_delegate_->is_tether_running());
 
   fake_power_manager_client_->SendSuspendImminent();
 
   EXPECT_EQ(
@@ skipping 201 matching lines @@
             tether_service_->updated_technology_state_count());
 
   SetTetherTechnologyStateEnabled(true);
   SetTetherTechnologyStateEnabled(true);
   SetTetherTechnologyStateEnabled(true);
 
   updated_technology_state_count++;
   EXPECT_EQ(updated_technology_state_count,
             tether_service_->updated_technology_state_count());
 }