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

Unified Diff: device/bluetooth/bluetooth_discovery_filter_unittest.cc

Issue 2063353002: device/bluetooth: split out transport enum (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: renaming header, moving TransportMask back into filter Created 4 years, 6 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
Index: device/bluetooth/bluetooth_discovery_filter_unittest.cc
diff --git a/device/bluetooth/bluetooth_discovery_filter_unittest.cc b/device/bluetooth/bluetooth_discovery_filter_unittest.cc
index 63135b613f5009b395a6a199bcaee5c87cd31237..aff1257e80437ae1b3729083023f4c9ef4e526ab 100644
--- a/device/bluetooth/bluetooth_discovery_filter_unittest.cc
+++ b/device/bluetooth/bluetooth_discovery_filter_unittest.cc
@@ -8,6 +8,7 @@
#include "base/macros.h"
#include "device/bluetooth/bluetooth_discovery_session.h"
+#include "device/bluetooth/bluetooth_types.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace {
@@ -21,14 +22,12 @@ const device::BluetoothUUID uuid1020("1020");
namespace device {
TEST(BluetoothDiscoveryFilterTest, Equal) {
- BluetoothDiscoveryFilter df1(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_CLASSIC);
+ BluetoothDiscoveryFilter df1(BluetoothTransport::TRANSPORT_CLASSIC);
df1.SetRSSI(-65);
df1.AddUUID(uuid1020);
df1.AddUUID(uuid1003);
- BluetoothDiscoveryFilter df2(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_CLASSIC);
+ BluetoothDiscoveryFilter df2(BluetoothTransport::TRANSPORT_CLASSIC);
df2.SetRSSI(-65);
df2.AddUUID(uuid1020);
df2.AddUUID(uuid1004);
@@ -42,19 +41,18 @@ TEST(BluetoothDiscoveryFilterTest, Equal) {
ASSERT_TRUE(df1.Equals(df2));
// now transport don't match
- df1.SetTransport(BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+ df1.SetTransport(BluetoothTransport::TRANSPORT_LE);
ASSERT_FALSE(df1.Equals(df2));
// now everything is back matching
- df1.SetTransport(BluetoothDiscoveryFilter::Transport::TRANSPORT_CLASSIC);
+ df1.SetTransport(BluetoothTransport::TRANSPORT_CLASSIC);
ASSERT_TRUE(df1.Equals(df2));
// now rssi don't match
df1.SetRSSI(-30);
ASSERT_FALSE(df1.Equals(df2));
- BluetoothDiscoveryFilter df3(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_CLASSIC);
+ BluetoothDiscoveryFilter df3(BluetoothTransport::TRANSPORT_CLASSIC);
df3.SetPathloss(45);
df3.AddUUID(uuid1020);
df3.AddUUID(uuid1003);
@@ -65,14 +63,12 @@ TEST(BluetoothDiscoveryFilterTest, Equal) {
}
TEST(BluetoothDiscoveryFilterTest, CopyFrom) {
- BluetoothDiscoveryFilter df1(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_CLASSIC);
+ BluetoothDiscoveryFilter df1(BluetoothTransport::TRANSPORT_CLASSIC);
df1.SetRSSI(-65);
df1.AddUUID(uuid1020);
df1.AddUUID(uuid1003);
- BluetoothDiscoveryFilter df2(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_CLASSIC);
+ BluetoothDiscoveryFilter df2(BluetoothTransport::TRANSPORT_CLASSIC);
df2.CopyFrom(df1);
@@ -83,8 +79,7 @@ TEST(BluetoothDiscoveryFilterTest, CopyFrom) {
df2.GetRSSI(&out_rssi);
EXPECT_EQ(-65, out_rssi);
- EXPECT_EQ(BluetoothDiscoveryFilter::Transport::TRANSPORT_CLASSIC,
- df2.GetTransport());
+ EXPECT_EQ(BluetoothTransport::TRANSPORT_CLASSIC, df2.GetTransport());
df2.GetUUIDs(out_uuids);
EXPECT_TRUE(out_uuids.find(uuid1020) != out_uuids.end());
@@ -92,13 +87,11 @@ TEST(BluetoothDiscoveryFilterTest, CopyFrom) {
}
TEST(BluetoothDiscoveryFilterTest, MergeUUIDs) {
- BluetoothDiscoveryFilter df1(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+ BluetoothDiscoveryFilter df1(BluetoothTransport::TRANSPORT_LE);
df1.AddUUID(uuid1020);
df1.AddUUID(uuid1003);
- BluetoothDiscoveryFilter df2(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+ BluetoothDiscoveryFilter df2(BluetoothTransport::TRANSPORT_LE);
df2.AddUUID(uuid1020);
df2.AddUUID(uuid1004);
@@ -120,20 +113,16 @@ TEST(BluetoothDiscoveryFilterTest, MergeUUIDs) {
}
TEST(BluetoothDiscoveryFilterTest, MergeProximity) {
- BluetoothDiscoveryFilter df1(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+ BluetoothDiscoveryFilter df1(BluetoothTransport::TRANSPORT_LE);
df1.SetRSSI(-50);
- BluetoothDiscoveryFilter df2(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+ BluetoothDiscoveryFilter df2(BluetoothTransport::TRANSPORT_LE);
df2.SetRSSI(-70);
- BluetoothDiscoveryFilter df3(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+ BluetoothDiscoveryFilter df3(BluetoothTransport::TRANSPORT_LE);
df3.SetPathloss(70);
- BluetoothDiscoveryFilter df4(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+ BluetoothDiscoveryFilter df4(BluetoothTransport::TRANSPORT_LE);
df4.SetPathloss(20);
std::unique_ptr<BluetoothDiscoveryFilter> result =
@@ -157,25 +146,20 @@ TEST(BluetoothDiscoveryFilterTest, MergeProximity) {
}
TEST(BluetoothDiscoveryFilterTest, MergeTransport) {
- BluetoothDiscoveryFilter df1(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_CLASSIC);
+ BluetoothDiscoveryFilter df1(BluetoothTransport::TRANSPORT_CLASSIC);
- BluetoothDiscoveryFilter df2(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_LE);
+ BluetoothDiscoveryFilter df2(BluetoothTransport::TRANSPORT_LE);
- BluetoothDiscoveryFilter df3(
- BluetoothDiscoveryFilter::Transport::TRANSPORT_DUAL);
+ BluetoothDiscoveryFilter df3(BluetoothTransport::TRANSPORT_DUAL);
std::unique_ptr<BluetoothDiscoveryFilter> result =
BluetoothDiscoveryFilter::Merge(&df1, &df2);
// Merging LE and CLASSIC should result in both being set
- EXPECT_EQ(BluetoothDiscoveryFilter::Transport::TRANSPORT_DUAL,
- result->GetTransport());
+ EXPECT_EQ(BluetoothTransport::TRANSPORT_DUAL, result->GetTransport());
result = BluetoothDiscoveryFilter::Merge(&df1, &df3);
- EXPECT_EQ(BluetoothDiscoveryFilter::Transport::TRANSPORT_DUAL,
- result->GetTransport());
+ EXPECT_EQ(BluetoothTransport::TRANSPORT_DUAL, result->GetTransport());
// Merging with null should alway result with empty filter.
result = BluetoothDiscoveryFilter::Merge(&df1, nullptr);

Powered by Google App Engine
This is Rietveld 408576698