Defer NetworkConfigurationHandler::CreateShillConfiguration callback

chromeos/network/auto_connect_handler_unittest.cc

 // Copyright 2014 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/network/auto_connect_handler.h"
 
 #include <memory>
 #include <string>
 
 #include "base/bind.h"
@@ skipping 119 matching lines @@
     auto_connect_handler_.reset(new AutoConnectHandler());
     auto_connect_handler_->Init(client_cert_resolver_.get(),
                                 nullptr,  // no connection handler
                                 network_state_handler_.get(),
                                 managed_config_handler_.get());
 
     base::RunLoop().RunUntilIdle();
   }
 
   void TearDown() override {
+    network_state_handler_->Shutdown();

[tbarzic, 2017/02/10 22:47:48]

not strictly part of this cl, but I wonder if we could introduce a test
NetworkHandler instance that sets up and manages test networking handlers.

[stevenjb, 2017/02/10 22:50:20]

Yep, we could. There are a lot of them though, so it would need a way to specify
which handlers are needed. *shrug*

     auto_connect_handler_.reset();
     client_cert_resolver_.reset();
     managed_config_handler_.reset();
     network_profile_handler_.reset();
     network_config_handler_.reset();
     network_state_handler_.reset();
     CertLoader::Shutdown();
     LoginState::Shutdown();
     DBusThreadManager::Shutdown();
   }
@@ skipping 351 matching lines @@
   // Applying the user policy after login would usually trigger connecting to
   // the 'best' network. But the manual connect prevents this.
   SetupPolicy(std::string(),            // no network configs
               base::DictionaryValue(),  // no global config
               true);                    // load as user policy
   EXPECT_EQ(shill::kStateOnline, GetServiceState("wifi0"));
   EXPECT_EQ(shill::kStateIdle, GetServiceState("wifi1"));
 }
 
 }  // namespace chromeos