diff options
author | Alex Blasche <alexander.blasche@theqtcompany.com> | 2015-05-07 14:03:23 +0200 |
---|---|---|
committer | Alex Blasche <alexander.blasche@theqtcompany.com> | 2015-05-12 09:00:45 +0000 |
commit | 193ed20a9d6f4b65fdffd4d3c43d7824eb450089 (patch) | |
tree | 08f42394b5fbc9d8a18acff92cd122f3b96e6d32 /tests/auto/qlowenergycontroller | |
parent | 26009d55b503e4bffbddf1f68530797ef81a1127 (diff) |
Reduce platform specific code lines
QLEController(QBluetoothAddress) ctor is not usable on iOS and OSX.
On those platforms we have to use QLECOntroller(QBluetoothDeviceInfo).
We mark the QBluetoothAddress based ctors of QLEController obsolete
and remove most uses of them. This cleans up some ifdefs throughout
the QtBluetooth code base.
Change-Id: Ibe1e3fa271a29461e39fc3c0eeba7a910a250077
Reviewed-by: Timur Pocheptsov <Timur.Pocheptsov@digia.com>
Diffstat (limited to 'tests/auto/qlowenergycontroller')
-rw-r--r-- | tests/auto/qlowenergycontroller/tst_qlowenergycontroller.cpp | 98 |
1 files changed, 39 insertions, 59 deletions
diff --git a/tests/auto/qlowenergycontroller/tst_qlowenergycontroller.cpp b/tests/auto/qlowenergycontroller/tst_qlowenergycontroller.cpp index 5faf5fee..50ecc099 100644 --- a/tests/auto/qlowenergycontroller/tst_qlowenergycontroller.cpp +++ b/tests/auto/qlowenergycontroller/tst_qlowenergycontroller.cpp @@ -156,8 +156,8 @@ void tst_QLowEnergyController::initTestCase() // On OS X/iOS the only way to find the device we are // interested in - is to use device's name. if (info.name().contains("Sensor") && info.name().contains("Tag")) { - remoteDeviceInfo = info; #endif + remoteDeviceInfo = info; deviceFound = true; break; } @@ -249,31 +249,23 @@ void tst_QLowEnergyController::tst_connect() { QList<QBluetoothHostInfo> localAdapters = QBluetoothLocalDevice::allDevices(); -#ifndef Q_OS_MAC - if (localAdapters.isEmpty() || remoteDevice.isNull()) -#elif defined(Q_OS_OSX) - if (localAdapters.isEmpty() || remoteDeviceInfo.deviceUuid().isNull()) -#elif defined (Q_OS_IOS) - if (remoteDeviceInfo.deviceUuid().isNull()) +#ifdef Q_OS_IOS + if (remoteDeviceInfo.isValid()) +#else + if (localAdapters.isEmpty() || !remoteDeviceInfo.isValid()) #endif QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); -#ifndef Q_OS_IOS - const QBluetoothAddress localAdapter = localAdapters.at(0).address(); -#endif - -#ifndef Q_OS_MAC - QLowEnergyController control(remoteDevice); -#else - // Create a low energy controller using Apple's - // uuid (NSUUID). QLowEnergyController control(remoteDeviceInfo); -#endif - QSignalSpy connectedSpy(&control, SIGNAL(connected())); QSignalSpy disconnectedSpy(&control, SIGNAL(disconnected())); + if (remoteDeviceInfo.name().isEmpty()) + QVERIFY(control.remoteName().isEmpty()); + else + QCOMPARE(control.remoteName(), remoteDeviceInfo.name()); #ifndef Q_OS_IOS + const QBluetoothAddress localAdapter = localAdapters.at(0).address(); QCOMPARE(control.localAddress(), localAdapter); QVERIFY(!control.localAddress().isNull()); #endif @@ -417,16 +409,14 @@ void tst_QLowEnergyController::tst_concurrentDiscovery() { #ifndef Q_OS_MAC QList<QBluetoothHostInfo> localAdapters = QBluetoothLocalDevice::allDevices(); - if (localAdapters.isEmpty() || remoteDevice.isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); + if (localAdapters.isEmpty()) + QSKIP("No local Bluetooth device found. Skipping test."); +#endif - // quick setup - more elaborate test is done by connectNew() - QLowEnergyController control(remoteDevice); -#else - if (remoteDeviceInfo.deviceUuid().isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); + if (!remoteDeviceInfo.isValid()) + QSKIP("No remote BTLE device found. Skipping test."); QLowEnergyController control(remoteDeviceInfo); -#endif + QCOMPARE(control.state(), QLowEnergyController::UnconnectedState); QCOMPARE(control.error(), QLowEnergyController::NoError); @@ -1659,16 +1649,13 @@ void tst_QLowEnergyController::tst_writeCharacteristic() { #ifndef Q_OS_MAC QList<QBluetoothHostInfo> localAdapters = QBluetoothLocalDevice::allDevices(); - if (localAdapters.isEmpty() || remoteDevice.isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); + if (localAdapters.isEmpty()) + QSKIP("No local Bluetooth device found. Skipping test."); +#endif - // quick setup - more elaborate test is done by connect() - QLowEnergyController control(remoteDevice); -#else - if (remoteDeviceInfo.deviceUuid().isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); + if (!remoteDeviceInfo.isValid()) + QSKIP("No remote BTLE device found. Skipping test."); QLowEnergyController control(remoteDeviceInfo); -#endif QCOMPARE(control.error(), QLowEnergyController::NoError); @@ -1836,14 +1823,14 @@ void tst_QLowEnergyController::tst_readWriteDescriptor() { #ifndef Q_OS_MAC QList<QBluetoothHostInfo> localAdapters = QBluetoothLocalDevice::allDevices(); - if (localAdapters.isEmpty() || remoteDevice.isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); - QLowEnergyController control(remoteDevice); -#else - if (remoteDeviceInfo.deviceUuid().isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); - QLowEnergyController control(remoteDeviceInfo); + if (localAdapters.isEmpty()) + QSKIP("No local Bluetooth device found. Skipping test."); #endif + + if (!remoteDeviceInfo.isValid()) + QSKIP("No remote BTLE device found. Skipping test."); + QLowEnergyController control(remoteDeviceInfo); + // quick setup - more elaborate test is done by connect() control.connectToDevice(); { @@ -2259,17 +2246,13 @@ void tst_QLowEnergyController::tst_errorCases() { #ifndef Q_OS_MAC QList<QBluetoothHostInfo> localAdapters = QBluetoothLocalDevice::allDevices(); - if (localAdapters.isEmpty() || remoteDevice.isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); - - // quick setup - more elaborate test is done by connect() - QLowEnergyController control(remoteDevice); -#else - if (remoteDeviceInfo.deviceUuid().isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); + if (localAdapters.isEmpty()) + QSKIP("No local Bluetooth device found. Skipping test."); +#endif + if (!remoteDeviceInfo.isValid()) + QSKIP("No remote BTLE device found. Skipping test."); QLowEnergyController control(remoteDeviceInfo); -#endif QCOMPARE(control.error(), QLowEnergyController::NoError); control.connectToDevice(); @@ -2485,17 +2468,14 @@ void tst_QLowEnergyController::tst_writeCharacteristicNoResponse() { #ifndef Q_OS_MAC QList<QBluetoothHostInfo> localAdapters = QBluetoothLocalDevice::allDevices(); - if (localAdapters.isEmpty() || remoteDevice.isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); - - // quick setup - more elaborate test is done by connect() - QLowEnergyController control(remoteDevice); -#else - if (remoteDeviceInfo.deviceUuid().isNull()) - QSKIP("No local Bluetooth or remote BTLE device found. Skipping test."); + if (localAdapters.isEmpty()) + QSKIP("No local Bluetooth device found. Skipping test."); +#endif + if (!remoteDeviceInfo.isValid()) + QSKIP("No remote BTLE device found. Skipping test."); QLowEnergyController control(remoteDeviceInfo); -#endif + QCOMPARE(control.error(), QLowEnergyController::NoError); control.connectToDevice(); |