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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: device/bluetooth/bluetooth_audio_sink_chromeos_unittest.cc

Issue 939753004: device/bluetooth: Implement Unregister() of BlueotoothAudioSinkChromeOS and disconnection-related c… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changed volume_ back to uint16_t with a defined range. Created 5 years, 10 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_audio_sink_chromeos.cc ('K') | « device/bluetooth/bluetooth_audio_sink_chromeos.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: device/bluetooth/bluetooth_audio_sink_chromeos_unittest.cc
diff --git a/device/bluetooth/bluetooth_audio_sink_chromeos_unittest.cc b/device/bluetooth/bluetooth_audio_sink_chromeos_unittest.cc
index 9aa727f08b4c862a8e40bf8df65961c8bb2c5b6e..e1db0be2f27d4124f6923574247ef0fe11ddf33b 100644
--- a/device/bluetooth/bluetooth_audio_sink_chromeos_unittest.cc
+++ b/device/bluetooth/bluetooth_audio_sink_chromeos_unittest.cc
@@ -81,12 +81,20 @@ class BluetoothAudioSinkChromeOSTest : public testing::Test {
properties_.volume.reset(
new uint16_t(FakeBluetoothMediaTransportClient::kTransportVolume));
+ media_ = static_cast<FakeBluetoothMediaClient*>(
+ DBusThreadManager::Get()->GetBluetoothMediaClient());
+ transport_ = static_cast<FakeBluetoothMediaTransportClient*>(
+ DBusThreadManager::Get()->GetBluetoothMediaTransportClient());
+
GetAdapter();
}
void TearDown() override {
callback_count_ = 0;
error_callback_count_ = 0;
+ observer_.reset();
+
+ media_->SetVisible(true);
// The adapter should outlive audio sink.
audio_sink_ = nullptr;
@@ -138,8 +146,20 @@ class BluetoothAudioSinkChromeOSTest : public testing::Test {
base::Bind(&BluetoothAudioSinkChromeOSTest::RegisterErrorCallback,
base::Unretained(this)));
+ observer_.reset(new TestAudioSinkObserver(audio_sink_));
EXPECT_EQ(callback_count_, 1);
EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+ }
+
+ void GetFakeMediaEndpoint() {
+ BluetoothAudioSinkChromeOS* audio_sink_chromeos =
+ static_cast<BluetoothAudioSinkChromeOS*>(audio_sink_.get());
+ ASSERT_NE(audio_sink_chromeos, nullptr);
+
+ media_endpoint_ = static_cast<FakeBluetoothMediaEndpointServiceProvider*>(
+ audio_sink_chromeos->GetEndpointServiceProvider());
}
// Called whenever RegisterAudioSink is completed successfully.
@@ -147,6 +167,11 @@ class BluetoothAudioSinkChromeOSTest : public testing::Test {
scoped_refptr<BluetoothAudioSink> audio_sink) {
++callback_count_;
audio_sink_ = audio_sink;
+
+ GetFakeMediaEndpoint();
+ ASSERT_NE(media_endpoint_, nullptr);
+ media_->SetEndpointRegistered(media_endpoint_, true);
+
ASSERT_NE(audio_sink_.get(), nullptr);
ASSERT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
}
@@ -166,6 +191,11 @@ class BluetoothAudioSinkChromeOSTest : public testing::Test {
EXPECT_EQ(capabilities, std::vector<uint8_t>({0x3f, 0xff, 0x12, 0x35}));
}
+ void UnregisterErrorCallback(BluetoothAudioSink::ErrorCode error_code) {
+ ++error_callback_count_;
+ EXPECT_EQ(error_code, BluetoothAudioSink::ERROR_NOT_UNREGISTERED);
+ }
+
// Generic callbacks.
void Callback() {
++callback_count_;
@@ -181,6 +211,10 @@ class BluetoothAudioSinkChromeOSTest : public testing::Test {
base::MessageLoop message_loop_;
+ FakeBluetoothMediaClient* media_;
+ FakeBluetoothMediaTransportClient* transport_;
+ FakeBluetoothMediaEndpointServiceProvider* media_endpoint_;
+ scoped_ptr<TestAudioSinkObserver> observer_;
scoped_refptr<BluetoothAdapter> adapter_;
scoped_refptr<BluetoothAudioSink> audio_sink_;
@@ -192,11 +226,6 @@ class BluetoothAudioSinkChromeOSTest : public testing::Test {
TEST_F(BluetoothAudioSinkChromeOSTest, RegisterSucceeded) {
GetAudioSink();
-
- // Adds an observer for |audio_sink_|.
- TestAudioSinkObserver observer(audio_sink_);
- EXPECT_EQ(observer.state_changed_count_, 0);
- EXPECT_EQ(observer.volume_changed_count_, 0);
}
TEST_F(BluetoothAudioSinkChromeOSTest, RegisterFailedWithInvalidOptions) {
@@ -232,21 +261,9 @@ TEST_F(BluetoothAudioSinkChromeOSTest, RegisterFailedWithInvalidOptions) {
TEST_F(BluetoothAudioSinkChromeOSTest, SelectConfiguration) {
GetAudioSink();
- // Adds an observer for |audio_sink_|.
- TestAudioSinkObserver observer(audio_sink_);
- EXPECT_EQ(observer.state_changed_count_, 0);
- EXPECT_EQ(observer.volume_changed_count_, 0);
-
// Simulates calling SelectConfiguration on the media endpoint object owned by
// |audio_sink_| with some fake capabilities.
- BluetoothAudioSinkChromeOS* audio_sink_chromeos =
- static_cast<BluetoothAudioSinkChromeOS*>(audio_sink_.get());
- FakeBluetoothMediaEndpointServiceProvider* media_endpoint =
- static_cast<FakeBluetoothMediaEndpointServiceProvider*>(
- audio_sink_chromeos->GetEndpointServiceProvider());
- ASSERT_NE(media_endpoint, nullptr);
-
- media_endpoint->SelectConfiguration(
+ media_endpoint_->SelectConfiguration(
std::vector<uint8_t>({0x21, 0x15, 0x33, 0x2C}),
base::Bind(&BluetoothAudioSinkChromeOSTest::SelectConfigurationCallback,
base::Unretained(this)));
@@ -254,138 +271,304 @@ TEST_F(BluetoothAudioSinkChromeOSTest, SelectConfiguration) {
EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
EXPECT_EQ(callback_count_, 2);
EXPECT_EQ(error_callback_count_, 0);
- EXPECT_EQ(observer.state_changed_count_, 0);
- EXPECT_EQ(observer.volume_changed_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
}
TEST_F(BluetoothAudioSinkChromeOSTest, SetConfiguration) {
GetAudioSink();
- // Adds an observer for |audio_sink_|.
- TestAudioSinkObserver observer(audio_sink_);
- EXPECT_EQ(observer.state_changed_count_, 0);
- EXPECT_EQ(observer.volume_changed_count_, 0);
+ media_endpoint_->SelectConfiguration(
+ std::vector<uint8_t>({0x21, 0x15, 0x33, 0x2C}),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::SelectConfigurationCallback,
+ base::Unretained(this)));
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
// Simulates calling SetConfiguration on the media endpoint object owned by
// |audio_sink_| with a fake transport path and a
// Delegate::TransportProperties structure.
- BluetoothAudioSinkChromeOS* audio_sink_chromeos =
- static_cast<BluetoothAudioSinkChromeOS*>(audio_sink_.get());
- FakeBluetoothMediaEndpointServiceProvider* media_endpoint =
- static_cast<FakeBluetoothMediaEndpointServiceProvider*>(
- audio_sink_chromeos->GetEndpointServiceProvider());
- ASSERT_NE(media_endpoint, nullptr);
-
- media_endpoint->SetConfiguration(
- ObjectPath(FakeBluetoothMediaTransportClient::kTransportPath),
+ media_endpoint_->SetConfiguration(
+ transport_->GetTransportPath(media_endpoint_->object_path()),
properties_);
EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_IDLE);
- EXPECT_EQ(callback_count_, 1);
+ EXPECT_EQ(callback_count_, 2);
EXPECT_EQ(error_callback_count_, 0);
- EXPECT_EQ(observer.state_changed_count_, 1);
- EXPECT_EQ(observer.volume_changed_count_, 1);
+ EXPECT_EQ(observer_->state_changed_count_, 1);
+ EXPECT_EQ(observer_->volume_changed_count_, 1);
}
TEST_F(BluetoothAudioSinkChromeOSTest, SetConfigurationWithUnexpectedState) {
GetAudioSink();
- // Adds an observer for |audio_sink_|.
- TestAudioSinkObserver observer(audio_sink_);
- EXPECT_EQ(observer.state_changed_count_, 0);
- EXPECT_EQ(observer.volume_changed_count_, 0);
+ media_endpoint_->SelectConfiguration(
+ std::vector<uint8_t>({0x21, 0x15, 0x33, 0x2C}),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::SelectConfigurationCallback,
+ base::Unretained(this)));
- // Simulates calling SetConfiguration on the media endpoint object owned by
- // |audio_sink_| with a fake transport path and a
- // Delegate::TransportProperties structure.
- BluetoothAudioSinkChromeOS* audio_sink_chromeos =
- static_cast<BluetoothAudioSinkChromeOS*>(audio_sink_.get());
- FakeBluetoothMediaEndpointServiceProvider* media_endpoint =
- static_cast<FakeBluetoothMediaEndpointServiceProvider*>(
- audio_sink_chromeos->GetEndpointServiceProvider());
- ASSERT_NE(media_endpoint, nullptr);
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
// Set state of Delegate::TransportProperties with an unexpected value.
- properties_.state = "active";
+ properties_.state = "pending";
- media_endpoint->SetConfiguration(
- ObjectPath(FakeBluetoothMediaTransportClient::kTransportPath),
+ media_endpoint_->SetConfiguration(
+ transport_->GetTransportPath(media_endpoint_->object_path()),
properties_);
EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
- EXPECT_EQ(callback_count_, 1);
+ EXPECT_EQ(callback_count_, 2);
EXPECT_EQ(error_callback_count_, 0);
- EXPECT_EQ(observer.state_changed_count_, 0);
- EXPECT_EQ(observer.volume_changed_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
}
-// Checks if the observer is notified while the media object is
-// removed(invisible). Once the media object is removed, the audio sink is no
-// longer valid.
-TEST_F(BluetoothAudioSinkChromeOSTest, ObserverNotifiedWhenMediaRemoved) {
+// TODO(mcchou): Adds test on media-removed events for STATE_PENDING and
+// STATE_ACTIVE.
+// Checks if the observer is notified on media-removed event when the state of
+// |audio_sink_| is STATE_DISCONNECTED. Once the media object is removed, the
+// audio sink is no longer valid.
+TEST_F(BluetoothAudioSinkChromeOSTest, MediaRemovedDuringDisconnectedState) {
GetAudioSink();
- // Adds an observer for |audio_sink_|.
- TestAudioSinkObserver observer(audio_sink_);
- EXPECT_EQ(observer.state_changed_count_, 0);
- EXPECT_EQ(observer.volume_changed_count_, 0);
+ // Gets the media object and makes it invisible to see if the state of the
+ // audio sink changes accordingly.
+ media_->SetVisible(false);
+
+ GetFakeMediaEndpoint();
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_INVALID);
+ EXPECT_EQ(media_endpoint_, nullptr);
+ EXPECT_EQ(observer_->state_changed_count_, 1);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+}
+
+// Checks if the observer is notified on media-removed event when the state of
+// |audio_sink_| is STATE_IDLE. Once the media object is removed, the audio sink
+// is no longer valid.
+TEST_F(BluetoothAudioSinkChromeOSTest, MediaRemovedDuringIdleState) {
+ GetAudioSink();
+
+ media_endpoint_->SelectConfiguration(
+ std::vector<uint8_t>({0x21, 0x15, 0x33, 0x2C}),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::SelectConfigurationCallback,
+ base::Unretained(this)));
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+
+ media_endpoint_->SetConfiguration(
+ transport_->GetTransportPath(media_endpoint_->object_path()),
+ properties_);
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_IDLE);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 1);
+ EXPECT_EQ(observer_->volume_changed_count_, 1);
// Gets the media object and makes it invisible to see if the state of the
// audio sink changes accordingly.
- FakeBluetoothMediaClient* media = static_cast<FakeBluetoothMediaClient*>(
- DBusThreadManager::Get()->GetBluetoothMediaClient());
- media->SetVisible(false);
+ media_->SetVisible(false);
- BluetoothAudioSinkChromeOS* audio_sink_chromeos =
- static_cast<BluetoothAudioSinkChromeOS*>(audio_sink_.get());
- FakeBluetoothMediaEndpointServiceProvider* media_endpoint =
- static_cast<FakeBluetoothMediaEndpointServiceProvider*>(
- audio_sink_chromeos->GetEndpointServiceProvider());
+ GetFakeMediaEndpoint();
EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_INVALID);
- EXPECT_EQ(media_endpoint, nullptr);
+ EXPECT_EQ(media_endpoint_, nullptr);
+
+ // The state becomes disconnted and then invalid, since the removal of
+ // transport object should happend before media becomes invisible.
+ // State: STATE_IDLE -> STATE_DISCONNECTED -> STATE_INVALID
+ EXPECT_EQ(observer_->state_changed_count_, 3);
+ EXPECT_EQ(observer_->volume_changed_count_, 2);
}
-// Checks if the observer is notified while the media transport is
-// removed(invisible). Once the media transport object is removed, the audio
-// sink is disconnected.
-TEST_F(BluetoothAudioSinkChromeOSTest, ObserverNotifiedWhenTransportRemoved) {
+// TODO(mcchou): Add tests on transport-removed event for STATE_PENDING and
+// STATE_ACTIVE.
+// Checks if the observer is notified on transport-removed event when the state
+// of |audio_sink_| is STATE_IDEL. Once the media transport object is removed,
+// the audio sink is disconnected.
+TEST_F(BluetoothAudioSinkChromeOSTest, TransportRemovedDuringIdleState) {
GetAudioSink();
- // Adds an observer for |audio_sink_|.
- TestAudioSinkObserver observer(audio_sink_);
- EXPECT_EQ(observer.state_changed_count_, 0);
- EXPECT_EQ(observer.volume_changed_count_, 0);
+ media_endpoint_->SelectConfiguration(
+ std::vector<uint8_t>({0x21, 0x15, 0x33, 0x2C}),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::SelectConfigurationCallback,
+ base::Unretained(this)));
- // Simulates calling SetConfiguration on the media endpoint object owned by
- // |audio_sink_| with a fake transport path and a
- // Delegate::TransportProperties structure.
- BluetoothAudioSinkChromeOS* audio_sink_chromeos =
- static_cast<BluetoothAudioSinkChromeOS*>(audio_sink_.get());
- FakeBluetoothMediaEndpointServiceProvider* media_endpoint =
- static_cast<FakeBluetoothMediaEndpointServiceProvider*>(
- audio_sink_chromeos->GetEndpointServiceProvider());
- ASSERT_NE(media_endpoint, nullptr);
-
- media_endpoint->SetConfiguration(
- ObjectPath(FakeBluetoothMediaTransportClient::kTransportPath),
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+
+ media_endpoint_->SetConfiguration(
+ transport_->GetTransportPath(media_endpoint_->object_path()),
+ properties_);
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_IDLE);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 1);
+ EXPECT_EQ(observer_->volume_changed_count_, 1);
+
+ // Makes the transport object invalid to see if the state of the audio sink
+ // changes accordingly.
+ transport_->SetValid(media_endpoint_, false);
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_NE(media_endpoint_, nullptr);
+ EXPECT_EQ(observer_->state_changed_count_, 2);
+ EXPECT_EQ(observer_->volume_changed_count_, 2);
+}
+
+TEST_F(BluetoothAudioSinkChromeOSTest,
+ AdapterPoweredChangedDuringDisconnectedState) {
+ GetAudioSink();
+
+ adapter_->SetPowered(
+ false,
+ base::Bind(&BluetoothAudioSinkChromeOSTest::Callback,
+ base::Unretained(this)),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::ErrorCallback,
+ base::Unretained(this)));
+
+ EXPECT_TRUE(adapter_->IsPresent());
+ EXPECT_FALSE(adapter_->IsPowered());
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+
+ adapter_->SetPowered(
+ true,
+ base::Bind(&BluetoothAudioSinkChromeOSTest::Callback,
+ base::Unretained(this)),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::ErrorCallback,
+ base::Unretained(this)));
+
+ EXPECT_TRUE(adapter_->IsPresent());
+ EXPECT_TRUE(adapter_->IsPowered());
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_EQ(callback_count_, 3);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+}
+
+TEST_F(BluetoothAudioSinkChromeOSTest, AdapterPoweredChangedDuringIdleState) {
+ GetAudioSink();
+
+ media_endpoint_->SelectConfiguration(
+ std::vector<uint8_t>({0x21, 0x15, 0x33, 0x2C}),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::SelectConfigurationCallback,
+ base::Unretained(this)));
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+
+ media_endpoint_->SetConfiguration(
+ transport_->GetTransportPath(media_endpoint_->object_path()),
properties_);
EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_IDLE);
- EXPECT_EQ(callback_count_, 1);
+ EXPECT_EQ(callback_count_, 2);
EXPECT_EQ(error_callback_count_, 0);
- EXPECT_EQ(observer.state_changed_count_, 1);
- EXPECT_EQ(observer.volume_changed_count_, 1);
+ EXPECT_EQ(observer_->state_changed_count_, 1);
+ EXPECT_EQ(observer_->volume_changed_count_, 1);
- // Gets the media transport object and makes it invalid to see if the state
- // of the audio sink changes accordingly.
- FakeBluetoothMediaTransportClient* transport =
- static_cast<FakeBluetoothMediaTransportClient*>(
- DBusThreadManager::Get()->GetBluetoothMediaTransportClient());
+ adapter_->SetPowered(
+ false,
+ base::Bind(&BluetoothAudioSinkChromeOSTest::Callback,
+ base::Unretained(this)),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::ErrorCallback,
+ base::Unretained(this)));
+ GetFakeMediaEndpoint();
- transport->SetValid(media_endpoint->object_path(), false);
+ EXPECT_TRUE(adapter_->IsPresent());
+ EXPECT_FALSE(adapter_->IsPowered());
+ EXPECT_NE(media_endpoint_, nullptr);
EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
- EXPECT_NE(media_endpoint, nullptr);
+ EXPECT_EQ(callback_count_, 3);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 2);
+ EXPECT_EQ(observer_->volume_changed_count_, 2);
+}
+
+// TODO(mcchou): Add tests on UnregisterAudioSink for STATE_PENDING and
+// STATE_ACTIVE.
+
+TEST_F(BluetoothAudioSinkChromeOSTest,
+ UnregisterAudioSinkDuringDisconnectedState) {
+ GetAudioSink();
+
+ audio_sink_->Unregister(
+ base::Bind(&BluetoothAudioSinkChromeOSTest::Callback,
+ base::Unretained(this)),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::UnregisterErrorCallback,
+ base::Unretained(this)));
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_INVALID);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 1);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+}
+
+TEST_F(BluetoothAudioSinkChromeOSTest, UnregisterAudioSinkDuringIdleState) {
+ GetAudioSink();
+
+ media_endpoint_->SelectConfiguration(
+ std::vector<uint8_t>({0x21, 0x15, 0x33, 0x2C}),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::SelectConfigurationCallback,
+ base::Unretained(this)));
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_DISCONNECTED);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 0);
+ EXPECT_EQ(observer_->volume_changed_count_, 0);
+
+ media_endpoint_->SetConfiguration(
+ transport_->GetTransportPath(media_endpoint_->object_path()),
+ properties_);
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_IDLE);
+ EXPECT_EQ(callback_count_, 2);
+ EXPECT_EQ(error_callback_count_, 0);
+ EXPECT_EQ(observer_->state_changed_count_, 1);
+ EXPECT_EQ(observer_->volume_changed_count_, 1);
+
+ audio_sink_->Unregister(
+ base::Bind(&BluetoothAudioSinkChromeOSTest::Callback,
+ base::Unretained(this)),
+ base::Bind(&BluetoothAudioSinkChromeOSTest::UnregisterErrorCallback,
+ base::Unretained(this)));
+
+ EXPECT_EQ(audio_sink_->GetState(), BluetoothAudioSink::STATE_INVALID);
+ EXPECT_EQ(callback_count_, 3);
+ EXPECT_EQ(error_callback_count_, 0);
+
+ // The state becomes disconnted and then invalid, since the removal of
+ // transport object should happend before the unregistration of endpoint.
+ // State: STATE_IDLE -> STATE_DISCONNECTED -> STATE_INVALID
+ EXPECT_EQ(observer_->state_changed_count_, 3);
+ EXPECT_EQ(observer_->volume_changed_count_, 2);
}
} // namespace chromeos
« device/bluetooth/bluetooth_audio_sink_chromeos.cc ('K') | « device/bluetooth/bluetooth_audio_sink_chromeos.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698