Fix failing pep gate
Recently, pep257 our python docstring checker, was updated to comply with new rules in PEP 257. No newline before a class docstring is now the standard. This patch removes the empty line before each class docstring. Change-Id: I37261111334c7982291bed7b55afbb2e9689439b
This commit is contained in:
parent
82f19487fd
commit
a87979a45c
@ -111,7 +111,6 @@ log.register_options(CONF)
|
||||
|
||||
|
||||
class JSONErrorHook(pecan.hooks.PecanHook):
|
||||
|
||||
"""A pecan hook that translates webob HTTP errors into a JSON format."""
|
||||
|
||||
def __init__(self):
|
||||
@ -154,7 +153,6 @@ class JSONErrorHook(pecan.hooks.PecanHook):
|
||||
|
||||
|
||||
class CORSHook(pecan.hooks.PecanHook):
|
||||
|
||||
"""A pecan hook that handles Cross-Origin Resource Sharing."""
|
||||
|
||||
def __init__(self):
|
||||
|
@ -92,7 +92,6 @@ CONF.register_opts(OPENID_OPTS, opt_group)
|
||||
|
||||
|
||||
class AuthController(rest.RestController):
|
||||
|
||||
"""Controller provides user authentication in OpenID 2.0 IdP."""
|
||||
|
||||
_custom_actions = {
|
||||
|
@ -33,7 +33,6 @@ requests_cache.install_cache(cache_name='github_cache',
|
||||
|
||||
|
||||
class CapabilitiesController(rest.RestController):
|
||||
|
||||
"""/v1/capabilities handler.
|
||||
|
||||
This acts as a proxy for retrieving capability files
|
||||
|
@ -34,7 +34,6 @@ CONF = cfg.CONF
|
||||
|
||||
@api_utils.check_permissions(level=const.ROLE_USER)
|
||||
class MetadataController(rest.RestController):
|
||||
|
||||
"""/v1/results/<test_id>/meta handler."""
|
||||
|
||||
rw_access_keys = ('shared',)
|
||||
@ -66,7 +65,6 @@ class MetadataController(rest.RestController):
|
||||
|
||||
|
||||
class ResultsController(validation.BaseRestControllerWithValidation):
|
||||
|
||||
"""/v1/results handler."""
|
||||
|
||||
__validator__ = validators.TestResultValidator
|
||||
|
@ -24,8 +24,7 @@ CONF = cfg.CONF
|
||||
|
||||
|
||||
class RootController(object):
|
||||
|
||||
"""root handler."""
|
||||
"""Root handler."""
|
||||
|
||||
v1 = v1.V1Controller()
|
||||
|
||||
|
@ -26,7 +26,6 @@ from refstack import db
|
||||
|
||||
|
||||
class PublicKeysController(validation.BaseRestControllerWithValidation):
|
||||
|
||||
"""/v1/profile/pubkeys handler."""
|
||||
|
||||
__validator__ = validators.PubkeyValidator
|
||||
@ -68,7 +67,6 @@ class PublicKeysController(validation.BaseRestControllerWithValidation):
|
||||
|
||||
|
||||
class ProfileController(rest.RestController):
|
||||
|
||||
"""Controller provides user information in OpenID 2.0 IdP.
|
||||
|
||||
/v1/profile handler
|
||||
|
@ -22,7 +22,6 @@ from refstack.api.controllers import user
|
||||
|
||||
|
||||
class V1Controller(object):
|
||||
|
||||
"""Version 1 API controller root."""
|
||||
|
||||
results = results.ResultsController()
|
||||
|
@ -22,7 +22,6 @@ from pecan import rest
|
||||
|
||||
|
||||
class BaseRestControllerWithValidation(rest.RestController):
|
||||
|
||||
"""Rest controller with validation.
|
||||
|
||||
Controller provides validation for POSTed data
|
||||
|
@ -17,14 +17,12 @@
|
||||
|
||||
|
||||
class ParseInputsError(Exception):
|
||||
|
||||
"""Raise if input params are invalid."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class ValidationError(Exception):
|
||||
|
||||
"""Raise if request doesn't pass trough validation process."""
|
||||
|
||||
def __init__(self, title, exc=None):
|
||||
|
@ -47,7 +47,6 @@ def checker_uuid(inst):
|
||||
|
||||
|
||||
class BaseValidator(object):
|
||||
|
||||
"""Base class for validators."""
|
||||
|
||||
schema = {}
|
||||
@ -75,7 +74,6 @@ class BaseValidator(object):
|
||||
|
||||
|
||||
class TestResultValidator(BaseValidator):
|
||||
|
||||
"""Validator for incoming test results."""
|
||||
|
||||
schema = {
|
||||
@ -141,7 +139,6 @@ class TestResultValidator(BaseValidator):
|
||||
|
||||
|
||||
class PubkeyValidator(BaseValidator):
|
||||
|
||||
"""Validator for uploaded public pubkeys."""
|
||||
|
||||
schema = {
|
||||
|
@ -37,14 +37,12 @@ db_options.set_defaults(cfg.CONF)
|
||||
|
||||
|
||||
class NotFound(Exception):
|
||||
|
||||
"""Raise if item not found in db."""
|
||||
|
||||
pass
|
||||
|
||||
|
||||
class Duplication(Exception):
|
||||
|
||||
"""Raise if unique constraint violates."""
|
||||
|
||||
pass
|
||||
|
@ -32,7 +32,6 @@ BASE = declarative_base()
|
||||
class RefStackBase(models.ModelBase,
|
||||
models.TimestampMixin,
|
||||
models.SoftDeleteMixin):
|
||||
|
||||
"""Base class for RefStack Models."""
|
||||
|
||||
__table_args__ = {'mysql_engine': 'InnoDB'}
|
||||
@ -51,7 +50,6 @@ class RefStackBase(models.ModelBase,
|
||||
|
||||
|
||||
class Test(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
"""Test."""
|
||||
|
||||
__tablename__ = 'test'
|
||||
@ -80,7 +78,6 @@ class Test(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
|
||||
class TestResults(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
"""Test results."""
|
||||
|
||||
__tablename__ = 'results'
|
||||
@ -104,7 +101,6 @@ class TestResults(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
|
||||
class TestMeta(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
"""Test metadata."""
|
||||
|
||||
__tablename__ = 'meta'
|
||||
@ -124,7 +120,6 @@ class TestMeta(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
|
||||
class User(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
"""User information."""
|
||||
|
||||
__tablename__ = 'user'
|
||||
@ -147,7 +142,6 @@ class User(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
|
||||
class PubKey(BASE, RefStackBase): # pragma: no cover
|
||||
|
||||
"""User public pubkeys."""
|
||||
|
||||
__tablename__ = 'pubkeys'
|
||||
|
@ -22,7 +22,6 @@ LOG = log.getLogger(__name__)
|
||||
|
||||
|
||||
class PluggableBackend(object):
|
||||
|
||||
"""A pluggable backend loaded lazily based on some value."""
|
||||
|
||||
def __init__(self, pivot, **backends):
|
||||
|
@ -33,7 +33,6 @@ from refstack.db import migration
|
||||
|
||||
|
||||
class FunctionalTest(base.BaseTestCase):
|
||||
|
||||
"""Base class for functional test case.
|
||||
|
||||
Used for functional tests where you need to test your.
|
||||
|
@ -19,7 +19,6 @@ from oslotest import base
|
||||
|
||||
|
||||
class RefstackBaseTestCase(base.BaseTestCase):
|
||||
|
||||
"""Refstack test base class."""
|
||||
|
||||
def setup_mock(self, *args, **kwargs):
|
||||
|
Loading…
x
Reference in New Issue
Block a user