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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux