varlink: rename OperationNotSupported error argument

To be more consistent across all the objects and errors, rename "entry"
to "name". It was always the name. When the API expects a whole entry
object we should use "entry". "name" is a string.

Signed-off-by: John Mulligan <[email protected]>

Author: phlogistonjohn

sambacc/varlink/interfaces/org.samba.containers.keybridge.varlink

@@ -73,4 +73,4 @@ error ReadOnlyScope (name: string)
 
 # OperationNotSupported may be returned if an entry method is not supported by
 # the given scope.
-error OperationNotSupported (op: string, entry: string, scope: string)
+error OperationNotSupported (op: string, name: string, scope: string)

sambacc/varlink/keybridge.py

@@ -97,7 +97,7 @@ class OperationNotSupported(KeyBridgeError):
     """
 
     _name = "OperationNotSupported"
-    _expect = {"op", "entry", "scope"}
+    _expect = {"op", "name", "scope"}
 
 
 class OpKind(str, enum.Enum):
@@ -245,7 +245,7 @@ def _verify(
         )
         raise OperationNotSupported(
             op=op.value,
-            entry=name,
+            name=name,
             scope=scope_name,
         )
 
@@ -316,7 +316,7 @@ def Set(self, entry: dict, _request: _Request) -> None:
         if not _set:
             raise OperationNotSupported(
                 op=OpKind.SET.value,
-                entry=_entry.name,
+                name=_entry.name,
                 scope=_entry.scope,
             )
         _set(_entry.name, _entry.kind, _entry.data)
@@ -335,7 +335,7 @@ def Delete(self, name: str, scope: str, _request: _Request) -> None:
         if not _delete:
             raise OperationNotSupported(
                 op=OpKind.DELETE.value,
-                entry=name,
+                name=name,
                 scope=scope,
             )
         _delete(name)
@@ -482,7 +482,7 @@ def set(self, key: str, kind: EntryKind, value: str) -> None:
         if not _set:
             raise OperationNotSupported(
                 op=OpKind.SET.value,
-                entry=key,
+                name=key,
                 scope=self.other.name(),
             )
         _set(key, kind, value)
@@ -492,7 +492,7 @@ def delete(self, key: str) -> None:
         if not _delete:
             raise OperationNotSupported(
                 op=OpKind.DELETE.value,
-                entry=key,
+                name=key,
                 scope=self.other.name(),
             )
         _delete(key)