fixed the flake8 and pylint warnings

This commit is contained in:
jamespcole 2015-03-22 15:32:47 +11:00
parent 20f52a7fb1
commit 58fd07e4c6

View File

@ -42,6 +42,7 @@ from homeassistant.const import CONF_API_KEY
_LOGGER = logging.getLogger(__name__) _LOGGER = logging.getLogger(__name__)
# pylint: disable=unused-variable
def get_service(hass, config): def get_service(hass, config):
""" Get the pushover notification service. """ """ Get the pushover notification service. """
@ -51,8 +52,8 @@ def get_service(hass, config):
return None return None
try: try:
# pylint: disable=unused-variable # pylint: disable=no-name-in-module, unused-variable
from pushover import Client, InitError, RequestError from pushover import InitError
except ImportError: except ImportError:
_LOGGER.exception( _LOGGER.exception(
@ -65,8 +66,7 @@ def get_service(hass, config):
api_token = config[DOMAIN].get(CONF_API_KEY) api_token = config[DOMAIN].get(CONF_API_KEY)
return PushoverNotificationService( return PushoverNotificationService(
config[DOMAIN]['user_key'], config[DOMAIN]['user_key'],
api_token api_token)
)
except InitError: except InitError:
_LOGGER.error( _LOGGER.error(
@ -79,14 +79,18 @@ class PushoverNotificationService(BaseNotificationService):
""" Implements notification service for Pushover. """ """ Implements notification service for Pushover. """
def __init__(self, user_key, api_token): def __init__(self, user_key, api_token):
from pushover import Client, RequestError # pylint: disable=no-name-in-module, unused-variable
from pushover import Client
self._user_key = user_key self._user_key = user_key
self._api_token = api_token self._api_token = api_token
self.pushover = client = Client(self._user_key, api_token=self._api_token) self.pushover = Client(
self._user_key, api_token=self._api_token)
def send_message(self, message="", **kwargs): def send_message(self, message="", **kwargs):
""" Send a message to a user. """ """ Send a message to a user. """
# pylint: disable=no-name-in-module
from pushover import RequestError
title = kwargs.get(ATTR_TITLE) title = kwargs.get(ATTR_TITLE)
try: try:
self.pushover.send_message(message, title=title) self.pushover.send_message(message, title=title)