commit 90a84ea8c22affb82196233cb675dcd9857be100
author Allen Li <ayatane@chromium.org> Thu Oct 27 15:07:42 2016 -0700
committer chrome-bot <chrome-bot@chromium.org> Tue Nov 08 23:25:13 2016 -0800
tree 50892511a977845b8c05873e4c9b51729eb69349
parent 854ef83fdcba4033b476b5952affc161024179c5

[autotest] Refactor atest server list

This is prep work for adding a JSON output format to atest so various
scripts can use atest to access server_db.

BUG=None
TEST=Run atest server list

Change-Id: If037def77393f60bc87e2b855bb088c72e1c34ab
Reviewed-on: https://chromium-review.googlesource.com/404496
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

cli/server.py

diff --git a/cli/server.py b/cli/server.py
index e0c5717..f517046 100644
--- a/cli/server.py
+++ b/cli/server.py
@@ -173,13 +173,18 @@
         @param results: return of the execute call, a list of server object that
                         contains server information.
         """
-        if not results:
+        if results:
+            if self.table:
+                formatter = server_manager_utils.format_servers_table
+            elif self.summary:
+                formatter = server_manager_utils.format_servers_summary
+            else:
+                formatter = server_manager_utils.format_servers
+            print formatter(results)
+        else:
             self.failure('No server is found.',
                          what_failed='Failed to find servers',
                          item=self.hostname, fatal=True)
-        else:
-            print server_manager_utils.get_server_details(results, self.table,
-                                                          self.summary)
 
 
 class server_create(server):