[virt-tools-list] [virt-bootstrap] [PATCH v3 06/12] tests: Add test suite

Cedric Bosdonnat cbosdonnat at suse.com
Thu Jul 20 12:58:25 UTC 2017


On Thu, 2017-07-20 at 12:29 +0100, Radostin Stoyanov wrote:
> Tests can be started with: `./setup.py test`

Could you please mention it in the README as well (in that commit)?
I guess newcomers will more likely read a HACKING section in the README
than the git log.

> ---
>  setup.py | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/setup.py b/setup.py
> index b2e17ac..c6a35ad 100755
> --- a/setup.py
> +++ b/setup.py
> @@ -50,7 +50,7 @@ class CheckPylint(Command):
>          """
>  
>          res = 0
> -        files = ' '.join(["setup.py", "src/virtBootstrap/*.py"])
> +        files = ' '.join(["setup.py", "src/virtBootstrap/*.py", "tests/*.py"])
>          output_format = "colorized" if sys.stdout.isatty() else "text"
>  
>          print(">>> Running pycodestyle ...")
> @@ -82,6 +82,7 @@ setup(
>      keywords='virtualization container rootfs',
>      package_dir={"": "src"},
>      packages=['virtBootstrap'],
> +    test_suite='tests',
>      entry_points={
>          'console_scripts': [
>              'virt-bootstrap=virtBootstrap.virt_bootstrap:main',
> @@ -117,6 +118,8 @@ setup(
>      # root password for root file system.
>      install_requires=['passlib>=1.7.1'],
>  
> +    tests_require=['mock>=2.0'],
> +
>      extras_require={
>          'dev': [
>              'pylint',

ACK with the invocation being added to README

--
Cedric




More information about the virt-tools-list mailing list