@param method: The name of the remote method to invoke.
@param args: The positional arguments to pass to the remote method.
- @param args: The keyword arguments to pass to the remote method.
+ @param kwargs: The keyword arguments to pass to the remote method.
"""
return self.callRemote(MethodCall, method=method, args=args, kwargs=kwargs)
[2] There's another argument copy. I removed it and the tests run fine.
[3] It would be nice but certainly not required to see a unit test with an exception in the remote method implementation, being caught by the client. Mostly as an example. Triggering it by hand i get an UnknownAmpRemoteError, which i guess is to be expected.
Looks good to me +1. I dig the generic method call api over amp. diffs below are my suggested changes against the branch.
[1] doc string typo. lib/amp. py 2010-01-18 08:24:51 +0000 lib/amp. py 2010-01-19 17:57:36 +0000
--- landscape/
+++ landscape/
@@ -82,7 +82,7 @@
@param method: The name of the remote method to invoke. (MethodCall,
method= method, args=args, kwargs=kwargs)
@param args: The positional arguments to pass to the remote method.
- @param args: The keyword arguments to pass to the remote method.
+ @param kwargs: The keyword arguments to pass to the remote method.
"""
return self.callRemote
[2] There's another argument copy. I removed it and the tests run fine.
@@ -147,8 +147,8 @@
def send_method_ call(*args, **kwargs): send_method_ call(method= method, kwargs. copy()) addCallback( lambda response: response["result"])
result = self._protocol.
- args=args[:],
- kwargs=
+ args=args,
+ kwargs=kwargs)
return result.
[3] It would be nice but certainly not required to see a unit test with an exception in the remote method implementation, being caught by the client. Mostly as an example. Triggering it by hand i get an UnknownAmpRemot eError, which i guess is to be expected.