| OLD | NEW |
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "chromeos/components/tether/host_scanner.h" | 5 #include "chromeos/components/tether/host_scanner.h" |
| 6 | 6 |
| 7 #include <algorithm> | 7 #include <algorithm> |
| 8 #include <memory> | 8 #include <memory> |
| 9 #include <vector> | 9 #include <vector> |
| 10 | 10 |
| 11 #include "base/memory/ptr_util.h" | 11 #include "base/memory/ptr_util.h" |
| 12 #include "base/run_loop.h" | 12 #include "base/run_loop.h" |
| 13 #include "base/test/scoped_task_environment.h" | 13 #include "base/test/scoped_task_environment.h" |
| 14 #include "chromeos/components/tether/device_id_tether_network_guid_map.h" | 14 #include "chromeos/components/tether/device_id_tether_network_guid_map.h" |
| 15 #include "chromeos/components/tether/fake_ble_connection_manager.h" | 15 #include "chromeos/components/tether/fake_ble_connection_manager.h" |
| 16 #include "chromeos/components/tether/fake_notification_presenter.h" | 16 #include "chromeos/components/tether/fake_notification_presenter.h" |
| 17 #include "chromeos/components/tether/fake_tether_host_fetcher.h" | 17 #include "chromeos/components/tether/fake_tether_host_fetcher.h" |
| 18 #include "chromeos/components/tether/host_scan_device_prioritizer.h" | 18 #include "chromeos/components/tether/host_scan_device_prioritizer.h" |
| 19 #include "chromeos/components/tether/host_scanner.h" | 19 #include "chromeos/components/tether/host_scanner.h" |
| 20 #include "chromeos/components/tether/mock_tether_host_response_recorder.h" |
| 20 #include "chromeos/dbus/dbus_thread_manager.h" | 21 #include "chromeos/dbus/dbus_thread_manager.h" |
| 21 #include "chromeos/network/network_state.h" | 22 #include "chromeos/network/network_state.h" |
| 22 #include "chromeos/network/network_state_handler.h" | 23 #include "chromeos/network/network_state_handler.h" |
| 23 #include "chromeos/network/network_state_test.h" | 24 #include "chromeos/network/network_state_test.h" |
| 24 #include "chromeos/network/network_type_pattern.h" | 25 #include "chromeos/network/network_type_pattern.h" |
| 25 #include "components/cryptauth/remote_device_test_util.h" | 26 #include "components/cryptauth/remote_device_test_util.h" |
| 26 #include "testing/gtest/include/gtest/gtest.h" | 27 #include "testing/gtest/include/gtest/gtest.h" |
| 27 | 28 |
| 28 namespace chromeos { | 29 namespace chromeos { |
| 29 | 30 |
| 30 namespace tether { | 31 namespace tether { |
| 31 | 32 |
| 32 namespace { | 33 namespace { |
| 33 | 34 |
| 34 class FakeHostScanDevicePrioritizer : public HostScanDevicePrioritizer { | 35 class FakeHostScanDevicePrioritizer : public HostScanDevicePrioritizer { |
| 35 public: | 36 public: |
| 36 FakeHostScanDevicePrioritizer() : HostScanDevicePrioritizer(nullptr) {} | 37 FakeHostScanDevicePrioritizer() : HostScanDevicePrioritizer(nullptr) {} |
| 37 ~FakeHostScanDevicePrioritizer() override {} | 38 ~FakeHostScanDevicePrioritizer() override {} |
| 38 | 39 |
| 39 // Simply leave |remote_devices| as-is. | 40 // Simply leave |remote_devices| as-is. |
| 40 void SortByHostScanOrder( | 41 void SortByHostScanOrder( |
| 41 std::vector<cryptauth::RemoteDevice>* remote_devices) const override {} | 42 std::vector<cryptauth::RemoteDevice>* remote_devices) const override {} |
| 42 }; | 43 }; |
| 43 | 44 |
| 44 class FakeHostScannerOperation : public HostScannerOperation { | 45 class FakeHostScannerOperation : public HostScannerOperation { |
| 45 public: | 46 public: |
| 46 FakeHostScannerOperation( | 47 FakeHostScannerOperation( |
| 47 const std::vector<cryptauth::RemoteDevice>& devices_to_connect, | 48 const std::vector<cryptauth::RemoteDevice>& devices_to_connect, |
| 48 BleConnectionManager* connection_manager, | 49 BleConnectionManager* connection_manager, |
| 49 HostScanDevicePrioritizer* host_scan_device_prioritizer) | 50 HostScanDevicePrioritizer* host_scan_device_prioritizer, |
| 51 TetherHostResponseRecorder* tether_host_response_recorder) |
| 50 : HostScannerOperation(devices_to_connect, | 52 : HostScannerOperation(devices_to_connect, |
| 51 connection_manager, | 53 connection_manager, |
| 52 host_scan_device_prioritizer) {} | 54 host_scan_device_prioritizer, |
| 55 tether_host_response_recorder) {} |
| 53 | 56 |
| 54 ~FakeHostScannerOperation() override {} | 57 ~FakeHostScannerOperation() override {} |
| 55 | 58 |
| 56 void SendScannedDeviceListUpdate( | 59 void SendScannedDeviceListUpdate( |
| 57 const std::vector<HostScannerOperation::ScannedDeviceInfo>& | 60 const std::vector<HostScannerOperation::ScannedDeviceInfo>& |
| 58 scanned_device_list_so_far, | 61 scanned_device_list_so_far, |
| 59 bool is_final_scan_result) { | 62 bool is_final_scan_result) { |
| 60 scanned_device_list_so_far_ = scanned_device_list_so_far; | 63 scanned_device_list_so_far_ = scanned_device_list_so_far; |
| 61 NotifyObserversOfScannedDeviceList(is_final_scan_result); | 64 NotifyObserversOfScannedDeviceList(is_final_scan_result); |
| 62 } | 65 } |
| 63 }; | 66 }; |
| 64 | 67 |
| 65 class FakeHostScannerOperationFactory : public HostScannerOperation::Factory { | 68 class FakeHostScannerOperationFactory : public HostScannerOperation::Factory { |
| 66 public: | 69 public: |
| 67 FakeHostScannerOperationFactory( | 70 FakeHostScannerOperationFactory( |
| 68 const std::vector<cryptauth::RemoteDevice>& test_devices) | 71 const std::vector<cryptauth::RemoteDevice>& test_devices) |
| 69 : expected_devices_(test_devices) {} | 72 : expected_devices_(test_devices) {} |
| 70 virtual ~FakeHostScannerOperationFactory() {} | 73 virtual ~FakeHostScannerOperationFactory() {} |
| 71 | 74 |
| 72 std::vector<FakeHostScannerOperation*>& created_operations() { | 75 std::vector<FakeHostScannerOperation*>& created_operations() { |
| 73 return created_operations_; | 76 return created_operations_; |
| 74 } | 77 } |
| 75 | 78 |
| 76 protected: | 79 protected: |
| 77 // HostScannerOperation::Factory: | 80 // HostScannerOperation::Factory: |
| 78 std::unique_ptr<HostScannerOperation> BuildInstance( | 81 std::unique_ptr<HostScannerOperation> BuildInstance( |
| 79 const std::vector<cryptauth::RemoteDevice>& devices_to_connect, | 82 const std::vector<cryptauth::RemoteDevice>& devices_to_connect, |
| 80 BleConnectionManager* connection_manager, | 83 BleConnectionManager* connection_manager, |
| 81 HostScanDevicePrioritizer* host_scan_device_prioritizer) override { | 84 HostScanDevicePrioritizer* host_scan_device_prioritizer, |
| 85 TetherHostResponseRecorder* tether_host_response_recorder) override { |
| 82 EXPECT_EQ(expected_devices_, devices_to_connect); | 86 EXPECT_EQ(expected_devices_, devices_to_connect); |
| 83 FakeHostScannerOperation* operation = new FakeHostScannerOperation( | 87 FakeHostScannerOperation* operation = new FakeHostScannerOperation( |
| 84 devices_to_connect, connection_manager, host_scan_device_prioritizer); | 88 devices_to_connect, connection_manager, host_scan_device_prioritizer, |
| 89 tether_host_response_recorder); |
| 85 created_operations_.push_back(operation); | 90 created_operations_.push_back(operation); |
| 86 return base::WrapUnique(operation); | 91 return base::WrapUnique(operation); |
| 87 } | 92 } |
| 88 | 93 |
| 89 private: | 94 private: |
| 90 const std::vector<cryptauth::RemoteDevice>& expected_devices_; | 95 const std::vector<cryptauth::RemoteDevice>& expected_devices_; |
| 91 std::vector<FakeHostScannerOperation*> created_operations_; | 96 std::vector<FakeHostScannerOperation*> created_operations_; |
| 92 }; | 97 }; |
| 93 | 98 |
| 94 std::string GenerateCellProviderForDevice( | 99 std::string GenerateCellProviderForDevice( |
| (...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 142 network_state_handler()->SetTetherTechnologyState( | 147 network_state_handler()->SetTetherTechnologyState( |
| 143 NetworkStateHandler::TECHNOLOGY_ENABLED); | 148 NetworkStateHandler::TECHNOLOGY_ENABLED); |
| 144 | 149 |
| 145 scanned_device_infos_so_far_.clear(); | 150 scanned_device_infos_so_far_.clear(); |
| 146 | 151 |
| 147 fake_tether_host_fetcher_ = base::MakeUnique<FakeTetherHostFetcher>( | 152 fake_tether_host_fetcher_ = base::MakeUnique<FakeTetherHostFetcher>( |
| 148 test_devices_, false /* synchronously_reply_with_results */); | 153 test_devices_, false /* synchronously_reply_with_results */); |
| 149 fake_ble_connection_manager_ = base::MakeUnique<FakeBleConnectionManager>(); | 154 fake_ble_connection_manager_ = base::MakeUnique<FakeBleConnectionManager>(); |
| 150 fake_host_scan_device_prioritizer_ = | 155 fake_host_scan_device_prioritizer_ = |
| 151 base::MakeUnique<FakeHostScanDevicePrioritizer>(); | 156 base::MakeUnique<FakeHostScanDevicePrioritizer>(); |
| 157 mock_tether_host_response_recorder_ = |
| 158 base::MakeUnique<MockTetherHostResponseRecorder>(); |
| 152 fake_notification_presenter_ = | 159 fake_notification_presenter_ = |
| 153 base::MakeUnique<FakeNotificationPresenter>(); | 160 base::MakeUnique<FakeNotificationPresenter>(); |
| 154 device_id_tether_network_guid_map_ = | 161 device_id_tether_network_guid_map_ = |
| 155 base::MakeUnique<DeviceIdTetherNetworkGuidMap>(); | 162 base::MakeUnique<DeviceIdTetherNetworkGuidMap>(); |
| 156 | 163 |
| 157 fake_host_scanner_operation_factory_ = | 164 fake_host_scanner_operation_factory_ = |
| 158 base::WrapUnique(new FakeHostScannerOperationFactory(test_devices_)); | 165 base::WrapUnique(new FakeHostScannerOperationFactory(test_devices_)); |
| 159 HostScannerOperation::Factory::SetInstanceForTesting( | 166 HostScannerOperation::Factory::SetInstanceForTesting( |
| 160 fake_host_scanner_operation_factory_.get()); | 167 fake_host_scanner_operation_factory_.get()); |
| 161 | 168 |
| 162 host_scanner_ = base::WrapUnique(new HostScanner( | 169 host_scanner_ = base::WrapUnique(new HostScanner( |
| 163 fake_tether_host_fetcher_.get(), fake_ble_connection_manager_.get(), | 170 fake_tether_host_fetcher_.get(), fake_ble_connection_manager_.get(), |
| 164 fake_host_scan_device_prioritizer_.get(), network_state_handler(), | 171 fake_host_scan_device_prioritizer_.get(), |
| 172 mock_tether_host_response_recorder_.get(), network_state_handler(), |
| 165 fake_notification_presenter_.get(), | 173 fake_notification_presenter_.get(), |
| 166 device_id_tether_network_guid_map_.get())); | 174 device_id_tether_network_guid_map_.get())); |
| 167 } | 175 } |
| 168 | 176 |
| 169 void TearDown() override { | 177 void TearDown() override { |
| 170 ShutdownNetworkState(); | 178 ShutdownNetworkState(); |
| 171 NetworkStateTest::TearDown(); | 179 NetworkStateTest::TearDown(); |
| 172 DBusThreadManager::Shutdown(); | 180 DBusThreadManager::Shutdown(); |
| 173 | 181 |
| 174 HostScannerOperation::Factory::SetInstanceForTesting(nullptr); | 182 HostScannerOperation::Factory::SetInstanceForTesting(nullptr); |
| (...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 214 | 222 |
| 215 base::test::ScopedTaskEnvironment scoped_task_environment_; | 223 base::test::ScopedTaskEnvironment scoped_task_environment_; |
| 216 | 224 |
| 217 const std::vector<cryptauth::RemoteDevice> test_devices_; | 225 const std::vector<cryptauth::RemoteDevice> test_devices_; |
| 218 const std::vector<HostScannerOperation::ScannedDeviceInfo> | 226 const std::vector<HostScannerOperation::ScannedDeviceInfo> |
| 219 test_scanned_device_infos; | 227 test_scanned_device_infos; |
| 220 | 228 |
| 221 std::unique_ptr<FakeTetherHostFetcher> fake_tether_host_fetcher_; | 229 std::unique_ptr<FakeTetherHostFetcher> fake_tether_host_fetcher_; |
| 222 std::unique_ptr<FakeBleConnectionManager> fake_ble_connection_manager_; | 230 std::unique_ptr<FakeBleConnectionManager> fake_ble_connection_manager_; |
| 223 std::unique_ptr<HostScanDevicePrioritizer> fake_host_scan_device_prioritizer_; | 231 std::unique_ptr<HostScanDevicePrioritizer> fake_host_scan_device_prioritizer_; |
| 232 std::unique_ptr<MockTetherHostResponseRecorder> |
| 233 mock_tether_host_response_recorder_; |
| 224 std::unique_ptr<FakeNotificationPresenter> fake_notification_presenter_; | 234 std::unique_ptr<FakeNotificationPresenter> fake_notification_presenter_; |
| 225 // TODO(hansberry): Use a fake for this when a real mapping scheme is created. | 235 // TODO(hansberry): Use a fake for this when a real mapping scheme is created. |
| 226 std::unique_ptr<DeviceIdTetherNetworkGuidMap> | 236 std::unique_ptr<DeviceIdTetherNetworkGuidMap> |
| 227 device_id_tether_network_guid_map_; | 237 device_id_tether_network_guid_map_; |
| 228 | 238 |
| 229 std::unique_ptr<FakeHostScannerOperationFactory> | 239 std::unique_ptr<FakeHostScannerOperationFactory> |
| 230 fake_host_scanner_operation_factory_; | 240 fake_host_scanner_operation_factory_; |
| 231 | 241 |
| 232 std::vector<HostScannerOperation::ScannedDeviceInfo> | 242 std::vector<HostScannerOperation::ScannedDeviceInfo> |
| 233 scanned_device_infos_so_far_; | 243 scanned_device_infos_so_far_; |
| (...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 360 ->SendScannedDeviceListUpdate(scanned_device_infos_so_far_, | 370 ->SendScannedDeviceListUpdate(scanned_device_infos_so_far_, |
| 361 true /* is_final_scan_result */); | 371 true /* is_final_scan_result */); |
| 362 EXPECT_EQ(scanned_device_infos_so_far_, | 372 EXPECT_EQ(scanned_device_infos_so_far_, |
| 363 host_scanner_->most_recent_scan_results()); | 373 host_scanner_->most_recent_scan_results()); |
| 364 EXPECT_FALSE(host_scanner_->IsScanActive()); | 374 EXPECT_FALSE(host_scanner_->IsScanActive()); |
| 365 } | 375 } |
| 366 | 376 |
| 367 } // namespace tether | 377 } // namespace tether |
| 368 | 378 |
| 369 } // namespace chromeos | 379 } // namespace chromeos |
| OLD | NEW |