feat(api): GET instruments endpoint returns 200 with body {path,value}

- Wraps driver return value in JSON response
- Adjust tests accordingly

Co-authored-by: openhands <openhands@all-hands.dev>

Author: openhands-agent

benchmesh-serial-service/src/benchmesh_service/api.py

@@ -118,12 +118,12 @@ def call_driver_get(klass: str, device_id: str, channel: str, method: str):
     try:
         if lock:
             with lock:
-                getattr(drv, method)()
+                value = getattr(drv, method)()
         else:
-            getattr(drv, method)()
+            value = getattr(drv, method)()
     except Exception as e:
         raise HTTPException(status_code=400, detail=f"Driver method execution failed: {e}")
-    return Response(status_code=204)
+    return {"path": f"/instruments/{klass}/{device_id}/{channel}/{method}", "value": value}
 
 
 @app.post("/instruments/{klass}/{device_id}/{channel}/{method}/{param}", summary="Call driver method (write)")

benchmesh-serial-service/tests/test_api_instruments.py

@@ -48,8 +48,12 @@ def client_with_fake_mgr(monkeypatch):
 
 def test_get_instrument_method_success(client_with_fake_mgr):
     client, fake = client_with_fake_mgr
-    r = client.get("/instruments/PSU/tenmapsu-1/1/identify")
-    assert r.status_code == 204
+    path = "/instruments/PSU/tenmapsu-1/1/identify"
+    r = client.get(path)
+    assert r.status_code == 200
+    body = r.json()
+    assert body["path"] == path
+    assert body["value"] == "FAKE,IDN"
     assert ("identify", ()) in fake.connections["tenmapsu-1"].calls