summaryrefslogtreecommitdiffstats
path: root/tox.ini
Commit message (Collapse)AuthorAgeFilesLines
* Fix and enable H403 testsDirk Mueller2013-06-121-1/+1
| | | | | | Multi-line doc-strings should end on a new, separate line. Change-Id: I4cf0cfe92b634ef77971863a4df41ef43531bc20
* Merge "Delete unused bin directory"Jenkins2013-06-051-1/+0
|\
| * Delete unused bin directoryJoe Gordon2013-06-031-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | Delete last bits of bin/. With the move to entrypoints these aren't needed anymore. Update CONF.bindir to default to os.path.join(sys.prefix, 'local', 'bin') Part of blueprint entrypoints-plugins Change-Id: I95250d3779433e7b85aaa889a873b16c86a7d2be
* | python3: Add py33 to tox.iniChuck Short2013-06-011-1/+1
|/ | | | | | | | Introduce py33 to tox.ini to make testing with python3 easier. Change-Id: I3f966efc1a845aaa8e317d810bd6a1366acd323c Signed-off-by: Chuck Short <chuck.short@canonical.com>
* Add posargs support to flake8 callJoe Gordon2013-05-281-1/+1
| | | | | | | | Add posargs to flake8 call in tox.ini, with this you can pass arguments directly into flake8 using the following notation tox -epep8 -- --FLAKE8-ARG Change-Id: I1296eac3df46438ef050bf99ca33eb38d3b02efa
* Enumerate Flake8 E12x ignoresJoe Gordon2013-05-281-1/+1
| | | | | | | Instead of ignoring all E12x errors, ignore only the failing ones, so its easy to fix each one separately. Change-Id: Id365f4d61be56b51823ead8c7c1948e034ea609b
* Fix and enable flake8 F823Joe Gordon2013-05-281-1/+1
| | | | | | | F823 local variable 'blah' (defined in enclosing scope on line xx) referenced before assignment Change-Id: I721e1339ef023558032980bbf59e8dc44b391d52
* Fix and enable flake8 F812Joe Gordon2013-05-281-1/+1
| | | | | | | | | F812 list comprehension redefines List comprehension don't have scope so redefining variables can lead to unexpected behaviors Change-Id: I85484dfcd4fa391195ea7be75aadc7897da4739c
* Enumerate Flake8 Fxxx ignoresJoe Gordon2013-05-281-1/+1
| | | | | | | Instead of ignoring all Fxxx errors, ignore only the failing ones, so its easy to fix each one separately. Change-Id: I4ccc1d38247fb76b6dad4388b76c6e8d696f081f
* Enable flake8 E721Joe Gordon2013-05-281-1/+1
| | | | | | We already pass the test, so just remove from ignore list Change-Id: I28bfeb99d9841532133a3825f7336e7b712d6561
* Rename requires files to standard names.Ruby Loo2013-05-221-2/+2
| | | | | | | | | Rename tools/pip-requires to requirements.txt and tools/test-requires to test-requirements.txt. These are standard files, and tools in the general world are growing intelligence about them. Change-Id: I68ece8406fb1d4e082a42db8e76e17b1aaa7e775 Fixes: bug #1179008
* Switch to flake8+hacking.Monty Taylor2013-05-171-7/+12
| | | | | | | | Remove a bunch of local custom scripts. Replace with configurable external tools. Use local hacking checks for nova specifics. Change-Id: I75a01375ba0ec36d2ff05abc47abe0a3f225eda5
* Enable tox use of site-packages for libvirt.Clark Boylan2013-03-191-0/+1
| | | | | | | | | | | | | Enable the use of site-packages in tox which will allow the use of the system install of libvirt while testing. Hardcode the libvirt host UUID for tests that check this UUID when system libvirt is being used. Without this hardcoding eight tests would fail when using the system libvirt install. Partially fixes bug #1113181 Change-Id: I59c5fbd45639962c0963298203c39759b6ca2d11
* Update tox.ini to support RHEL 6.x.Dan Prince2013-03-071-1/+5
| | | | | | | | | | | | | | | | | | In order to support running unit tests on RHEL 6.x we need to patch eventlet with contrib/redhat-eventlet.patch. We already have support for this in the tools/install_venv_common.py but we need to make a couple changes to allow tox to consume this: 1) Sync in the latest intall_venv_common.py from oslo. This changes patch to use the -N option (ignore already applied patches) and makes it safe to call the patching function more than once. 2) Add a new patch_tox_venv.py script in tools. 3) Update tox.ini to call patch_tox_venv.py before it runs tests and coverage. Change-Id: I3e2b07c3f718e4aede5c5f231ff0cdb7721ec885
* Use oslo-config-2013.1b4Mark McLoughlin2013-02-191-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | The cfg API is now available via the oslo-config library, so switch to it and remove the copied-and-pasted version. Add the 2013.1b4 tarball to tools/pip-requires - this will be changed to 'oslo-config>=2013.1' when oslo-config is published to pypi. This will happen in time for grizzly final. Add dependency_links to setup.py so that oslo-config can be installed from the tarball URL specified in pip-requires. Remove the 'deps = pep8==1.3.3' from tox.ini as it means all the other deps get installed with easy_install which can't install oslo-config from the URL. Make tools/hacking.py include oslo in IMPORT_EXCEPTIONS like it already does for paste. It turns out imp.find_module() doesn't correct handle namespace packages. Retain dummy cfg.py file until keystoneclient middleware has been updated (I18c450174277c8e2d15ed93879da6cd92074c27a). Change-Id: I4815aeb8a9341a31a250e920157f15ee15cfc5bc
* Harmonize PEP8 checking between tox and run_tests.shRick Harris2013-02-151-12/+1
| | | | | | | | | | | | | | | | | Tox and run_tests.sh were running PEP8 checks against different file-sets. This patch refactors the logic to determine which files to run PEP8 checks on into `tools/run_pep8.sh` where it can be called by both tox and `run_tests.sh`. Additional fixes: Some of our Python XenAPI Dom0 plugins don't end in *.py but should still be checked by PEP8. This patches fixes the hacking.py violations in the files and adds them back to the srcfiles list. Merged tools/unused_imports.sh into tools/run_pep8.sh Change-Id: Id5edd1acb644ab938beffc3473494a179d9d8cda
* Module import style checking changesAttila Fazekas2013-02-131-4/+9
| | | | | | | | | | | | | | * Implementing the * import detection (it is disabled for now) * New style relative import testing based on syntax rules * Old style relative import testing based on module search * Inspection based solution replaced by PYTHONPATH search in order to avoid module compile and initialization steps (code execution) in a syntax checking phase. This solution is faster and safer, but does not able to recognize modules added dynamically to the module scope. Change-Id: Ifc871f4fdbcd4a9a736170ceb4475f4f2cbe66bc
* Added the build directory to the tox.ini list pep8 ignores.Anita Kuno2013-02-071-1/+1
| | | | Change-Id: Ie3c0d5ce021058de8ac868c736660fa0c75b5af5
* Make sure there are no unused importJoe Gordon2013-01-281-1/+4
| | | | | | | Remove all currently unused imports Prevent future unused imports Change-Id: I6ac26d5c71b79952a7732db300355a00310c712e
* Enable N302: Import modules onlyJoe Gordon2013-01-251-4/+2
| | | | | | Fix all N302 issues, and re-enable. Change-Id: Ic94d144c915b228b7ff2fd9c5951875e159ffcdd
* Fix hacking N302 import only modulesJoe Gordon2013-01-241-2/+4
| | | | | | | | * Includes some general tools/hacking cleanup * Fix several N302 cases * Disable N302 until all cases are fixed Change-Id: Iddba07ff13e10dc41a6930749044bb8c0572d279
* Fix nova coverage.Clark Boylan2013-01-211-1/+1
| | | | | | | | | | Nova coverage was broken because additional coverage tests were added that interfere with collecting coverage during unit testing. These new tests were not properly excluded by the coverage unit test regex. Modify the regex to exclude any tests of the form '.*test.*coverage.*'. This allows `tox -ecover` and `run_tests.sh --coverage` to run properly. Change-Id: Id35f50ec335707c9eadb0e7fb3aa0833661f7333
* Fix multi line docstring tests in hacking.pyJoe Gordon2013-01-171-2/+2
| | | | | | | | | * Fixes N403, along with docstring test * Adds N404, multi line start * Disable N403 and N404 until all cases are fixed * Remove obsolote '--repeat' flag from tox.ini Change-Id: Ibf6371efc1cdc764e66f6f7c5c4362440101f963
* Merge "Use testrepository setuptools support."Jenkins2013-01-161-9/+6
|\
| * Use testrepository setuptools support.Monty Taylor2013-01-141-9/+6
| | | | | | | | | | | | The previous false-positive bug was fixed upstream in 0.0.13. Change-Id: Iceb07a511992249c104bbc7c34b7c35e01bd9fbd
| * Revert "Use testr setuptools commands."James E. Blair2013-01-141-6/+9
| | | | | | | | | | | | | | This reverts commit c945b71929d7dc872ea27103ca8b6ec3d5cc7cd5 which caused the unit test command to always exit sucessfully. Change-Id: I410cbc878675503f0ce6e50bddc4fd0552370c46
| * Use testr setuptools commands.Monty Taylor2013-01-121-9/+6
| | | | | | | | | | | | | | Replace the embedded shell scripts with setuptools commands provided by upstream. Cleans up a few warnings that this cause tox to show. Change-Id: I9eb47142eafb6ed038b358b94984462ca1632542
* | enable hacking.py self testsSean Dague2013-01-141-0/+1
|/ | | | | | | | | | | | | | | | | | | | | | | | | | pep8 and hacking.py (by inheritance) have the ability to run self tests via docstrings, if we format the comments correctly, and change the docstring test regex. Include self testing in our normal pep8 tests. fix the comments to actually be test case compliant fix nova_import_alphabetical to pass on short import lists (only 2 lines, starting at the beginning of a file) comment where it doesn't appear that hacking.py tests are working (though there could be deeper black magic in some of them). fix a doc string in pep8 self tests that breaks N301 fix the fact that pep8 error strings are supposed to start with \w\d\d\d, we were just lucking out on some formatting because the string NOVA happened to be only 4 chars long. Add a couple of escapes so that we don't recursively fail on our own fail self tests for comments. (boy that's fun) Change-Id: I14d5e04cfc092b37ec4967cd7485e9975a5eb820
* make runtests -p act more like toxSean Dague2013-01-101-2/+2
| | | | | | now that all the N402 fixes are in, start enforcing it Change-Id: I9d495849bf698e5af142d0ccafe551ac0e8a600c
* Merge "fix N401 errors, stop ignoring all N4* errors"Jenkins2013-01-091-2/+2
|\
| * fix N401 errors, stop ignoring all N4* errorsSean Dague2013-01-071-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We had previously been ignoring all our custom N4xx hacking.py errors. This fixes all the N401 errors "doc strings should not start with a space" and reduces the ignore set down to N402 only "single line docstrings should end with period". It also fixes the N401 parser to catch only docstrings, and not tripple quoted string blocks used later on in a function. Clean up a few of the more crazy uses of """ in our code Clean up additional funky comments to make indents a bit more consistent, and pull in lines when possible. Change-Id: I9040a1d2ca7efda83bd5e425b95d1408b5b63577
* | Invert test stream capture logic for debugging.Clark Boylan2013-01-071-3/+0
|/ | | | | | | | | | | | | | | | | | | | Previously the default test behavior was to capture stdout and stderr. Make the new default to not capture stdout and stderr then set .testr.conf to always capture these streams when running tests under testr. The motiviation behind this change is that you will want these streams to be captured when running under testr, but when not running under testr you may not want to capture them. An example of this would be running `python -m testtools.run test_name` with a change to test_name to invoke the python debugger (capturing stdout and stderr interferes with normal debugger functionality). Also, only invoke the test timeout by default when running under testr. This is done for the same reason as above. When running a test under the debugger the timeout interferes with debugging. Change-Id: I42cbbdadb2f221ec439e92a6800d14e8436bb77b
* Use testr to run nova unittests.Clark Boylan2012-12-141-8/+11
| | | | | | | | | | | | | | | | | | | | Convert nova from using nosetests to testr for its test runner. Some tests had to be modified to get them to run properly under testr. run_tests.sh has been updated to run testr instead of nosetests. Coverage is collected by running subunit.run under coverage.py when the coverage environment is selected. Note that you will need to rebuild your virtualenvs as nose is being removed from the dependency lists and is being replaced by testr. Tests will run in different processes once this test is merged so you cannot use test classes to pass information between tests. Each test should be a proper independent unit. Additionally the -x and -d flags to run_tests.sh have been removed as there are currently no decent approximations for those functions. Change-Id: I019ca098972ca749b195f59968cf21edd5ba9109
* Fix pep8 exclude logic for 1.3.3Joe Gordon2012-12-061-1/+1
| | | | | | | | | | * With pep8 1.3.3 the exclude logic monkey patch broke causing tools/hacking to run on openstack/common code * Also add .git to exclude list Fix bug 1086410 Change-Id: Ifc50ac1963c50c338432bbb4b1e15cdf58ed2128
* Add pyflakes option to toxJoe Gordon2012-12-041-0/+4
| | | | | | | * Uses pyflakes wrapper to ignore gettext errors * Based on: https://bugs.launchpad.net/pyflakes/+bug/844592 Change-Id: I4fd947176f5258f89f517acf5dda2ded4a2da15e
* Pin pep8 to 1.3.3Chuck Short2012-11-191-3/+3
| | | | | | | | | | | Apart of making pep8 version standard across all openstack projects. With this change we ignore E712 since it is normal to use "column == True" in sqlalchemy. Change-Id: I73a162847a79558cb158112878033edf18039805 Signed-off-by: Chuck Short <chuck.short@canonical.com>
* Upgrade pylint version to 0.26.0Yun Mao2012-11-121-1/+1
| | | | | | Upgrade pylint to reduce false positives in lintstack. Change-Id: I10ab7a46c6a70daf9151f03cd9fea3c4851e6ebb
* Make tox.ini run pep8/hacking checks on bin.Dan Prince2012-11-091-0/+2
| | | | | | | This updates the pep8 hacking checks in our tox.ini file so that we are also scanning the bin directory for HACKING violations. Change-Id: I4b010ef4dd799f0afe0fcd56c823a611021e8f60
* Fix and enable pep8 E502, E712Joe Gordon2012-10-221-1/+1
| | | | | | pep8 E502 and E712 were temporarily disabld when upgrading to pep8 1.2 Change-Id: If5158304cb8e73ef844dea9f726df6f8fe305269
* Update tools hacking for pep8 1.2 and beyondJoe Gordon2012-10-011-2/+2
| | | | | | | | * https://github.com/jcrocholl/pep8/commit/b9f72b16011aac981ce9e3a47fd0ffb1d3d2e085 broke tools/hacking.py * Upgrade pep8 to 1.2 and disable the following new tests: E12,E711,E712,E721,E502 Change-Id: I32a8808c6c9cccfedcc4d2a26649333aca1cd713
* Revert "Add full test environment."James E. Blair2012-09-281-6/+1
| | | | | | | | | | | This reverts commit 30e513309fe25af358b4d9ed47da2c7574fc0f06. The python mysql module was removed from test-requires in favor of test-options, but the work to actually use that was never completed. Revert the change so that mysql testing is actually performed again. Change-Id: Iaf35811cc748272229260f2dab95520995a69b44
* Do not run pylint by defaultJoe Gordon2012-08-281-1/+1
| | | | | | | | pylint creates some files, and leaves git in a detached HEAD. pylint can still be run via 'tox -epylint' Change-Id: Ic0193f2488f1c6eb4a9f0fabbe24ac9869239890
* Add lintstack error checker based on pylintYun Mao2012-08-241-1/+7
| | | | | | | | | Add a new silent test target lintstack based on pylint in Jenkins. lintstack will compare the pylint errors between HEAD and HEAD~1, filter out known false positives and report violations in pylint original parseable format for Jenkins to consume. Change-Id: Ic2962cb42a174bfe9535aea88475ff4ede3bf9ff
* Disable I18N in Nova's test suitesJian Wen2012-08-181-0/+4
| | | | | | | | | Messages translated into other language like Chinese cause some unit tests to fail. Fixes LP bug #1037946. Change-Id: I01d04333d28bab52d6f424432524b206f3845fba
* Fix broken pep8 exclude processing.Monty Taylor2012-08-031-1/+4
| | | | | | | | | | | | | | | | First of all, our pep8 exclude was excluding openstack, to trap for not doing pep8 checks on openstack/common, which comes from elsewhere. But, pep8 strips filenames down to basename when doing exclude checks on them, which makes no sense. To fix this, grab the two functions from pep8, fix them, and monkeypatch them from within hacking.py. Patch has been submitted upstream as: https://github.com/jcrocholl/pep8/pull/111 Also, changed the exclude to catch just openstack/common. Change-Id: If0b18ae828e74203f84a8b6f8b4ba0100b3bbc59
* Use all deps for tools/hacking.py tests in toxJoe Gordon2012-07-131-1/+0
| | | | | | tools/hacking needs pip modules to accuratly run nova_import_module_only Change-Id: I19eb624cc32a15438666e5710394a77d13b50763
* Exclude openstack-common from pep8 checks.Dan Prince2012-07-131-1/+1
| | | | | | | | | Updates tox.ini and run_tests.sh to exclude openstack-common code from pep8 checks. Fixes LP Bug #1020792 Change-Id: I6bb9747f86f7677f0e9c4169c04d6f3fbd4b3480
* Run hacking tests as part of the gate.Monty Taylor2012-07-021-1/+1
| | | | | | | | | | Nova has additional pep8 "plugins" that they expect to run as part of the gate. This patch will run tools/hacking.py instead of pep8 directly. Also, it fixes the hacking violaions in contrib, plugins and smoketests. Fixes bug 1010136 Change-Id: I86d8789218c197d5d4a43d1201465d340646a395
* Merge "Use setuptools-git plugin for MANIFEST."Jenkins2012-06-281-1/+1
|\
| * Use setuptools-git plugin for MANIFEST.Monty Taylor2012-06-281-1/+1
| | | | | | | | | | | | | | | | | | Instead of a hard-coded MANIFEST.in to list non-code files to be included in the tarball, use the setuptools-git plugin to ensure that every file that's in git will wind up in the tarball. The only things needed in MANIFEST.in now are files that are generated as part of the sdist run. Change-Id: I76df685d58f7b56977d859e8df92e845f450b3e4
generated by cgit (git 2.34.1) at 2024-05-24 22:00:34 +0000