Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id D6552200C8B for ; Mon, 22 May 2017 14:17:35 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id D4EAC160BD8; Mon, 22 May 2017 12:17:35 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 0063D160BA5 for ; Mon, 22 May 2017 14:17:34 +0200 (CEST) Received: (qmail 13420 invoked by uid 500); 22 May 2017 12:17:34 -0000 Mailing-List: contact dev-help@ariatosca.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ariatosca.incubator.apache.org Delivered-To: mailing list dev@ariatosca.incubator.apache.org Received: (qmail 13408 invoked by uid 99); 22 May 2017 12:17:34 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 22 May 2017 12:17:34 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id BD11A190A20 for ; Mon, 22 May 2017 12:17:33 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -4.222 X-Spam-Level: X-Spam-Status: No, score=-4.222 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id MKCj4lxQM71v for ; Mon, 22 May 2017 12:17:31 +0000 (UTC) Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with SMTP id 4F17A60D9F for ; Mon, 22 May 2017 12:17:29 +0000 (UTC) Received: (qmail 12405 invoked by uid 99); 22 May 2017 12:17:28 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 22 May 2017 12:17:28 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 46937E9687; Mon, 22 May 2017 12:17:28 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: ran@apache.org To: dev@ariatosca.incubator.apache.org Date: Mon, 22 May 2017 12:17:37 -0000 Message-Id: In-Reply-To: References: X-Mailer: ASF-Git Admin Mailer Subject: [10/19] incubator-ariatosca git commit: ARIA-62 Apply pylint on extensions archived-at: Mon, 22 May 2017 12:17:36 -0000 ARIA-62 Apply pylint on extensions Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/0af9e638 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/0af9e638 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/0af9e638 Branch: refs/heads/ARIA-208-Missing-back-refrences-for-models Commit: 0af9e638b7c75eea6eac39374b7176f9dc2c645a Parents: eae44d0 Author: Ran Ziv Authored: Wed May 10 12:08:01 2017 +0300 Committer: Ran Ziv Committed: Wed May 10 12:08:01 2017 +0300 ---------------------------------------------------------------------- aria/.pylintrc | 2 +- .../aria_extension_tosca/simple_nfv_v1_0/presenter.py | 2 +- .../aria_extension_tosca/simple_v1_0/assignments.py | 2 +- extensions/aria_extension_tosca/simple_v1_0/functions.py | 8 ++++---- .../aria_extension_tosca/simple_v1_0/modeling/__init__.py | 10 ++++++---- extensions/aria_extension_tosca/simple_v1_0/presenter.py | 2 +- tests/.pylintrc | 2 +- tox.ini | 2 +- 8 files changed, 16 insertions(+), 14 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/0af9e638/aria/.pylintrc ---------------------------------------------------------------------- diff --git a/aria/.pylintrc b/aria/.pylintrc index 7da8c56..4d77556 100644 --- a/aria/.pylintrc +++ b/aria/.pylintrc @@ -360,7 +360,7 @@ max-locals=20 max-returns=10 # Maximum number of branch for function / method body -max-branches=12 +max-branches=15 # Maximum number of statements in function / method body max-statements=50 http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/0af9e638/extensions/aria_extension_tosca/simple_nfv_v1_0/presenter.py ---------------------------------------------------------------------- diff --git a/extensions/aria_extension_tosca/simple_nfv_v1_0/presenter.py b/extensions/aria_extension_tosca/simple_nfv_v1_0/presenter.py index 0ce918e..cd07f42 100644 --- a/extensions/aria_extension_tosca/simple_nfv_v1_0/presenter.py +++ b/extensions/aria_extension_tosca/simple_nfv_v1_0/presenter.py @@ -19,7 +19,7 @@ from aria.utils.caching import cachedmethod from ..simple_v1_0 import ToscaSimplePresenter1_0 -class ToscaSimpleNfvPresenter1_0(ToscaSimplePresenter1_0): # pylint: disable=invalid-name +class ToscaSimpleNfvPresenter1_0(ToscaSimplePresenter1_0): # pylint: disable=invalid-name,abstract-method """ ARIA presenter for the `TOSCA Simple Profile for NFV v1.0 csd03 `__. http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/0af9e638/extensions/aria_extension_tosca/simple_v1_0/assignments.py ---------------------------------------------------------------------- diff --git a/extensions/aria_extension_tosca/simple_v1_0/assignments.py b/extensions/aria_extension_tosca/simple_v1_0/assignments.py index 6e36ba8..9a2179a 100644 --- a/extensions/aria_extension_tosca/simple_v1_0/assignments.py +++ b/extensions/aria_extension_tosca/simple_v1_0/assignments.py @@ -94,7 +94,7 @@ class OperationAssignment(ExtensiblePresentation): if operation_definition is not None: if operation_definition._extensions: extensions.update(operation_definition._extensions) - + extensions = {} update_inherited_extensions(extensions, self._container._get_type(context)) if self._container._extensions: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/0af9e638/extensions/aria_extension_tosca/simple_v1_0/functions.py ---------------------------------------------------------------------- diff --git a/extensions/aria_extension_tosca/simple_v1_0/functions.py b/extensions/aria_extension_tosca/simple_v1_0/functions.py index 405aa8f..2f77420 100644 --- a/extensions/aria_extension_tosca/simple_v1_0/functions.py +++ b/extensions/aria_extension_tosca/simple_v1_0/functions.py @@ -176,7 +176,7 @@ class GetProperty(Function): for modelable_entity in modelable_entities: properties = None - + if hasattr(modelable_entity, 'requirement_templates') \ and modelable_entity.requirement_templates \ and (req_or_cap_name in [v.name for v in modelable_entity.requirement_templates]): @@ -185,7 +185,7 @@ class GetProperty(Function): # First argument refers to a requirement # TODO: should follow to matched capability in other node... raise CannotEvaluateFunctionException() - break + # break nested_property_name_or_index = self.nested_property_name_or_index[1:] elif hasattr(modelable_entity, 'capability_templates') \ and modelable_entity.capability_templates \ @@ -254,7 +254,7 @@ class GetAttribute(Function): # Operation # -@dsl_specification('4.6.1', 'tosca-simple-1.0') +@dsl_specification('4.6.1', 'tosca-simple-1.0') # pylint: disable=abstract-method class GetOperationOutput(Function): """ The :code:`get_operation_output` function is used to retrieve the values of variables exposed / @@ -334,7 +334,7 @@ class GetNodesOfType(Function): # Artifact # -@dsl_specification('4.8.1', 'tosca-simple-1.0') +@dsl_specification('4.8.1', 'tosca-simple-1.0') # pylint: disable=abstract-method class GetArtifact(Function): """ The :code:`get_artifact` function is used to retrieve artifact location between modelable http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/0af9e638/extensions/aria_extension_tosca/simple_v1_0/modeling/__init__.py ---------------------------------------------------------------------- diff --git a/extensions/aria_extension_tosca/simple_v1_0/modeling/__init__.py b/extensions/aria_extension_tosca/simple_v1_0/modeling/__init__.py index 9576260..3bda7e2 100644 --- a/extensions/aria_extension_tosca/simple_v1_0/modeling/__init__.py +++ b/extensions/aria_extension_tosca/simple_v1_0/modeling/__init__.py @@ -16,9 +16,11 @@ """ Creates ARIA service template models based on the TOSCA presentation. -Relies on many helper methods in the presentation classes. +Relies on many helper methods in the presentation classes. """ +#pylint: disable=unsubscriptable-object + import os import re from types import FunctionType @@ -155,7 +157,7 @@ def create_node_template_model(context, service_template, node_template): node_type = service_template.node_types.get_descendant(node_type._name) model = NodeTemplate(name=node_template._name, type=node_type) - + model.default_instances = 1 model.min_instances = 0 @@ -367,7 +369,7 @@ def create_operation_template_model(context, service_template, operation): model.description = operation.description.value implementation = operation.implementation - if implementation is not None: + if implementation is not None: primary = implementation.primary parse_implementation_string(context, service_template, operation, model, primary) relationship_edge = operation._get_extensions(context).get('relationship_edge') @@ -376,7 +378,7 @@ def create_operation_template_model(context, service_template, operation): model.relationship_edge = False elif relationship_edge == 'target': model.relationship_edge = True - + dependencies = implementation.dependencies if dependencies: for dependency in dependencies: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/0af9e638/extensions/aria_extension_tosca/simple_v1_0/presenter.py ---------------------------------------------------------------------- diff --git a/extensions/aria_extension_tosca/simple_v1_0/presenter.py b/extensions/aria_extension_tosca/simple_v1_0/presenter.py index 96cc763..c88decd 100644 --- a/extensions/aria_extension_tosca/simple_v1_0/presenter.py +++ b/extensions/aria_extension_tosca/simple_v1_0/presenter.py @@ -22,7 +22,7 @@ from .functions import (Concat, Token, GetInput, GetProperty, GetAttribute, GetO from .modeling import create_service_template_model from .templates import ServiceTemplate -class ToscaSimplePresenter1_0(Presenter): # pylint: disable=invalid-name +class ToscaSimplePresenter1_0(Presenter): # pylint: disable=invalid-name,abstract-method """ ARIA presenter for the `TOSCA Simple Profile v1.0 cos01 `__. http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/0af9e638/tests/.pylintrc ---------------------------------------------------------------------- diff --git a/tests/.pylintrc b/tests/.pylintrc index 9795bfc..0352dd3 100644 --- a/tests/.pylintrc +++ b/tests/.pylintrc @@ -360,7 +360,7 @@ max-locals=30 max-returns=10 # Maximum number of branch for function / method body -max-branches=12 +max-branches=15 # Maximum number of statements in function / method body max-statements=50 http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/0af9e638/tox.ini ---------------------------------------------------------------------- diff --git a/tox.ini b/tox.ini index f4e9871..6ab97cb 100644 --- a/tox.ini +++ b/tox.ini @@ -49,7 +49,7 @@ commands=pytest tests/end2end --cov-report term-missing --cov aria commands=pytest tests --ignore=tests/end2end --cov-report term-missing --cov aria [testenv:pylint_code] -commands=pylint --rcfile=aria/.pylintrc --disable=fixme,missing-docstring aria +commands=pylint --rcfile=aria/.pylintrc --disable=fixme,missing-docstring aria extensions/aria_extension_tosca/ [testenv:pylint_tests] commands=pylint --rcfile=tests/.pylintrc --disable=fixme,missing-docstring tests