Bluetooth: macOS: DidModifyServices can happens while scanning

device/bluetooth/test/bluetooth_test_mac.h

 // Copyright 2015 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.
 
 #ifndef DEVICE_BLUETOOTH_TEST_BLUETOOTH_TEST_MAC_H_
 #define DEVICE_BLUETOOTH_TEST_BLUETOOTH_TEST_MAC_H_
 
 #include <memory>
 
 #include "base/test/test_simple_task_runner.h"
@@ skipping 73 matching lines @@
       const std::vector<uint8_t>& value) override;
   void SimulateGattCharacteristicRemoved(
       BluetoothRemoteGattService* service,
       BluetoothRemoteGattCharacteristic* characteristic) override;
   void ExpectedChangeNotifyValueAttempts(int attempts) override;
   void ExpectedNotifyValue(NotifyValueState expected_value_state) override;
 
   // macOS is the only platform for which we need to discover each set of
   // attributes individually so we need a method to simulate discovering each
   // set of attributes.
-  void SimulateDidDiscoverServices(BluetoothDevice* device,
-                                   const std::vector<std::string>& uuids);
+  // Simulates service discovery for a device.
+  void SimulateDidDiscoverServices(BluetoothDevice* device);
+  // Simulates characteristic discovery for a service.
+  void SimulateDidDiscoverCharacteristics(BluetoothRemoteGattService* service);
+  // Simulates descriptor discovery for a characteristic.
+  void SimulateDidDiscoverDescriptors(
+      BluetoothRemoteGattCharacteristic* characteristic);
+  // Adds services in MockCBPeripheral.
+  void AddServicesToDevice(BluetoothDevice* device,
+                           const std::vector<std::string>& uuids);
+  // Adds a characteristic in MockCBService.
+  void AddCharacteristicToService(BluetoothRemoteGattService* service,
+                                  const std::string& characteristic_uuid,
+                                  int properties);
+  // Adds a descriptor in MockCBCharacteristic.
+  void AddDescriptorToCharacteristic(
+      BluetoothRemoteGattCharacteristic* characteristic,
+      const std::string& uuid);
 
   // Callback for the bluetooth central manager mock.
   void OnFakeBluetoothDeviceConnectGattCalled();
   void OnFakeBluetoothGattDisconnect();
 
   // Callback for the bluetooth peripheral mock.
   void OnFakeBluetoothServiceDiscovery();
   void OnFakeBluetoothCharacteristicDiscovery();
   void OnFakeBluetoothCharacteristicReadValue();
   void OnFakeBluetoothCharacteristicWriteValue(std::vector<uint8_t> value);
@@ skipping 11 matching lines @@
   MockCBPeripheral* GetMockCBPeripheral(BluetoothDevice* device) const;
   // Returns MockCBPeripheral from BluetoothRemoteGattService.
   MockCBPeripheral* GetMockCBPeripheral(
       BluetoothRemoteGattService* service) const;
   // Returns MockCBPeripheral from BluetoothRemoteGattCharacteristic.
   MockCBPeripheral* GetMockCBPeripheral(
       BluetoothRemoteGattCharacteristic* characteristic) const;
   // Returns MockCBCharacteristic from BluetoothRemoteGattCharacteristic.
   MockCBCharacteristic* GetCBMockCharacteristic(
       BluetoothRemoteGattCharacteristic* characteristic) const;
-  // Adds services in MockCBPeripheral.
-  void AddServicesToDevice(BluetoothDevice* device,
-                           const std::vector<std::string>& uuids);
 
   // Utility function for finding CBUUIDs with relatively nice SHA256 hashes.
   std::string FindCBUUIDForHashTarget();
 
   BluetoothAdapterMac* adapter_mac_ = nullptr;
   std::unique_ptr<ScopedMockCentralManager> mock_central_manager_;
 
   // Value set by -[CBPeripheral setNotifyValue:forCharacteristic:] call.
   bool last_notify_value_ = false;
   int gatt_characteristic_discovery_attempts_ = 0;
 };
 
 // Defines common test fixture name. Use TEST_F(BluetoothTest, YourTestName).
 typedef BluetoothTestMac BluetoothTest;
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_TEST_BLUETOOTH_TEST_MAC_H_