Adopt reviewer's suggestions

* Correct the StatusCode
* Format code
* Use @staticmethod
* Fix typo

1 files changed, 22 insertions, 9 deletions

diff --git a/src/python/grpcio/grpc/_cython/_cygrpc/channelz.pyx.pxi b/src/python/grpcio/grpc/_cython/_cygrpc/channelz.pyx.pxi
index 572a473d3e..113f7976dd 100644
--- a/src/python/grpcio/grpc/_cython/_cygrpc/channelz.pyx.pxi
+++ b/src/python/grpcio/grpc/_cython/_cygrpc/channelz.pyx.pxi
@@ -14,43 +14,56 @@
 
 
 def channelz_get_top_channels(start_channel_id):
-    cdef char *c_returned_str = grpc_channelz_get_top_channels(start_channel_id)
+    cdef char *c_returned_str = grpc_channelz_get_top_channels(
+        start_channel_id,
+    )
     if c_returned_str == NULL:
-        raise ValueError('Failed to get top channels, please ensure your start_channel_id==%s is valid' % start_channel_id)
+        raise ValueError('Failed to get top channels, please ensure your' \
+                         ' start_channel_id==%s is valid' % start_channel_id)
     return c_returned_str
     
 def channelz_get_servers(start_server_id):
     cdef char *c_returned_str = grpc_channelz_get_servers(start_server_id)
     if c_returned_str == NULL:
-        raise ValueError('Failed to get servers, please ensure your start_server_id==%s is valid' % start_server_id)
+        raise ValueError('Failed to get servers, please ensure your' \
+                         ' start_server_id==%s is valid' % start_server_id)
     return c_returned_str
     
 def channelz_get_server(server_id):
     cdef char *c_returned_str = grpc_channelz_get_server(server_id)
     if c_returned_str == NULL:
-        raise ValueError('Failed to get the server, please ensure your server_id==%s is valid' % server_id)
+        raise ValueError('Failed to get the server, please ensure your' \
+                         ' server_id==%s is valid' % server_id)
     return c_returned_str
     
 def channelz_get_server_sockets(server_id, start_socket_id):
-    cdef char *c_returned_str = grpc_channelz_get_server_sockets(server_id, start_socket_id)
+    cdef char *c_returned_str = grpc_channelz_get_server_sockets(
+        server_id,
+        start_socket_id,
+    )
     if c_returned_str == NULL:
-        raise ValueError('Failed to get server sockets, please ensure your server_id==%s and start_socket_id==%s is valid' % (server_id, start_socket_id))
+        raise ValueError('Failed to get server sockets, please ensure your' \
+                         ' server_id==%s and start_socket_id==%s is valid' %
+                         (server_id, start_socket_id))
     return c_returned_str
     
 def channelz_get_channel(channel_id):
     cdef char *c_returned_str = grpc_channelz_get_channel(channel_id)
     if c_returned_str == NULL:
-        raise ValueError('Failed to get the channel, please ensure your channel_id==%s is valid' % (channel_id))
+        raise ValueError('Failed to get the channel, please ensure your' \
+                         ' channel_id==%s is valid' % (channel_id))
     return c_returned_str
     
 def channelz_get_subchannel(subchannel_id):
     cdef char *c_returned_str = grpc_channelz_get_subchannel(subchannel_id)
     if c_returned_str == NULL:
-        raise ValueError('Failed to get the subchannel, please ensure your subchannel_id==%s is valid' % (subchannel_id))
+        raise ValueError('Failed to get the subchannel, please ensure your' \
+                         ' subchannel_id==%s is valid' % (subchannel_id))
     return c_returned_str
     
 def channelz_get_socket(socket_id):
     cdef char *c_returned_str = grpc_channelz_get_socket(socket_id)
     if c_returned_str == NULL:
-        raise ValueError('Failed to get the socket, please ensure your socket_id==%s is valid' % (socket_id))
+        raise ValueError('Failed to get the socket, please ensure your' \
+                         ' socket_id==%s is valid' % (socket_id))
     return c_returned_str