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

Side by Side Diff: device/bluetooth/bluetooth_adapter_android_unittest.cc

Issue 1129683002: bluetooth: Android adapter can be created with and without Bluetooth permission. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@bta-jni-
Patch Set: Created 5 years, 7 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "base/memory/ref_counted.h" 5 #include "base/memory/ref_counted.h"
6 #include "device/bluetooth/bluetooth_adapter_android.h" 6 #include "device/bluetooth/bluetooth_adapter_android.h"
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 namespace device { 9 namespace device {
10 10
11 class BluetoothAdapterAndroidTest : public testing::Test { 11 class BluetoothAdapterAndroidTest : public testing::Test {
12 protected: 12 protected:
13 BluetoothAdapterAndroidTest() { 13 void InitWithPermission() {
14 adapter_ = BluetoothAdapterAndroid::CreateAdapter().get(); 14 adapter_ = BluetoothAdapterAndroid::CreateAdapter().get();
15 } 15 }
16 16
17 void InitWithoutPermission() {
18 adapter_ =
19 BluetoothAdapterAndroid::CreateAdapterWithoutPermissionForTesting()
20 .get();
21 }
22
17 scoped_refptr<BluetoothAdapterAndroid> adapter_; 23 scoped_refptr<BluetoothAdapterAndroid> adapter_;
18 }; 24 };
19 25
20 TEST_F(BluetoothAdapterAndroidTest, Construct) { 26 TEST_F(BluetoothAdapterAndroidTest, Construct) {
27 InitWithPermission();
28 ASSERT_TRUE(adapter_.get());
29 EXPECT_TRUE(adapter_->HasBluetoothPermission());
Ted C 2015/05/07 22:23:57 what happens if these tests are running on a devic
scheib 2015/05/07 23:59:17 Good point. I've guarded the unit test with a dete
30 EXPECT_GT(adapter_->GetAddress().length(), 0u);
31 EXPECT_GT(adapter_->GetName().length(), 0u);
32 EXPECT_TRUE(adapter_->IsPresent());
33 EXPECT_TRUE(adapter_->IsPowered());
34 EXPECT_FALSE(adapter_->IsDiscoverable());
35 EXPECT_FALSE(adapter_->IsDiscovering());
36 }
37
38 TEST_F(BluetoothAdapterAndroidTest, ConstructNoPermision) {
39 InitWithoutPermission();
21 ASSERT_TRUE(adapter_.get()); 40 ASSERT_TRUE(adapter_.get());
22 EXPECT_FALSE(adapter_->HasBluetoothPermission()); 41 EXPECT_FALSE(adapter_->HasBluetoothPermission());
42 EXPECT_EQ(adapter_->GetAddress().length(), 0u);
43 EXPECT_EQ(adapter_->GetName().length(), 0u);
44 EXPECT_FALSE(adapter_->IsPresent());
45 EXPECT_FALSE(adapter_->IsPowered());
46 EXPECT_FALSE(adapter_->IsDiscoverable());
47 EXPECT_FALSE(adapter_->IsDiscovering());
23 } 48 }
24 49
25 } // namespace device 50 } // namespace device
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698