jacebrowning / coverage-space-cli

coveragespace/client.py

"""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
            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
        break

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

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

    return response