[Kimchi-devel] [project-kimchi] [PATCH] Organize python imports

Ramon Medeiros ramonn at linux.vnet.ibm.com
Wed Dec 18 20:13:38 UTC 2013


First of all, congratulations. I think it's a very worth effort to keep 
the code complaint.

I have some questions!

why did you separated this imports? They can't be together?
>   import kimchi.mockmodel
>
>   #utils.silence_server()
> diff --git a/tests/test_vmtemplate.py b/tests/test_vmtemplate.py
> index 81382c7..92c7385 100644
> --- a/tests/test_vmtemplate.py
> +++ b/tests/test_vmtemplate.py
> @@ -23,9 +23,11 @@
>   import unittest
>   import uuid
>
> +
>   from kimchi.vmtemplate import *
>   from kimchi.xmlutils import xpath_get_text
>
> +
>   class VMTemplateTests(unittest.TestCase):
>       def test_minimal_construct(self):
>           fields = (('name', 'test'), ('os_distro', 'unknown'),
> diff --git a/tests/utils.py b/tests/utils.py
> index c114813..a7596e8 100644
> --- a/tests/utils.py
> +++ b/tests/utils.py
> @@ -21,16 +21,19 @@
>   # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
>   #
>
> -import httplib
> +import base64
>   import cherrypy
> -import threading
> -import time
> +import httplib
>   import os
> -import sys
>   import socket
> -from contextlib import closing
> +import sys
> +import threading
> +import time
>   import unittest
> -import base64
> +
> +
> +from contextlib import closing
> +
>
>   import kimchi.server
>   import kimchi.model
The same question here


-- 
Ramon Nunes Medeiros
Software Engineer - Linux Technology Center Brazil
IBM Systems & Technology Group
Phone : +55 19 2132 7878
ramonn at br.ibm.com

-- 
project-kimchi mailing list <project-kimchi at googlegroups.com>
https://groups.google.com/forum/#!forum/project-kimchi
--- 
You received this message because you are subscribed to the Google Groups "project-kimchi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to project-kimchi+unsubscribe at googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.



More information about the Kimchi-devel mailing list