Using a single test run orchestrator makes it easier to maintain
the project, especially as pytest is actively maintained.
Change-Id: I5c843984bc0a1b9264e744373e6a3fd9d43e99cd
Switches queries testing to use of pytest which provides the following:
- test generator for each query (parametrize)
- ability to test a single query test
- generate html report with test results, making easier to investigate
failures.
- parallel executions
- minor bugfix which prevented running queries from running with py38
as the config parser requires only strings (None being invalid).
Change-Id: I982c694a5160a9ecfd117d177d30b911cfe53425
- Upgraded hacking(flake8)
- Added more modern tox linters environment (pep8 alias)
- Temporary added skips for broken newer rules
- Fixed few basic rule violations
- Moved flake8 config to setup.cfg (tox.ini is not recommended)
Change-Id: I75b3ce5d2ce965a9dc5bdfaa49b2aacd8f0195ad
This change converts the repo to the new PTI workflow
and modernises the docs by using the OpenStack docs theme.
Change-Id: I5a3706c7cf40ee0c3405131b89f956bf75cae244
- This change increaes the sphinx versions
to be inline with lower-constratints and
moderen pbr.
- This change resolves the docs build error.
"sphinx.errors.SphinxError: Builder name h not registered"
Change-Id: I4dd70bd4e50108028f46d255dfd58cebbf781277
The pbr release 2.0.0 is breaking in that it removes the use of
warnerrors in build_sphinux. Hacking <0.11 had a similar issue so
use a newer version of that.
Change-Id: I4b27f38b2962dc70e078b8dbfd23d9df4dc415c3
Fixtures 3.0.0 adds another parameter to MonkeyPatch fixture usage
which is some kind of link back to the class being patched out. This
makes the ER code pass under fixtures 3.0.
Change-Id: Id19d0dde8fa558cbf3c76a66430170d0d5b0cd94
We couldn't see the ImportError in bug
1423431 because we have testtools capped.
This updates the capped test requirements
to match what's currently in global-requirements
so we can see ImportErrors.
Change-Id: Ie69a779c2135ab38dd79050afa2b9c2be10a3bcc
Related-Bug: #1423431
this changes the interface to move the readiness check out of
the classifier and into the stream object. This massively
simplifies the logic connecting these pieces, as classifier is
now just a thin wrapper to elastic search.
This also adds unit testing for the stream processing through the
creation of a fake_gerrit mock class. That lets us run gerrit
event interactions in a sane way.
It also drops all the unit testing for the classifier which is now
largely useless, because all it tests is we can execute a for loop.
Change-Id: I1971c121276412e31f01eb5680b9c41fc7e442d3
Distutils and Sphinx 1.2 break `python setup.py build_sphinx`. Work
around this by capping Sphinx to <1.2.
Change-Id: I6985aee2f9abec58d7a9bcc9452524527b35a1bc
Make repo support 'tox -epep8' as we want this to be the first gating
check.
Don't make any actual code changes, just change setup and add files.
Further work is needed to re-organize the code, but that can be in a
separate patch.