| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
Remove a bunch of local custom scripts. Replace with configurable
external tools.
Use local hacking checks for nova specifics.
Change-Id: I75a01375ba0ec36d2ff05abc47abe0a3f225eda5
|
| |
|
|
|
|
|
|
|
|
|
| |
Declarative approach to packaging, obviates the need for copying files.
Also, removes the need for setup.py to import files from the nova
module.
./run_tests.sh now ensure's that nova.egg-info/entry_points.txt is up to
date before running tests.
Change-Id: I7b7a18e065a62491ade54fbed5d8400db1d4e6c7
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added test case and support functions for CI opportunistic testing
using a PostgreSQL database, as part of our postgresql as a first
class citizen effort.
rebased again on top of test migration with data framework
clean up debug, and ensure pg will fail to connect gracefully
fix previous migration test which was inserting a key by id, which
postgresql doesn't like, as it doesn't adjust the autoincrement
counter.
added driver to connection string for mysql
Updated comment to expose how to do this on pg easily
Fixes:
bug 1084567
Change-Id: If0bfe5cf748f6d5724d17082be6d18b6f96f6ee4
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Python coverage lower than 3.6 causes issues when running the whole
test suite. Some arguments are randomly mixed up with script names,
resulting in unittest failures like this:
UnexpectedMethodCallError: Unexpected method call. unexpected:- expected:+
- exists.__call__('/tmp/buildd/.../nova/virt/images.py') -> None
+ exists.__call__('/some/path') -> True
Change-Id: Iadb6d046e3acc75c6ed5abffcfc1abda82aeee4f
|
| |/
|
|
|
|
|
|
| |
The current tests are very opaque when something fails. This change makes the
failure easier to understand without a debugger. The testtools version is
bumped to get a new HasLength matcher.
Change-Id: Iceafa70f88a8cc31a1b0ba912fe32a9fef1b2f18
|
| |
|
|
| |
Change-Id: If760a7b6f8771d9f9ac2722c300032e715e34151
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Fixes bug 1102400
testtools 0.9.25 introduces a new dependency on the 'extras' package, but
it does so in a way to prevents itself from being installed without the
'extras' package already installed.
Work around the problem for now by avoiding the problematic release.
Change-Id: Ib42ffc67f00db62f9b829069c00476632daae94f
|
| |
|
|
|
|
| |
The previous false-positive bug was fixed upstream in 0.0.13.
Change-Id: Iceb07a511992249c104bbc7c34b7c35e01bd9fbd
|
| |
|
|
|
|
|
| |
This reverts commit c945b71929d7dc872ea27103ca8b6ec3d5cc7cd5
which caused the unit test command to always exit sucessfully.
Change-Id: I410cbc878675503f0ce6e50bddc4fd0552370c46
|
| |
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
Fixtures 0.3.12 adds StringStream as a successor to DetailStream (this
change was made for compatibility with python 3). Update Nova's tests to
use StringStream instead of DetailStream.
Change-Id: If697859a52f3a7db9dab38bd080b779bb3270287
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
| |
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>
|
| |
|
|
|
|
|
|
|
| |
This fixes bug 1078548.
This is done by adding a dependency on the fixtures library, which has
dedicated code for just this sort of thing, and using it in the base class.
Change-Id: I841fbf912b1a3ab43ca8c52c779d930aaf4a0e96
|
| |
|
|
|
|
|
|
|
| |
python-cinderclient is now actually a requirement, move
this from test-requires to pip-requires to reflect that.
Fixes bug #1074085
Change-Id: Iea6c58faab2462d137049d690d58d951bd4e2c9e
|
| |
|
|
|
|
|
|
|
|
|
|
| |
As a step towards parallelizing the test runs, use testtools as the
base unittest test case class. In this patch, it has caused us to catch
places where setUp and tearDown were not being chained properly.
Additionally, we get to remove our custom skip decorators and a few
backported assert calls.
Part of blueprint grizzly-testtools
Change-Id: Ia2025e1b7191b65130b85b30ce6239d5c431c8ba
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
* 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
|
| |/
|
|
|
|
|
|
|
|
|
|
|
| |
We have a fake ldap implementation for testing, but the tests still
fail if ldap isn't installed. This modifies the usage of ldap in
the LdapDNS tests to stub the module before attempting to import
it. This allows us to test without having python-ldap installed.
It removes the unneccessary FakeLdapDNS in favor of using the real
LdapDNS with the module stubbed out. It also removes the dependency
on python-ldap from test-requires.
Change-Id: Ifbe09e16436afa6999dcb7385763492b0026556c
|
| |
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
| |
Also removes the auth dir from nova.
fix bug lp:1054848
Change-Id: I9c2710c9789ff8413bc80044eba94f3a851d8544
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add optional flag to allow cinder endpoint to be defined by flag rather
than extracted from the request context's service catalog. No change to
default behavior.
This flag allows deployers to use a seperate management endpoint for
cinder which compute can use exclusively to send messages to cinder, or
just use an auth system that doesn't add/forward the catalog along with
the request (e.g. noauth).
Add python-cinderclient to tools/test-requires
fix lp bug #1048798
Change-Id: Icb416bf4df2a6e37024f1fbc866006d46d30bcf2
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |\ \
| |/
|/| |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Adding nosehtmloutput as a test dependency allows nose to output its
results to an html file. This will be used by Jenkins to save logs on
a different server.
Change-Id: I45191e872a41eed8e6a9e14757675ee7a28b3836
|
| |/
|
|
|
|
|
|
|
| |
Fix bug 1039745
* move feedparser to tools/test-requires
* remove python-daemon
Change-Id: I8da6ee6854e2064b44f3161b84f28d0bfc7f438d
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| | |
tox uses pep8 1.1, but tools/test-requires was set to pep8 1.2.
This sets pep8 back to 1.1
Change-Id: I63612d84a245def20367e192f1dda88aff061cbc
|
| |\ \
| |/
|/| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a tox environment that will install all of the optional
packages and which will run with system site packages enabled
so that we pick up those tests as well. This should allow us
to run expansive tests on jenkins that aren't normally required
locally, but which are possible.
Change-Id: Iabd61874061e24950d4d2082a2fba158b4601624
|
| |/
|
|
|
|
| |
As found by pep8 version 1.2.
Change-Id: I5d40294416db0410fa7d9c82ea8efe8efadd0f58
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Shrink tox.ini to the new short version.
Fix the test cases to be able to be run in nosetets plus the
openstack.nose_plugin, which finally removes the need for
nova/testing/runner.py
Also, now we'll just output directly to stdout, which will
make nose collect the trace logging directly and either output
it at the end of the run, or inject it into the xunit output
appropriately.
Change-Id: I1456e18a11a840145492038108bdfe812c8230d1
|
| |
|
|
|
|
|
|
| |
The latest release of pep8 adds stricter rules for extraneous
whitespace within lines. This patch also addresses these new
violations acoss the codebase.
Change-Id: Ib7e50281870473df1704ed50868c5c2e26bdb02e
|
| |
|
|
|
|
|
| |
Nails the pep8 deps for tox and test-requires to 1.0.1.
Fixes an issues causing pep8 failures due to a new pep8 release.
Change-Id: I30be4909f052c29834afbfddba6007045afa519b
|
| |
|
|
|
|
|
|
| |
Addresses bug #978304
Refactor _walk_versions for readability and pep8 compliance
Change-Id: I94d17f2ec81a4115ae3ea3863c79b88462ceb835
|
| |
|
|
|
|
|
| |
Update test-requires to use pep8>=1.0. Removed PEP8 warning suppression
around 3-arg raises.
Change-Id: Ib4ed42adc167aa1e8078619a36b409b76b9f5d73
|
|
|
Change-Id: I6a0a616a979688595762e6b772d77a14f1968070
|