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

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: tedchoc comments addressed. 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) {
jbudorick 2015/05/11 21:20:03 This test failed on its first run through Android
27 InitWithPermission();
28 ASSERT_TRUE(adapter_.get());
29 EXPECT_TRUE(adapter_->HasBluetoothPermission());
30 if (!adapter_->IsPresent()) {
31 LOG(WARNING) << "Bluetooth adapter not present; skipping unit test.";
32 return;
33 }
34 EXPECT_GT(adapter_->GetAddress().length(), 0u);
35 EXPECT_GT(adapter_->GetName().length(), 0u);
36 EXPECT_TRUE(adapter_->IsPresent());
37 EXPECT_TRUE(adapter_->IsPowered());
38 EXPECT_FALSE(adapter_->IsDiscoverable());
39 EXPECT_FALSE(adapter_->IsDiscovering());
40 }
41
42 TEST_F(BluetoothAdapterAndroidTest, ConstructNoPermision) {
43 InitWithoutPermission();
21 ASSERT_TRUE(adapter_.get()); 44 ASSERT_TRUE(adapter_.get());
22 EXPECT_FALSE(adapter_->HasBluetoothPermission()); 45 EXPECT_FALSE(adapter_->HasBluetoothPermission());
46 EXPECT_EQ(adapter_->GetAddress().length(), 0u);
47 EXPECT_EQ(adapter_->GetName().length(), 0u);
48 EXPECT_FALSE(adapter_->IsPresent());
49 EXPECT_FALSE(adapter_->IsPowered());
50 EXPECT_FALSE(adapter_->IsDiscoverable());
51 EXPECT_FALSE(adapter_->IsDiscovering());
23 } 52 }
24 53
25 } // namespace device 54 } // namespace device
OLDNEW
« no previous file with comments | « device/bluetooth/bluetooth_adapter_android.cc ('k') | testing/android/native_test/java/AndroidManifest.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698