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

Side by Side Diff: chromeos/components/tether/device_status_util_unittest.cc

Issue 2926503002: [CrOS Tether] Create proto_test_util.cc/h and move CreateTestDeviceStatus() there. (Closed)
Patch Set: Included missing dep. Created 3 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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/device_status_util.h" 5 #include "chromeos/components/tether/device_status_util.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
11 #include "base/macros.h" 11 #include "base/macros.h"
12 #include "base/memory/ptr_util.h" 12 #include "base/memory/ptr_util.h"
13 #include "chromeos/components/tether/fake_tether_host_fetcher.h" 13 #include "chromeos/components/tether/fake_tether_host_fetcher.h"
14 #include "chromeos/components/tether/proto_test_util.h"
14 #include "components/cryptauth/remote_device.h" 15 #include "components/cryptauth/remote_device.h"
15 #include "components/cryptauth/remote_device_test_util.h" 16 #include "components/cryptauth/remote_device_test_util.h"
16 #include "components/prefs/testing_pref_service.h" 17 #include "components/prefs/testing_pref_service.h"
17 #include "testing/gtest/include/gtest/gtest.h" 18 #include "testing/gtest/include/gtest/gtest.h"
18 19
19 namespace chromeos { 20 namespace chromeos {
20 21
21 namespace tether { 22 namespace tether {
22 23
23 namespace {
24
25 const char kDoNotSetStringField[] = "doNotSetField";
26 const int kDoNotSetIntField = -100;
27
28 // Creates a DeviceStatus object using the parameters provided. If
29 // |kDoNotSetStringField| or |kDoNotSetIntField| are passed, these fields will
30 // not be set in the output.
31 DeviceStatus CreateFakeDeviceStatus(const std::string& cell_provider_name,
32 int battery_percentage,
33 int connection_strength) {
34 // TODO(khorimoto): Once a ConnectedWifiSsid field is added as a property of
35 // Tether networks, give an option to pass a parameter for that field as well.
36 WifiStatus wifi_status;
37 wifi_status.set_status_code(
38 WifiStatus_StatusCode::WifiStatus_StatusCode_CONNECTED);
39 wifi_status.set_ssid("Google A");
40
41 DeviceStatus device_status;
42 if (battery_percentage != kDoNotSetIntField) {
43 device_status.set_battery_percentage(battery_percentage);
44 }
45 if (cell_provider_name != kDoNotSetStringField) {
46 device_status.set_cell_provider(cell_provider_name);
47 }
48 if (connection_strength != kDoNotSetIntField) {
49 device_status.set_connection_strength(connection_strength);
50 }
51
52 device_status.mutable_wifi_status()->CopyFrom(wifi_status);
53
54 return device_status;
55 }
56
57 } // namespace
58
59 class DeviceStatusUtilTest : public testing::Test { 24 class DeviceStatusUtilTest : public testing::Test {
60 public: 25 public:
61 DeviceStatusUtilTest() {} 26 DeviceStatusUtilTest() {}
62 27
63 private: 28 private:
64 DISALLOW_COPY_AND_ASSIGN(DeviceStatusUtilTest); 29 DISALLOW_COPY_AND_ASSIGN(DeviceStatusUtilTest);
65 }; 30 };
66 31
67 TEST_F(DeviceStatusUtilTest, TestNotPresent) { 32 TEST_F(DeviceStatusUtilTest, TestNotPresent) {
68 DeviceStatus status = 33 DeviceStatus status = CreateTestDeviceStatus(
69 CreateFakeDeviceStatus(kDoNotSetStringField /* cell_provider_name */, 34 proto_test_util::kDoNotSetStringField /* cell_provider_name */,
70 kDoNotSetIntField /* battery_percentage */, 35 proto_test_util::kDoNotSetIntField /* battery_percentage */,
71 kDoNotSetIntField /* connection_strength */); 36 proto_test_util::kDoNotSetIntField /* connection_strength */);
72 37
73 std::string carrier; 38 std::string carrier;
74 int32_t battery_percentage; 39 int32_t battery_percentage;
75 int32_t signal_strength; 40 int32_t signal_strength;
76 41
77 NormalizeDeviceStatus(status, &carrier, &battery_percentage, 42 NormalizeDeviceStatus(status, &carrier, &battery_percentage,
78 &signal_strength); 43 &signal_strength);
79 44
80 EXPECT_EQ("unknown-carrier", carrier); 45 EXPECT_EQ("unknown-carrier", carrier);
81 EXPECT_EQ(100, battery_percentage); 46 EXPECT_EQ(100, battery_percentage);
82 EXPECT_EQ(100, signal_strength); 47 EXPECT_EQ(100, signal_strength);
83 } 48 }
84 49
85 TEST_F(DeviceStatusUtilTest, TestEmptyCellProvider) { 50 TEST_F(DeviceStatusUtilTest, TestEmptyCellProvider) {
86 DeviceStatus status = CreateFakeDeviceStatus( 51 DeviceStatus status = CreateTestDeviceStatus(
87 "" /* cell_provider_name */, kDoNotSetIntField /* battery_percentage */, 52 "" /* cell_provider_name */,
88 kDoNotSetIntField /* connection_strength */); 53 proto_test_util::kDoNotSetIntField /* battery_percentage */,
54 proto_test_util::kDoNotSetIntField /* connection_strength */);
89 55
90 std::string carrier; 56 std::string carrier;
91 int32_t battery_percentage; 57 int32_t battery_percentage;
92 int32_t signal_strength; 58 int32_t signal_strength;
93 59
94 NormalizeDeviceStatus(status, &carrier, &battery_percentage, 60 NormalizeDeviceStatus(status, &carrier, &battery_percentage,
95 &signal_strength); 61 &signal_strength);
96 62
97 EXPECT_EQ("unknown-carrier", carrier); 63 EXPECT_EQ("unknown-carrier", carrier);
98 EXPECT_EQ(100, battery_percentage); 64 EXPECT_EQ(100, battery_percentage);
99 EXPECT_EQ(100, signal_strength); 65 EXPECT_EQ(100, signal_strength);
100 } 66 }
101 67
102 TEST_F(DeviceStatusUtilTest, TestBelowMinValue) { 68 TEST_F(DeviceStatusUtilTest, TestBelowMinValue) {
103 DeviceStatus status = CreateFakeDeviceStatus( 69 DeviceStatus status = CreateTestDeviceStatus(
104 "cellProvider" /* cell_provider_name */, -1 /* battery_percentage */, 70 "cellProvider" /* cell_provider_name */, -1 /* battery_percentage */,
105 -1 /* connection_strength */); 71 -1 /* connection_strength */);
106 72
107 std::string carrier; 73 std::string carrier;
108 int32_t battery_percentage; 74 int32_t battery_percentage;
109 int32_t signal_strength; 75 int32_t signal_strength;
110 76
111 NormalizeDeviceStatus(status, &carrier, &battery_percentage, 77 NormalizeDeviceStatus(status, &carrier, &battery_percentage,
112 &signal_strength); 78 &signal_strength);
113 79
114 EXPECT_EQ("cellProvider", carrier); 80 EXPECT_EQ("cellProvider", carrier);
115 EXPECT_EQ(0, battery_percentage); 81 EXPECT_EQ(0, battery_percentage);
116 EXPECT_EQ(0, signal_strength); 82 EXPECT_EQ(0, signal_strength);
117 } 83 }
118 84
119 TEST_F(DeviceStatusUtilTest, TestAboveMaxValue) { 85 TEST_F(DeviceStatusUtilTest, TestAboveMaxValue) {
120 DeviceStatus status = CreateFakeDeviceStatus( 86 DeviceStatus status = CreateTestDeviceStatus(
121 "cellProvider" /* cell_provider_name */, 101 /* battery_percentage */, 87 "cellProvider" /* cell_provider_name */, 101 /* battery_percentage */,
122 5 /* connection_strength */); 88 5 /* connection_strength */);
123 89
124 std::string carrier; 90 std::string carrier;
125 int32_t battery_percentage; 91 int32_t battery_percentage;
126 int32_t signal_strength; 92 int32_t signal_strength;
127 93
128 NormalizeDeviceStatus(status, &carrier, &battery_percentage, 94 NormalizeDeviceStatus(status, &carrier, &battery_percentage,
129 &signal_strength); 95 &signal_strength);
130 96
131 EXPECT_EQ("cellProvider", carrier); 97 EXPECT_EQ("cellProvider", carrier);
132 EXPECT_EQ(100, battery_percentage); 98 EXPECT_EQ(100, battery_percentage);
133 EXPECT_EQ(100, signal_strength); 99 EXPECT_EQ(100, signal_strength);
134 } 100 }
135 101
136 TEST_F(DeviceStatusUtilTest, TestValidValues) { 102 TEST_F(DeviceStatusUtilTest, TestValidValues) {
137 DeviceStatus status = CreateFakeDeviceStatus( 103 DeviceStatus status = CreateTestDeviceStatus(
138 "cellProvider" /* cell_provider_name */, 50 /* battery_percentage */, 104 "cellProvider" /* cell_provider_name */, 50 /* battery_percentage */,
139 2 /* connection_strength */); 105 2 /* connection_strength */);
140 106
141 std::string carrier; 107 std::string carrier;
142 int32_t battery_percentage; 108 int32_t battery_percentage;
143 int32_t signal_strength; 109 int32_t signal_strength;
144 110
145 NormalizeDeviceStatus(status, &carrier, &battery_percentage, 111 NormalizeDeviceStatus(status, &carrier, &battery_percentage,
146 &signal_strength); 112 &signal_strength);
147 113
148 EXPECT_EQ("cellProvider", carrier); 114 EXPECT_EQ("cellProvider", carrier);
149 EXPECT_EQ(50, battery_percentage); 115 EXPECT_EQ(50, battery_percentage);
150 EXPECT_EQ(50, signal_strength); 116 EXPECT_EQ(50, signal_strength);
151 } 117 }
152 118
153 } // namespace tether 119 } // namespace tether
154 120
155 } // namespace cryptauth 121 } // namespace cryptauth
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698