| Index: chromeos/components/tether/ble_advertiser_unittest.cc
|
| diff --git a/chromeos/components/tether/ble_advertiser_unittest.cc b/chromeos/components/tether/ble_advertiser_unittest.cc
|
| index d8caac628c414b10fcd125da53ceeb1e1e25b863..78f3cc1f04f4ef9ae228f0bd1657113832e40fdf 100644
|
| --- a/chromeos/components/tether/ble_advertiser_unittest.cc
|
| +++ b/chromeos/components/tether/ble_advertiser_unittest.cc
|
| @@ -27,7 +27,7 @@ namespace tether {
|
| namespace {
|
| uint8_t kInvertedConnectionFlag = 0x01;
|
|
|
| -const std::string fake_public_key = "fakePublicKey";
|
| +const char kFakePublicKey[] = "fakePublicKey";
|
|
|
| struct RegisterAdvertisementArgs
|
| : public base::RefCounted<RegisterAdvertisementArgs> {
|
| @@ -80,15 +80,12 @@ class MockBluetoothAdapterWithAdvertisements
|
| ~MockBluetoothAdapterWithAdvertisements() override {}
|
| };
|
|
|
| -std::vector<cryptauth::ForegroundEidGenerator::DataWithTimestamp>
|
| -GenerateFakeAdvertisements() {
|
| - cryptauth::ForegroundEidGenerator::DataWithTimestamp advertisement1(
|
| - "advertisement1", 1000L, 2000L);
|
| - cryptauth::ForegroundEidGenerator::DataWithTimestamp advertisement2(
|
| - "advertisement2", 2000L, 3000L);
|
| +std::vector<cryptauth::DataWithTimestamp> GenerateFakeAdvertisements() {
|
| + cryptauth::DataWithTimestamp advertisement1("advertisement1", 1000L, 2000L);
|
| + cryptauth::DataWithTimestamp advertisement2("advertisement2", 2000L, 3000L);
|
|
|
| - std::vector<cryptauth::ForegroundEidGenerator::DataWithTimestamp>
|
| - advertisements = {advertisement1, advertisement2};
|
| + std::vector<cryptauth::DataWithTimestamp> advertisements = {advertisement1,
|
| + advertisement2};
|
| return advertisements;
|
| }
|
|
|
| @@ -106,7 +103,7 @@ std::vector<cryptauth::BeaconSeed> CreateFakeBeaconSeedsForDevice(
|
|
|
| std::vector<cryptauth::BeaconSeed> seeds = {seed1, seed2};
|
| return seeds;
|
| -};
|
| +}
|
|
|
| } // namespace
|
|
|
| @@ -178,7 +175,7 @@ class BleAdvertiserTest : public testing::Test {
|
|
|
| mock_local_data_provider_ = base::MakeUnique<MockLocalDeviceDataProvider>();
|
| mock_local_data_provider_->SetPublicKey(
|
| - base::MakeUnique<std::string>(fake_public_key));
|
| + base::MakeUnique<std::string>(kFakePublicKey));
|
|
|
| ble_advertiser_ = base::WrapUnique(new BleAdvertiser(
|
| mock_adapter_, base::WrapUnique(test_unregister_handler_),
|
| @@ -271,8 +268,7 @@ class BleAdvertiserTest : public testing::Test {
|
| individual_advertisements_;
|
|
|
| const std::vector<cryptauth::RemoteDevice> fake_devices_;
|
| - const std::vector<cryptauth::ForegroundEidGenerator::DataWithTimestamp>
|
| - fake_advertisements_;
|
| + const std::vector<cryptauth::DataWithTimestamp> fake_advertisements_;
|
|
|
| private:
|
| DISALLOW_COPY_AND_ASSIGN(BleAdvertiserTest);
|
| @@ -325,8 +321,7 @@ TEST_F(BleAdvertiserTest, AdapterPoweredOffWhenAdvertisementRegistered) {
|
| EXPECT_CALL(*mock_adapter_, IsPowered()).Times(1).WillOnce(Return(false));
|
|
|
| mock_eid_generator_->set_advertisement(
|
| - base::MakeUnique<cryptauth::ForegroundEidGenerator::DataWithTimestamp>(
|
| - fake_advertisements_[0]));
|
| + base::MakeUnique<cryptauth::DataWithTimestamp>(fake_advertisements_[0]));
|
|
|
| EXPECT_TRUE(ble_advertiser_->StartAdvertisingToDevice(fake_devices_[0]));
|
| EXPECT_EQ(static_cast<size_t>(1), individual_advertisements_.size());
|
| @@ -342,8 +337,7 @@ TEST_F(BleAdvertiserTest, RegisteringAdvertisementFails) {
|
| EXPECT_CALL(*mock_adapter_, RegisterAdvertisementWithArgsStruct(_)).Times(1);
|
|
|
| mock_eid_generator_->set_advertisement(
|
| - base::MakeUnique<cryptauth::ForegroundEidGenerator::DataWithTimestamp>(
|
| - fake_advertisements_[0]));
|
| + base::MakeUnique<cryptauth::DataWithTimestamp>(fake_advertisements_[0]));
|
|
|
| EXPECT_TRUE(ble_advertiser_->StartAdvertisingToDevice(fake_devices_[0]));
|
| EXPECT_EQ(static_cast<size_t>(1), individual_advertisements_.size());
|
| @@ -362,8 +356,7 @@ TEST_F(BleAdvertiserTest, AdvertisementRegisteredSuccessfully) {
|
| EXPECT_CALL(*mock_adapter_, RegisterAdvertisementWithArgsStruct(_)).Times(1);
|
|
|
| mock_eid_generator_->set_advertisement(
|
| - base::MakeUnique<cryptauth::ForegroundEidGenerator::DataWithTimestamp>(
|
| - fake_advertisements_[0]));
|
| + base::MakeUnique<cryptauth::DataWithTimestamp>(fake_advertisements_[0]));
|
|
|
| EXPECT_TRUE(ble_advertiser_->StartAdvertisingToDevice(fake_devices_[0]));
|
| EXPECT_EQ(static_cast<size_t>(1), individual_advertisements_.size());
|
| @@ -388,8 +381,7 @@ TEST_F(BleAdvertiserTest, AdvertisementRegisteredSuccessfully_TwoDevices) {
|
|
|
| // First device.
|
| mock_eid_generator_->set_advertisement(
|
| - base::MakeUnique<cryptauth::ForegroundEidGenerator::DataWithTimestamp>(
|
| - fake_advertisements_[0]));
|
| + base::MakeUnique<cryptauth::DataWithTimestamp>(fake_advertisements_[0]));
|
|
|
| EXPECT_TRUE(ble_advertiser_->StartAdvertisingToDevice(fake_devices_[0]));
|
| EXPECT_EQ(static_cast<size_t>(1), individual_advertisements_.size());
|
| @@ -402,8 +394,7 @@ TEST_F(BleAdvertiserTest, AdvertisementRegisteredSuccessfully_TwoDevices) {
|
|
|
| // Second device.
|
| mock_eid_generator_->set_advertisement(
|
| - base::MakeUnique<cryptauth::ForegroundEidGenerator::DataWithTimestamp>(
|
| - fake_advertisements_[1]));
|
| + base::MakeUnique<cryptauth::DataWithTimestamp>(fake_advertisements_[1]));
|
|
|
| EXPECT_TRUE(ble_advertiser_->StartAdvertisingToDevice(fake_devices_[1]));
|
| EXPECT_EQ(static_cast<size_t>(2), individual_advertisements_.size());
|
| @@ -433,8 +424,7 @@ TEST_F(BleAdvertiserTest, TooManyDevicesRegistered) {
|
| EXPECT_CALL(*mock_adapter_, RegisterAdvertisementWithArgsStruct(_)).Times(3);
|
|
|
| mock_eid_generator_->set_advertisement(
|
| - base::MakeUnique<cryptauth::ForegroundEidGenerator::DataWithTimestamp>(
|
| - fake_advertisements_[0]));
|
| + base::MakeUnique<cryptauth::DataWithTimestamp>(fake_advertisements_[0]));
|
|
|
| // Should succeed for the first two devices.
|
| EXPECT_TRUE(ble_advertiser_->StartAdvertisingToDevice(fake_devices_[0]));
|
| @@ -464,8 +454,7 @@ TEST_F(BleAdvertiserTest, AdapterPowerChange_StartsOffThenTurnsOn) {
|
| EXPECT_CALL(*mock_adapter_, RegisterAdvertisementWithArgsStruct(_)).Times(1);
|
|
|
| mock_eid_generator_->set_advertisement(
|
| - base::MakeUnique<cryptauth::ForegroundEidGenerator::DataWithTimestamp>(
|
| - fake_advertisements_[0]));
|
| + base::MakeUnique<cryptauth::DataWithTimestamp>(fake_advertisements_[0]));
|
|
|
| EXPECT_TRUE(ble_advertiser_->StartAdvertisingToDevice(fake_devices_[0]));
|
| EXPECT_EQ(static_cast<size_t>(1), individual_advertisements_.size());
|
| @@ -490,8 +479,7 @@ TEST_F(BleAdvertiserTest, AdvertisementReleased) {
|
| EXPECT_CALL(*mock_adapter_, RegisterAdvertisementWithArgsStruct(_)).Times(2);
|
|
|
| mock_eid_generator_->set_advertisement(
|
| - base::MakeUnique<cryptauth::ForegroundEidGenerator::DataWithTimestamp>(
|
| - fake_advertisements_[0]));
|
| + base::MakeUnique<cryptauth::DataWithTimestamp>(fake_advertisements_[0]));
|
|
|
| EXPECT_TRUE(ble_advertiser_->StartAdvertisingToDevice(fake_devices_[0]));
|
| EXPECT_EQ(static_cast<size_t>(1), individual_advertisements_.size());
|
| @@ -520,4 +508,4 @@ TEST_F(BleAdvertiserTest, AdvertisementReleased) {
|
|
|
| } // namespace tether
|
|
|
| -} // namespace cryptauth
|
| +} // namespace chromeos
|
|
|