Create Flake8 extension for tempest checks

Implements bp Flake8 extensions for specific tempest style checks

Change-Id: Ica93bcba872ef8b1ecee12c61d8f602f4ae37c2f
This commit is contained in:
Giampaolo Lauria 2013-05-01 11:22:07 -04:00
parent e148a6f4f4
commit 1b837ce2ae
4 changed files with 45 additions and 0 deletions

0
tools/__init__.py Normal file
View File

View File

42
tools/hacking/tempest.py Normal file
View File

@ -0,0 +1,42 @@
# vim: tabstop=4 shiftwidth=4 softtabstop=4
# Copyright 2013 IBM Corp.
#
# Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain
# a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
import re
SKIP_DECORATOR = '@testtools.skip('
def skip_bugs(physical_line):
"""Check skip lines for proper bug entries
T101: Bug not in skip line
T102: Bug in message formatted incorrectly
"""
pos = physical_line.find(SKIP_DECORATOR)
skip_re = re.compile(r'^\s*@testtools.skip.*')
if pos != -1 and skip_re.match(physical_line):
bug = re.compile(r'^.*\bbug\b.*', re.IGNORECASE)
if bug.match(physical_line) is None:
return (pos, 'T101: skips must have an associated bug')
bug_re = re.compile(r'.*skip\(.*Bug\s\#\d+', re.IGNORECASE)
if bug_re.match(physical_line) is None:
return (pos, 'T102: Bug number formatted incorrectly')

View File

@ -54,6 +54,9 @@ commands = flake8
deps = -r{toxinidir}/tools/pip-requires
-r{toxinidir}/tools/test-requires
[hacking]
local-check = tools.hacking.tempest.skip_bugs
[flake8]
ignore = E125,H302,H404
show-source = True