Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Unified Diff: chromeos/components/tether/host_scanner_unittest.cc

Issue 2913313002: Tether: Persist if first-time setup is required to HostScanCache. (Closed)
Patch Set: khorimoto@ comments. Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chromeos/components/tether/host_scanner_unittest.cc
diff --git a/chromeos/components/tether/host_scanner_unittest.cc b/chromeos/components/tether/host_scanner_unittest.cc
index e793b40ced80fe6b61372412f5916a7fc0f0e6ca..b954d3cffce1a459ab7db5f449a6490e4db4f7a5 100644
--- a/chromeos/components/tether/host_scanner_unittest.cc
+++ b/chromeos/components/tether/host_scanner_unittest.cc
@@ -196,10 +196,10 @@ CreateFakeScannedDeviceInfos(
cell_provider_name, battery_percentage, connection_strength);
// Require set-up for odd-numbered device indices.
- bool set_up_required = i % 2 == 0;
+ bool setup_required = i % 2 == 0;
scanned_device_infos.push_back(HostScannerOperation::ScannedDeviceInfo(
- remote_devices[i], device_status, set_up_required));
+ remote_devices[i], device_status, setup_required));
}
return scanned_device_infos;
@@ -332,6 +332,8 @@ class HostScannerTest : public testing::Test {
EXPECT_EQ(0, cache_item.signal_strength);
else
EXPECT_EQ(status.connection_strength() * 25, cache_item.signal_strength);
+
+ EXPECT_EQ(scanned_device_info.setup_required, cache_item.setup_required);
}
const std::vector<cryptauth::RemoteDevice> test_devices_;

Powered by Google App Engine
This is Rietveld 408576698