Commit Graph

16 Commits

Author SHA1 Message Date
Takashi Kajinami 813f229a08 Bump hacking
hacking 3.0.x is too old.

Also fix failures detected by the new hacking version.

Change-Id: Ifccbbf2ff7b0824d2079937526d3eee1c536349b
2024-01-18 19:04:37 +09:00
Pavlo Shchelokovskyy 51f566c56c Clean up test requirements
remove os-testr and testrepository in favor of stestr,
and remove qpid-python (does not seem to be imported anywhere).

Also adjust docs to describe running tests with stestr instead of
testrepository.

Change-Id: I12088ea2bf2475963db58d6e8e83fd6abe6f9b3f
2018-07-27 13:38:27 +00:00
Zane Bitter 3f98c3d6fb Docs: modernise links
Switch to https, replace outdated URLs with canonical ones.

Change-Id: I7c474dbf0b21f922b6625e31d6ae7df1e198b8de
Co-Authored-By: chenghuiyu <yuchenghui@unionpay.com>
Co-Authored-By: Max Pavlukhin <pavlukhinm@gmail.com>
Co-Authored-By: yfzhao <dsware@126.com>
Co-Authored-By: ricolin <rico.lin@easystack.cn>
2018-04-11 16:46:07 -04:00
Andreas Jaeger 6e5593f11b Prepare for using standard python tests
Add simple script to setup mysql and postgresql databases, this script
can be run by users during testing and will be run by CI systems for
specific setup before running unit tests. This is exactly what is
currently done by OpenStack CI in project-config.

This allows to change in project-config the python-db jobs to
python-jobs since python-jobs will call this script initially.

See also
http://lists.openstack.org/pipermail/openstack-dev/2016-November/107784.html

Update HACKING for this.

Needed-By: I98c1b799e520cad102b9e47cf51477f54b07e732
Change-Id: Id5e3b67190866acdebee2b89f4393d5c407a5b96
2017-02-04 09:49:18 +01:00
abhishekkekane 2d95f8c7d0 Restrict use of iteritems, iterkeys and itervalues
Enabled hacking checks to check dict.iteritems, dict.itervalues and
dict.iterkeys should not be used in the future and raise error while
running pep8 using tox.

Change-Id: I1239bb5c651d0193141d5bb830872caf172ec561
2016-03-23 10:05:54 +00:00
Shilla Saebi 1a433f5a42 cleanup of hacking.rst heat doc
removed unneccessary capitalizations
Heat should be lower case
titles should be lower case as well
changed document to documents and features
https://wiki.openstack.org/wiki/Documentation/Conventions

Change-Id: I55a4c93f518834b5d508f6ef5237ee202df37560
2015-04-28 16:47:07 -04:00
divakar-padiyar-nandavar 9ae2e7cb0a Fixed the reference to 'Creating Unit Tests' in HACKING.rst
Change-Id: Ie9778aceabb4239058468cf64009792253995ec2
Closes-bug: #1332920
2014-06-22 02:00:59 -06:00
Keshava Bharadwaj 690c0341b4 Updates OpenStack Style Commandments link
The current link in the HACKING file is broken. This references the
correct location for contributors to view.

Change-Id: I9edc768bca825534771d79082e7824befa51e1b8
2013-10-16 22:44:44 +05:30
Joe Gordon 3b6e8d63e2 Cleanup and make HACKING.rst DRYer
Reference the OpenStack hacking guide in HACKING.rst and remove
duplicate entries.  Add placeholder section for heat specific
rules.  heat specific rules can be created using hacking's local check
support.

Change-Id: Ib6967ae769bd73857abb7ef89368c407c8b22053
2013-07-13 09:17:14 -07:00
Monty Taylor 77afc90b5d Migrate test base class to testtools.
This is stricter, but also provides plumbing for more advanced things
like fixtures and testscenarios.

Change-Id: I607b50390708ca3caa9799ef0b27007df5045a6c
2013-05-25 09:43:40 +02:00
Zane Bitter 6d6b993287 Remove Nova dependencies from hacking test
References to Nova in the docstrings(!) of the hacking.py tests caused the
run_pep8.sh script to fail if the Grizzly development version of Nova was
not installed. Update all the references to use Heat packages instead.

Change-Id: I2e84e9a231451705d61fa9acddc65ee95a98e294
2013-03-13 12:41:34 +01:00
Steven Dake 137f9c0ad2 Add N802 rule to hacking.py
Gate on commit headers < 50 characters

Change-Id: Ia6a616d8a9b0eabca05a23476e284f261108b43a
Fixes: Bug #1136386
2013-03-04 11:20:31 -07:00
Steven Dake 637740a4f1 Update infrastructure to more closely match other OpenStack projects
This patch imports nova's HACKING.rst guidelines altered for Heat.
This patch imports nova's run_pep8.sh with some minor alterations.
This patch imports nova's hacking.py extensions to pep8.

This patch places the run_pep8 logic in a separate file rather than
having it spread between tox.ini and run_tests.sh.  The HACKING.rst
requirements as well as general pep8 requirements are then enforced
with run_pep8.sh.

Avoid running once the N802 rule until dependency changelogs are
less then 50 characters.

Change-Id: If46c14668eae3cf11b8fbff46d766cc589d701aa
Fixes: Bug #1131275
2013-02-28 20:18:01 +00:00
Chris Alfonso 9482127223 Directives to not use variable names that conflict with pdb 2012-03-20 07:16:16 -04:00
Mark McLoughlin cdc7f5a49c Sync cfg from openstack-common
Use openstack-common's update script to sync it to the latest.

Add some dire warnings that changes should be made in the upstream
copy of the code first.
2012-03-16 11:49:46 -04:00
Angus Salkeld 41d167c560 Add setup.py and friends
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
2012-03-14 09:25:54 +11:00