coveragespace.client

Complexity

Total Complexity

7

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Test Coverage

Coverage

93.1%

Importance

Changes

0

2 Functions

Rating	Name	Duplication	Size	Complexity
A	delete()	0	14	3
A	get()	0	17	4

"""API client functions."""

import logging
import time

import requests

from .cache import Cache


log = logging.getLogger(__name__)
cache = Cache()


def get(url, data):
    log.info("Getting %s: %s", url, data)

    response = cache.get((url, data))
    if response is None:
        for i in range(3):
            response = requests.put(url, data=data)
            if response.status_code == 500:
                time.sleep(i + 1)
                continue
            else:
                break
        cache.set((url, data), response)

    log.info("Response: %s", response)

    return response


def delete(url, data):
    log.info("Deleting %s: %s", url, data)

    for i in range(3):
        response = requests.delete(url, data=data)
        if response.status_code == 500:
            time.sleep(i + 1)
            continue
        else:
            break

    log.info("Response: %s", response)

The variable response does not seem to be defined in case the for loop on line 37 is not entered. Are you sure this can never be the case?

    return response