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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: device/bluetooth/bluetooth_remote_gatt_service_unittest.cc

Issue 2798193002: Use test UUIDs for device bluetooth unit tests (Closed)
Patch Set: use test UUIDs for device bluetooth unit tests Created 3 years, 8 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
« device/bluetooth/bluetooth_device_unittest.cc ('K') | « device/bluetooth/bluetooth_remote_gatt_descriptor_unittest.cc ('k') | device/bluetooth/test/bluetooth_test.h » ('j') | device/bluetooth/test/bluetooth_test.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: device/bluetooth/bluetooth_remote_gatt_service_unittest.cc
diff --git a/device/bluetooth/bluetooth_remote_gatt_service_unittest.cc b/device/bluetooth/bluetooth_remote_gatt_service_unittest.cc
index c5118645f99ade3e2ed65c1bc579aa36082184fd..33f505a33fc8aa9a7324f168ded69dfe0e48cdf2 100644
--- a/device/bluetooth/bluetooth_remote_gatt_service_unittest.cc
+++ b/device/bluetooth/bluetooth_remote_gatt_service_unittest.cc
@@ -44,9 +44,8 @@ TEST_F(BluetoothRemoteGattServiceTest, GetIdentifier) {
// 2 duplicate UUIDs creating 2 service instances on each device.
std::vector<std::string> services;
- std::string uuid = "00000000-0000-1000-8000-00805f9b34fb";
- services.push_back(uuid);
- services.push_back(uuid);
+ services.push_back(kTestUUIDGenericAccess);
+ services.push_back(kTestUUIDGenericAccess);
SimulateGattServicesDiscovered(device1, services);
SimulateGattServicesDiscovered(device2, services);
base::RunLoop().RunUntilIdle();
@@ -82,7 +81,7 @@ TEST_F(BluetoothRemoteGattServiceTest, GetUUID) {
base::RunLoop().RunUntilIdle();
// Create multiple instances with the same UUID.
- BluetoothUUID uuid("00000000-0000-1000-8000-00805f9b34fb");
+ BluetoothUUID uuid(kTestUUIDGenericAccess);
std::vector<std::string> services;
services.push_back(uuid.canonical_value());
services.push_back(uuid.canonical_value());
@@ -111,7 +110,7 @@ TEST_F(BluetoothRemoteGattServiceTest, GetCharacteristics_FindNone) {
// Simulate a service, with no Characteristics:
std::vector<std::string> services;
- services.push_back("00000000-0000-1000-8000-00805f9b34fb");
+ services.push_back(kTestUUIDGenericAccess);
SimulateGattServicesDiscovered(device, services);
base::RunLoop().RunUntilIdle();
BluetoothRemoteGattService* service = device->GetGattServices()[0];
@@ -137,18 +136,18 @@ TEST_F(BluetoothRemoteGattServiceTest,
// Simulate a service, with several Characteristics:
std::vector<std::string> services;
- services.push_back("00000000-0000-1000-8000-00805f9b34fb");
+ services.push_back(kTestUUIDGenericAccess);
SimulateGattServicesDiscovered(device, services);
base::RunLoop().RunUntilIdle();
BluetoothRemoteGattService* service = device->GetGattServices()[0];
- std::string characteristic_uuid1 = "11111111-0000-1000-8000-00805f9b34fb";
- std::string characteristic_uuid2 = "22222222-0000-1000-8000-00805f9b34fb";
- std::string characteristic_uuid3 = characteristic_uuid2; // Duplicate UUID.
- std::string characteristic_uuid4 = "33333333-0000-1000-8000-00805f9b34fb";
- SimulateGattCharacteristic(service, characteristic_uuid1, /* properties */ 0);
- SimulateGattCharacteristic(service, characteristic_uuid2, /* properties */ 0);
- SimulateGattCharacteristic(service, characteristic_uuid3, /* properties */ 0);
- SimulateGattCharacteristic(service, characteristic_uuid4, /* properties */ 0);
+ SimulateGattCharacteristic(service, kTestUUIDDeviceName, /* properties */ 0);
+ SimulateGattCharacteristic(service, kTestUUIDAppearance,
+ /* properties */ 0);
+ // Duplicate UUID.
+ SimulateGattCharacteristic(service, kTestUUIDAppearance,
+ /* properties */ 0);
+ SimulateGattCharacteristic(service, kTestUUIDReconnectionAddress,
+ /* properties */ 0);
// Verify that GetCharacteristic can retrieve characteristics again by ID,
// and that the same Characteristics come back.
@@ -189,37 +188,36 @@ TEST_F(BluetoothRemoteGattServiceTest, GetCharacteristicsByUUID) {
// Simulate two primary GATT services.
std::vector<std::string> services;
- services.push_back("00000000-0000-1000-8000-00805f9b34fb");
- services.push_back("01010101-0101-1000-8000-00805f9b34fb");
+ services.push_back(kTestUUIDGenericAccess);
+ services.push_back(kTestUUIDHeartRate);
SimulateGattServicesDiscovered(device, services);
base::RunLoop().RunUntilIdle();
BluetoothRemoteGattService* service1 = device->GetGattServices()[0];
BluetoothRemoteGattService* service2 = device->GetGattServices()[1];
- std::string characteristic_uuid1 = "11111111-0000-1000-8000-00805f9b34fb";
- std::string characteristic_uuid2 = "22222222-0000-1000-8000-00805f9b34fb";
- SimulateGattCharacteristic(service1, characteristic_uuid1,
+ SimulateGattCharacteristic(service1, kTestUUIDDeviceName,
/* properties */ 0);
// 2 duplicate UUIDs creating 2 instances.
- SimulateGattCharacteristic(service2, characteristic_uuid2,
+ SimulateGattCharacteristic(service2, kTestUUIDHeartRateMeasurement,
/* properties */ 0);
- SimulateGattCharacteristic(service2, characteristic_uuid2,
+ SimulateGattCharacteristic(service2, kTestUUIDHeartRateMeasurement,
/* properties */ 0);
{
std::vector<BluetoothRemoteGattCharacteristic*> characteristics =
- service1->GetCharacteristicsByUUID(BluetoothUUID(characteristic_uuid1));
+ service1->GetCharacteristicsByUUID(BluetoothUUID(kTestUUIDDeviceName));
EXPECT_EQ(1u, characteristics.size());
- EXPECT_EQ(characteristic_uuid1,
+ EXPECT_EQ(kTestUUIDDeviceName,
characteristics[0]->GetUUID().canonical_value());
}
{
std::vector<BluetoothRemoteGattCharacteristic*> characteristics =
- service2->GetCharacteristicsByUUID(BluetoothUUID(characteristic_uuid2));
+ service2->GetCharacteristicsByUUID(
+ BluetoothUUID(kTestUUIDHeartRateMeasurement));
EXPECT_EQ(2u, characteristics.size());
- EXPECT_EQ(characteristic_uuid2,
+ EXPECT_EQ(kTestUUIDHeartRateMeasurement,
characteristics[0]->GetUUID().canonical_value());
- EXPECT_EQ(characteristic_uuid2,
+ EXPECT_EQ(kTestUUIDHeartRateMeasurement,
characteristics[1]->GetUUID().canonical_value());
EXPECT_NE(characteristics[0]->GetIdentifier(),
characteristics[1]->GetIdentifier());
@@ -254,18 +252,18 @@ TEST_F(BluetoothRemoteGattServiceTest, GattCharacteristics_ObserversCalls) {
// Simulate a service, with several Characteristics:
std::vector<std::string> services;
- services.push_back("00000000-0000-1000-8000-00805f9b34fb");
+ services.push_back(kTestUUIDGenericAccess);
SimulateGattServicesDiscovered(device, services);
base::RunLoop().RunUntilIdle();
BluetoothRemoteGattService* service = device->GetGattServices()[0];
- std::string characteristic_uuid1 = "11111111-0000-1000-8000-00805f9b34fb";
- std::string characteristic_uuid2 = "22222222-0000-1000-8000-00805f9b34fb";
- std::string characteristic_uuid3 = characteristic_uuid2; // Duplicate UUID.
- std::string characteristic_uuid4 = "33333333-0000-1000-8000-00805f9b34fb";
- SimulateGattCharacteristic(service, characteristic_uuid1, /* properties */ 0);
- SimulateGattCharacteristic(service, characteristic_uuid2, /* properties */ 0);
- SimulateGattCharacteristic(service, characteristic_uuid3, /* properties */ 0);
- SimulateGattCharacteristic(service, characteristic_uuid4, /* properties */ 0);
+ SimulateGattCharacteristic(service, kTestUUIDDeviceName, /* properties */ 0);
+ SimulateGattCharacteristic(service, kTestUUIDAppearance,
+ /* properties */ 0);
+ // Duplicate UUID.
+ SimulateGattCharacteristic(service, kTestUUIDAppearance,
+ /* properties */ 0);
+ SimulateGattCharacteristic(service, kTestUUIDReconnectionAddress,
+ /* properties */ 0);
#if !defined(OS_WIN)
// TODO(620895) GattCharacteristicAdded has to be implemented for Windows.
EXPECT_EQ(4, observer.gatt_characteristic_added_count());
@@ -327,8 +325,8 @@ TEST_F(BluetoothRemoteGattServiceTest, SimulateGattServiceRemove) {
// Simulate two primary GATT services.
std::vector<std::string> services;
- services.push_back("00000000-0000-1000-8000-00805f9b34fb");
- services.push_back("01010101-0101-1000-8000-00805f9b34fb");
+ services.push_back(kTestUUIDGenericAccess);
+ services.push_back(kTestUUIDHeartRate);
SimulateGattServicesDiscovered(device, services);
EXPECT_EQ(2u, device->GetGattServices().size());
« device/bluetooth/bluetooth_device_unittest.cc ('K') | « device/bluetooth/bluetooth_remote_gatt_descriptor_unittest.cc ('k') | device/bluetooth/test/bluetooth_test.h » ('j') | device/bluetooth/test/bluetooth_test.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698