remote/client: replace manual place checks with helper method calls

Signed-off-by: Bastian Krause <[email protected]>

Author: Bastian-Krause

labgrid/remote/client.py

@@ -669,10 +669,7 @@ def check_matches(self, place):
 
     async def acquire(self):
         """Acquire a place, marking it unavailable for other clients"""
-        place = self.get_place()
-        if place.acquired:
-            raise UserError(f"place {place.name} is already acquired by {place.acquired}")
-
+        place = self.get_idle_place()
         if not self.args.allow_unmatched:
             self.check_matches(place)
 

tests/test_client.py

@@ -262,7 +262,7 @@ def test_place_add_no_name(coordinator):
 
 def test_place_del_no_name(coordinator):
     with pexpect.spawn('python -m labgrid.remote.client delete') as spawn:
-        spawn.expect("name was not a string")
+        spawn.expect("place pattern not specified")
         spawn.expect(pexpect.EOF)
         spawn.close()
         assert spawn.exitstatus != 0, spawn.before.strip()