Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(12)

Unified Diff: testing/legion/examples/hello_world/host_test.py

Issue 870103005: Adding a server side, non-blocking subprocess mechanism. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changed the GetStd*** names and added some usage docs. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « testing/legion/common_lib.py ('k') | testing/legion/examples/subprocess/client.isolate » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: testing/legion/examples/hello_world/host_test.py
diff --git a/testing/legion/examples/hello_world/host_test.py b/testing/legion/examples/hello_world/host_test.py
index 0094d9bd5611c2b7aefb872be3b4b367e9c3a5a5..7a7875b7a311d5f6099550777efa8ec5af487f47 100755
--- a/testing/legion/examples/hello_world/host_test.py
+++ b/testing/legion/examples/hello_world/host_test.py
@@ -60,13 +60,16 @@ class ExampleController(host_controller.HostController):
def CallEcho(self, client):
"""Call rpc.Echo on a client."""
logging.info('Calling Echo on %s', client.name)
- logging.info(self.client1.rpc.Echo(client.name))
+ logging.info(client.rpc.Echo(client.name))
def CallClientTest(self, client):
"""Call client_test.py name on a client."""
logging.info('Calling Subprocess to run "./client_test.py %s"', client.name)
- retcode, stdout, stderr = client.rpc.Subprocess(
- ['./client_test.py', client.name])
+ proc = client.rpc.subprocess.Popen(['./client_test.py', client.name])
+ client.rpc.subprocess.Wait(proc)
+ retcode = client.rpc.subprocess.GetReturncode(proc)
+ stdout = client.rpc.subprocess.ReadStdout(proc)
+ stderr = client.rpc.subprocess.ReadStderr(proc)
logging.info('retcode: %s, stdout: %s, stderr: %s', retcode, stdout, stderr)
« no previous file with comments | « testing/legion/common_lib.py ('k') | testing/legion/examples/subprocess/client.isolate » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698