Sign upLogin

18F/autoapi

View on GitHub
Star

Showing 9 of 11 total issues

Function _index_table has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.
Open

def _index_table(tablename, metadata, engine, case_insensitive=False):
    """Index all columns on `tablename`, optionally using case-insensitive
    indexes on string columns when supported by the database.
    """
    table = sa.Table(tablename, metadata, autoload_with=engine)
Severity: Minor
Found in utils.py - About 1 hr to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function make_app has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.
Open

def make_app():
    app = sandman2.get_app(config.SQLA_URI, Base=utils.AutomapModel)

    app.json_encoder = utils.APIJSONEncoder
    app.config['CASE_INSENSITIVE'] = config.CASE_INSENSITIVE
Severity: Minor
Found in app.py - About 1 hr to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function apify has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
Open

def apify(filename, tablename=None):
    try:
        filenames = glob.glob(filename, recursive=True)
    except TypeError:  # recursive glob in Python 3.5+ only
        filenames = glob.glob(filename)
Severity: Minor
Found in tasks.py - About 55 mins to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function load_table has 6 arguments (exceeds 4 allowed). Consider refactoring.
Open

def load_table(filename,
Severity: Minor
Found in utils.py - About 45 mins to fix

    Function register_rules has 5 arguments (exceeds 4 allowed). Consider refactoring.
    Open

    def register_rules(app, spec, service, schema, page_schema):
    Severity: Minor
    Found in swagger.py - About 35 mins to fix

      Function to_sql has 5 arguments (exceeds 4 allowed). Consider refactoring.
      Open

      def to_sql(name, engine, frame, chunksize=None, **kwargs):
      Severity: Minor
      Found in utils.py - About 35 mins to fix

        Function entrypoint has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
        Open

        def entrypoint(argv):
    '''
    This is a Docker entrypoint that configures the container to run
    as the same uid of the user on the host container, rather than
    the Docker default of root. Aside from following security best
        Severity: Minor
        Found in entrypoint.py - About 35 mins to fix

        Cognitive Complexity

        Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

        A method's cognitive complexity is based on a few simple rules:

        • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
        • Code is considered more complex for each "break in the linear flow of the code"
        • Code is considered more complex when "flow breaking structures are nested"

        Further reading

        Function get_message has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
        Open

        def get_message(payload):
    if payload['Type'] == 'Notification':
        keys = NOTIFICATION_KEYS
    elif payload['Type'] in ['SubscriptionConfirmation',
                             'UnsubscribeConfirmation']:
        Severity: Minor
        Found in signing.py - About 25 mins to fix

        Cognitive Complexity

        Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

        A method's cognitive complexity is based on a few simple rules:

        • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
        • Code is considered more complex for each "break in the linear flow of the code"
        • Code is considered more complex when "flow breaking structures are nested"

        Further reading

        Function handle_notification has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
        Open

            def handle_notification(self, data):
        client = boto3.client('s3')
        message = json.loads(data['Message'])
        for record in message.get('Records', []):
            bucket = record['s3']['bucket']
        Severity: Minor
        Found in aws.py - About 25 mins to fix

        Cognitive Complexity

        Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

        A method's cognitive complexity is based on a few simple rules:

        • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
        • Code is considered more complex for each "break in the linear flow of the code"
        • Code is considered more complex when "flow breaking structures are nested"

        Further reading

        Severity
        Category
        Status
        Source
        Language