bluez: Set Powered before other properties

object_set_properties sets the properties one by one, but the bluez
interface doesn't allow setting Discoverable or Pairable unless the
adaptor is powered. So before setting those, power it on first.

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
Differential Revision: https://phabricator.apertis.org/D2208

apertis_tests_lib/bluez.py

@@ -305,7 +305,7 @@ def object_wait_for_properties_blocking(obj, properties_interface, properties,
     main_loop.run()
 
 
-def adapters_ensure_powered(adapters, result_func):
+def adapters_ensure_powered(adapters, result_func=None):
     assert(len(adapters) > 0)
     remaining = set(adapters)
 
@@ -314,7 +314,7 @@ def adapters_ensure_powered(adapters, result_func):
 
         print('Adapter ‘%s’ is powered' % obj.object_path)
         remaining.remove(obj)
-        if len(remaining) == 0:
+        if not remaining and result_func:
             result_func()
 
     for adapter in adapters:
@@ -336,10 +336,11 @@ def adapters_make_pairable(adapters, result_func=None):
         if not remaining and result_func:
             result_func()
 
+    adapters_ensure_powered(adapters)
+
     for adapter in adapters:
         print('Making adapter ‘%s’ pairable…' % adapter.object_path)
         object_set_properties(adapter, 'org.bluez.Adapter1', {
-            'Powered': True,
             'Pairable': True,
             'Discoverable': True,
         }, inner_result_func)